Opened 17 years ago

Closed 17 years ago

#10 closed defect (fixed)

Newview doesn't pass quoted parameters to launched executables correctly

Reported by: cla Owned by: RBRi
Priority: major Milestone: Version 2.19
Component: NewView Version: 2.18
Keywords: link double quote launch Cc:

Description

For links with reftype=launch the parameters within the attribute data are not handled correctly, if parameters are enclosed in double quotes.

I include a testcase to demo the filename case. Note that the IBM viewer does handle double quotes correctly. See further explanation in the testcase panel itself.

The testcase is compiled with HyperText?/2, you may want to download htex107.zip from hobbes for evaluation.

I would greatly appreciate if you could pass me an email to cla@… when the error is about to be fixed. I could implement a workaround in HyperText?/2, but that would not help for lots of old INFs being already around (BTW, this is why I would consider this being important, so the major prio). And it would not solve the basic problem, also affecting filenames and such.

Attachments (2)

testcase.zip (2.6 KB) - added by cla 17 years ago.
doublequotes.zip (2.7 KB) - added by Andreas Schnellbacher 17 years ago.
Testcases for double quotes in links and in the commandline

Download all attachments as: .zip

Change History (5)

Changed 17 years ago by cla

Attachment: testcase.zip added

comment:1 Changed 17 years ago by Andreas Schnellbacher

Keywords: link double quote launch added

The issue described in testcase.zip works for me with 2.19.b2. With 2.19.b3 all external links make NewView crash, either on displaying the panel containing it or on selecting such a link, see also ticket:8.

Apparently 2.19.b3 fixes the issues with double quotes in links (viewing the status bar text when hovering about a link). But there were some other issues added: additional ampersands and doubled backslashes. See my following testcase (doublequotes.zip, containing doublequotes.ipf and doublequotes.inf).

On the second page of it, I listed the results when calling NewView with search strings from CMD.EXE. Ok, that issue is about commandline parsing, but I found it more convenient to have both testcases in one file. The testcase shows, that 2.19.b2 and 2.19.b3 handle strings well. Thanks for fixing that! It works now exactly as IBMVIEW.EXE. There remain issues with specifying a multi-word string for NewView's /s parameter: It works with tripled double quotes, but that's a bit unusual and should at least be documented.

Changed 17 years ago by Andreas Schnellbacher

Attachment: doublequotes.zip added

Testcases for double quotes in links and in the commandline

comment:2 Changed 17 years ago by anonymous

Milestone: Version 2.19

comment:3 Changed 17 years ago by RBRi

Resolution: fixed
Status: newclosed

fixed in beta4

Note: See TracTickets for help on using tickets.