Ensure show prop is always of type bool to avoid unintentional items …

…to be shown
mattermost · Apr 3, 2019 · 31c8e67 · 31c8e67
1 parent 56613c6
commit 31c8e67
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/components/admin_console/system_users/system_users_dropdown/system_users_dropdown.jsx b/components/admin_console/system_users/system_users_dropdown/system_users_dropdown.jsx
@@ -338,7 +338,7 @@ export default class SystemUsersDropdown extends React.Component {
         let showMakeNotActive = !Utils.isSystemAdmin(user.roles);
         let showManageTeams = true;
         let showRevokeSessions = true;
-        const showMfaReset = this.props.mfaEnabled && user.mfa_active && !user.is_bot;
+        const showMfaReset = this.props.mfaEnabled && !!user.mfa_active && !user.is_bot;
 
         if (user.delete_at > 0) {
             currentRoles = (
@@ -410,7 +410,7 @@ export default class SystemUsersDropdown extends React.Component {
                                 text={Utils.localizeMessage('admin.user_item.resetMfa', 'Remove MFA')}
                             />
                             <MenuItemAction
-                                show={user.auth_service && this.props.experimentalEnableAuthenticationTransfer && !user.is_bot}
+                                show={!!user.auth_service && this.props.experimentalEnableAuthenticationTransfer && !user.is_bot}
                                 onClick={this.handleResetPassword}
                                 text={Utils.localizeMessage('admin.user_item.switchToEmail', 'Switch to Email/Password')}
                             />