Fix: Community membership (not the authorization) check had a bug

Author: birdofpreyru

.exchange-rates.cache

@@ -1 +1 @@
-{"disclaimer":"Usage subject to terms: https://openexchangerates.org/terms","license":"https://openexchangerates.org/license","timestamp":1510308000,"base":"USD","rates":{"AED":3.673014,"AFN":68.569,"ALL":114.47,"AMD":486.485,"ANG":1.783061,"AOA":165.9225,"ARS":17.505,"AUD":1.302652,"AWG":1.786752,"AZN":1.6985,"BAM":1.678487,"BBD":2,"BDT":82.922356,"BGN":1.679945,"BHD":0.377902,"BIF":1754.75,"BMD":1,"BND":1.361374,"BOB":6.927598,"BRL":3.2524,"BSD":1,"BTC":0.000140623331,"BTN":64.903859,"BWP":10.537513,"BYN":1.986014,"BZD":2.013587,"CAD":1.267198,"CDF":1574.900794,"CHF":0.992823,"CLF":0.02344,"CLP":630.6,"CNH":6.64695,"CNY":6.6374,"COP":3009.48,"CRC":571.285,"CUC":1,"CUP":25.5,"CVE":95.45,"CZK":21.897075,"DJF":178.61,"DKK":6.382137,"DOP":48.025,"DZD":115.0585,"EGP":17.6515,"ERN":15.191062,"ETB":27.305454,"EUR":0.857638,"FJD":2.0568,"FKP":0.76033,"GBP":0.76033,"GEL":2.561079,"GGP":0.76033,"GHS":4.434771,"GIP":0.76033,"GMD":47.225,"GNF":9034.4,"GTQ":7.325694,"GYD":207.34,"HKD":7.8014,"HNL":23.529964,"HRK":6.4689,"HTG":63.127007,"HUF":267.671,"IDR":13522.113381,"ILS":3.531805,"IMP":0.76033,"INR":65.1005,"IQD":1165.75,"IRR":34668.946723,"ISK":103.787037,"JEP":0.76033,"JMD":126.352796,"JOD":0.709001,"JPY":113.37411765,"KES":103.599979,"KGS":69.056478,"KHR":4034.7,"KMF":423.525,"KPW":900,"KRW":1118.5875,"KWD":0.302753,"KYD":0.832442,"KZT":332.745,"LAK":8318.15,"LBP":1511,"LKR":153.435116,"LRD":122.926453,"LSL":14.105316,"LYD":1.37756,"MAD":9.494657,"MDL":17.639888,"MGA":3156.65,"MKD":52.806078,"MMK":1363.2,"MNT":2451.645814,"MOP":8.023813,"MRO":354.62,"MUR":34.4025,"MVR":15.390384,"MWK":726.429716,"MXN":19.05501,"MYR":4.19094,"MZN":60.946857,"NAD":14.105316,"NGN":359.615,"NIO":30.6,"NOK":8.118593,"NPR":103.824326,"NZD":1.439299,"OMR":0.385022,"PAB":1,"PEN":3.24468,"PGK":3.20136,"PHP":51.2275,"PKR":105.01998,"PLN":3.628435,"PYG":5647.05,"QAR":3.81595,"RON":3.978357,"RSD":101.645,"RUB":59.2175,"RWF":834.38494,"SAR":3.75065,"SBD":7.786861,"SCR":13.405088,"SDG":6.669494,"SEK":8.361287,"SGD":1.359918,"SHP":0.76033,"SLL":7643.328121,"SOS":577.905,"SRD":7.448,"SSP":130.2634,"STD":20904.400676,"SVC":8.740713,"SYP":514.98999,"SZL":14.104479,"THB":33.11975,"TJS":8.797495,"TMT":3.50998,"TND":2.511905,"TOP":2.254482,"TRY":3.869115,"TTD":6.743762,"TWD":30.177,"TZS":2255.35,"UAH":26.510801,"UGX":3626.1,"USD":1,"UYU":29.192808,"UZS":8059.05,"VEF":10.785743,"VND":22717.253308,"VUV":105.856667,"WST":2.54744,"XAF":562.573348,"XAG":0.05875449,"XAU":0.00077839,"XCD":2.70255,"XDR":0.712478,"XOF":562.573348,"XPD":0.00099085,"XPF":102.343382,"XPT":0.00106596,"YER":250.281642,"ZAR":14.337335,"ZMW":9.914463,"ZWL":322.355011}}
+{"disclaimer":"Usage subject to terms: https://openexchangerates.org/terms","license":"https://openexchangerates.org/license","timestamp":1510430426,"base":"USD","rates":{"AED":3.673018,"AFN":68.5905,"ALL":114.268015,"AMD":485.835,"ANG":1.783866,"AOA":165.9225,"ARS":17.49,"AUD":1.305262,"AWG":1.786752,"AZN":1.6985,"BAM":1.676693,"BBD":2,"BDT":83.148415,"BGN":1.67641,"BHD":0.377032,"BIF":1760,"BMD":1,"BND":1.357824,"BOB":6.909874,"BRL":3.27715,"BSD":1,"BTC":0.000158230047,"BTN":65.009917,"BWP":10.60399,"BYN":1.98819,"BZD":2.005603,"CAD":1.268115,"CDF":1579,"CHF":0.99603,"CLF":0.02347,"CLP":631.5,"CNH":6.661808,"CNY":6.64125,"COP":3008,"CRC":568.23,"CUC":1,"CUP":25.5,"CVE":94.95,"CZK":21.9065,"DJF":178.97,"DKK":6.379455,"DOP":47.92,"DZD":115.078,"EGP":17.6505,"ERN":15.336062,"ETB":27.275,"EUR":0.857192,"FJD":2.0568,"FKP":0.75816,"GBP":0.75816,"GEL":2.561854,"GGP":0.75816,"GHS":4.43036,"GIP":0.75816,"GMD":47.26,"GNF":9013.35,"GTQ":7.32654,"GYD":208.292635,"HKD":7.80107,"HNL":23.591593,"HRK":6.4643,"HTG":63.169196,"HUF":267.154,"IDR":13524.113381,"ILS":3.549105,"IMP":0.75816,"INR":65.368,"IQD":1164.45,"IRR":34668.946723,"ISK":103.613694,"JEP":0.75816,"JMD":126.058505,"JOD":0.709001,"JPY":113.525,"KES":103.602489,"KGS":69.119478,"KHR":4025.25,"KMF":422.25,"KPW":900,"KRW":1121.1,"KWD":0.302532,"KYD":0.831547,"KZT":332.157791,"LAK":8307.85,"LBP":1512.7,"LKR":153.291485,"LRD":122.988952,"LSL":14.335761,"LYD":1.365714,"MAD":9.479031,"MDL":17.601594,"MGA":3152.15,"MKD":52.786,"MMK":1352.15,"MNT":2451.70248,"MOP":8.01931,"MRO":354.39,"MUR":34.400227,"MVR":15.390067,"MWK":722.69,"MXN":19.1046,"MYR":4.191994,"MZN":60.994761,"NAD":14.365,"NGN":359.215,"NIO":30.684016,"NOK":8.13405,"NPR":104.024834,"NZD":1.4431,"OMR":0.385115,"PAB":1,"PEN":3.2425,"PGK":3.197794,"PHP":51.13782,"PKR":104.82,"PLN":3.6236,"PYG":5640.75,"QAR":3.8115,"RON":3.982002,"RSD":101.59,"RUB":59.135,"RWF":852.62,"SAR":3.750225,"SBD":7.802011,"SCR":13.988,"SDG":6.662214,"SEK":8.35636,"SGD":1.3605,"SHP":0.75816,"SLL":7643.328121,"SOS":577.205,"SRD":7.448,"SSP":130.2634,"STD":20725.000286,"SVC":8.731481,"SYP":514.98999,"SZL":14.331779,"THB":33.043,"TJS":8.790913,"TMT":3.50998,"TND":2.500007,"TOP":2.254482,"TRY":3.86361,"TTD":6.715426,"TWD":30.171,"TZS":2252.2,"UAH":26.451281,"UGX":3634.1,"USD":1,"UYU":29.133845,"UZS":8080,"VEF":9.984033,"VND":22714.053308,"VUV":106.093334,"WST":2.547073,"XAF":562.280992,"XAG":0.05917511,"XAU":0.00078379,"XCD":2.70255,"XDR":0.712515,"XOF":562.280992,"XPD":0.00100377,"XPF":102.290196,"XPT":0.00107614,"YER":250.25,"ZAR":14.37934,"ZMW":9.914463,"ZWL":322.355011}}

src/shared/containers/tc-communities/Loader.jsx

@@ -74,8 +74,8 @@ class Loader extends React.Component {
 
     const visitorGroupIds = visitorGroups ? visitorGroups.map(g => g.id) : [];
 
-    const member = visitorGroups && meta.groupIds
-      && Boolean(_.intersection(meta.groupIds, visitorGroupIds.length));
+    const member = Boolean(visitorGroups && meta.groupIds
+      && _.intersection(meta.groupIds, visitorGroupIds).length);
 
     /* Community does not require authorization. */
     if (!meta.authorizedGroupIds) return Community({ member, meta });