1309883 - error in "every weekday" recurrence in lightning-item-iframe.js (onDays is not defined)

Status: RESOLVED FIXED

(Calendar :: Calendar Frontend, defect)

Description

[Markus Adrario [:Taraman]]

When setting a daily recurrence to "every weekday", they get shown on weekends as well.

Error Message:
JavaScript error: chrome://lightning/content/lightning-item-iframe.js, line 2709: ReferenceError: onDays is not defined
[1]

When editing the item afterwards, Recurrence Rule shows "daily" again in accordance with the display in the view.

[1]:
https://dxr.mozilla.org/comm-central/rev/593964199b659fdb8b5b7d8ce765c1976a783c99/calendar/lightning/content/lightning-item-iframe.js#2709

Comment 1

[Markus Adrario [:Taraman]]

Attachment: Fix V1

This should fix the problem.

Comment 2

[Stefan Sitter [:ssitter]]

Looks like a regression from https://hg.mozilla.org/comm-central/rev/880d5e4d567c

Comment 3

[Philipp Kewisch [:Fallen][📱📆]]

Comment on attachment 8800657 [details] [diff] [review]
Fix V1

Review of attachment 8800657 [details] [diff] [review]:
-----------------------------------------------------------------

::: calendar/lightning/content/lightning-item-iframe.js
@@ +2705,4 @@
>                  recRule.type = "WEEKLY";
>                  break;
>              case "every.weekday":
> +                let onDays = [2, 3, 4, 5, 6];

I don't think this will pass eslint, because the let variable is in a switch case. I'd suggest to just pass 5 instead of onDays.length, or use {} for the case.

Comment 4

[Markus Adrario [:Taraman]]

Attachment: Fix V2

passing in 5 sounds reasonable and since no variable is declared, it is clear in reading as well.

Patch updated

Comment 5

[Philipp Kewisch [:Fallen][📱📆]]

Comment on attachment 8803607 [details] [diff] [review]
Fix V2

Review of attachment 8803607 [details] [diff] [review]:
-----------------------------------------------------------------

lgtm, thanks for the patch! r=philipp

Comment 6

[Markus Adrario [:Taraman]]

pushed to

comm-central -> https://hg.mozilla.org/comm-central/rev/279d84a45b1db499195f2c6cc72407693bf8d07c
comm-aurora-> https://hg.mozilla.org/releases/comm-aurora/rev/2817bc16dce87746c98c552142c5c92304e398da