Correction for problem found by Christopher Culver. The code which was

closing the popup was also calling editedObject uncondition on the selection
owner which may or may not be an editor.


git-svn-id: svn+ssh://svn.gna.org/svn/gnustep/apps/gorm/trunk@19848 72102866-910b-0410-8b05-ffd578937521
This commit is contained in:
Gregory John Casamento 2004-08-08 20:14:31 +00:00
parent 074d1c7493
commit 6b57e185f3
2 changed files with 18 additions and 2 deletions

View file

@ -1,3 +1,10 @@
2004-08-08 16:43 Gregory John Casamento <greg_casamento@yahoo.com>
* Palettes/0Menus/GormNSMenu.m: Changed the code in
handleNotification to not call edited object blindly.
It now checks to make sure the object responds to the
message before calling.
2004-08-08 13:43 Gregory John Casamento <greg_casamento@yahoo.com>
* Palettes/1Windows/GormNSWindow.m: Removed some debugging

View file

@ -99,9 +99,18 @@
id<IBEditors> object = [notification object];
if(object != nil)
{
id edited = [object editedObject];
if(self != edited && [self _ownedByPopUp])
// don't call, unless it does respond...
if([(id)object respondsToSelector: @selector(editedObject)])
{
id edited = [object editedObject];
if(self != edited && [self _ownedByPopUp])
{
[self close];
}
}
else
{
// Close anyway if the editor doesn't respond.
[self close];
}
}