Tapestry
  1. Tapestry
  2. TAPESTRY-1739

Add byte, short and float Translators

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 5.0
    • Fix Version/s: 5.0.8
    • Component/s: tapestry-core
    • Labels:
      None

      Description

      Add byte, short and float Translators. Don't know if char and boolean need translators.

      I would be happy to write the code as it's just cut and paste work. I coded short and was able to build core, deploy to my local maven repository and use it in my T5 project.

      There are no Sub-Tasks for this issue.

        Activity

        Hide
        Dave Dombrosky added a comment -

        This issue should be reopened. Tapestry 5.0.11 still does not contain a Short translator.

        Show
        Dave Dombrosky added a comment - This issue should be reopened. Tapestry 5.0.11 still does not contain a Short translator.
        Hide
        Howard M. Lewis Ship added a comment -

        I took this opportunity to change the semantics of Translator such that nulls and blank strings are not passed in (this is now handled via FieldValidationSupportImpl). That simplifies the translators even further since the endlessly duplicated null checks are gone.

        Show
        Howard M. Lewis Ship added a comment - I took this opportunity to change the semantics of Translator such that nulls and blank strings are not passed in (this is now handled via FieldValidationSupportImpl). That simplifies the translators even further since the endlessly duplicated null checks are gone.

          People

          • Assignee:
            Howard M. Lewis Ship
            Reporter:
            Bruce Myers
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development