From 981302192ca2747963ee0df593b5e65be200c9f2 Mon Sep 17 00:00:00 2001
From: Ramon Wenger <donramonloco+github@gmail.com>
Date: Mon, 29 Jul 2019 09:40:12 +0200
Subject: [PATCH 1/6] Fix content block bug

---
 client/src/components/ContentBlock.vue | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/client/src/components/ContentBlock.vue b/client/src/components/ContentBlock.vue
index 641d76e6..34880f00 100644
--- a/client/src/components/ContentBlock.vue
+++ b/client/src/components/ContentBlock.vue
@@ -174,6 +174,9 @@
         return this.me.selectedClass;
       },
       hidden() {
+        if (!this.contentBlock.id || !this.contentBlock.visibleFor || !this.contentBlock.hiddenFor) {
+          return false;
+        }
         if (this.contentBlock.userCreated) {
           if (this.schoolClass.id === '') {
             return false;
@@ -260,7 +263,7 @@
           left: 0;
           bottom: 0;
           right: 0;
-          background: rgba(255,255,255,0.5);
+          background: rgba(255, 255, 255, 0.5);
           z-index: 10;
         }
       }

From fae0043138cb34450386fd9fab647a0128bdcefb Mon Sep 17 00:00:00 2001
From: Ramon Wenger <donramonloco+github@gmail.com>
Date: Mon, 29 Jul 2019 10:34:12 +0200
Subject: [PATCH 2/6] Only display edit actions for content block when edit
 mode is on

---
 client/src/components/ContentBlock.vue | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/client/src/components/ContentBlock.vue b/client/src/components/ContentBlock.vue
index 34880f00..509ee994 100644
--- a/client/src/components/ContentBlock.vue
+++ b/client/src/components/ContentBlock.vue
@@ -1,14 +1,14 @@
 <template>
   <div class="content-block__container" :class="{'content-block__container--hidden': hidden}">
     <div class="content-block" :class="specialClass">
-      <div class="content-block__actions" v-if="canEditContentBlock">
+      <div class="content-block__actions" v-if="canEditContentBlock && editModule">
         <user-widget v-bind="me" class="content-block__user-widget"></user-widget>
         <more-options-widget>
           <li class="popover-links__link"><a @click="deleteContentBlock()">Löschen</a></li>
           <li class="popover-links__link"><a @click="editContentBlock()">Bearbeiten</a></li>
         </more-options-widget>
       </div>
-      <div class="content-block__visibility">
+      <div class="content-block__visibility" v-if="editModule">
         <visibility-action
           v-if="!contentBlock.indent"
           :block="contentBlock"></visibility-action>
@@ -288,7 +288,7 @@
     &__actions {
       position: absolute;
       top: 10px;
-      right: -45px;
+      right: -85px;
       display: flex;
       flex-direction: column;
       align-items: center;

From 4483175dbe9b30fb4832df7be9671af1a6b192f7 Mon Sep 17 00:00:00 2001
From: Ramon Wenger <donramonloco+github@gmail.com>
Date: Mon, 29 Jul 2019 10:53:31 +0200
Subject: [PATCH 3/6] Hide module subnavigation for students

---
 client/src/components/modules/ModuleNavigation.vue | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/client/src/components/modules/ModuleNavigation.vue b/client/src/components/modules/ModuleNavigation.vue
index 5006c0dc..5b774647 100644
--- a/client/src/components/modules/ModuleNavigation.vue
+++ b/client/src/components/modules/ModuleNavigation.vue
@@ -1,5 +1,5 @@
 <template>
-  <nav class="module-navigation sub-navigation">
+  <nav class="module-navigation sub-navigation" v-if="canManageContent">
     <div class="sub-navigation-item">
       <router-link class="sub-navigation-item__title" :to="moduleContentLink">Modul</router-link>
     </div>
@@ -32,7 +32,7 @@
       </div>
     </div>
 
-    <div class="module-navigation__toggle-menu" v-if="canManageContent">
+    <div class="module-navigation__toggle-menu">
       <toggle-editing></toggle-editing>
     </div>
 

From 5ae47029a3e5f759ed301d78e346bbb7f8191622 Mon Sep 17 00:00:00 2001
From: Ramon Wenger <donramonloco+github@gmail.com>
Date: Mon, 29 Jul 2019 11:09:51 +0200
Subject: [PATCH 4/6] Hide content blocks for teachers when not in edit mode

---
 client/src/components/Chapter.vue      | 28 ++++++++++++++++++++++++--
 client/src/components/ContentBlock.vue | 14 +++----------
 client/src/helpers/content-block.js    | 14 +++++++++++++
 3 files changed, 43 insertions(+), 13 deletions(-)

diff --git a/client/src/components/Chapter.vue b/client/src/components/Chapter.vue
index 19a50389..e7de2161 100644
--- a/client/src/components/Chapter.vue
+++ b/client/src/components/Chapter.vue
@@ -10,7 +10,7 @@
 
     <content-block :contentBlock="contentBlock"
                    :parent="chapter.id"
-                   :key="contentBlock.id" v-for="contentBlock in chapter.contentBlocks">
+                   :key="contentBlock.id" v-for="contentBlock in filteredContentBlocks">
     </content-block>
   </div>
 </template>
@@ -20,6 +20,8 @@
   import AddContentBlockButton from '@/components/AddContentBlockButton';
 
   import {mapGetters} from 'vuex';
+  import {isHidden} from '@/helpers/content-block';
+  import {meQuery} from '@/graphql/queries';
 
   export default {
     props: ['chapter', 'index'],
@@ -30,8 +32,30 @@
     },
 
     computed: {
-      ...mapGetters(['editModule'])
+      ...mapGetters(['editModule']),
+      filteredContentBlocks() {
+        if (!(this.chapter && this.chapter.contentBlocks)) {
+          return []
+        }
+        if (this.editModule) {
+          return this.chapter.contentBlocks;
+        }
+        return this.chapter.contentBlocks.filter(contentBlock => !isHidden(contentBlock, this.schoolClass));
+      },
+      schoolClass() {
+        return this.me.selectedClass;
+      },
     },
+
+    data() {
+      return {
+        me: {}
+      }
+    },
+
+    apollo: {
+      me: meQuery
+    }
   }
 </script>
 
diff --git a/client/src/components/ContentBlock.vue b/client/src/components/ContentBlock.vue
index 509ee994..8e77b257 100644
--- a/client/src/components/ContentBlock.vue
+++ b/client/src/components/ContentBlock.vue
@@ -69,6 +69,8 @@
 
   import {mapGetters} from 'vuex';
 
+  import {isHidden} from '@/helpers/content-block';
+
   const instruments = {
     base_communication: 'Sprache & Kommunikation',
     base_society: 'Gesellschaft'
@@ -174,17 +176,7 @@
         return this.me.selectedClass;
       },
       hidden() {
-        if (!this.contentBlock.id || !this.contentBlock.visibleFor || !this.contentBlock.hiddenFor) {
-          return false;
-        }
-        if (this.contentBlock.userCreated) {
-          if (this.schoolClass.id === '') {
-            return false;
-          }
-          return !this.contentBlock.visibleFor.map(entry => entry.id).includes(this.schoolClass.id);
-        } else {
-          return this.contentBlock.hiddenFor.map(entry => entry.id).includes(this.schoolClass.id);
-        }
+        return isHidden(this.contentBlock, this.schoolClass);
       }
     },
 
diff --git a/client/src/helpers/content-block.js b/client/src/helpers/content-block.js
index 563cfc1e..bad90b65 100644
--- a/client/src/helpers/content-block.js
+++ b/client/src/helpers/content-block.js
@@ -1,3 +1,17 @@
 export function setUserBlockType(isAssignment) {
   return isAssignment ? 'TASK' : 'NORMAL';
 }
+
+export const isHidden = (contentBlock, schoolClass) => {
+  if (!contentBlock.id || !contentBlock.visibleFor || !contentBlock.hiddenFor) {
+    return false;
+  }
+  if (contentBlock.userCreated) {
+    if (schoolClass.id === '') {
+      return false;
+    }
+    return !contentBlock.visibleFor.map(entry => entry.id).includes(schoolClass.id);
+  } else {
+    return contentBlock.hiddenFor.map(entry => entry.id).includes(schoolClass.id);
+  }
+};

From bdfd6ee510d7d3efc83e24d9d280251acee7311e Mon Sep 17 00:00:00 2001
From: Ramon Wenger <donramonloco+github@gmail.com>
Date: Mon, 29 Jul 2019 11:22:00 +0200
Subject: [PATCH 5/6] Change the content block type only if the user specifies
 a type

---
 server/books/schema/mutations/contentblock.py | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/server/books/schema/mutations/contentblock.py b/server/books/schema/mutations/contentblock.py
index 6de57be7..c557dfa9 100644
--- a/server/books/schema/mutations/contentblock.py
+++ b/server/books/schema/mutations/contentblock.py
@@ -33,7 +33,10 @@ class MutateContentBlock(relay.ClientIDMutation):
 
             content_block = get_object(ContentBlock, id_param)
 
-            block_type = content_block_data.get('type', content_block.type.upper())
+            block_type = content_block_data.get('type')
+            if block_type is not None:  # only change the type if the user has set a type
+                content_block.type = set_user_defined_block_type(block_type)
+
             module = content_block.get_parent().get_parent().specific
 
             if visibility_list is not None:
@@ -48,7 +51,7 @@ class MutateContentBlock(relay.ClientIDMutation):
             if contents is not None:
                 content_block.contents = json.dumps([handle_content_block(c, info.context, module) for c in contents])
 
-            content_block.type = set_user_defined_block_type(block_type)
+
             content_block.save()
 
             return cls(content_block=content_block)

From 0b0942ba4bc0ed759012b394f38e125232f2a8ec Mon Sep 17 00:00:00 2001
From: Ramon Wenger <donramonloco+github@gmail.com>
Date: Mon, 29 Jul 2019 11:28:21 +0200
Subject: [PATCH 6/6] Display the assignment text, not the title for assignment
 submission page

---
 client/src/components/AssignmentWithSubmissions.vue | 2 +-
 client/src/components/modules/ModuleNavigation.vue  | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/client/src/components/AssignmentWithSubmissions.vue b/client/src/components/AssignmentWithSubmissions.vue
index 6c88ba23..02cdcc96 100644
--- a/client/src/components/AssignmentWithSubmissions.vue
+++ b/client/src/components/AssignmentWithSubmissions.vue
@@ -1,6 +1,6 @@
 <template>
   <div class="assignment-with-submissions">
-    <h1 class="assignment-with-submissions__title">{{assignment.title}}</h1>
+    <!--<h1 class="assignment-with-submissions__title">{{assignment.assignment}}</h1>-->
       <h4 class="assignment-with-submissions__heading">Aufgabe</h4>
 
     <p class="assignment-with-submissions__text">{{assignment.assignment}}</p>
diff --git a/client/src/components/modules/ModuleNavigation.vue b/client/src/components/modules/ModuleNavigation.vue
index 5b774647..097c306f 100644
--- a/client/src/components/modules/ModuleNavigation.vue
+++ b/client/src/components/modules/ModuleNavigation.vue
@@ -13,7 +13,7 @@
       >{{assignmentTitle(assignment)}}
       </router-link>
     </sub-navigation-item>
-    <div class="module-navigation__module-content" v-if="false">
+    <div class="module-navigation__module-content" v-if="false"> <!-- Do not display this for now, might be used later again though -->
       <router-link
         tag="h3"
         :to="moduleContentLink"
@@ -107,7 +107,7 @@
         return `/module/${this.module.slug}/submissions/${assignment.id}`;
       },
       assignmentTitle(assignment) {
-        return assignment.title.length > 25 ? assignment.title.substring(0, 22) + '...' : assignment.title;
+        return assignment.assignment.length > 25 ? assignment.assignment.substring(0, 22) + '...' : assignment.assignment;
       }
     }
   }