Gregor Imboden wrote:
Hi Devs,
The Deactivate/Publish menu items in the Default Publication are
selectable even if the workflow event cannot be invoked.
Shouldn't we use addErrorMessage() instead of addInfoMessage() in
doCheckPreconditions() to filter these usecases out ?
IMO the default behaviour is arguable.
Consider the following situation:
- foo (live)
- bar (review)
- baz (review)
If you want to publish "bar" and "baz" in one go, you could use
PublishSubtree on "foo". IMO it should therefore be allowed even
though the event can't be invoked on "foo" itself.
An alternative would be to add the workflow transition
publish: live -> live
but IMO this is confusing.
-- Andreas
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
