[Zope-CMF] TypesTool, Content Actions.. move to ActionInformation?

[Chris Withers]

alan runyan wrote:
> 
> instances.  And one of the most powerful methods of a ActionInformation
> instance is hte condition attribute Expression.  .. This way you can
> conditionally show a action in the ActionsTool.

...I worry about executing every single condition for every single page request. I know
you don't worry much about efficiency in Plone, but that's not true for all CMF sites, and
that change would affect all CMF sites ;-)

> would people agree that we should move this to ActionInformation instances?

not until there's a real use case.

> I would be more than happy to dedicate time doing this.  But i dont know
> anything about the internals of the ActionsTool at large.  is this 'fairly
> simple' to do?  or is it huge undertaking?

I very much of the "let sleeping dogs lie" opinion.

> i.e.
> * object is locked through webdav
> * in portal_types/Document/actions/edit should have a condition: check to
> make sure this object is not locked
> * we want to conditionally show the edit action.

...agreed this is a nice to have, but the speed impact worries me...

cheers,

Chris