From ffd8314a115b49c9dc7eeff582bd480fbaec0a8e Mon Sep 17 00:00:00 2001
From: Jaco Greeff <jacogr@gmail.com>
Date: Fri, 9 Dec 2016 17:52:25 +0100
Subject: [PATCH] Be lenient around invalid owners map (#3764)

* Be lenient around invalid owners map

* Filter invalid owners before render
---
 js/src/views/Accounts/Summary/summary.js |  6 ++++--
 js/src/views/Accounts/accounts.js        | 18 +++++++++++-------
 2 files changed, 15 insertions(+), 9 deletions(-)

diff --git a/js/src/views/Accounts/Summary/summary.js b/js/src/views/Accounts/Summary/summary.js
index 764f24edf..aeff8a2e5 100644
--- a/js/src/views/Accounts/Summary/summary.js
+++ b/js/src/views/Accounts/Summary/summary.js
@@ -14,6 +14,7 @@
 // You should have received a copy of the GNU General Public License
 // along with Parity.  If not, see <http://www.gnu.org/licenses/>.
 
+import BigNumber from 'bignumber.js';
 import React, { Component, PropTypes } from 'react';
 import { Link } from 'react-router';
 import { isEqual } from 'lodash';
@@ -113,15 +114,16 @@ export default class Summary extends Component {
 
   renderOwners () {
     const { owners } = this.props;
+    const ownersValid = (owners || []).filter((owner) => owner.address && new BigNumber(owner.address).gt(0));
 
-    if (!owners || owners.length === 0) {
+    if (!ownersValid || ownersValid.length === 0) {
       return null;
     }
 
     return (
       <div className={ styles.owners }>
         {
-          owners.map((owner) => (
+          ownersValid.map((owner) => (
             <div key={ owner.address }>
               <div
                 data-tip
diff --git a/js/src/views/Accounts/accounts.js b/js/src/views/Accounts/accounts.js
index 4383b7662..a844ae989 100644
--- a/js/src/views/Accounts/accounts.js
+++ b/js/src/views/Accounts/accounts.js
@@ -293,13 +293,17 @@ function mapStateToProps (state) {
 
   const walletsOwners = Object
     .keys(walletsInfo)
-    .map((wallet) => ({
-      owners: walletsInfo[wallet].owners.map((owner) => ({
-        address: owner,
-        name: accountsInfo[owner] && accountsInfo[owner].name || owner
-      })),
-      address: wallet
-    }))
+    .map((wallet) => {
+      const owners = walletsInfo[wallet].owners || [];
+
+      return {
+        owners: owners.map((owner) => ({
+          address: owner,
+          name: accountsInfo[owner] && accountsInfo[owner].name || owner
+        })),
+        address: wallet
+      };
+    })
     .reduce((walletsOwners, wallet) => {
       walletsOwners[wallet.address] = wallet.owners;
       return walletsOwners;