Adding validation for incident type

Netflix · Oct 23, 2024 · 14375e4 · 14375e4
1 parent ddb0de0
commit 14375e4
Show file tree

Hide file tree

Showing 3 changed files with 33 additions and 11 deletions.
diff --git a/src/dispatch/static/dispatch/src/case/type/CaseTypeSelect.vue b/src/dispatch/static/dispatch/src/case/type/CaseTypeSelect.vue
@@ -130,7 +130,7 @@ export default {
         }
       }
 
-      filterOptions = SearchUtils.createParametersFromTableOptions({ ...filterOptions })
+      filterOptions = SearchUtils.createParametersFromTableOptions({ ...filterOptions }, "CaseType")
 
       CaseTypeApi.getAll(filterOptions)
         .then((response) => {
@@ -170,6 +170,7 @@ export default {
 
   watch: {
     project() {
+      this.validateType()
       this.fetchData()
     },
   },

diff --git a/src/dispatch/static/dispatch/src/incident/priority/IncidentPrioritySelect.vue b/src/dispatch/static/dispatch/src/incident/priority/IncidentPrioritySelect.vue
@@ -131,6 +131,7 @@ export default {
 
   watch: {
     project() {
+      this.validatePriority()
       this.fetchData()
     },
     status() {

diff --git a/src/dispatch/static/dispatch/src/incident/type/IncidentTypeSelect.vue b/src/dispatch/static/dispatch/src/incident/type/IncidentTypeSelect.vue
@@ -7,7 +7,7 @@
     :label="label"
     return-object
     :loading="loading"
-    :rules="[validationRule]"
+    :rules="[is_type_in_project]"
   >
     <template #item="{ props, item }">
       <v-list-item v-bind="props" :title="null">
@@ -61,34 +61,51 @@ export default {
       numItems: 5,
       total: 0,
       lastProjectId: null,
+      error: null,
+      is_type_in_project: () => {
+        this.validateType()
+        return this.error
+      },
     }
   },
 
   computed: {
     selectedIncidentType: {
       get() {
-        return this.modelValue || null
+        if (!this.modelValue) return null
+        if (this.modelValue.id) {
+          return this.items.find((item) => item.id === this.modelValue.id) || null
+        }
+        // If we only have a name (e.g., from URL params), find by name
+        if (this.modelValue.name) {
+          return this.items.find((item) => item.name === this.modelValue.name) || null
+        }
+        return null
       },
       set(value) {
         this.$emit("update:modelValue", value)
+        this.validateType()
       },
     },
-    isTypeValid() {
-      const project_id = this.project?.id || 0
-      return this.selectedIncidentType?.project?.id == project_id
-    },
-    validationRule() {
-      return this.isTypeValid || "Only types in selected project are allowed"
-    },
   },
 
   watch: {
     project() {
+      this.validateType()
       this.fetchData()
     },
   },
 
   methods: {
+    validateType() {
+      const project_id = this.project?.id || 0
+      const in_project = this.selectedIncidentType?.project?.id == project_id
+      if (in_project) {
+        this.error = true
+      } else {
+        this.error = "Only types in selected project are allowed"
+      }
+    },
     clearSelection() {
       this.selectedIncidentType = null
     },
@@ -112,7 +129,10 @@ export default {
         }
       }
 
-      filterOptions = SearchUtils.createParametersFromTableOptions(filterOptions)
+      filterOptions = SearchUtils.createParametersFromTableOptions(
+        { ...filterOptions },
+        "IncidentType"
+      )
 
       IncidentTypeApi.getAll(filterOptions)
         .then((response) => {