Uploaded image for project: 'Struts 2'
  1. Struts 2
  2. WW-4651

Negative number is considered an arithmetic expression

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.3.29
    • Fix Version/s: 2.3.30, 2.5.2
    • Component/s: None
    • Labels:
      None

      Description

      With the arithmetic expression check that has been introduced with 2.3.29, negative numbers are considered to be arithmetic expressions.
      This makes it impossible to use number fields with negative values as KeyProperty in collection type conversion.

      I can understand that one does not want to have arithmetic expressions in parameter names. OTOH, I would not consider a negative number as an arithmetic expression.

      Please let me know if I should rather open this issue in the ognl project.

        Activity

        Hide
        lukaszlenart Lukasz Lenart added a comment -

        I think it would be good to post this issue there as well to allow keep track of changes for other projects that are using OGNL.

        Show
        lukaszlenart Lukasz Lenart added a comment - I think it would be good to post this issue there as well to allow keep track of changes for other projects that are using OGNL.
        Hide
        clenggenhager Christoph Lenggenhager added a comment -
        Show
        clenggenhager Christoph Lenggenhager added a comment - Okay. I created https://github.com/jkuhnert/ognl/issues/28
        Hide
        lukaszlenart Lukasz Lenart added a comment -

        Done by releasing new version of OGNL

        Show
        lukaszlenart Lukasz Lenart added a comment - Done by releasing new version of OGNL

          People

          • Assignee:
            lukaszlenart Lukasz Lenart
            Reporter:
            clenggenhager Christoph Lenggenhager
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development