From 2511e99409c21c7abe139d2fb90f98280836019c Mon Sep 17 00:00:00 2001
From: Sebastian Sdorra <sebastian.sdorra@cloudogu.com>
Date: Mon, 30 Jul 2018 11:17:47 +0200
Subject: [PATCH] small fixes in users module

---
 scm-ui/src/users/modules/users.js      | 16 +++++++---------
 scm-ui/src/users/modules/users.test.js |  2 +-
 2 files changed, 8 insertions(+), 10 deletions(-)

diff --git a/scm-ui/src/users/modules/users.js b/scm-ui/src/users/modules/users.js
index ca3a064cf2..823f389f86 100644
--- a/scm-ui/src/users/modules/users.js
+++ b/scm-ui/src/users/modules/users.js
@@ -111,7 +111,7 @@ export function fetchUser(name: string) {
 export function fetchUserPending(name: string): Action {
   return {
     type: FETCH_USER_PENDING,
-    payload: { name }
+    payload: name
   };
 }
 
@@ -122,11 +122,11 @@ export function fetchUserSuccess(user: any): Action {
   };
 }
 
-export function fetchUserFailure(username: string, error: Error): Action {
+export function fetchUserFailure(name: string, error: Error): Action {
   return {
     type: FETCH_USER_FAILURE,
     payload: {
-      username,
+      name,
       error
     }
   };
@@ -148,7 +148,6 @@ export function createUser(user: User, callback?: () => void) {
       .catch(err =>
         dispatch(
           createUserFailure(
-            user,
             new Error(`failed to add user ${user.name}: ${err.message}`)
           )
         )
@@ -169,11 +168,10 @@ export function createUserSuccess(): Action {
   };
 }
 
-export function createUserFailure(user: User, err: Error): Action {
+export function createUserFailure(error: Error): Action {
   return {
     type: CREATE_USER_FAILURE,
-    payload: err,
-    user
+    payload: error
   };
 }
 
@@ -392,7 +390,7 @@ function byNamesReducer(state: any = {}, action: any = {}) {
 
     // Fetch single user actions
     case FETCH_USER_PENDING:
-      return reducerByName(state, action.payload.name, {
+      return reducerByName(state, action.payload, {
         loading: true,
         error: null
       });
@@ -403,7 +401,7 @@ function byNamesReducer(state: any = {}, action: any = {}) {
         entry: action.payload
       });
     case FETCH_USER_FAILURE:
-      return reducerByName(state, action.payload.username, {
+      return reducerByName(state, action.payload.name, {
         loading: false,
         error: action.payload.error
       });
diff --git a/scm-ui/src/users/modules/users.test.js b/scm-ui/src/users/modules/users.test.js
index 464c524a3b..d4bc5b3726 100644
--- a/scm-ui/src/users/modules/users.test.js
+++ b/scm-ui/src/users/modules/users.test.js
@@ -517,7 +517,7 @@ describe("users reducer", () => {
   it("should set the loading to false and the error if user could not be created", () => {
     const newState = reducer(
       { create: { loading: true, error: null } },
-      createUserFailure(userFord, new Error("kaputt kaputt"))
+      createUserFailure(new Error("kaputt kaputt"))
     );
     expect(newState.create.loading).toBeFalsy();
     expect(newState.create.error).toEqual(new Error("kaputt kaputt"));