#13 No notification sound on ios

Author: EddyVerbruggen

README.md

@@ -60,7 +60,7 @@ You can pass several options to this function, everything is optional:
     body: 'The body',
     ticker: 'The ticker',
     badge: 1,
-    sound: null, // suppress sound on Android
+    sound: null, // suppress the default sound
     at: new Date(new Date().getTime() + (10 * 1000)) // 10 seconds from now
   }]).then(
       function() {

local-notifications.android.js

@@ -31,12 +31,11 @@ LocalNotifications.schedule = function (arg) {
         options.icon = application.android.nativeApp.getApplicationInfo().icon;
         options.atTime = options.at ? options.at.getTime() : new Date().getTime();
 
-        // note that setting options.sound to explicitly to null will not add the default sound
-        if (options.sound === undefined) {
+        if (options.sound === undefined || options.sound === "default") {
           options.sound = android.media.RingtoneManager.getDefaultUri(android.media.RingtoneManager.TYPE_NOTIFICATION).toString();
         }
 
-        // TODO best move this to native lib so we can reuse it in the restorereceiver (dupl for now)
+        // TODO best move this to native lib so we can reuse it in the restorereceiver (dupe for now)
         var builder = new android.support.v4.app.NotificationCompat.Builder(context)
             .setDefaults(0)
             .setContentTitle(options.title)

local-notifications.ios.js

@@ -107,9 +107,13 @@ LocalNotifications._schedulePendingNotifications = function () {
     userInfoDict.setObjectForKey(options.body, "body");
     notification.userInfo = userInfoDict;
 
+    if (options.sound === undefined || options.sound === "default") {
+      notification.soundName = UILocalNotificationDefaultSoundName;
+    }
+
     // TODO add these after v1
     // notification.repeatInterval = 1;
-    // notification.soundName = null;
+    // notification.soundName = custom..;
     // notification.resumeApplicationInBackground = true;
 
     console.log("--- scheduling " + notification);

package.json

@@ -1,6 +1,6 @@
 {
   "name": "nativescript-local-notifications",
-  "version": "1.0.2",
+  "version": "1.0.3",
   "description": "The Local Notifications plugin allows your app to show notifications when the app is not running. Just like remote push notifications, but a few orders of magnitude easier to set up.",
   "main" : "local-notifications.js",
   "nativescript": {