backslashes doubled

[bill]

In files like _RichText.js, the style correction will convert \ into
,
into

, etc.

[Shane O'Sullivan]

I can not replicate this issue on the latest code. Can you please retest, and close this ticket if it is no longer a problem?

[bill]

I just grabbed the latest code. Here's what I did:

1. run "sh checkstyle.sh"
2. open checkstyleReport.html
3. click dijit/_editor in left hand column
4. click first "Richtext.js" entry in right grid
5. click "make fixes" button
6. click "save" button

Now comparing _Richtext.js to _Richtext.js.checkstyle.js:

old:

"\\": exec("insertunorderedlist")

vs. new:

"\\\\": exec("insertunorderedlist")

And so on.

This was running on mac, and using safari, although I doubt that makes a difference.

This was running against RichText.js [20573], I'm about to check in some fixes to that file that may affect the results so you should try running against that version.

[Shane O'Sullivan]

(In [20595]) Refs #10093 Fixes the issue where the Checkstyle tool failed to correctly handle backslashes in front of quotes and double quotes. This was caused by the PHP file used to save modified JavaScript? files, which had code that manipulated the file contents to remove unnecessary backslashes. The solution is to use the stripslashes() PHP function instead.