On 16 Jul 2008, at 12:54 AM, Maxwell, Adam R wrote:
> Nice...that's easily reproducible, and it looks like an Apple bug.
> You can
> overwrite the citekey text with any invalid character (e.g. {) and
> reproduce
> the problem, also, so it's not limited to spaces. I wonder how long
> this
> has been happening?
>
> If you file a bug report, it might be worth noting that it's feeding
> an
> NSRange of { 0, NSNotFound } to -[NSTextStorage
> _undoRedoAttributedSubstringFromRange:].
Definitely an Apple bug. I can reproduce it in an almost trivial test
program (only thing needed is a formatter that modifies the edited
string, and that's explicitly allowed by the API). Just filed a bug
report (rdar://problem/6078086 if you want to refer to it).
BTW, even without the recent changes to the formatters I can reproduce
this bug. So as this bug wasn't reported before (and it should have
been there already for a very long time), I strongly suspect a recent
change in AppKit (probably in 10.5.4).
Christiaan
> On 07/15/08 15:39, "Christiaan Hofman" <[EMAIL PROTECTED]> wrote:
>
>> Thanks Adam, I see the problem now (the fact that you hit Space
>> instead of Delete, as you said earlier, is crucial). It seems to be
>> partly bug in the built-in undo for text cells (as that's where it
>> takes place). But anyway, I found a fix.
>>
>> Christiaan
>>
>> On 15 Jul 2008, at 10:44 PM, Adam M. Goldstein wrote:
>>
>>> 1. Open a get info window for a pub that has something in its
>>> attached
>>> files pane.
>>>
>>> 2. Select one of the files by clicking on it once.
>>>
>>> 3. Select the cite key by double-clicking on it.
>>>
>>> 4. Delete the cite key by pushing "space"---thinking you were
>>> going to
>>> quick-look the selected attached file.
>>>
>>> 5. Realize your mistake and cmd-z to undo. continue until error 1
>>> appears. Eventually its dialogue is dismissed and the message about
>>> not allowing blank cite keys appears.
>>>
>>> 6. keep trying to undo. the second error appears eventually.
>>>
>>> 7. To get the cite key, after all dialogues are dismissed, use the
>>> work menu (gear) at the bottom left of the record.
>>>
>>> This seems to work if there are no attached files, just by
>>> clicking in
>>> the empty pane.
>>>
>>> I can reproduce the errors at will, even after quitting BD and
>>> restarting it again.
>>>
>>> -Adam
>>>
>>> On Jul 15, 2008, at 4:29 PM, Christiaan Hofman wrote:
>>>
>>>> Do you have exact steps to reproduce? I don't see any problem,
>>>> and it
>>>> is even impossible to delete a cite key.
>>>>
>>>> Christiaan
>>>>
>>>> On 15 Jul 2008, at 10:21 PM, Adam M. Goldstein wrote:
>>>>
>>>>> undo: NSCellUndoManager 0x16d95c20 is in invalid state, undo was
>>>>> called with too many nested undo groups
>>>>>
>>>>> This is the error I got after I accidentally deleted a cite key
>>>>> while
>>>>> in "Get info" mode, and trying to undo.
>>>>>
>>>>> I had got another error but accidentally copied over it with this
>>>>> one
>>>>> so can't paste it here.
>>>>>
>>>>> BD didn't crash, though.
>>>>>
>>>>> Using Version 1.3.17 (1188)
>>>>>
>>>>> -Adam
>>>>>
>>>>> Console log with more information:
>>>>>
>>>>> 7/15/08 4:11:24 PM BibDesk[1444] *** -[NSBigMutableString
>>>>> substringWithRange:]: Range or index out of bounds
>>>>> 7/15/08 4:11:25 PM BibDesk[1444] __raiseError (in CoreFoundation)
>>>>> objc_exception_throw (in libobjc.A.dylib)
>>>>> +[NSException raise:format:arguments:] (in CoreFoundation)
>>>>> +[NSException raise:format:] (in CoreFoundation)
>>>>> -[NSString substringWithRange:] (in Foundation)
>>>>> -[NSTextStorage(NSUndo) _undoRedoAttributedSubstringFromRange:]
>>>>> (in
>>>>> AppKit)
>>>>> -[NSUndoTyping undoRedo:] (in AppKit)
>>>>> -[_NSUndoLightInvocation invoke] (in Foundation)
>>>>> -[_NSUndoStack popAndInvoke] (in Foundation)
>>>>> -[NSUndoManager undoNestedGroup] (in Foundation)
>>>>> -[NSCellUndoManager undo] (in AppKit)
>>>>> -[NSApplication sendAction:to:from:] (in AppKit)
>>>>> -[BDSKApplication sendAction:to:from:] (in BibDesk)
>>>>> -[NSMenu performActionForItemAtIndex:] (in AppKit)
>>>>> -[NSCarbonMenuImpl performActionWithHighlightingForItemAtIndex:]
>>>>> (in
>>>>> AppKit)
>>>>> -[NSMenu performKeyEquivalent:] (in AppKit)
>>>>> -[NSApplication _handleKeyEquivalent:] (in AppKit)
>>>>> -[NSApplication sendEvent:] (in AppKit)
>>>>> -[OAApplication sendEvent:] (in OmniAppKit)
>>>>> -[NSApplication run] (in AppKit)
>>>>> -[OAApplication run] (in OmniAppKit)
>>>>> NSApplicationMain (in AppKit)
>>>>> main (in BibDesk)
>>>>> _start (in BibDesk)
>>>>> start (in BibDesk)
>>>>>
>>>>> 7/15/08 4:11:38 PM BibDesk[1444] undo: NSCellUndoManager
>>>>> 0x16d95c20
>>>>> is
>>>>> in invalid state, undo was called with too many nested undo groups
>>>>> 7/15/08 4:11:39 PM BibDesk[1444] __raiseError (in CoreFoundation)
>>>>> objc_exception_throw (in libobjc.A.dylib)
>>>>> +[NSException raise:format:arguments:] (in CoreFoundation)
>>>>> +[NSException raise:format:] (in CoreFoundation)
>>>>> -[NSUndoManager undo] (in Foundation)
>>>>> -[NSCellUndoManager undo] (in AppKit)
>>>>> -[NSApplication sendAction:to:from:] (in AppKit)
>>>>> -[BDSKApplication sendAction:to:from:] (in BibDesk)
>>>>> -[NSMenu performActionForItemAtIndex:] (in AppKit)
>>>>> -[NSCarbonMenuImpl performActionWithHighlightingForItemAtIndex:]
>>>>> (in
>>>>> AppKit)
>>>>> -[NSMenu performKeyEquivalent:] (in AppKit)
>>>>> -[NSApplication _handleKeyEquivalent:] (in AppKit)
>>>>> -[NSApplication sendEvent:] (in AppKit)
>>>>> -[OAApplication sendEvent:] (in OmniAppKit)
>>>>> -[NSApplication run] (in AppKit)
>>>>> -[OAApplication run] (in OmniAppKit)
>>>>> NSApplicationMain (in AppKit)
>>>>> main (in BibDesk)
>>>>> _start (in BibDesk)
>>>>> start (in BibDesk)
>>>>>
>>>>> =================================
>>>>> Adam M. Goldstein PhD MSLIS
>>>>> Assistant Professor of Philosophy
>>>>> Iona College
>>>>> --
>>>>> email 1 [EMAIL PROTECTED]
>>>>> email 2 [EMAIL PROTECTED]
>>>>> web http://www.iona.edu/faculty/agoldstein/
>>>>> tel (914) 637-2717
>>>>> post Iona College
>>>>> Department of Philosophy
>>>>> 715 North Avenue
>>>>> New Rochelle, NY 10801
>>>>>
>>>>>
>>>>>
>>>>>
>>>>> =================================
>>>>> Adam M. Goldstein PhD MSLIS
>>>>> Assistant Professor of Philosophy
>>>>> Iona College
>>>>> --
>>>>> email 1 [EMAIL PROTECTED]
>>>>> email 2 [EMAIL PROTECTED]
>>>>> web http://www.iona.edu/faculty/agoldstein/
>>>>> tel (914) 637-2717
>>>>> post Iona College
>>>>> Department of Philosophy
>>>>> 715 North Avenue
>>>>> New Rochelle, NY 10801
>>>>>
>>>>>
>>>>> -------------------------------------------------------------------------
>>>>> This SF.Net email is sponsored by the Moblin Your Move Developer's
>>>>> challenge
>>>>> Build the coolest Linux based applications with Moblin SDK & win
>>>>> great prizes
>>>>> Grand prize is a trip for two to an Open Source event anywhere in
>>>>> the world
>>>>> http://moblin-contest.org/redirect.php?banner_id=100&url=/
>>>>> _______________________________________________
>>>>> Bibdesk-users mailing list
>>>>> [email protected]
>>>>> https://lists.sourceforge.net/lists/listinfo/bibdesk-users
>>>>
>>>>
>>>> -------------------------------------------------------------------------
>>>> This SF.Net email is sponsored by the Moblin Your Move Developer's
>>>> challenge
>>>> Build the coolest Linux based applications with Moblin SDK & win
>>>> great prizes
>>>> Grand prize is a trip for two to an Open Source event anywhere in
>>>> the world
>>>> http://moblin-contest.org/redirect.php?banner_id=100&url=/
>>>> _______________________________________________
>>>> Bibdesk-users mailing list
>>>> [email protected]
>>>> https://lists.sourceforge.net/lists/listinfo/bibdesk-users
>>>
>>>
>>> =================================
>>> Adam M. Goldstein PhD MSLIS
>>> Assistant Professor of Philosophy
>>> Iona College
>>> --
>>> email 1 [EMAIL PROTECTED]
>>> email 2 [EMAIL PROTECTED]
>>> web http://www.iona.edu/faculty/agoldstein/
>>> tel (914) 637-2717
>>> post Iona College
>>> Department of Philosophy
>>> 715 North Avenue
>>> New Rochelle, NY 10801
>>>
>>>
>>> -------------------------------------------------------------------------
>>> This SF.Net email is sponsored by the Moblin Your Move Developer's
>>> challenge
>>> Build the coolest Linux based applications with Moblin SDK & win
>>> great prizes
>>> Grand prize is a trip for two to an Open Source event anywhere in
>>> the world
>>> http://moblin-contest.org/redirect.php?banner_id=100&url=/
>>> _______________________________________________
>>> Bibdesk-users mailing list
>>> [email protected]
>>> https://lists.sourceforge.net/lists/listinfo/bibdesk-users
>>
>>
>> -------------------------------------------------------------------------
>> This SF.Net email is sponsored by the Moblin Your Move Developer's
>> challenge
>> Build the coolest Linux based applications with Moblin SDK & win
>> great prizes
>> Grand prize is a trip for two to an Open Source event anywhere in
>> the world
>> http://moblin-contest.org/redirect.php?banner_id=100&url=/
>> _______________________________________________
>> Bibdesk-users mailing list
>> [email protected]
>> https://lists.sourceforge.net/lists/listinfo/bibdesk-users
>
>
> -------------------------------------------------------------------------
> This SF.Net email is sponsored by the Moblin Your Move Developer's
> challenge
> Build the coolest Linux based applications with Moblin SDK & win
> great prizes
> Grand prize is a trip for two to an Open Source event anywhere in
> the world
> http://moblin-contest.org/redirect.php?banner_id=100&url=/
> _______________________________________________
> Bibdesk-users mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/bibdesk-users
-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Bibdesk-users mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/bibdesk-users
