Fix old consumers of .stringValue

RESOLVED FIXED

Status

RESOLVED FIXED
13 years ago
12 years ago

People

(Reporter: jminta, Unassigned)

Tracking

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

13 years ago
bug 307948 changed the way ical parameters, expecially text values, should be stored and accessed.  Not all of the old consumers of .stringValue were updated however.  LXR tells me there are some in calAttendee.js and in calendarUtils.js.  

The calendarUtils.js calls are responsible for the reappearance of bug 311983.

This would be another [good first bug].
Created attachment 204223 [details] [diff] [review]
Patch for calendarUtils.js
Attachment #204223 - Flags: first-review?(jminta)
Created attachment 204225 [details] [diff] [review]
Patch for calendarUtils.js and calAttendee.js
Attachment #204223 - Attachment is obsolete: true
Attachment #204225 - Flags: first-review?
Attachment #204223 - Flags: first-review?(jminta)
Attachment #204225 - Flags: first-review? → first-review?(jminta)
(Reporter)

Comment 3

13 years ago
Comment on attachment 204225 [details] [diff] [review]
Patch for calendarUtils.js and calAttendee.js

Thanks for the patch!
r=jminta
Attachment #204225 - Flags: first-review?(jminta) → first-review+
(Reporter)

Comment 4

13 years ago
patch checked in
Status: NEW → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
Blocks: 317754
The bugspam monkeys have struck again. They are currently chewing on default assignees for Calendar. Be afraid for your sanity!
Assignee: base → nobody
You need to log in before you can comment on or make changes to this bug.