PrivateCoffee/element-web
4
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 3

Make completionStates an object

Browse source 
So that we can sensibly track the number completed by taking the length of it.
This commit is contained in:
David Baker 2016-08-10 18:50:56 +01:00
parent fa498eb8c6
commit d6f7358f81
1 changed files with 9 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
src/components/views/dialogs/MultiInviteDialog.js
View file

@ -28,8 +28,8 @@ export default class MultiInviteDialog extends React.Component {
this.state = {
busy: false,
completionStates: [], // State of each address (invited or error)
errorTexts: [], // Textual error per address
completionStates: {}, // State of each address (invited or error)
errorTexts: {}, // Textual error per address
done: false,
};
for (let i = 0; i < this.props.inputs.length; ++i) {
@ -114,16 +114,19 @@ export default class MultiInviteDialog extends React.Component {
}
_getProgressIndicator() {
const numErrors = this.state.completionStates.filter((s) => {
return s == 'error';
}).length;
let numErrors = 0;
for (const k of Object.keys(this.state.completionStates)) {
if (this.state.completionStates[k] == 'error') {
++numErrors;
}
}
let errorText;
if (numErrors > 0) {
const plural = numErrors > 1 ? 's' : '';
errorText = <span className="error">({numErrors} error{plural})</span>
}
return <span>
{this.state.completionStates.length} / {this.props.inputs.length} {errorText}
{Object.keys(this.state.completionStates).length} / {this.props.inputs.length} {errorText}
</span>;
}