Uploaded image for project: 'Tapestry'
  1. Tapestry
  2. TAPESTRY-2236

Strip trailing punctuation (_ and $) from member names, as is done with leading punctuation

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 5.0
    • Fix Version/s: 5.0.15
    • Component/s: None
    • Labels:
      None
    • Environment:
      Tapestry 5.0.11-SNAPSHOT

      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

        Attachments

          Activity

            People

            • Assignee:
              hlship Howard M. Lewis Ship
              Reporter:
              adriaanjoubert Adriaan Joubert
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: