Merge pull request #180 from 67P/feature/add_contribution_with_params

Add "give kredits" button to contributor profiles

app/components/add-contribution/component.js

@@ -1,7 +1,7 @@
 import Component from '@ember/component';
 import { computed } from '@ember/object';
 import { and, notEmpty } from '@ember/object/computed';
-import { isEmpty } from '@ember/utils';
+import { assign } from '@ember/polyfills';
 import moment from 'moment';
 
 export default Component.extend({
@@ -24,19 +24,17 @@ export default Component.extend({
   init () {
     this._super(...arguments);
     this.set('defaultDate', moment().startOf('hour').toDate());
+    this.set('defaultAttr', {
+      contributorId: null,
+      kind: null,
+      date: this.defaultDate,
+      amount: null,
+      description: null,
+      url: null,
+      details: null
+    });
 
-    // Default attributes used by reset
+    this.set('attributes', assign({}, this.defaultAttr, this.attributes));
-    if (isEmpty(this.attributes)) {
-      this.set('attributes', {
-        contributorId: null,
-        kind: null,
-        date: this.defaultDate,
-        amount: null,
-        description: null,
-        url: null,
-        details: null
-      });
-    }
 
     this.reset();
   },

app/controllers/contributions/new.js

@@ -6,6 +6,8 @@ export default Controller.extend({
 
   kredits: service(),
 
+  queryParams: ['contributorId', 'kind', 'amount'],
+
   contributors: alias('kredits.contributors'),
   minedContributors: filterBy('contributors', 'id'),
 

app/router.js

@@ -20,7 +20,7 @@ Router.map(function() {
     this.route('new');
   });
   this.route('contributions', function() {
-    this.route('new');
+    this.route('new', { queryParams: ['contributorId', 'kind', 'amount'] });
     this.route('resubmit', { path: ':id/resubmit' });
   });
   this.route('contributors', function() {

app/routes/contributions/new.js

@@ -0,0 +1,9 @@
+import Route from '@ember/routing/route';
+
+export default Route.extend({
+
+  model (params) {
+    return { params };
+  }
+
+});

app/styles/components/_contributor-profile.scss

@@ -42,6 +42,10 @@ section#contributor-profile {
         }
       }
     }
+
+    .actions {
+      margin-top: 2rem;
+    }
   }
 
   .actions {

app/templates/contributions/new.hbs

@@ -6,7 +6,9 @@
     </header>
 
     <div class="content">
-      {{add-contribution contributors=sortedContributors save=(action "save")}}
+      {{add-contribution contributors=sortedContributors
+                         attributes=model.params
+                         save=(action "save")}}
     </div>
   </section>
 

app/templates/dashboard/contributors/show.hbs

@@ -22,6 +22,13 @@
         </li>
       {{/if}}
     </ul>
+    <div class="actions">
+      <p>
+        {{link-to "♥ Give kredits" "contributions.new"
+                  (query-params contributorId=model.id)
+                  class="button green"}}
+      </p>
+    </div>
   </div>
 
   <div class="actions">

tests/unit/components/add-contribution-test.js

@@ -0,0 +1,41 @@
+import { module, test } from 'qunit';
+import { setupTest } from 'ember-qunit';
+import moment from 'moment';
+
+module('Unit | Component | add-contribution', function(hooks) {
+  setupTest(hooks);
+
+  test('it exists', function(assert) {
+    let component = this.owner.factoryFor('component:add-contribution').create();
+    assert.ok(component);
+  });
+
+  test('default attributes', function(assert) {
+    let component = this.owner.factoryFor('component:add-contribution').create();
+
+    ['contributorId', 'kind', 'amount', 'description', 'url', 'details'].forEach(a => {
+      assert.equal(component.attributes[a], null, `sets the default ${a} attribute`);
+      assert.equal(component.get(a), null, `sets the ${a} property`);
+    });
+
+    assert.ok(moment.isDate(component.attributes.date), 'sets the default date attribute');
+    assert.ok(moment.isDate(component.date), 'sets the default date');
+  });
+
+  test('override default attributes', function(assert) {
+    let component = this.owner.factoryFor('component:add-contribution').create({
+      attributes: { contributorId: '1' }
+    });
+
+    assert.equal(component.contributorId, '1', `overrides the default property`);
+
+    ['kind', 'amount', 'description', 'url', 'details'].forEach(a => {
+      assert.equal(component.attributes[a], null, `sets the default ${a} attribute`);
+      assert.equal(component.get(a), null, `sets the ${a} property`);
+    });
+
+    assert.ok(moment.isDate(component.attributes.date), 'sets the default date attribute');
+    assert.ok(moment.isDate(component.date), 'sets the default date');
+  });
+
+});