[TRINIDAD-810] Faces plugin copies the property-values to attribute-values incorrectly if there is a sibling property-metadata element

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.2.3-plugins
• Fix Version/s: 1.2.4-plugins
• Component/s: Plugins
• Assignee: Andrew Robinson
• Reporter: Andrew Robinson
• Created: 09/Nov/07 02:18
• Updated: 16/Nov/07 02:29
• Resolved: 09/Nov/07 02:30

Description

Faces plugin copies the property-values to attribute-values incorrectly if there is a sibling property-metadata element

example XML source:

<property>
<description><![CDATA[test]]></description>
<property-name>myProperty</property-name>
<property-class>java.lang.String</property-class>
<default-value>one</default-value>
<property-extension>
<mfp:required>false</mfp:required>
<mfp:property-values>one two three four</mfp:property-values>
<mfp:property-metadata>
<mfp:preferred>true</mfp:preferred>
</mfp:property-metadata>
</property-extension>
</property>

Currently generates:

<property>
<description>test</description>
<property-name>myProperty</property-name>
<property-class>java.lang.String</property-class>
<default-value>one</default-value>
<property-extension>
<attribute-values>one two three four</attribute-values>
<property-metadata>
<preferred>true</preferred>
</property-metadata>
</property-extension>
</property>

Desired result:

<property>
<description>test</description>
<property-name>myProperty</property-name>
<property-class>java.lang.String</property-class>
<default-value>one</default-value>
<property-extension>
<property-metadata>
<preferred>true</preferred>
<attribute-values>one two three four</attribute-values>
</property-metadata>
</property-extension>
</property>

notice the new location of "<attribute-values>"