Closed Bug 394174 Opened 13 years ago Closed 13 years ago

Attendees list is blank in new event summary dialog

Categories

(Calendar :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: omarb.public, Assigned: michael.buettner)

Details

Attachments

(1 file)

STEPS TO REPRODUCE:
===================

- Subscribe to http://sd-calendar.staroffice.de/
- Mark the calendar as a read only
- double click on event with added attendees

RESULT:
=======

- Attendees list is blank in new event summary dialog

EXPECTED RESULT:
================

- Attendees are shown in new event summary dialog

REPRODUCIBLE:
=============

- always
Event summary dialog is a new enhancement and doesn't work as expected---> requesting blocking 0.7
Flags: blocking-calendar0.7?
I can confirm with an event in read-only storage calendar. The Console shows:

  Error: organizer is not defined
  Source File: chrome://calendar/content/calendar-summary-dialog.js
  Line: 264
Component: Provider: WCAP → General
OS: Windows XP → Windows 2000
QA Contact: wcap-provider → general
need to be resolved
Flags: blocking-calendar0.7? → blocking-calendar0.7+
Assignee: nobody → michael.buettner
Status: NEW → ASSIGNED
OS: Windows 2000 → All
Hardware: PC → All
Attached patch patch v1Splinter Review
Inadvertent typo. Admittedly, I'm really missing a decent compiler that bails out under these circumstances. Anyway, I also took the liberty to correct the wrong handling of the 'mailto' prefix which I came across.
Attachment #279074 - Flags: review?(daniel.boelzle)
Comment on attachment 279074 [details] [diff] [review]
patch v1

>+            } else {
>+                email = email;
remove that else-block

>+                    if (email && email.length) {
>+                        if (re.test(email)) {
>+                            email = RegExp.$1;
>+                        } else {
>+                            email = email;
>+                        }
>                     }
just
if (email && re.test(email)) {
    email = RegExp.$1;
}
here

btw: RegExp.$1 is deprecated, use exec()

r=dbo
Attachment #279074 - Flags: review?(daniel.boelzle) → review+
patch checked in on trunk and MOZILLA_1_8_BRANCH

-> FIXED
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → 0.7
Verified with lightning 2007090504
Status: RESOLVED → VERIFIED
Verified with Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1.7pre) Gecko/20070906 Calendar/0.7pre
You need to log in before you can comment on or make changes to this bug.