FLUID-76: Ids generated by RSF must not contain open parenthesis, i.e. '('.

Metadata

Source
FLUID-76
Type
Bug
Priority
Major
Status
Closed
Resolution
Fixed
Assignee
Antranig Basman
Reporter
Joseph Scheuhammer
Created
2007-10-25T17:21:41.000-0400
Updated
2007-11-26T14:16:09.000-0500
Versions
N/A
Fixed Versions
  1. 0.1
Component
  1. Image Reorderer

Description

The ids of, for example, the thumbnail divs as generated by RSF include a unique string (a UUID?). This string occurs right after the 'page-replace:' prefix, and before the "gallery: ..." (in the case of the lightbox template). Sometimes, and it is rare, this unique string contains an open parenthesis, '('. If it does, it causes the javascript function fluid.Utilities.seekNodesById() to fail with a syntax error, along the lines of "unmatched parenthesis".

My guess is that the parenthesis must be escaped in order to be seen as literal. But, it would be better if they just didn't occur. According to W3C, html 4.01 id's don't contain parentheses:
http://www.w3.org/TR/html401/types.html#type-id

Comments

  • Antranig Basman commented 2007-11-06T11:40:34.000-0500

    Apologies for this awful oversight. There is a new utility class "XMLUtil" which now does the work of flattening out "Eighteen IDs" into an XML-compatible form with very little bit-loss... this utility is now in RSF trunk also and when the next RSF release appears we can remove the copy of the utility I placed in Gallery.