Wicket
  1. Wicket
  2. WICKET-3941

Provide hooks to alias the class names generated/consumed by PackageMapper

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.5-RC5.1
    • Fix Version/s: 1.5-RC6
    • Component/s: wicket
    • Labels:
      None

      Description

      One small feature that I feel it is really useful.

        Activity

        Bruno Borges created issue -
        Bruno Borges made changes -
        Field Original Value New Value
        Attachment PackageMapper.patch [ 12488342 ]
        Martin Grigorov made changes -
        Summary Provide lowercase class name for PackageMapper Provide hooks to alias the class names generated/consumed by PackageMapper
        Hide
        Martin Grigorov added a comment -

        Added overrideable methods with which the sub-implementation can manipulate however they want the class names. See PackageMapperTest for examples.

        Show
        Martin Grigorov added a comment - Added overrideable methods with which the sub-implementation can manipulate however they want the class names. See PackageMapperTest for examples.
        Martin Grigorov made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Assignee Martin Grigorov [ mgrigorov ]
        Fix Version/s 1.5-RC6 [ 12316657 ]
        Resolution Fixed [ 1 ]
        Hide
        Bruno Borges added a comment -

        Martin, thanks for taking a look at this issue.

        But I do think the PackageMapper should already offer the lower-case pattern. URLs are mostly lower-case, so I think this class could already come with that feature (disabled by default), but as simple as setLowerCase(true); as possible

        Show
        Bruno Borges added a comment - Martin, thanks for taking a look at this issue. But I do think the PackageMapper should already offer the lower-case pattern. URLs are mostly lower-case, so I think this class could already come with that feature (disabled by default), but as simple as setLowerCase(true); as possible

          People

          • Assignee:
            Martin Grigorov
            Reporter:
            Bruno Borges
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development