Opened 18 years ago
Closed 18 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)
Change History (5)
by , 18 years ago
| Attachment: | testcase.zip added |
|---|
comment:1 by , 18 years ago
| Keywords: | link double quote launch added |
|---|
by , 18 years ago
| Attachment: | doublequotes.zip added |
|---|
Testcases for double quotes in links and in the commandline
comment:2 by , 18 years ago
| Milestone: | → Version 2.19 |
|---|
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.