refactor: some methods to computed values

Author: kevinkosterr

src/formGroup.vue

@@ -1,7 +1,6 @@
 <template>
   <div class="form-group" :class="getFieldRowClasses(field)">
-    <label v-if="fieldTypeHasLabel(field)" :for="getFieldID(field)" :class="field.labelClasses">
-      <span>{{ field.label }}</span>
+    <label v-if="hasLabel" :for="fieldId" :class="field.labelClasses">
       <span>
         <i v-if="hasIconBefore" :class="field.labelIcon.iconClass" />
         {{ field.label }}
@@ -91,6 +90,29 @@ export default {
     hasIconAfter () {
       return this.field.labelIcon && this.field.labelIcon.iconClass && this.field.labelIcon.position === 'after'
     },
+    hasLabel() {
+      if (isNil(this.field.label)) return false
+
+      let relevantType = ''
+      if (this.field.type === 'input') {
+        relevantType = this.field.inputType
+      } else {
+        relevantType = this.field.type
+      }
+
+      switch (relevantType) {
+        case 'button':
+        case 'submit':
+        case 'reset':
+          return false
+        default:
+          return true
+      }
+    },
+    fieldId () {
+      const idPrefix = this.options?.fieldIdPrefix ?? ''
+      return slugifyFormID(this.field, idPrefix)
+    }
   },
   methods: {
     onBlur (newValue, schema) {
@@ -110,29 +132,6 @@ export default {
       }
       return res
     },
-    fieldTypeHasLabel(field) {
-      if (isNil(field.label)) return false
-
-      let relevantType = ''
-      if (field.type === 'input') {
-        relevantType = field.inputType
-      } else {
-        relevantType = field.type
-      }
-
-      switch (relevantType) {
-        case 'button':
-        case 'submit':
-        case 'reset':
-          return false
-        default:
-          return true
-      }
-    },
-    getFieldID(schema) {
-      const idPrefix = objGet(this.options, 'fieldIdPrefix', '')
-      return slugifyFormID(schema, idPrefix)
-    },
     // Get type of field 'field-xxx'. It'll be the name of HTML element
     getFieldType(fieldSchema) {
       return 'field-' + fieldSchema.type