Merge pull request #609 from shopgate/PWA-1664-remove-redundant-selec…

…tor-getUserLogin

Removed duplicate getUserLogin selector

libraries/common/selectors/user.js

@@ -8,32 +8,40 @@ import { getUrl } from './url';
 export const getUserState = state => state.user;
 
 /**
- * @param {Object} state The current application state.
- * @return {Object}
+ * Gets user.login from the redux store.
+ * @param {Object} state The application state.
+ * @return {Object|null}
  */
-export const getUserData = createSelector(
+export const getLoginData = createSelector(
   getUserState,
   (user) => {
-    if (!user.data) {
+    if (!user || !user.login) {
       return null;
     }
 
-    return user.data;
+    return user.login;
   }
 );
 
 /**
  * @param {Object} state The current application state.
  * @return {Object|null}
+ * @deprecated
  */
-export const getUserLogin = createSelector(
+export const getUserLogin = getLoginData;
+
+/**
+ * @param {Object} state The current application state.
+ * @return {Object}
+ */
+export const getUserData = createSelector(
   getUserState,
-  (userState) => {
-    if (!userState) {
+  (user) => {
+    if (!user.data) {
       return null;
     }
 
-    return userState.login;
+    return user.data;
   }
 );
 
@@ -43,7 +51,7 @@ export const getUserLogin = createSelector(
  * @return {boolean}
  */
 export const isUserLoggedIn = createSelector(
-  getUserLogin,
+  getLoginData,
   (login) => {
     if (!login) {
       return false;
@@ -98,19 +106,21 @@ export const getUserEmail = createSelector(
   }
 );
 
-/**
- * Gets user.login from the redux store.
- * @param {Object} state The application state.
- * @return {Object|null}
- */
-export const getLoginData = state => state.user.login;
-
 /**
  * Selects the disabled state of the login action from the redux store.
  * @param {Object} state The application state.
  * @return {Object|null}
  */
-export const isUserLoginDisabled = state => getLoginData(state).disabled;
+export const isUserLoginDisabled = createSelector(
+  getLoginData,
+  (login) => {
+    if (!login) {
+      return false;
+    }
+
+    return login.disabled;
+  }
+);
 
 /**
  * Gets the register url.