From 2d7d22f8478a402bdaf3458c33f0f3d17c95c9ee Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Thu, 5 Nov 2015 17:42:50 -0700
Subject: [PATCH 01/35] Set up preliminary tests for users

---
 app/mirage/config.js                          |  3 +
 app/pods/protected/users/index/template.hbs   |  2 +-
 .../protected/users/user-form/template.hbs    |  4 +-
 app/pods/users/new/template.hbs               | 10 +--
 tests/acceptance/users-test.js                | 77 +++++++++++++++++++
 5 files changed, 88 insertions(+), 8 deletions(-)
 create mode 100644 tests/acceptance/users-test.js

diff --git a/app/mirage/config.js b/app/mirage/config.js
index 929ea8a..c03b469 100644
--- a/app/mirage/config.js
+++ b/app/mirage/config.js
@@ -10,7 +10,10 @@ export function testConfig() {
   this.namespace = '/api/hymenobacter';
   this.timing = 0;
 
+  this.get('/users');
+  this.post('/users');
   this.get('/users/:id');
+  this.put('/users/:id');
 
   this.get('/species');
   this.post('/species');
diff --git a/app/pods/protected/users/index/template.hbs b/app/pods/protected/users/index/template.hbs
index adb5b55..5354e95 100644
--- a/app/pods/protected/users/index/template.hbs
+++ b/app/pods/protected/users/index/template.hbs
@@ -1,5 +1,5 @@
 <h2>{{genus-name}} Users</h2>
-<h3>Total users: {{model.length}}</h3>
+<h3 id="total-users">Total users: {{model.length}}</h3>
 
 <table class="flakes-table">
   <thead>
diff --git a/app/pods/protected/users/user-form/template.hbs b/app/pods/protected/users/user-form/template.hbs
index c6209a9..e7a8b02 100644
--- a/app/pods/protected/users/user-form/template.hbs
+++ b/app/pods/protected/users/user-form/template.hbs
@@ -4,7 +4,7 @@
     <div data-row-span="1">
       <div data-field-span="1">
         <label>Name</label>
-        {{input value=user.name}}
+        {{input value=user.name class="user-name"}}
       </div>
     </div>
     <div data-row-span="1">
@@ -33,7 +33,7 @@
     Cancel
   </a>
   {{#if user.hasDirtyAttributes}}
-    <button type="submit" class="button-green smaller">
+    <button type="submit" class="button-green smaller save-user">
       Save
     </button>
   {{/if}}
diff --git a/app/pods/users/new/template.hbs b/app/pods/users/new/template.hbs
index d6e81c5..e8e6e2c 100644
--- a/app/pods/users/new/template.hbs
+++ b/app/pods/users/new/template.hbs
@@ -6,22 +6,22 @@
         <ul>
           <li>
             <label>Name</label>
-            {{input value=user.name}}
+            {{input value=user.name class="user-name"}}
           </li>
           <li>
             <label>Email</label>
-            {{input value=user.email}}
+            {{input value=user.email class="email"}}
           </li>
           <li>
             <label>Password</label>
-            {{input type="password" value=user.password}}
+            {{input type="password" value=user.password class="password"}}
           </li>
           <li>
             <label>Password (confirm)</label>
-            {{input type="password" value=passwordConfirm}}
+            {{input type="password" value=passwordConfirm class="password-verify"}}
           </li>
           <li>
-            <button type="submit" class="button-green smaller">
+            <button type="submit" class="button-green smaller save-user">
               Submit
             </button>
           </li>
diff --git a/tests/acceptance/users-test.js b/tests/acceptance/users-test.js
new file mode 100644
index 0000000..ed08d72
--- /dev/null
+++ b/tests/acceptance/users-test.js
@@ -0,0 +1,77 @@
+import Ember from 'ember';
+import { module, test } from 'qunit';
+import startApp from '../helpers/start-app';
+import { invalidateSession, authenticateSession } from '../helpers/ember-simple-auth';
+
+module('Acceptance | users', {
+  beforeEach: function() {
+    this.application = startApp();
+    authenticateSession(this.application, {
+      access_token: "eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJiYWN0ZGIiLCJzdWIiOiIxIiwiZXhwIjoxNDQ2NTAyMjI2LCJpYXQiOjE0NDY0OTg2MjZ9.vIjKHAsp2TkCV505EbtCo2xQT-2oQkB-Nv5y0b6E7Mg"
+    });
+    server.create('users', { role: 'A', canEdit: true });
+  },
+
+  afterEach: function() {
+    Ember.run(this.application, 'destroy');
+  }
+});
+
+test('visiting /users', function(assert) {
+  const users = server.createList('users', 19); // We already created one user in beforeEach
+  visit('/users');
+
+  andThen(function() {
+    assert.equal(currentURL(), '/users');
+    assert.equal(find(".flakes-table > tbody > tr").length, users.length + 1);
+    assert.equal(find("#total-users").text(), "Total users: 20");
+  });
+});
+
+
+test('visiting /users/:id', function(assert) {
+  const user = server.create('users');
+  visit(`/users/${user.id}`);
+
+  andThen(function() {
+    assert.equal(currentURL(), `/users/${user.id}`);
+    assert.equal(find(".flakes-information-box > legend").text().trim(), user.name);
+  });
+});
+
+test('editing /users/:id/edit', function(assert) {
+  const user = server.create('users', { 'canEdit': true });
+  visit(`/users/${user.id}/edit`);
+
+  andThen(function() {
+    assert.equal(currentURL(), `/users/${user.id}/edit`);
+
+    fillIn('.user-name', 'Revised User Name');
+    click('.save-user');
+
+    andThen(function() {
+      assert.equal(currentURL(), `/users/${user.id}`);
+      assert.equal(find(".flakes-information-box > legend").text().trim(), 'Revised User Name');
+    });
+  });
+});
+
+test('creating /users/new', function(assert) {
+  invalidateSession(this.application);
+  visit(`/users/new`);
+
+  andThen(function() {
+    assert.equal(currentURL(), `/users/new`);
+    fillIn('.user-name', 'New User Name');
+    fillIn('.email', 'example@example.com');
+    fillIn('.password', 'Password1');
+    fillIn('.password-verify', 'Password1');
+    click('.save-user');
+
+    andThen(function() {
+      assert.equal(currentURL(), `/login`);
+      assert.equal(find(".flakes-message").text().trim(), `✖ You have successfully signed up.
+              Please check your email for further instructions.`);
+    });
+  });
+});

From e6510992dc0ecef7ef3bf46972fa939d2a829571 Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Thu, 5 Nov 2015 17:48:58 -0700
Subject: [PATCH 02/35] Refactor users/index

---
 app/pods/protected/users/index/route.js       |  6 ++--
 app/pods/protected/users/index/template.hbs   | 35 +++----------------
 .../users/index/users-table/component.js      |  7 ++++
 .../users/index/users-table/template.hbs      | 33 +++++++++++++++++
 app/pods/protected/users/loading/template.hbs |  1 +
 5 files changed, 49 insertions(+), 33 deletions(-)
 create mode 100644 app/pods/protected/users/index/users-table/component.js
 create mode 100644 app/pods/protected/users/index/users-table/template.hbs
 create mode 100644 app/pods/protected/users/loading/template.hbs

diff --git a/app/pods/protected/users/index/route.js b/app/pods/protected/users/index/route.js
index 8429726..2752983 100644
--- a/app/pods/protected/users/index/route.js
+++ b/app/pods/protected/users/index/route.js
@@ -1,7 +1,9 @@
 import Ember from 'ember';
 
-export default Ember.Route.extend({
-  currentUser: Ember.inject.service('session-account'),
+const { Route, inject: { service } } = Ember;
+
+export default Route.extend({
+  currentUser: service('session-account'),
 
   beforeModel: function(transition) {
     this._super(transition);
diff --git a/app/pods/protected/users/index/template.hbs b/app/pods/protected/users/index/template.hbs
index 5354e95..f75072a 100644
--- a/app/pods/protected/users/index/template.hbs
+++ b/app/pods/protected/users/index/template.hbs
@@ -1,33 +1,6 @@
 <h2>{{genus-name}} Users</h2>
-<h3 id="total-users">Total users: {{model.length}}</h3>
 
-<table class="flakes-table">
-  <thead>
-    <tr>
-      <th>Name</th>
-      <th>Email</th>
-      <th>Role</th>
-      <th>Date Registered</th>
-    </tr>
-  </thead>
-  <tbody>
-    {{#each model as |row|}}
-      <tr>
-        <td>
-          {{#link-to 'protected.users.show' row}}
-            {{row.name}}
-          {{/link-to}}
-        </td>
-        <td>
-          {{row.email}}
-        </td>
-        <td>
-          {{row.fullRole}}
-        </td>
-        <td>
-          {{null-time row.createdAt 'LL'}}
-        </td>
-      </tr>
-    {{/each}}
-  </tbody>
-</table>
+{{
+  protected/users/index/users-table
+  users=model
+}}
diff --git a/app/pods/protected/users/index/users-table/component.js b/app/pods/protected/users/index/users-table/component.js
new file mode 100644
index 0000000..2abad45
--- /dev/null
+++ b/app/pods/protected/users/index/users-table/component.js
@@ -0,0 +1,7 @@
+import Ember from 'ember';
+
+const { Component } = Ember;
+
+export default Component.extend({
+  users: null,
+});
diff --git a/app/pods/protected/users/index/users-table/template.hbs b/app/pods/protected/users/index/users-table/template.hbs
new file mode 100644
index 0000000..ad2a064
--- /dev/null
+++ b/app/pods/protected/users/index/users-table/template.hbs
@@ -0,0 +1,33 @@
+
+<h3 id="total-users">Total users: {{users.length}}</h3>
+
+<table class="flakes-table">
+  <thead>
+    <tr>
+      <th>Name</th>
+      <th>Email</th>
+      <th>Role</th>
+      <th>Date Registered</th>
+    </tr>
+  </thead>
+  <tbody>
+    {{#each users as |user|}}
+      <tr>
+        <td>
+          {{#link-to 'protected.users.show' user}}
+            {{user.name}}
+          {{/link-to}}
+        </td>
+        <td>
+          {{user.email}}
+        </td>
+        <td>
+          {{user.fullRole}}
+        </td>
+        <td>
+          {{null-time user.createdAt 'LL'}}
+        </td>
+      </tr>
+    {{/each}}
+  </tbody>
+</table>
diff --git a/app/pods/protected/users/loading/template.hbs b/app/pods/protected/users/loading/template.hbs
new file mode 100644
index 0000000..e5a3e05
--- /dev/null
+++ b/app/pods/protected/users/loading/template.hbs
@@ -0,0 +1 @@
+{{loading-panel}}

From b742ddbb518c7fbb6b170edc07b584a6c9920eb8 Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Thu, 5 Nov 2015 20:24:07 -0700
Subject: [PATCH 03/35] refactored users/show

---
 app/pods/protected/users/show/controller.js   | 10 ++--
 app/pods/protected/users/show/route.js        | 16 +++--
 app/pods/protected/users/show/template.hbs    | 58 ++-----------------
 .../users/show/user-card/component.js         | 11 ++++
 .../users/show/user-card/template.hbs         | 53 +++++++++++++++++
 5 files changed, 84 insertions(+), 64 deletions(-)
 create mode 100644 app/pods/protected/users/show/user-card/component.js
 create mode 100644 app/pods/protected/users/show/user-card/template.hbs

diff --git a/app/pods/protected/users/show/controller.js b/app/pods/protected/users/show/controller.js
index 3c6c549..26c243d 100644
--- a/app/pods/protected/users/show/controller.js
+++ b/app/pods/protected/users/show/controller.js
@@ -1,9 +1,9 @@
 import Ember from 'ember';
+import DeleteModel from '../../../../mixins/delete-model';
 
-export default Ember.Controller.extend({
-  currentUser: Ember.inject.service('session-account'),
+const { Controller } = Ember;
 
-  isUser: Ember.computed('model.id', 'currentUser.account.id', function() {
-    return this.get('model.id') === this.get('currentUser.account.id');
-  }),
+export default Controller.extend(DeleteModel, {
+  // Required for DeleteModel mixin
+  transitionRoute: 'protected.index',
 });
diff --git a/app/pods/protected/users/show/route.js b/app/pods/protected/users/show/route.js
index a0c9fd9..798787d 100644
--- a/app/pods/protected/users/show/route.js
+++ b/app/pods/protected/users/show/route.js
@@ -1,21 +1,25 @@
 import Ember from 'ember';
 
-export default Ember.Route.extend({
-  currentUser: Ember.inject.service('session-account'),
+const { Route, inject: { service } } = Ember;
 
+export default Route.extend({
+  currentUser: service('session-account'),
+
+  // Not using ElevatedAccess Mixin because the rules for viewing user accounts
+  // is slightly different.
   beforeModel: function(transition) {
     this._super(transition);
 
-    this.get('currentUser.account').then((currentUser) => {
-      let user_id = transition.params['protected.users.show'].user_id;
-      if (!currentUser.get('isAdmin') && currentUser.get('id') !== user_id) {
+    this.get('currentUser.account').then((user) => {
+      const user_id = transition.params['protected.users.show'].user_id;
+      if (!user.get('isAdmin') && user.get('id') !== user_id) {
         this.transitionTo('protected.users.index');
       }
     });
   },
 
   model: function(params) {
-    return this.store.findRecord('user', params.user_id, { reload: true });
+    return this.store.findRecord('user', params.user_id);
   },
 
 });
diff --git a/app/pods/protected/users/show/template.hbs b/app/pods/protected/users/show/template.hbs
index ca89688..a8648c4 100644
--- a/app/pods/protected/users/show/template.hbs
+++ b/app/pods/protected/users/show/template.hbs
@@ -1,53 +1,5 @@
-<div class="span-1">
-  <fieldset class="flakes-information-box">
-    <legend>
-      {{model.name}}
-    </legend>
-
-    {{! ROW 1 }}
-    <div class="grid-2 gutter-20">
-      <dl class="span-1">
-        <dt>Email</dt>
-        <dd>
-          {{model.email}}
-        </dd>
-      </dl>
-      <dl class="span-1">
-        <dt>Role</dt>
-        <dd>
-          {{model.fullRole}}
-        </dd>
-      </dl>
-    </div>
-
-    {{! ROW 2 }}
-    <div class="grid-2 gutter-20">
-      <dl class="span-1">
-        <dt>Record Created</dt>
-        <dd>{{null-time model.createdAt 'LL'}}</dd>
-      </dl>
-      <dl class="span-1">
-        <dt>Record Updated</dt>
-        <dd>{{null-time model.updatedAt 'LL'}}</dd>
-      </dl>
-    </div>
-  </fieldset>
-</div>
-<br>
-<div class="grid-2 gutter-20">
-  {{#if isUser}}
-    <div class="span-1">
-      {{#link-to 'protected.users.changepassword' model.id class="button-gray smaller"}}
-        Change Password
-      {{/link-to}}
-    </div>
-  {{/if}}
-
-  <div class="span-1">
-    {{#if model.canEdit}}
-      {{#link-to 'protected.users.edit' model.id class="button-gray smaller"}}
-        Edit
-      {{/link-to}}
-    {{/if}}
-  </div>
-</div>
+{{
+  protected/users/show/user-card
+  user=model
+  on-delete=(action 'delete')
+}}
diff --git a/app/pods/protected/users/show/user-card/component.js b/app/pods/protected/users/show/user-card/component.js
new file mode 100644
index 0000000..41dcaef
--- /dev/null
+++ b/app/pods/protected/users/show/user-card/component.js
@@ -0,0 +1,11 @@
+import Ember from 'ember';
+
+const { Component, computed, inject: { service } } = Ember;
+
+export default Component.extend({
+  currentUser: service('session-account'),
+
+  isUser: computed('model.id', 'currentUser.account.id', function() {
+    return this.get('model.id') === this.get('currentUser.account.id');
+  }),
+});
diff --git a/app/pods/protected/users/show/user-card/template.hbs b/app/pods/protected/users/show/user-card/template.hbs
new file mode 100644
index 0000000..6aa0d84
--- /dev/null
+++ b/app/pods/protected/users/show/user-card/template.hbs
@@ -0,0 +1,53 @@
+<div class="span-1">
+  <fieldset class="flakes-information-box">
+    <legend>
+      {{user.name}}
+    </legend>
+
+    {{! ROW 1 }}
+    <div class="grid-2 gutter-20">
+      <dl class="span-1">
+        <dt>Email</dt>
+        <dd>
+          {{user.email}}
+        </dd>
+      </dl>
+      <dl class="span-1">
+        <dt>Role</dt>
+        <dd>
+          {{user.fullRole}}
+        </dd>
+      </dl>
+    </div>
+
+    {{! ROW 2 }}
+    <div class="grid-2 gutter-20">
+      <dl class="span-1">
+        <dt>Record Created</dt>
+        <dd>{{null-time user.createdAt 'LL'}}</dd>
+      </dl>
+      <dl class="span-1">
+        <dt>Record Updated</dt>
+        <dd>{{null-time user.updatedAt 'LL'}}</dd>
+      </dl>
+    </div>
+  </fieldset>
+</div>
+<br>
+<div class="grid-2 gutter-20">
+  {{#if isUser}}
+    <div class="span-1">
+      {{#link-to 'protected.users.changepassword' user.id class="button-gray smaller"}}
+        Change Password
+      {{/link-to}}
+    </div>
+  {{/if}}
+
+  <div class="span-1">
+    {{#if model.canEdit}}
+      {{#link-to 'protected.users.edit' user.id class="button-gray smaller"}}
+        Edit
+      {{/link-to}}
+    {{/if}}
+  </div>
+</div>

From 1dd0910ed1711e6025f580dd78550b865069badb Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Fri, 6 Nov 2015 11:20:28 -0700
Subject: [PATCH 04/35] Refactored users/edit

---
 app/helpers/equal.js                          |  4 +-
 app/mirage/config.js                          |  2 +-
 app/mixins/save-model.js                      |  1 +
 app/mixins/setup-metadata.js                  |  2 +
 .../characteristics/edit/controller.js        |  2 +-
 app/pods/protected/users/edit/controller.js   | 42 +++------------
 app/pods/protected/users/edit/route.js        | 10 ++--
 app/pods/protected/users/edit/template.hbs    |  5 +-
 .../users/show/user-card/component.js         |  6 ++-
 .../users/show/user-card/template.hbs         |  2 +-
 .../protected/users/user-form/component.js    | 54 +++++++++++++++++--
 .../protected/users/user-form/template.hbs    | 16 +++---
 app/services/session-account.js               | 11 ++--
 13 files changed, 92 insertions(+), 65 deletions(-)

diff --git a/app/helpers/equal.js b/app/helpers/equal.js
index db3e867..a003400 100644
--- a/app/helpers/equal.js
+++ b/app/helpers/equal.js
@@ -1,7 +1,9 @@
 import Ember from 'ember';
 
+const { Helper: { helper } } = Ember;
+
 export function equalHelper(params) {
   return params[0] === params[1];
 }
 
-export default Ember.HTMLBars.makeBoundHelper(equalHelper);
+export default helper(equalHelper);
diff --git a/app/mirage/config.js b/app/mirage/config.js
index c03b469..3982c0c 100644
--- a/app/mirage/config.js
+++ b/app/mirage/config.js
@@ -2,7 +2,7 @@ export default function() {
   // Don't use mirage for development (for now)
   this.urlPrefix = 'http://127.0.0.1:8901';
   this.namespace = '/api';
-  this.passthrough();
+  this.passthrough('http://localhost:4200/**', 'http://127.0.0.1:8901/**');
 }
 
 export function testConfig() {
diff --git a/app/mixins/save-model.js b/app/mixins/save-model.js
index 53efda9..8d3b233 100644
--- a/app/mixins/save-model.js
+++ b/app/mixins/save-model.js
@@ -16,6 +16,7 @@ export default Mixin.create({
 
       if (model.get('hasDirtyAttributes')) {
         model.save().then((model) => {
+          this.get('flashMessages').clearMessages();
           this.transitionToRoute(fallbackRoute, model);
         }, () => {
           ajaxError(model.get('errors'), this.get('flashMessages'));
diff --git a/app/mixins/setup-metadata.js b/app/mixins/setup-metadata.js
index 183f93f..bdfd886 100644
--- a/app/mixins/setup-metadata.js
+++ b/app/mixins/setup-metadata.js
@@ -5,10 +5,12 @@ const { Mixin, inject: { service }} = Ember;
 export default Mixin.create({
   currentUser: service('session-account'),
   metaData: null,
+  isAdmin: null,
 
   setupMetaDataOnInit: Ember.on('init', function() {
     this.get('currentUser.account').then((user) => {
       this.set('metaData', user.get('metaData'));
+      this.set('isAdmin', user.get('isAdmin'));
     });
   }),
 });
diff --git a/app/pods/protected/characteristics/edit/controller.js b/app/pods/protected/characteristics/edit/controller.js
index 0b83924..aff45e1 100644
--- a/app/pods/protected/characteristics/edit/controller.js
+++ b/app/pods/protected/characteristics/edit/controller.js
@@ -6,5 +6,5 @@ const { Controller } = Ember;
 export default Controller.extend(SaveModel, {
   // Required for SaveModel mixin
   fallbackRouteSave: 'protected.characteristics.show',
-  fallbackRouteCancel: 'protected.characteristics.index',
+  fallbackRouteCancel: 'protected.characteristics.show',
 });
diff --git a/app/pods/protected/users/edit/controller.js b/app/pods/protected/users/edit/controller.js
index 955c328..283631c 100644
--- a/app/pods/protected/users/edit/controller.js
+++ b/app/pods/protected/users/edit/controller.js
@@ -1,40 +1,10 @@
 import Ember from 'ember';
-import ajaxError from '../../../../utils/ajax-error';
+import SaveModel from '../../../../mixins/save-model';
 
-export default Ember.Controller.extend({
-  actions: {
-    save: function() {
-      let user = this.get('model');
+const { Controller } = Ember;
 
-      if (user.get('hasDirtyAttributes')) {
-        let attrs = user.changedAttributes(), roleChanged = false;
-        if (attrs.role) {
-          roleChanged = true;
-        }
-        user.save().then((user) => {
-          this.get('flashMessages').clearMessages();
-          if (roleChanged) {
-            // Need to clear the store so that canEdit and canAdd
-            // attributes reflect the new role.
-            this.get('store').unloadAll();
-          }
-          this.transitionToRoute('protected.users.show', user);
-        }, () => {
-          ajaxError(user.get('errors'), this.get('flashMessages'));
-        });
-      } else {
-        this.transitionToRoute('protected.users.show', user);
-      }
-    },
-
-    cancel: function() {
-      let user = this.get('model');
-
-      user.get('errors').clear();
-      user.rollbackAttributes();
-
-      this.transitionToRoute('protected.users.show', user);
-    },
-
-  },
+export default Controller.extend(SaveModel, {
+  // Required for SaveModel mixin
+  fallbackRouteSave: 'protected.users.show',
+  fallbackRouteCancel: 'protected.users.show',
 });
diff --git a/app/pods/protected/users/edit/route.js b/app/pods/protected/users/edit/route.js
index b95bdb2..c91288e 100644
--- a/app/pods/protected/users/edit/route.js
+++ b/app/pods/protected/users/edit/route.js
@@ -1,8 +1,12 @@
 import Ember from 'ember';
 
-export default Ember.Route.extend({
-  currentUser: Ember.inject.service('session-account'),
+const { Route, inject: { service } } = Ember;
 
+export default Route.extend({
+  currentUser: service('session-account'),
+
+  // Not using ElevatedAccess Mixin because the rules for viewing user accounts
+  // is slightly different.
   beforeModel: function(transition) {
     this._super(transition);
 
@@ -16,7 +20,7 @@ export default Ember.Route.extend({
   },
 
   model: function(params) {
-    return this.store.findRecord('user', params.user_id, { reload: true });
+    return this.store.findRecord('user', params.user_id);
   },
 
 });
diff --git a/app/pods/protected/users/edit/template.hbs b/app/pods/protected/users/edit/template.hbs
index e53c6e8..60cbe0d 100644
--- a/app/pods/protected/users/edit/template.hbs
+++ b/app/pods/protected/users/edit/template.hbs
@@ -1,7 +1,6 @@
 {{
   protected/users/user-form
   user=model
-  currentUser=currentUser.account
-  save="save"
-  cancel="cancel"
+  on-save=(action "save")
+  on-cancel=(action "cancel")
 }}
diff --git a/app/pods/protected/users/show/user-card/component.js b/app/pods/protected/users/show/user-card/component.js
index 41dcaef..00e3164 100644
--- a/app/pods/protected/users/show/user-card/component.js
+++ b/app/pods/protected/users/show/user-card/component.js
@@ -5,7 +5,9 @@ const { Component, computed, inject: { service } } = Ember;
 export default Component.extend({
   currentUser: service('session-account'),
 
-  isUser: computed('model.id', 'currentUser.account.id', function() {
-    return this.get('model.id') === this.get('currentUser.account.id');
+  user: null,
+
+  isUser: computed('user.id', 'currentUser.account.id', function() {
+    return this.get('user.id') === this.get('currentUser.account.id');
   }),
 });
diff --git a/app/pods/protected/users/show/user-card/template.hbs b/app/pods/protected/users/show/user-card/template.hbs
index 6aa0d84..5ced125 100644
--- a/app/pods/protected/users/show/user-card/template.hbs
+++ b/app/pods/protected/users/show/user-card/template.hbs
@@ -44,7 +44,7 @@
   {{/if}}
 
   <div class="span-1">
-    {{#if model.canEdit}}
+    {{#if user.canEdit}}
       {{#link-to 'protected.users.edit' user.id class="button-gray smaller"}}
         Edit
       {{/link-to}}
diff --git a/app/pods/protected/users/user-form/component.js b/app/pods/protected/users/user-form/component.js
index 0a706f3..28fb03a 100644
--- a/app/pods/protected/users/user-form/component.js
+++ b/app/pods/protected/users/user-form/component.js
@@ -1,15 +1,61 @@
 import Ember from 'ember';
+import SetupMetaData from '../../../../mixins/setup-metadata';
 
-export default Ember.Component.extend({
+const { Component } = Ember;
+
+export default Component.extend(SetupMetaData, {
+  // Read-only attributes
+  user: null,
+  isDirty: false,
   roles: Ember.String.w('A R W'),
 
+  // Actions
+  "on-save": null,
+  "on-cancel": null,
+  "on-update": null,
+
+  // Property mapping
+  propertiesList: ['name', 'email', 'role'],
+  name: null,
+  email: null,
+  role: null,
+
+  resetOnInit: Ember.on('init', function() {
+    this.get('propertiesList').forEach((field) => {
+      const valueInUser = this.get('user').get(field);
+      this.set(field, valueInUser);
+    });
+  }),
+
+  updateField: function(property, value) {
+    this.set(property, value);
+    // Manually compare against passed in value
+    if (this.get('user').get(property) !== value) {
+      this.set('isDirty', true);
+    } else {
+      this.set('isDirty', false);
+    }
+  },
+
   actions: {
     save: function() {
-      this.sendAction('save');
+      return this.attrs['on-save'](this.getProperties(this.get('propertiesList')));
     },
 
     cancel: function() {
-      this.sendAction('cancel');
+      return this.attrs['on-cancel']();
     },
-  }
+
+    nameDidChange: function(value) {
+      this.updateField('name', value);
+    },
+
+    emailDidChange: function(value) {
+      this.updateField('email', value);
+    },
+
+    roleDidChange: function(value) {
+      this.updateField('role', value);
+    },
+  },
 });
diff --git a/app/pods/protected/users/user-form/template.hbs b/app/pods/protected/users/user-form/template.hbs
index e7a8b02..cb0e27f 100644
--- a/app/pods/protected/users/user-form/template.hbs
+++ b/app/pods/protected/users/user-form/template.hbs
@@ -1,29 +1,29 @@
 <form class="grid-form" {{action 'save' on='submit'}}>
   <fieldset>
-    <legend><em>{{user.name}}</em></legend>
+    <legend><em>{{name}}</em></legend>
     <div data-row-span="1">
       <div data-field-span="1">
         <label>Name</label>
-        {{input value=user.name class="user-name"}}
+        {{one-way-input type="text" class="user-name" value=name update=(action "nameDidChange")}}
       </div>
     </div>
     <div data-row-span="1">
       <div data-field-span="1">
         <label>Email</label>
-        {{input value=user.email}}
+        {{one-way-input type="text" class="email" value=email update=(action "emailDidChange")}}
       </div>
     </div>
     <div data-row-span="1">
       <div data-field-span="1">
         <label>Role</label>
-        {{#if session.currentUser.isAdmin}}
-          <select onchange={{action (mut user.role) value="target.value"}}>
+        {{#if isAdmin}}
+          <select onchange={{action "roleDidChange" value="target.value"}}>
             {{#each roles as |roleChoice|}}
-              <option value={{roleChoice}} selected={{equal user.role roleChoice}}>{{roleChoice}}</option>
+              <option value={{roleChoice}} selected={{equal role roleChoice}}>{{roleChoice}}</option>
             {{/each}}
           </select>
         {{else}}
-          {{user.role}}
+          {{role}} {{!-- Not editable --}}
         {{/if}}
       </div>
     </div>
@@ -32,7 +32,7 @@
   <a class="button-red smaller" {{action 'cancel'}}>
     Cancel
   </a>
-  {{#if user.hasDirtyAttributes}}
+  {{#if isDirty}}
     <button type="submit" class="button-green smaller save-user">
       Save
     </button>
diff --git a/app/services/session-account.js b/app/services/session-account.js
index 1b34554..ee09f3a 100644
--- a/app/services/session-account.js
+++ b/app/services/session-account.js
@@ -2,19 +2,20 @@ import Ember from 'ember';
 import DS from 'ember-data';
 import parseBase64 from '../utils/parse-base64';
 
-const { service } = Ember.inject;
+const { Service, computed, isEmpty, inject: { service } } = Ember;
+const { PromiseObject } = DS;
 
-export default Ember.Service.extend({
+export default Service.extend({
   session: service('session'),
   store: service(),
 
-  account: Ember.computed('session.data.authenticated.access_token', function() {
+  account: computed('session.data.authenticated.access_token', function() {
     const token = this.get('session.data.authenticated.access_token');
     const claims = parseBase64(token);
     const id = claims['sub'];
 
-    if (!Ember.isEmpty(id)) {
-      return DS.PromiseObject.create({
+    if (!isEmpty(id)) {
+      return PromiseObject.create({
         promise: this.get('store').findRecord('user', id),
       });
     }

From 45703e67ee5aaac58e6b0ef43d064d80aa9182d3 Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Fri, 6 Nov 2015 11:56:38 -0700
Subject: [PATCH 05/35] refactor user password change

---
 .../users/changepassword/controller.js        | 31 +++++------
 .../changepassword/password-form/component.js | 52 +++++++++++++++++++
 .../changepassword/password-form/template.hbs | 29 +++++++++++
 .../protected/users/changepassword/route.js   | 11 ++--
 .../users/changepassword/template.hbs         | 29 ++---------
 5 files changed, 107 insertions(+), 45 deletions(-)
 create mode 100644 app/pods/protected/users/changepassword/password-form/component.js
 create mode 100644 app/pods/protected/users/changepassword/password-form/template.hbs

diff --git a/app/pods/protected/users/changepassword/controller.js b/app/pods/protected/users/changepassword/controller.js
index b8d100e..3aed813 100644
--- a/app/pods/protected/users/changepassword/controller.js
+++ b/app/pods/protected/users/changepassword/controller.js
@@ -1,33 +1,30 @@
 import Ember from 'ember';
 import ajaxRequest from '../../../../utils/ajax-request';
 
-export default Ember.Controller.extend({
-  session: Ember.inject.service('session'),
-  currentUser: Ember.inject.service('session-account'),
+const { Controller, inject: { service } } = Ember;
 
-  passwordConfirm: null,
+export default Controller.extend({
+  session: service(),
+  currentUser: service('session-account'),
 
   actions: {
-    save: function() {
-      if (this.get('password') !== this.get('passwordConfirm')) {
-        this.get('flashMessages').clearMessages();
-        this.get('flashMessages').error("Password fields don't match");
-        return;
-      }
-
-      let url = `${this.get('globals.apiURL')}/api/${this.get('globals.genus')}/users/password`;
-      let options = {
+    save: function(password) {
+      const url = `${this.get('globals.apiURL')}/api/${this.get('globals.genus')}/users/password`;
+      const id = this.get('currentUser.account.id');
+      const options = {
         method: 'POST',
         data: {
-          id: this.get('currentUser.account.id'),
-          password: this.get('password'),
+          id: id,
+          password: password,
         },
       };
       ajaxRequest(url, options, this.get('session'));
-      this.transitionToRoute('protected.users.index');
+      this.transitionToRoute('protected.users.show', id);
       this.get('flashMessages').information('Your password has been changed.');
     },
 
+    cancel: function() {
+      this.transitionToRoute('protected.users.show', this.get('currentUser.account.id'));
+    },
   },
-
 });
diff --git a/app/pods/protected/users/changepassword/password-form/component.js b/app/pods/protected/users/changepassword/password-form/component.js
new file mode 100644
index 0000000..494885e
--- /dev/null
+++ b/app/pods/protected/users/changepassword/password-form/component.js
@@ -0,0 +1,52 @@
+import Ember from 'ember';
+
+const { Component } = Ember;
+
+export default Component.extend({
+  password: null,
+  passwordConfirm: null,
+  matches: false,
+
+  // Actions
+  "on-save": null,
+  "on-cancel": null,
+
+  updateField: function(property, value) {
+    this.set(property, value);
+    this.verifyPassword(this.get('password'), this.get('passwordConfirm'));
+  },
+
+  verifyPassword: function(password, passwordConfirm) {
+    if (password && passwordConfirm) {
+      if (password !== passwordConfirm) {
+        this.get('flashMessages').clearMessages();
+        this.get('flashMessages').error("Password fields don't match");
+        this.set('matches', false);
+      } else {
+        this.get('flashMessages').clearMessages();
+        this.set('matches', true);
+      }
+    }
+  },
+
+  actions: {
+    save: function() {
+      this.verifyPassword(this.get('password'), this.get('passwordConfirm'));
+      if (this.get('matches')) {
+        return this.attrs['on-save'](this.get('password'));
+      }
+    },
+
+    cancel: function() {
+      return this.attrs['on-cancel']();
+    },
+
+    passwordDidChange: function(value) {
+      this.updateField('password', value);
+    },
+
+    passwordConfirmDidChange: function(value) {
+      this.updateField('passwordConfirm', value);
+    },
+  },
+});
diff --git a/app/pods/protected/users/changepassword/password-form/template.hbs b/app/pods/protected/users/changepassword/password-form/template.hbs
new file mode 100644
index 0000000..5d30c35
--- /dev/null
+++ b/app/pods/protected/users/changepassword/password-form/template.hbs
@@ -0,0 +1,29 @@
+<div class="grid-1">
+  <div class="span-1">
+    <fieldset>
+      <legend>Change password</legend>
+      <form {{action 'save' on='submit'}}>
+        <ul>
+          <li>
+            <label>New Password</label>
+            {{one-way-input type="password" class="password" value=password update=(action "passwordDidChange")}}
+          </li>
+          <li>
+            <label>New Password (confirm)</label>
+            {{one-way-input type="password" class="password-confirm" value=passwordConfirm update=(action "passwordConfirmDidChange")}}
+          </li>
+          <li>
+            <a class="button-red smaller" {{action 'cancel'}}>
+              Cancel
+            </a>
+            {{#if matches}}
+              <button type="submit" class="button-green smaller submit-password">
+                Submit
+              </button>
+            {{/if}}
+          </li>
+        </ul>
+      </form>
+    </fieldset>
+  </div>
+</div>
diff --git a/app/pods/protected/users/changepassword/route.js b/app/pods/protected/users/changepassword/route.js
index ce869dd..f6da398 100644
--- a/app/pods/protected/users/changepassword/route.js
+++ b/app/pods/protected/users/changepassword/route.js
@@ -1,16 +1,19 @@
 import Ember from 'ember';
 
-export default Ember.Route.extend({
-  currentUser: Ember.inject.service('session-account'),
+const { Route, inject: { service } } = Ember;
+
+export default Route.extend({
+  currentUser: service('session-account'),
 
   beforeModel: function(transition) {
     this._super(transition);
 
-    let user_id = transition.params['protected.users.changepassword'].user_id;
+    // Only the logged in user can change their password
+    const user_id = transition.params['protected.users.changepassword'].user_id;
 
     this.get('currentUser.account').then((user) => {
       if (user.get('id') !== user_id) {
-        this.transitionTo('protected.users.index');
+        this.transitionTo('protected.users.show', user.get('id'));
       }
     });
   }
diff --git a/app/pods/protected/users/changepassword/template.hbs b/app/pods/protected/users/changepassword/template.hbs
index b22db6c..b7509c1 100644
--- a/app/pods/protected/users/changepassword/template.hbs
+++ b/app/pods/protected/users/changepassword/template.hbs
@@ -1,24 +1,5 @@
-<div class="grid-1">
-  <div class="span-1">
-    <fieldset>
-      <legend>Change password</legend>
-      <form {{action 'save' on='submit'}}>
-        <ul>
-          <li>
-            <label>New Password</label>
-            {{input type="password" value=password}}
-          </li>
-          <li>
-            <label>New Password (confirm)</label>
-            {{input type="password" value=passwordConfirm}}
-          </li>
-          <li>
-            <button type="submit" class="button-green smaller">
-              Submit
-            </button>
-          </li>
-        </ul>
-      </form>
-    </fieldset>
-  </div>
-</div>
+{{
+  protected/users/changepassword/password-form
+  on-save=(action "save")
+  on-cancel=(action "cancel")
+}}

From 367341c7803ebc5c4c6ca1817416a0685bdc3e35 Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Tue, 10 Nov 2015 07:55:08 -0700
Subject: [PATCH 06/35] Lint user lockout

---
 app/pods/users/lockoutauthenticate/controller.js |  6 ++++--
 app/pods/users/lockoutauthenticate/route.js      | 10 ++++++----
 2 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/app/pods/users/lockoutauthenticate/controller.js b/app/pods/users/lockoutauthenticate/controller.js
index 79734b5..4adee77 100644
--- a/app/pods/users/lockoutauthenticate/controller.js
+++ b/app/pods/users/lockoutauthenticate/controller.js
@@ -1,6 +1,8 @@
 import Ember from 'ember';
 
-export default Ember.Controller.extend({
-  queryParams: ['token'],
+const { Controller } = Ember;
 
+export default Controller.extend({
+  queryParams: ['token'],
+  token: null,
 });
diff --git a/app/pods/users/lockoutauthenticate/route.js b/app/pods/users/lockoutauthenticate/route.js
index 8ace7ef..21f807d 100644
--- a/app/pods/users/lockoutauthenticate/route.js
+++ b/app/pods/users/lockoutauthenticate/route.js
@@ -1,14 +1,16 @@
 import Ember from 'ember';
 import UnauthenticatedRouteMixin from 'ember-simple-auth/mixins/unauthenticated-route-mixin';
 
-export default Ember.Route.extend(UnauthenticatedRouteMixin, {
-  session: Ember.inject.service('session'),
-  currentUser: Ember.inject.service('session-account'),
+const { Route, get, inject: { service } } = Ember;
+
+export default Route.extend(UnauthenticatedRouteMixin, {
+  session: service(),
+  currentUser: service('session-account'),
 
   beforeModel: function(transition) {
     this._super(transition);
 
-    let token = Ember.get(transition, 'queryParams.token');
+    const token = get(transition, 'queryParams.token');
     this.get('session').authenticate('authenticator:jwt-resolved', token).then(() => {
       this.get('currentUser.account').then((account) => {
         this.transitionTo('protected.users.changepassword', account.get('id'));

From ec597cd419ab02633bc31194887d64dc7ac5176e Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Tue, 10 Nov 2015 09:15:28 -0700
Subject: [PATCH 07/35] Refactor new user creation

---
 app/pods/users/new/controller.js              | 23 ++++---
 app/pods/users/new/new-user-form/component.js | 66 +++++++++++++++++++
 app/pods/users/new/new-user-form/template.hbs | 36 ++++++++++
 app/pods/users/new/route.js                   | 13 ++--
 app/pods/users/new/template.hbs               | 38 ++---------
 app/pods/users/new/verify/route.js            | 19 ++----
 app/pods/users/new/verify/template.hbs        |  1 -
 7 files changed, 130 insertions(+), 66 deletions(-)
 create mode 100644 app/pods/users/new/new-user-form/component.js
 create mode 100644 app/pods/users/new/new-user-form/template.hbs
 delete mode 100644 app/pods/users/new/verify/template.hbs

diff --git a/app/pods/users/new/controller.js b/app/pods/users/new/controller.js
index 3285b1a..e2bb0d8 100644
--- a/app/pods/users/new/controller.js
+++ b/app/pods/users/new/controller.js
@@ -1,30 +1,29 @@
 import Ember from 'ember';
 import ajaxError from '../../../utils/ajax-error';
 
-export default Ember.Controller.extend({
-  passwordConfirm: null,
+const { Controller } = Ember;
+
+export default Controller.extend({
+  isLoading: false,
 
   actions: {
-    save: function() {
-      let user = this.get('user');
-
-      // All validation is server-side, except for password verification matching
-      if (user.get('password') !== this.get('passwordConfirm')) {
-        this.get('flashMessages').clearMessages();
-        this.get('flashMessages').error("Password fields don't match");
-        return;
-      }
+    save: function(properties) {
+      const user = this.get('model');
+      user.setProperties(properties);
 
       if (user.get('hasDirtyAttributes')) {
+        this.set('isLoading', true);
         user.save().then(() => {
-          this.transitionTo('login').then(() => {
+          this.transitionToRoute('login').then(() => {
             this.get('flashMessages').information(`You have successfully signed up.
               Please check your email for further instructions.`);
           });
         }, () => {
+          this.set('isLoading', false);
           ajaxError(user.get('errors'), this.get('flashMessages'));
         });
       }
     },
+
   },
 });
diff --git a/app/pods/users/new/new-user-form/component.js b/app/pods/users/new/new-user-form/component.js
new file mode 100644
index 0000000..947080c
--- /dev/null
+++ b/app/pods/users/new/new-user-form/component.js
@@ -0,0 +1,66 @@
+import Ember from 'ember';
+
+const { Component } = Ember;
+
+export default Component.extend({
+  // Read-only attributes
+  user: null,
+  isLoading: null,
+
+  // Actions
+  "on-save": null,
+  "on-cancel": null,
+
+  // Property mapping
+  propertiesList: ['name', 'email', 'password', 'passwordConfirm'],
+  name: null,
+  email: null,
+  password: null,
+  passwordConfirm: null,
+
+  resetOnInit: Ember.on('init', function() {
+    this.get('propertiesList').forEach((field) => {
+      const valueInUser = this.get('user').get(field);
+      this.set(field, valueInUser);
+    });
+  }),
+
+  updateField: function(property, value) {
+    this.set(property, value);
+    // Manually compare against passed in value
+    if (this.get('user').get(property) !== value) {
+      this.set('isDirty', true);
+    } else {
+      this.set('isDirty', false);
+    }
+  },
+
+  actions: {
+    save: function() {
+      // All validation is server-side, except for password verification matching
+      if (this.get('password') !== this.get('passwordConfirm')) {
+        this.get('flashMessages').clearMessages();
+        this.get('flashMessages').error("Password fields don't match");
+        return;
+      }
+
+      return this.attrs['on-save'](this.getProperties(this.get('propertiesList')));
+    },
+
+    nameDidChange: function(value) {
+      this.updateField('name', value);
+    },
+
+    emailDidChange: function(value) {
+      this.updateField('email', value);
+    },
+
+    passwordDidChange: function(value) {
+      this.updateField('password', value);
+    },
+
+    passwordConfirmDidChange: function(value) {
+      this.updateField('passwordConfirm', value);
+    }
+  }
+});
diff --git a/app/pods/users/new/new-user-form/template.hbs b/app/pods/users/new/new-user-form/template.hbs
new file mode 100644
index 0000000..980eec9
--- /dev/null
+++ b/app/pods/users/new/new-user-form/template.hbs
@@ -0,0 +1,36 @@
+{{#if isLoading}}
+  {{loading-panel}}
+{{else}}
+  <div class="grid-1">
+    <div class="span-1">
+      <fieldset>
+        <legend>New User Signup</legend>
+        <form {{action 'save' on='submit'}}>
+          <ul>
+            <li>
+              <label>Name</label>
+              {{one-way-input type="text" class="user-name" value=name update=(action "nameDidChange")}}
+            </li>
+            <li>
+              <label>Email</label>
+              {{one-way-input type="text" class="email" value=email update=(action "emailDidChange")}}
+            </li>
+            <li>
+              <label>Password</label>
+              {{one-way-input type="password" class="password" value=password update=(action "passwordDidChange")}}
+            </li>
+            <li>
+              <label>Password (confirm)</label>
+              {{one-way-input type="password" class="password-verify" value=passwordConfirm update=(action "passwordConfirmDidChange")}}
+            </li>
+            <li>
+              <button type="submit" class="button-green smaller save-user">
+                Submit
+              </button>
+            </li>
+          </ul>
+        </form>
+      </fieldset>
+    </div>
+  </div>
+{{/if}}
diff --git a/app/pods/users/new/route.js b/app/pods/users/new/route.js
index c3e757a..afdcdac 100644
--- a/app/pods/users/new/route.js
+++ b/app/pods/users/new/route.js
@@ -1,15 +1,10 @@
 import Ember from 'ember';
 import UnauthenticatedRouteMixin from 'ember-simple-auth/mixins/unauthenticated-route-mixin';
 
-export default Ember.Route.extend(UnauthenticatedRouteMixin, {
+const { Route } = Ember;
+
+export default Route.extend(UnauthenticatedRouteMixin, {
   model: function() {
-    return Ember.RSVP.hash({
-      user: this.store.createRecord('user'),
-    });
+    return this.store.createRecord('user');
   },
-
-  setupController: function(controller, model) {
-    controller.setProperties(model);
-  },
-
 });
diff --git a/app/pods/users/new/template.hbs b/app/pods/users/new/template.hbs
index e8e6e2c..ac728bf 100644
--- a/app/pods/users/new/template.hbs
+++ b/app/pods/users/new/template.hbs
@@ -1,32 +1,6 @@
-<div class="grid-1">
-  <div class="span-1">
-    <fieldset>
-      <legend>New User Signup</legend>
-      <form {{action 'save' on='submit'}}>
-        <ul>
-          <li>
-            <label>Name</label>
-            {{input value=user.name class="user-name"}}
-          </li>
-          <li>
-            <label>Email</label>
-            {{input value=user.email class="email"}}
-          </li>
-          <li>
-            <label>Password</label>
-            {{input type="password" value=user.password class="password"}}
-          </li>
-          <li>
-            <label>Password (confirm)</label>
-            {{input type="password" value=passwordConfirm class="password-verify"}}
-          </li>
-          <li>
-            <button type="submit" class="button-green smaller save-user">
-              Submit
-            </button>
-          </li>
-        </ul>
-      </form>
-    </fieldset>
-  </div>
-</div>
+{{
+  users/new/new-user-form
+  user=model
+  isLoading=isLoading
+  on-save=(action "save")
+}}
diff --git a/app/pods/users/new/verify/route.js b/app/pods/users/new/verify/route.js
index 7c50823..06755b3 100644
--- a/app/pods/users/new/verify/route.js
+++ b/app/pods/users/new/verify/route.js
@@ -1,31 +1,26 @@
 import Ember from 'ember';
 import ajaxRequest from '../../../../utils/ajax-request';
 
-export default Ember.Route.extend({
-  session: Ember.inject.service('session'),
+const { Route, inject: { service } } = Ember;
 
-  apiURL: function() {
-    return this.get('globals.apiURL');
-  }.property(),
-
-  genus: function() {
-    return this.get('globals.genus');
-  }.property(),
+export default Route.extend({
+  session: service(),
+  globals: service(),
 
   model: function(params) {
-    let url = `${this.get('apiURL')}/api/${this.get('genus')}/users/verify/${params.nonce}`;
+    const url = `${this.get('globals.apiURL')}/api/${this.get('globals.genus')}/users/verify/${params.nonce}`;
     return ajaxRequest(url, {}, this.get('session'));
   },
 
 
   afterModel: function(model/*, transition*/) {
-    this.get('flashMessages').success(model.msg);
+    this.get('flashMessages').success(model.get('msg'));
     this.transitionTo('login');
   },
 
   actions: {
     error: function(error/*, transition*/) {
-      let err = Ember.$.parseJSON(error.responseText);
+      const err = Ember.$.parseJSON(error.responseText);
       this.get('flashMessages').error(err.error);
       this.transitionTo('login');
     }
diff --git a/app/pods/users/new/verify/template.hbs b/app/pods/users/new/verify/template.hbs
deleted file mode 100644
index c24cd68..0000000
--- a/app/pods/users/new/verify/template.hbs
+++ /dev/null
@@ -1 +0,0 @@
-{{outlet}}

From 079b2c59f61d8d048b15b9390e0ea8c0a2e63ae3 Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Tue, 10 Nov 2015 09:22:35 -0700
Subject: [PATCH 08/35] Refactor user lockout/pw reset

---
 .../users/requestlockouthelp/controller.js    | 15 ++++++++-----
 .../lockout-form/component.js                 | 18 +++++++++++++++
 .../lockout-form/template.hbs                 | 18 +++++++++++++++
 app/pods/users/requestlockouthelp/route.js    |  8 -------
 .../users/requestlockouthelp/template.hbs     | 22 ++++---------------
 5 files changed, 49 insertions(+), 32 deletions(-)
 create mode 100644 app/pods/users/requestlockouthelp/lockout-form/component.js
 create mode 100644 app/pods/users/requestlockouthelp/lockout-form/template.hbs
 delete mode 100644 app/pods/users/requestlockouthelp/route.js

diff --git a/app/pods/users/requestlockouthelp/controller.js b/app/pods/users/requestlockouthelp/controller.js
index ccda273..2930b31 100644
--- a/app/pods/users/requestlockouthelp/controller.js
+++ b/app/pods/users/requestlockouthelp/controller.js
@@ -1,15 +1,18 @@
 import Ember from 'ember';
 import ajaxRequest from '../../../utils/ajax-request';
 
-export default Ember.Controller.extend({
-  session: Ember.inject.service('session'),
+const { Controller, inject: { service } } = Ember;
+
+export default Controller.extend({
+  session: service(),
+  globals: service(),
 
   actions: {
-    save: function() {
-      let url = `${this.get('globals.apiURL')}/api/${this.get('globals.genus')}/users/lockout`;
-      let options = {
+    submit: function(email) {
+      const url = `${this.get('globals.apiURL')}/api/${this.get('globals.genus')}/users/lockout`;
+      const options = {
         method: 'POST',
-        data: { email: this.get('email') },
+        data: { email: email },
       };
       ajaxRequest(url, options, this.get('session'));
       this.transitionToRoute('login');
diff --git a/app/pods/users/requestlockouthelp/lockout-form/component.js b/app/pods/users/requestlockouthelp/lockout-form/component.js
new file mode 100644
index 0000000..2b7426f
--- /dev/null
+++ b/app/pods/users/requestlockouthelp/lockout-form/component.js
@@ -0,0 +1,18 @@
+import Ember from 'ember';
+
+const { Component } = Ember;
+
+export default Component.extend({
+  email: null,
+  "on-submit": null,
+
+  actions: {
+    save: function() {
+      return this.attrs["on-submit"](this.get('email'));
+    },
+
+    emailDidChange: function(value) {
+      this.set('email', value);
+    },
+  }
+});
diff --git a/app/pods/users/requestlockouthelp/lockout-form/template.hbs b/app/pods/users/requestlockouthelp/lockout-form/template.hbs
new file mode 100644
index 0000000..5f18a09
--- /dev/null
+++ b/app/pods/users/requestlockouthelp/lockout-form/template.hbs
@@ -0,0 +1,18 @@
+<div class="grid-1">
+  <div class="span-1">
+    <fieldset>
+      <legend>Account Lockout/Password Reset</legend>
+      <form {{action 'save' on='submit'}}>
+        <ul>
+          <li>
+            <label>Email</label>
+            {{one-way-input type="text" class="email" value=email update=(action "emailDidChange")}}
+          </li>
+          <li>
+            <button type="submit" class="button-green smaller">Submit</button>
+          </li>
+        </ul>
+      </form>
+    </fieldset>
+  </div>
+</div>
diff --git a/app/pods/users/requestlockouthelp/route.js b/app/pods/users/requestlockouthelp/route.js
deleted file mode 100644
index 19831b4..0000000
--- a/app/pods/users/requestlockouthelp/route.js
+++ /dev/null
@@ -1,8 +0,0 @@
-import Ember from 'ember';
-
-export default Ember.Route.extend({
-  deactivate: function() {
-    this.controller.set('email', null);
-  },
-
-});
diff --git a/app/pods/users/requestlockouthelp/template.hbs b/app/pods/users/requestlockouthelp/template.hbs
index 7cdd25f..c482b70 100644
--- a/app/pods/users/requestlockouthelp/template.hbs
+++ b/app/pods/users/requestlockouthelp/template.hbs
@@ -1,18 +1,4 @@
-<div class="grid-1">
-  <div class="span-1">
-    <fieldset>
-      <legend>Account Lockout/Password Reset</legend>
-      <form {{action 'save' on='submit'}}>
-        <ul>
-          <li>
-            <label>Email</label>
-            {{input value=email}}
-          </li>
-          <li>
-            <button type="submit" class="button-green smaller">Submit</button>
-          </li>
-        </ul>
-      </form>
-    </fieldset>
-  </div>
-</div>
+{{
+  users/requestlockouthelp/lockout-form
+  on-submit=(action "submit")
+}}

From 35475c2b920ac1ec8d526acb61795884f3241645 Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Tue, 10 Nov 2015 11:02:23 -0700
Subject: [PATCH 09/35] Rough in some tests

---
 app/mirage/config.js                          | 15 ++++
 app/mirage/factories/strains.js               | 17 +++++
 app/pods/protected/strains/index/template.hbs |  2 +-
 app/pods/protected/strains/show/template.hbs  |  2 +-
 .../strains/strain-form/template.hbs          |  4 +-
 tests/acceptance/strains-test.js              | 76 +++++++++++++++++++
 6 files changed, 112 insertions(+), 4 deletions(-)
 create mode 100644 app/mirage/factories/strains.js
 create mode 100644 tests/acceptance/strains-test.js

diff --git a/app/mirage/config.js b/app/mirage/config.js
index 3982c0c..e39ccee 100644
--- a/app/mirage/config.js
+++ b/app/mirage/config.js
@@ -24,4 +24,19 @@ export function testConfig() {
   this.post('/characteristics');
   this.get('/characteristics/:id');
   this.put('/characteristics/:id');
+
+  this.get('/strains', function(db /*, request*/) {
+    return {
+      strains: db.strains,
+      species: db.species,
+    };
+  });
+  this.post('/strains');
+  this.get('/strains/:id', function(db, request) {
+    return {
+      strain: db.strains.find(request.params.id),
+      species: db.species, // Just send back everything we've got
+    }
+  });
+  this.put('/strains/:id');
 }
diff --git a/app/mirage/factories/strains.js b/app/mirage/factories/strains.js
new file mode 100644
index 0000000..6165528
--- /dev/null
+++ b/app/mirage/factories/strains.js
@@ -0,0 +1,17 @@
+import Mirage, { faker } from 'ember-cli-mirage';
+
+export default Mirage.Factory.extend({
+  measurements: [],
+  characteristics: [],
+  species: 0,
+  strainName() { return faker.lorem.words().join(' '); },
+  typeStrain: faker.random.boolean(),
+  accessionNumbers() { return faker.lorem.words().join(' '); },
+  genbank() { return faker.lorem.words().join(' '); },
+  wholeGenomeSequence() { return faker.lorem.words().join(' '); },
+  isolatedFrom: faker.lorem.sentences(),
+  notes: faker.lorem.sentences(),
+  totalMeasurements: 0,
+  sortOrder: faker.random.number(),
+  canEdit: faker.random.boolean(),
+});
diff --git a/app/pods/protected/strains/index/template.hbs b/app/pods/protected/strains/index/template.hbs
index 077d22a..851f93b 100644
--- a/app/pods/protected/strains/index/template.hbs
+++ b/app/pods/protected/strains/index/template.hbs
@@ -1,5 +1,5 @@
 <h2>{{genus-name}} Strains</h2>
-<h3>Total strains: {{model.length}}</h3>
+<h3 id="total-strains">Total strains: {{model.length}}</h3>
 
 {{add-button label="Add Strain" link="protected.strains.new" canAdd=metaData.canAdd}}
 
diff --git a/app/pods/protected/strains/show/template.hbs b/app/pods/protected/strains/show/template.hbs
index 8a10e60..87d5d4f 100644
--- a/app/pods/protected/strains/show/template.hbs
+++ b/app/pods/protected/strains/show/template.hbs
@@ -1,7 +1,7 @@
 <div class="span-1">
   <fieldset class="flakes-information-box">
     <legend>
-      Strain {{model.strainNameMU}}
+      {{model.strainNameMU}}
     </legend>
 
     {{! ROW 1 }}
diff --git a/app/pods/protected/strains/strain-form/template.hbs b/app/pods/protected/strains/strain-form/template.hbs
index 109d037..fef122d 100644
--- a/app/pods/protected/strains/strain-form/template.hbs
+++ b/app/pods/protected/strains/strain-form/template.hbs
@@ -4,7 +4,7 @@
     <div data-row-span="2">
       <div data-field-span="1">
         <label>Strain Name</label>
-        {{input value=strain.strainName}}
+        {{input value=strain.strainName class="strain-name"}}
       </div>
       <div data-field-span="1">
         <label>Type Strain?</label>
@@ -62,7 +62,7 @@
     Cancel
   </a>
   {{#if strain.hasDirtyAttributes}}
-    <button type="submit" class="button-green smaller">
+    <button type="submit" class="button-green smaller save-strain">
       Save
     </button>
   {{/if}}
diff --git a/tests/acceptance/strains-test.js b/tests/acceptance/strains-test.js
new file mode 100644
index 0000000..a52e7ed
--- /dev/null
+++ b/tests/acceptance/strains-test.js
@@ -0,0 +1,76 @@
+import Ember from 'ember';
+import { module, test } from 'qunit';
+import startApp from '../helpers/start-app';
+import { authenticateSession } from '../helpers/ember-simple-auth';
+
+module('Acceptance | strains', {
+  beforeEach: function() {
+    this.application = startApp();
+    authenticateSession(this.application, {
+      access_token: "eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJiYWN0ZGIiLCJzdWIiOiIxIiwiZXhwIjoxNDQ2NTAyMjI2LCJpYXQiOjE0NDY0OTg2MjZ9.vIjKHAsp2TkCV505EbtCo2xQT-2oQkB-Nv5y0b6E7Mg"
+    });
+    server.create('users', { role: 'A', canEdit: true });
+  },
+
+  afterEach: function() {
+    Ember.run(this.application, 'destroy');
+  }
+});
+
+test('visiting /strains', function(assert) {
+  const species = server.create('species');
+  const strains = server.createList('strains', 20, { species: species.id });
+  visit('/strains');
+
+  andThen(function() {
+    assert.equal(currentURL(), '/strains');
+    assert.equal(find(".flakes-table > tbody > tr").length, strains.length);
+    assert.equal(find("#total-strains").text(), "Total strains: 20");
+  });
+});
+
+test('visiting /strains/:id', function(assert) {
+  const species = server.create('species');
+  const strain = server.create('strains', { species: species.id });
+  visit(`/strains/${strain.id}`);
+
+  andThen(function() {
+    assert.equal(currentURL(), `/strains/${strain.id}`);
+    const typeStrain = strain.typeStrain ? 'T' : '';
+    assert.equal(find(".flakes-information-box > legend").text().trim(), `${strain.strainName}${typeStrain}`);
+  });
+});
+
+test('editing /strains/:id/edit', function(assert) {
+  const species = server.create('species');
+  const strain = server.create('strains', { canEdit: true , species: species.id });
+  visit(`/strains/${strain.id}/edit`);
+
+  andThen(function() {
+    assert.equal(currentURL(), `/strains/${strain.id}/edit`);
+
+    fillIn('.strain-name', 'Revised Strain Name');
+    click('.save-strain');
+
+    andThen(function() {
+      assert.equal(currentURL(), `/strains/${strain.id}`);
+      const typeStrain = strain.typeStrain ? 'T' : '';
+      assert.equal(find(".flakes-information-box > legend").text().trim(), `Revised Strain Name${typeStrain}`);
+    });
+  });
+});
+
+test('creating /strains/new', function(assert) {
+  visit(`/strains/new`);
+
+  andThen(function() {
+    assert.equal(currentURL(), `/strains/new`);
+    fillIn('.strain-name', 'New Strain Name');
+    click('.save-strain');
+
+    andThen(function() {
+      assert.equal(find(".flakes-information-box > legend").text().trim(), `New Strain Name`);
+      assert.equal(server.db.strains.length, 1);
+    });
+  });
+});

From 85d861da27fa10c71b867e112dd72ba34474a0fc Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Tue, 10 Nov 2015 11:13:22 -0700
Subject: [PATCH 10/35] Refactor strains/index

---
 .../species/index/species-table/component.js  |  4 +--
 .../protected/strains/index/controller.js     |  6 ----
 app/pods/protected/strains/index/route.js     | 11 ++-----
 .../strains/index/strain-table/component.js   | 12 ++++++++
 .../strains/index/strain-table/template.hbs   | 26 +++++++++++++++++
 app/pods/protected/strains/index/template.hbs | 29 +++----------------
 6 files changed, 46 insertions(+), 42 deletions(-)
 delete mode 100644 app/pods/protected/strains/index/controller.js
 create mode 100644 app/pods/protected/strains/index/strain-table/component.js
 create mode 100644 app/pods/protected/strains/index/strain-table/template.hbs

diff --git a/app/pods/protected/species/index/species-table/component.js b/app/pods/protected/species/index/species-table/component.js
index d58f509..528c48d 100644
--- a/app/pods/protected/species/index/species-table/component.js
+++ b/app/pods/protected/species/index/species-table/component.js
@@ -1,12 +1,12 @@
 import Ember from 'ember';
 import SetupMetaData from '../../../../../mixins/setup-metadata';
 
-const { Component } = Ember;
+const { Component, computed: { sort } } = Ember;
 
 export default Component.extend(SetupMetaData, {
   species: null,
 
   sortParams: ['speciesName', 'strainCount'],
-  sortedSpecies: Ember.computed.sort('species', 'sortParams'),
+  sortedSpecies: sort('species', 'sortParams'),
 
 });
diff --git a/app/pods/protected/strains/index/controller.js b/app/pods/protected/strains/index/controller.js
deleted file mode 100644
index ec348db..0000000
--- a/app/pods/protected/strains/index/controller.js
+++ /dev/null
@@ -1,6 +0,0 @@
-import Ember from 'ember';
-
-export default Ember.Controller.extend({
-  sortParams: ['sortOrder'],
-  sortedStrains: Ember.computed.sort('model', 'sortParams'),
-});
diff --git a/app/pods/protected/strains/index/route.js b/app/pods/protected/strains/index/route.js
index e5582d7..627760f 100644
--- a/app/pods/protected/strains/index/route.js
+++ b/app/pods/protected/strains/index/route.js
@@ -1,17 +1,10 @@
 import Ember from 'ember';
 
-export default Ember.Route.extend({
-  currentUser: Ember.inject.service('session-account'),
+const { Route } = Ember;
 
+export default Route.extend({
   model: function() {
     return this.store.findAll('strain');
   },
 
-  setupController: function(controller, model) {
-    controller.set('model', model);
-    this.get('currentUser.account').then((user) => {
-      controller.set('metaData', user.get('metaData'));
-    });
-  },
-
 });
diff --git a/app/pods/protected/strains/index/strain-table/component.js b/app/pods/protected/strains/index/strain-table/component.js
new file mode 100644
index 0000000..0e801ca
--- /dev/null
+++ b/app/pods/protected/strains/index/strain-table/component.js
@@ -0,0 +1,12 @@
+import Ember from 'ember';
+import SetupMetaData from '../../../../../mixins/setup-metadata';
+
+const { Component, computed: { sort } } = Ember;
+
+export default Component.extend(SetupMetaData, {
+  strains: null,
+
+  sortParams: ['fullNameMU'],
+  sortedStrains: sort('strains', 'sortParams'),
+
+});
diff --git a/app/pods/protected/strains/index/strain-table/template.hbs b/app/pods/protected/strains/index/strain-table/template.hbs
new file mode 100644
index 0000000..bb13d76
--- /dev/null
+++ b/app/pods/protected/strains/index/strain-table/template.hbs
@@ -0,0 +1,26 @@
+<h3 id="total-strains">Total strains: {{strains.length}}</h3>
+
+{{add-button label="Add Strain" link="protected.strains.new" canAdd=metaData.canAdd}}
+
+<table class="flakes-table">
+  <thead>
+    <tr>
+      <th>Species</th>
+      <th>Total Measurements</th>
+    </tr>
+  </thead>
+  <tbody>
+    {{#each sortedStrains as |strain|}}
+      <tr>
+        <td>
+          {{#link-to 'protected.strains.show' strain classBinding="data.typeStrain:type-strain"}}
+            {{strain.fullNameMU}}
+          {{/link-to}}
+        </td>
+        <td>
+          {{strain.totalMeasurements}}
+        </td>
+      </tr>
+    {{/each}}
+  </tbody>
+</table>
diff --git a/app/pods/protected/strains/index/template.hbs b/app/pods/protected/strains/index/template.hbs
index 851f93b..d441386 100644
--- a/app/pods/protected/strains/index/template.hbs
+++ b/app/pods/protected/strains/index/template.hbs
@@ -1,27 +1,6 @@
 <h2>{{genus-name}} Strains</h2>
-<h3 id="total-strains">Total strains: {{model.length}}</h3>
 
-{{add-button label="Add Strain" link="protected.strains.new" canAdd=metaData.canAdd}}
-
-<table class="flakes-table">
-  <thead>
-    <tr>
-      <th>Species</th>
-      <th>Total Measurements</th>
-    </tr>
-  </thead>
-  <tbody>
-    {{#each sortedStrains as |row|}}
-      <tr>
-        <td>
-          {{#link-to 'protected.strains.show' row classBinding="data.typeStrain:type-strain"}}
-            {{row.fullNameMU}}
-          {{/link-to}}
-        </td>
-        <td>
-          {{row.totalMeasurements}}
-        </td>
-      </tr>
-    {{/each}}
-  </tbody>
-</table>
+{{
+  protected/strains/index/strain-table
+  strains=model
+}}

From 04486880a09ea8e0d0a30cb3ca2292764c1a6997 Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Tue, 10 Nov 2015 11:41:04 -0700
Subject: [PATCH 11/35] Begin refactoring strains/show

Will address measurements table later
---
 app/pods/protected/strains/show/controller.js |  13 +--
 .../show/measurements-table-row/component.js  |   4 +-
 .../show/measurements-table/component.js      |   4 +-
 app/pods/protected/strains/show/route.js      |   6 +-
 .../strains/show/strain-card/component.js     |  14 +++
 .../strains/show/strain-card/template.hbs     | 105 +++++++++++++++++
 app/pods/protected/strains/show/template.hbs  | 110 +-----------------
 7 files changed, 139 insertions(+), 117 deletions(-)
 create mode 100644 app/pods/protected/strains/show/strain-card/component.js
 create mode 100644 app/pods/protected/strains/show/strain-card/template.hbs

diff --git a/app/pods/protected/strains/show/controller.js b/app/pods/protected/strains/show/controller.js
index 7128b9a..e5e3e77 100644
--- a/app/pods/protected/strains/show/controller.js
+++ b/app/pods/protected/strains/show/controller.js
@@ -1,12 +1,9 @@
 import Ember from 'ember';
+import DeleteModel from '../../../../mixins/delete-model';
 
-export default Ember.Controller.extend({
-  actions: {
-    delete: function() {
-      this.get('model').destroyRecord().then(() => {
-        this.transitionToRoute('protected.strains.index');
-      });
-    },
-  },
+const { Controller } = Ember;
 
+export default Controller.extend(DeleteModel, {
+  // Required for DeleteModel mixin
+  transitionRoute: 'protected.strains.index',
 });
diff --git a/app/pods/protected/strains/show/measurements-table-row/component.js b/app/pods/protected/strains/show/measurements-table-row/component.js
index b4220cd..8ad1ba7 100644
--- a/app/pods/protected/strains/show/measurements-table-row/component.js
+++ b/app/pods/protected/strains/show/measurements-table-row/component.js
@@ -1,7 +1,9 @@
 import Ember from 'ember';
 import ajaxError from '../../../../../utils/ajax-error';
 
-export default Ember.Component.extend({
+const { Component } = Ember;
+
+export default Component.extend({
   tagName: 'tr',
   isEditing: false,
 
diff --git a/app/pods/protected/strains/show/measurements-table/component.js b/app/pods/protected/strains/show/measurements-table/component.js
index 07d94c6..6cbf880 100644
--- a/app/pods/protected/strains/show/measurements-table/component.js
+++ b/app/pods/protected/strains/show/measurements-table/component.js
@@ -1,6 +1,8 @@
 import Ember from 'ember';
 
-export default Ember.Component.extend({
+const { Component } = Ember;
+
+export default Component.extend({
   measurementsPresent: function() {
     return this.get('model.measurements.length') > 0;
   }.property('model.measurements'),
diff --git a/app/pods/protected/strains/show/route.js b/app/pods/protected/strains/show/route.js
index 0a17d8b..ea320bd 100644
--- a/app/pods/protected/strains/show/route.js
+++ b/app/pods/protected/strains/show/route.js
@@ -1,8 +1,10 @@
 import Ember from 'ember';
 
-export default Ember.Route.extend({
+const { Route } = Ember;
+
+export default Route.extend({
   model: function(params) {
-    return this.store.findRecord('strain', params.strain_id, { reload: true });
+    return this.store.findRecord('strain', params.strain_id);
   },
 
 });
diff --git a/app/pods/protected/strains/show/strain-card/component.js b/app/pods/protected/strains/show/strain-card/component.js
new file mode 100644
index 0000000..40802c8
--- /dev/null
+++ b/app/pods/protected/strains/show/strain-card/component.js
@@ -0,0 +1,14 @@
+import Ember from 'ember';
+
+const { Component } = Ember;
+
+export default Component.extend({
+  strain: null,
+  "on-delete": null,
+
+  actions: {
+    deleteStrain: function() {
+      return this.attrs['on-delete']();
+    },
+  },
+});
diff --git a/app/pods/protected/strains/show/strain-card/template.hbs b/app/pods/protected/strains/show/strain-card/template.hbs
new file mode 100644
index 0000000..989db71
--- /dev/null
+++ b/app/pods/protected/strains/show/strain-card/template.hbs
@@ -0,0 +1,105 @@
+<div class="span-1">
+  <fieldset class="flakes-information-box">
+    <legend>
+      {{strain.strainNameMU}}
+    </legend>
+
+    {{! ROW 1 }}
+    <div class="grid-2 gutter-20">
+      <dl class="span-1">
+        <dt>Species</dt>
+        <dd>
+          {{#link-to 'protected.species.show' strain.species.id}}
+            <em>{{strain.species.speciesNameMU}}</em>
+          {{/link-to}}
+        </dd>
+      </dl>
+      <dl class="span-1">
+        <dt>Type Strain?</dt>
+        <dd>
+          {{if strain.typeStrain 'Yes' 'No'}}
+        </dd>
+      </dl>
+    </div>
+
+    {{! ROW 2 }}
+    <div class="grid-3 gutter-20">
+      <dl class="span-1">
+        <dt>Accession Numbers</dt>
+        <dd>
+          {{strain.accessionNumbers}}
+        </dd>
+      </dl>
+      <dl class="span-1">
+        <dt>Genbank</dt>
+        <dd>
+          {{genbank-url genbank=strain.genbank}}
+        </dd>
+      </dl>
+      <dl class="span-1">
+        <dt>Whole Genome Sequence</dt>
+        <dd>
+          {{genbank-url genbank=strain.wholeGenomeSequence}}
+        </dd>
+      </dl>
+    </div>
+
+    {{! ROW 3 }}
+    <div class="grid-1 gutter-20">
+      <dl class="span-1">
+        <dt>Isolated From</dt>
+        <dd>
+          {{{strain.isolatedFrom}}}
+        </dd>
+      </dl>
+    </div>
+
+    {{! ROW 4 }}
+    <div class="grid-1 gutter-20">
+      <dl class="span-1">
+        <dt>Notes</dt>
+        <dd>
+          {{#if strain.notes}}
+            {{{strain.notes}}}
+          {{else}}
+            No notes.
+          {{/if}}
+        </dd>
+      </dl>
+    </div>
+
+    {{! ROW 5 }}
+    <div class="grid-1 gutter-20">
+      <dl class="span-1">
+        <dt>Characteristics</dt>
+        <dd>
+          {{
+            protected/strains/show/measurements-table
+            model=strain
+            canEdit=false
+            canAdd=false
+          }}
+        </dd>
+      </dl>
+    </div>
+
+    {{! ROW 6 }}
+    <div class="grid-2 gutter-20">
+      <dl class="span-1">
+        <dt>Record Created</dt>
+        <dd>{{null-time strain.createdAt 'LL'}}</dd>
+      </dl>
+      <dl class="span-1">
+        <dt>Record Updated</dt>
+        <dd>{{null-time strain.updatedAt 'LL'}}</dd>
+      </dl>
+    </div>
+  </fieldset>
+</div>
+{{#if strain.canEdit}}
+  <br>
+  {{#link-to 'protected.strains.edit' strain class="button-gray smaller"}}
+    Edit
+  {{/link-to}}
+  {{delete-button delete=(action 'deleteStrain')}}
+{{/if}}
diff --git a/app/pods/protected/strains/show/template.hbs b/app/pods/protected/strains/show/template.hbs
index 87d5d4f..32d23d6 100644
--- a/app/pods/protected/strains/show/template.hbs
+++ b/app/pods/protected/strains/show/template.hbs
@@ -1,105 +1,5 @@
-<div class="span-1">
-  <fieldset class="flakes-information-box">
-    <legend>
-      {{model.strainNameMU}}
-    </legend>
-
-    {{! ROW 1 }}
-    <div class="grid-2 gutter-20">
-      <dl class="span-1">
-        <dt>Species</dt>
-        <dd>
-          {{#link-to 'protected.species.show' model.species.id}}
-            <em>{{model.species.speciesNameMU}}</em>
-          {{/link-to}}
-        </dd>
-      </dl>
-      <dl class="span-1">
-        <dt>Type Strain?</dt>
-        <dd>
-          {{if model.typeStrain 'Yes' 'No'}}
-        </dd>
-      </dl>
-    </div>
-
-    {{! ROW 2 }}
-    <div class="grid-3 gutter-20">
-      <dl class="span-1">
-        <dt>Accession Numbers</dt>
-        <dd>
-          {{model.accessionNumbers}}
-        </dd>
-      </dl>
-      <dl class="span-1">
-        <dt>Genbank</dt>
-        <dd>
-          {{genbank-url genbank=model.genbank}}
-        </dd>
-      </dl>
-      <dl class="span-1">
-        <dt>Whole Genome Sequence</dt>
-        <dd>
-          {{genbank-url genbank=model.wholeGenomeSequence}}
-        </dd>
-      </dl>
-    </div>
-
-    {{! ROW 3 }}
-    <div class="grid-1 gutter-20">
-      <dl class="span-1">
-        <dt>Isolated From</dt>
-        <dd>
-          {{{model.isolatedFrom}}}
-        </dd>
-      </dl>
-    </div>
-
-    {{! ROW 4 }}
-    <div class="grid-1 gutter-20">
-      <dl class="span-1">
-        <dt>Notes</dt>
-        <dd>
-          {{#if model.notes}}
-            {{{model.notes}}}
-          {{else}}
-            No notes.
-          {{/if}}
-        </dd>
-      </dl>
-    </div>
-
-    {{! ROW 5 }}
-    <div class="grid-1 gutter-20">
-      <dl class="span-1">
-        <dt>Characteristics</dt>
-        <dd>
-          {{
-            protected/strains/show/measurements-table
-            model=model
-            canEdit=false
-            canAdd=false
-          }}
-        </dd>
-      </dl>
-    </div>
-
-    {{! ROW 6 }}
-    <div class="grid-2 gutter-20">
-      <dl class="span-1">
-        <dt>Record Created</dt>
-        <dd>{{null-time model.createdAt 'LL'}}</dd>
-      </dl>
-      <dl class="span-1">
-        <dt>Record Updated</dt>
-        <dd>{{null-time model.updatedAt 'LL'}}</dd>
-      </dl>
-    </div>
-  </fieldset>
-</div>
-{{#if model.canEdit}}
-  <br>
-  {{#link-to 'protected.strains.edit' model.id class="button-gray smaller"}}
-    Edit
-  {{/link-to}}
-  {{delete-button delete=(action 'delete')}}
-{{/if}}
+{{
+  protected/strains/show/strain-card
+  strain=model
+  on-delete=(action 'delete')
+}}

From 29c507af6b4b834e4078c6ab526e24d1bb238e4b Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Tue, 10 Nov 2015 12:57:12 -0700
Subject: [PATCH 12/35] Refactor strain form and edit strain

---
 app/pods/protected/strains/edit/controller.js | 34 ++------
 app/pods/protected/strains/edit/route.js      | 41 ++++++----
 app/pods/protected/strains/edit/template.hbs  |  8 +-
 .../strains/strain-form/component.js          | 78 +++++++++++++++++--
 .../strains/strain-form/template.hbs          | 30 +++----
 5 files changed, 122 insertions(+), 69 deletions(-)

diff --git a/app/pods/protected/strains/edit/controller.js b/app/pods/protected/strains/edit/controller.js
index 2f84780..a44ea40 100644
--- a/app/pods/protected/strains/edit/controller.js
+++ b/app/pods/protected/strains/edit/controller.js
@@ -1,32 +1,10 @@
 import Ember from 'ember';
-import ajaxError from '../../../../utils/ajax-error';
+import SaveModel from '../../../../mixins/save-model';
 
-export default Ember.Controller.extend({
-  actions: {
-    save: function() {
-      let strain = this.get('strain');
+const { Controller } = Ember;
 
-      if (strain.get('hasDirtyAttributes')) {
-        strain.save().then((strain) => {
-          this.transitionToRoute('protected.strains.show', strain);
-        }, () => {
-          ajaxError(strain.get('errors'), this.get('flashMessages'));
-        });
-      } else {
-        strain.destroyRecord().then(() => {
-          this.transitionToRoute('protected.strains.show', strain);
-        });
-      }
-    },
-
-    cancel: function() {
-      let strain = this.get('strain');
-
-      strain.get('errors').clear();
-      strain.rollbackAttributes();
-
-      this.transitionToRoute('protected.strains.show', strain);
-    },
-
-  },
+export default Controller.extend(SaveModel, {
+  // Required for SaveModel mixin
+  fallbackRouteSave: 'protected.strains.show',
+  fallbackRouteCancel: 'protected.strains.show',
 });
diff --git a/app/pods/protected/strains/edit/route.js b/app/pods/protected/strains/edit/route.js
index ee75d51..7efe43d 100644
--- a/app/pods/protected/strains/edit/route.js
+++ b/app/pods/protected/strains/edit/route.js
@@ -1,35 +1,42 @@
 import Ember from 'ember';
+import ElevatedAccess from '../../../../mixins/elevated-access';
 
-export default Ember.Route.extend({
-  currentUser: Ember.inject.service('session-account'),
+const { Route } = Ember;
 
-  beforeModel: function(transition) {
-    this._super(transition);
-    this.get('currentUser.account').then((user) => {
-      if (user.get('isReader')) {
-        this.transitionTo('protected.strains.index');
-      }
-    });
-  },
+export default Route.extend(ElevatedAccess, {
+  // Required for ElevatedAccess mixin
+  fallbackRouteBefore: 'protected.strains.index',
+  fallbackRouteAfter: 'protected.strains.show',
 
   model: function(params) {
     return Ember.RSVP.hash({
-      strain: this.store.find('strain', params.strain_id),
+      strain: this.store.findRecord('strain', params.strain_id),
       species: this.store.findAll('species'), // Need for dropdown
     });
   },
 
+  // Overriding afterModel because of RSVP hash
   afterModel: function(models) {
-    if (!models.strain.get('canEdit')) {
-      this.transitionTo('strains.show', models.strain.get('id'));
+    if (!models.strain.get('isNew') && !models.strain.get('canEdit')) {
+      this.transitionTo(this.get('fallbackRouteAfter'), models.strain.get('id'));
     }
   },
 
+  // Setting up controller because of RSVP hash
   setupController: function(controller, models) {
-    controller.setProperties(models);
-    this.get('currentUser.account').then((user) => {
-      controller.set('metaData', user.get('metaData'));
-    });
+    controller.set('model', models.strain);
+    controller.set('speciesList', models.species);
   },
 
+  actions: {
+    // Overriding willTransition because of RSVP hash
+    willTransition: function(/*transition*/) {
+      const controller = this.get('controller');
+      const model = controller.get('model');
+
+      if (model.get('isNew')) {
+        model.destroyRecord();
+      }
+    },
+  },
 });
diff --git a/app/pods/protected/strains/edit/template.hbs b/app/pods/protected/strains/edit/template.hbs
index a4885d6..822ac93 100644
--- a/app/pods/protected/strains/edit/template.hbs
+++ b/app/pods/protected/strains/edit/template.hbs
@@ -1,8 +1,8 @@
 {{
   protected/strains/strain-form
-  strain=strain
-  species=species
+  strain=model
+  speciesList=speciesList
   canAdd=metaData.canAdd
-  save="save"
-  cancel="cancel"
+  on-save=(action "save")
+  on-cancel=(action "cancel")
 }}
diff --git a/app/pods/protected/strains/strain-form/component.js b/app/pods/protected/strains/strain-form/component.js
index 8ac9027..07841a7 100644
--- a/app/pods/protected/strains/strain-form/component.js
+++ b/app/pods/protected/strains/strain-form/component.js
@@ -1,21 +1,89 @@
 import Ember from 'ember';
+import SetupMetaData from '../../../../mixins/setup-metadata';
+
+const { Component } = Ember;
+
+export default Component.extend(SetupMetaData, {
+  // Read-only attributes
+  strain: null,
+  isNew: null,
+  isDirty: false,
+  speciesList: null,
+
+  // Actions
+  "on-save": null,
+  "on-cancel": null,
+  "on-update": null,
+
+  // Property mapping
+  propertiesList: ['strainName', 'typeStrain', 'species', 'isolatedFrom', 'accessionNumbers', 'genbank', 'wholeGenomeSequence', 'notes'],
+  strainName: null,
+  typeStrain: null,
+  species: null,
+  isolatedFrom: null,
+  accessionNumbers: null,
+  genbank: null,
+  wholeGenomeSequence: null,
+  notes: null,
+
+  resetOnInit: Ember.on('init', function() {
+    this.get('propertiesList').forEach((field) => {
+      const valueInStrain = this.get('strain').get(field);
+      this.set(field, valueInStrain);
+    });
+    // Read-only attributes
+    this.set('isNew', this.get('strain.isNew'));
+  }),
+
+  updateField: function(property, value) {
+    this.set(property, value);
+    // Manually compare against passed in value
+    if (this.get('strain').get(property) !== value) {
+      this.set('isDirty', true);
+    } else {
+      this.set('isDirty', false);
+    }
+  },
 
-export default Ember.Component.extend({
   actions: {
     save: function() {
-      this.sendAction('save');
+      return this.attrs['on-save'](this.getProperties(this.get('propertiesList')));
     },
 
     cancel: function() {
-      this.sendAction('cancel');
+      return this.attrs['on-cancel']();
+    },
+
+    strainNameDidChange: function(value) {
+      this.updateField('strainName', value);
+    },
+
+    typeStrainDidChange: function() {
+      this.updateField('typeStrain', !this.get('typeStrain'));
+    },
+
+    speciesDidChange: function(value) {
+      this.updateField('species', value);
     },
 
     isolatedFromDidChange: function(value) {
-      this.set('strain.isolatedFrom', value);
+      this.updateField('isolatedFrom', value);
+    },
+
+    accessionNumbersNameDidChange: function(value) {
+      this.updateField('accessionNumbers', value);
+    },
+
+    genbankDidChange: function(value) {
+      this.updateField('genbank', value);
+    },
+
+    wholeGenomeSequenceDidChange: function(value) {
+      this.updateField('wholeGenomeSequence', value);
     },
 
     notesDidChange: function(value) {
-      this.set('strain.notes', value);
+      this.updateField('strain.notes', value);
     },
   },
 });
diff --git a/app/pods/protected/strains/strain-form/template.hbs b/app/pods/protected/strains/strain-form/template.hbs
index fef122d..a959f4f 100644
--- a/app/pods/protected/strains/strain-form/template.hbs
+++ b/app/pods/protected/strains/strain-form/template.hbs
@@ -1,51 +1,51 @@
 <form class="grid-form" {{action 'save' on='submit'}}>
   <fieldset>
-    <legend><em>{{strain.strainName}}</em></legend>
+    <legend><em>{{strainName}}</em></legend>
     <div data-row-span="2">
       <div data-field-span="1">
         <label>Strain Name</label>
-        {{input value=strain.strainName class="strain-name"}}
+        {{one-way-input type="text" class="strain-name" value=strainName update=(action "strainNameDidChange")}}
       </div>
       <div data-field-span="1">
         <label>Type Strain?</label>
-        {{input type="checkbox" checked=strain.typeStrain}} {{if strain.typeStrain 'Yes' 'No'}}
+        <input type="checkbox" checked={{typeStrain}} value="{{typeStrain}}" onchange={{action "typeStrainDidChange"}}>
+        {{if typeStrain 'Yes' 'No'}}
       </div>
     </div>
     <div data-row-span="2">
       <div data-field-span="2">
         <label>Species</label>
-          {{
-            select-2
-            content=species
-            optionLabelPath="speciesName"
-            value=strain.species
-          }}
+        <select onchange={{action "speciesDidChange" value="target.value"}}>
+          {{#each speciesList as |speciesChoice|}}
+            <option value={{speciesChoice}} selected={{equal species speciesChoice}}>{{speciesChoice.speciesName}}</option>
+          {{/each}}
+        </select>
       </div>
     </div>
     <div data-row-span="2">
       <div data-field-span="2">
         <label>Isolated From</label>
-        {{text-editor value=strain.isolatedFrom update=(action "isolatedFromDidChange")}}
+        {{text-editor value=isolatedFrom update=(action "isolatedFromDidChange")}}
       </div>
     </div>
     <div data-row-span="3">
       <div data-field-span="1">
         <label>Accession Numbers</label>
-        {{input value=strain.accessionNumbers}}
+        {{one-way-input type="text" class="accession-numbers" value=accessionNumbers update=(action "accessionNumbersNameDidChange")}}
       </div>
       <div data-field-span="1">
         <label>GenBank</label>
-        {{input value=strain.genbank}}
+        {{one-way-input type="text" class="genbank" value=genbank update=(action "genbankDidChange")}}
       </div>
       <div data-field-span="1">
         <label>Whole Genome Sequence</label>
-        {{input value=strain.wholeGenomeSequence}}
+        {{one-way-input type="text" class="whole-genome-sequenc" value=wholeGenomeSequence update=(action "wholeGenomeSequenceDidChange")}}
       </div>
     </div>
     <div data-row-span="2">
       <div data-field-span="2">
         <label>Notes</label>
-        {{text-editor value=strain.notes update=(action "notesDidChange")}}
+        {{text-editor value=notes update=(action "notesDidChange")}}
       </div>
     </div>
   </fieldset>
@@ -61,7 +61,7 @@
   <a class="button-red smaller" {{action 'cancel'}}>
     Cancel
   </a>
-  {{#if strain.hasDirtyAttributes}}
+  {{#if isDirty}}
     <button type="submit" class="button-green smaller save-strain">
       Save
     </button>

From fa5b741e357a40f7b8ae66d7536d2cd84e15637e Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Tue, 10 Nov 2015 14:31:38 -0700
Subject: [PATCH 13/35] Refactor strains/new

---
 app/mirage/config.js                          |  2 +-
 app/mixins/save-model.js                      | 15 +++-----
 app/models/strain.js                          |  4 +++
 app/pods/protected/strains/edit/template.hbs  |  1 -
 .../strains/index/strain-table/component.js   |  2 +-
 app/pods/protected/strains/new/controller.js  | 31 ++++------------
 app/pods/protected/strains/new/route.js       | 35 +++++++++++--------
 app/pods/protected/strains/new/template.hbs   |  8 ++---
 .../strains/show/strain-card/template.hbs     |  2 +-
 .../strains/strain-form/component.js          |  3 +-
 .../strains/strain-form/template.hbs          |  2 +-
 11 files changed, 45 insertions(+), 60 deletions(-)

diff --git a/app/mirage/config.js b/app/mirage/config.js
index e39ccee..935eb17 100644
--- a/app/mirage/config.js
+++ b/app/mirage/config.js
@@ -36,7 +36,7 @@ export function testConfig() {
     return {
       strain: db.strains.find(request.params.id),
       species: db.species, // Just send back everything we've got
-    }
+    };
   });
   this.put('/strains/:id');
 }
diff --git a/app/mixins/save-model.js b/app/mixins/save-model.js
index 8d3b233..d4459e3 100644
--- a/app/mixins/save-model.js
+++ b/app/mixins/save-model.js
@@ -13,17 +13,12 @@ export default Mixin.create({
       const fallbackRoute = this.get('fallbackRouteSave');
 
       model.setProperties(properties);
-
-      if (model.get('hasDirtyAttributes')) {
-        model.save().then((model) => {
-          this.get('flashMessages').clearMessages();
-          this.transitionToRoute(fallbackRoute, model);
-        }, () => {
-          ajaxError(model.get('errors'), this.get('flashMessages'));
-        });
-      } else {
+      model.save().then((model) => {
+        this.get('flashMessages').clearMessages();
         this.transitionToRoute(fallbackRoute, model);
-      }
+      }, () => {
+        ajaxError(model.get('errors'), this.get('flashMessages'));
+      });
     },
 
     cancel: function() {
diff --git a/app/models/strain.js b/app/models/strain.js
index 72d4ff6..0c59f2f 100644
--- a/app/models/strain.js
+++ b/app/models/strain.js
@@ -25,6 +25,10 @@ export default DS.Model.extend({
     return Ember.String.htmlSafe(`${this.get('strainName')}${type}`);
   }.property('strainName', 'typeStrain').readOnly(),
 
+  fullName: Ember.computed('species', 'strainName', function() {
+    return `${this.get('species.speciesName')} ${this.get('strainNameMU')}`;
+  }),
+
   fullNameMU: function() {
     return Ember.String.htmlSafe(`<em>${this.get('species.speciesName')}</em> ${this.get('strainNameMU')}`);
   }.property('species', 'strainNameMU').readOnly(),
diff --git a/app/pods/protected/strains/edit/template.hbs b/app/pods/protected/strains/edit/template.hbs
index 822ac93..1ae2d5b 100644
--- a/app/pods/protected/strains/edit/template.hbs
+++ b/app/pods/protected/strains/edit/template.hbs
@@ -2,7 +2,6 @@
   protected/strains/strain-form
   strain=model
   speciesList=speciesList
-  canAdd=metaData.canAdd
   on-save=(action "save")
   on-cancel=(action "cancel")
 }}
diff --git a/app/pods/protected/strains/index/strain-table/component.js b/app/pods/protected/strains/index/strain-table/component.js
index 0e801ca..7b3b21c 100644
--- a/app/pods/protected/strains/index/strain-table/component.js
+++ b/app/pods/protected/strains/index/strain-table/component.js
@@ -6,7 +6,7 @@ const { Component, computed: { sort } } = Ember;
 export default Component.extend(SetupMetaData, {
   strains: null,
 
-  sortParams: ['fullNameMU'],
+  sortParams: ['fullName'],
   sortedStrains: sort('strains', 'sortParams'),
 
 });
diff --git a/app/pods/protected/strains/new/controller.js b/app/pods/protected/strains/new/controller.js
index d3e1b36..a38edbb 100644
--- a/app/pods/protected/strains/new/controller.js
+++ b/app/pods/protected/strains/new/controller.js
@@ -1,29 +1,10 @@
 import Ember from 'ember';
-import ajaxError from '../../../../utils/ajax-error';
+import SaveModel from '../../../../mixins/save-model';
 
-export default Ember.Controller.extend({
-  actions: {
-    save: function() {
-      let strain = this.get('strain');
+const { Controller } = Ember;
 
-      if (strain.get('hasDirtyAttributes')) {
-        strain.save().then((strain) => {
-          this.transitionToRoute('protected.strains.show', strain);
-        }, () => {
-          ajaxError(strain.get('errors'), this.get('flashMessages'));
-        });
-      } else {
-        strain.destroyRecord().then(() => {
-          this.transitionToRoute('protected.strains.index');
-        });
-      }
-    },
-
-    cancel: function() {
-      this.get('strain').destroyRecord().then(() => {
-        this.transitionToRoute('protected.strains.index');
-      });
-    },
-
-  },
+export default Controller.extend(SaveModel, {
+  // Required for SaveModel mixin
+  fallbackRouteSave: 'protected.strains.show',
+  fallbackRouteCancel: 'protected.strains.index',
 });
diff --git a/app/pods/protected/strains/new/route.js b/app/pods/protected/strains/new/route.js
index 837b713..a9f4c6f 100644
--- a/app/pods/protected/strains/new/route.js
+++ b/app/pods/protected/strains/new/route.js
@@ -1,16 +1,12 @@
 import Ember from 'ember';
+import ElevatedAccess from '../../../../mixins/elevated-access';
 
-export default Ember.Route.extend({
-  currentUser: Ember.inject.service('session-account'),
+const { Route } = Ember;
 
-  beforeModel: function(transition) {
-    this._super(transition);
-    this.get('currentUser.account').then((user) => {
-      if (user.get('isReader')) {
-        this.transitionTo('protected.strains.index');
-      }
-    });
-  },
+export default Route.extend(ElevatedAccess, {
+  // Required for ElevatedAccess mixin
+  fallbackRouteBefore: 'protected.strains.index',
+  fallbackRouteAfter: 'protected.strains.show',
 
   model: function() {
     return Ember.RSVP.hash({
@@ -19,19 +15,28 @@ export default Ember.Route.extend({
     });
   },
 
+  // Overriding afterModel because of RSVP hash
+  afterModel: function(models) {
+    if (!models.strain.get('isNew') && !models.strain.get('canEdit')) {
+      this.transitionTo(this.get('fallbackRouteAfter'), models.strain.get('id'));
+    }
+  },
+
+  // Setting up controller because of RSVP hash
   setupController: function(controller, models) {
-    controller.setProperties(models);
+    controller.set('model', models.strain);
+    controller.set('speciesList', models.species);
   },
 
   actions: {
+    // Overriding willTransition because of RSVP hash
     willTransition: function(/*transition*/) {
       const controller = this.get('controller');
-      const strain = controller.get('strain');
+      const model = controller.get('model');
 
-      if (strain.get('isNew')) {
-        strain.destroyRecord();
+      if (model.get('isNew')) {
+        model.destroyRecord();
       }
     },
   },
-
 });
diff --git a/app/pods/protected/strains/new/template.hbs b/app/pods/protected/strains/new/template.hbs
index d9c4d43..1ae2d5b 100644
--- a/app/pods/protected/strains/new/template.hbs
+++ b/app/pods/protected/strains/new/template.hbs
@@ -1,7 +1,7 @@
 {{
   protected/strains/strain-form
-  strain=strain
-  species=species
-  save="save"
-  cancel="cancel"
+  strain=model
+  speciesList=speciesList
+  on-save=(action "save")
+  on-cancel=(action "cancel")
 }}
diff --git a/app/pods/protected/strains/show/strain-card/template.hbs b/app/pods/protected/strains/show/strain-card/template.hbs
index 989db71..58ff553 100644
--- a/app/pods/protected/strains/show/strain-card/template.hbs
+++ b/app/pods/protected/strains/show/strain-card/template.hbs
@@ -98,7 +98,7 @@
 </div>
 {{#if strain.canEdit}}
   <br>
-  {{#link-to 'protected.strains.edit' strain class="button-gray smaller"}}
+  {{#link-to 'protected.strains.edit' strain.id class="button-gray smaller"}}
     Edit
   {{/link-to}}
   {{delete-button delete=(action 'deleteStrain')}}
diff --git a/app/pods/protected/strains/strain-form/component.js b/app/pods/protected/strains/strain-form/component.js
index 07841a7..b5544f3 100644
--- a/app/pods/protected/strains/strain-form/component.js
+++ b/app/pods/protected/strains/strain-form/component.js
@@ -63,7 +63,8 @@ export default Component.extend(SetupMetaData, {
     },
 
     speciesDidChange: function(value) {
-      this.updateField('species', value);
+      const newSpecies = this.get('speciesList').findBy('id', value);
+      this.updateField('species', newSpecies);
     },
 
     isolatedFromDidChange: function(value) {
diff --git a/app/pods/protected/strains/strain-form/template.hbs b/app/pods/protected/strains/strain-form/template.hbs
index a959f4f..ea27230 100644
--- a/app/pods/protected/strains/strain-form/template.hbs
+++ b/app/pods/protected/strains/strain-form/template.hbs
@@ -17,7 +17,7 @@
         <label>Species</label>
         <select onchange={{action "speciesDidChange" value="target.value"}}>
           {{#each speciesList as |speciesChoice|}}
-            <option value={{speciesChoice}} selected={{equal species speciesChoice}}>{{speciesChoice.speciesName}}</option>
+            <option value={{speciesChoice.id}} selected={{equal species.id speciesChoice.id}}>{{speciesChoice.speciesName}}</option>
           {{/each}}
         </select>
       </div>

From 48cd1f4f39e82513287e992662e1cb02b2dda555 Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Tue, 10 Nov 2015 14:35:17 -0700
Subject: [PATCH 14/35] Step one, move up to pod root

---
 .../strains/{show => }/measurements-table-row/component.js      | 0
 .../strains/{show => }/measurements-table-row/template.hbs      | 0
 .../strains/{show => }/measurements-table/component.js          | 0
 .../strains/{show => measurements-table}/loading/template.hbs   | 0
 .../strains/{show => }/measurements-table/template.hbs          | 2 +-
 .../strains/show/measurements-table/loading/template.hbs        | 1 -
 app/pods/protected/strains/show/strain-card/template.hbs        | 2 +-
 app/pods/protected/strains/strain-form/template.hbs             | 2 +-
 8 files changed, 3 insertions(+), 4 deletions(-)
 rename app/pods/protected/strains/{show => }/measurements-table-row/component.js (100%)
 rename app/pods/protected/strains/{show => }/measurements-table-row/template.hbs (100%)
 rename app/pods/protected/strains/{show => }/measurements-table/component.js (100%)
 rename app/pods/protected/strains/{show => measurements-table}/loading/template.hbs (100%)
 rename app/pods/protected/strains/{show => }/measurements-table/template.hbs (96%)
 delete mode 100644 app/pods/protected/strains/show/measurements-table/loading/template.hbs

diff --git a/app/pods/protected/strains/show/measurements-table-row/component.js b/app/pods/protected/strains/measurements-table-row/component.js
similarity index 100%
rename from app/pods/protected/strains/show/measurements-table-row/component.js
rename to app/pods/protected/strains/measurements-table-row/component.js
diff --git a/app/pods/protected/strains/show/measurements-table-row/template.hbs b/app/pods/protected/strains/measurements-table-row/template.hbs
similarity index 100%
rename from app/pods/protected/strains/show/measurements-table-row/template.hbs
rename to app/pods/protected/strains/measurements-table-row/template.hbs
diff --git a/app/pods/protected/strains/show/measurements-table/component.js b/app/pods/protected/strains/measurements-table/component.js
similarity index 100%
rename from app/pods/protected/strains/show/measurements-table/component.js
rename to app/pods/protected/strains/measurements-table/component.js
diff --git a/app/pods/protected/strains/show/loading/template.hbs b/app/pods/protected/strains/measurements-table/loading/template.hbs
similarity index 100%
rename from app/pods/protected/strains/show/loading/template.hbs
rename to app/pods/protected/strains/measurements-table/loading/template.hbs
diff --git a/app/pods/protected/strains/show/measurements-table/template.hbs b/app/pods/protected/strains/measurements-table/template.hbs
similarity index 96%
rename from app/pods/protected/strains/show/measurements-table/template.hbs
rename to app/pods/protected/strains/measurements-table/template.hbs
index a9a45dd..25b0c27 100644
--- a/app/pods/protected/strains/show/measurements-table/template.hbs
+++ b/app/pods/protected/strains/measurements-table/template.hbs
@@ -41,7 +41,7 @@
   <tbody>
     {{#each sortedMeasurements as |measurement|}}
       {{
-        protected/strains/show/measurements-table-row
+        protected/strains/measurements-table-row
         row=measurement
         canEdit=canEdit
       }}
diff --git a/app/pods/protected/strains/show/measurements-table/loading/template.hbs b/app/pods/protected/strains/show/measurements-table/loading/template.hbs
deleted file mode 100644
index e5a3e05..0000000
--- a/app/pods/protected/strains/show/measurements-table/loading/template.hbs
+++ /dev/null
@@ -1 +0,0 @@
-{{loading-panel}}
diff --git a/app/pods/protected/strains/show/strain-card/template.hbs b/app/pods/protected/strains/show/strain-card/template.hbs
index 58ff553..86e281f 100644
--- a/app/pods/protected/strains/show/strain-card/template.hbs
+++ b/app/pods/protected/strains/show/strain-card/template.hbs
@@ -74,7 +74,7 @@
         <dt>Characteristics</dt>
         <dd>
           {{
-            protected/strains/show/measurements-table
+            protected/strains/measurements-table
             model=strain
             canEdit=false
             canAdd=false
diff --git a/app/pods/protected/strains/strain-form/template.hbs b/app/pods/protected/strains/strain-form/template.hbs
index ea27230..908020f 100644
--- a/app/pods/protected/strains/strain-form/template.hbs
+++ b/app/pods/protected/strains/strain-form/template.hbs
@@ -51,7 +51,7 @@
   </fieldset>
   <div>
     {{
-      protected/strains/show/measurements-table
+      protected/strains/measurements-table
       model=strain
       canEdit=strain.canEdit
       canAdd=canAdd

From fbfbdf19d1bbddf92ddb1b38137b2136c8401a28 Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Tue, 10 Nov 2015 15:42:20 -0700
Subject: [PATCH 15/35] WIP

---
 app/pods/protected/strains/edit/controller.js |  8 ++++
 app/pods/protected/strains/edit/template.hbs  |  1 +
 .../measurements-table-row/component.js       |  3 +-
 .../strains/measurements-table/component.js   | 47 ++++++++++---------
 .../strains/measurements-table/template.hbs   | 20 ++++----
 .../strains/show/strain-card/template.hbs     |  2 +-
 .../strains/strain-form/component.js          |  5 ++
 .../strains/strain-form/template.hbs          |  5 +-
 8 files changed, 54 insertions(+), 37 deletions(-)

diff --git a/app/pods/protected/strains/edit/controller.js b/app/pods/protected/strains/edit/controller.js
index a44ea40..d3640c7 100644
--- a/app/pods/protected/strains/edit/controller.js
+++ b/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 }),
+      });
+    },
+  },
 });
diff --git a/app/pods/protected/strains/edit/template.hbs b/app/pods/protected/strains/edit/template.hbs
index 1ae2d5b..c36aa54 100644
--- a/app/pods/protected/strains/edit/template.hbs
+++ b/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")
 }}
diff --git a/app/pods/protected/strains/measurements-table-row/component.js b/app/pods/protected/strains/measurements-table-row/component.js
index 8ad1ba7..c6983d7 100644
--- a/app/pods/protected/strains/measurements-table-row/component.js
+++ b/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();
diff --git a/app/pods/protected/strains/measurements-table/component.js b/app/pods/protected/strains/measurements-table/component.js
index 6cbf880..a83e25e 100644
--- a/app/pods/protected/strains/measurements-table/component.js
+++ b/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() {
-    return this.get('model.measurements.length') > 0;
-  }.property('model.measurements'),
-
-  fetchCharacteristics: function() {
-    if (this.get('canEdit')) {
-      this.store.findAll('characteristic');
-    }
-  }.on('didInsertElement'),
+  // Passed in
+  strain: null,
+  allCharacteristics: null,
+  canEdit: false,
+  canAdd: false,
+  "add-characteristic": null,
 
+  // 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', {
-        sortOrder: -999
-      });
-      const m = this.store.createRecord('measurement', {
-        characteristic: c
-      });
-      this.get('model.measurements').addObject(m);
+      const newChar = this.attrs['add-characteristic']();
+      this.get('strain.measurements').addObject(newChar);
     },
 
     changeSortParam: function(col) {
-      let sort = this.get('sortAsc') ? 'asc' : 'desc';
-      let sortCol = `${col}:${sort}`;
-      this.set('sortParams', [sortCol]);
+      const sort = this.get('sortAsc') ? 'asc' : 'desc';
+      this.set('sortParams', [`${col}:${sort}`]);
       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;
     },
   },
 
diff --git a/app/pods/protected/strains/measurements-table/template.hbs b/app/pods/protected/strains/measurements-table/template.hbs
index 25b0c27..3f635f1 100644
--- a/app/pods/protected/strains/measurements-table/template.hbs
+++ b/app/pods/protected/strains/measurements-table/template.hbs
@@ -1,17 +1,17 @@
 {{#if canAdd}}
-<br>
-<button class="button-green smaller" {{action "addCharacteristic"}}>
-  Add characteristic
-</button>
-<br><br>
+  <br>
+  <button class="button-green smaller" {{action "addCharacteristic"}}>
+    Add characteristic
+  </button>
+  <br><br>
 {{/if}}
 
 {{#if measurementsPresent}}
-{{#if paramsChanged}}
-  <button class="button-gray smaller" {{action 'resetSortParam'}}>
-    Reset sort
-  </button>
-{{/if}}
+  {{#if paramsChanged}}
+    <button class="button-gray smaller" {{action 'resetSortParam'}}>
+      Reset sort
+    </button>
+  {{/if}}
 <table class="flakes-table">
   <colgroup>
     {{#if canEdit}}
diff --git a/app/pods/protected/strains/show/strain-card/template.hbs b/app/pods/protected/strains/show/strain-card/template.hbs
index 86e281f..995326a 100644
--- a/app/pods/protected/strains/show/strain-card/template.hbs
+++ b/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
           }}
diff --git a/app/pods/protected/strains/strain-form/component.js b/app/pods/protected/strains/strain-form/component.js
index b5544f3..11ef764 100644
--- a/app/pods/protected/strains/strain-form/component.js
+++ b/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);
     },
diff --git a/app/pods/protected/strains/strain-form/template.hbs b/app/pods/protected/strains/strain-form/template.hbs
index 908020f..21e5bcf 100644
--- a/app/pods/protected/strains/strain-form/template.hbs
+++ b/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>

From ba15af411ee2f674cfe6c340f495519bd0ad8453 Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Tue, 10 Nov 2015 15:49:55 -0700
Subject: [PATCH 16/35] Stop loading chars in measurements-table

---
 app/pods/protected/strains/edit/route.js                 | 2 ++
 app/pods/protected/strains/edit/template.hbs             | 1 +
 .../strains/measurements-table-row/component.js          | 2 --
 .../strains/measurements-table-row/template.hbs          | 2 +-
 .../protected/strains/measurements-table/component.js    | 9 ++-------
 .../protected/strains/measurements-table/template.hbs    | 1 +
 app/pods/protected/strains/strain-form/component.js      | 1 +
 app/pods/protected/strains/strain-form/template.hbs      | 1 +
 8 files changed, 9 insertions(+), 10 deletions(-)

diff --git a/app/pods/protected/strains/edit/route.js b/app/pods/protected/strains/edit/route.js
index 7efe43d..f8d8619 100644
--- a/app/pods/protected/strains/edit/route.js
+++ b/app/pods/protected/strains/edit/route.js
@@ -12,6 +12,7 @@ export default Route.extend(ElevatedAccess, {
     return Ember.RSVP.hash({
       strain: this.store.findRecord('strain', params.strain_id),
       species: this.store.findAll('species'), // Need for dropdown
+      characteristics: this.store.findAll('characteristic'), // Need for dropdown
     });
   },
 
@@ -26,6 +27,7 @@ export default Route.extend(ElevatedAccess, {
   setupController: function(controller, models) {
     controller.set('model', models.strain);
     controller.set('speciesList', models.species);
+    controller.set('allCharacteristics', models.characteristics);
   },
 
   actions: {
diff --git a/app/pods/protected/strains/edit/template.hbs b/app/pods/protected/strains/edit/template.hbs
index c36aa54..727510c 100644
--- a/app/pods/protected/strains/edit/template.hbs
+++ b/app/pods/protected/strains/edit/template.hbs
@@ -3,6 +3,7 @@
   strain=model
   speciesList=speciesList
   add-characteristic=(action "addCharacteristic")
+  allCharacteristics=allCharacteristics
   on-save=(action "save")
   on-cancel=(action "cancel")
 }}
diff --git a/app/pods/protected/strains/measurements-table-row/component.js b/app/pods/protected/strains/measurements-table-row/component.js
index c6983d7..8c265bc 100644
--- a/app/pods/protected/strains/measurements-table-row/component.js
+++ b/app/pods/protected/strains/measurements-table-row/component.js
@@ -20,8 +20,6 @@ export default Component.extend({
 
   actions: {
     edit: function() {
-      // The parent table fetches all of the characteristics ahead of time
-      this.set('characteristics', this.store.peekAll('characteristic'));
       this.toggleProperty('isEditing');
     },
 
diff --git a/app/pods/protected/strains/measurements-table-row/template.hbs b/app/pods/protected/strains/measurements-table-row/template.hbs
index a26fef4..61a7140 100644
--- a/app/pods/protected/strains/measurements-table-row/template.hbs
+++ b/app/pods/protected/strains/measurements-table-row/template.hbs
@@ -4,7 +4,7 @@
     {{
       select-2
       multiple=false
-      content=characteristics
+      content=allCharacteristics
       value=row.characteristic
       optionLabelPath="characteristicName"
     }}
diff --git a/app/pods/protected/strains/measurements-table/component.js b/app/pods/protected/strains/measurements-table/component.js
index a83e25e..497dd39 100644
--- a/app/pods/protected/strains/measurements-table/component.js
+++ b/app/pods/protected/strains/measurements-table/component.js
@@ -9,6 +9,8 @@ export default Component.extend({
   allCharacteristics: null,
   canEdit: false,
   canAdd: false,
+
+  // Actions
   "add-characteristic": null,
 
   // Properties
@@ -20,13 +22,6 @@ export default Component.extend({
     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 newChar = this.attrs['add-characteristic']();
diff --git a/app/pods/protected/strains/measurements-table/template.hbs b/app/pods/protected/strains/measurements-table/template.hbs
index 3f635f1..288421e 100644
--- a/app/pods/protected/strains/measurements-table/template.hbs
+++ b/app/pods/protected/strains/measurements-table/template.hbs
@@ -43,6 +43,7 @@
       {{
         protected/strains/measurements-table-row
         row=measurement
+        allCharacteristics=allCharacteristics
         canEdit=canEdit
       }}
     {{/each}}
diff --git a/app/pods/protected/strains/strain-form/component.js b/app/pods/protected/strains/strain-form/component.js
index 11ef764..b224aef 100644
--- a/app/pods/protected/strains/strain-form/component.js
+++ b/app/pods/protected/strains/strain-form/component.js
@@ -9,6 +9,7 @@ export default Component.extend(SetupMetaData, {
   isNew: null,
   isDirty: false,
   speciesList: null,
+  allCharacteristics: null,
 
   // Actions
   "on-save": null,
diff --git a/app/pods/protected/strains/strain-form/template.hbs b/app/pods/protected/strains/strain-form/template.hbs
index 21e5bcf..0200ea7 100644
--- a/app/pods/protected/strains/strain-form/template.hbs
+++ b/app/pods/protected/strains/strain-form/template.hbs
@@ -54,6 +54,7 @@
       protected/strains/measurements-table
       strain=strain
       add-characteristic=(action "addCharacteristic")
+      allCharacteristics=allCharacteristics
       canEdit=strain.canEdit
       canAdd=metaData.canAdd
     }}

From 15474c76f39e65a628fc88998e7c1998b4311023 Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Tue, 10 Nov 2015 15:55:52 -0700
Subject: [PATCH 17/35] Rename row -> measurement

---
 .../measurements-table-row/component.js       | 19 ++++++++++---------
 .../measurements-table-row/template.hbs       | 16 ++++++++--------
 .../strains/measurements-table/template.hbs   |  2 +-
 3 files changed, 19 insertions(+), 18 deletions(-)

diff --git a/app/pods/protected/strains/measurements-table-row/component.js b/app/pods/protected/strains/measurements-table-row/component.js
index 8c265bc..62bf56f 100644
--- a/app/pods/protected/strains/measurements-table-row/component.js
+++ b/app/pods/protected/strains/measurements-table-row/component.js
@@ -1,21 +1,22 @@
 import Ember from 'ember';
 import ajaxError from '../../../../utils/ajax-error';
 
-const { Component } = Ember;
+const { Component, computed } = Ember;
 
 export default Component.extend({
   tagName: 'tr',
   isEditing: false,
   allCharacteristics: null,
+  measurement: null,
 
   oldCharacteristicId: function() {
-    let json = this.get('row').toJSON();
+    let json = this.get('measurement').toJSON();
     return json.characteristic;
   }.property(),
 
-  rowChanged: Ember.computed('row.notes', 'row.value', 'row.characteristic.id', function() {
-    return this.get('row.hasDirtyAttributes') ||
-      this.get('oldCharacteristicId') !== this.get('row.characteristic.id');
+  rowChanged: computed('measurement.notes', 'measurement.value', 'measurement.characteristic.id', function() {
+    return this.get('measurement.hasDirtyAttributes') ||
+      this.get('oldCharacteristicId') !== this.get('measurement.characteristic.id');
   }),
 
   actions: {
@@ -25,11 +26,11 @@ export default Component.extend({
 
     save: function() {
       if (this.get('rowChanged')) {
-        this.get('row').save().then(() => {
+        this.get('measurement').save().then(() => {
           this.get('flashMessages').clearMessages();
           this.toggleProperty('isEditing');
         }, () => {
-          ajaxError(this.get('row.errors'), this.get('flashMessages'));
+          ajaxError(this.get('measurement.errors'), this.get('flashMessages'));
         });
       } else {
         this.toggleProperty('isEditing');
@@ -37,11 +38,11 @@ export default Component.extend({
     },
 
     delete: function() {
-      let char = this.get('row.characteristic');
+      let char = this.get('measurement.characteristic');
       if (char.get('isNew')) {
         char.destroyRecord();
       }
-      this.get('row').destroyRecord();
+      this.get('measurement').destroyRecord();
     }
 
   },
diff --git a/app/pods/protected/strains/measurements-table-row/template.hbs b/app/pods/protected/strains/measurements-table-row/template.hbs
index 61a7140..cab5554 100644
--- a/app/pods/protected/strains/measurements-table-row/template.hbs
+++ b/app/pods/protected/strains/measurements-table-row/template.hbs
@@ -5,15 +5,15 @@
       select-2
       multiple=false
       content=allCharacteristics
-      value=row.characteristic
+      value=measurement.characteristic
       optionLabelPath="characteristicName"
     }}
   </td>
   <td>
-    {{input value=row.value}}
+    {{input value=measurement.value}}
   </td>
   <td>
-    {{input value=row.notes}}
+    {{input value=measurement.notes}}
   </td>
   {{#if canEdit}}
     <td>
@@ -30,18 +30,18 @@
   {{/if}}
 {{else}}
   <td>
-    {{{row.characteristic.characteristicTypeName}}}
+    {{{measurement.characteristic.characteristicTypeName}}}
   </td>
   <td>
-    {{#link-to 'protected.characteristics.show' row.characteristic.id}}
-      {{{row.characteristic.characteristicName}}}
+    {{#link-to 'protected.characteristics.show' measurement.characteristic.id}}
+      {{{measurement.characteristic.characteristicName}}}
     {{/link-to}}
   </td>
   <td>
-    {{row.value}}
+    {{measurement.value}}
   </td>
   <td>
-    {{row.notes}}
+    {{measurement.notes}}
   </td>
   {{#if canEdit}}
     <td>
diff --git a/app/pods/protected/strains/measurements-table/template.hbs b/app/pods/protected/strains/measurements-table/template.hbs
index 288421e..146d44e 100644
--- a/app/pods/protected/strains/measurements-table/template.hbs
+++ b/app/pods/protected/strains/measurements-table/template.hbs
@@ -42,7 +42,7 @@
     {{#each sortedMeasurements as |measurement|}}
       {{
         protected/strains/measurements-table-row
-        row=measurement
+        measurement=measurement
         allCharacteristics=allCharacteristics
         canEdit=canEdit
       }}

From b2fa171d1776364fd29f89b452f511531d7610ce Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Tue, 10 Nov 2015 16:08:40 -0700
Subject: [PATCH 18/35] Save measurement

---
 app/pods/protected/strains/edit/controller.js       | 10 ++++++++++
 app/pods/protected/strains/edit/template.hbs        |  1 +
 .../strains/measurements-table-row/component.js     | 13 ++-----------
 .../strains/measurements-table/component.js         |  5 +++++
 .../strains/measurements-table/template.hbs         |  1 +
 app/pods/protected/strains/strain-form/component.js |  5 +++++
 app/pods/protected/strains/strain-form/template.hbs |  1 +
 7 files changed, 25 insertions(+), 11 deletions(-)

diff --git a/app/pods/protected/strains/edit/controller.js b/app/pods/protected/strains/edit/controller.js
index d3640c7..0bc7bc0 100644
--- a/app/pods/protected/strains/edit/controller.js
+++ b/app/pods/protected/strains/edit/controller.js
@@ -1,5 +1,6 @@
 import Ember from 'ember';
 import SaveModel from '../../../../mixins/save-model';
+import ajaxError from '../../../../utils/ajax-error';
 
 const { Controller } = Ember;
 
@@ -14,5 +15,14 @@ export default Controller.extend(SaveModel, {
         characteristic: this.store.createRecord('characteristic', { sortOrder: -999 }),
       });
     },
+
+    saveMeasurement: function(measurement) {
+      measurement.save().then(() => {
+        this.get('flashMessages').clearMessages();
+      }, () => {
+        ajaxError(measurement.get('errors'), this.get('flashMessages'));
+      });
+    },
+
   },
 });
diff --git a/app/pods/protected/strains/edit/template.hbs b/app/pods/protected/strains/edit/template.hbs
index 727510c..f252090 100644
--- a/app/pods/protected/strains/edit/template.hbs
+++ b/app/pods/protected/strains/edit/template.hbs
@@ -4,6 +4,7 @@
   speciesList=speciesList
   add-characteristic=(action "addCharacteristic")
   allCharacteristics=allCharacteristics
+  save-measurement=(action "saveMeasurement")
   on-save=(action "save")
   on-cancel=(action "cancel")
 }}
diff --git a/app/pods/protected/strains/measurements-table-row/component.js b/app/pods/protected/strains/measurements-table-row/component.js
index 62bf56f..0c7f96c 100644
--- a/app/pods/protected/strains/measurements-table-row/component.js
+++ b/app/pods/protected/strains/measurements-table-row/component.js
@@ -1,5 +1,4 @@
 import Ember from 'ember';
-import ajaxError from '../../../../utils/ajax-error';
 
 const { Component, computed } = Ember;
 
@@ -25,16 +24,8 @@ export default Component.extend({
     },
 
     save: function() {
-      if (this.get('rowChanged')) {
-        this.get('measurement').save().then(() => {
-          this.get('flashMessages').clearMessages();
-          this.toggleProperty('isEditing');
-        }, () => {
-          ajaxError(this.get('measurement.errors'), this.get('flashMessages'));
-        });
-      } else {
-        this.toggleProperty('isEditing');
-      }
+      this.attrs['save-measurement'](this.get('measurement'));
+      this.toggleProperty('isEditing');
     },
 
     delete: function() {
diff --git a/app/pods/protected/strains/measurements-table/component.js b/app/pods/protected/strains/measurements-table/component.js
index 497dd39..a797815 100644
--- a/app/pods/protected/strains/measurements-table/component.js
+++ b/app/pods/protected/strains/measurements-table/component.js
@@ -12,6 +12,7 @@ export default Component.extend({
 
   // Actions
   "add-characteristic": null,
+  "save-measurement": null,
 
   // Properties
   sortParams: ['characteristic.characteristicTypeName', 'characteristic.sortOrder', 'characteristic.characteristicName'],
@@ -40,6 +41,10 @@ export default Component.extend({
       this.set('paramsChanged', false);
       this.set('sortAsc', true);
     },
+
+    saveMeasurement: function(measurement) {
+      return this.attrs['save-measurement'](measurement);
+    },
   },
 
 });
diff --git a/app/pods/protected/strains/measurements-table/template.hbs b/app/pods/protected/strains/measurements-table/template.hbs
index 146d44e..5a813d7 100644
--- a/app/pods/protected/strains/measurements-table/template.hbs
+++ b/app/pods/protected/strains/measurements-table/template.hbs
@@ -43,6 +43,7 @@
       {{
         protected/strains/measurements-table-row
         measurement=measurement
+        save-measurement=(action "saveMeasurement")
         allCharacteristics=allCharacteristics
         canEdit=canEdit
       }}
diff --git a/app/pods/protected/strains/strain-form/component.js b/app/pods/protected/strains/strain-form/component.js
index b224aef..c291343 100644
--- a/app/pods/protected/strains/strain-form/component.js
+++ b/app/pods/protected/strains/strain-form/component.js
@@ -16,6 +16,7 @@ export default Component.extend(SetupMetaData, {
   "on-cancel": null,
   "on-update": null,
   "add-characteristic": null,
+  "save-measurement": null,
 
   // Property mapping
   propertiesList: ['strainName', 'typeStrain', 'species', 'isolatedFrom', 'accessionNumbers', 'genbank', 'wholeGenomeSequence', 'notes'],
@@ -60,6 +61,10 @@ export default Component.extend(SetupMetaData, {
       return this.attrs['add-characteristic']();
     },
 
+    saveMeasurement: function(measurement) {
+      return this.attrs['save-measurement'](measurement);
+    },
+
     strainNameDidChange: function(value) {
       this.updateField('strainName', value);
     },
diff --git a/app/pods/protected/strains/strain-form/template.hbs b/app/pods/protected/strains/strain-form/template.hbs
index 0200ea7..9999954 100644
--- a/app/pods/protected/strains/strain-form/template.hbs
+++ b/app/pods/protected/strains/strain-form/template.hbs
@@ -55,6 +55,7 @@
       strain=strain
       add-characteristic=(action "addCharacteristic")
       allCharacteristics=allCharacteristics
+      save-measurement=(action "saveMeasurement")
       canEdit=strain.canEdit
       canAdd=metaData.canAdd
     }}

From c1ff0d57b46cec89379001c9caa141effa147ba8 Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Tue, 10 Nov 2015 16:14:02 -0700
Subject: [PATCH 19/35] Delete measurement

---
 app/pods/protected/strains/edit/controller.js      |  8 ++++++++
 app/pods/protected/strains/edit/template.hbs       |  1 +
 .../strains/measurements-table-row/component.js    | 14 +++++++-------
 .../strains/measurements-table/component.js        |  5 +++++
 .../strains/measurements-table/template.hbs        |  1 +
 .../protected/strains/strain-form/component.js     |  5 +++++
 .../protected/strains/strain-form/template.hbs     |  1 +
 7 files changed, 28 insertions(+), 7 deletions(-)

diff --git a/app/pods/protected/strains/edit/controller.js b/app/pods/protected/strains/edit/controller.js
index 0bc7bc0..17f47d6 100644
--- a/app/pods/protected/strains/edit/controller.js
+++ b/app/pods/protected/strains/edit/controller.js
@@ -24,5 +24,13 @@ export default Controller.extend(SaveModel, {
       });
     },
 
+    deleteMeasurement: function(measurement) {
+      const characteristic = measurement.get('characteristic');
+      if (characteristic.get('isNew')) {
+        characteristic.destroyRecord();
+      }
+      measurement.destroyRecord();
+    },
+
   },
 });
diff --git a/app/pods/protected/strains/edit/template.hbs b/app/pods/protected/strains/edit/template.hbs
index f252090..59d1633 100644
--- a/app/pods/protected/strains/edit/template.hbs
+++ b/app/pods/protected/strains/edit/template.hbs
@@ -5,6 +5,7 @@
   add-characteristic=(action "addCharacteristic")
   allCharacteristics=allCharacteristics
   save-measurement=(action "saveMeasurement")
+  delete-measurement=(action "deleteMeasurement")
   on-save=(action "save")
   on-cancel=(action "cancel")
 }}
diff --git a/app/pods/protected/strains/measurements-table-row/component.js b/app/pods/protected/strains/measurements-table-row/component.js
index 0c7f96c..cba52a6 100644
--- a/app/pods/protected/strains/measurements-table-row/component.js
+++ b/app/pods/protected/strains/measurements-table-row/component.js
@@ -8,8 +8,12 @@ export default Component.extend({
   allCharacteristics: null,
   measurement: null,
 
+  // Actions
+  "save-measurement": null,
+  "delete-measurement": null,
+
   oldCharacteristicId: function() {
-    let json = this.get('measurement').toJSON();
+    const json = this.get('measurement').toJSON();
     return json.characteristic;
   }.property(),
 
@@ -29,12 +33,8 @@ export default Component.extend({
     },
 
     delete: function() {
-      let char = this.get('measurement.characteristic');
-      if (char.get('isNew')) {
-        char.destroyRecord();
-      }
-      this.get('measurement').destroyRecord();
-    }
+      this.attrs['delete-measurement'](this.get('measurement'));
+    },
 
   },
 });
diff --git a/app/pods/protected/strains/measurements-table/component.js b/app/pods/protected/strains/measurements-table/component.js
index a797815..ab0b72e 100644
--- a/app/pods/protected/strains/measurements-table/component.js
+++ b/app/pods/protected/strains/measurements-table/component.js
@@ -13,6 +13,7 @@ export default Component.extend({
   // Actions
   "add-characteristic": null,
   "save-measurement": null,
+  "delete-measurement": null,
 
   // Properties
   sortParams: ['characteristic.characteristicTypeName', 'characteristic.sortOrder', 'characteristic.characteristicName'],
@@ -45,6 +46,10 @@ export default Component.extend({
     saveMeasurement: function(measurement) {
       return this.attrs['save-measurement'](measurement);
     },
+
+    deleteMeasurement: function(measurement) {
+      return this.attrs['delete-measurement'](measurement);
+    },
   },
 
 });
diff --git a/app/pods/protected/strains/measurements-table/template.hbs b/app/pods/protected/strains/measurements-table/template.hbs
index 5a813d7..9e22393 100644
--- a/app/pods/protected/strains/measurements-table/template.hbs
+++ b/app/pods/protected/strains/measurements-table/template.hbs
@@ -44,6 +44,7 @@
         protected/strains/measurements-table-row
         measurement=measurement
         save-measurement=(action "saveMeasurement")
+        delete-measurement=(action "deleteMeasurement")
         allCharacteristics=allCharacteristics
         canEdit=canEdit
       }}
diff --git a/app/pods/protected/strains/strain-form/component.js b/app/pods/protected/strains/strain-form/component.js
index c291343..be823bc 100644
--- a/app/pods/protected/strains/strain-form/component.js
+++ b/app/pods/protected/strains/strain-form/component.js
@@ -17,6 +17,7 @@ export default Component.extend(SetupMetaData, {
   "on-update": null,
   "add-characteristic": null,
   "save-measurement": null,
+  "delete-measurement": null,
 
   // Property mapping
   propertiesList: ['strainName', 'typeStrain', 'species', 'isolatedFrom', 'accessionNumbers', 'genbank', 'wholeGenomeSequence', 'notes'],
@@ -65,6 +66,10 @@ export default Component.extend(SetupMetaData, {
       return this.attrs['save-measurement'](measurement);
     },
 
+    deleteMeasurement: function(measurement) {
+      return this.attrs['delete-measurement'](measurement);
+    },
+
     strainNameDidChange: function(value) {
       this.updateField('strainName', value);
     },
diff --git a/app/pods/protected/strains/strain-form/template.hbs b/app/pods/protected/strains/strain-form/template.hbs
index 9999954..47c5112 100644
--- a/app/pods/protected/strains/strain-form/template.hbs
+++ b/app/pods/protected/strains/strain-form/template.hbs
@@ -56,6 +56,7 @@
       add-characteristic=(action "addCharacteristic")
       allCharacteristics=allCharacteristics
       save-measurement=(action "saveMeasurement")
+      delete-measurement=(action "deleteMeasurement")
       canEdit=strain.canEdit
       canAdd=metaData.canAdd
     }}

From 1ee0e0e5ed80707035a7abbee2295582f0cfdfc9 Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Tue, 10 Nov 2015 21:21:59 -0700
Subject: [PATCH 20/35] DDAU (WIP)

---
 app/pods/protected/strains/edit/controller.js |  3 +-
 .../measurements-table-row/component.js       | 44 +++++++++++++++----
 .../measurements-table-row/template.hbs       |  8 ++--
 .../strains/measurements-table/component.js   |  4 +-
 .../strains/strain-form/component.js          |  4 +-
 5 files changed, 46 insertions(+), 17 deletions(-)

diff --git a/app/pods/protected/strains/edit/controller.js b/app/pods/protected/strains/edit/controller.js
index 17f47d6..4d68665 100644
--- a/app/pods/protected/strains/edit/controller.js
+++ b/app/pods/protected/strains/edit/controller.js
@@ -16,7 +16,8 @@ export default Controller.extend(SaveModel, {
       });
     },
 
-    saveMeasurement: function(measurement) {
+    saveMeasurement: function(measurement, properties) {
+      measurement.setProperties(properties);
       measurement.save().then(() => {
         this.get('flashMessages').clearMessages();
       }, () => {
diff --git a/app/pods/protected/strains/measurements-table-row/component.js b/app/pods/protected/strains/measurements-table-row/component.js
index cba52a6..b818de6 100644
--- a/app/pods/protected/strains/measurements-table-row/component.js
+++ b/app/pods/protected/strains/measurements-table-row/component.js
@@ -4,31 +4,47 @@ const { Component, computed } = Ember;
 
 export default Component.extend({
   tagName: 'tr',
+
+  // Read-only attributes
   isEditing: false,
   allCharacteristics: null,
   measurement: null,
+  isDirty: null,
 
   // Actions
   "save-measurement": null,
   "delete-measurement": null,
 
-  oldCharacteristicId: function() {
-    const json = this.get('measurement').toJSON();
-    return json.characteristic;
-  }.property(),
+  // Property mapping
+  propertiesList: ['characteristic', 'value', 'notes'],
+  characteristic: null,
+  value: null,
+  notes: null,
 
-  rowChanged: computed('measurement.notes', 'measurement.value', 'measurement.characteristic.id', function() {
-    return this.get('measurement.hasDirtyAttributes') ||
-      this.get('oldCharacteristicId') !== this.get('measurement.characteristic.id');
+  resetOnInit: Ember.on('init', function() {
+    this.get('propertiesList').forEach((field) => {
+      const valueInMeasurement = this.get('measurement').get(field);
+      this.set(field, valueInMeasurement);
+    });
   }),
 
+  updateField: function(property, value) {
+    this.set(property, value);
+    // Manually compare against passed in value
+    if (this.get('measurement').get(property) !== value) {
+      this.set('isDirty', true);
+    } else {
+      this.set('isDirty', false);
+    }
+  },
+
   actions: {
     edit: function() {
       this.toggleProperty('isEditing');
     },
 
     save: function() {
-      this.attrs['save-measurement'](this.get('measurement'));
+      this.attrs['save-measurement'](this.get('measurement'), this.getProperties(this.get('propertiesList')));
       this.toggleProperty('isEditing');
     },
 
@@ -36,5 +52,17 @@ export default Component.extend({
       this.attrs['delete-measurement'](this.get('measurement'));
     },
 
+    characteristicDidChange: function(value) {
+      this.updateField('characteristic', value);
+    },
+
+    valueDidChange: function(value) {
+      this.updateField('value', value);
+    },
+
+    notesDidChange: function(value) {
+      this.updateField('notes', value);
+    },
+
   },
 });
diff --git a/app/pods/protected/strains/measurements-table-row/template.hbs b/app/pods/protected/strains/measurements-table-row/template.hbs
index cab5554..5bb44f0 100644
--- a/app/pods/protected/strains/measurements-table-row/template.hbs
+++ b/app/pods/protected/strains/measurements-table-row/template.hbs
@@ -5,19 +5,19 @@
       select-2
       multiple=false
       content=allCharacteristics
-      value=measurement.characteristic
+      value=characteristic
       optionLabelPath="characteristicName"
     }}
   </td>
   <td>
-    {{input value=measurement.value}}
+    {{one-way-input type="text" class="measurement-value" value=value update=(action "valueDidChange")}}
   </td>
   <td>
-    {{input value=measurement.notes}}
+    {{one-way-input type="text" class="measurement-notes" value=notes update=(action "notesDidChange")}}
   </td>
   {{#if canEdit}}
     <td>
-      {{#if rowChanged}}
+      {{#if isDirty}}
       <button class="button-green smaller" {{action 'save'}}>
         Save
       </button>
diff --git a/app/pods/protected/strains/measurements-table/component.js b/app/pods/protected/strains/measurements-table/component.js
index ab0b72e..e4fd3a1 100644
--- a/app/pods/protected/strains/measurements-table/component.js
+++ b/app/pods/protected/strains/measurements-table/component.js
@@ -43,8 +43,8 @@ export default Component.extend({
       this.set('sortAsc', true);
     },
 
-    saveMeasurement: function(measurement) {
-      return this.attrs['save-measurement'](measurement);
+    saveMeasurement: function(measurement, properties) {
+      return this.attrs['save-measurement'](measurement, properties);
     },
 
     deleteMeasurement: function(measurement) {
diff --git a/app/pods/protected/strains/strain-form/component.js b/app/pods/protected/strains/strain-form/component.js
index be823bc..420db5f 100644
--- a/app/pods/protected/strains/strain-form/component.js
+++ b/app/pods/protected/strains/strain-form/component.js
@@ -62,8 +62,8 @@ export default Component.extend(SetupMetaData, {
       return this.attrs['add-characteristic']();
     },
 
-    saveMeasurement: function(measurement) {
-      return this.attrs['save-measurement'](measurement);
+    saveMeasurement: function(measurement, properties) {
+      return this.attrs['save-measurement'](measurement, properties);
     },
 
     deleteMeasurement: function(measurement) {

From d46b3fc48be4e5858d26307b33da9c65776e7aa0 Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Tue, 10 Nov 2015 21:32:49 -0700
Subject: [PATCH 21/35] Dropping select 2 for now

---
 .../strains/measurements-table-row/component.js      |  3 ++-
 .../strains/measurements-table-row/template.hbs      | 12 +++++-------
 2 files changed, 7 insertions(+), 8 deletions(-)

diff --git a/app/pods/protected/strains/measurements-table-row/component.js b/app/pods/protected/strains/measurements-table-row/component.js
index b818de6..c3e5c89 100644
--- a/app/pods/protected/strains/measurements-table-row/component.js
+++ b/app/pods/protected/strains/measurements-table-row/component.js
@@ -53,7 +53,8 @@ export default Component.extend({
     },
 
     characteristicDidChange: function(value) {
-      this.updateField('characteristic', value);
+      const newCharacteristic = this.get('allCharacteristics').findBy('id', value);
+      this.updateField('characteristic', newCharacteristic);
     },
 
     valueDidChange: function(value) {
diff --git a/app/pods/protected/strains/measurements-table-row/template.hbs b/app/pods/protected/strains/measurements-table-row/template.hbs
index 5bb44f0..0dc03b7 100644
--- a/app/pods/protected/strains/measurements-table-row/template.hbs
+++ b/app/pods/protected/strains/measurements-table-row/template.hbs
@@ -1,13 +1,11 @@
 {{#if isEditing}}
   <td></td>
   <td>
-    {{
-      select-2
-      multiple=false
-      content=allCharacteristics
-      value=characteristic
-      optionLabelPath="characteristicName"
-    }}
+    <select onchange={{action "characteristicDidChange" value="target.value"}}>
+      {{#each allCharacteristics as |characteristicChoice|}}
+        <option value={{characteristicChoice.id}} selected={{equal characteristic.id characteristicChoice.id}}>{{characteristicChoice.characteristicName}}</option>
+      {{/each}}
+    </select>
   </td>
   <td>
     {{one-way-input type="text" class="measurement-value" value=value update=(action "valueDidChange")}}

From 90deef228bf18e5290b34c3053d692551cbea44d Mon Sep 17 00:00:00 2001
From: Matthew Dillon <matthewrdillon@gmail.com>
Date: Wed, 11 Nov 2015 11:52:52 -0700
Subject: [PATCH 22/35] Linting

---
 app/pods/protected/strains/measurements-table-row/component.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/pods/protected/strains/measurements-table-row/component.js b/app/pods/protected/strains/measurements-table-row/component.js
index c3e5c89..ad63dc9 100644
--- a/app/pods/protected/strains/measurements-table-row/component.js
+++ b/app/pods/protected/strains/measurements-table-row/component.js
@@ -1,6 +1,6 @@
 import Ember from 'ember';
 
-const { Component, computed } = Ember;
+const { Component } = Ember;
 
 export default Component.extend({
   tagName: 'tr',

From 2d5e2c4be76c4c2ca66b0049a3b46f71b11da91e Mon Sep 17 00:00:00 2001
From: Matthew Dillon <matthewrdillon@gmail.com>
Date: Wed, 11 Nov 2015 12:48:21 -0700
Subject: [PATCH 23/35] WIP

---
 app/pods/login/login-form/component.js | 22 ++++++++++++++++++++++
 app/pods/login/login-form/template.hbs | 10 ++++++++++
 app/pods/login/route.js                |  4 +++-
 app/pods/login/template.hbs            | 11 +----------
 4 files changed, 36 insertions(+), 11 deletions(-)
 create mode 100644 app/pods/login/login-form/component.js
 create mode 100644 app/pods/login/login-form/template.hbs

diff --git a/app/pods/login/login-form/component.js b/app/pods/login/login-form/component.js
new file mode 100644
index 0000000..56a0f99
--- /dev/null
+++ b/app/pods/login/login-form/component.js
@@ -0,0 +1,22 @@
+import Ember from 'ember';
+
+const { Component } = Ember;
+
+export default Component.extend({
+  // Actions
+  "on-save": null,
+  "on-cancel": null,
+  "on-update": null,
+  "add-characteristic": null,
+  "save-measurement": null,
+  "delete-measurement": null,
+
+  // Property mapping
+  propertiesList: ['identification', 'password'],
+  identification: null,
+  password: null,
+
+  updateField: function(property, value) {
+    this.set(property, value);
+  },
+});
diff --git a/app/pods/login/login-form/template.hbs b/app/pods/login/login-form/template.hbs
new file mode 100644
index 0000000..8492f1c
--- /dev/null
+++ b/app/pods/login/login-form/template.hbs
@@ -0,0 +1,10 @@
+<form {{action "authenticate" on="submit"}}>
+  <h2>Log In</h2>
+    {{input value=identification type="text" placeholder="Email"}}
+    {{input value=password type="password" placeholder="Password"}}
+    {{input class="button-gray" type="submit" value="Log In"}}
+</form>
+<br>
+<div>
+  {{link-to 'Forget your password?' 'users.requestlockouthelp'}}
+</div>
diff --git a/app/pods/login/route.js b/app/pods/login/route.js
index c38d19e..a9687fd 100644
--- a/app/pods/login/route.js
+++ b/app/pods/login/route.js
@@ -1,4 +1,6 @@
 import Ember from 'ember';
 import UnauthenticatedRouteMixin from 'ember-simple-auth/mixins/unauthenticated-route-mixin';
 
-export default Ember.Route.extend(UnauthenticatedRouteMixin, {});
+const { Route } = Ember;
+
+export default Route.extend(UnauthenticatedRouteMixin, {});
diff --git a/app/pods/login/template.hbs b/app/pods/login/template.hbs
index 246c876..6a6fec6 100644
--- a/app/pods/login/template.hbs
+++ b/app/pods/login/template.hbs
@@ -1,12 +1,3 @@
 {{#x-application invalidateSession="invalidateSession"}}
-  <form {{action "authenticate" on="submit"}}>
-    <h2>Log In</h2>
-      {{input value=identification type="text" placeholder="Email"}}
-      {{input value=password type="password" placeholder="Password"}}
-      {{input class="button-gray" type="submit" value="Log In"}}
-  </form>
-  <br>
-  <div>
-    {{link-to 'Forget your password?' 'users.requestlockouthelp'}}
-  </div>
+  {{login/login-form}}
 {{/x-application}}

From 73461d40483f252125ca0e438698493ca6806fb3 Mon Sep 17 00:00:00 2001
From: Matthew Dillon <matthewrdillon@gmail.com>
Date: Wed, 11 Nov 2015 13:01:23 -0700
Subject: [PATCH 24/35] WIP

---
 app/pods/login/controller.js           |  9 +++++----
 app/pods/login/login-form/component.js | 13 +++++--------
 app/pods/login/login-form/template.hbs |  2 +-
 app/pods/login/template.hbs            |  5 ++++-
 4 files changed, 15 insertions(+), 14 deletions(-)

diff --git a/app/pods/login/controller.js b/app/pods/login/controller.js
index ec1077b..910a88a 100644
--- a/app/pods/login/controller.js
+++ b/app/pods/login/controller.js
@@ -1,14 +1,15 @@
 import Ember from 'ember';
 
-export default Ember.Controller.extend({
-  session: Ember.inject.service('session'),
+const { Controller, inject: { service } } = Ember;
+
+export default Controller.extend({
+  session: service(),
 
   actions: {
-    authenticate: function() {
+    authenticate: function(identification, password) {
       // Manually clean up because there might not be a transition
       this.get('flashMessages').clearMessages();
 
-      let { identification, password } = this.getProperties('identification', 'password');
       this.transitionToRoute('loading').then(() => {
         this.get('session').authenticate('authenticator:oauth2', identification, password).catch((error) => {
           this.transitionToRoute('login').then(() => {
diff --git a/app/pods/login/login-form/component.js b/app/pods/login/login-form/component.js
index 56a0f99..22e2cff 100644
--- a/app/pods/login/login-form/component.js
+++ b/app/pods/login/login-form/component.js
@@ -4,19 +4,16 @@ const { Component } = Ember;
 
 export default Component.extend({
   // Actions
-  "on-save": null,
-  "on-cancel": null,
-  "on-update": null,
-  "add-characteristic": null,
-  "save-measurement": null,
-  "delete-measurement": null,
+  "on-submit": null,
 
   // Property mapping
   propertiesList: ['identification', 'password'],
   identification: null,
   password: null,
 
-  updateField: function(property, value) {
-    this.set(property, value);
+  actions: {
+    submit: function() {
+      return this.attrs['on-submit'](this.get('identification'), this.get('password'));
+    },
   },
 });
diff --git a/app/pods/login/login-form/template.hbs b/app/pods/login/login-form/template.hbs
index 8492f1c..65f5adb 100644
--- a/app/pods/login/login-form/template.hbs
+++ b/app/pods/login/login-form/template.hbs
@@ -1,4 +1,4 @@
-<form {{action "authenticate" on="submit"}}>
+<form {{action "submit" on="submit"}}>
   <h2>Log In</h2>
     {{input value=identification type="text" placeholder="Email"}}
     {{input value=password type="password" placeholder="Password"}}
diff --git a/app/pods/login/template.hbs b/app/pods/login/template.hbs
index 6a6fec6..870b977 100644
--- a/app/pods/login/template.hbs
+++ b/app/pods/login/template.hbs
@@ -1,3 +1,6 @@
 {{#x-application invalidateSession="invalidateSession"}}
-  {{login/login-form}}
+  {{
+    login/login-form
+    on-submit=(action "authenticate")
+  }}
 {{/x-application}}

From 2beabf817c3b1a9a7a487e3cd551864579ae9ac3 Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Wed, 11 Nov 2015 16:53:46 -0700
Subject: [PATCH 25/35] DDAU (dropping two-way helpers)

---
 app/pods/login/login-form/component.js | 8 ++++++++
 app/pods/login/login-form/template.hbs | 8 +++++---
 2 files changed, 13 insertions(+), 3 deletions(-)

diff --git a/app/pods/login/login-form/component.js b/app/pods/login/login-form/component.js
index 22e2cff..638cba4 100644
--- a/app/pods/login/login-form/component.js
+++ b/app/pods/login/login-form/component.js
@@ -15,5 +15,13 @@ export default Component.extend({
     submit: function() {
       return this.attrs['on-submit'](this.get('identification'), this.get('password'));
     },
+
+    identificationDidChange: function(value) {
+      this.set('identification', value);
+    },
+
+    passwordDidChange: function(value) {
+      this.set('password', value);
+    },
   },
 });
diff --git a/app/pods/login/login-form/template.hbs b/app/pods/login/login-form/template.hbs
index 65f5adb..45afb9b 100644
--- a/app/pods/login/login-form/template.hbs
+++ b/app/pods/login/login-form/template.hbs
@@ -1,8 +1,10 @@
 <form {{action "submit" on="submit"}}>
   <h2>Log In</h2>
-    {{input value=identification type="text" placeholder="Email"}}
-    {{input value=password type="password" placeholder="Password"}}
-    {{input class="button-gray" type="submit" value="Log In"}}
+    {{one-way-input type="text" class="identification" value=identification update=(action "identificationDidChange") placeholder="Email"}}
+    {{one-way-input type="password" class="password" value=password update=(action "passwordDidChange") placeholder="Password"}}
+    <button type="submit" class="button-gray log-in">
+      Log In
+    </button>
 </form>
 <br>
 <div>

From 268ab18cf6e92618332ee4e380b6425523cc0ca6 Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Wed, 11 Nov 2015 17:07:45 -0700
Subject: [PATCH 26/35] Compare, begin components

---
 app/pods/protected/about/route.js             |  4 +-
 app/pods/protected/compare/controller.js      | 40 ++-----------
 app/pods/protected/compare/route.js           |  4 +-
 .../compare/select-form/component.js          | 49 +++++++++++++++
 .../compare/select-form/template.hbs          | 53 +++++++++++++++++
 app/pods/protected/compare/template.hbs       | 59 ++-----------------
 app/pods/protected/index/route.js             |  4 +-
 app/pods/protected/index/template.hbs         |  1 -
 8 files changed, 122 insertions(+), 92 deletions(-)
 create mode 100644 app/pods/protected/compare/select-form/component.js
 create mode 100644 app/pods/protected/compare/select-form/template.hbs
 delete mode 100644 app/pods/protected/index/template.hbs

diff --git a/app/pods/protected/about/route.js b/app/pods/protected/about/route.js
index 096e3c5..5f46de6 100644
--- a/app/pods/protected/about/route.js
+++ b/app/pods/protected/about/route.js
@@ -1,3 +1,5 @@
 import Ember from 'ember';
 
-export default Ember.Route.extend({});
+const { Route } = Ember;
+
+export default Route.extend({});
diff --git a/app/pods/protected/compare/controller.js b/app/pods/protected/compare/controller.js
index 00ef991..bc04ca2 100644
--- a/app/pods/protected/compare/controller.js
+++ b/app/pods/protected/compare/controller.js
@@ -1,41 +1,11 @@
 import Ember from 'ember';
 
-export default Ember.Controller.extend({
+const { Controller } = Ember;
+
+export default Controller.extend({
   actions: {
-    search: function() {
-      let query = {
-        strain_ids: this.get('selectedStrains'),
-        characteristic_ids: this.get('selectedCharacteristics'),
-      };
-
-      this.transitionToRoute('protected.compare.results', {queryParams: query});
+    search: function(query) {
+      this.transitionToRoute('protected.compare.results', { queryParams: query });
     },
-
-    selectAllStrains: function() {
-      let strains = this.get('strains');
-      let strain_ids = [];
-      strains.forEach((strain) => {
-        strain_ids.push(strain.id);
-      });
-      this.set('selectedStrains', strain_ids.join(","));
-    },
-
-    deselectAllStrains: function() {
-      this.set('selectedStrains', '');
-    },
-
-    selectAllCharacteristics: function() {
-      let chars = this.get('characteristics');
-      let char_ids = [];
-      chars.forEach((char) => {
-        char_ids.push(char.id);
-      });
-      this.set('selectedCharacteristics', char_ids.join(","));
-    },
-
-    deselectAllCharacteristics: function() {
-      this.set('selectedCharacteristics', '');
-    },
-
   }
 });
diff --git a/app/pods/protected/compare/route.js b/app/pods/protected/compare/route.js
index 19c8e94..199ade7 100644
--- a/app/pods/protected/compare/route.js
+++ b/app/pods/protected/compare/route.js
@@ -1,6 +1,8 @@
 import Ember from 'ember';
 
-export default Ember.Route.extend({
+const { Route } = Ember;
+
+export default Route.extend({
   model: function() {
     return this.store.findAll('characteristic');
   },
diff --git a/app/pods/protected/compare/select-form/component.js b/app/pods/protected/compare/select-form/component.js
new file mode 100644
index 0000000..34f5bff
--- /dev/null
+++ b/app/pods/protected/compare/select-form/component.js
@@ -0,0 +1,49 @@
+import Ember from 'ember';
+
+const { Component } = Ember;
+
+export default Component.extend({
+  characteristics: null,
+  strains: null,
+
+  "on-search": null,
+
+  selectedStrains: null,
+  selectedCharacteristics: null,
+
+  actions: {
+    search: function() {
+      const query = {
+        strain_ids: this.get('selectedStrains'),
+        characteristic_ids: this.get('selectedCharacteristics'),
+      };
+      this.attrs['on-search'](query);
+    },
+
+    selectAllStrains: function() {
+      const strains = this.get('strains');
+      const strain_ids = [];
+      strains.forEach((strain) => {
+        strain_ids.push(strain.get('id'));
+      });
+      this.set('selectedStrains', strain_ids.join(","));
+    },
+
+    deselectAllStrains: function() {
+      this.set('selectedStrains', '');
+    },
+
+    selectAllCharacteristics: function() {
+      const chars = this.get('characteristics');
+      const char_ids = [];
+      chars.forEach((char) => {
+        char_ids.push(char.get('id'));
+      });
+      this.set('selectedCharacteristics', char_ids.join(","));
+    },
+
+    deselectAllCharacteristics: function() {
+      this.set('selectedCharacteristics', '');
+    },
+  },
+});
diff --git a/app/pods/protected/compare/select-form/template.hbs b/app/pods/protected/compare/select-form/template.hbs
new file mode 100644
index 0000000..61a07a5
--- /dev/null
+++ b/app/pods/protected/compare/select-form/template.hbs
@@ -0,0 +1,53 @@
+<div class="span-1">
+  <fieldset>
+    <form {{action 'search' on='submit'}}>
+      <ul>
+        <li>
+          <label>Strains</label>
+          {{
+            select-2
+            multiple=true
+            content=strains
+            value=selectedStrains
+            optionValuePath="id"
+            optionLabelPath="fullNameMU"
+            placeholder="Select one or more strains"
+          }}
+        </li>
+        <li>
+          <button class="action button-green smaller right" {{action 'selectAllStrains'}}>
+            Select All
+          </button>
+          <button class="action button-red smaller right" {{action 'deselectAllStrains'}}>
+            Deselect All
+          </button>
+        </li>
+        <li>
+          <label>Characteristics</label>
+          {{
+            select-2
+            multiple=true
+            content=characteristics
+            value=selectedCharacteristics
+            optionValuePath="id"
+            optionLabelPath="characteristicName"
+            placeholder="Select one or more characteristics"
+          }}
+        </li>
+        <li>
+          <button class="action button-green smaller right" {{action 'selectAllCharacteristics'}}>
+            Select All
+          </button>
+          <button class="action button-red smaller right" {{action 'deselectAllCharacteristics'}}>
+            Deselect All
+          </button>
+        </li>
+        <li>
+          <button type="submit" class="action button-gray smaller right">
+            Search
+          </button>
+        </li>
+      </ul>
+    </form>
+  </fieldset>
+</div>
diff --git a/app/pods/protected/compare/template.hbs b/app/pods/protected/compare/template.hbs
index fefe9a6..6dc89e8 100644
--- a/app/pods/protected/compare/template.hbs
+++ b/app/pods/protected/compare/template.hbs
@@ -1,57 +1,10 @@
 <h2>{{genus-name}} - Compare Strains</h2>
 
-<div class="span-1">
-  <fieldset>
-    <form {{action 'search' on='submit'}}>
-      <ul>
-        <li>
-          <label>Strains</label>
-          {{
-            select-2
-            multiple=true
-            content=strains
-            value=selectedStrains
-            optionValuePath="id"
-            optionLabelPath="fullNameMU"
-            placeholder="Select one or more strains"
-          }}
-        </li>
-        <li>
-          <button class="action button-green smaller right" {{action 'selectAllStrains'}}>
-            Select All
-          </button>
-          <button class="action button-red smaller right" {{action 'deselectAllStrains'}}>
-            Deselect All
-          </button>
-        </li>
-        <li>
-          <label>Characteristics</label>
-          {{
-            select-2
-            multiple=true
-            content=characteristics
-            value=selectedCharacteristics
-            optionValuePath="id"
-            optionLabelPath="characteristicName"
-            placeholder="Select one or more characteristics"
-          }}
-        </li>
-        <li>
-          <button class="action button-green smaller right" {{action 'selectAllCharacteristics'}}>
-            Select All
-          </button>
-          <button class="action button-red smaller right" {{action 'deselectAllCharacteristics'}}>
-            Deselect All
-          </button>
-        </li>
-        <li>
-          <button type="submit" class="action button-gray smaller right">
-            Search
-          </button>
-        </li>
-      </ul>
-    </form>
-  </fieldset>
-</div>
+{{
+  protected/compare/select-form
+  characteristics=characteristics
+  strains=strains
+  on-search=(action "search")
+}}
 
 {{outlet}}
diff --git a/app/pods/protected/index/route.js b/app/pods/protected/index/route.js
index bfdf329..fdf337b 100644
--- a/app/pods/protected/index/route.js
+++ b/app/pods/protected/index/route.js
@@ -1,6 +1,8 @@
 import Ember from 'ember';
 
-export default Ember.Route.extend({
+const { Route } = Ember;
+
+export default Route.extend({
   beforeModel: function(transition) {
     this._super(transition);
     this.transitionTo('protected.compare');
diff --git a/app/pods/protected/index/template.hbs b/app/pods/protected/index/template.hbs
deleted file mode 100644
index ba4c514..0000000
--- a/app/pods/protected/index/template.hbs
+++ /dev/null
@@ -1 +0,0 @@
-Welcome

From aeb3206fd9a9ad821185c6c69e374c703c18f63a Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Wed, 11 Nov 2015 20:29:50 -0700
Subject: [PATCH 27/35] compare/results component

---
 app/pods/protected/compare/controller.js      | 11 ++++++
 .../protected/compare/results/controller.js   | 29 ++-------------
 .../results/results-table/component.js        | 19 ++++++++++
 .../results/results-table/template.hbs        | 25 +++++++++++++
 app/pods/protected/compare/results/route.js   | 36 ++++++++++++++-----
 .../protected/compare/results/template.hbs    | 32 ++++-------------
 .../compare/select-form/component.js          | 20 ++++++++---
 app/pods/protected/compare/template.hbs       |  4 +++
 8 files changed, 113 insertions(+), 63 deletions(-)
 create mode 100644 app/pods/protected/compare/results/results-table/component.js
 create mode 100644 app/pods/protected/compare/results/results-table/template.hbs

diff --git a/app/pods/protected/compare/controller.js b/app/pods/protected/compare/controller.js
index bc04ca2..0ba05a4 100644
--- a/app/pods/protected/compare/controller.js
+++ b/app/pods/protected/compare/controller.js
@@ -3,9 +3,20 @@ import Ember from 'ember';
 const { Controller } = Ember;
 
 export default Controller.extend({
+  selectedStrains: null,
+  selectedCharacteristics: null,
+
   actions: {
     search: function(query) {
       this.transitionToRoute('protected.compare.results', { queryParams: query });
     },
+
+    updateStrainSelection: function(selection) {
+      this.set('selectedStrains', selection);
+    },
+
+    updateCharacteristicSelection: function(selection) {
+      this.set('selectedCharacteristics', selection);
+    },
   }
 });
diff --git a/app/pods/protected/compare/results/controller.js b/app/pods/protected/compare/results/controller.js
index 391985a..c054c46 100644
--- a/app/pods/protected/compare/results/controller.js
+++ b/app/pods/protected/compare/results/controller.js
@@ -1,31 +1,8 @@
 import Ember from 'ember';
 
-export default Ember.Controller.extend({
+const { Controller } = Ember;
+
+export default Controller.extend({
   queryParams: ['strain_ids', 'characteristic_ids'],
 
-  csvLink: function() {
-    let token = encodeURIComponent(this.get('session.secure.token'));
-    return `${this.get('globals.apiURL')}/api/${this.get('globals.genus')}/` +
-      `compare?token=${token}&strain_ids=${this.get('strain_ids')}&` +
-      `characteristic_ids=${this.get('characteristic_ids')}&mimeType=csv`;
-  }.property('strain_ids', 'characteristic_ids').readOnly(),
-
-  strains: function() {
-    let strains = [];
-    let strain_ids = this.get('strain_ids').split(',');
-    strain_ids.forEach((id) => {
-      strains.push(this.store.peekRecord('strain', id));
-    });
-    return strains;
-  }.property('strain_ids'),
-
-  characteristics: function() {
-    let characteristics = [];
-    let characteristic_ids = this.get('characteristic_ids').split(',');
-    characteristic_ids.forEach((id) => {
-      characteristics.push(this.store.peekRecord('characteristic', id));
-    });
-    return characteristics;
-  }.property('characteristic_ids'),
-
 });
diff --git a/app/pods/protected/compare/results/results-table/component.js b/app/pods/protected/compare/results/results-table/component.js
new file mode 100644
index 0000000..87b2369
--- /dev/null
+++ b/app/pods/protected/compare/results/results-table/component.js
@@ -0,0 +1,19 @@
+import Ember from 'ember';
+
+const { Component, computed, inject: { service } } = Ember;
+
+export default Component.extend({
+  session: service(),
+
+  strains: null,
+  characteristics: null,
+  strain_ids: null,
+  characteristic_ids: null,
+
+  csvLink: computed('strain_ids', 'characteristic_ids', function() {
+    const token = encodeURIComponent(this.get('session.data.authenticated.access_token'));
+    return `${this.get('globals.apiURL')}/api/${this.get('globals.genus')}/` +
+      `compare?token=${token}&strain_ids=${this.get('strain_ids')}&` +
+      `characteristic_ids=${this.get('characteristic_ids')}&mimeType=csv`;
+  }),
+});
diff --git a/app/pods/protected/compare/results/results-table/template.hbs b/app/pods/protected/compare/results/results-table/template.hbs
new file mode 100644
index 0000000..c90d3bc
--- /dev/null
+++ b/app/pods/protected/compare/results/results-table/template.hbs
@@ -0,0 +1,25 @@
+<table class="flakes-table">
+  <thead>
+    <tr>
+      <th>Characteristic</th>
+      {{#each strains as |strain|}}
+        <th>
+          {{#link-to 'protected.strains.show' strain.id classBinding="data.typeStrain:type-strain"}}
+            {{strain.fullNameMU}}
+          {{/link-to}}
+        </th>
+      {{/each}}
+    </tr>
+  </thead>
+  <tbody>
+    {{#each characteristics as |row|}}
+      <tr>
+        {{#each row key="@index" as |col|}}
+          <td>{{col}}</td>
+        {{/each}}
+      </tr>
+    {{/each}}
+  </tbody>
+</table>
+<hr>
+<a href="{{csvLink}}" download>Download as CSV</a>
diff --git a/app/pods/protected/compare/results/route.js b/app/pods/protected/compare/results/route.js
index 981fa36..5a8ec60 100644
--- a/app/pods/protected/compare/results/route.js
+++ b/app/pods/protected/compare/results/route.js
@@ -1,8 +1,10 @@
 import Ember from 'ember';
 import ajaxRequest from '../../../../utils/ajax-request';
 
-export default Ember.Route.extend({
-  session: Ember.inject.service('session'),
+const { Route, $: { isEmptyObject }, inject: { service } } = Ember;
+
+export default Route.extend({
+  session: service(),
 
   queryParams: {
     strain_ids: {
@@ -15,8 +17,9 @@ export default Ember.Route.extend({
 
   beforeModel: function(transition) {
     this._super(transition);
-    if (Ember.$.isEmptyObject(transition.queryParams.strain_ids) ||
-        Ember.$.isEmptyObject(transition.queryParams.characteristic_ids)) {
+    const strain_ids = transition.queryParams.strain_ids;
+    const characteristic_ids = transition.queryParams.characteristic_ids;
+    if (isEmptyObject(strain_ids) || isEmptyObject(characteristic_ids)) {
       this.transitionTo('protected.compare');
     }
   },
@@ -26,12 +29,12 @@ export default Ember.Route.extend({
       this.transitionTo('protected.compare');
     }
 
-    let compare = this.controllerFor('protected.compare');
+    const compare = this.controllerFor('protected.compare');
     compare.set('selectedStrains', params.strain_ids);
     compare.set('selectedCharacteristics', params.characteristic_ids);
 
-    let url = `${this.get('globals.apiURL')}/api/${this.get('globals.genus')}/compare`;
-    let options = {
+    const url = `${this.get('globals.apiURL')}/api/${this.get('globals.genus')}/compare`;
+    const options = {
       method: 'GET',
       data: params,
     };
@@ -40,9 +43,26 @@ export default Ember.Route.extend({
 
   setupController: function(controller, model) {
     model.forEach((m, i) => {
-      let c = this.store.peekRecord('characteristic', m[0]);
+      const c = this.store.peekRecord('characteristic', m[0]);
       model[i][0] = c.get('characteristicName');
     });
+
+    const compare = this.controllerFor('protected.compare');
+
+    const strains = [];
+    const strain_ids = compare.get('selectedStrains').split(',');
+    strain_ids.forEach((id) => {
+      strains.push(this.store.peekRecord('strain', id));
+    });
+    controller.set('strains', strains);
+
+    const characteristics = [];
+    const characteristic_ids = compare.get('selectedCharacteristics').split(',');
+    characteristic_ids.forEach((id) => {
+      characteristics.push(this.store.peekRecord('characteristic', id));
+    });
+    controller.set('characteristics', characteristics);
+
     controller.set('model', model);
   },
 
diff --git a/app/pods/protected/compare/results/template.hbs b/app/pods/protected/compare/results/template.hbs
index f40869c..18542e7 100644
--- a/app/pods/protected/compare/results/template.hbs
+++ b/app/pods/protected/compare/results/template.hbs
@@ -1,25 +1,7 @@
-<table class="flakes-table">
-  <thead>
-    <tr>
-      <th>Characteristic</th>
-      {{#each strains as |strain|}}
-        <th>
-          {{#link-to 'protected.strains.show' strain.id classBinding="data.typeStrain:type-strain"}}
-            {{strain.fullNameMU}}
-          {{/link-to}}
-        </th>
-      {{/each}}
-    </tr>
-  </thead>
-  <tbody>
-    {{#each model as |row|}}
-      <tr>
-        {{#each row key="@index" as |col|}}
-          <td>{{col}}</td>
-        {{/each}}
-      </tr>
-    {{/each}}
-  </tbody>
-</table>
-<hr>
-<a href="{{csvLink}}" download>Download as CSV</a>
+{{
+  protected/compare/results/results-table
+  strains=strains
+  characteristics=model
+  strain_ids=strain_ids
+  characteristic_ids=characteristic_ids
+}}
diff --git a/app/pods/protected/compare/select-form/component.js b/app/pods/protected/compare/select-form/component.js
index 34f5bff..db58626 100644
--- a/app/pods/protected/compare/select-form/component.js
+++ b/app/pods/protected/compare/select-form/component.js
@@ -7,10 +7,22 @@ export default Component.extend({
   strains: null,
 
   "on-search": null,
+  "update-strains": null,
+  "update-characteristics": null,
 
   selectedStrains: null,
   selectedCharacteristics: null,
 
+  updateStrains: function(selection) {
+    this.set('selectedStrains', selection);
+    this.attrs['update-strains'](this.get('selectedStrains'));
+  },
+
+  updateCharacteristics: function(selection) {
+    this.set('selectedCharacteristics', selection);
+    this.attrs['update-characteristics'](this.get('selectedCharacteristics'));
+  },
+
   actions: {
     search: function() {
       const query = {
@@ -26,11 +38,11 @@ export default Component.extend({
       strains.forEach((strain) => {
         strain_ids.push(strain.get('id'));
       });
-      this.set('selectedStrains', strain_ids.join(","));
+      this.updateStrains(strain_ids.join(","));
     },
 
     deselectAllStrains: function() {
-      this.set('selectedStrains', '');
+      this.updateStrains("");
     },
 
     selectAllCharacteristics: function() {
@@ -39,11 +51,11 @@ export default Component.extend({
       chars.forEach((char) => {
         char_ids.push(char.get('id'));
       });
-      this.set('selectedCharacteristics', char_ids.join(","));
+      this.updateCharacteristics(char_ids.join(","));
     },
 
     deselectAllCharacteristics: function() {
-      this.set('selectedCharacteristics', '');
+      this.updateCharacteristics("");
     },
   },
 });
diff --git a/app/pods/protected/compare/template.hbs b/app/pods/protected/compare/template.hbs
index 6dc89e8..af8b15a 100644
--- a/app/pods/protected/compare/template.hbs
+++ b/app/pods/protected/compare/template.hbs
@@ -4,7 +4,11 @@
   protected/compare/select-form
   characteristics=characteristics
   strains=strains
+  selectedStrains=selectedStrains
+  selectedCharacteristics=selectedCharacteristics
   on-search=(action "search")
+  update-strains=(action "updateStrainSelection")
+  update-characteristics=(action "updateCharacteristicSelection")
 }}
 
 {{outlet}}

From 21e0e6c6248158bb3a8b0bd4b287c8d609422d7d Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Thu, 12 Nov 2015 05:54:18 -0700
Subject: [PATCH 28/35] Linting

---
 app/models/characteristic.js    | 24 ++++++++++---------
 app/models/measurement.js       | 26 +++++++++++----------
 app/models/species.js           | 29 ++++++++++++-----------
 app/models/strain.js            | 41 ++++++++++++++++++---------------
 app/models/user.js              | 41 ++++++++++++++++++---------------
 app/pods/application/adapter.js |  4 +++-
 app/pods/application/route.js   |  6 ++---
 app/pods/not-found/route.js     |  6 +++--
 app/pods/protected/route.js     |  7 ++++--
 app/serializers/application.js  | 15 +++++++-----
 10 files changed, 112 insertions(+), 87 deletions(-)

diff --git a/app/models/characteristic.js b/app/models/characteristic.js
index 766fdbf..ea6a6d5 100644
--- a/app/models/characteristic.js
+++ b/app/models/characteristic.js
@@ -1,14 +1,16 @@
 import DS from 'ember-data';
 
-export default DS.Model.extend({
-  characteristicName    : DS.attr('string'),
-  characteristicTypeName: DS.attr('string'),
-  strains               : DS.hasMany('strain', { async: false }),
-  measurements          : DS.hasMany('measurements', { async: false }),
-  createdAt             : DS.attr('date'),
-  updatedAt             : DS.attr('date'),
-  createdBy             : DS.attr('number'),
-  updatedBy             : DS.attr('number'),
-  sortOrder             : DS.attr('number'),
-  canEdit               : DS.attr('boolean'),
+const { Model, attr, hasMany } = DS;
+
+export default Model.extend({
+  characteristicName    : attr('string'),
+  characteristicTypeName: attr('string'),
+  strains               : hasMany('strain', { async: false }),
+  measurements          : hasMany('measurements', { async: false }),
+  createdAt             : attr('date'),
+  updatedAt             : attr('date'),
+  createdBy             : attr('number'),
+  updatedBy             : attr('number'),
+  sortOrder             : attr('number'),
+  canEdit               : attr('boolean'),
 });
diff --git a/app/models/measurement.js b/app/models/measurement.js
index 41a7644..009aa8b 100644
--- a/app/models/measurement.js
+++ b/app/models/measurement.js
@@ -1,15 +1,17 @@
 import DS from 'ember-data';
 
-export default DS.Model.extend({
-  strain             : DS.belongsTo('strain', { async: false }),
-  characteristic     : DS.belongsTo('characteristic', { async: false }),
-  value              : DS.attr('string'),
-  confidenceInterval : DS.attr('number'),
-  unitType           : DS.attr('string'),
-  notes              : DS.attr('string'),
-  testMethod         : DS.attr('string'),
-  createdAt          : DS.attr('date'),
-  updatedAt          : DS.attr('date'),
-  createdBy          : DS.attr('number'),
-  updatedBy          : DS.attr('number'),
+const { Model, belongsTo, attr } = DS;
+
+export default Model.extend({
+  strain             : belongsTo('strain', { async: false }),
+  characteristic     : belongsTo('characteristic', { async: false }),
+  value              : attr('string'),
+  confidenceInterval : attr('number'),
+  unitType           : attr('string'),
+  notes              : attr('string'),
+  testMethod         : attr('string'),
+  createdAt          : attr('date'),
+  updatedAt          : attr('date'),
+  createdBy          : attr('number'),
+  updatedBy          : attr('number'),
 });
diff --git a/app/models/species.js b/app/models/species.js
index bb27451..7581d10 100644
--- a/app/models/species.js
+++ b/app/models/species.js
@@ -2,20 +2,23 @@ import DS from 'ember-data';
 import config from '../config/environment';
 import Ember from 'ember';
 
-export default DS.Model.extend({
-  speciesName : DS.attr('string'),
-  typeSpecies : DS.attr('boolean'),
-  etymology   : DS.attr('string'),
-  genusName   : DS.attr('string', { defaultValue: config.APP.genus }),
-  strains     : DS.hasMany('strain', { async: false }),
-  totalStrains: DS.attr('number'),
-  createdAt   : DS.attr('date'),
-  updatedAt   : DS.attr('date'),
-  createdBy   : DS.attr('number'),
-  updatedBy   : DS.attr('number'),
-  sortOrder   : DS.attr('number'),
-  canEdit     : DS.attr('boolean'),
+const { Model, attr, hasMany } = DS;
 
+export default Model.extend({
+  speciesName : attr('string'),
+  typeSpecies : attr('boolean'),
+  etymology   : attr('string'),
+  genusName   : attr('string', { defaultValue: config.APP.genus }),
+  strains     : hasMany('strain', { async: false }),
+  totalStrains: attr('number'),
+  createdAt   : attr('date'),
+  updatedAt   : attr('date'),
+  createdBy   : attr('number'),
+  updatedBy   : attr('number'),
+  sortOrder   : attr('number'),
+  canEdit     : attr('boolean'),
+
+  // TODO: move this to component/helper
   speciesNameMU: function() {
     return Ember.String.htmlSafe(`<em>${this.get('speciesName')}</em>`);
   }.property('speciesName').readOnly(),
diff --git a/app/models/strain.js b/app/models/strain.js
index 0c59f2f..b202a78 100644
--- a/app/models/strain.js
+++ b/app/models/strain.js
@@ -1,34 +1,39 @@
 import DS from 'ember-data';
 import Ember from 'ember';
 
-export default DS.Model.extend({
-  measurements       : DS.hasMany('measurements', { async: false }),
-  characteristics    : DS.hasMany('characteristics', { async: false }),
-  species            : DS.belongsTo('species', { async: false }),
-  strainName         : DS.attr('string'),
-  typeStrain         : DS.attr('boolean'),
-  accessionNumbers   : DS.attr('string'),
-  genbank            : DS.attr('string'),
-  wholeGenomeSequence: DS.attr('string'),
-  isolatedFrom       : DS.attr('string'),
-  notes              : DS.attr('string'),
-  createdAt          : DS.attr('date'),
-  updatedAt          : DS.attr('date'),
-  createdBy          : DS.attr('number'),
-  updatedBy          : DS.attr('number'),
-  totalMeasurements  : DS.attr('number'),
-  sortOrder          : DS.attr('number'),
-  canEdit            : DS.attr('boolean'),
+const { Model, hasMany, belongsTo, attr } = DS;
 
+export default Model.extend({
+  measurements       : hasMany('measurements', { async: false }),
+  characteristics    : hasMany('characteristics', { async: false }),
+  species            : belongsTo('species', { async: false }),
+  strainName         : attr('string'),
+  typeStrain         : attr('boolean'),
+  accessionNumbers   : attr('string'),
+  genbank            : attr('string'),
+  wholeGenomeSequence: attr('string'),
+  isolatedFrom       : attr('string'),
+  notes              : attr('string'),
+  createdAt          : attr('date'),
+  updatedAt          : attr('date'),
+  createdBy          : attr('number'),
+  updatedBy          : attr('number'),
+  totalMeasurements  : attr('number'),
+  sortOrder          : attr('number'),
+  canEdit            : attr('boolean'),
+
+  // TODO: move this to component/helper
   strainNameMU: function() {
     let type = this.get('typeStrain') ? '<sup>T</sup>' : '';
     return Ember.String.htmlSafe(`${this.get('strainName')}${type}`);
   }.property('strainName', 'typeStrain').readOnly(),
 
+  // TODO: move this to component/helper
   fullName: Ember.computed('species', 'strainName', function() {
     return `${this.get('species.speciesName')} ${this.get('strainNameMU')}`;
   }),
 
+  // TODO: move this to component/helper
   fullNameMU: function() {
     return Ember.String.htmlSafe(`<em>${this.get('species.speciesName')}</em> ${this.get('strainNameMU')}`);
   }.property('species', 'strainNameMU').readOnly(),
diff --git a/app/models/user.js b/app/models/user.js
index 198f451..89b7c0f 100644
--- a/app/models/user.js
+++ b/app/models/user.js
@@ -1,29 +1,32 @@
 import Ember from 'ember';
 import DS from 'ember-data';
 
-export default DS.Model.extend({
-  email    : DS.attr('string'),
-  password : DS.attr('string'),
-  name     : DS.attr('string'),
-  role     : DS.attr('string'),
-  canEdit  : DS.attr('boolean'),
-  createdAt: DS.attr('date'),
-  updatedAt: DS.attr('date'),
+const { Model, attr } = DS;
+const { computed } = Ember;
 
-  isAdmin: function() {
+export default Model.extend({
+  email    : attr('string'),
+  password : attr('string'),
+  name     : attr('string'),
+  role     : attr('string'),
+  canEdit  : attr('boolean'),
+  createdAt: attr('date'),
+  updatedAt: attr('date'),
+
+  isAdmin: computed('role', function() {
     return this.get('role') === 'A';
-  }.property('role'),
+  }),
 
-  isWriter: function() {
+  isWriter: computed('role', function() {
     return this.get('role') === 'W';
-  }.property('role'),
+  }),
 
-  isReader: function() {
+  isReader: computed('role', function() {
     return this.get('role') === 'R';
-  }.property('role'),
+  }),
 
-  fullRole: function() {
-    let role = this.get('role');
+  fullRole: computed('role', function() {
+    const role = this.get('role');
     if (role === 'R') {
       return 'Read-Only';
     } else if (role === 'W') {
@@ -33,13 +36,13 @@ export default DS.Model.extend({
     } else {
       return 'Error';
     }
-  }.property('role'),
+  }),
 
-  canWrite: Ember.computed('role', function() {
+  canWrite: computed('role', function() {
     return this.get('role') !== 'R';
   }),
 
-  metaData: Ember.computed('canWrite', function() {
+  metaData: computed('canWrite', function() {
     return { 'canAdd': this.get('canWrite') };
   }),
 
diff --git a/app/pods/application/adapter.js b/app/pods/application/adapter.js
index 96daa1a..0d7ed91 100644
--- a/app/pods/application/adapter.js
+++ b/app/pods/application/adapter.js
@@ -1,7 +1,9 @@
 import DS from 'ember-data';
 import DataAdapterMixin from 'ember-simple-auth/mixins/data-adapter-mixin';
 
-export default DS.RESTAdapter.extend(DataAdapterMixin, {
+const { RESTAdapter } = DS;
+
+export default RESTAdapter.extend(DataAdapterMixin, {
   authorizer: 'authorizer:application',
 
   namespace: function() {
diff --git a/app/pods/application/route.js b/app/pods/application/route.js
index ca03050..e677da9 100644
--- a/app/pods/application/route.js
+++ b/app/pods/application/route.js
@@ -1,7 +1,9 @@
 import Ember from 'ember';
 import ApplicationRouteMixin from 'ember-simple-auth/mixins/application-route-mixin';
 
-export default Ember.Route.extend(ApplicationRouteMixin, {
+const { Route } = Ember;
+
+export default Route.extend(ApplicationRouteMixin, {
   actions: {
     invalidateSession: function() {
       this.get('session').invalidate().then(() => {
@@ -9,7 +11,5 @@ export default Ember.Route.extend(ApplicationRouteMixin, {
         return true;
       });
     },
-
   },
-
 });
diff --git a/app/pods/not-found/route.js b/app/pods/not-found/route.js
index 5be93bf..c14f72d 100644
--- a/app/pods/not-found/route.js
+++ b/app/pods/not-found/route.js
@@ -1,8 +1,10 @@
 import Ember from 'ember';
 
-export default Ember.Route.extend({
+const { Route } = Ember;
+
+export default Route.extend({
   redirect: function() {
-    let url = this.router.location.formatURL('/not-found');
+    const url = this.router.location.formatURL('/not-found');
 
     if (window.location.pathname !== url) {
       this.transitionTo('/not-found');
diff --git a/app/pods/protected/route.js b/app/pods/protected/route.js
index b1cdc95..64bc25d 100644
--- a/app/pods/protected/route.js
+++ b/app/pods/protected/route.js
@@ -1,9 +1,12 @@
 import Ember from 'ember';
 import AuthenticatedRouteMixin from 'ember-simple-auth/mixins/authenticated-route-mixin';
 
-export default Ember.Route.extend(AuthenticatedRouteMixin, {
+const { Route } = Ember;
+
+export default Route.extend(AuthenticatedRouteMixin, {
   actions: {
-    error: function() {
+    error: function(err) {
+      console.log(err);
       this.transitionTo('/not-found');
     },
 
diff --git a/app/serializers/application.js b/app/serializers/application.js
index 52f555e..d40a2bf 100644
--- a/app/serializers/application.js
+++ b/app/serializers/application.js
@@ -1,19 +1,22 @@
 import DS from 'ember-data';
 import Ember from 'ember';
 
-export default DS.RESTSerializer.extend({
+const { RESTSerializer } = DS;
+const { isNone } = Ember;
+
+export default RESTSerializer.extend({
   isNewSerializerAPI: true,
 
   serializeBelongsTo: function(snapshot, json, relationship) {
-    var key = relationship.key;
-    var belongsTo = snapshot.belongsTo(key);
+    let key = relationship.key;
+    const belongsTo = snapshot.belongsTo(key);
     key = this.keyForRelationship ? this.keyForRelationship(key, "belongsTo", "serialize") : key;
-    json[key] = Ember.isNone(belongsTo) ? belongsTo : +belongsTo.record.id;
+    json[key] = isNone(belongsTo) ? belongsTo : +belongsTo.record.id;
   },
 
   serializeHasMany: function(snapshot, json, relationship) {
-    var key = relationship.key;
-    var hasMany = snapshot.hasMany(key);
+    let key = relationship.key;
+    const hasMany = snapshot.hasMany(key);
     key = this.keyForRelationship ? this.keyForRelationship(key, "hasMany", "serialize") : key;
 
     json[key] = [];

From 1fe7702430548c00a3096f639e32eb3acb0bba7c Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Thu, 12 Nov 2015 06:36:22 -0700
Subject: [PATCH 29/35] ember-cli 1.13.11

---
 .watchmanconfig                        |  2 +-
 app/app.js                             |  4 ++--
 app/router.js                          |  2 +-
 bower.json                             | 16 +++++++--------
 config/environment.js                  |  2 ++
 ember-cli-build.js                     |  1 +
 package.json                           | 28 +++++++++++++-------------
 tests/helpers/destroy-app.js           |  5 +++++
 tests/helpers/module-for-acceptance.js | 23 +++++++++++++++++++++
 tests/helpers/resolver.js              |  2 +-
 tests/helpers/start-app.js             |  6 +++---
 tests/index.html                       |  5 +++--
 12 files changed, 64 insertions(+), 32 deletions(-)
 create mode 100644 tests/helpers/destroy-app.js
 create mode 100644 tests/helpers/module-for-acceptance.js

diff --git a/.watchmanconfig b/.watchmanconfig
index 5e9462c..e7834e3 100644
--- a/.watchmanconfig
+++ b/.watchmanconfig
@@ -1,3 +1,3 @@
 {
-  "ignore_dirs": ["tmp"]
+  "ignore_dirs": ["tmp", "dist"]
 }
diff --git a/app/app.js b/app/app.js
index 8d66b95..8b234d6 100644
--- a/app/app.js
+++ b/app/app.js
@@ -3,14 +3,14 @@ import Resolver from 'ember/resolver';
 import loadInitializers from 'ember/load-initializers';
 import config from './config/environment';
 
-var App;
+let App;
 
 Ember.MODEL_FACTORY_INJECTIONS = true;
 
 App = Ember.Application.extend({
   modulePrefix: config.modulePrefix,
   podModulePrefix: config.podModulePrefix,
-  Resolver: Resolver
+  Resolver
 });
 
 loadInitializers(App, config.modulePrefix);
diff --git a/app/router.js b/app/router.js
index f5cc4af..13b6c88 100644
--- a/app/router.js
+++ b/app/router.js
@@ -1,7 +1,7 @@
 import Ember from 'ember';
 import config from './config/environment';
 
-var Router = Ember.Router.extend({
+const Router = Ember.Router.extend({
   location: config.locationType
 });
 
diff --git a/bower.json b/bower.json
index 0c58bd9..db98942 100644
--- a/bower.json
+++ b/bower.json
@@ -3,15 +3,15 @@
   "dependencies": {
     "jquery": "~2.1.1",
     "ember": "1.13.10",
-    "ember-cli-shims": "ember-cli/ember-cli-shims#0.0.3",
-    "ember-cli-test-loader": "ember-cli-test-loader#0.1.3",
-    "ember-data": "1.13.13",
-    "ember-load-initializers": "ember-cli/ember-load-initializers#0.1.5",
-    "ember-qunit": "0.4.9",
-    "ember-qunit-notifications": "0.0.7",
-    "ember-resolver": "~0.1.18",
+    "ember-cli-shims": "0.0.6",
+    "ember-cli-test-loader": "0.2.1",
+    "ember-data": "1.13.15",
+    "ember-load-initializers": "0.1.7",
+    "ember-qunit": "0.4.16",
+    "ember-qunit-notifications": "0.1.0",
+    "ember-resolver": "~0.1.20",
     "loader.js": "ember-cli/loader.js#3.2.1",
-    "qunit": "~1.18.0",
+    "qunit": "~1.20.0",
     "flakes": "~1.0.0",
     "moment": "~2.10.6",
     "select2": "3.5.2",
diff --git a/config/environment.js b/config/environment.js
index 6351cb1..752e4c9 100644
--- a/config/environment.js
+++ b/config/environment.js
@@ -37,6 +37,8 @@ module.exports = function(environment) {
     ENV.APP.LOG_ACTIVE_GENERATION = false;
     ENV.APP.LOG_VIEW_LOOKUPS = false;
     ENV.locationType = 'none';
+    ENV.baseURL = '/';
+    ENV.APP.rootElement = '#ember-testing';
   }
 
   if (environment === 'staging') {
diff --git a/ember-cli-build.js b/ember-cli-build.js
index 995161c..3ae8de0 100644
--- a/ember-cli-build.js
+++ b/ember-cli-build.js
@@ -1,3 +1,4 @@
+/*jshint node:true*/
 /* global require, module */
 var EmberApp = require('ember-cli/lib/broccoli/ember-app');
 
diff --git a/package.json b/package.json
index 9b901cb..6925274 100644
--- a/package.json
+++ b/package.json
@@ -19,25 +19,25 @@
   "author": "",
   "license": "MIT",
   "devDependencies": {
-    "broccoli-asset-rev": "^2.1.2",
-    "ember-cli": "1.13.8",
-    "ember-cli-app-version": "0.5.0",
-    "ember-cli-babel": "^5.1.3",
-    "ember-cli-dependency-checker": "^1.0.1",
-    "ember-cli-divshot": "^0.1.7",
+    "broccoli-asset-rev": "^2.2.0",
+    "ember-cli": "1.13.11",
+    "ember-cli-app-version": "^1.0.0",
+    "ember-cli-babel": "^5.1.5",
+    "ember-cli-content-security-policy": "0.4.0",
+    "ember-cli-dependency-checker": "^1.1.0",
     "ember-cli-flash": "1.3.6",
-    "ember-cli-htmlbars": "0.7.9",
-    "ember-cli-htmlbars-inline-precompile": "^0.2.0",
+    "ember-cli-htmlbars": "^1.0.1",
+    "ember-cli-htmlbars-inline-precompile": "^0.3.1",
     "ember-cli-ic-ajax": "0.2.1",
     "ember-cli-inject-live-reload": "^1.3.1",
     "ember-cli-mirage": "0.1.11",
-    "ember-cli-qunit": "^1.0.0",
-    "ember-cli-release": "0.2.3",
-    "ember-cli-sri": "^1.0.3",
+    "ember-cli-qunit": "^1.0.4",
+    "ember-cli-release": "0.2.8",
+    "ember-cli-sri": "^1.1.0",
     "ember-cli-uglify": "^1.2.0",
-    "ember-data": "1.13.13",
-    "ember-disable-proxy-controllers": "^1.0.0",
-    "ember-export-application-global": "^1.0.3",
+    "ember-data": "1.13.15",
+    "ember-disable-proxy-controllers": "^1.0.1",
+    "ember-export-application-global": "^1.0.4",
     "ember-one-way-input": "0.1.3",
     "ember-select-2": "1.3.0",
     "ember-simple-auth": "1.0.0"
diff --git a/tests/helpers/destroy-app.js b/tests/helpers/destroy-app.js
new file mode 100644
index 0000000..c3d4d1a
--- /dev/null
+++ b/tests/helpers/destroy-app.js
@@ -0,0 +1,5 @@
+import Ember from 'ember';
+
+export default function destroyApp(application) {
+  Ember.run(application, 'destroy');
+}
diff --git a/tests/helpers/module-for-acceptance.js b/tests/helpers/module-for-acceptance.js
new file mode 100644
index 0000000..ed23003
--- /dev/null
+++ b/tests/helpers/module-for-acceptance.js
@@ -0,0 +1,23 @@
+import { module } from 'qunit';
+import startApp from '../helpers/start-app';
+import destroyApp from '../helpers/destroy-app';
+
+export default function(name, options = {}) {
+  module(name, {
+    beforeEach() {
+      this.application = startApp();
+
+      if (options.beforeEach) {
+        options.beforeEach.apply(this, arguments);
+      }
+    },
+
+    afterEach() {
+      destroyApp(this.application);
+
+      if (options.afterEach) {
+        options.afterEach.apply(this, arguments);
+      }
+    }
+  });
+}
diff --git a/tests/helpers/resolver.js b/tests/helpers/resolver.js
index 28f4ece..ebfb4e4 100644
--- a/tests/helpers/resolver.js
+++ b/tests/helpers/resolver.js
@@ -1,7 +1,7 @@
 import Resolver from 'ember/resolver';
 import config from '../../config/environment';
 
-var resolver = Resolver.create();
+const resolver = Resolver.create();
 
 resolver.namespace = {
   modulePrefix: config.modulePrefix,
diff --git a/tests/helpers/start-app.js b/tests/helpers/start-app.js
index 0f7aab1..e098f1d 100644
--- a/tests/helpers/start-app.js
+++ b/tests/helpers/start-app.js
@@ -3,12 +3,12 @@ import Application from '../../app';
 import config from '../../config/environment';
 
 export default function startApp(attrs) {
-  var application;
+  let application;
 
-  var attributes = Ember.merge({}, config.APP);
+  let attributes = Ember.merge({}, config.APP);
   attributes = Ember.merge(attributes, attrs); // use defaults, but you can override;
 
-  Ember.run(function() {
+  Ember.run(() => {
     application = Application.create(attributes);
     application.setupForTesting();
     application.injectTestHelpers();
diff --git a/tests/index.html b/tests/index.html
index 648f884..de2e29b 100644
--- a/tests/index.html
+++ b/tests/index.html
@@ -18,13 +18,14 @@
     {{content-for 'test-head-footer'}}
   </head>
   <body>
-
     {{content-for 'body'}}
     {{content-for 'test-body'}}
+
     <script src="assets/vendor.js"></script>
     <script src="assets/test-support.js"></script>
     <script src="assets/hymenobacterdotinfo.js"></script>
-    <script src="testem.js"></script>
+    <script src="testem.js" integrity=""></script>
+    <script src="assets/tests.js"></script>
     <script src="assets/test-loader.js"></script>
 
     {{content-for 'body-footer'}}

From e54c6fcb2f959d68a22710d4da56a4dffc695c50 Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Thu, 12 Nov 2015 07:08:14 -0700
Subject: [PATCH 30/35] Bring back CSP

---
 config/environment.js | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/config/environment.js b/config/environment.js
index 752e4c9..f163a3d 100644
--- a/config/environment.js
+++ b/config/environment.js
@@ -24,6 +24,15 @@ module.exports = function(environment) {
       routeAfterAuthentication: 'protected.compare',
       routeIfAlreadyAuthenticated: 'protected.compare',
     },
+    contentSecurityPolicy: {
+      'default-src': "'none'",
+      'script-src': "'self'",
+      'font-src': "'self'",
+      'connect-src': "'self'",
+      'img-src': "'self'",
+      'style-src': "'self' 'unsafe-inline'",
+      'media-src': "'self'"
+    }
   };
 
   var apiURL;
@@ -52,6 +61,7 @@ module.exports = function(environment) {
   }
 
   ENV.apiURL = apiURL;
+  ENV.contentSecurityPolicy['connect-src'] = `'self' ${apiURL}`;
 
   return ENV;
 };

From b12ff0eb2058fe7c33471087aa9e1f2aa873f9ee Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Thu, 12 Nov 2015 13:32:41 -0700
Subject: [PATCH 31/35] ember-ajax

Fixes #37
Fixes #38
---
 app/pods/protected/compare/results/route.js   |  9 ++------
 .../users/changepassword/controller.js        | 13 +++--------
 app/pods/users/new/verify/route.js            |  9 +++-----
 .../users/requestlockouthelp/controller.js    |  9 ++------
 app/services/ajax.js                          | 23 +++++++++++++++++++
 app/utils/ajax-request.js                     | 20 ----------------
 package.json                                  |  2 +-
 7 files changed, 34 insertions(+), 51 deletions(-)
 create mode 100644 app/services/ajax.js
 delete mode 100644 app/utils/ajax-request.js

diff --git a/app/pods/protected/compare/results/route.js b/app/pods/protected/compare/results/route.js
index 5a8ec60..f71de62 100644
--- a/app/pods/protected/compare/results/route.js
+++ b/app/pods/protected/compare/results/route.js
@@ -1,10 +1,10 @@
 import Ember from 'ember';
-import ajaxRequest from '../../../../utils/ajax-request';
 
 const { Route, $: { isEmptyObject }, inject: { service } } = Ember;
 
 export default Route.extend({
   session: service(),
+  ajax: service(),
 
   queryParams: {
     strain_ids: {
@@ -33,12 +33,7 @@ export default Route.extend({
     compare.set('selectedStrains', params.strain_ids);
     compare.set('selectedCharacteristics', params.characteristic_ids);
 
-    const url = `${this.get('globals.apiURL')}/api/${this.get('globals.genus')}/compare`;
-    const options = {
-      method: 'GET',
-      data: params,
-    };
-    return ajaxRequest(url, options, this.get('session'));
+    return this.get('ajax').request('/compare', { data: params })
   },
 
   setupController: function(controller, model) {
diff --git a/app/pods/protected/users/changepassword/controller.js b/app/pods/protected/users/changepassword/controller.js
index 3aed813..9119f1a 100644
--- a/app/pods/protected/users/changepassword/controller.js
+++ b/app/pods/protected/users/changepassword/controller.js
@@ -1,24 +1,17 @@
 import Ember from 'ember';
-import ajaxRequest from '../../../../utils/ajax-request';
 
 const { Controller, inject: { service } } = Ember;
 
 export default Controller.extend({
   session: service(),
+  ajax: service(),
   currentUser: service('session-account'),
 
   actions: {
     save: function(password) {
-      const url = `${this.get('globals.apiURL')}/api/${this.get('globals.genus')}/users/password`;
       const id = this.get('currentUser.account.id');
-      const options = {
-        method: 'POST',
-        data: {
-          id: id,
-          password: password,
-        },
-      };
-      ajaxRequest(url, options, this.get('session'));
+      const data = { id: id, password: password };
+      this.get('ajax').post('/users/password', { data: data });
       this.transitionToRoute('protected.users.show', id);
       this.get('flashMessages').information('Your password has been changed.');
     },
diff --git a/app/pods/users/new/verify/route.js b/app/pods/users/new/verify/route.js
index 06755b3..a2d26c2 100644
--- a/app/pods/users/new/verify/route.js
+++ b/app/pods/users/new/verify/route.js
@@ -1,20 +1,17 @@
 import Ember from 'ember';
-import ajaxRequest from '../../../../utils/ajax-request';
 
 const { Route, inject: { service } } = Ember;
 
 export default Route.extend({
   session: service(),
-  globals: service(),
+  ajax: service(),
 
   model: function(params) {
-    const url = `${this.get('globals.apiURL')}/api/${this.get('globals.genus')}/users/verify/${params.nonce}`;
-    return ajaxRequest(url, {}, this.get('session'));
+    return this.get('ajax').request(`/users/verify/${params.nonce}`);
   },
 
-
   afterModel: function(model/*, transition*/) {
-    this.get('flashMessages').success(model.get('msg'));
+    this.get('flashMessages').success(model.msg);
     this.transitionTo('login');
   },
 
diff --git a/app/pods/users/requestlockouthelp/controller.js b/app/pods/users/requestlockouthelp/controller.js
index 2930b31..321045d 100644
--- a/app/pods/users/requestlockouthelp/controller.js
+++ b/app/pods/users/requestlockouthelp/controller.js
@@ -1,20 +1,15 @@
 import Ember from 'ember';
-import ajaxRequest from '../../../utils/ajax-request';
 
 const { Controller, inject: { service } } = Ember;
 
 export default Controller.extend({
   session: service(),
   globals: service(),
+  ajax: service(),
 
   actions: {
     submit: function(email) {
-      const url = `${this.get('globals.apiURL')}/api/${this.get('globals.genus')}/users/lockout`;
-      const options = {
-        method: 'POST',
-        data: { email: email },
-      };
-      ajaxRequest(url, options, this.get('session'));
+      this.get('ajax').post('/users/lockout', { data: { email: email } } );
       this.transitionToRoute('login');
       this.get('flashMessages').information('Please check your email');
     },
diff --git a/app/services/ajax.js b/app/services/ajax.js
new file mode 100644
index 0000000..a0a8cef
--- /dev/null
+++ b/app/services/ajax.js
@@ -0,0 +1,23 @@
+import Ember from 'ember';
+import AjaxService from 'ember-ajax/services/ajax';
+
+const { computed, inject: { service } } = Ember;
+
+export default AjaxService.extend({
+  session: service(),
+  globals: service(),
+
+  host: computed('globals.apiURL', function() {
+    return `${this.get('globals.apiURL')}/api/${this.get('globals.genus')}`;
+  }),
+
+  headers: computed('session.authToken', {
+    get: function() {
+      let headers = {};
+      this.get('session').authorize('authorizer:application', (headerName, headerValue) => {
+        headers[headerName] = headerValue;
+      });
+      return headers;
+    }
+  })
+});
diff --git a/app/utils/ajax-request.js b/app/utils/ajax-request.js
deleted file mode 100644
index a4c8de9..0000000
--- a/app/utils/ajax-request.js
+++ /dev/null
@@ -1,20 +0,0 @@
-import Ember from 'ember';
-
-export default function ajaxRequest(url, options, session) {
-  return new Ember.RSVP.Promise(function(resolve, reject) {
-    options = options || {};
-    options.url = url;
-    session.authorize('authorizer:application', (headerName, headerValue) => {
-      let authHeader = {};
-      authHeader[headerName] = headerValue;
-      options.headers = authHeader;
-    });
-    options.success = function(data) {
-      resolve(data);
-    };
-    options.error = function(jqXHR, status, error) {
-      reject(jqXHR, status, error);
-    };
-    Ember.$.ajax(options);
-  });
-}
diff --git a/package.json b/package.json
index 6925274..45d3b65 100644
--- a/package.json
+++ b/package.json
@@ -28,7 +28,7 @@
     "ember-cli-flash": "1.3.6",
     "ember-cli-htmlbars": "^1.0.1",
     "ember-cli-htmlbars-inline-precompile": "^0.3.1",
-    "ember-cli-ic-ajax": "0.2.1",
+    "ember-ajax": "0.7.0",
     "ember-cli-inject-live-reload": "^1.3.1",
     "ember-cli-mirage": "0.1.11",
     "ember-cli-qunit": "^1.0.4",

From 2d2950ed3347e038999a99374aeb0104bce20952 Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Thu, 12 Nov 2015 13:49:32 -0700
Subject: [PATCH 32/35] Linting

---
 app/pods/protected/compare/results/route.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/pods/protected/compare/results/route.js b/app/pods/protected/compare/results/route.js
index f71de62..4ae7fd0 100644
--- a/app/pods/protected/compare/results/route.js
+++ b/app/pods/protected/compare/results/route.js
@@ -33,7 +33,7 @@ export default Route.extend({
     compare.set('selectedStrains', params.strain_ids);
     compare.set('selectedCharacteristics', params.characteristic_ids);
 
-    return this.get('ajax').request('/compare', { data: params })
+    return this.get('ajax').request('/compare', { data: params });
   },
 
   setupController: function(controller, model) {

From e5dc115607203ffe123a56d8efe26c3b21ea3479 Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Thu, 12 Nov 2015 13:49:55 -0700
Subject: [PATCH 33/35] Drop store initializer

Fixes #39
---
 app/initializers/component-store.js           |  8 -------
 .../unit/initializers/component-store-test.js | 23 -------------------
 2 files changed, 31 deletions(-)
 delete mode 100644 app/initializers/component-store.js
 delete mode 100644 tests/unit/initializers/component-store-test.js

diff --git a/app/initializers/component-store.js b/app/initializers/component-store.js
deleted file mode 100644
index 54a3064..0000000
--- a/app/initializers/component-store.js
+++ /dev/null
@@ -1,8 +0,0 @@
-export function initialize(container, application) {
-  application.inject('component', 'store', 'service:store');
-}
-
-export default {
-  name: 'component-store',
-  initialize: initialize
-};
diff --git a/tests/unit/initializers/component-store-test.js b/tests/unit/initializers/component-store-test.js
deleted file mode 100644
index b0429e5..0000000
--- a/tests/unit/initializers/component-store-test.js
+++ /dev/null
@@ -1,23 +0,0 @@
-import Ember from 'ember';
-import { initialize } from '../../../initializers/component-store';
-import { module, test } from 'qunit';
-
-var container, application;
-
-module('Unit | Initializer | component store', {
-  beforeEach: function() {
-    Ember.run(function() {
-      application = Ember.Application.create();
-      container = application.__container__;
-      application.deferReadiness();
-    });
-  }
-});
-
-// Replace this with your real tests.
-test('it works', function(assert) {
-  initialize(container, application);
-
-  // you would normally confirm the results of the initializer here
-  assert.ok(true);
-});

From 305c108825b2a70347b1b8f75cbf94adcd711a58 Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Thu, 12 Nov 2015 13:53:54 -0700
Subject: [PATCH 34/35] Clean up README

Fixes #36
---
 README.md | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/README.md b/README.md
index 16e11f0..aa5a107 100644
--- a/README.md
+++ b/README.md
@@ -1,9 +1,6 @@
 # hymenobacterdotinfo
 
-Detailed information to come --- for now see the ember-cli boilerplate below.
-
-This README outlines the details of collaborating on this Ember application.
-A short introduction of this app could easily go here.
+This ember application is an interface for the [bactdb](https://github.com/thermokarst/bactdb).
 
 ## Prerequisites
 
@@ -17,13 +14,14 @@ You will need the following things properly installed on your computer.
 
 ## Installation
 
-* `git clone <repository-url>` this repository
+* `git clone https://github.com/thermokarst/hymenobacterdotinfo` this repository
 * change into the new directory
 * `npm install`
 * `bower install`
 
 ## Running / Development
 
+* Launch `bactdb`
 * `ember server`
 * Visit your app at [http://localhost:4200](http://localhost:4200).
 
@@ -43,7 +41,8 @@ Make use of the many generators for code, try `ember help generate` for more det
 
 ### Deploying
 
-Specify what it takes to deploy your app.
+* `ember build -e staging`
+* `firebase deploy`
 
 ## Further Reading / Useful Links
 

From bb05e114d746289167f2ec6c289fb7b2ee4d85e4 Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Fri, 13 Nov 2015 08:29:48 -0700
Subject: [PATCH 35/35] Clean up Quill Component

Fixes #54
---
 app/pods/components/text-editor/component.js | 20 ++++++++++++++++----
 1 file changed, 16 insertions(+), 4 deletions(-)

diff --git a/app/pods/components/text-editor/component.js b/app/pods/components/text-editor/component.js
index 01d00fd..5fca82d 100644
--- a/app/pods/components/text-editor/component.js
+++ b/app/pods/components/text-editor/component.js
@@ -1,13 +1,25 @@
 import Ember from 'ember';
 /* global Quill */
 
-export default Ember.Component.extend({
-  quill: null,
+const { Component } = Ember;
+
+export default Component.extend({
+  // Passed in
   value: null,
-  update: null,
+
+  // Internal
+  quill: null,
+
+  didReceiveAttrs() {
+    this._super(...arguments);
+
+    if (!this.attrs.update) {
+      throw new Error(`You must provide an \`update\` action.`);
+    }
+  },
 
   didInsertElement: function() {
-    let quill = new Quill(`#${this.get('elementId')} .editor`, {
+    const quill = new Quill(`#${this.get('elementId')} .editor`, {
       formats: ['bold', 'italic', 'underline'],
       modules: {
         'toolbar': { container: `#${this.get('elementId')} .toolbar` }