Hello Dieter Maurer, Thursday, October 20, 2005, 1:44:27 AM, you wrote:
DM> Victor Safronovich wrote at 2005-10-17 13:10 +0600: >> ... >>By the way i think code: >> for w in wfs: >> w.notifySuccess(ob, action, res) >> if reindex: >> self._reindexWorkflowVariables(ob) >>in _invokeWithNotification should be turn upside-down to: >> if reindex: >> self._reindexWorkflowVariables(ob) >> for w in wfs: >> w.notifySuccess(ob, action, res) >>am i right? DM> Why do you think so? DM> I find the original order more natural (in case "notifySuccess" DM> would change some workflow variables). My product used DCWorkflow in a wrong way. It is override notifySuccess method and called many scripts in this method, instead of adding before and after scripts to transition. And to my product it is necessary ( i think because of allowedRolesAndUsers index ) that ob must be reindexed before notifySuccess was called. By the way is it correct to skip ObjectMoved, ObjectDeleted errors from notifyBefore, notifySuccess, notifyException methods in _invokeWithNotification method? -- Best regards, Victor Safronovich NauMen.NauDoc.SoftwareDeveloper http://www.naumen.ru _______________________________________________ Zope-CMF maillist - Zope-CMF@lists.zope.org http://mail.zope.org/mailman/listinfo/zope-cmf See http://collector.zope.org/CMF for bug reports and feature requests
