From 66325ee3b7f7f22015a98bddb504924c4b0931cd Mon Sep 17 00:00:00 2001
From: hms243 <hms243@cornell.edu>
Date: Wed, 30 Nov 2022 12:43:43 -0500
Subject: [PATCH 1/4] [BI-1614] - Add Term Type to Ontology Term display and
 form creation/modification

---
 src/breeding-insight/model/Sort.ts           |  3 +-
 src/breeding-insight/model/Trait.ts          |  8 +++--
 src/breeding-insight/model/TraitSelector.ts  |  6 ++++
 src/components/ontology/OntologyTable.vue    | 13 ++++++++
 src/components/trait/TraitDetailPanel.vue    | 10 ++++++
 src/components/trait/forms/BaseTraitForm.vue | 34 +++++++++++++++++---
 6 files changed, 66 insertions(+), 8 deletions(-)

diff --git a/src/breeding-insight/model/Sort.ts b/src/breeding-insight/model/Sort.ts
index 4ae6b2898..fddd724a6 100644
--- a/src/breeding-insight/model/Sort.ts
+++ b/src/breeding-insight/model/Sort.ts
@@ -55,7 +55,8 @@ export enum OntologySortField {
   MethodDescription = 'methodDescription',
   ScaleClass = 'scaleClass',
   ScaleName = 'scaleName',
-  entityAttributeSortLabel = 'entityAttribute'
+  entityAttributeSortLabel = 'entityAttribute',
+  TermType = 'termType'
 }
 
 export class OntologySort {
diff --git a/src/breeding-insight/model/Trait.ts b/src/breeding-insight/model/Trait.ts
index 214f9ce8a..358210600 100644
--- a/src/breeding-insight/model/Trait.ts
+++ b/src/breeding-insight/model/Trait.ts
@@ -18,11 +18,13 @@
 import {ProgramObservationLevel} from "@/breeding-insight/model/ProgramObservationLevel";
 import {Method} from "@/breeding-insight/model/Method";
 import {Scale} from "@/breeding-insight/model/Scale";
+import {TermType} from "@/breeding-insight/model/TraitSelector";
 
 export class Trait {
   id?: string;
   traitName?: string;
   observationVariableName?: string;
+  termType?: TermType;
   programObservationLevel?: ProgramObservationLevel;
   entity?: string;
   attribute?: string;
@@ -39,6 +41,7 @@ export class Trait {
   constructor(id?: string,
               traitName?: string,
               observationVariableName?: string,
+              termType?: TermType,
               programObservationLevel?: ProgramObservationLevel,
               entity?: string,
               attribute?: string,
@@ -49,7 +52,7 @@ export class Trait {
               active?: boolean,
               tags?: string[],
               fullName?: string,
-              isDup?: boolean,
+              isDup?: boolean
               ) {
     this.id = id;
     this.traitName = traitName;
@@ -89,7 +92,7 @@ export class Trait {
 
   static assign(trait: Trait): Trait {
     return new Trait(trait.id, trait.traitName, trait.observationVariableName, trait.programObservationLevel, trait.entity, trait.attribute,
-        trait.traitDescription, trait.method, trait.scale, trait.synonyms, trait.active, trait.tags, trait.fullName, trait.isDup);
+        trait.traitDescription, trait.method, trait.scale, trait.synonyms, trait.active, trait.tags, trait.fullName, trait.isDup, trait.termType);
   }
 
   checkStringListEquals(list: string[] | undefined, otherList: string[] | undefined): boolean {
@@ -111,6 +114,7 @@ export class Trait {
       (this.traitName === trait.traitName) &&
       (this.observationVariableName === trait.observationVariableName) &&
       (this.fullName === trait.fullName) &&
+      (this.termType === trait.termType) &&
       (this.checkStringListEquals(this.synonyms, trait.synonyms)) &&
       (this.mainAbbreviation === trait.mainAbbreviation) &&
         (this.entity === trait.entity) &&
diff --git a/src/breeding-insight/model/TraitSelector.ts b/src/breeding-insight/model/TraitSelector.ts
index 9b651ac1b..b7101ff50 100644
--- a/src/breeding-insight/model/TraitSelector.ts
+++ b/src/breeding-insight/model/TraitSelector.ts
@@ -21,6 +21,12 @@ export enum TraitField {
   UPDATED_BY_USER_NAME = 'updatedByUserName'
 }
 
+export enum TermType {
+  PHENOTYPE = 'Phenotype',
+  GERM_ATTRIBUTE = 'Germplasm Attribute',
+  GERM_PASSPORT = 'Germplasm Passport'
+}
+
 export class TraitFilter {
   field?: TraitField;
   value?: string | number | boolean;
diff --git a/src/components/ontology/OntologyTable.vue b/src/components/ontology/OntologyTable.vue
index 804a5c7d8..1fdb28552 100644
--- a/src/components/ontology/OntologyTable.vue
+++ b/src/components/ontology/OntologyTable.vue
@@ -122,6 +122,18 @@
         >
           {{ data.observationVariableName }}
         </TableColumn>
+        <TableColumn
+            name="termType"
+            v-bind:label="'Term Type'"
+            v-bind:sortField="ontologySort.field"
+            v-bind:sortFieldLabel="termTypeSortLabel"
+            v-bind:sortable="true"
+            v-bind:sortOrder="ontologySort.order"
+            v-on:newSortColumn="$emit('newSortColumn', $event)"
+            v-on:toggleSortOrder="$emit('toggleSortOrder')"
+        >
+          {{ data.termType }}
+        </TableColumn>
         <TableColumn
           name="trait"
           v-bind:label="'Trait'"
@@ -314,6 +326,7 @@ export default class OntologyTable extends Vue {
   private scaleClassSortLabel: string = OntologySortField.ScaleClass;
   private unitSortLabel: string = OntologySortField.ScaleName;
   private entityAttributeSortLabel: string = OntologySortField.entityAttributeSortLabel;
+  private termTypeSortLabel: string = OntologySortField.TermType;
 
   // New trait form
   private newTraitActive: boolean = false;
diff --git a/src/components/trait/TraitDetailPanel.vue b/src/components/trait/TraitDetailPanel.vue
index baccaf3a5..45699d565 100644
--- a/src/components/trait/TraitDetailPanel.vue
+++ b/src/components/trait/TraitDetailPanel.vue
@@ -31,6 +31,16 @@
           <span class="is-size-7 mb-0">{{data.traitDescription}}</span>
         </div>
       </div>
+
+      <div v-if="data.termType" class="columns is-desktop pt-1 pl-3">
+        <div class="column is-one-third pt-0 pb-0 has-text-right-desktop">
+          <span class="has-text-weight-bold">Term Type</span>
+        </div>
+        <div class="column pt-0 pb-0">
+          <span class="is-size-7 mb-0">{{data.termType}}</span>
+        </div>
+      </div>
+
       <!-- just shows first abbreviation AKA main abbreviation and first synonym -->
       <template v-if="abbreviationsSynonymsString">
         <div class="columns is-desktop pt-1 pl-3">
diff --git a/src/components/trait/forms/BaseTraitForm.vue b/src/components/trait/forms/BaseTraitForm.vue
index 6da6d84ec..d3b03d858 100644
--- a/src/components/trait/forms/BaseTraitForm.vue
+++ b/src/components/trait/forms/BaseTraitForm.vue
@@ -14,6 +14,21 @@
       <label for="newTermActiveToggle" class="is-pulled-right">{{trait.active ? 'Active' : 'Archived'}}</label>
     </div>
 
+<!-- term type -->
+  <div class="column is-2">
+    <span class="is-pulled-right required new-term pb-2 pr-3">Term Type</span>
+  </div>
+  <div class="column new-term is-10">
+    <BasicSelectField
+        class="pb-2"
+        v-bind:selected-id="trait.termType"
+        v-bind:options="termTypes"
+        v-bind:field-name="'Term Type'"
+        v-bind:show-label="false"
+        v-on:input="setTermType($event)"
+    />
+  </div>
+
 <!--    term name-->
     <div class="column is-2">
       <span class="is-pulled-right required new-term pb-2 pr-3">Name</span>
@@ -265,22 +280,22 @@ import BasicInputField from "@/components/forms/BasicInputField.vue";
 import BasicSelectField from "@/components/forms/BasicSelectField.vue";
 import {Trait} from "@/breeding-insight/model/Trait";
 import {Method, MethodClass} from "@/breeding-insight/model/Method";
-import { Scale, DataType } from '@/breeding-insight/model/Scale';
-import { ProgramObservationLevel } from '@/breeding-insight/model/ProgramObservationLevel';
+import {DataType, Scale} from '@/breeding-insight/model/Scale';
+import {ProgramObservationLevel} from '@/breeding-insight/model/ProgramObservationLevel';
 import OrdinalTraitForm from "@/components/trait/forms/CategoryTraitForm.vue";
+import CategoryTraitForm from "@/components/trait/forms/CategoryTraitForm.vue";
 import TextTraitForm from "@/components/trait/forms/TextTraitForm.vue";
 import DateTraitForm from "@/components/trait/forms/DateTraitForm.vue";
 import DurationTraitForm from "@/components/trait/forms/DurationTraitForm.vue";
 import NumericalTraitForm from "@/components/trait/forms/NumericalTraitForm.vue";
-import CategoryTraitForm from "@/components/trait/forms/CategoryTraitForm.vue";
 import {TraitError} from "@/breeding-insight/model/errors/TraitError";
 import {ValidationError} from "@/breeding-insight/model/errors/ValidationError";
 import AutoCompleteField from "@/components/forms/AutoCompleteField.vue";
-import { StringFormatters } from '@/breeding-insight/utils/StringFormatters';
+import {StringFormatters} from '@/breeding-insight/utils/StringFormatters';
 import {Category} from "@/breeding-insight/model/Category";
-import {integer} from "vuelidate/lib/validators";
 import TagField from "@/components/forms/TagField.vue";
 import BaseFieldWrapper from "@/components/forms/BaseFieldWrapper.vue";
+import {TermType} from "@/breeding-insight/model/TraitSelector";
 
 @Component({
   components: {
@@ -323,6 +338,8 @@ export default class BaseTraitForm extends Vue {
   @Prop()
   tags?: string[];
 
+  private termTypes: TermType[] = Object.values(TermType);
+
   private methodHistory: {[key: string]: Method} = {};
   private scaleHistory: {[key: string]: Scale} = {};
   private lastCategoryType: string = '';
@@ -362,6 +379,9 @@ export default class BaseTraitForm extends Vue {
     if ((this.trait.scale) && (this.trait.scale.categories)) {
       this.categories = this.trait.scale.categories;
     }
+    if (!this.trait.termType) {
+      this.trait.termType = TermType.PHENOTYPE;
+    }
   }
 
   @Watch('trait', {deep: true})
@@ -529,6 +549,10 @@ export default class BaseTraitForm extends Vue {
     }
   }
 
+  setTermType(value: TermType) {
+    this.trait.termType = value;
+  }
+
   setFullName(value: string) {
     this.trait.fullName = value;
     this.trait.synonyms = this.trait.synonyms || [];

From 7814a6bbebc6911ed8cbaad94d63aa10f07e516f Mon Sep 17 00:00:00 2001
From: hms243 <hms243@cornell.edu>
Date: Wed, 30 Nov 2022 13:45:49 -0500
Subject: [PATCH 2/4] [BI-1614] - Fix attempt

---
 src/breeding-insight/model/Trait.ts | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/breeding-insight/model/Trait.ts b/src/breeding-insight/model/Trait.ts
index 358210600..df7458791 100644
--- a/src/breeding-insight/model/Trait.ts
+++ b/src/breeding-insight/model/Trait.ts
@@ -24,7 +24,6 @@ export class Trait {
   id?: string;
   traitName?: string;
   observationVariableName?: string;
-  termType?: TermType;
   programObservationLevel?: ProgramObservationLevel;
   entity?: string;
   attribute?: string;
@@ -37,11 +36,11 @@ export class Trait {
   tags?: string[] = [];
   fullName?: string;
   isDup?: boolean;
+  termType?: TermType;
 
   constructor(id?: string,
               traitName?: string,
               observationVariableName?: string,
-              termType?: TermType,
               programObservationLevel?: ProgramObservationLevel,
               entity?: string,
               attribute?: string,
@@ -52,7 +51,8 @@ export class Trait {
               active?: boolean,
               tags?: string[],
               fullName?: string,
-              isDup?: boolean
+              isDup?: boolean,
+              termType?: TermType
               ) {
     this.id = id;
     this.traitName = traitName;

From a1e3237e367a5d29b000c4b71219cf71f34e6382 Mon Sep 17 00:00:00 2001
From: hms243 <hms243@cornell.edu>
Date: Fri, 2 Dec 2022 13:42:17 -0500
Subject: [PATCH 3/4] [BI-1615] - Add termType to Trait DB Table

---
 src/breeding-insight/model/Trait.ts                 | 1 +
 src/breeding-insight/model/TraitSelector.ts         | 9 +++++----
 src/breeding-insight/utils/TraitStringFormatters.ts | 6 ++++++
 src/components/ontology/OntologyTable.vue           | 9 +++++----
 src/components/trait/TraitDetailPanel.vue           | 5 +++--
 5 files changed, 20 insertions(+), 10 deletions(-)

diff --git a/src/breeding-insight/model/Trait.ts b/src/breeding-insight/model/Trait.ts
index df7458791..28f8abbe0 100644
--- a/src/breeding-insight/model/Trait.ts
+++ b/src/breeding-insight/model/Trait.ts
@@ -88,6 +88,7 @@ export class Trait {
     }
     this.fullName = fullName;
     this.isDup = isDup;
+    this.termType = termType;
   }
 
   static assign(trait: Trait): Trait {
diff --git a/src/breeding-insight/model/TraitSelector.ts b/src/breeding-insight/model/TraitSelector.ts
index b7101ff50..098f6e29a 100644
--- a/src/breeding-insight/model/TraitSelector.ts
+++ b/src/breeding-insight/model/TraitSelector.ts
@@ -18,13 +18,14 @@ export enum TraitField {
   CREATED_BY_USER_ID = 'createdByUserId',
   CREATED_BY_USER_NAME = 'createdByUserName',
   UPDATED_BY_USER_ID = 'updatedByUserId',
-  UPDATED_BY_USER_NAME = 'updatedByUserName'
+  UPDATED_BY_USER_NAME = 'updatedByUserName',
+  TERM_TYPE = 'termType'
 }
 
 export enum TermType {
-  PHENOTYPE = 'Phenotype',
-  GERM_ATTRIBUTE = 'Germplasm Attribute',
-  GERM_PASSPORT = 'Germplasm Passport'
+  PHENOTYPE = 'PHENOTYPE',
+  GERM_ATTRIBUTE = 'GERM_ATTRIBUTE',
+  GERM_PASSPORT = 'GERM_PASSPORT'
 }
 
 export class TraitFilter {
diff --git a/src/breeding-insight/utils/TraitStringFormatters.ts b/src/breeding-insight/utils/TraitStringFormatters.ts
index 48a61c111..c221150d9 100644
--- a/src/breeding-insight/utils/TraitStringFormatters.ts
+++ b/src/breeding-insight/utils/TraitStringFormatters.ts
@@ -17,6 +17,7 @@
 
 import {Scale, DataType} from "@/breeding-insight/model/Scale";
 import {StringFormatters} from "@/breeding-insight/utils/StringFormatters";
+import {TermType} from "@/breeding-insight/model/TraitSelector";
 
 export class TraitStringFormatters {
   
@@ -32,4 +33,9 @@ export class TraitStringFormatters {
     return undefined;
   }
 
+  static getTermTypeString(termType: TermType): string | undefined {
+    return StringFormatters.toStartCase(termType);
+    //todo see if can be used to convert to more readable form
+    }
+
 }
\ No newline at end of file
diff --git a/src/components/ontology/OntologyTable.vue b/src/components/ontology/OntologyTable.vue
index 1fdb28552..20bc4c777 100644
--- a/src/components/ontology/OntologyTable.vue
+++ b/src/components/ontology/OntologyTable.vue
@@ -132,7 +132,7 @@
             v-on:newSortColumn="$emit('newSortColumn', $event)"
             v-on:toggleSortOrder="$emit('toggleSortOrder')"
         >
-          {{ data.termType }}
+          {{ TraitStringFormatters.getTermTypeString(data.termType) }}
         </TableColumn>
         <TableColumn
           name="trait"
@@ -250,7 +250,7 @@ import WarningModal from '@/components/modals/WarningModal.vue'
 import {PlusCircleIcon} from 'vue-feather-icons'
 import {validationMixin} from 'vuelidate';
 import {Trait} from '@/breeding-insight/model/Trait'
-import {mapGetters, mapActions} from 'vuex'
+import {mapActions, mapGetters} from 'vuex'
 import {Program} from "@/breeding-insight/model/Program";
 import NewDataForm from '@/components/forms/NewDataForm.vue'
 import BasicInputField from "@/components/forms/BasicInputField.vue";
@@ -271,8 +271,8 @@ import {DataType, Scale} from "@/breeding-insight/model/Scale";
 import {SidePanelTableEventBusHandler} from "@/components/tables/SidePanelTableEventBus";
 import {DataFormEventBusHandler} from '@/components/forms/DataFormEventBusHandler';
 import {integer, maxLength} from "vuelidate/lib/validators";
-import {TraitField, TraitFilter} from "@/breeding-insight/model/TraitSelector";
-import {OntologySort, OntologySortField, SortOrder, TraitSortField} from "@/breeding-insight/model/Sort";
+import {TermType, TraitField, TraitFilter} from "@/breeding-insight/model/TraitSelector";
+import {OntologySort, OntologySortField} from "@/breeding-insight/model/Sort";
 import {BackendPaginationController} from "@/breeding-insight/model/view_models/BackendPaginationController";
 import {Category} from "@/breeding-insight/model/Category";
 
@@ -623,6 +623,7 @@ export default class OntologyTable extends Vue {
   }
 
   prepareScaleCategoriesForSave(inputTrait: Trait){
+    //todo investigate means to convert term type
     let traitToSave = JSON.parse(JSON.stringify(inputTrait));
     if ((traitToSave) && (traitToSave.scale) && (traitToSave.scale.dataType) && (Scale.dataTypeEquals(traitToSave.scale.dataType, DataType.Nominal)) && (traitToSave.scale.categories)) {
       traitToSave.scale.categories.forEach((category: Category) => {
diff --git a/src/components/trait/TraitDetailPanel.vue b/src/components/trait/TraitDetailPanel.vue
index 45699d565..6a16fc9db 100644
--- a/src/components/trait/TraitDetailPanel.vue
+++ b/src/components/trait/TraitDetailPanel.vue
@@ -37,7 +37,7 @@
           <span class="has-text-weight-bold">Term Type</span>
         </div>
         <div class="column pt-0 pb-0">
-          <span class="is-size-7 mb-0">{{data.termType}}</span>
+          <span class="is-size-7 mb-0">{{TraitStringFormatters.getTermTypeString(data.termType)}}</span>
         </div>
       </div>
 
@@ -262,6 +262,7 @@
   import { DataFormEventBusHandler } from '@/components/forms/DataFormEventBusHandler';
   import { HelpCircleIcon } from 'vue-feather-icons'
   import ProgressBar from '@/components/forms/ProgressBar.vue'
+  import {TraitStringFormatters} from '@/breeding-insight/utils/TraitStringFormatters';
 
   @Component({
     components: {EditDataForm, SidePanel, BaseTraitForm, HelpCircleIcon, ProgressBar},
@@ -270,7 +271,7 @@
         'isSubscribed'
       ])
     },
-    data: () => ({DataType, MethodClass, Scale, Method, StringFormatters}),
+    data: () => ({DataType, MethodClass, Scale, Method, StringFormatters, TraitStringFormatters}),
     filters: {
       capitalize: function(value: string | undefined) : string | undefined {
         if (value === undefined) value = '';

From 3b9a58a9bd33400f86f8b288601b2103c80092e2 Mon Sep 17 00:00:00 2001
From: hms243 <hms243@cornell.edu>
Date: Mon, 5 Dec 2022 15:36:54 -0500
Subject: [PATCH 4/4] [BI-1615] - Enable conversion to user readable format

---
 src/breeding-insight/model/Trait.ts           |  6 ++--
 src/breeding-insight/model/TraitSelector.ts   |  6 ++--
 src/breeding-insight/utils/EnumUtils.ts       | 28 +++++++++++++++++++
 .../utils/TraitStringFormatters.ts            |  4 +--
 src/components/ontology/OntologyTable.vue     |  4 ++-
 src/components/trait/forms/BaseTraitForm.vue  |  6 ++--
 6 files changed, 44 insertions(+), 10 deletions(-)
 create mode 100644 src/breeding-insight/utils/EnumUtils.ts

diff --git a/src/breeding-insight/model/Trait.ts b/src/breeding-insight/model/Trait.ts
index 28f8abbe0..9b1ad4da1 100644
--- a/src/breeding-insight/model/Trait.ts
+++ b/src/breeding-insight/model/Trait.ts
@@ -36,7 +36,7 @@ export class Trait {
   tags?: string[] = [];
   fullName?: string;
   isDup?: boolean;
-  termType?: TermType;
+  termType: TermType =  TermType.PHENOTYPE; //Phenotype is default
 
   constructor(id?: string,
               traitName?: string,
@@ -88,7 +88,9 @@ export class Trait {
     }
     this.fullName = fullName;
     this.isDup = isDup;
-    this.termType = termType;
+    if (termType) {
+      this.termType = termType;
+    }
   }
 
   static assign(trait: Trait): Trait {
diff --git a/src/breeding-insight/model/TraitSelector.ts b/src/breeding-insight/model/TraitSelector.ts
index 098f6e29a..55e4db337 100644
--- a/src/breeding-insight/model/TraitSelector.ts
+++ b/src/breeding-insight/model/TraitSelector.ts
@@ -23,9 +23,9 @@ export enum TraitField {
 }
 
 export enum TermType {
-  PHENOTYPE = 'PHENOTYPE',
-  GERM_ATTRIBUTE = 'GERM_ATTRIBUTE',
-  GERM_PASSPORT = 'GERM_PASSPORT'
+  PHENOTYPE = 'Phenotype',
+  GERM_ATTRIBUTE = 'Germplasm Attribute',
+  GERM_PASSPORT = 'Germplasm Passport'
 }
 
 export class TraitFilter {
diff --git a/src/breeding-insight/utils/EnumUtils.ts b/src/breeding-insight/utils/EnumUtils.ts
new file mode 100644
index 000000000..58cb6082e
--- /dev/null
+++ b/src/breeding-insight/utils/EnumUtils.ts
@@ -0,0 +1,28 @@
+/*
+ * See the NOTICE file distributed with this work for additional information
+ * regarding copyright ownership.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+export class EnumUtils {
+    static enumKeyToValue(enumKey: string, enumType: Object){
+        let index = Object.keys(enumType).indexOf(enumKey);
+        return Object.values(enumType)[index];
+    }
+
+    static enumValueToKey(enumVal: string, enumType: Object){
+        let index = Object.values(enumType).indexOf(enumVal);
+        return Object.keys(enumType)[index];
+    }
+}
\ No newline at end of file
diff --git a/src/breeding-insight/utils/TraitStringFormatters.ts b/src/breeding-insight/utils/TraitStringFormatters.ts
index c221150d9..1ae08827a 100644
--- a/src/breeding-insight/utils/TraitStringFormatters.ts
+++ b/src/breeding-insight/utils/TraitStringFormatters.ts
@@ -18,6 +18,7 @@
 import {Scale, DataType} from "@/breeding-insight/model/Scale";
 import {StringFormatters} from "@/breeding-insight/utils/StringFormatters";
 import {TermType} from "@/breeding-insight/model/TraitSelector";
+import {EnumUtils} from "@/breeding-insight/utils/EnumUtils";
 
 export class TraitStringFormatters {
   
@@ -34,8 +35,7 @@ export class TraitStringFormatters {
   }
 
   static getTermTypeString(termType: TermType): string | undefined {
-    return StringFormatters.toStartCase(termType);
-    //todo see if can be used to convert to more readable form
+    return EnumUtils.enumKeyToValue(termType,TermType);
     }
 
 }
\ No newline at end of file
diff --git a/src/components/ontology/OntologyTable.vue b/src/components/ontology/OntologyTable.vue
index 20bc4c777..0ae2c148e 100644
--- a/src/components/ontology/OntologyTable.vue
+++ b/src/components/ontology/OntologyTable.vue
@@ -275,6 +275,7 @@ import {TermType, TraitField, TraitFilter} from "@/breeding-insight/model/TraitS
 import {OntologySort, OntologySortField} from "@/breeding-insight/model/Sort";
 import {BackendPaginationController} from "@/breeding-insight/model/view_models/BackendPaginationController";
 import {Category} from "@/breeding-insight/model/Category";
+import {EnumUtils} from "@/breeding-insight/utils/EnumUtils";
 
 @Component({
   mixins: [validationMixin],
@@ -623,7 +624,6 @@ export default class OntologyTable extends Vue {
   }
 
   prepareScaleCategoriesForSave(inputTrait: Trait){
-    //todo investigate means to convert term type
     let traitToSave = JSON.parse(JSON.stringify(inputTrait));
     if ((traitToSave) && (traitToSave.scale) && (traitToSave.scale.dataType) && (Scale.dataTypeEquals(traitToSave.scale.dataType, DataType.Nominal)) && (traitToSave.scale.categories)) {
       traitToSave.scale.categories.forEach((category: Category) => {
@@ -631,6 +631,8 @@ export default class OntologyTable extends Vue {
         category.label = undefined;
       });
     }
+    //Translate TermType from user readable to backend storage format
+    traitToSave.termType = EnumUtils.enumValueToKey(inputTrait.termType, TermType);
     return traitToSave;
   }
 
diff --git a/src/components/trait/forms/BaseTraitForm.vue b/src/components/trait/forms/BaseTraitForm.vue
index d3b03d858..0e53729af 100644
--- a/src/components/trait/forms/BaseTraitForm.vue
+++ b/src/components/trait/forms/BaseTraitForm.vue
@@ -296,6 +296,7 @@ import {Category} from "@/breeding-insight/model/Category";
 import TagField from "@/components/forms/TagField.vue";
 import BaseFieldWrapper from "@/components/forms/BaseFieldWrapper.vue";
 import {TermType} from "@/breeding-insight/model/TraitSelector";
+import {EnumUtils} from "@/breeding-insight/utils/EnumUtils";
 
 @Component({
   components: {
@@ -379,8 +380,9 @@ export default class BaseTraitForm extends Vue {
     if ((this.trait.scale) && (this.trait.scale.categories)) {
       this.categories = this.trait.scale.categories;
     }
-    if (!this.trait.termType) {
-      this.trait.termType = TermType.PHENOTYPE;
+    //If termType pulled from backend (rather than the default for new terms), set to display friendly version
+    if (this.trait.termType != TermType.PHENOTYPE) {
+      this.trait.termType = EnumUtils.enumKeyToValue(this.trait.termType, TermType);
     }
   }