[jira] Created: (JEXL-39) BSF engine impl

BSF engine impl
---------------

                 Key: JEXL-39
                 URL: https://issues.apache.org/jira/browse/JEXL-39
             Project: Commons JEXL
          Issue Type: New Feature
    Affects Versions: 1.1
            Reporter: Rahul Akolkar
             Fix For: 1.1.1


See thread titled "[JEXL] Any plans for a BSF wrapper?", started on dev list 09/11/07. On markmail, its at:

    http://commons.markmail.org/message/onjxo7rjmhxnc2as?q=order:date-backward&page=3

This is meant to provide a starting point, if there is interest. Attachment coming, the fix version is purely speculative.


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

     [ https://issues.apache.org/jira/browse/JEXL-39?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Rahul Akolkar updated JEXL-39:
------------------------------

    Attachment: JexlEngine.java

BSFEngine for Commons JEXL.

> BSF engine impl
> ---------------
>
>                 Key: JEXL-39
>                 URL: https://issues.apache.org/jira/browse/JEXL-39
>             Project: Commons JEXL
>          Issue Type: New Feature
>    Affects Versions: 1.1
>            Reporter: Rahul Akolkar
>             Fix For: 1.1.1
>
>         Attachments: JexlEngine.java
>
>
> See thread titled "[JEXL] Any plans for a BSF wrapper?", started on dev list 09/11/07. On markmail, its at:
>     http://commons.markmail.org/message/onjxo7rjmhxnc2as?q=order:date-backward&page=3
> This is meant to provide a starting point, if there is interest. Attachment coming, the fix version is purely speculative.

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

    [ https://issues.apache.org/jira/browse/JEXL-39?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12596368#action_12596368 ]

Sebb commented on JEXL-39:
--------------------------

Just tried adding the class to JMeter. I found that two minor changes are needed:

public void initialize(BSFManager mpr,
should be
public void initialize(BSFManager mgr,

Also, terminate() may be called multiple times, so need to add check for jc != null

Otherwise it seems to work fine.

> BSF engine impl
> ---------------
>
>                 Key: JEXL-39
>                 URL: https://issues.apache.org/jira/browse/JEXL-39
>             Project: Commons JEXL
>          Issue Type: New Feature
>    Affects Versions: 1.1
>            Reporter: Rahul Akolkar
>             Fix For: 1.1.1
>
>         Attachments: JexlEngine.java
>
>
> See thread titled "[JEXL] Any plans for a BSF wrapper?", started on dev list 09/11/07. On markmail, its at:
>     http://commons.markmail.org/message/onjxo7rjmhxnc2as?q=order:date-backward&page=3
> This is meant to provide a starting point, if there is interest. Attachment coming, the fix version is purely speculative.

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

    [ https://issues.apache.org/jira/browse/JEXL-39?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12596521#action_12596521 ]

Rahul Akolkar commented on JEXL-39:
-----------------------------------

Cool, thanks for the feedback. I never followed up with the BSF folks, need to post on bsf-dev (hopefully soon).

> BSF engine impl
> ---------------
>
>                 Key: JEXL-39
>                 URL: https://issues.apache.org/jira/browse/JEXL-39
>             Project: Commons JEXL
>          Issue Type: New Feature
>    Affects Versions: 1.1
>            Reporter: Rahul Akolkar
>             Fix For: 1.1.1
>
>         Attachments: JexlEngine.java
>
>
> See thread titled "[JEXL] Any plans for a BSF wrapper?", started on dev list 09/11/07. On markmail, its at:
>     http://commons.markmail.org/message/onjxo7rjmhxnc2as?q=order:date-backward&page=3
> This is meant to provide a starting point, if there is interest. Attachment coming, the fix version is purely speculative.

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

    [ https://issues.apache.org/jira/browse/JEXL-39?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12596593#action_12596593 ]

Rahul Akolkar commented on JEXL-39:
-----------------------------------

Use oacj.Script instead so multiple statements can be evaluated. See dev@ thread:

  http://markmail.org/message/txj2a7nq5lsjnfnq

> BSF engine impl
> ---------------
>
>                 Key: JEXL-39
>                 URL: https://issues.apache.org/jira/browse/JEXL-39
>             Project: Commons JEXL
>          Issue Type: New Feature
>    Affects Versions: 1.1
>            Reporter: Rahul Akolkar
>             Fix For: 1.1.1
>
>         Attachments: JexlEngine.java
>
>
> See thread titled "[JEXL] Any plans for a BSF wrapper?", started on dev list 09/11/07. On markmail, its at:
>     http://commons.markmail.org/message/onjxo7rjmhxnc2as?q=order:date-backward&page=3
> This is meant to provide a starting point, if there is interest. Attachment coming, the fix version is purely speculative.

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

     [ https://issues.apache.org/jira/browse/JEXL-39?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Sebb updated JEXL-39:
---------------------

    Attachment: JexlEngine2.java

Updated version of Engine:
- corrected typo in initialize() parameter name
- initialize() no longer swallows Exception
- terminate() checks jc for null
- eval() uses script() rather than evaluate() so can pass several statements and return the last value.

> BSF engine impl
> ---------------
>
>                 Key: JEXL-39
>                 URL: https://issues.apache.org/jira/browse/JEXL-39
>             Project: Commons JEXL
>          Issue Type: New Feature
>    Affects Versions: 1.1
>            Reporter: Rahul Akolkar
>             Fix For: 1.1.1
>
>         Attachments: JexlEngine.java, JexlEngine2.java
>
>
> See thread titled "[JEXL] Any plans for a BSF wrapper?", started on dev list 09/11/07. On markmail, its at:
>     http://commons.markmail.org/message/onjxo7rjmhxnc2as?q=order:date-backward&page=3
> This is meant to provide a starting point, if there is interest. Attachment coming, the fix version is purely speculative.

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

    [ https://issues.apache.org/jira/browse/JEXL-39?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12596697#action_12596697 ]

Sebb commented on JEXL-39:
--------------------------

If this is not added to BSF, then the user has to do the following:

{code}
        BSFManager.registerScriptingEngine("jexl",
                "org.apache.commons.jexl.bsf.JexlEngine",
                new String[]{"jexl"});

{code}

This needs to be done once (e.g. in a static block) before attempting to use Jexl as a BSF language.

> BSF engine impl
> ---------------
>
>                 Key: JEXL-39
>                 URL: https://issues.apache.org/jira/browse/JEXL-39
>             Project: Commons JEXL
>          Issue Type: New Feature
>    Affects Versions: 1.1
>            Reporter: Rahul Akolkar
>             Fix For: 1.1.1
>
>         Attachments: JexlEngine.java, JexlEngine2.java
>
>
> See thread titled "[JEXL] Any plans for a BSF wrapper?", started on dev list 09/11/07. On markmail, its at:
>     http://commons.markmail.org/message/onjxo7rjmhxnc2as?q=order:date-backward&page=3
> This is meant to provide a starting point, if there is interest. Attachment coming, the fix version is purely speculative.

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

     [ https://issues.apache.org/jira/browse/JEXL-39?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Rahul Akolkar resolved JEXL-39.
-------------------------------

    Resolution: Won't Fix

Added to Jakarta BSF in r656487, so won't be added to Commons JEXL.

> BSF engine impl
> ---------------
>
>                 Key: JEXL-39
>                 URL: https://issues.apache.org/jira/browse/JEXL-39
>             Project: Commons JEXL
>          Issue Type: New Feature
>    Affects Versions: 1.1
>            Reporter: Rahul Akolkar
>             Fix For: 1.1.1
>
>         Attachments: JexlEngine.java, JexlEngine2.java
>
>
> See thread titled "[JEXL] Any plans for a BSF wrapper?", started on dev list 09/11/07. On markmail, its at:
>     http://commons.markmail.org/message/onjxo7rjmhxnc2as?q=order:date-backward&page=3
> This is meant to provide a starting point, if there is interest. Attachment coming, the fix version is purely speculative.

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