bugs/issue-81d3997adeb3beb0cdaa42795f4030842094ee16.yaml

   1 --- !ditz.rubyforge.org,2008-03-06/issue
   2 title: SequentialHandler doesn't copy points
   3 desc: |-
   4   SUMMARY
   5
   6   The sequential handler doesn't make copies of the point dictionaries (or, depending on the commit, makes only one) ever since the introduction of the call_children method.
   7
   8   REPRODUCTION
   9
  10   This bug turns up when you use multiple handlers that decide it's okay to mangle point data. Usually this is because they want to pretty-print the UNIX timestamp (FastFileWriter does this, and so does EchoHandler).
  11
  12   CAUSE?
  13
  14   I think this is because the arguments to the call_children method (or any method for that matter) are only evaluated once. So, even though child handlers all get a copy, they all get the *same* copy. This is obviously not what we want.
  15
  16   FIX?
  17
  18   A possible fix is to pass args and kwargs to children using callables. Another possible fix is to tell call_children to copy/deepcopy its arguments before feeding it to the children. The former is probably the best solution.
  19
  20
  21 type: :bugfix
  22 component: handlers
  23 release: "0.1"
  24 reporter: Laurens Van Houtven <lvh@laurensvh.be>
  25 status: :unstarted
  26 disposition:
  27 creation_time: 2009-03-01 09:27:16.954057 Z
  28 references: []
  29
  30 id: 81d3997adeb3beb0cdaa42795f4030842094ee16
  31 log_events:
  32 - - 2009-03-01 09:27:42.729810 Z
  33   - Laurens Van Houtven <lvh@laurensvh.be>
  34   - created
  35   - The callable args/kwargs fix will be added to master ASAP, since it's a much nicer way of doing things.