From f9c9610059d740e0c9db43670a0384e6ec37c4e5 Mon Sep 17 00:00:00 2001 From: NejcZdovc Date: Mon, 13 Mar 2017 10:40:15 +0100 Subject: [PATCH] Auto-include propagates correctly Resolves #7451 Auditors: @mrose17 Test Plan: - Enable payments in settings - Disable auto-include switch - Visit brianbondy.com in a new tab, site is added in the list and is not included --- app/renderer/components/preferences/payment/ledgerTable.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/app/renderer/components/preferences/payment/ledgerTable.js b/app/renderer/components/preferences/payment/ledgerTable.js index 122a8cef12b..e270bf0fac1 100644 --- a/app/renderer/components/preferences/payment/ledgerTable.js +++ b/app/renderer/components/preferences/payment/ledgerTable.js @@ -59,7 +59,7 @@ class LedgerTable extends ImmutableComponent { return result } } - return true + return getSetting(settings.AUTO_SUGGEST_SITES, this.props.settings) } shouldShow (synopsis) { @@ -101,7 +101,7 @@ class LedgerTable extends ImmutableComponent { const publisherURL = synopsis.get('publisherURL') const percentage = synopsis.get('percentage') const site = synopsis.get('site') - const defaultSiteSetting = true + const defaultAutoInclude = this.enabledForSite(synopsis) return [ { @@ -131,7 +131,7 @@ class LedgerTable extends ImmutableComponent { { html: ,