grassrootseconomics/openethereum
5
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Be lenient around invalid owners map (#3764)

Browse Source 
* Be lenient around invalid owners map

* Filter invalid owners before render
This commit is contained in:
Jaco Greeff 2016-12-09 17:52:25 +01:00 committed by Nicolas Gotchac
parent 1213ada59c
commit ffd8314a11
2 changed files with 15 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
js/src/views/Accounts/Summary/summary.js
View File

@ -14,6 +14,7 @@
// You should have received a copy of the GNU General Public License // You should have received a copy of the GNU General Public License
// along with Parity. If not, see <http://www.gnu.org/licenses/>. // along with Parity. If not, see <http://www.gnu.org/licenses/>.
import BigNumber from 'bignumber.js';
import React, { Component, PropTypes } from 'react'; import React, { Component, PropTypes } from 'react';
import { Link } from 'react-router'; import { Link } from 'react-router';
import { isEqual } from 'lodash'; import { isEqual } from 'lodash';
@ -113,15 +114,16 @@ export default class Summary extends Component {
renderOwners () { renderOwners () {
const { owners } = this.props; 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 null;
} }
return ( return (
<div className={ styles.owners }> <div className={ styles.owners }>
{ {
owners.map((owner) => ( ownersValid.map((owner) => (
<div key={ owner.address }> <div key={ owner.address }>
<div <div
data-tip data-tip

18
js/src/views/Accounts/accounts.js
View File

@ -293,13 +293,17 @@ function mapStateToProps (state) {
const walletsOwners = Object const walletsOwners = Object
.keys(walletsInfo) .keys(walletsInfo)
.map((wallet) => ({ .map((wallet) => {
owners: walletsInfo[wallet].owners.map((owner) => ({ const owners = walletsInfo[wallet].owners || [];
address: owner,
name: accountsInfo[owner] && accountsInfo[owner].name || owner return {
})), owners: owners.map((owner) => ({
address: wallet address: owner,
})) name: accountsInfo[owner] && accountsInfo[owner].name || owner
})),
address: wallet
};
})
.reduce((walletsOwners, wallet) => { .reduce((walletsOwners, wallet) => {
walletsOwners[wallet.address] = wallet.owners; walletsOwners[wallet.address] = wallet.owners;
return walletsOwners; return walletsOwners;