430999 - Console log error message: .sdef error: Operation could not be completed. (NSXMLParserErrorDomain error 27.)

Status: VERIFIED FIXED

(Camino Graveyard :: General, defect)

Description

[philippe (part-time)]

With the 2.0a1pre (1.9pre 2008042601) build
4/27/08 9:09:34 AM Camino[26806] ChimericalConsole loaded 
4/27/08 9:34:32 AM Camino[26806] .sdef error: Operation could not be completed. (NSXMLParserErrorDomain error 27.) 
4/27/08 9:34:32 AM Camino[26806] line number: 307 
4/27/08 9:34:32 AM Camino[26806] .sdef error: Operation could not be completed. (NSXMLParserErrorDomain error 27.) 
4/27/08 9:34:32 AM Camino[26806] line number: 307 

(seen also with a home made build, cvs checkout 20080425 ~19:00 PDT)

I suspect this started after bug 403346 landed. Not bad things happened so far

Comment 1

[philippe (part-time)]

PS - I forgot to mention:
> 4/27/08 9:09:34 AM Camino[26806] ChimericalConsole loaded 

is unrelated and not an error, I included it to note the start up time of Camino.

Comment 2

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Attachment: don't use &[lr]dquo;

Apparently I didn't watch the console.log when I tested after switching ' to the double-quote entities.

' is already in the file, so use it instead.

Comment 3

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Attachment: Use " instead

Except they're not really apostrophes, so use " instead.  It also seems free of console spew.

Comment 4

[Stuart Morgan]

Comment on attachment 317951 [details] [diff] [review]
Use " instead

sr=smorgan

Comment 5

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

I checked this in on the branch as well as the trunk since even though the plist parser is OK with those entities (or they're stripped in conversion to .script* files), they're clearly wrong in an sdef file.  In addition, at least on 10.5, the .script* files are converted back to an sdef by ScriptEditor.

Comment 6

[philippe (part-time)]

v. with 2.0a1pre (1.9pre 2008042700)
Thanks.

[one way to repro this was: click on a link in an email client (mail.app)].