0
@@ -4,10 +4,10 @@
0
 
0
 == ruote - 0.9.19    not yet released
0
 
0
+- bug   #21306 : made sure duplicate insert bug for ActiveParticipant is fixed.
0
 - todo  #21334 : optimized fulldup() (almost 2 times faster).
0
 - bug   #21322 : reserve mutex not freed upon cancel. Fixed.
0
 - bug   #21305 : replay_at_error was broken for subprocesses. Fixed.
0
-- bug   #21306 : made sure duplicate insert bug for ActiveParticipant is fixed.
0
 - todo  #21167 : exptree.to_code_s() now respecting non-string att values.
0
 - bug   #20714 : engine.lookup_variable was coaxing target value into a regex. 
0
                  Fixed.

0
@@ -177,8 +177,6 @@ module Extras
0
     #
0
     def Workitem.from_owfe_workitem (wi, store_name=nil)
0
 
0
-      i = nil
0
-
0
       i = Workitem.new
0
       i.fei = wi.fei.to_s
0
       i.wfid = wi.fei.wfid
0
@@ -198,6 +196,9 @@ module Extras
0
         # sometimes, the saved workitem id wasn't updated, was remaining at 0
0
         # thus finding if necessary...
0
 
0
+      #i.fields.delete_all
0
+        # why do I need that ??? fields were getting recycled...
0
+
0
       # This is a field set by the active participant immediately
0
       # before calling this method.
0
       # the default behavior is "use field method"
0
@@ -670,7 +671,8 @@ module Extras
0
     #
0
     def cancel (cancelitem)
0
 
0
-      Workitem.delete_all([ "fei = ?", cancelitem.fei.to_s ])
0
+      Workitem.destroy_all([ "fei = ?", cancelitem.fei.to_s ])
0
+        # note that delete_all was not removing workitem fields
0
     end
0
 
0
     #