From 36cb35c1cec5e213ac6339f325ad3086eeda02e6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?R=C3=A9becca=20Tinchon?= <rebecca.tinchon@hotmail.fr>
Date: Tue, 14 May 2024 17:54:38 +0200
Subject: [PATCH] course creation

---
 .../modules/course/components/creation.jsx    | 11 +---
 .../modules/course/components/empty.jsx       | 50 +++++++++++++++++
 .../modules/course/components/parameters.jsx  | 10 +++-
 .../course/components/registration.jsx        |  9 +++-
 .../Resources/modules/course/store/actions.js |  9 +++-
 .../modules/tools/events/components/tool.jsx  | 41 ++++++++++----
 .../modules/tools/events/containers/tool.jsx  | 54 ++++++++++---------
 .../modules/tools/events/store/reducer.js     |  7 ++-
 .../trainings/catalog/containers/main.jsx     | 36 +++++++------
 .../Resources/translations/cursus.en.json     |  1 +
 .../Resources/translations/cursus.fr.json     |  1 +
 11 files changed, 158 insertions(+), 71 deletions(-)
 create mode 100644 src/plugin/cursus/Resources/modules/course/components/empty.jsx

diff --git a/src/plugin/cursus/Resources/modules/course/components/creation.jsx b/src/plugin/cursus/Resources/modules/course/components/creation.jsx
index 64d115f941..57276b0863 100644
--- a/src/plugin/cursus/Resources/modules/course/components/creation.jsx
+++ b/src/plugin/cursus/Resources/modules/course/components/creation.jsx
@@ -25,18 +25,9 @@ const CourseCreationComponent = (props) =>
     }]}
     title={trans('trainings', {}, 'tools')}
     subtitle={trans('new_course', {}, 'cursus')}
-    primaryAction="add"
-    actions={[{
-      name: 'add',
-      type: LINK_BUTTON,
-      icon: 'fa fa-fw fa-plus',
-      label: trans('add_course', {}, 'cursus'),
-      target: props.path,
-      group: trans('management'),
-      primary: true
-    }]}
   >
     <CourseForm
+      contextType={props.contextType}
       path={props.path}
       name={selectors.FORM_NAME}
     />
diff --git a/src/plugin/cursus/Resources/modules/course/components/empty.jsx b/src/plugin/cursus/Resources/modules/course/components/empty.jsx
new file mode 100644
index 0000000000..1deefd510a
--- /dev/null
+++ b/src/plugin/cursus/Resources/modules/course/components/empty.jsx
@@ -0,0 +1,50 @@
+import React from 'react'
+import {PropTypes as T} from 'prop-types'
+
+import {trans} from '#/main/app/intl/translation'
+import {LINK_BUTTON} from '#/main/app/buttons'
+import {ToolPage} from '#/main/core/tool/containers/page'
+import {Button} from '#/main/app/action/components/button'
+import {ContentSizing} from '#/main/app/content/components/sizing'
+import {ContentPlaceholder} from '#/main/app/content/components/placeholder'
+
+const EmptyCourse = (props) =>
+  <ToolPage
+    primaryAction="add"
+    actions={[
+      {
+        name: 'add',
+        type: LINK_BUTTON,
+        icon: 'fa fa-fw fa-plus',
+        label: trans('add_course', {}, 'cursus'),
+        target: `${props.path}/new`,
+        group: trans('management'),
+        displayed: props.canEdit,
+        primary: true
+      }
+    ]}
+  >
+    <ContentSizing size="lg" className="mt-4">
+      <ContentPlaceholder
+        size="lg"
+        title={trans('no_course', {}, 'cursus')}
+        help={trans('no_course_help', {}, 'cursus')}
+      />
+      <Button
+        className="btn btn-primary w-100 my-3"
+        size="lg"
+        type={LINK_BUTTON}
+        target={`${props.path}/new`}
+        label={trans('add_course', {}, 'cursus')}
+      />
+    </ContentSizing>
+  </ToolPage>
+
+EmptyCourse.propTypes = {
+  path: T.string.isRequired,
+  canEdit: T.bool.isRequired
+}
+
+export {
+  EmptyCourse
+}
diff --git a/src/plugin/cursus/Resources/modules/course/components/parameters.jsx b/src/plugin/cursus/Resources/modules/course/components/parameters.jsx
index 649e3227ca..0bd87317b6 100644
--- a/src/plugin/cursus/Resources/modules/course/components/parameters.jsx
+++ b/src/plugin/cursus/Resources/modules/course/components/parameters.jsx
@@ -19,11 +19,16 @@ const CourseParameters = (props) => {
       className="mt-3"
       name={props.name}
       buttons={true}
+      locked={'workspace' === props.contextType ? ['workspace', '_workspaceType'] : []}
       save={{
         type: CALLBACK_BUTTON,
         callback: () => props.save(props.course, props.isNew, props.name).then(course => {
           if (props.isNew) {
-            history.push(route(course))
+            if ('workspace' === props.contextType) {
+              history.push(props.path)
+            } else {
+              history.push(route(course))
+            }
           }
         })
       }}
@@ -281,7 +286,8 @@ CourseParameters.propTypes = {
   course: T.shape(
     CourseTypes.propTypes
   ),
-  update: T.func.isRequired
+  update: T.func.isRequired,
+  contextType: T.string.isRequired
 }
 
 export {
diff --git a/src/plugin/cursus/Resources/modules/course/components/registration.jsx b/src/plugin/cursus/Resources/modules/course/components/registration.jsx
index b93aa085bf..5b85500d40 100644
--- a/src/plugin/cursus/Resources/modules/course/components/registration.jsx
+++ b/src/plugin/cursus/Resources/modules/course/components/registration.jsx
@@ -24,7 +24,11 @@ const CourseRegistration = (props) => {
         type: CALLBACK_BUTTON,
         callback: () => props.save(props.course, props.isNew, props.name).then(course => {
           if (props.isNew) {
-            history.push(route(course))
+            if ('workspace' === props.contextType) {
+              history.push(props.path)
+            } else {
+              history.push(route(course))
+            }
           }
         })
       }}
@@ -166,7 +170,8 @@ CourseRegistration.propTypes = {
   course: T.shape(
     CourseTypes.propTypes
   ),
-  update: T.func.isRequired
+  update: T.func.isRequired,
+  contextType: T.string.isRequired
 }
 
 export {
diff --git a/src/plugin/cursus/Resources/modules/course/store/actions.js b/src/plugin/cursus/Resources/modules/course/store/actions.js
index c00f17f342..d19c6292d0 100644
--- a/src/plugin/cursus/Resources/modules/course/store/actions.js
+++ b/src/plugin/cursus/Resources/modules/course/store/actions.js
@@ -36,8 +36,15 @@ actions.open = (courseSlug, force = false) => (dispatch, getState) => {
   }
 }
 
-actions.openForm = (courseSlug = null, defaultProps = {}) => (dispatch) => {
+actions.openForm = (courseSlug = null, defaultProps = {}, workspace = null) => (dispatch) => {
   if (!courseSlug) {
+    if(workspace) {
+      defaultProps = {
+        ...defaultProps,
+        _workspaceType: 'workspace',
+        workspace: workspace
+      }
+    }
     return dispatch(formActions.resetForm(selectors.FORM_NAME, defaultProps, true))
   }
 
diff --git a/src/plugin/cursus/Resources/modules/tools/events/components/tool.jsx b/src/plugin/cursus/Resources/modules/tools/events/components/tool.jsx
index c50f09ba26..96ba23986b 100644
--- a/src/plugin/cursus/Resources/modules/tools/events/components/tool.jsx
+++ b/src/plugin/cursus/Resources/modules/tools/events/components/tool.jsx
@@ -3,8 +3,11 @@ import get from 'lodash/get'
 import {PropTypes as T} from 'prop-types'
 
 import {Routes} from '#/main/app/router'
-
 import {Course} from '#/plugin/cursus/course/containers/main'
+import {Course as CourseTypes} from '#/plugin/cursus/prop-types'
+import {CourseCreation} from '#/plugin/cursus/course/components/creation'
+
+import {EmptyCourse} from '#/plugin/cursus/course/components/empty'
 import {EventsAll} from '#/plugin/cursus/tools/events/components/all'
 import {EventsPublic} from '#/plugin/cursus/tools/events/components/public'
 import {EventsDetails} from '#/plugin/cursus/tools/events/containers/details'
@@ -19,15 +22,29 @@ const EventsTool = (props) =>
     ]}
     routes={[
       {
+        path: '/new',
+        onEnter: () => props.openForm(null, CourseTypes.defaultProps, props.currentContext.data),
+        disabled: !props.canEdit,
+        component: CourseCreation
+      }, {
         path: '/about',
-        onEnter: () => props.openCourse(props.course.slug),
-        render: (params = {}) => (
-          <Course
-            path={props.path+'/about'}
-            slug={props.course.slug}
-            history={params.history}
-          />
-        )
+        onEnter: () => {
+          if (props.course) {
+            return props.openCourse(props.course.slug)
+          }
+        },
+        render: (params = {}) => {
+          if (props.course) {
+            return (
+              <Course
+                path={props.path + '/about'}
+                slug={props.course.slug}
+                history={params.history}
+              />)
+          } else {
+            return (<EmptyCourse path={props.path} canEdit={props.canEdit}/>)
+          }
+        }
       }, {
         path: '/registered',
         onEnter: props.invalidateList,
@@ -50,13 +67,13 @@ const EventsTool = (props) =>
       }, {
         path: '/all',
         onEnter: props.invalidateList,
+        disabled: !props.canEdit && !props.canRegister,
         render: () => (
           <EventsAll
             path={props.path}
             contextId={get(props.currentContext, 'data.id')}
           />
-        ),
-        disabled: !props.canEdit && !props.canRegister
+        )
       }, {
         path: '/presences',
         component: EventsPresences
@@ -68,6 +85,7 @@ const EventsTool = (props) =>
     ]}
   />
 
+
 EventsTool.propTypes = {
   path: T.string.isRequired,
   currentContext: T.shape({
@@ -78,6 +96,7 @@ EventsTool.propTypes = {
   canRegister: T.bool.isRequired,
   invalidateList: T.func.isRequired,
   open: T.func.isRequired,
+  openForm: T.func.isRequired,
   openCourse: T.func,
   course: T.shape({
     slug: T.string
diff --git a/src/plugin/cursus/Resources/modules/tools/events/containers/tool.jsx b/src/plugin/cursus/Resources/modules/tools/events/containers/tool.jsx
index 9c82d66c8f..490603dc92 100644
--- a/src/plugin/cursus/Resources/modules/tools/events/containers/tool.jsx
+++ b/src/plugin/cursus/Resources/modules/tools/events/containers/tool.jsx
@@ -5,35 +5,37 @@ import {withReducer} from '#/main/app/store/reducer'
 
 import {selectors} from '#/plugin/cursus/tools/events/store'
 import {selectors as toolSelectors} from '#/main/core/tool/store'
-
 import {actions as listActions} from '#/main/app/content/list/store'
-import {actions as courseActions} from '#/plugin/cursus/course/store'
-
 import {EventsTool as EventsToolComponent} from '#/plugin/cursus/tools/events/components/tool'
-import {actions as eventActions, reducer as eventReducer, selectors as eventSelectors} from '#/plugin/cursus/event/store'
-
+import {actions as eventActions, reducer as eventReducer, selectors as eventSelectors}             from '#/plugin/cursus/event/store'
+import {actions as courseActions, reducer as courseReducer, selectors as courseSelectors} from '#/plugin/cursus/course/store'
 
-const EventsTool = withReducer(eventSelectors.STORE_NAME, eventReducer)( // not the best place to do it
-  connect(
-    (state) => ({
-      path: toolSelectors.path(state),
-      course: selectors.course(state),
-      currentContext: toolSelectors.context(state),
-      canEdit: hasPermission('edit', toolSelectors.toolData(state)),
-      canRegister: hasPermission('register', toolSelectors.toolData(state))
-    }),
-    (dispatch) => ({
-      open(id) {
-        dispatch(eventActions.open(id))
-      },
-      openCourse(slug) {
-        dispatch(courseActions.open(slug))
-      },
-      invalidateList() {
-        dispatch(listActions.invalidateData(selectors.LIST_NAME))
-      }
-    })
-  )(EventsToolComponent)
+const EventsTool = withReducer(courseSelectors.STORE_NAME, courseReducer)(
+  withReducer(eventSelectors.STORE_NAME, eventReducer)(
+    connect(
+      (state) => ({
+        path: toolSelectors.path(state),
+        course: selectors.course(state),
+        currentContext: toolSelectors.context(state),
+        canEdit: hasPermission('edit', toolSelectors.toolData(state)),
+        canRegister: hasPermission('register', toolSelectors.toolData(state))
+      }),
+      (dispatch) => ({
+        open(id) {
+          dispatch(eventActions.open(id))
+        },
+        openCourse(slug) {
+          dispatch(courseActions.open(slug))
+        },
+        openForm(slug, defaultProps, workspace) {
+          dispatch(courseActions.openForm(slug, defaultProps, workspace))
+        },
+        invalidateList() {
+          dispatch(listActions.invalidateData(selectors.LIST_NAME))
+        }
+      })
+    )(EventsToolComponent)
+  )
 )
 
 export {
diff --git a/src/plugin/cursus/Resources/modules/tools/events/store/reducer.js b/src/plugin/cursus/Resources/modules/tools/events/store/reducer.js
index 4d9d0329bc..4948f550af 100644
--- a/src/plugin/cursus/Resources/modules/tools/events/store/reducer.js
+++ b/src/plugin/cursus/Resources/modules/tools/events/store/reducer.js
@@ -3,7 +3,9 @@ import {makeListReducer} from '#/main/app/content/list/store'
 import {makeReducer, combineReducers} from '#/main/app/store/reducer'
 
 import {TOOL_LOAD} from '#/main/core/tool/store'
+import {FORM_SUBMIT_SUCCESS} from '#/main/app/content/form/store'
 import {selectors} from '#/plugin/cursus/tools/events/store/selectors'
+import {selectors as courseSelectors} from '#/plugin/cursus/course/store'
 
 const reducer = combineReducers({
   events: makeListReducer(selectors.LIST_NAME, {
@@ -15,8 +17,9 @@ const reducer = combineReducers({
   }),
   course: makeReducer({}, {
     [makeInstanceAction(TOOL_LOAD, selectors.STORE_NAME)]: (state, action) => {
-      return action.toolData.course
-    }
+      return typeof action.toolData.course !== 'undefined' ? action.toolData.course : null
+    },
+    [FORM_SUBMIT_SUCCESS+'/'+courseSelectors.FORM_NAME]: (state, action) => action.updatedData
   })
 })
 
diff --git a/src/plugin/cursus/Resources/modules/tools/trainings/catalog/containers/main.jsx b/src/plugin/cursus/Resources/modules/tools/trainings/catalog/containers/main.jsx
index c9a1d8124a..7b6172de5c 100644
--- a/src/plugin/cursus/Resources/modules/tools/trainings/catalog/containers/main.jsx
+++ b/src/plugin/cursus/Resources/modules/tools/trainings/catalog/containers/main.jsx
@@ -4,26 +4,28 @@ import {hasPermission} from '#/main/app/security'
 import {withReducer} from '#/main/app/store/reducer'
 
 import {selectors as toolSelectors} from '#/main/core/tool/store'
-import {actions as courseActions} from '#/plugin/cursus/course/store'
 import {reducer, selectors} from '#/plugin/cursus/tools/trainings/catalog/store'
 import {CatalogMain as CatalogMainComponent} from '#/plugin/cursus/tools/trainings/catalog/components/main'
+import {actions as courseActions, reducer as courseReducer, selectors as courseSelectors} from '#/plugin/cursus/course/store'
 
-const CatalogMain = withReducer(selectors.STORE_NAME, reducer)(
-  connect(
-    (state) => ({
-      path: toolSelectors.path(state),
-      course: selectors.course(state),
-      canEdit: hasPermission('edit', toolSelectors.toolData(state))
-    }),
-    (dispatch) => ({
-      open(slug) {
-        dispatch(courseActions.open(slug))
-      },
-      openForm(slug, defaultProps) {
-        dispatch(courseActions.openForm(slug, defaultProps))
-      }
-    })
-  )(CatalogMainComponent)
+const CatalogMain = withReducer(courseSelectors.STORE_NAME, courseReducer)(
+  withReducer(selectors.STORE_NAME, reducer)(
+    connect(
+      (state) => ({
+        path: toolSelectors.path(state),
+        course: selectors.course(state),
+        canEdit: hasPermission('edit', toolSelectors.toolData(state))
+      }),
+      (dispatch) => ({
+        open(slug) {
+          dispatch(courseActions.open(slug))
+        },
+        openForm(slug, defaultProps) {
+          dispatch(courseActions.openForm(slug, defaultProps))
+        }
+      })
+    )(CatalogMainComponent)
+  )
 )
 
 export {
diff --git a/src/plugin/cursus/Resources/translations/cursus.en.json b/src/plugin/cursus/Resources/translations/cursus.en.json
index 63256c47c7..92ffd9ab70 100644
--- a/src/plugin/cursus/Resources/translations/cursus.en.json
+++ b/src/plugin/cursus/Resources/translations/cursus.en.json
@@ -105,6 +105,7 @@
     "export-presences-empty": "Download the attendance list (empty)",
     "export-presences-filled": "Download the attendance list (filled)",
     "no_course": "No training",
+    "no_course_help": "No training is linked to this workspace",
     "no_session": "No session",
     "session_info": "Session information",
     "download_presence": "Download the certificate of presence (.pdf)",
diff --git a/src/plugin/cursus/Resources/translations/cursus.fr.json b/src/plugin/cursus/Resources/translations/cursus.fr.json
index e7a1e1a907..0fcb3c6da6 100644
--- a/src/plugin/cursus/Resources/translations/cursus.fr.json
+++ b/src/plugin/cursus/Resources/translations/cursus.fr.json
@@ -107,6 +107,7 @@
     "export-presences-empty": "Télécharger la liste de présences (vide)",
     "export-presences-filled": "Télécharger la liste de présences (remplie)",
     "no_course": "Aucune formation",
+    "no_course_help": "Aucune formation n'est liée à cet espace d'activités",
     "no_session": "Aucune session",
     "session_info": "Informations sur la session",
     "download_presence": "Télécharger l'attestation de présence (.pdf)",