Merge pull request #58 from ickata/master

Ability to set custom notification sound

Author: EddyVerbruggen

README.md

@@ -48,7 +48,7 @@ You can pass several options to this function, everything is optional:
 |`ticker` |On Android you can show a different text in the statusbar, instead of the `body`. Default not set, so `body` is used.|
 |`at`     |A JavaScript Date object indicating when the notification should be shown. Default 'now'.|
 |`badge`  |On iOS (and some Android devices) you see a number on top of the app icon. On most Android devices you'll see this number in the notification center. Default not set (0).|
-|`sound`  |Set this to `null` to suppress the default sound.|
+|`sound`  |Notification sound. For custom notification sound, copy sound file in ```App_Resources/iOS``` (iOS) and ```App_Resources/Android/raw``` (Android). Set this to "default" (or do not set at all) in order to use default OS sound. Set this to `null` to suppress sound.|
 |`interval` |Set to one of `second minute hour day week month quarter year` if you want a recurring notification.|
 |`smallIcon` |On Android you can set a custom icon in the system tray. Pass in 'res://filename' (without the extension) which lives in App_Resouces/Android/drawable folders. If not passed, we look for a file named 'ic_stat_notify.png' in the App_Resources/Android/drawable folders. Default: the app icon.|
 |`largeIcon` |Same as `smallIcon`, but this one is shown when you expand the notification center. The optional file we look for is not 'ic_stat_notify.png' but 'ic_notify.png'.|
@@ -69,7 +69,7 @@ Note that after a reboot the `smallIcon` and `largeIcon` are not restored but fa
     ongoing: true, // makes the notification ongoing (Android only)
     smallIcon: 'res://heart',
     interval: 'minute',
-    sound: null, // suppress the default sound
+    sound: require("application").ios ? "customsound-ios.wav" : "customsound-android", // can be also `null`, "default"
     at: new Date(new Date().getTime() + (10 * 1000)) // 10 seconds from now
   }]).then(
       function() {
@@ -195,5 +195,4 @@ If the `requestPermission` or `schedule` functions previously ran you may want t
 Let us know what you need by opening a Github issue.
 
 We're thinking about adding support for things like:
-- [Custom Notification sounds](https://github.com/EddyVerbruggen/nativescript-local-notifications/issues/3)
 - Interactive Notifications on iOS

local-notifications.android.js

@@ -65,8 +65,14 @@ LocalNotifications.schedule = function (arg) {
 
         options.atTime = options.at ? options.at.getTime() : new Date().getTime();
 
-        if (options.sound === undefined || options.sound === "default") {
-          options.sound = android.media.RingtoneManager.getDefaultUri(android.media.RingtoneManager.TYPE_NOTIFICATION).toString();
+        switch (options.sound) {
+          case false:
+            options.sound = null;
+            break;
+          case undefined:
+          case "default":
+            options.sound = android.media.RingtoneManager.getDefaultUri(android.media.RingtoneManager.TYPE_NOTIFICATION).toString();
+            break;
         }
 
         // TODO best move this to native lib so we can reuse it in the restorereceiver (dupe for now)
@@ -272,7 +278,7 @@ LocalNotifications.requestPermission = function (arg) {
 };
 
 LocalNotifications._getSharedPreferences = function () {
-  var PREF_KEY = com.telerik.localnotifications.NotificationRestoreReceiver.SHARED_PREFERENCES_KEY;
+  var PREF_KEY = "LocalNotificationsPlugin"; // TODO: For some reason this is `null` and causes Java error...
   return context.getSharedPreferences(PREF_KEY, android.content.Context.MODE_PRIVATE);
 };
 

local-notifications.ios.js

@@ -114,8 +114,17 @@ LocalNotifications._schedulePendingNotifications = function () {
     userInfoDict.setObjectForKey(options.interval, "interval");
     notification.userInfo = userInfoDict;
 
-    if (options.sound === undefined || options.sound === "default") {
-      notification.soundName = UILocalNotificationDefaultSoundName;
+    switch (options.sound) {
+      case null:
+      case false:
+        break;
+      case undefined:
+      case "default":
+        notification.soundName = UILocalNotificationDefaultSoundName;
+        break;
+      default:
+        notification.soundName = options.sound;
+        break;
     }
 
     if (options.interval !== 0) {

native-src/android/src/com/telerik/localnotifications/NotificationPublisher.java

@@ -5,15 +5,20 @@
 import android.content.BroadcastReceiver;
 import android.content.Context;
 import android.content.Intent;
+import android.net.Uri;
+import static android.R.attr.data;
 
 public class NotificationPublisher extends BroadcastReceiver {
 
   public static String NOTIFICATION_ID = "notification-id";
   public static String NOTIFICATION = "notification";
+  public static String SOUND = "sound";
 
   public void onReceive(Context context, Intent intent) {
     final Notification notification = intent.getParcelableExtra(NOTIFICATION);
+    Uri d_sound = Uri.parse((String)("android.resource://" + context.getPackageName() + "/raw/" + notification.sound));
+    notification.sound = d_sound;
     final int id = intent.getIntExtra(NOTIFICATION_ID, 0);
     ((NotificationManager)context.getSystemService(Context.NOTIFICATION_SERVICE)).notify(id, notification);
   }
-}
+}

native-src/android/src/com/telerik/localnotifications/NotificationRestoreReceiver.java

@@ -49,7 +49,7 @@ public void onReceive(Context context, Intent intent) {
             .setContentText(options.optString("body"))
             .setSmallIcon(options.optInt("icon"))
             .setAutoCancel(true)
-            .setSound(options.has("sound") ? Uri.parse(options.getString("sound")) : null)
+            .setSound(options.has("sound") ? Uri.parse((String)("android.resource://" + context.getPackageName() + "/raw/" + options.optString("sound"))) : Uri.parse((String)("android.resource://" + context.getPackageName() + "/raw/notify")) )
             .setNumber(options.optInt("badge"))
             .setTicker(options.optString("ticker"));
 
@@ -67,6 +67,7 @@ public void onReceive(Context context, Intent intent) {
         final Intent notificationIntent = new Intent(context, NotificationPublisher.class)
             .setAction(options.getString("id"))
             .putExtra(com.telerik.localnotifications.NotificationPublisher.NOTIFICATION_ID, options.optInt("id"))
+            .putExtra(NotificationPublisher.SOUND, options.optString("sound"))
             .putExtra(com.telerik.localnotifications.NotificationPublisher.NOTIFICATION, notification);
 
         final PendingIntent pendingIntent = PendingIntent.getBroadcast(context, options.optInt("id")+200, notificationIntent, PendingIntent.FLAG_CANCEL_CURRENT);