WIP

app/pods/protected/strains/edit/controller.js

@@ -7,4 +7,12 @@ export default Controller.extend(SaveModel, {
   // Required for SaveModel mixin
   fallbackRouteSave: 'protected.strains.show',
   fallbackRouteCancel: 'protected.strains.show',
+
+  actions: {
+    addCharacteristic: function() {
+      return this.store.createRecord('measurement', {
+        characteristic: this.store.createRecord('characteristic', { sortOrder: -999 }),
+      });
+    },
+  },
 });

app/pods/protected/strains/edit/template.hbs

@@ -2,6 +2,7 @@
   protected/strains/strain-form
   strain=model
   speciesList=speciesList
+  add-characteristic=(action "addCharacteristic")
   on-save=(action "save")
   on-cancel=(action "cancel")
 }}

app/pods/protected/strains/measurements-table-row/component.js

@@ -1,11 +1,12 @@
 import Ember from 'ember';
-import ajaxError from '../../../../../utils/ajax-error';
+import ajaxError from '../../../../utils/ajax-error';
 
 const { Component } = Ember;
 
 export default Component.extend({
   tagName: 'tr',
   isEditing: false,
+  allCharacteristics: null,
 
   oldCharacteristicId: function() {
     let json = this.get('row').toJSON();

app/pods/protected/strains/measurements-table/component.js

@@ -1,48 +1,49 @@
 import Ember from 'ember';
 
-const { Component } = Ember;
+const { Component, computed } = Ember;
+const { sort } = computed;
 
 export default Component.extend({
-  measurementsPresent: function() {
+  // Passed in
-    return this.get('model.measurements.length') > 0;
+  strain: null,
-  }.property('model.measurements'),
+  allCharacteristics: null,
-
+  canEdit: false,
-  fetchCharacteristics: function() {
+  canAdd: false,
-    if (this.get('canEdit')) {
+  "add-characteristic": null,
-      this.store.findAll('characteristic');
-    }
-  }.on('didInsertElement'),
 
+  // Properties
   sortParams: ['characteristic.characteristicTypeName', 'characteristic.sortOrder', 'characteristic.characteristicName'],
   sortAsc: true,
   paramsChanged: false,
-  sortedMeasurements: Ember.computed.sort('model.measurements', 'sortParams'),
+  sortedMeasurements: sort('strain.measurements', 'sortParams'),
+  measurementsPresent: computed('strain.measurements', function() {
+    return this.get('strain.measurements.length') > 0;
+  }),
+
+  // TODO: remove this
+  // fetchCharacteristics: function() {
+  //   if (this.get('canEdit')) {
+  //     this.store.findAll('characteristic');
+  //   }
+  // }.on('didInsertElement'),
 
   actions: {
     addCharacteristic: function() {
-      const c = this.store.createRecord('characteristic', {
+      const newChar = this.attrs['add-characteristic']();
-        sortOrder: -999
+      this.get('strain.measurements').addObject(newChar);
-      });
-      const m = this.store.createRecord('measurement', {
-        characteristic: c
-      });
-      this.get('model.measurements').addObject(m);
     },
 
     changeSortParam: function(col) {
-      let sort = this.get('sortAsc') ? 'asc' : 'desc';
+      const sort = this.get('sortAsc') ? 'asc' : 'desc';
-      let sortCol = `${col}:${sort}`;
+      this.set('sortParams', [`${col}:${sort}`]);
-      this.set('sortParams', [sortCol]);
       this.set('paramsChanged', true);
       this.toggleProperty('sortAsc');
-      return false;
     },
 
     resetSortParam: function() {
       this.set('sortParams', ['characteristic.characteristicTypeName', 'characteristic.sortOrder', 'characteristic.characteristicName']);
       this.set('paramsChanged', false);
       this.set('sortAsc', true);
-      return false;
     },
   },
 

app/pods/protected/strains/measurements-table/template.hbs

@@ -1,17 +1,17 @@
 {{#if canAdd}}
-<br>
+  <br>
-<button class="button-green smaller" {{action "addCharacteristic"}}>
+  <button class="button-green smaller" {{action "addCharacteristic"}}>
-  Add characteristic
+    Add characteristic
-</button>
+  </button>
-<br><br>
+  <br><br>
 {{/if}}
 
 {{#if measurementsPresent}}
-{{#if paramsChanged}}
+  {{#if paramsChanged}}
-  <button class="button-gray smaller" {{action 'resetSortParam'}}>
+    <button class="button-gray smaller" {{action 'resetSortParam'}}>
-    Reset sort
+      Reset sort
-  </button>
+    </button>
-{{/if}}
+  {{/if}}
 <table class="flakes-table">
   <colgroup>
     {{#if canEdit}}

app/pods/protected/strains/show/strain-card/template.hbs

@@ -75,7 +75,7 @@
         <dd>
           {{
             protected/strains/measurements-table
-            model=strain
+            strain=strain
             canEdit=false
             canAdd=false
           }}

app/pods/protected/strains/strain-form/component.js

@@ -14,6 +14,7 @@ export default Component.extend(SetupMetaData, {
   "on-save": null,
   "on-cancel": null,
   "on-update": null,
+  "add-characteristic": null,
 
   // Property mapping
   propertiesList: ['strainName', 'typeStrain', 'species', 'isolatedFrom', 'accessionNumbers', 'genbank', 'wholeGenomeSequence', 'notes'],
@@ -54,6 +55,10 @@ export default Component.extend(SetupMetaData, {
       return this.attrs['on-cancel']();
     },
 
+    addCharacteristic: function() {
+      return this.attrs['add-characteristic']();
+    },
+
     strainNameDidChange: function(value) {
       this.updateField('strainName', value);
     },

app/pods/protected/strains/strain-form/template.hbs

@@ -52,9 +52,10 @@
   <div>
     {{
       protected/strains/measurements-table
-      model=strain
+      strain=strain
+      add-characteristic=(action "addCharacteristic")
       canEdit=strain.canEdit
-      canAdd=canAdd
+      canAdd=metaData.canAdd
     }}
   </div>
   <br>