724204 - "visit count" bookmark AppleScript property broken in 2.1

Status: RESOLVED FIXED

(Camino Graveyard :: OS Integration, defect)

Description

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

Attachment: Update sdef to match code

When Stuart unified the Bookmarks and History visit count interfaces in bug 642427, he forgot to change the cocoaKey element in the .sdef to match.

This currently causes an exception when the property is accessed:

Warning An exception was thrown during execution of an NSScriptCommand... 
Warning [<Bookmark 0x13e1dbe0> valueForUndefinedKey:]: this class is not key value coding-compliant for the key numberOfVisits.

Luckily, this is a true 1-liner to fix!  We should take it in 2.1.1.

(:sigh: Bug 453202 :sigh:  Sadly, this is twice now this has been broken, though the last time--bug 512628--wasn't really our fault.)

(Lisa, if anyone's complained about your "Show Bookmark Usage" script not working in 2.1, this is why :P )

Comment 1

[Stuart Morgan]

Comment on attachment 594391 [details] [diff] [review]
Update sdef to match code

Sorry :(

sr=smorgan

Comment 2

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

http://hg.mozilla.org/camino/rev/301fe151ff94