Closed
Bug 433759
Opened 16 years ago
Closed 16 years ago
Empty description for hCalendar causes error in console
Categories
(Toolkit Graveyard :: Microformats, defect)
Toolkit Graveyard
Microformats
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mkaply, Assigned: mkaply)
Details
Attachments
(1 file)
1.76 KB,
patch
|
sayrer
:
review+
mconnor
:
approval1.9.0.4-
|
Details | Diff | Splinter Review |
If description is empty in an hCalendar, the creation of the object that is contstructed to represent the HTML or string of the description throws an error because it doesn't test value first.
Assignee | ||
Updated•16 years ago
|
Attachment #320978 -
Flags: review?(sayrer)
Updated•16 years ago
|
Attachment #320978 -
Flags: review?(sayrer) → review+
Assignee | ||
Comment 1•16 years ago
|
||
Comment on attachment 320978 [details] [diff] [review] Add a check for value, and if it is undefined, set the value of description to "" Correctness fix for microformats.
Attachment #320978 -
Flags: approval1.9.0.3?
Assignee | ||
Comment 2•16 years ago
|
||
fixed in hg.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Comment 3•16 years ago
|
||
Comment on attachment 320978 [details] [diff] [review] Add a check for value, and if it is undefined, set the value of description to "" Approved for 1.9.0.4, a=dveditz for release-drivers
Attachment #320978 -
Flags: approval1.9.0.4? → approval1.9.0.4+
Comment 4•16 years ago
|
||
Comment on attachment 320978 [details] [diff] [review] Add a check for value, and if it is undefined, set the value of description to "" sorry, unapproving. You'll need to appeal to the project leaders (mconnor,beltzner, etc) because these don't fit the narrow regression/security bug criteria we now use.
Attachment #320978 -
Flags: approval1.9.0.4+ → approval1.9.0.4?
Comment 5•16 years ago
|
||
Comment on attachment 320978 [details] [diff] [review] Add a check for value, and if it is undefined, set the value of description to "" In order to reduce risk and minimize overhead for stability releases, we are not accepting "nice to have" patches in stability releases. As 3.1 is fast approaching, this should not unduly impact time to get these fixes to users.
Attachment #320978 -
Flags: approval1.9.0.4? → approval1.9.0.4-
Updated•5 years ago
|
Product: Toolkit → Toolkit Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•