From 8f3b159f96cd07bb36011159a644dfb36d80fd96 Mon Sep 17 00:00:00 2001
From: jendas <jendas1@yahoo.com>
Date: Mon, 27 Jul 2015 14:46:41 +0200
Subject: [PATCH] Fixed compilation on mac + added possibility to display os x
 notification without bundle identifier (terminal programs)

---
 .../osxnotificationcenter.mm                  | 53 ++++++++++++++++---
 1 file changed, 46 insertions(+), 7 deletions(-)

diff --git a/src/plugins/backends/osxnotificationcenter/osxnotificationcenter.mm b/src/plugins/backends/osxnotificationcenter/osxnotificationcenter.mm
index f9cc6bd..1d70017 100644
--- a/src/plugins/backends/osxnotificationcenter/osxnotificationcenter.mm
+++ b/src/plugins/backends/osxnotificationcenter/osxnotificationcenter.mm
@@ -4,12 +4,12 @@
 #include "libsnore/utils.h"
 #include "libsnore/snore.h"
 #include "libsnore/log.h"
-
+#include <QDebug.h>
 #import <QThread.h>
 #import <QApplication.h>
 #import <QMap>
 #include <Foundation/Foundation.h>
-
+#import <objc/runtime.h>
 using namespace Snore;
 
 QMap<int, Notification> m_IdToNotification;
@@ -20,6 +20,36 @@ void emitNotificationClicked(Notification notification) {
     emit SnoreCore::instance().actionInvoked(notification);
 }
 
+//
+// Overcome need for bundle identifier to display notification
+//
+
+#pragma mark - Swizzle NSBundle
+
+@implementation NSBundle(swizle)
+// Overriding bundleIdentifier works, but overriding NSUserNotificationAlertStyle does not work.
+- (NSString *)__bundleIdentifier
+{
+    if (self == [NSBundle mainBundle] && ![[self __bundleIdentifier] length]) {
+        return @"com.apple.terminal";
+    } else {
+        return [self __bundleIdentifier];
+    }
+}
+
+@end
+BOOL installNSBundleHook()
+{
+    Class cls = objc_getClass("NSBundle");
+    if (cls) {
+        method_exchangeImplementations(class_getInstanceMethod(cls, @selector(bundleIdentifier)),
+                                       class_getInstanceMethod(cls, @selector(__bundleIdentifier)));
+        return YES;
+    }
+    return NO;
+}
+
+
 //
 // Enable reaction when user clicks on NSUserNotification
 //
@@ -70,20 +100,29 @@ public:
 };
 
 
+
+
 // store some variables that are needed (since obj-c++ does not allow having obj-c classes as c++ members)
 namespace {
     
     NSString * NSStringFromQString(QString qstr) {
-        return [NSString stringWithUTF8String: qstr.toUtf8().constData()];
+        return qstr.toNSString();
+    }
+    QString QStringFromNSString(NSString * str) {
+        return QString::fromNSString(str);
     }
 }
 
-UserNotificationItemClass * delegate;
+static UserNotificationItemClass * delegate = 0;
 
 OSXNotificationCenter::OSXNotificationCenter()
 {
-    m_IdToNSNotification = [[[NSMutableDictionary alloc] init] autorelease];
-    delegate = new UserNotificationItemClass();
+    installNSBundleHook();
+    m_IdToNSNotification = [[NSMutableDictionary alloc] init];
+    if (not delegate) {
+        delegate = new UserNotificationItemClass();
+    }
+    
 }
 
 OSXNotificationCenter::~OSXNotificationCenter()
@@ -95,7 +134,7 @@ void OSXNotificationCenter::slotNotify(Snore::Notification notification)
 {
     NSUserNotification * osxNotification = [[[NSUserNotification alloc] init] autorelease];
     NSString * notificationId = [NSString stringWithFormat:@"%d",notification.id()];
-    osxNotification.title = NSStringFromQString(notification.title());
+    osxNotification.title = notification.title().toNSString();
     osxNotification.userInfo = [NSDictionary dictionaryWithObjectsAndKeys:notificationId, @"id", nil];
     osxNotification.informativeText = NSStringFromQString(notification.text());