fix(admin-ui): resolve all bugs displayed for admin-ui bug #308

Author: syntrydy

plugins/auth-server/redux/reducers/OIDCReducer.js

@@ -25,15 +25,9 @@ const reducerName = 'oidcReducer'
 export default function oidcReducer(state = INIT_STATE, action) {
   switch (action.type) {
     case GET_OPENID_CLIENTS:
-      return {
-        ...state,
-        loading: true,
-      }
+      return handleLoading()
     case SEARCH_CLIENTS:
-      return {
-        ...state,
-        loading: true,
-      }
+      return handleLoading()
     case GET_OPENID_CLIENTS_RESPONSE:
       if (action.payload.data) {
         return {
@@ -46,10 +40,7 @@ export default function oidcReducer(state = INIT_STATE, action) {
       }
 
     case ADD_NEW_CLIENT:
-      return {
-        ...state,
-        loading: true,
-      }
+      return handleLoading()
     case ADD_CLIENT_RESPONSE:
       if (action.payload.data) {
         return {
@@ -83,10 +74,7 @@ export default function oidcReducer(state = INIT_STATE, action) {
       }
 
     case DELETE_CLIENT:
-      return {
-        ...state,
-        loading: true,
-      }
+      return handleLoading()
 
     case DELETE_CLIENT_RESPONSE:
       if (action.payload.data) {
@@ -132,5 +120,11 @@ export default function oidcReducer(state = INIT_STATE, action) {
       loading: false,
     }
   }
+  function handleLoading() {
+    return {
+      ...state,
+      loading: true,
+    }
+  }
 }
 reducerRegistry.register(reducerName, oidcReducer)

plugins/auth-server/redux/reducers/ScopeReducer.js

@@ -27,15 +27,9 @@ const reducerName = 'scopeReducer'
 export default function scopeReducer(state = INIT_STATE, action) {
   switch (action.type) {
     case GET_SCOPES:
-      return {
-        ...state,
-        loading: true,
-      }
+      return handleLoading()
     case SEARCH_SCOPES:
-      return {
-        ...state,
-        loading: true,
-      }
+      return handleLoading()
     case GET_SCOPES_RESPONSE:
       if (action.payload.data) {
         return {
@@ -47,10 +41,7 @@ export default function scopeReducer(state = INIT_STATE, action) {
         return handleDefault()
       }
     case GET_SCOPE_BY_INUM:
-      return {
-        ...state,
-        loading: true,
-      }
+      return handleLoading()
     case GET_SCOPE_BY_INUM_RESPONSE:
       if (action.payload.data) {
         return {
@@ -62,10 +53,7 @@ export default function scopeReducer(state = INIT_STATE, action) {
         return handleDefault()
       }
     case GET_SCOPE_BY_PATTERN:
-      return {
-        ...state,
-        loading: true,
-      }
+      return handleLoading()
     case GET_SCOPE_BY_PATTERN_RESPONSE:
       if (action.payload.data) {
         return {
@@ -78,10 +66,7 @@ export default function scopeReducer(state = INIT_STATE, action) {
       }
 
     case ADD_SCOPE:
-      return {
-        ...state,
-        loading: true,
-      }
+      return handleLoading()
     case ADD_SCOPE_RESPONSE:
       if (action.payload.data) {
         return {
@@ -94,10 +79,7 @@ export default function scopeReducer(state = INIT_STATE, action) {
       }
 
     case EDIT_SCOPE:
-      return {
-        ...state,
-        loading: true,
-      }
+      return handleLoading()
 
     case EDIT_SCOPE_RESPONSE:
       if (action.payload.data) {
@@ -110,10 +92,7 @@ export default function scopeReducer(state = INIT_STATE, action) {
         return handleDefault()
       }
     case DELETE_SCOPE:
-      return {
-        ...state,
-        loading: true,
-      }
+      return handleLoading()
     case DELETE_SCOPE_RESPONSE:
       if (action.payload.data) {
         return {
@@ -140,7 +119,12 @@ export default function scopeReducer(state = INIT_STATE, action) {
     default:
       return handleDefault()
   }
-
+  function handleLoading() {
+    return {
+      ...state,
+      loading: true,
+    }
+  }
   function handleDefault() {
     return {
       ...state,