Enforce that the current date is between min/max date (adopted-ember-addons#121)

* The component can now react if the current date value is below the minDate or above the maxDate.
 - New component property enforceDateIntervals is used to activate this behavior
 - When `enforceDateIntervals` is `true` the component will set the pikaday date to minDate or maxDate (currentDate < minDate or currentDate > maxDate) and call `onSelection` with the new date.

* Added documentation for enfoceDateIntervals option and behavior

* Fixed documentation about enforceDateIntervals

* Only call onSelection when enforceDateIntervals is true. We don't need to call pikaday.setDate

* Updated documentation

* Testing againts day instead of dates in tests for enforceDateIntervals

* When enforceDateIntervals is active we call onSelection on the next sync queue to prevent reassigments when rendering

* Removed unused code in the dummy application

* Removed everything related to enforceDateIntervals. The component by default reacts to minDate and maxDate changes, and change the pikaday current date accordingly

* Modified test names

* Instead of calling onSelection to force the date to be between minDate and maxDate we call pikaday.setDate directly

* Removed traces of enforceDateIntervals property used in some tests

* Moved the complete logic of checking for minDate and maxDates outside interval to the queued function

Author: showy

.travis.yml

@@ -14,6 +14,7 @@ env:
   - EMBER_TRY_SCENARIO=ember-release
   - EMBER_TRY_SCENARIO=ember-beta
   - EMBER_TRY_SCENARIO=ember-canary
+  - EMBER_TRY_SCENARIO=ember-2.8.3
   - EMBER_TRY_SCENARIO=ember-2.4.3
   - EMBER_TRY_SCENARIO=ember-2.3.1
   - EMBER_TRY_SCENARIO=ember-2.2.2

addon/mixins/pikaday.js

@@ -5,10 +5,13 @@ import moment from 'moment';
 const {
   isPresent,
   run,
+  getProperties
 } = Ember;
+
 const assign = Ember.assign || Ember.merge;
 
 export default Ember.Mixin.create({
+
   _options: Ember.computed('options', 'i18n', {
     get() {
       let options = this._defaultOptions();
@@ -99,17 +102,35 @@ export default Ember.Mixin.create({
   },
 
   setMinDate: function() {
-    if (this.get('minDate')) {
+    const { pikaday, minDate, value } = getProperties(this, [ 'pikaday', 'minDate', 'value' ]);
+
+    if (minDate) {
       run.later(() => {
-        this.get('pikaday').setMinDate(this.get('minDate'));
+        pikaday.setMinDate(minDate);
+      });
+
+      // If the current date is lower than minDate we set date to minDate
+      run.schedule('sync', () => {
+        if (value < minDate) {
+          pikaday.setDate(minDate);
+        }
       });
     }
   },
 
   setMaxDate: function() {
-    if (this.get('maxDate')) {
+    const { pikaday, maxDate, value }  = getProperties(this, [ 'pikaday', 'maxDate', 'value' ]);
+
+    if (maxDate) {
       run.later(() => {
-        this.get('pikaday').setMaxDate(this.get('maxDate'));
+        pikaday.setMaxDate(maxDate);
+      });
+
+      // If the current date is greater than maxDate we set date to maxDate
+      run.schedule('sync', () => {
+        if (value > maxDate) {
+          pikaday.setDate(maxDate);
+        }
       });
     }
   },

config/ember-try.js

@@ -39,6 +39,17 @@ module.exports = {
         }
       }
     },
+    {
+      name: 'ember-2.8.3',
+      bower: {
+        dependencies: {
+          'ember': '2.8.3'
+        },
+        resolutions: {
+          'ember': '2.8.3'
+        }
+      }
+    },
     {
       name: 'ember-2.4.3',
       bower: {

index.js

@@ -3,7 +3,7 @@
 
 module.exports = {
   name: 'ember-pikaday',
-
+  
   options: {
     nodeAssets: {
       pikaday: {

tests/dummy/app/controllers/application.js

@@ -9,5 +9,5 @@ export default Ember.Controller.extend({
     doSomethingWithSelectedValue(value) {
       console.log(value);
     }
-  }
-});
+  },
+});

tests/dummy/app/templates/application.hbs

@@ -21,4 +21,4 @@ Datepicker already disabled before it's rendered:
 
 Inputless datepicker:
 {{pikaday-inputless onSelection=(action (mut someDate))}}
-{{someDate}}
+{{someDate}}

tests/integration/components/pikaday-input-test.js

@@ -388,3 +388,31 @@ test('if updates pikaday config if options hash is changed', function(assert) {
   openDatepicker(this.$('input'));
   assert.notOk($(`td[data-day=${weekendDay}]`).hasClass('is-disabled'));
 });
+
+test('if minDate is greater than value we we set pikaday\'s current date to minDate' ,function(assert) {
+  assert.expect(1);
+
+  let today = new Date();
+  let tomorrow = new Date( Date.now() + (60 * 60 * 24 * 1000));
+
+  this.set('currentDate', today);
+  this.set('minDate', today);
+  this.render(hbs`{{pikaday-input minDate=minDate value=currentDate onSelection=(action (mut currentDate)) }}`);
+
+  this.set('minDate', tomorrow);
+  assert.equal(this.get('currentDate').getDate(), tomorrow.getDate(), 'value should change');
+});
+
+test('if maxDate is lower than value we set pikaday\'s current date to maxDate', function(assert) {
+  assert.expect(1);
+
+  let today = new Date();
+  let tomorrow = new Date( Date.now() + (60 * 60 * 24 * 1000));
+
+  this.set('currentDate', tomorrow);
+  this.set('maxDate', tomorrow);
+  this.render(hbs`{{pikaday-input maxDate=maxDate value=currentDate onSelection=(action (mut currentDate)) }}`);
+
+  this.set('maxDate', today);
+  assert.equal(this.get('currentDate').getDate(), today.getDate(), 'value should change');
+});