From ebfeec41841a1dfef1cdc59c9529a42af570f575 Mon Sep 17 00:00:00 2001
From: Daniel Egger <daniel.egger@gmail.com>
Date: Fri, 7 Oct 2022 17:08:55 +0200
Subject: [PATCH] Refactor competence profile

---
 .../competences/CompetenceDetail.vue          |  4 ++-
 .../competences/PerformanceCriteriaRow.vue    |  4 ++-
 .../learningPath/SelfEvaluation.vue           |  4 ++-
 .../CompetenceIndexPage.vue}                  | 26 ++++++++++----
 .../CompetenceListPage.vue}                   |  5 ++-
 .../CompetenceParentPage.vue}                 |  2 +-
 .../PerformanceCriteriaPage.vue}              |  7 ++--
 client/src/router/index.ts                    | 12 +++----
 .../create_default_competence_profile.py      | 34 +++++++++----------
 server/vbv_lernwelt/competence/models.py      |  3 ++
 10 files changed, 64 insertions(+), 37 deletions(-)
 rename client/src/pages/{CompetencesMainView.vue => competence/CompetenceIndexPage.vue} (85%)
 rename client/src/pages/{CompetencesDetailView.vue => competence/CompetenceListPage.vue} (88%)
 rename client/src/pages/{ComptencesView.vue => competence/CompetenceParentPage.vue} (93%)
 rename client/src/pages/{CompetencePerfomanceCriteriaView.vue => competence/PerformanceCriteriaPage.vue} (93%)

diff --git a/client/src/components/competences/CompetenceDetail.vue b/client/src/components/competences/CompetenceDetail.vue
index e080236a..67998432 100644
--- a/client/src/components/competences/CompetenceDetail.vue
+++ b/client/src/components/competences/CompetenceDetail.vue
@@ -22,7 +22,9 @@ const togglePerformanceCriteria = () => {
   <div>
     <div :class="{ 'pb-8 border-b border-gray-500 mb-4': isOpen }" class="-mx-8 px-8">
       <div class="mb-4 flex flex-row justify-between items-center">
-        <h2 class="text-large">{{ competence.title }}</h2>
+        <h2 class="text-large">
+          {{ competence.competence_id }} {{ competence.title }}
+        </h2>
         <button class="transition-transform" :class="{ 'rotate-180': isOpen }">
           <it-icon-arrow-down
             class="h-10 w-10"
diff --git a/client/src/components/competences/PerformanceCriteriaRow.vue b/client/src/components/competences/PerformanceCriteriaRow.vue
index caa676f4..464b171b 100644
--- a/client/src/components/competences/PerformanceCriteriaRow.vue
+++ b/client/src/components/competences/PerformanceCriteriaRow.vue
@@ -25,7 +25,9 @@ const props = withDefaults(defineProps<Props>(), {
         <it-icon-smiley-neutral v-else></it-icon-smiley-neutral>
       </div>
       <div class="pr-5 lg:mr-10 mb-4 lg:mb-0">
-        <h4 class="text-bold mb-2">{{ criteria.title }}</h4>
+        <h4 class="text-bold mb-2">
+          {{ criteria.competence_id }} {{ criteria.title }}
+        </h4>
         <p>
           Lerneinheit:
           <a class="link" :href="criteria.learning_unit.frontend_url">
diff --git a/client/src/components/learningPath/SelfEvaluation.vue b/client/src/components/learningPath/SelfEvaluation.vue
index be53ba5b..d8c216bf 100644
--- a/client/src/components/learningPath/SelfEvaluation.vue
+++ b/client/src/components/learningPath/SelfEvaluation.vue
@@ -72,7 +72,9 @@ function handleContinue() {
       </p>
 
       <div class="mt-4 lg:mt-8 p-6 lg:p-12 border border-gray-500">
-        <h2 class="heading-2">{{ currentQuestion.title }}</h2>
+        <h2 class="heading-2">
+          {{ currentQuestion.competence_id }} {{ currentQuestion.title }}
+        </h2>
 
         <div class="mt-4 lg:mt-8 flex flex-col lg:flex-row justify-between gap-6">
           <button
diff --git a/client/src/pages/CompetencesMainView.vue b/client/src/pages/competence/CompetenceIndexPage.vue
similarity index 85%
rename from client/src/pages/CompetencesMainView.vue
rename to client/src/pages/competence/CompetenceIndexPage.vue
index 3f7d33ba..58202c37 100644
--- a/client/src/pages/CompetencesMainView.vue
+++ b/client/src/pages/competence/CompetenceIndexPage.vue
@@ -6,7 +6,7 @@ import _ from "lodash";
 import * as log from "loglevel";
 import { computed } from "vue";
 
-log.debug("CompetencesMainView created");
+log.debug("CompetenceIndexPage created");
 
 const competenceStore = useCompetenceStore();
 
@@ -43,8 +43,11 @@ const countStatus = computed(() => {
 </script>
 
 <template>
-  <div class="container-large lg:mt-8">
-    <div class="flex flex-col lg:flex-row items-center justify-between mb-10">
+  <div class="container-large lg:mt-4">
+    <div
+      v-if="competenceStore.competenceProfilePage"
+      class="flex flex-col lg:flex-row items-center justify-between mb-10"
+    >
       <h1>Kompetenzprofil</h1>
       <!--      <ItDropdownSelect-->
       <!--        v-model="dropdownSelected"-->
@@ -60,14 +63,17 @@ const countStatus = computed(() => {
             class="py-4 flex flex-col lg:flex-row lg:items-center border-b border-gray-500 last:mb-8"
           >
             <p class="mb-4 lg:mb-0 lg:w-1/4 inline-block lg:mr-5">
-              {{ competence.title }}
+              {{ competence.competence_id }} {{ competence.title }}
             </p>
             <CompetenceProgress
               :status-count="competenceStore.calcStatusCount(competence.children)"
             ></CompetenceProgress>
           </li>
         </ul>
-        <router-link to="/mediacenter/handlungsfeldlist" class="flex items-center">
+        <router-link
+          :to="`${competenceStore.competenceProfilePage?.frontend_url}/competences`"
+          class="flex items-center"
+        >
           <span>Alle anschauen</span>
           <it-icon-arrow-right></it-icon-arrow-right>
         </router-link>
@@ -104,7 +110,10 @@ const countStatus = computed(() => {
           </div>
         </li>
       </ul>
-      <router-link to="/mediacenter/handlungsfeldlist" class="flex items-center">
+      <router-link
+        :to="`${competenceStore.competenceProfilePage?.frontend_url}/criteria`"
+        class="flex items-center"
+      >
         <span>Alle anschauen</span>
         <it-icon-arrow-right></it-icon-arrow-right>
       </router-link>
@@ -123,7 +132,10 @@ const countStatus = computed(() => {
           <PerformanceCriteriaRow :criteria="criteria"></PerformanceCriteriaRow>
         </li>
       </ul>
-      <router-link to="/mediacenter/handlungsfeldlist" class="flex items-center">
+      <router-link
+        :to="`${competenceStore.competenceProfilePage?.frontend_url}/criteria`"
+        class="flex items-center"
+      >
         <span>Alle anschauen</span>
         <it-icon-arrow-right></it-icon-arrow-right>
       </router-link>
diff --git a/client/src/pages/CompetencesDetailView.vue b/client/src/pages/competence/CompetenceListPage.vue
similarity index 88%
rename from client/src/pages/CompetencesDetailView.vue
rename to client/src/pages/competence/CompetenceListPage.vue
index 6aea6903..f8de78c9 100644
--- a/client/src/pages/CompetencesDetailView.vue
+++ b/client/src/pages/competence/CompetenceListPage.vue
@@ -11,7 +11,10 @@ const competenceStore = useCompetenceStore();
 <template>
   <div class="container-large">
     <nav class="lg:mt-4">
-      <a class="block mb-8 cursor-pointer flex items-center" @click="router.go(-1)">
+      <a
+        class="block mb-8 cursor-pointer flex items-center"
+        :href="competenceStore.competenceProfilePage?.frontend_url"
+      >
         <it-icon-arrow-left />
         <span>zurück</span></a
       >
diff --git a/client/src/pages/ComptencesView.vue b/client/src/pages/competence/CompetenceParentPage.vue
similarity index 93%
rename from client/src/pages/ComptencesView.vue
rename to client/src/pages/competence/CompetenceParentPage.vue
index 70406407..aea76a37 100644
--- a/client/src/pages/ComptencesView.vue
+++ b/client/src/pages/competence/CompetenceParentPage.vue
@@ -3,7 +3,7 @@ import { useCompetenceStore } from "@/stores/competence";
 import * as log from "loglevel";
 import { onMounted } from "vue";
 
-log.debug("CometencesView created");
+log.debug("CompetenceParentPage created");
 
 const props = defineProps<{
   competenceProfilePageSlug: string;
diff --git a/client/src/pages/CompetencePerfomanceCriteriaView.vue b/client/src/pages/competence/PerformanceCriteriaPage.vue
similarity index 93%
rename from client/src/pages/CompetencePerfomanceCriteriaView.vue
rename to client/src/pages/competence/PerformanceCriteriaPage.vue
index a22ca4ad..5fc30015 100644
--- a/client/src/pages/CompetencePerfomanceCriteriaView.vue
+++ b/client/src/pages/competence/PerformanceCriteriaPage.vue
@@ -20,8 +20,11 @@ const shownCriteria = computed(() => {
 
 <template>
   <div class="container-large">
-    <nav>
-      <a class="block mb-8 cursor-pointer flex items-center" @click="router.go(-1)">
+    <nav class="lg:mt-4">
+      <a
+        class="block mb-8 cursor-pointer flex items-center"
+        :href="`${competenceStore.competenceProfilePage?.frontend_url}`"
+      >
         <it-icon-arrow-left />
         <span>zurück</span></a
       >
diff --git a/client/src/router/index.ts b/client/src/router/index.ts
index 3609707b..9d799c65 100644
--- a/client/src/router/index.ts
+++ b/client/src/router/index.ts
@@ -55,19 +55,19 @@ const router = createRouter({
     {
       path: "/competence/:competenceProfilePageSlug",
       props: true,
-      component: () => import("@/pages/ComptencesView.vue"),
+      component: () => import("@/pages/competence/CompetenceParentPage.vue"),
       children: [
         {
           path: "",
-          component: () => import("@/pages/CompetencesMainView.vue"),
+          component: () => import("@/pages/competence/CompetenceIndexPage.vue"),
         },
         {
-          path: "detail",
-          component: () => import("@/pages/CompetencesDetailView.vue"),
+          path: "competences",
+          component: () => import("@/pages/competence/CompetenceListPage.vue"),
         },
         {
-          path: "performancecriteria",
-          component: () => import("@/pages/CompetencePerfomanceCriteriaView.vue"),
+          path: "criteria",
+          component: () => import("@/pages/competence/PerformanceCriteriaPage.vue"),
         },
       ],
     },
diff --git a/server/vbv_lernwelt/competence/create_default_competence_profile.py b/server/vbv_lernwelt/competence/create_default_competence_profile.py
index c4491463..242bb288 100644
--- a/server/vbv_lernwelt/competence/create_default_competence_profile.py
+++ b/server/vbv_lernwelt/competence/create_default_competence_profile.py
@@ -373,7 +373,7 @@ def create_default_competence_profile():
         competence_id="B4.2",
         title="Innerhalb des Handlungsfelds «Fahrzeug» bin ich fähig, dem Kunden die Vorschläge verständlich zu erläutern und die entsprechenden Informationspflichten zu erfüllen",
         learning_unit=LearningUnit.objects.get(
-            slug="versicherungsvermittlerin-lp-circle-analyse-lu-fahrzeug"
+            slug="versicherungsvermittlerin-lp-circle-lösung-lu-fahrzeug"
         ),
     )
     PerformanceCriteriaFactory(
@@ -381,7 +381,7 @@ def create_default_competence_profile():
         competence_id="B4.3",
         title="Innerhalb des Handlungsfelds «Fahrzeug» bin ich fähig, auf Vorbehalte und/oder Fragen sachlich korrekt und (verhandlungs-)sicher einzugehen.",
         learning_unit=LearningUnit.objects.get(
-            slug="versicherungsvermittlerin-lp-circle-analyse-lu-fahrzeug"
+            slug="versicherungsvermittlerin-lp-circle-lösung-lu-fahrzeug"
         ),
     )
     PerformanceCriteriaFactory(
@@ -389,7 +389,7 @@ def create_default_competence_profile():
         competence_id="B4.4",
         title="Innerhalb des Handlungsfelds «Fahrzeug» bin ich fähig, nötige Anpassungen flexibel vorzunehmen.",
         learning_unit=LearningUnit.objects.get(
-            slug="versicherungsvermittlerin-lp-circle-analyse-lu-fahrzeug"
+            slug="versicherungsvermittlerin-lp-circle-lösung-lu-fahrzeug"
         ),
     )
     PerformanceCriteriaFactory(
@@ -397,7 +397,7 @@ def create_default_competence_profile():
         competence_id="C1.1",
         title="Innerhalb des Handlungsfelds «Fahrzeug» bin ich fähig durch eine Bestandesaufnahme der aktuellen Policen zu prüfen, ob die Leistungen dem Bedarf des Kunden entsprechen",
         learning_unit=LearningUnit.objects.get(
-            slug="versicherungsvermittlerin-lp-circle-analyse-lu-fahrzeug"
+            slug="versicherungsvermittlerin-lp-circle-lösung-lu-fahrzeug"
         ),
     )
     PerformanceCriteriaFactory(
@@ -405,7 +405,7 @@ def create_default_competence_profile():
         competence_id="C1.2",
         title="Innerhalb des Handlungsfelds «Fahrzeug» bin ich fähig, den Kunden bedarfsgerechte Vorschläge für Anpassungen der Versicherungslösung zu unterbreiten (Up-Selling).",
         learning_unit=LearningUnit.objects.get(
-            slug="versicherungsvermittlerin-lp-circle-analyse-lu-fahrzeug"
+            slug="versicherungsvermittlerin-lp-circle-lösung-lu-fahrzeug"
         ),
     )
 
@@ -415,7 +415,7 @@ def create_default_competence_profile():
         competence_id="B3.2",
         title="Innerhalb des Handlungsfelds «Reisen» bin ich fähig, eine Unterversicherung, eine Doppel- oder Überversicherung, einen fehlenden Versicherungsschutz und mögliches Optimierungspotential festzustellen.",
         learning_unit=LearningUnit.objects.get(
-            slug="versicherungsvermittlerin-lp-circle-analyse-lu-reisen"
+            slug="versicherungsvermittlerin-lp-circle-lösung-lu-reisen"
         ),
     )
     PerformanceCriteriaFactory(
@@ -423,7 +423,7 @@ def create_default_competence_profile():
         competence_id="B4.2",
         title="Innerhalb des Handlungsfelds «Reisen» bin ich fähig, dem Kunden die Vorschläge verständlich zu erläutern und die entsprechenden Informationspflichten zu erfüllen.",
         learning_unit=LearningUnit.objects.get(
-            slug="versicherungsvermittlerin-lp-circle-analyse-lu-reisen"
+            slug="versicherungsvermittlerin-lp-circle-lösung-lu-reisen"
         ),
     )
     PerformanceCriteriaFactory(
@@ -431,7 +431,7 @@ def create_default_competence_profile():
         competence_id="B4.3",
         title="Innerhalb des Handlungsfelds «Reisen» bin ich fähig, auf Vorbehalte und/oder Fragen sachlich korrekt und (verhandlungs-)sicher einzugehen.",
         learning_unit=LearningUnit.objects.get(
-            slug="versicherungsvermittlerin-lp-circle-analyse-lu-reisen"
+            slug="versicherungsvermittlerin-lp-circle-lösung-lu-reisen"
         ),
     )
     PerformanceCriteriaFactory(
@@ -439,7 +439,7 @@ def create_default_competence_profile():
         competence_id="B4.4",
         title="Innerhalb des Handlungsfelds «Reisen» bin ich fähig, nötige Anpassungen flexibel vorzunehmen.",
         learning_unit=LearningUnit.objects.get(
-            slug="versicherungsvermittlerin-lp-circle-analyse-lu-reisen"
+            slug="versicherungsvermittlerin-lp-circle-lösung-lu-reisen"
         ),
     )
     PerformanceCriteriaFactory(
@@ -447,7 +447,7 @@ def create_default_competence_profile():
         competence_id="C1.2",
         title="Innerhalb des Handlungsfelds «Reisen» bin ich fähig, den Kunden bedarfsgerechte Vorschläge für Anpassungen der Versicherungslösung zu unterbreiten (Up-Selling).",
         learning_unit=LearningUnit.objects.get(
-            slug="versicherungsvermittlerin-lp-circle-analyse-lu-reisen"
+            slug="versicherungsvermittlerin-lp-circle-lösung-lu-reisen"
         ),
     )
     PerformanceCriteriaFactory(
@@ -455,7 +455,7 @@ def create_default_competence_profile():
         competence_id="C1.3",
         title="Innerhalb des Handlungsfelds «Reisen» bin ich fähig aufgrund des Portfolios passende Zusatzprodukte anzubieten (Cross-Selling).",
         learning_unit=LearningUnit.objects.get(
-            slug="versicherungsvermittlerin-lp-circle-analyse-lu-reisen"
+            slug="versicherungsvermittlerin-lp-circle-lösung-lu-reisen"
         ),
     )
 
@@ -465,7 +465,7 @@ def create_default_competence_profile():
         competence_id="A1.2",
         title="Innerhalb des Handlungsfelds «Fahrzeug» bin ich fähig, geeignete Personen wie z.B. Garagisten, Architekten, Treuhänder auf die Vermittlung/Zusammenarbeit anzusprechen.",
         learning_unit=LearningUnit.objects.get(
-            slug="versicherungsvermittlerin-lp-circle-analyse-lu-fahrzeug"
+            slug="versicherungsvermittlerin-lp-circle-abschluss-lu-fahrzeug"
         ),
     )
     PerformanceCriteriaFactory(
@@ -473,7 +473,7 @@ def create_default_competence_profile():
         competence_id="A4.1",
         title="Innerhalb des Handlungsfelds «Fahrzeug» bin ich fähig, Kundendaten in Datenbanken (CRM) korrekt zu erfassen.",
         learning_unit=LearningUnit.objects.get(
-            slug="versicherungsvermittlerin-lp-circle-analyse-lu-fahrzeug"
+            slug="versicherungsvermittlerin-lp-circle-abschluss-lu-fahrzeug"
         ),
     )
     PerformanceCriteriaFactory(
@@ -481,7 +481,7 @@ def create_default_competence_profile():
         competence_id="B4.6",
         title="Innerhalb des Handlungsfelds «Fahrzeug» bin ich fähig, Anträge korrekt auszufüllen.",
         learning_unit=LearningUnit.objects.get(
-            slug="versicherungsvermittlerin-lp-circle-analyse-lu-fahrzeug"
+            slug="versicherungsvermittlerin-lp-circle-abschluss-lu-fahrzeug"
         ),
     )
     PerformanceCriteriaFactory(
@@ -489,7 +489,7 @@ def create_default_competence_profile():
         competence_id="C1.3",
         title="Innerhalb des Handlungsfelds «Fahrzeug» bin ich fähig, aufgrund des Portfolios passende Zusatzprodukte anzubieten (Cross-Selling).",
         learning_unit=LearningUnit.objects.get(
-            slug="versicherungsvermittlerin-lp-circle-analyse-lu-fahrzeug"
+            slug="versicherungsvermittlerin-lp-circle-abschluss-lu-fahrzeug"
         ),
     )
 
@@ -499,7 +499,7 @@ def create_default_competence_profile():
         competence_id="B4.6",
         title="Innerhalb des Handlungsfelds «Reisen» bin ich fähig, Anträge korrekt auszufüllen.",
         learning_unit=LearningUnit.objects.get(
-            slug="versicherungsvermittlerin-lp-circle-analyse-lu-reisen"
+            slug="versicherungsvermittlerin-lp-circle-abschluss-lu-reisen"
         ),
     )
     PerformanceCriteriaFactory(
@@ -507,6 +507,6 @@ def create_default_competence_profile():
         competence_id="C3.1",
         title="Innerhalb des Handlungsfelds «Reisen» bin ich fähig, Kunden die Vorgehensweise für die Meldung des Schadens nachvollziehbar zu erläutern und sie bei Bedarf zu unterstützen.",
         learning_unit=LearningUnit.objects.get(
-            slug="versicherungsvermittlerin-lp-circle-analyse-lu-reisen"
+            slug="versicherungsvermittlerin-lp-circle-abschluss-lu-reisen"
         ),
     )
diff --git a/server/vbv_lernwelt/competence/models.py b/server/vbv_lernwelt/competence/models.py
index 370a70bd..ce1b4162 100644
--- a/server/vbv_lernwelt/competence/models.py
+++ b/server/vbv_lernwelt/competence/models.py
@@ -23,6 +23,9 @@ class CompetenceProfilePage(Page):
         )
         super(CompetenceProfilePage, self).full_clean(*args, **kwargs)
 
+    def get_frontend_url(self):
+        return f"/competence/{self.slug}"
+
     @classmethod
     def get_serializer_class(cls):
         return get_it_serializer_class(