Mdas 516 sst management c optimierung frontend (#62)

* ♻️ Eingabe GueltigBis Datum nicht mehr verpflichtend. * ♻️ Eingabe GueltigBis Datum nicht mehr verpflichtend. * ♻️ Tooltips for list elements and Email format check implemented. Additional check date gueltigAb <= gueltigBis implemented in the frontend. * ♻️ Uniform naming cancel button. Back button to navigate to interface overview implemented. Width Zugewiesene Personen refactored/formated. * ♻️ Uniform naming cancel button. Back button to navigate to interface overview implemented. Width Zugewiesene Personen refactored/formated. * ♻️ Mark expired assignments. Set Anlage-/Aenderungsdatum readonly and remove tooltip. * ♻️ Fixing linter errors. * Release notes complemented * ♻️ Review notes refactored * ♻️ More review notes refactored
it-at-m · Oct 7, 2024 · e628f14 · e628f14
1 parent b3342e7
commit e628f14
Show file tree

Hide file tree

Showing 14 changed files with 224 additions and 85 deletions.
diff --git a/docs/src/releasenotes/README.md b/docs/src/releasenotes/README.md
@@ -2,6 +2,16 @@
 ## Sprint 17 (10.09.2024 - 01.10.2024)
 ## Hinzugefügt
 - Vue 3 und Vuetify 3 Hebung
+- SST-Management Optimierung (Frontend)
+    - Keine Eingabe GueltigBis Datum
+    - Tooltip
+    - Prüfung EMail Format
+    - Spaltenbreite
+    - Zurück Springen
+    - Abgelaufene Personen markieren
+    - Einheitliche Bennenung Abbrechen Button
+    - Anzeige Anlage-, Aenderungsdatum
+    - Prüfung GueltigAb < GueltigBis Datum
 
 ## Sprint 15 (30.07.2024 - 20.08.2024)
 ## Hinzugefügt

diff --git a/mobidam-sst-management-backend/src/main/java/de/muenchen/mobidam/domain/Zuordnung.java b/mobidam-sst-management-backend/src/main/java/de/muenchen/mobidam/domain/Zuordnung.java
@@ -75,8 +75,6 @@ public class Zuordnung extends BaseEntity {
     @FutureOrPresent
     private LocalDate gueltigAb;
 
-    @Column(nullable = false)
-    @NotNull
     @FutureOrPresent
     private LocalDate gueltigBis;
 

diff --git a/...-management-backend/src/main/java/de/muenchen/mobidam/domain/dtos/ZuordnungCreateDTO.java b/...-management-backend/src/main/java/de/muenchen/mobidam/domain/dtos/ZuordnungCreateDTO.java
@@ -52,6 +52,5 @@ public class ZuordnungCreateDTO {
     private String funktionsadresse;
     @NotNull
     private LocalDate gueltigAb;
-    @NotNull
     private LocalDate gueltigBis;
 }
diff --git a/...am-sst-management-backend/src/main/java/de/muenchen/mobidam/domain/dtos/ZuordnungDTO.java b/...am-sst-management-backend/src/main/java/de/muenchen/mobidam/domain/dtos/ZuordnungDTO.java
@@ -51,6 +51,5 @@ public class ZuordnungDTO {
     private String funktionsadresse;
     @NotNull
     private LocalDate gueltigAb;
-    @NotNull
     private LocalDate gueltigBis;
 }
diff --git a/...am-sst-management-backend/src/main/java/de/muenchen/mobidam/rest/ZuordnungController.java b/...am-sst-management-backend/src/main/java/de/muenchen/mobidam/rest/ZuordnungController.java
@@ -28,10 +28,7 @@
 import de.muenchen.mobidam.service.ZuordnungService;
 import io.swagger.v3.oas.annotations.Operation;
 import jakarta.validation.Valid;
-import java.util.ArrayList;
-import java.util.Comparator;
-import java.util.List;
-import java.util.Optional;
+import java.util.*;
 import lombok.AllArgsConstructor;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
@@ -71,7 +68,8 @@ public ResponseEntity<Iterable<ZuordnungDTO>> getAllById(@PathVariable String id
         List<ZuordnungDTO> personDTOList = new ArrayList<>();
         zuordnungService.getAllById(id)
                 .forEach(task -> personDTOList.add(zuordnungMapper.toDTO(task)));
-        personDTOList.sort(Comparator.comparing(ZuordnungDTO::getGueltigBis));
+        personDTOList.sort(
+                Comparator.comparing(ZuordnungDTO::getGueltigBis, Comparator.nullsLast(Comparator.naturalOrder())).thenComparing(ZuordnungDTO::getGueltigAb));
         return new ResponseEntity<>(personDTOList, HttpStatus.OK);
     }
 

diff --git a/...ckend/src/main/java/de/muenchen/mobidam/validators/StartDateNotAfterEndDateValidator.java b/...ckend/src/main/java/de/muenchen/mobidam/validators/StartDateNotAfterEndDateValidator.java
@@ -62,6 +62,8 @@ public boolean isValid(final Object o, final ConstraintValidatorContext constrai
 
             final Method endDateGetter = o.getClass().getMethod(endDateFieldName);
             final LocalDate endDate = (LocalDate) endDateGetter.invoke(o);
+            if (startDate != null && endDate == null)
+                return true;
             if (endDate.isBefore(startDate)) {
                 throw new ValidationException(message);
             } else {

diff --git a/...agement-backend/src/test/java/de/muenchen/mobidam/repository/ZuordnungRepositoryTest.java b/...agement-backend/src/test/java/de/muenchen/mobidam/repository/ZuordnungRepositoryTest.java
@@ -82,4 +82,30 @@ void testSave() {
         assertEquals(original, persisted);
     }
 
+    @Test
+    @Transactional(propagation = Propagation.REQUIRED, noRollbackFor = Exception.class)
+    void testSaveGueltigBisNull() {
+
+        // initialize
+        Zuordnung original = new Zuordnung();
+        Schnittstelle schnittstelle = new Schnittstelle();
+        schnittstelle.setName("test");
+        schnittstelle.setAnlagedatum(LocalDate.now());
+        schnittstelle.setStatus(SchnittstellenStatus.AKTIVIERT);
+        schnittstelle.setId(UUID.randomUUID());
+        original.setSchnittstelle(schnittstelle);
+        original.setBenutzerkennung(UUID.randomUUID().toString());
+        original.setFachbereich("dep");
+        original.setFunktionsadresse("adr");
+        original.setGueltigAb(LocalDate.now());
+
+        // persist
+        original = repository.save(original);
+
+        // check
+        Zuordnung persisted = repository.findById(original.getId()).orElse(null);
+        assertNotNull(persisted);
+        assertEquals(original, persisted);
+    }
+
 }
diff --git a/...nt-backend/src/test/java/de/muenchen/mobidam/validators/StartDateNotAfterEndDateTest.java b/...nt-backend/src/test/java/de/muenchen/mobidam/validators/StartDateNotAfterEndDateTest.java
@@ -60,15 +60,15 @@ void testIsValidWithValidObjects() {
     }
 
     @Test
-    void testIsValidWithInvalidObjectThrowingExceptionInValidator() {
+    void testIsValidWithValidObjectsEndDateNull() {
         final LocalDate startDate = LocalDate.now();
         final LocalDate endDate = null;
 
         final TestClass testObject = new TestClass(
                 startDate,
                 endDate);
 
-        assertThrows(Exception.class, () -> validator.validate(testObject));
+        assertTrue(validator.validate(testObject).isEmpty());
     }
 
     @Test

diff --git a/mobidam-sst-management-frontend/frontend/src/components/AddPersonDialog.vue b/mobidam-sst-management-frontend/frontend/src/components/AddPersonDialog.vue
@@ -42,7 +42,7 @@
                     <v-text-field
                         ref="person"
                         v-model="zuordnung.benutzerkennung"
-                        label="Person angeben"
+                        label="Benutzerkennung"
                         hint="Welche Person soll der
                     Schnittstelle zugewiesen werden?"
                         :rules="textInputRules"
@@ -61,12 +61,13 @@
                     <v-text-field
                         ref="address"
                         v-model="zuordnung.funktionsadresse"
-                        label="Funktionsadresse"
-                        hint="Welchem Gruppenpostfach gehört diese Person an?"
+                        label="Funktionspostfach"
+                        hint="Wie lautet das Funktionspostfach?"
+                        placeholder="[Ändere mich]@muenchen.de"
                         :counter="textMaxLength"
                         :maxlength="textMaxLength"
                         :minlength="textMinLength"
-                        :rules="textInputRules"
+                        :rules="emailInputRules"
                     ></v-text-field>
                     &nbsp;
                     <v-row>
@@ -112,8 +113,15 @@
                                         label="Gültig bis"
                                         readonly
                                         variant="outlined"
-                                        :rules="textInputRules"
+                                        clearable
                                         v-bind="props"
+                                        :rules="[
+                                            validationRules.isGueltigAbBeforeGueltigBis(
+                                                zuordnung.gueltigAb,
+                                                zuordnung.gueltigBis,
+                                                'Gültig-ab muss kleiner gleich Gültig-bis sein.'
+                                            ),
+                                        ]"
                                         @click="gueltigBisMenu = true"
                                     ></v-text-field>
                                 </template>
@@ -124,7 +132,7 @@
                                         color="primary"
                                         header-color="primary"
                                         :min="today"
-                                        @update:model-value="updateGueltiBis"
+                                        @update:model-value="updateGueltigBis"
                                     >
                                     </v-date-picker>
                                 </v-locale-provider>
@@ -134,7 +142,7 @@
                     &nbsp;
                     <v-divider class="divider"></v-divider>
                     <v-card-actions>
-                        <v-btn @click="closeDialog">Schließen</v-btn>
+                        <v-btn @click="closeDialog">Abbrechen</v-btn>
                         <v-spacer></v-spacer>
                         <v-btn
                             class="text-white"
@@ -170,6 +178,10 @@ const textInputRules = [
             " Zeichen lang sein."
     ),
 ];
+const emailInputRules = textInputRules.concat([
+    validationRules.isValidEmail("Kein gültiges EMail Format."),
+]);
+
 const gueltigBisMenu = ref(false);
 const gueltigBis = ref(new Date());
 const gueltigAbMenu = ref(false);
@@ -209,7 +221,7 @@ function closeDialog() {
     form.value?.reset();
 }
 
-function updateGueltiBis(date: Date) {
+function updateGueltigBis(date: Date) {
     gueltigBisMenu.value = false;
     zuordnung.value.gueltigBis = formatDate(date.toLocaleDateString());
 }

diff --git a/mobidam-sst-management-frontend/frontend/src/components/ManageSchnittstelleDialog.vue b/mobidam-sst-management-frontend/frontend/src/components/ManageSchnittstelleDialog.vue
@@ -107,7 +107,14 @@
                                 v-for="zuordnung in mutableZuordnungen"
                                 :key="zuordnung.benutzerkennung"
                                 closable
-                                style="margin-right: 1%"
+                                :style="[
+                                    'margin-right: 1%',
+                                    validationRules.isExpired(
+                                        zuordnung.gueltigBis
+                                    )
+                                        ? { color: 'red' }
+                                        : {},
+                                ]"
                                 @click:close="removeZuordnung(zuordnung)"
                             >
                                 {{ zuordnung.benutzerkennung }}

diff --git a/mobidam-sst-management-frontend/frontend/src/composables/rules.ts b/mobidam-sst-management-frontend/frontend/src/composables/rules.ts
@@ -27,19 +27,48 @@ export function useRules() {
             (value && value.length <= length) || message;
     }
 
-    function notEmptyDateRule(message = "error") {
+    function notEmptyRule(message = "error") {
         return (value: string | null | undefined) =>
-            (value && value.trim() != "-") || message;
+            (value && value.trim() != "") || message;
     }
 
-    function notEmptyRule(message = "error") {
+    function isValidEmail(message = "error") {
+        const emailPattern = /^[^\s@]+@[^\s@]+\.[^\s@]+$/;
         return (value: string | null | undefined) =>
-            (value && value.trim() != "") || message;
+            (value && emailPattern.test(value.trim())) || message;
+    }
+
+    function isExpired(toDate: string | undefined) {
+        if (!toDate) {
+            return false;
+        }
+        const today = new Date();
+        const to = new Date(toDate);
+        return to < today;
+    }
+
+    function isGueltigAbBeforeGueltigBis(
+        gueltigAb: string,
+        gueltigBis: string | number | Date | undefined,
+        message: string
+    ) {
+        if (!gueltigBis && !gueltigAb) {
+            return true;
+        }
+        if (!gueltigBis && gueltigAb) {
+            return true;
+        } else {
+            const to = new Date(gueltigBis ?? "");
+            const from = new Date(gueltigAb);
+            return from <= to || message;
+        }
     }
 
     return {
         maxLengthRule,
-        notEmptyDateRule,
         notEmptyRule,
+        isValidEmail,
+        isExpired,
+        isGueltigAbBeforeGueltigBis,
     };
 }
diff --git a/mobidam-sst-management-frontend/frontend/src/types/Zuordnung.ts b/mobidam-sst-management-frontend/frontend/src/types/Zuordnung.ts
@@ -28,7 +28,7 @@ export default class Zuordnung {
         public fachbereich: string,
         public funktionsadresse: string,
         public gueltigAb: string,
-        public gueltigBis: string,
+        public gueltigBis?: string,
         public id?: string
     ) {}
 }
diff --git a/mobidam-sst-management-frontend/frontend/src/views/MainView.vue b/mobidam-sst-management-frontend/frontend/src/views/MainView.vue
@@ -61,7 +61,8 @@
                     <v-tooltip location="top">
                         <template #activator="{ props }">
                             <v-col
-                                cols="2"
+                                cols="3"
+                                md="3"
                                 v-bind="props"
                             >
                                 {{ schnittstelle.name }}