brettpoole / growl

Automatically exported from code.google.com/p/growl
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

growlNotificationTimedOut: sent to Growl delegate when unimplemented (2.0b3 SDK) #515

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Post a Growl notification that appears in Notification Center
2. Make sure growlNotificationTimedOut: is not implemented in the delegate
3. Error will appear in Console

What is the expected output?

Error does not appear. (method not called)

What do you see instead?

Crash occurs (method called)

What version of the product are you using? On what operating system?

2.0b3 SDK on 10.8.0

Please provide any additional information below.

Crash: http://pastebin.com/EHyCKRtg

Code called to post notification:

NSMutableDictionary * clickContext = [NSMutableDictionary dictionaryWithObject: 
GROWL_DOWNLOAD_COMPLETE forKey: @"Type"];

        if (location)
            [clickContext setObject: location forKey: @"Location"];

        [GrowlApplicationBridge notifyWithTitle: NSLocalizedString(@"Download Complete", "Growl notification title")
                                    description: [torrent name] notificationName: GROWL_DOWNLOAD_COMPLETE
                                    iconData: nil priority: 0 isSticky: NO clickContext: clickContext];

Original issue reported on code.google.com by livings...@gmail.com on 4 Aug 2012 at 9:44

GoogleCodeExporter commented 8 years ago
Thanks for catching this, it will be fixed =)

Original comment by dan...@growl.info on 5 Aug 2012 at 3:18

GoogleCodeExporter commented 8 years ago
Fixed in source as of [e3f30ffa0982] in Default, will either be in the next 
beta or the SDK release depending on if anything else crops up that needs more 
work than adding a respondsToSelector: to a couple calls

Original comment by dan...@growl.info on 5 Aug 2012 at 3:27

GoogleCodeExporter commented 8 years ago
Growl 2 is released, marking these as fixed.

Original comment by ch...@growl.info on 20 Sep 2012 at 3:16