Uploaded image for project: 'Seam International'
  1. Seam International
  2. SEAMINTL-16

Implement an interpolator for strings containing expressions

    Details

    • Type: Task
    • Status: Open (View Workflow)
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: 3.2.0.Alpha1
    • Component/s: Messages
    • Labels:
      None
    • Affects:
      Documentation (Ref Guide, User Guide, etc.), Release Notes
    • Patch Instructions:
      Provide an interpolator and align it with efforts in other Seam modules.
    • Estimated Difficulty:
      Medium

      Description

      Since Weld Extensions has adopted the EL expression resolver, it's fitting that it also provides an interpolator for strings containing EL expression.

      Provide an interpolator that resolves EL expressions and position parameters in strings. This feature was available in Seam 2 [1]. Since it's not provided yet in Seam 3, we see it showing up as a utility in various modules, including the International [2] and REST [3] modules. We should consolidate this to a single, authoritative source.

      It should probably be made an interface with implementations that process specific expression languages, such as EL. For instance, one implementation might interpret MVEL. Just an idea.

      [1] http://anonsvn.jboss.org/repos/seam/branches/community/Seam_2_2/src/main/org/jboss/seam/core/Interpolator.java
      [2] https://github.com/seam/international/raw/master/impl/src/main/java/org/jboss/seam/international/status/builder/Interpolator.java
      [3] https://github.com/seam/rest/raw/master/impl/src/main/java/org/jboss/seam/rest/util/ExpressionLanguageInterpolator.java

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            pmuir Pete Muir added a comment -

            Considered and rejected. Keeping the jar deps for logging to 1.

            Show
            pmuir Pete Muir added a comment - Considered and rejected. Keeping the jar deps for logging to 1.
            Hide
            dan.j.allen Dan Allen added a comment -

            Ah, how about this. Keep the 1 JAR, just put Messages and MessageBundle in the package org.jboss.i18n. I think that's a import developers are going to feel a lot more comfortable about.

            Show
            dan.j.allen Dan Allen added a comment - Ah, how about this. Keep the 1 JAR, just put Messages and MessageBundle in the package org.jboss.i18n. I think that's a import developers are going to feel a lot more comfortable about.
            Hide
            pmuir Pete Muir added a comment -

            Sugggest you raise a JBLOGGING issue.

            Show
            pmuir Pete Muir added a comment - Sugggest you raise a JBLOGGING issue.
            Show
            dan.j.allen Dan Allen added a comment - https://jira.jboss.org/browse/JBLOGGING-49
            Hide
            dan.j.allen Dan Allen added a comment -

            Ken, after this got bumped around, it has become more of an alignment than a new feature, since I know that i18n has an Interpolator already.

            Show
            dan.j.allen Dan Allen added a comment - Ken, after this got bumped around, it has become more of an alignment than a new feature, since I know that i18n has an Interpolator already.

              People

              • Assignee:
                Unassigned
                Reporter:
                dan.j.allen Dan Allen
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:

                  Time Tracking

                  Estimated:
                  Original Estimate - 5 hours
                  5h
                  Remaining:
                  Remaining Estimate - 5 hours
                  5h
                  Logged:
                  Time Spent - Not Specified
                  Not Specified

                    Development