bugGNUstep - Bugs: bug #26101, NSBeginAlertSheet() doesn't call...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #26101: NSBeginAlertSheet() doesn't call the didEndSelector

Submitted by:  Doug Simons <theeggcamefirst>
Submitted on:  Tue 07 Apr 2009 04:04:36 AM UTC  
 
Category: Gui/AppKitSeverity: 3 - Normal
Item Group: BugStatus: Fixed
Privacy: PublicAssigned to: Fred Kiefer <FredKiefer>
Open/Closed: Closed

Wed 08 Apr 2009 06:55:41 AM UTC, comment #3:

Reported as being fixed.

Fred Kiefer <FredKiefer>
Project MemberIn charge of this item.
Tue 07 Apr 2009 11:10:15 PM UTC, comment #2:

Yeah, thank Apple for changing the parameter names. Anyway, I thought you might want to change your parameters to match Apple's current naming.

In any case, the changes you made work perfectly for me. Thanks!

Doug Simons <theeggcamefirst>
Tue 07 Apr 2009 08:56:38 PM UTC, comment #1:

The name of the delegate methods may be a bit misleading, but it is what Apple called them in 10.1. We just sticked to that.

As for the missing calls, I added them to SVN code, please give it a try and report back whether this works now.

Fred Kiefer <FredKiefer>
Project MemberIn charge of this item.
Tue 07 Apr 2009 04:04:36 AM UTC, original submission:

The NSBeginAlertSheet() function accepts two call-back selectors that Apple calls the didEndSelector and the didDismissSelector. It appears that GNUstep refers to them as the willEndSelector and didEndSelector instead, which is confusing, and ignores the first of them.

It appears to me that renaming the parameters to the function to match Apple's docs and then calling the didDismissSelector (if one is given) after the sheet/panel has been closed would fix the problem and provide compatibility with Cocoa.

Doug Simons <theeggcamefirst>

 

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by FredKiefer (Posted a comment)
  • -unavailable- added by theeggcamefirst (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 5 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Wed 08 Apr 2009 06:55:41 AM UTCFredKieferStatusReady For Test=>Fixed
      Open/ClosedIn Test=>Closed
    Tue 07 Apr 2009 08:56:38 PM UTCFredKieferStatusNone=>Ready For Test
      Assigned toNone=>FredKiefer
      Open/ClosedOpen=>In Test

    Back to the top


    Powered by Savane 3.1-cleanup1