Remove lint warnings
authorKevin Sawicki <kevinsawicki@gmail.com>
Wed, 8 Jun 2016 00:00:53 +0000 (17:00 -0700)
committerKevin Sawicki <kevinsawicki@gmail.com>
Thu, 9 Jun 2016 20:57:08 +0000 (13:57 -0700)
lib/renderer/chrome-api.js
lib/renderer/extensions/i18n.js

index b22fe74..4a3556c 100644 (file)
@@ -200,5 +200,5 @@ exports.injectTo = function (extensionId, isBackgroundPage, context) {
     getPopup () {}
   }
 
-  chrome.i18n = require('./extensions/i18n.js')
+  chrome.i18n = require('./extensions/i18n.js').setup(extensionId)
 }
index d6362e3..cee0fb1 100644 (file)
@@ -10,15 +10,15 @@ const path = require('path')
 
 let metadata
 
-const getExtensionMetadata = () => {
+const getExtensionMetadata = (extensionId) => {
   if (!metadata) {
-    metadata = ipcRenderer.sendSync('CHROME_I18N_MANIFEST', chrome.runtime.id)
+    metadata = ipcRenderer.sendSync('CHROME_I18N_MANIFEST', extensionId)
   }
   return metadata
 }
 
-const getMessagesPath = (language) => {
-  const {srcDirectory, default_locale} = getExtensionMetadata()
+const getMessagesPath = (extensionId, language) => {
+  const {srcDirectory, default_locale} = getExtensionMetadata(extensionId)
   const localesDirectory = path.join(srcDirectory, '_locales')
   let messagesPath = path.join(localesDirectory, language, 'messages.json')
   if (!fs.statSyncNoException(messagesPath)) {
@@ -27,9 +27,10 @@ const getMessagesPath = (language) => {
   return messagesPath
 }
 
-const getMessages = (language) => {
+const getMessages = (extensionId, language) => {
   try {
-    return JSON.parse(fs.readFileSync(getMessagesPath(language))) || {}
+    const messagesPath = getMessagesPath(extensionId, language)
+    return JSON.parse(fs.readFileSync(messagesPath)) || {}
   } catch (error) {
     return {}
   }
@@ -65,13 +66,18 @@ const replacePlaceholders = (message, placeholders, substitutions) => {
   return replaceNumberedSubstitutions(message, substitutions)
 }
 
-module.exports = {
-  getMessage (messageName, substitutions) {
-    const language = getLanguage()
-    const messages = getMessages(language)
-    if (messages.hasOwnProperty(messageName)) {
-      const {message, placeholders} = messages[messageName]
-      return replacePlaceholders(message, placeholders, substitutions)
+const getMessage = (extensionId, messageName, substitutions) => {
+  const messages = getMessages(extensionId, getLanguage())
+  if (messages.hasOwnProperty(messageName)) {
+    const {message, placeholders} = messages[messageName]
+    return replacePlaceholders(message, placeholders, substitutions)
+  }
+}
+
+exports.setup = (extensionId) => {
+  return {
+    getMessage (messageName, substitutions) {
+      return getMessage(extensionId, messageName, substitutions)
     }
   }
 }