pieforms fieldset makes legend ids starting with numbers (invalid xhtml)

[Penny Leach]

This really is a pieforms bug, but I don't know where to file those.

htdocs/lib/pieforms/pieform/elements/fieldset.php

$id = substr(md5(microtime()), 0, 4);

I made a temporary workaround with

$id = preg_replace('/[0-9]/', 'a', $id);

Which is probably not unique enough...

This bug was imported from eduforge.org, see:
https://eduforge.org/tracker/index.php?func=detail&aid=3134&group_id=176&atid=739

[Nigel-catalyst (nigel-catalyst)]

http://www.w3.org/TR/html4/types.html#h-6.2 is the part of the spec that says IDs must start with a letter (which took a bit of finding).

I'll fix it in pieforms.

Pieforms bugs should normally be reported to http://sourceforge.net/tracker/?atid=901453&group_id=182497&func=browse but I don't mind them here.

[François Marier (fmarier)]

Hi Penny,

Do you remember which branch you fixed this on? Also, instead of replacing all of the digits with 'a', why not just prepend the letter 'a' to the id string?

[Penny Leach (mjollnir)]

I didn't fix it, I just reported it. I'm not sure if Nigel fixed it in pieforms directly or not.