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 type: :bugfix
  20 component: handlers
  21 release: "0.1"
  22 reporter: Laurens Van Houtven <lvh@laurensvh.be>
  23 status: :closed
  24 disposition: :fixed
  25 creation_time: 2009-03-01 09:27:16.954057 Z
  26 references: []
  27
  28 id: 81d3997adeb3beb0cdaa42795f4030842094ee16
  29 log_events:
  30 - - 2009-03-01 09:27:42.729810 Z
  31   - Laurens Van Houtven <lvh@laurensvh.be>
  32   - created
  33   - The callable args/kwargs fix will be added to master ASAP, since it's a much nicer way of doing things.
  34 - - 2009-03-02 11:20:48.365787 Z
  35   - Laurens Van Houtven <lvh@laurensvh.be>
  36   - closed with disposition fixed
  37   - Fixed in master with the callable solution.