Refs #264 - Provide environment information when opening newtab notifications

Author: ThomasGreiner

lib/notificationHelper.js

@@ -36,6 +36,16 @@ const displayMethods = new Map([
 ]);
 const defaultDisplayMethods = ["popup"];
 
+const linkPlaceholders = new Set([
+  ["LANG", () => browser.i18n.getUILanguage().replace("-", "_")],
+  ["ADDON_NAME", () => info.addonName],
+  ["ADDON_VERSION", () => info.addonVersion],
+  ["APPLICATION_NAME", () => info.application],
+  ["APPLICATION_VERSION", () => info.applicationVersion],
+  ["PLATFORM_NAME", () => info.platform],
+  ["PLATFORM_VERSION", () => info.platformVersion]
+]);
+
 // We must hard code any "abp:" prefixed notification links here, otherwise
 // notifications linking to them will not be displayed at all.
 const localNotificationPages = new Map([
@@ -224,10 +234,14 @@ function notificationDismissed(notificationId, stashedInNotificationCenter)
 
 function openNotificationInNewTab(notification)
 {
+  let [url] = notification.links;
+  for (let [key, getValue] of linkPlaceholders)
+    url = url.replace(`%${key}%`, encodeURIComponent(getValue()));
+
   let onTabCreated = () =>
   {
     browser.tabs.onCreated.removeListener(onTabCreated);
-    browser.tabs.create({url: notification.links[0]});
+    browser.tabs.create({url});
     notifications.markAsShown(notification.id);
   };