Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Later
    • Affects Version/s: 4.0.2
    • Fix Version/s: 4.1.2
    • Component/s: Framework
    • Labels:
      None

      Description

      ExpressionEvaluator abstracts OGNL by providing an OGNL independent interface for evaluating expressions. It is very useful not only for bindings but for custom usage as well. I'd like to propose some enhancements to make it more useable:

      • add more configuration options, for example allow specifying a custom OGNL class resolver
      • allow custom context variables like

      public Object read(Object target, String expression, Map<String, ?> variables);

      The values in "variables" would be accessible as OGNL global variables, eg. #mycustomvariable
      For more information see http://www.ognl.org/2.6.9/Documentation/html/LanguageGuide/varref.html
      (- maybe the interface should be renamed to OgnlExpressionEvaluator)
      (- maybe there could be configurable default variables when evaluating bindings, like #locale -> requestCycle.engine.locale. These would help for application developers to simplify the page files. I know that it is another way to implement existing functionality - <inject> - but maybe it would be useful.)

      The two latter are only quick ideas but the rest would be very useful!

      If you think so then I send my own ExpressionEvaluator implementation (which is compatible with the current one in Tapestry).

      Regards,
      Norbi

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        298d 8h 45m 1 Jesse Kuhnert 19/Mar/07 00:56
        Mark Thomas made changes -
        Workflow Default workflow, editable Closed status [ 12567740 ] jira [ 12590868 ]
        Mark Thomas made changes -
        Workflow jira [ 12372364 ] Default workflow, editable Closed status [ 12567740 ]
        Jesse Kuhnert made changes -
        Assignee Jesse Kuhnert [ jkuhnert ]
        Resolution Later [ 7 ]
        Status Open [ 1 ] Resolved [ 5 ]
        Hide
        Jesse Kuhnert added a comment -

        Not an invalid issue - I do actually plan on making these kinds of changes but want to wait until ognl 3.0 comes out for that. (3.0 will involve a lot of things to make it easier to expose all of the core functionality in IoC containers. )

        Show
        Jesse Kuhnert added a comment - Not an invalid issue - I do actually plan on making these kinds of changes but want to wait until ognl 3.0 comes out for that. (3.0 will involve a lot of things to make it easier to expose all of the core functionality in IoC containers. )
        Jesse Kuhnert made changes -
        Field Original Value New Value
        Fix Version/s 4.1.2 [ 12312202 ]
        Norbert Sándor created issue -

          People

          • Assignee:
            Jesse Kuhnert
            Reporter:
            Norbert Sándor
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development