Issue Details (XML | Word | Printable)

Key: TAPESTRY-241
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Jesse Kuhnert
Reporter: Kurtis Williams
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Tapestry

Converter property of contrib:FormTable does not work

Created: 28/Dec/04 06:07 PM   Updated: 30/Apr/06 04:53 PM
Return to search
Component/s: Contrib
Affects Version/s: 3.0.1
Fix Version/s: 3.0.5

Time Tracking:
Not Specified

File Attachments:
  Size
Text File Licensed for inclusion in ASF works TAPESTRY-241-branch-3.0.patch 2006-03-11 07:02 PM Brian K. Wallace 0.9 kB
Environment: Tomcat 5.5

Resolution Date: 14/Mar/06 09:36 PM


 Description  « Hide
I've been trying to use the "converter" property on the contrib:FormTable component without success. I dug into the source code and discovered that the "converter" property on the contained "tableRows" components is not marked as <inherited-binding.../>, it's just marked as a <binding.../> which, of course, will prevent the "converter" property of the owning table from being passed in properly. This all happens on line 237 of FormTable.jwc:
 
<component id="tableRows" type="TableFormRows">
  <inherited-binding name="row" parameter-name="row"/>
  <inherited-binding name="class" parameter-name="rowsClass"/>
  <!-HERE'S THE PROBLEM!!!-->
  <binding name="convertor" expression="convertor"/>
</component>

It seems like that line should read:

<inherited-binding name="converter" expression="converter"/>

 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Brian K. Wallace added a comment - 11/Mar/06 07:02 PM
While the binding change (from binding to inherited-binding) logic is correct, the property is actually "convertor" ("or", not "er").

Jesse Kuhnert added a comment - 14/Mar/06 09:36 PM
patch applied, thanks