[jira] Created: (VELOCITY-588) Provide a an ubespector that allows chaining other uberspectors

[Velocity - Dev mailing list-2]

Provide a an ubespector that allows chaining other uberspectors
---------------------------------------------------------------

                 Key: VELOCITY-588
                 URL: https://issues.apache.org/jira/browse/VELOCITY-588
             Project: Velocity
          Issue Type: Improvement
          Components: Engine
    Affects Versions: 1.5
            Reporter: Vincent Massol
            Priority: Minor


In XWiki project we're using Velocity and have a need to write several uberspectors that we'd like to chain (see http://jira.xwiki.org/jira/browse/XWIKI-2182). We're writing a custom uberspector that'll allow chaining other uberspectors but we think that uberspector should best be located in the Velocity project if you're interested.

Let us know what you think and if you agree we can donate it to you or you could create one from scratch if you agree with the use case.

Thanks

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...

[Velocity - Dev mailing list-2]

    [ https://issues.apache.org/jira/browse/VELOCITY-588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12575655#action_12575655 ]

Claude Brisson commented on VELOCITY-588:
-----------------------------------------

That is a brillant idea.

As I understand your use cases, the first uberspector in the chain would check for annotations and, for instance, add log or security.

We are interested, of course. Plus, using chained uberspectors can help us to factorize existing code.

and I already think to another use case :

a view tool chained uberspector that allows syntaxes like #set( $session.foo = bar ) instead of #set( $junk = $session.setAttribute('foo','bar')

I may have the time to code that these days, I'm not sure. Let us now if you start coding it, I'll also add a comment here if I do so first on my side.

> Provide a an ubespector that allows chaining other uberspectors
> ---------------------------------------------------------------
>
>                 Key: VELOCITY-588
>                 URL: https://issues.apache.org/jira/browse/VELOCITY-588
>             Project: Velocity
>          Issue Type: Improvement
>          Components: Engine
>    Affects Versions: 1.5
>            Reporter: Vincent Massol
>            Priority: Minor
>
> In XWiki project we're using Velocity and have a need to write several uberspectors that we'd like to chain (see http://jira.xwiki.org/jira/browse/XWIKI-2182). We're writing a custom uberspector that'll allow chaining other uberspectors but we think that uberspector should best be located in the Velocity project if you're interested.
> Let us know what you think and if you agree we can donate it to you or you could create one from scratch if you agree with the use case.
> Thanks

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...

[Velocity - Dev mailing list-2]

    [ https://issues.apache.org/jira/browse/VELOCITY-588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12575849#action_12575849 ]

Nathan Bubna commented on VELOCITY-588:
---------------------------------------

Yeah, this would be great.  I'm messing around with uberspectors that magically provide utility functions for all references  (e.g. instead of $display.measure($foo), i could just do $foo.measure()) and have been thinking about ways to make the uberspectors more extensible.  Chaining would be fantastic.

> Provide a an ubespector that allows chaining other uberspectors
> ---------------------------------------------------------------
>
>                 Key: VELOCITY-588
>                 URL: https://issues.apache.org/jira/browse/VELOCITY-588
>             Project: Velocity
>          Issue Type: Improvement
>          Components: Engine
>    Affects Versions: 1.5
>            Reporter: Vincent Massol
>            Priority: Minor
>
> In XWiki project we're using Velocity and have a need to write several uberspectors that we'd like to chain (see http://jira.xwiki.org/jira/browse/XWIKI-2182). We're writing a custom uberspector that'll allow chaining other uberspectors but we think that uberspector should best be located in the Velocity project if you're interested.
> Let us know what you think and if you agree we can donate it to you or you could create one from scratch if you agree with the use case.
> Thanks

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...

[Velocity - Dev mailing list-2]

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

Sergiu Dumitriu updated VELOCITY-588:
-------------------------------------

    Attachment: ChainableUberspectorsXWiki.patch

This is the first version of the implementation done for XWiki. Please review it, and if it looks good I can make a patch for the velocity trunk.

> Provide a an ubespector that allows chaining other uberspectors
> ---------------------------------------------------------------
>
>                 Key: VELOCITY-588
>                 URL: https://issues.apache.org/jira/browse/VELOCITY-588
>             Project: Velocity
>          Issue Type: Improvement
>          Components: Engine
>    Affects Versions: 1.5
>            Reporter: Vincent Massol
>            Priority: Minor
>         Attachments: ChainableUberspectorsXWiki.patch
>
>
> In XWiki project we're using Velocity and have a need to write several uberspectors that we'd like to chain (see http://jira.xwiki.org/jira/browse/XWIKI-2182). We're writing a custom uberspector that'll allow chaining other uberspectors but we think that uberspector should best be located in the Velocity project if you're interested.
> Let us know what you think and if you agree we can donate it to you or you could create one from scratch if you agree with the use case.
> Thanks

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...

[Velocity - Dev mailing list-2]

    [ https://issues.apache.org/jira/browse/VELOCITY-588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12604030#action_12604030 ]

Nathan Bubna commented on VELOCITY-588:
---------------------------------------

Hi Sergiu,

My apologies for my slow response.  But today i'm catching up on some Velocity stuff. :)

This looks pretty good.  Should i assume that it is working well for you?  I only wonder if it might be simpler to keep Uberspectors in an ordered list (in the root/parent Uberspector) and just interrogate them in order until one returns a non-null value.   This way you might be able to avoid requiring them to implement a ChainableUberspector interface.

Of course, i haven't tried implementing it that way.  Have you?  What do you think of the idea?

> Provide a an ubespector that allows chaining other uberspectors
> ---------------------------------------------------------------
>
>                 Key: VELOCITY-588
>                 URL: https://issues.apache.org/jira/browse/VELOCITY-588
>             Project: Velocity
>          Issue Type: Improvement
>          Components: Engine
>    Affects Versions: 1.5
>            Reporter: Vincent Massol
>            Priority: Minor
>         Attachments: ChainableUberspectorsXWiki.patch
>
>
> In XWiki project we're using Velocity and have a need to write several uberspectors that we'd like to chain (see http://jira.xwiki.org/jira/browse/XWIKI-2182). We're writing a custom uberspector that'll allow chaining other uberspectors but we think that uberspector should best be located in the Velocity project if you're interested.
> Let us know what you think and if you agree we can donate it to you or you could create one from scratch if you agree with the use case.
> Thanks

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...

[Velocity - Dev mailing list-2]

    [ https://issues.apache.org/jira/browse/VELOCITY-588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12604106#action_12604106 ]

Sergiu Dumitriu commented on VELOCITY-588:
------------------------------------------

The patch is outdated, a better source is in the XWiki repository, at http://svn.xwiki.org/svnroot/xwiki/xwiki-platform/core/trunk/xwiki-velocity/src/main/java/org/xwiki/velocity/introspection/

Yes, it works pretty well until now. I also have Chainable Introspectors locally, but the code requires some cleaning before committing. Here is a proposed architecture for how we're planning to use custom introspectors/uberspectors in XWiki: http://www.nabble.com/Deprecating-all-the-APIs-in-favor-of-a-Velocity-uberspecting-chain-td17208816.html#a17208816

Your idea is good, too. As a pro, it keeps things simple. As a con, it isn't as flexible as the chainable one, basically because it allows enhancing the list of returned methods, but doesn't allow restricting that list, or performing additional tasks with the methods found by previous introspectors/uberspectors. For example, we can't block access to methods that don't meet the access rights restrictions (see RightsCheckingIntrospector in the XWiki proposal), and we can't implement DeprecatedCheckIntrospector that relies entirely on the results of the previous introspectors.

I guess I could write a different Uberspector that uses simple linking, instead of chaining, and the user (application developer) can choose the one that better suits his needs.

> Provide a an ubespector that allows chaining other uberspectors
> ---------------------------------------------------------------
>
>                 Key: VELOCITY-588
>                 URL: https://issues.apache.org/jira/browse/VELOCITY-588
>             Project: Velocity
>          Issue Type: Improvement
>          Components: Engine
>    Affects Versions: 1.5
>            Reporter: Vincent Massol
>            Priority: Minor
>         Attachments: ChainableUberspectorsXWiki.patch
>
>
> In XWiki project we're using Velocity and have a need to write several uberspectors that we'd like to chain (see http://jira.xwiki.org/jira/browse/XWIKI-2182). We're writing a custom uberspector that'll allow chaining other uberspectors but we think that uberspector should best be located in the Velocity project if you're interested.
> Let us know what you think and if you agree we can donate it to you or you could create one from scratch if you agree with the use case.
> Thanks

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...

[Velocity - Dev mailing list-2]

    [ https://issues.apache.org/jira/browse/VELOCITY-588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12604118#action_12604118 ]

Nathan Bubna commented on VELOCITY-588:
---------------------------------------

You're right.  The wrapping mechanism allows a lot more flexibility.  I hadn't thought of that.  I'll try to find some more time (hopefully this week) to look at your more up to date stuff.  I really like this idea.  Since it seems we might be waiting a while on a bug fix before we roll out a 1.6 release, i might try to get something of this into the trunk soon.  It's a great idea, and i'm already thinking of ways i'd like to use it. :)

> Provide a an ubespector that allows chaining other uberspectors
> ---------------------------------------------------------------
>
>                 Key: VELOCITY-588
>                 URL: https://issues.apache.org/jira/browse/VELOCITY-588
>             Project: Velocity
>          Issue Type: Improvement
>          Components: Engine
>    Affects Versions: 1.5
>            Reporter: Vincent Massol
>            Priority: Minor
>         Attachments: ChainableUberspectorsXWiki.patch
>
>
> In XWiki project we're using Velocity and have a need to write several uberspectors that we'd like to chain (see http://jira.xwiki.org/jira/browse/XWIKI-2182). We're writing a custom uberspector that'll allow chaining other uberspectors but we think that uberspector should best be located in the Velocity project if you're interested.
> Let us know what you think and if you agree we can donate it to you or you could create one from scratch if you agree with the use case.
> Thanks

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...

[Velocity - Dev mailing list-2]

    [ https://issues.apache.org/jira/browse/VELOCITY-588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12609375#action_12609375 ]

Nathan Bubna commented on VELOCITY-588:
---------------------------------------

Sergiu,

I think this is something i'd like to move forward with implementing this in Velocity.  However, your patch and the SVN version are under LGPL.  Are you the creator/copyright owner?  If so, could you create an ASL 2.0  version for us to use?  If not, that's fine, i'll just make my own clean attempt at it.

> Provide a an ubespector that allows chaining other uberspectors
> ---------------------------------------------------------------
>
>                 Key: VELOCITY-588
>                 URL: https://issues.apache.org/jira/browse/VELOCITY-588
>             Project: Velocity
>          Issue Type: Improvement
>          Components: Engine
>    Affects Versions: 1.5
>            Reporter: Vincent Massol
>            Priority: Minor
>         Attachments: ChainableUberspectorsXWiki.patch
>
>
> In XWiki project we're using Velocity and have a need to write several uberspectors that we'd like to chain (see http://jira.xwiki.org/jira/browse/XWIKI-2182). We're writing a custom uberspector that'll allow chaining other uberspectors but we think that uberspector should best be located in the Velocity project if you're interested.
> Let us know what you think and if you agree we can donate it to you or you could create one from scratch if you agree with the use case.
> Thanks

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...

[Velocity - Dev mailing list-2]

    [ https://issues.apache.org/jira/browse/VELOCITY-588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12609400#action_12609400 ]

Sergiu Dumitriu commented on VELOCITY-588:
------------------------------------------

Hi,

Yes, I'm the creator of the code. It is currently under LGPL because that is the license for all XWiki code, but I'll gladly give the implementation to the Apache foundation. I'll work on providing a clean patch for the velocity trunk, but it will take some time, as I'm busy with my PhD and with XWiki.

> Provide a an ubespector that allows chaining other uberspectors
> ---------------------------------------------------------------
>
>                 Key: VELOCITY-588
>                 URL: https://issues.apache.org/jira/browse/VELOCITY-588
>             Project: Velocity
>          Issue Type: Improvement
>          Components: Engine
>    Affects Versions: 1.5
>            Reporter: Vincent Massol
>            Priority: Minor
>         Attachments: ChainableUberspectorsXWiki.patch
>
>
> In XWiki project we're using Velocity and have a need to write several uberspectors that we'd like to chain (see http://jira.xwiki.org/jira/browse/XWIKI-2182). We're writing a custom uberspector that'll allow chaining other uberspectors but we think that uberspector should best be located in the Velocity project if you're interested.
> Let us know what you think and if you agree we can donate it to you or you could create one from scratch if you agree with the use case.
> Thanks

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...

[Velocity - Dev mailing list-2]

    [ https://issues.apache.org/jira/browse/VELOCITY-588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12609403#action_12609403 ]

Nathan Bubna commented on VELOCITY-588:
---------------------------------------

Thanks, Sergiu.  I'll look forward to it!

> Provide a an ubespector that allows chaining other uberspectors
> ---------------------------------------------------------------
>
>                 Key: VELOCITY-588
>                 URL: https://issues.apache.org/jira/browse/VELOCITY-588
>             Project: Velocity
>          Issue Type: Improvement
>          Components: Engine
>    Affects Versions: 1.5
>            Reporter: Vincent Massol
>            Priority: Minor
>         Attachments: ChainableUberspectorsXWiki.patch
>
>
> In XWiki project we're using Velocity and have a need to write several uberspectors that we'd like to chain (see http://jira.xwiki.org/jira/browse/XWIKI-2182). We're writing a custom uberspector that'll allow chaining other uberspectors but we think that uberspector should best be located in the Velocity project if you're interested.
> Let us know what you think and if you agree we can donate it to you or you could create one from scratch if you agree with the use case.
> Thanks

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...