You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Vishal Goenka (JIRA)" <ji...@apache.org> on 2008/10/08 18:45:44 UTC

[jira] Created: (JEXL-49) Block statements aren't parsed

Block statements aren't parsed
------------------------------

                 Key: JEXL-49
                 URL: https://issues.apache.org/jira/browse/JEXL-49
             Project: Commons JEXL
          Issue Type: Bug
    Affects Versions: 1.1.1
         Environment: All
            Reporter: Vishal Goenka
            Priority: Critical
             Fix For: 1.1.1
         Attachments: astblock.patch

The ExpressionFactory throws an exception when parsing a block statement such as:

{ a='b'; c='d'; }

 I traced it down to a missing check for ASTBlock in the ExpressionFactory. The patch is attached although it was generated against a local checkin of this file in my local svn, hence version numbers are different. The patch is simple, so I decided to attach it as is, rather than generating it against the apache-commons svn trunk. To apply this patch, you will need to manually edit the code and add the two lines added.



-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (JEXL-49) Block statements aren't parsed

Posted by "Henri Biestro (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/JEXL-49?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12732553#action_12732553 ] 

Henri Biestro commented on JEXL-49:
-----------------------------------

Fixed in 2.0.

> Block statements aren't parsed
> ------------------------------
>
>                 Key: JEXL-49
>                 URL: https://issues.apache.org/jira/browse/JEXL-49
>             Project: Commons JEXL
>          Issue Type: Bug
>    Affects Versions: 1.1.1
>         Environment: All
>            Reporter: Vishal Goenka
>            Priority: Critical
>             Fix For: 1.1.1
>
>         Attachments: astblock.patch
>
>   Original Estimate: 0.17h
>  Remaining Estimate: 0.17h
>
> The ExpressionFactory throws an exception when parsing a block statement such as:
> { a='b'; c='d'; }
>  I traced it down to a missing check for ASTBlock in the ExpressionFactory. The patch is attached.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (JEXL-49) Block statements aren't parsed

Posted by "Rahul Akolkar (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/JEXL-49?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Rahul Akolkar updated JEXL-49:
------------------------------

         Fix Version/s:     (was: 1.1.1)
                        2.0
    Remaining Estimate:     (was: 0.17h)
     Original Estimate:     (was: 0.17h)

Set fix version to v2.0.


> Block statements aren't parsed
> ------------------------------
>
>                 Key: JEXL-49
>                 URL: https://issues.apache.org/jira/browse/JEXL-49
>             Project: Commons JEXL
>          Issue Type: Bug
>    Affects Versions: 1.1.1
>         Environment: All
>            Reporter: Vishal Goenka
>            Priority: Critical
>             Fix For: 2.0
>
>         Attachments: astblock.patch
>
>
> The ExpressionFactory throws an exception when parsing a block statement such as:
> { a='b'; c='d'; }
>  I traced it down to a missing check for ASTBlock in the ExpressionFactory. The patch is attached.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (JEXL-49) Block statements aren't parsed

Posted by "Vishal Goenka (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/JEXL-49?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Vishal Goenka updated JEXL-49:
------------------------------

    Attachment: astblock.patch

Pretty trivial patch

> Block statements aren't parsed
> ------------------------------
>
>                 Key: JEXL-49
>                 URL: https://issues.apache.org/jira/browse/JEXL-49
>             Project: Commons JEXL
>          Issue Type: Bug
>    Affects Versions: 1.1.1
>         Environment: All
>            Reporter: Vishal Goenka
>            Priority: Critical
>             Fix For: 1.1.1
>
>         Attachments: astblock.patch
>
>   Original Estimate: 0.17h
>  Remaining Estimate: 0.17h
>
> The ExpressionFactory throws an exception when parsing a block statement such as:
> { a='b'; c='d'; }
>  I traced it down to a missing check for ASTBlock in the ExpressionFactory. The patch is attached although it was generated against a local checkin of this file in my local svn, hence version numbers are different. The patch is simple, so I decided to attach it as is, rather than generating it against the apache-commons svn trunk. To apply this patch, you will need to manually edit the code and add the two lines added.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (JEXL-49) Block statements aren't parsed

Posted by "Vishal Goenka (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/JEXL-49?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Vishal Goenka updated JEXL-49:
------------------------------

    Description: 
The ExpressionFactory throws an exception when parsing a block statement such as:

{ a='b'; c='d'; }

 I traced it down to a missing check for ASTBlock in the ExpressionFactory. The patch is attached.


  was:
The ExpressionFactory throws an exception when parsing a block statement such as:

{ a='b'; c='d'; }

 I traced it down to a missing check for ASTBlock in the ExpressionFactory. The patch is attached although it was generated against a local checkin of this file in my local svn, hence version numbers are different. The patch is simple, so I decided to attach it as is, rather than generating it against the apache-commons svn trunk. To apply this patch, you will need to manually edit the code and add the two lines added.




> Block statements aren't parsed
> ------------------------------
>
>                 Key: JEXL-49
>                 URL: https://issues.apache.org/jira/browse/JEXL-49
>             Project: Commons JEXL
>          Issue Type: Bug
>    Affects Versions: 1.1.1
>         Environment: All
>            Reporter: Vishal Goenka
>            Priority: Critical
>             Fix For: 1.1.1
>
>         Attachments: astblock.patch
>
>   Original Estimate: 0.17h
>  Remaining Estimate: 0.17h
>
> The ExpressionFactory throws an exception when parsing a block statement such as:
> { a='b'; c='d'; }
>  I traced it down to a missing check for ASTBlock in the ExpressionFactory. The patch is attached.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Resolved: (JEXL-49) Block statements aren't parsed

Posted by "Rahul Akolkar (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/JEXL-49?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Rahul Akolkar resolved JEXL-49.
-------------------------------

    Resolution: Fixed

This should be fixed in the 2.0 branch in SVN. Please try it out (by building the branch) and if further work is needed, reopen this issue with a complete JUnit test that fails.


> Block statements aren't parsed
> ------------------------------
>
>                 Key: JEXL-49
>                 URL: https://issues.apache.org/jira/browse/JEXL-49
>             Project: Commons JEXL
>          Issue Type: Bug
>    Affects Versions: 1.1.1
>         Environment: All
>            Reporter: Vishal Goenka
>            Priority: Critical
>             Fix For: 2.0
>
>         Attachments: astblock.patch
>
>
> The ExpressionFactory throws an exception when parsing a block statement such as:
> { a='b'; c='d'; }
>  I traced it down to a missing check for ASTBlock in the ExpressionFactory. The patch is attached.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.