Clarify required auth stages

This commit is contained in:
J. Ryan Stinnett 2019-01-31 16:37:37 -06:00
parent fb39ba1ffc
commit ba50ef8445

View file

@ -270,11 +270,15 @@ module.exports = React.createClass({
this.validateField(FIELD_USERNAME, ev.type);
},
/**
* A step is required if all flows include that step.
*
* @param {string} step A stage name to check
* @returns {boolean} Whether it is required
*/
_authStepIsRequired(step) {
// A step is required if no flow exists which does not include that step
// (Notwithstanding setups like either email or msisdn being required)
return !this.props.flows.some((flow) => {
return !flow.stages.includes(step);
return this.props.flows.every((flow) => {
return flow.stages.includes(step);
});
},