Sergiu Dumitriu wrote:
On 05/07/2010 10:42 PM, cjdelisle (SVN) wrote:
Author: cjdelisle
Date: 2010-05-07 22:42:06 +0200 (Fri, 07 May 2010)
New Revision: 28794
Modified:
platform/xwiki-applications/trunk/administration/src/main/resources/XWiki/ConfigurableClass.xml
Log:
XAADMINISTRATION-138: ConfigurableClass producing invalid xhtml
Fixed regression introduced in r28787 preventing saving.
Modified:
platform/xwiki-applications/trunk/administration/src/main/resources/XWiki/ConfigurableClass.xml
===================================================================
---
platform/xwiki-applications/trunk/administration/src/main/resources/XWiki/ConfigurableClass.xml 2010-05-07
20:21:45 UTC (rev 28793)
+++
platform/xwiki-applications/trunk/administration/src/main/resources/XWiki/ConfigurableClass.xml 2010-05-07
20:42:06 UTC (rev 28794)
@@ -567,8 +567,8 @@
#end
</label>
## Have to prepend the app name to prevent id collisions.
Are you sure it's always going to be _0_ ?
It is always the first object but I concede it is a fallacy to assume it must be _0_.
Do you know any way to get the number of an object without refactoring the foreach loop to
call getObject("class", nb) each time?
-
$obj.display($propName,
"edit").replaceAll("$objClass.getName()_0_$propName",
-
"${escapedAppName}_$objClass.getName()_0_$propName")
+ $obj.display($propName, "edit").replaceAll("
id='$objClass.getName()_0_$propName",
+ "
id='${escapedAppName}_$objClass.getName()_0_$propName")
</p>
#end## If property is in propertiesToShow
#end## Foreach property in this class