Custom repeated task - cannot click on OK button

RESOLVED FIXED in 1.0b5

Status

Calendar
Tasks
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: hnsz2002, Assigned: Fallen)

Tracking

Lightning 1.0b4
1.0b5
All
Other
Bug Flags:
blocking-calendar1.0 +

Details

(Whiteboard: [needed beta][has l10n impact])

Attachments

(1 attachment)

(Reporter)

Description

7 years ago
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:5.0) Gecko/20100101 Firefox/5.0
Build ID: 2011061500

Steps to reproduce:

I want to create a custom repeated task until a date. New task, Repeat: Custom, and if select Repeat until radiobutton, if i click on OK, it has no effect, and cannot save the settings.
(This bug is also available in earlier beta versions...)


Actual results:

The dialog window not closed, and the settings are't saved.


Expected results:

Close the window, and create repeated task until the selected date.
(Assignee)

Comment 1

7 years ago
Do you have any error console messages afterwards?
(Reporter)

Comment 2

7 years ago
(In reply to comment #1)
> Do you have any error console messages afterwards?

Yeah, http://wstaw.org/m/2011/07/19/lightning.png
(Assignee)

Comment 3

7 years ago
Are you using a localized build, or is this en-US?
(Reporter)

Comment 5

7 years ago
(In reply to comment #3)
> Are you using a localized build, or is this en-US?

Its a localized version, hu-HU
(Assignee)

Comment 6

7 years ago
Ok, so we need to either guard better or fix the locales, as fast as possible. I am in the process of spinning 1.0b5rc1, but I think its reasonable to push in a fix for this bug too.

Decathlon, could you spin up a patch to guard better on the calendar side?
(Assignee)

Updated

7 years ago
Whiteboard: [needed beta][has l10n impact]
(Assignee)

Updated

7 years ago
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: blocking-calendar1.0+
(Assignee)

Comment 7

7 years ago
Created attachment 547389 [details] [diff] [review]
Fix - v1

This should take care of the calendar side and at least allow usage of the dialog again. I'll contact localizers to see that they fix the errors.
Assignee: nobody → philipp
Status: NEW → ASSIGNED
Attachment #547389 - Flags: review?(bv1578)
(Assignee)

Comment 8

7 years ago
Comment on attachment 547389 [details] [diff] [review]
Fix - v1

I'm going to check this in for the sake of a quicker release.
Attachment #547389 - Flags: review?(bv1578) → review+
(Assignee)

Comment 9

7 years ago
Pushed to comm-central <http://hg.mozilla.org/comm-central/rev/3654eb8aa8e9>
-> FIXED
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Trunk
(Assignee)

Comment 10

7 years ago
Backported to comm-miramar <http://hg.mozilla.org/releases/comm-miramar/rev/d5153e2e084a>
Target Milestone: Trunk → 1.0b5

Comment 12

7 years ago
Comment on attachment 547389 [details] [diff] [review]
Fix - v1

># HG changeset patch
># Parent 2978cc4dce9e7a1b35e72c737bd8a5e173d07a66
>Fix bug 672158 - Custom repeated task - cannot click on OK button. r=decathlon
>
>diff -r 2978cc4dce9e calendar/base/content/dialogs/calendar-event-dialog-recurrence.js
>--- a/calendar/base/content/dialogs/calendar-event-dialog-recurrence.js	Thu Jul 21 12:31:27 2011 +0200
>+++ b/calendar/base/content/dialogs/calendar-event-dialog-recurrence.js	Thu Jul 21 15:46:23 2011 +0200
>@@ -762,8 +762,11 @@
>     // Add elements in the right order, removing them from their old parent
>     for (var i = 0; i < aPropParams.length; i++) {
>         var newEl = document.getElementById(localeOrder[i]);
>-        parents[i].appendChild(newEl.parentNode.removeChild(newEl));
>-
>+        if (newEl) {
>+            parents[i].appendChild(newEl.parentNode.removeChild(newEl));
>+        } else {
>+            cal.ERROR("Localization error, could not find node '" + localeOrder[i] + "'. Please have your localizer check the string '" + aPropKey + "'");
>+        }
>     }
> }
> 
>diff -r 2978cc4dce9e calendar/locales/en-US/chrome/calendar/calendar-event-dialog.properties
>--- a/calendar/locales/en-US/chrome/calendar/calendar-event-dialog.properties	Thu Jul 21 12:31:27 2011 +0200
>+++ b/calendar/locales/en-US/chrome/calendar/calendar-event-dialog.properties	Thu Jul 21 15:46:23 2011 +0200
>@@ -130,6 +130,7 @@
> # %1$S - ordinal with article
> # %2$S - weekday noun
> # e.g. "'the first' 'Monday'"
>+# DONT_TRANSLATE: Make sure there are no extra words in this property, just variables.
> ordinalWeekdayOrder=%1$S %2$S
> 
> # LOCALIZATION NOTE (monthlyEveryOfEveryNounclass...):
>@@ -350,18 +351,21 @@
> # Edit recurrence window -> Recurrence pattern -> Repeat monthly
> # %1$S - ordinal with article, %2$S - weekday
> # e.g. "the First Saturday"
>+# DONT_TRANSLATE: Make sure there are no extra words in this property, just variables.
> monthlyOrder=%1$S %2$S
> 
> # Edit recurrence window -> Recurrence pattern -> Repeat yearly
> # %1$S - day of month, %2$S - of, %3$S - month
> # e.g. "6 [of] September"
> # If you don't need %2$S in your locale - please put this on the third place.
>+# DONT_TRANSLATE: Make sure there are no extra words in this property, just variables.
> yearlyOrder=%1$S %3$S %2$S
> 
> # Edit recurrence window -> Recurrence pattern -> Repeat yearly
> # %1$S - ordinal with article, %2$S - weekday, %3$S  - of, %4$S - month
> # e.g. "the First Saturday of September"
> # If you don't need %3$S in your locale - please put this on the third place.
>+# DONT_TRANSLATE: Make sure there are no extra words in this property, just variables.
> yearlyOrder2=%1$S %2$S %3$S %4$S
> 
> # LOCALIZATION NOTE (pluralForWeekdays):

Comment 13

7 years ago
Opps, sorry for the mess in the previous comment. I had not seen you already checked-in the patch.

There are other localization error related to this issue
http://mxr.mozilla.org/l10n-mozilla-aurora/source/eu/calendar/chrome/calendar/calendar-event-dialog.dtd#293

and maybe:
http://mxr.mozilla.org/l10n-mozilla-aurora/source/da/calendar/chrome/calendar/calendar-event-dialog.dtd#307

I'm going to send an email to the localizers.
(Assignee)

Comment 14

7 years ago
Thanks! I already have reply from two localizers from the first batch.
You need to log in before you can comment on or make changes to this bug.