[Nuxeo-tickets] [Nuxeo Repository] #1705: Users get bad subscriptions in case of republishing

[Nuxeo Repository]

#1705: Users get bad subscriptions in case of republishing
-------------------------+--------------------------------------------------
 Reporter:  gracinet     |       Owner:  janguenot
     Type:  defect       |      Status:  new      
 Priority:  P2           |   Milestone:  CPS 3.4.2
Component:  CPSWorkflow  |     Version:  TRUNK    
 Severity:  normal       |    Keywords:           
-------------------------+--------------------------------------------------
 They get a link to the pending proxy that has been merged and doesn't
 exist anymore.

 This extract from CPSWorkflow.workflow, lines 1089 and following, seem to
 be the culprit:

 {{{
         if delete_ob is not None:
             # XXX refactoring.
             evtool.notify('workflow_%s' % tdef.getId(),
                           delete_ob, {'kwargs': kwargs})
             container = aq_parent(aq_inner(delete_ob))
             container._delObject(delete_ob.getId())
             if moved_exc is not None:
                 raise moved_exc
             else:
                 raise ObjectDeleted
 }}}

 delete_ob is the object (proxy) that has been merged, and the one on which
 the transition is been executed.

 From CPSSubscriptions point of view, the event should be sent for dest_ob
 (merge target) even though it's not going through any transition. I'm not
 100% sure though, since other parts from CPS might actually expect this
 behaviour.

-- 
Ticket URL: <http://svn.nuxeo.org/trac/pub/ticket/1705>
Nuxeo Repository <http://www.cps-project.org/>
Nuxeo Repository