extra space in changeSiteActions function in \Data\Products\CMFPlone\migrations\v2_\betas.py

[P. Erickson]

In the function changeSiteActions(portal, out) in \Data\Products\CMFPlone\migrations\v2_\betas.py there is an extra space after "string:" lines # 981, # 988, # 995

This causes invalid html in site_actions after migrating. The rendered code (currently on the plone.org home page):

<li id="siteaction-accessibility"><a

href="  http://plone.org/accessibility-info" accesskey="0" title="Accessibility">Accessibility</a></li>

Notice the extra space between href=" and the URI.

This seems to happen in several files. If you search for "string: " (notice the trailing space) inside \Products\CMFPlone\, I find 33 files that have "string: ".

[jok2]

According to  http://www.w3.org/TR/xhtml1/#h-4.7 and  http://www.w3.org/TR/REC-xml/#AVNormalize, leading and trailing space in attributes MUST be discarded by the xhtml parser. So <a href="  http://plone.org/accessibility-info"">accessibility/a> is valid xhtml.

[limi]

Yup; should still be fixed, though. :)

[hannosch]

(In [9754]) Fixed extra space in changeSiteActions function in some migrations. This closes #5161.