Bug: 'Topic Save Error' if double quotes in attach comment field
If a user puts double quotes in the comment field when attaching a file to a topic the following error is generated:
Topic save error
During save of file Test.AttachError an error was found by the version control system. Please notify your TWiki administrator.
/usr/bin/ci -x,v -q -u -m"halfsize image of the "this close" boundary." -t-none -w"MattWilkie" /www/twiki/pub/Test/AttachError/image.png 2>&1 1>NUL
ci: close boundary.: No such file or directory
ci: -t-none option is ignored after pathnames
ci: -wMattWilkie option is ignored after pathnames
Go back in your browser and save your changes locally.
Test case
Environment
| TWiki version: |
2001 Dec 01 |
| TWiki plugins: |
|
| Server OS: |
Win2k (cygwin) |
| Web server: |
apache 1.3.22 |
| Perl version: |
|
| Client OS: |
Win2k |
| Web Browser: |
Mozilla 1.1b |
--
MattWilkie - 06 Aug 2002
Follow up
A work-around is to use single quotes instead:
halfsize image of the 'this close' boundary
TWiki.org does not have this problem (see attached image comment below)
--
MattWilkie - 06 Aug 2002
Fix record
TWiki.org runs a relatively recent
TWikiAlphaRelease, typically, often a
TWikiBetaRelease - so this may already be fixed. Try downloading the beta to see if it fixes this problem, but it may be Windows specific due to the use of different quote characters. You might also try tinkering with the quote char variable in TWiki.cfg, but the real solution is to escape such characters or perhaps drive
ci via a pipe into stdin.
--
RichardDonkin - 07 Aug 2002
If you are using
ActivePerl and
CygWin (you didn't specify this above), you can also have a look at setting
bash as the shell, by using
CookbookActivePerlSetup - there's a proposed fix for $quoteCmd that would help too. Let me know if this is still an issue with the alpha or beta, otherwise I'll close this.
--
RichardDonkin - 17 Oct 2002
I've since moved this twiki to Debian GNU/Linux so I can't test your suggestions. For the record though, it was a pure
CygWin install (ie: used
CygWin perl).
--
MattWilkie - 18 Oct 2002
Works fine for me, so closing (fixed in the interim, I guess).
--
CrawfordCurrie - 21 Jul 2004