Merge pull request #4339 from marmelab/useeditcontroller-inconsistency

Ensure useEditController specifies a default redirect to its caller
marmelab · Jan 23, 2020 · 53ce05c · 53ce05c
2 parents 4de9af2 + 15ff009
commit 53ce05c
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/packages/ra-core/src/controller/useEditController.ts b/packages/ra-core/src/controller/useEditController.ts
@@ -42,6 +42,7 @@ export interface EditControllerProps {
     resource: string;
     basePath: string;
     record?: Record;
+    redirect: RedirectionSideEffect;
     version: number;
     successMessage?: string;
 }
@@ -101,7 +102,7 @@ const useEditController = (props: EditProps): EditControllerProps => {
     const save = useCallback(
         (
             data: Partial<Record>,
-            redirectTo = 'list',
+            redirectTo = DefaultRedirect,
             { onSuccess, onFailure } = {}
         ) =>
             update(
@@ -146,8 +147,11 @@ const useEditController = (props: EditProps): EditControllerProps => {
         resource,
         basePath,
         record,
+        redirect: DefaultRedirect,
         version,
     };
 };
 
 export default useEditController;
+
+const DefaultRedirect = 'list';