Small tweaks:

- only perform check if payments are enabled
- check using Array.prototype.some (which returns on first truthy result)

Auditors: @mrose17

js/components/tab.js

@@ -256,12 +256,22 @@ class Tab extends ImmutableComponent {
   }
 }
 
-windowStore.addChangeListener(() => {
-  var presentP = false
-  const windowState = windowStore.getState().toJS()
-
-  windowState.tabs.forEach((tab) => { if (tab.location === 'about:preferences#payments') presentP = true })
+const paymentsEnabled = () => {
+  const getSetting = require('../settings').getSetting
+  const settings = require('../constants/settings')
+  return getSetting(settings.PAYMENTS_ENABLED)
+}
 
-  ipc.send(messages.LEDGER_PAYMENTS_PRESENT, presentP)
+windowStore.addChangeListener(() => {
+  if (paymentsEnabled()) {
+    const windowState = windowStore.getState()
+    const tabs = windowState && windowState.get('tabs')
+    if (tabs) {
+      const presentP = tabs.some((tab) => {
+        return tab.get('location') === 'about:preferences#payments'
+      })
+      ipc.send(messages.LEDGER_PAYMENTS_PRESENT, presentP)
+    }
+  }
 })
 module.exports = Tab