infra
/
etesync
2
0
Fork 0
Code Issues Pull Requests Packages Activity

TaskEdit: use null coalescing instead of ||. 

It technically works because undefined happens to be falsey, but still bad
practice and can lead to errors (especially when copy-pasting).
master
Tom Hacohen 2020-03-13 11:16:56 +02:00
parent 54302c4c07
commit 18967ddb16
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/components/Tasks/TaskEdit.tsx
View File

@ -94,7 +94,7 @@ class TaskEdit extends React.PureComponent<PropsType> {
this.state.uid = event.uid;
this.state.title = event.title ? event.title : '';
this.state.status = event.status;
this.state.priority = event.priority || TaskPriorityType.Undefined;
this.state.priority = event.priority ?? TaskPriorityType.Undefined;
if (event.startDate) {
this.state.allDay = event.startDate.isDate;
this.state.start = event.startDate.convertToZone(ICAL.Timezone.localTimezone).toJSDate();