Ticket #13885 (new defect)
Bug in dsp_edit_profile.cfm concerning outputting of extended attribute sets.
| Reported by: | spam2@… | Owned by: | |
|---|---|---|---|
| Priority: | minor | Component: | General |
| Version: | 5.1 | Keywords: | register edit form |
| Cc: | Time planned: | 1d | |
| Time remaining: | 1d | Time spent: | 1d |
Description
I have several attribute sets assigned assigned to the default subclass of the User entity, my extended attribute sets look like this:
Address (street, suite, city, etc…) Directory (description of company, make profile visible, etc…) Employee information (number of employees in province, country, world, etc…)
I would have expected dsp_edit_profile.cfm (the display object for rendering the user register / update form) to group each extended attribute set into a <fieldset> but the existing logic is wrong. It only throws the first field into a fieldset block, and the rest are OUTSIDE of the fieldset.
Take a look at http://www.splicedigital.com/test/mura_profile.gif
This is a slightly older (downloaded in march 2010), but I performed the following fix. In dsp_edit_profile.cfm on line 218, you had:
<cfif not started> </ul> </fieldset> <cfset started=true> </cfif>
To match the the line on 184
<cfif not started> <fieldset> …… </cfif>
The problem is that started gets set to true after the first field in the field set, so Mura closes the <fieldset> block after the first input. It should open it at the START of the inputs for that set, and close it at the END of the set so that all fields in that set are inside this <fieldset> block.
<cfif not started> <cfset started=true> </cfif> <cfif arrayLen(attributesArray) EQ a> </ul> </fieldset> <cfset started=true> </cfif>
Now I get the proper results: http://www.splicedigital.com/test/mura_profile_fixed.gif
Unless I am missing something, this is a bug.
-Brian
