1331265 - Error "now.getTime is not a function" when creating a task with entry date

Status: RESOLVED FIXED

(Calendar :: Tasks, defect)

Description

[Decathlon]

Step to reproduce:

As described in summary: create a new task, activate the entry date, save and close the dialog;

--> the error "now.getTime is not a function" appears in the console.

It's caused by a typo from bug 1063085.

Comment 1

[Decathlon]

Attachment: patch-v1

fix

Comment 2

[[:MakeMyDay]]

Comment on attachment 8826941 [details] [diff] [review]
patch-v1

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

Thanks, r=me

We should take this for 5.4, too.

Comment 3

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Comment on attachment 8826941 [details] [diff] [review]
patch-v1

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

Thanks for catching this! I assume 5.4+ is the only place where this is an issue? If not, blanket a=me for other branches.

Comment 4

[Decathlon]

Pushed to comm-central
https://hg.mozilla.org/comm-central/rev/753f701af7a2

Comment 5

[Decathlon]

Pushed to comm-aurora
https://hg.mozilla.org/releases/comm-aurora/rev/86b109c48a5a

This also have to go on beta.

MakeMyDay, is there currently any restriction about Beta (5.3)?

Comment 6

[Jorg K (CEST = GMT+2)]

You should have set the "Target Milestone", not the Version.

Comment 7

[Jorg K (CEST = GMT+2)]

Can someone please set approval-calendar-beta+ here, otherwise my queries don't see it.

Comment 8

[[:MakeMyDay]]

Comment on attachment 8826941 [details] [diff] [review]
patch-v1

No, there's none, go ahead. Given that the patch from the original bug landed in 3.6 (which also should be the version of this bug then), the patch should be uplifted up to ESR (carrying forward Philipp's blanko a+).

Now that beta builds are working again, there will probably one more beta of TB 51 this week if there are any uplifts.

Comment 9

[Jorg K (CEST = GMT+2)]

There are uplifts to beta 51, currently this bug and two others and I've already uplifted one.
Note that bug 1324006 is also awaiting calendar approval. Maybe you want to give that and I can include it in the uplift.

Comment 10

[Stefan Sitter [:ssitter]]

I don't think this has to go to ESR. If I read it correctly this regression was introduced in Lightning 5.3 with Bug 1280898 (https://hg.mozilla.org/comm-central/rev/fd02e757b142). Bug 1063085 is not related.

Comment 11

[[:MakeMyDay]]

Comment on attachment 8826941 [details] [diff] [review]
patch-v1

Yes, you're right. Thanks for cross checking!

Comment 12

[Decathlon]

Pushed to comm-beta:
https://hg.mozilla.org/releases/comm-beta/rev/edbc5069cfa0


(In reply to Stefan Sitter from comment #10)
> ...this regression was introduced in Lightning 5.3 with Bug 1280898
> (https://hg.mozilla.org/comm-central/rev/fd02e757b142). Bug 1063085 is not
> related.

Thanks for the right bug number.

(In reply to Jorg K (GMT+1) from comment #6)
> You should have set the "Target Milestone", not the Version.

Thanks for the correction and also for the explanation.