emezac / tubular

Automatically exported from code.google.com/p/tubular
0 stars 0 forks source link

Wrong inheritance between option for sibling steps #33

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
When two sibling steps have same option name, the second one get options from 
the first one if it dont declare it.
For exemple, in wrap-sequence-004 test, the sibling steps are wrap-sequence and 
wrap, they have the same option names (wrapper, wrapper-namespace ...) and wrap 
step doesn't declare wrapper-prefix option :
      <p:wrap-sequence wrapper="wrapper"
                       wrapper-namespace="http://baz.com" wrapper-prefix="baz"/>    
      <p:wrap match="/" wrapper="foo"/> 

In execution, input.getOptionValue(XProcOptions.WRAPPER_PREFIX, null) in 
WrapStepProcessor got "baz" whereas it must be null.

Original issue reported on code.google.com by emmanuel...@gmail.com on 13 Jun 2011 at 8:44

GoogleCodeExporter commented 9 years ago

Original comment by emmanuel...@gmail.com on 13 Jun 2011 at 2:08

GoogleCodeExporter commented 9 years ago
I was having a closer look in order to merge your changes but I can't figure 
out something about this issue or its fix. The only commit related to it is the 
following one:

http://code.google.com/r/emmanueltourdot-tubular/source/detail?r=49c1a32dd82bed5
b2a38472e29151c905d279239

But it is a just a trace removal. Is there something missing from the commit?

Original comment by herve.qu...@gmail.com on 18 Jun 2011 at 2:08

GoogleCodeExporter commented 9 years ago
Okay, in fact it looks like the following commit is also related to this issue:

http://code.google.com/r/emmanueltourdot-tubular/source/detail?r=588c22b239de749
b09b26e97d68036a8e7748be9

It was incorrectly associated with the issue 34 which does not exist yet.

Original comment by herve.qu...@gmail.com on 18 Jun 2011 at 2:10