From 30f8e89c404cba46ce2702e327eef38036ace968 Mon Sep 17 00:00:00 2001
From: Matthew Dillon <mrdillon@alaska.edu>
Date: Wed, 17 Jun 2015 14:40:28 -0800
Subject: [PATCH] Rough cut: compare.

---
 app/helpers/get-property.js                |  8 ++++
 app/pods/application/template.hbs          | 15 ++++---
 app/pods/compare/controller.js             | 47 ++++++++++++++++++++++
 app/pods/compare/route.js                  |  4 ++
 app/pods/compare/template.hbs              | 32 +++++++++++++++
 app/pods/index/route.js                    |  6 ++-
 app/router.js                              |  4 +-
 tests/unit/helpers/get-property-test.js    | 10 +++++
 tests/unit/pods/compare/controller-test.js | 12 ++++++
 tests/unit/pods/compare/route-test.js      | 11 +++++
 10 files changed, 140 insertions(+), 9 deletions(-)
 create mode 100644 app/helpers/get-property.js
 create mode 100644 app/pods/compare/controller.js
 create mode 100644 app/pods/compare/route.js
 create mode 100644 app/pods/compare/template.hbs
 create mode 100644 tests/unit/helpers/get-property-test.js
 create mode 100644 tests/unit/pods/compare/controller-test.js
 create mode 100644 tests/unit/pods/compare/route-test.js

diff --git a/app/helpers/get-property.js b/app/helpers/get-property.js
new file mode 100644
index 0000000..6b2308d
--- /dev/null
+++ b/app/helpers/get-property.js
@@ -0,0 +1,8 @@
+import Ember from 'ember';
+
+// This will be unneccesary when ember 2.0 lands
+export function getProperty(params) {
+  return Ember.get(params[0], params[1]);
+}
+
+export default Ember.HTMLBars.makeBoundHelper(getProperty);
diff --git a/app/pods/application/template.hbs b/app/pods/application/template.hbs
index 245a185..729e634 100644
--- a/app/pods/application/template.hbs
+++ b/app/pods/application/template.hbs
@@ -4,18 +4,21 @@
     {{/link-to}}
     {{#if session.isAuthenticated}}
       <ul>
+        {{#link-to 'compare' tagName='li' href=false}}
+          {{link-to 'Compare' 'compare'}}
+        {{/link-to}}
+        {{#link-to 'measurements' tagName='li' href=false}}
+          {{link-to 'Measurements' 'measurements'}}
+        {{/link-to}}
+        {{#link-to 'characteristics' tagName='li' href=false}}
+          {{link-to 'Characteristics' 'characteristics'}}
+        {{/link-to}}
         {{#link-to 'species' tagName='li' href=false}}
           {{link-to 'Species' 'species'}}
         {{/link-to}}
         {{#link-to 'strains' tagName='li' href=false}}
           {{link-to 'Strains' 'strains'}}
         {{/link-to}}
-        {{#link-to 'characteristics' tagName='li' href=false}}
-          {{link-to 'Characteristics' 'characteristics'}}
-        {{/link-to}}
-        {{#link-to 'measurements' tagName='li' href=false}}
-          {{link-to 'Measurements' 'measurements'}}
-        {{/link-to}}
         {{#link-to 'about' tagName='li' href=false}}
           {{link-to 'About' 'about'}}
         {{/link-to}}
diff --git a/app/pods/compare/controller.js b/app/pods/compare/controller.js
new file mode 100644
index 0000000..fe7ebe5
--- /dev/null
+++ b/app/pods/compare/controller.js
@@ -0,0 +1,47 @@
+import Ember from 'ember';
+
+export default Ember.Controller.extend({
+  strains: [],
+  dataEmpty: true,
+
+  actions: {
+    search: function(selectedStrains, selectedCharacteristics) {
+      if (Ember.isEmpty(selectedStrains) || Ember.isEmpty(selectedCharacteristics)) {
+        this.set('dataEmpty', true);
+        return false;
+      }
+
+      let data = Ember.A();
+      let strains = [];
+      selectedStrains.forEach((strain) => {
+        let s = this.store.getById('strain', strain);
+        strains.pushObject(s);
+      })
+      this.set('strains', strains);
+
+      this.store.find('measurement', {
+        strain: selectedStrains,
+        characteristic: selectedCharacteristics,
+      }).then((measurements) => {
+        selectedCharacteristics.forEach((characteristic) => {
+          let char = this.store.getById('characteristic', characteristic);
+          let row = {
+            characteristic: char.get('characteristicName'),
+          };
+          selectedStrains.forEach((strain) => {
+            let meas = measurements.filterBy('strain.id', strain)
+              .filterBy('characteristic.id', characteristic);
+            if (!Ember.isEmpty(meas)) {
+              row[strain] = meas[0].get('value');
+            } else {
+              row[strain] = '';
+            }
+          });
+          data.pushObject(row);
+        });
+        this.set('data', data);
+        this.set('dataEmpty', false);
+      });
+    }
+  },
+});
diff --git a/app/pods/compare/route.js b/app/pods/compare/route.js
new file mode 100644
index 0000000..5b14286
--- /dev/null
+++ b/app/pods/compare/route.js
@@ -0,0 +1,4 @@
+import Ember from 'ember';
+import AuthenticatedRouteMixin from 'simple-auth/mixins/authenticated-route-mixin';
+
+export default Ember.Route.extend(AuthenticatedRouteMixin);
diff --git a/app/pods/compare/template.hbs b/app/pods/compare/template.hbs
new file mode 100644
index 0000000..8cf1faa
--- /dev/null
+++ b/app/pods/compare/template.hbs
@@ -0,0 +1,32 @@
+<h2>{{genus-name}} - Compare Strains</h2>
+
+{{measurement-search-panel search='search'}}
+
+{{#if dataEmpty}}
+  <div class="flakes-message information">
+    Please select one or more strains and one or more characteristics.
+  </div>
+{{else}}
+<div>
+  <table class="flakes-table">
+    <thead>
+      <tr>
+        <th>Characteristic</th>
+        {{#each strains as |strain|}}
+          <th>{{strain.fullNameMU}}</th>
+        {{/each}}
+      </tr>
+    </thead>
+    <tbody>
+      {{#each data as |row|}}
+        <tr>
+          <td>{{row.characteristic}}</td>
+          {{#each strains as |strain|}}
+            <td>{{get-property row strain.id}}</td>
+          {{/each}}
+        </tr>
+      {{/each}}
+    </tbody>
+  </table>
+</div>
+{{/if}}
diff --git a/app/pods/index/route.js b/app/pods/index/route.js
index 5b14286..d38ae27 100644
--- a/app/pods/index/route.js
+++ b/app/pods/index/route.js
@@ -1,4 +1,8 @@
 import Ember from 'ember';
 import AuthenticatedRouteMixin from 'simple-auth/mixins/authenticated-route-mixin';
 
-export default Ember.Route.extend(AuthenticatedRouteMixin);
+export default Ember.Route.extend(AuthenticatedRouteMixin, {
+  beforeModel: function() {
+    this.transitionTo('compare');
+  }
+});
diff --git a/app/router.js b/app/router.js
index b46afb6..03f3b68 100644
--- a/app/router.js
+++ b/app/router.js
@@ -10,6 +10,8 @@ Router.map(function() {
   this.route('about');
   this.route('characteristics');
   this.route('users');
+  this.route('measurements');
+  this.route('compare');
 
   this.route('species', function() {
     this.route('new');
@@ -19,8 +21,6 @@ Router.map(function() {
     this.route('new');
     this.route('show', { path: ':strain_id' });
   });
-
-  this.route('measurements');
 });
 
 export default Router;
diff --git a/tests/unit/helpers/get-property-test.js b/tests/unit/helpers/get-property-test.js
new file mode 100644
index 0000000..25955ae
--- /dev/null
+++ b/tests/unit/helpers/get-property-test.js
@@ -0,0 +1,10 @@
+import { getProperty } from '../../../helpers/get-property';
+import { module, test } from 'qunit';
+
+module('Unit | Helper | get property');
+
+// Replace this with your real tests.
+test('it works', function(assert) {
+  var result = getProperty(42);
+  assert.ok(result);
+});
diff --git a/tests/unit/pods/compare/controller-test.js b/tests/unit/pods/compare/controller-test.js
new file mode 100644
index 0000000..c34abc7
--- /dev/null
+++ b/tests/unit/pods/compare/controller-test.js
@@ -0,0 +1,12 @@
+import { moduleFor, test } from 'ember-qunit';
+
+moduleFor('controller:compare', {
+  // Specify the other units that are required for this test.
+  // needs: ['controller:foo']
+});
+
+// Replace this with your real tests.
+test('it exists', function(assert) {
+  var controller = this.subject();
+  assert.ok(controller);
+});
diff --git a/tests/unit/pods/compare/route-test.js b/tests/unit/pods/compare/route-test.js
new file mode 100644
index 0000000..c940264
--- /dev/null
+++ b/tests/unit/pods/compare/route-test.js
@@ -0,0 +1,11 @@
+import { moduleFor, test } from 'ember-qunit';
+
+moduleFor('route:compare', 'Unit | Route | compare', {
+  // Specify the other units that are required for this test.
+  // needs: ['controller:foo']
+});
+
+test('it exists', function(assert) {
+  var route = this.subject();
+  assert.ok(route);
+});