Re: Porblems with pxf:copy xmlcalabash-1.5.7-120

Dear Gerrit,

I already stumbled over your great code at transpect and it served as a affirmation how I could use pxf:copy. Thanks for the advice with the downgrade!



> On 18. Apr 2024, at 23:43, Imsieke, Gerrit, le-tex <gerrit.imsieke@le-tex.de> wrote:
> 
> Strange. We use pxf:copy with $target as a URI in some places (https://github.com/search?q=org%3Atranspect+pxf+copy&type=code), and XML Calabash treats it as a string or URI.
> 
> We recently reverted to 1.4.1 (with Saxon 10) from 1.5.7 due to another issue:
> java.lang.NullPointerException: Cannot invoke "java.io.Reader.read(char[], int, int)" because "reader" is null (I noticed it first at an internal project, but it was also reported publicly at https://github.com/transpect/docx2tex/issues/105#issuecomment-1948441663).
> 
> I don’t know whether pxf:copy would have failed with 1.5.7; it probably didn’t get that far due to the other error. It seems to work fine with 1.4.1.
> 
> And I didn’t report the issue since I didn’t want to stress Norm, and I didn’t have much time available to create a proper repro – on the other hand, the state that docx2tex was in when issue 105 was filed might serve as a repro, with any docx file.
> But when I discovered it in the first place, I also thought that it’s probably a Saxon issue which isn’t Norm’s business anyw – oh wait…
> 
> Gerrit
> 
> 
> 
> On 18.04.2024 18:05, Norm Tovey-Walsh wrote:
>>>  Hopefully more clarification will surface ;-)
>>> 
>>> Yes, I am sure if Norm finds the time to read this he will tell you whether it is a quirk in Calabash or whether the step needs some different usage than you have tried.
>> It appears to be a straight-up bug in the declaration for the step. Some sort of copy-and-paste error, I guess. The implementation expects the target to be a URI, not a boolean. But that doesn’t help because checking the types is based on the declarations, not the implementation.
>> What’s harder to explain is how this slipped past even a cursory amount of testing.
>> I’ll try to see if I can dig a bit deeper, but that may take a little more time. Things are … busy.
>>                                         Be seeing you,
>>                                           norm
>> --
>> Norm Tovey-Walsh <ndw@nwalsh.com>
>> https://norm.tovey-walsh.com/
>>> Happiness is nothing more than good health and a bad memory.--Albert
>>> Schweitzer
> 
> 

Received on Sunday, 21 April 2024 21:30:04 UTC