From 9a6b9bf3e35f98218bb6cb2fdd62886ee7b8ffeb Mon Sep 17 00:00:00 2001
From: Ramon Wenger <donramonloco+github@gmail.com>
Date: Wed, 21 Nov 2018 14:56:43 +0100
Subject: [PATCH] Add edit objective form wizard to client

---
 client/src/App.vue                            |   4 +-
 .../EditObjectiveGroupWizard.vue              | 107 ++++++++++++++++++
 .../NewObjectiveGroupWizard.vue               |  47 ++------
 .../objective-groups/ObjectiveForm.vue        |   6 +-
 .../objective-groups/ObjectiveGroup.vue       |  14 ++-
 .../objective-groups/ObjectiveGroupForm.vue   |  64 +++++++++++
 client/src/graphql/client.js                  |   3 +-
 .../gql/fragments/objectiveGroupParts.gql     |   1 +
 .../gql/mutations/updateObjectiveGroup.gql    |  16 +++
 .../src/graphql/gql/objectiveGroupQuery.gql   |  14 +++
 client/src/store/index.js                     |  11 +-
 11 files changed, 245 insertions(+), 42 deletions(-)
 create mode 100644 client/src/components/objective-groups/EditObjectiveGroupWizard.vue
 create mode 100644 client/src/components/objective-groups/ObjectiveGroupForm.vue
 create mode 100644 client/src/graphql/gql/mutations/updateObjectiveGroup.gql
 create mode 100644 client/src/graphql/gql/objectiveGroupQuery.gql

diff --git a/client/src/App.vue b/client/src/App.vue
index 9e17d019..e78c6e57 100644
--- a/client/src/App.vue
+++ b/client/src/App.vue
@@ -15,6 +15,7 @@
   import NewRoomEntryWizard from '@/components/rooms/room-entries/NewRoomEntryWizard';
   import EditRoomEntryWizard from '@/components/rooms/room-entries/EditRoomEntryWizard';
   import NewObjectiveGroupWizard from '@/components/objective-groups/NewObjectiveGroupWizard';
+  import EditObjectiveGroupWizard from '@/components/objective-groups/EditObjectiveGroupWizard';
 
   export default {
     name: 'App',
@@ -28,7 +29,8 @@
       EditContentBlockWizard,
       NewRoomEntryWizard,
       EditRoomEntryWizard,
-      NewObjectiveGroupWizard
+      NewObjectiveGroupWizard,
+      EditObjectiveGroupWizard
     },
 
     computed: {
diff --git a/client/src/components/objective-groups/EditObjectiveGroupWizard.vue b/client/src/components/objective-groups/EditObjectiveGroupWizard.vue
new file mode 100644
index 00000000..9e67b362
--- /dev/null
+++ b/client/src/components/objective-groups/EditObjectiveGroupWizard.vue
@@ -0,0 +1,107 @@
+<template>
+  <objective-group-form
+    :title="title"
+    :objectives="objectives"
+    @save="saveObjectiveGroup"
+    @hide="hideModal"
+  ></objective-group-form>
+</template>
+
+<script>
+  import Modal from '@/components/Modal';
+  import ObjectiveGroupForm from '@/components/objective-groups/ObjectiveGroupForm';
+  import AddContentElement from '@/components/AddContentElement';
+
+  import UPDATE_OBJECTIVE_GROUP_MUTATION from '@/graphql/gql/mutations/updateObjectiveGroup.gql';
+  import MODULE_DETAILS_QUERY from '@/graphql/gql/moduleDetailsQuery.gql';
+  import OBJECTIVE_GROUP_QUERY from '@/graphql/gql/objectiveGroupQuery.gql';
+
+  export default {
+    components: {
+      AddContentElement,
+      Modal,
+      ObjectiveGroupForm
+    },
+
+    computed: {
+      title() {
+        if (this.$store.state.objectiveGroupType === 'society') {
+          return 'Gesellschaft';
+        }
+        return 'Sprache & Kommunikation';
+      },
+      objectives() {
+        return this.objectiveGroup.objectives.edges.map(edge => edge.node)
+      }
+    },
+
+    methods: {
+      saveObjectiveGroup(objectives) {
+        const objectiveGroup = {
+          id: this.$store.state.currentObjectiveGroup,
+          objectives,
+        };
+        this.$apollo.mutate({
+          mutation: UPDATE_OBJECTIVE_GROUP_MUTATION,
+          variables: {
+            input: {
+              objectiveGroup
+            }
+          },
+          // todo: make update work
+          // update: (store, {data: {addObjectiveGroup: {objectiveGroup}}}) => {
+          //   const query = MODULE_DETAILS_QUERY;
+          //   const variables = {slug: this.$route.params.slug};
+          //   const data = store.readQuery({query, variables});
+          //   debugger;
+          //   if (data.module && data.module.objectiveGroups) {
+          //     data.module.objectiveGroups.edges.push({
+          //       node: objectiveGroup,
+          //       __typename: 'ObjectiveGroupNode'
+          //     });
+          //     store.writeQuery({query, variables, data});
+          //   }
+          //
+          // }
+          refetchQueries: [{
+            query: MODULE_DETAILS_QUERY,
+            variables: {
+              slug: this.$route.params.slug
+            }
+          }]
+
+        }).then(() => {
+          this.hideModal();
+        });
+      },
+      hideModal() {
+        this.$store.dispatch('hideModal');
+      },
+    },
+
+    apollo: {
+      objectiveGroup() {
+        return {
+          query: OBJECTIVE_GROUP_QUERY,
+          variables: {
+            id: this.$store.state.currentObjectiveGroup
+          }
+        }
+      }
+    },
+
+    data() {
+      return {
+        objectiveGroup: {
+          objectives: {
+            edges: []
+          }
+        }
+      }
+    }
+  }
+</script>
+
+<style scoped lang="scss">
+
+</style>
diff --git a/client/src/components/objective-groups/NewObjectiveGroupWizard.vue b/client/src/components/objective-groups/NewObjectiveGroupWizard.vue
index 84497951..a09c69f9 100644
--- a/client/src/components/objective-groups/NewObjectiveGroupWizard.vue
+++ b/client/src/components/objective-groups/NewObjectiveGroupWizard.vue
@@ -1,27 +1,15 @@
 <template>
-  <modal class="new-objective-group-wizard">
-    <template slot="header">
-      <h4 class="new-objective-group-wizard__heading">Lernziele: {{title}}</h4>
-    </template>
-    
-    <objective-form
-      :objective="objective"
-      v-for="(objective, index) in objectives"
-      @input="updateObjective($event, index)"
-      :key="index"></objective-form>
-    <add-content-element @add-element="addObjective"></add-content-element>
-
-    <div slot="footer">
-      <a class="button button--primary" data-cy="modal-save-button" @click="saveObjectiveGroup">Speichern</a>
-      <a class="button" @click="hideModal">Abbrechen</a>
-    </div>
-
-  </modal>
+  <objective-group-form
+    :title="title"
+    :objectives="objectives"
+    @save="saveObjectiveGroup"
+    @hide="hideModal"
+  ></objective-group-form>
 </template>
 
 <script>
   import Modal from '@/components/Modal';
-  import ObjectiveForm from '@/components/objective-groups/ObjectiveForm';
+  import ObjectiveGroupForm from '@/components/objective-groups/ObjectiveGroupForm';
   import AddContentElement from '@/components/AddContentElement';
 
   import NEW_OBJECTIVE_GROUP_MUTATION from '@/graphql/gql/mutations/addObjectiveGroup.gql';
@@ -31,7 +19,7 @@
     components: {
       AddContentElement,
       Modal,
-      ObjectiveForm
+      ObjectiveGroupForm
     },
 
     computed: {
@@ -44,18 +32,11 @@
     },
 
     methods: {
-      addObjective() {
-        this.objectives.push({});
-      },
-      updateObjective(text, index) {
-        this.objectives.splice(index, 1, {text});
-      },
-      saveObjectiveGroup() {
-        console.log('saving');
+      saveObjectiveGroup(objectives) {
         const objectiveGroup = {
           title: this.$store.state.objectiveGroupType,
-          objectives: this.objectives,
-          module: this.$store.state.parentModule
+          module: this.$store.state.parentModule,
+          objectives,
         };
         this.$apollo.mutate({
           mutation: NEW_OBJECTIVE_GROUP_MUTATION,
@@ -106,9 +87,5 @@
 </script>
 
 <style scoped lang="scss">
-  .new-objective-group-wizard {
-    &__heading {
-      margin-bottom: 0;
-    }
-  }
+
 </style>
diff --git a/client/src/components/objective-groups/ObjectiveForm.vue b/client/src/components/objective-groups/ObjectiveForm.vue
index b1c5bc62..a2e94d15 100644
--- a/client/src/components/objective-groups/ObjectiveForm.vue
+++ b/client/src/components/objective-groups/ObjectiveForm.vue
@@ -2,10 +2,10 @@
   <div class="objective-form">
     <modal-input class="objective-form__input"
                  placeholder="Lernziel erfassen..."
-                 :value="objective.value"
+                 :value="objective.text"
                  @input="$emit('input', $event)"
     ></modal-input>
-    <a class="icon-button">
+    <a class="icon-button" @click="$emit('delete')">
       <trash-icon class="icon-button__icon"></trash-icon>
     </a>
   </div>
@@ -14,7 +14,7 @@
 
 <script>
   import TrashIcon from '@/components/icons/TrashIcon';
-  import ModalInput from "@/components/ModalInput";
+  import ModalInput from '@/components/ModalInput';
 
   export default {
     props: ['objective'],
diff --git a/client/src/components/objective-groups/ObjectiveGroup.vue b/client/src/components/objective-groups/ObjectiveGroup.vue
index 48dd2a59..89a46600 100644
--- a/client/src/components/objective-groups/ObjectiveGroup.vue
+++ b/client/src/components/objective-groups/ObjectiveGroup.vue
@@ -3,6 +3,10 @@
     <div class="objective-group__actions">
       <visibility-action :block="group">
       </visibility-action>
+      <a @click="editObjectiveGroup()" v-if="group.mine" class="objective-group__action-button">
+        <pen-icon class="objective-group__action-icon action-icon"></pen-icon>
+      </a>
+
     </div>
 
     <h4>{{group.displayTitle}}</h4>
@@ -19,6 +23,7 @@
 <script>
   import VisibilityAction from '@/components/visibility/VisibilityAction';
   import EyeIcon from '@/components/icons/EyeIcon';
+  import PenIcon from '@/components/icons/PenIcon';
 
   import ME_QUERY from '@/graphql/gql/meQuery.gql';
 
@@ -32,7 +37,8 @@
 
     components: {
       VisibilityAction,
-      EyeIcon
+      EyeIcon,
+      PenIcon
     },
 
     apollo: {
@@ -45,6 +51,12 @@
       canManageContent() {
         return this.me.permissions.includes('users.can_manage_school_class_content');
       }
+    },
+
+    methods: {
+      editObjectiveGroup() {
+        this.$store.dispatch('editObjectiveGroup', this.group.id);
+      }
     }
   }
 </script>
diff --git a/client/src/components/objective-groups/ObjectiveGroupForm.vue b/client/src/components/objective-groups/ObjectiveGroupForm.vue
new file mode 100644
index 00000000..b32ca67e
--- /dev/null
+++ b/client/src/components/objective-groups/ObjectiveGroupForm.vue
@@ -0,0 +1,64 @@
+<template>
+  <modal class="objective-group-form">
+    <template slot="header">
+      <h4 class="objective-group-form__heading">Lernziele: {{title}}</h4>
+    </template>
+
+    <objective-form
+      :objective="objective"
+      v-for="(objective, index) in initialObjectives"
+      @input="updateObjective($event, index)"
+      @delete="removeObjective(index)"
+      :key="index"></objective-form>
+    <add-content-element @add-element="addObjective"></add-content-element>
+
+    <div slot="footer">
+      <a class="button button--primary" data-cy="modal-save-button"
+         @click="$emit('save', initialObjectives)">Speichern</a>
+      <a class="button" @click="$emit('hide')">Abbrechen</a>
+    </div>
+
+  </modal>
+</template>
+
+<script>
+  import Modal from '@/components/Modal';
+  import ObjectiveForm from '@/components/objective-groups/ObjectiveForm';
+  import AddContentElement from '@/components/AddContentElement';
+
+  export default {
+    props: ['title', 'objectives'],
+
+    components: {
+      AddContentElement,
+      Modal,
+      ObjectiveForm
+    },
+
+    methods: {
+      addObjective() {
+        this.initialObjectives.push({});
+      },
+      updateObjective(text, index) {
+        this.initialObjectives.splice(index, 1, {text});
+      },
+      removeObjective(index) {
+        this.initialObjectives.splice(index, 1);
+      }
+    },
+
+    data() {
+      return {
+        initialObjectives: this.objectives
+      }
+    }
+  }
+</script>
+
+<style scoped lang="scss">
+  .objective-group-form {
+    &__heading {
+      margin-bottom: 0;
+    }
+  }
+</style>
diff --git a/client/src/graphql/client.js b/client/src/graphql/client.js
index 48b7f090..c7031b1a 100644
--- a/client/src/graphql/client.js
+++ b/client/src/graphql/client.js
@@ -44,7 +44,8 @@ const cache = new InMemoryCache({
       contentBlock: (_, args, {getCacheKey}) => getCacheKey({__typename: 'ContentBlockNode', id: args.id}),
       chapter: (_, args, {getCacheKey}) => getCacheKey({__typename: 'ChapterNode', id: args.id}),
       assignment: (_, args, {getCacheKey}) => getCacheKey({__typename: 'AssignmentNode', id: args.id}),
-      objective: (_, args, {getCacheKey}) => getCacheKey({__typename: 'ObjectiveNode', id: args.id})
+      objective: (_, args, {getCacheKey}) => getCacheKey({__typename: 'ObjectiveNode', id: args.id}),
+      objectiveGroup: (_, args, {getCacheKey}) => getCacheKey({__typename: 'ObjectiveGroupNode', id: args.id})
     }
   }
 });
diff --git a/client/src/graphql/gql/fragments/objectiveGroupParts.gql b/client/src/graphql/gql/fragments/objectiveGroupParts.gql
index 38e6f8b7..a806919a 100644
--- a/client/src/graphql/gql/fragments/objectiveGroupParts.gql
+++ b/client/src/graphql/gql/fragments/objectiveGroupParts.gql
@@ -2,6 +2,7 @@ fragment ObjectiveGroupParts on ObjectiveGroupNode {
   id
   title
   displayTitle
+  mine
   owner {
     id
   }
diff --git a/client/src/graphql/gql/mutations/updateObjectiveGroup.gql b/client/src/graphql/gql/mutations/updateObjectiveGroup.gql
new file mode 100644
index 00000000..4a0bfa68
--- /dev/null
+++ b/client/src/graphql/gql/mutations/updateObjectiveGroup.gql
@@ -0,0 +1,16 @@
+#import "../fragments/objectiveGroupParts.gql"
+mutation UpdateObjectiveGroup($input: UpdateObjectiveGroupInput!) {
+  updateObjectiveGroup(input: $input) {
+    objectiveGroup {
+      ...ObjectiveGroupParts
+      objectives {
+        edges {
+          node {
+            id
+            text
+          }
+        }
+      }
+    }
+  }
+}
diff --git a/client/src/graphql/gql/objectiveGroupQuery.gql b/client/src/graphql/gql/objectiveGroupQuery.gql
new file mode 100644
index 00000000..c4b6b653
--- /dev/null
+++ b/client/src/graphql/gql/objectiveGroupQuery.gql
@@ -0,0 +1,14 @@
+#import "./fragments/objectiveGroupParts.gql"
+query ObjectiveGroupQuery($id: ID!) {
+  objectiveGroup(id: $id) {
+    ...ObjectiveGroupParts
+    objectives {
+      edges {
+        node {
+          id
+          text
+        }
+      }
+    }
+  }
+}
diff --git a/client/src/store/index.js b/client/src/store/index.js
index 759dbd75..5950370a 100644
--- a/client/src/store/index.js
+++ b/client/src/store/index.js
@@ -16,7 +16,8 @@ export default new Vuex.Store({
     currentRoomEntry: '',
     parentRoom: null,
     parentModule: '',
-    objectiveGroupType: ''
+    objectiveGroupType: '',
+    currentObjectiveGroup: ''
   },
 
   getters: {},
@@ -37,6 +38,7 @@ export default new Vuex.Store({
       commit('setParentRoom', null);
       commit('setParentModule', '');
       commit('setObjectiveGroupType', '');
+      commit('setCurrentObjectiveGroup', '');
     },
     resetContentBlockPosition({commit}) {
       commit('setContentBlockPosition', {});
@@ -65,6 +67,10 @@ export default new Vuex.Store({
       commit('setObjectiveGroupType', type);
       dispatch('showModal', 'new-objective-group-wizard');
     },
+    editObjectiveGroup({commit, dispatch}, payload) {
+      commit('setCurrentObjectiveGroup', payload);
+      dispatch('showModal', 'edit-objective-group-wizard');
+    },
     showModal({commit}, payload) {
       document.body.classList.add('no-scroll'); // won't get at the body any other way
       commit('setModal', payload);
@@ -107,6 +113,9 @@ export default new Vuex.Store({
     },
     setObjectiveGroupType(state, payload) {
       state.objectiveGroupType = payload;
+    },
+    setCurrentObjectiveGroup(state, payload) {
+      state.currentObjectiveGroup = payload;
     }
   }
 })