[TAPESTRY-2236] Strip trailing punctuation (_ and $) from member names, as is done with leading punctuation

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 5.0
• Fix Version/s: 5.0.15
• Environment: Tapestry 5.0.11-SNAPSHOT
• Assignee: Howard Lewis Ship
• Reporter: Adriaan Joubert
• Created: 05/Mar/08 10:27
• Updated: 12/Sep/08 18:15
• Resolved: 12/Sep/08 18:15

Description

Hi,

our convention is to use an _ suffix for fields, rather than a prefix. Tapestry cannot naturally transform these field names where needed. Could you please change

org.apache.tapestry.ioc.internal.util.InternalUtils.stripMemberPrefix(String memberName)

to strip an _ suffix as well. The following addition should do the job

while (builder.length>0)

{ char ch = builder.charAt(builder.length-1); if (InternalUtils.NAME_PREFIX.indexOf(ch) < 0) break; builder.deleteCharAt(builder.length-1); }

Cheers,

Adriaan