From e045ab7481f2a0d5964a1b0aae27d71ffcfe54a6 Mon Sep 17 00:00:00 2001 From: Piotr Sarnacki <drogus@gmail.com> Date: Fri, 20 Jun 2014 15:47:37 +0200 Subject: [PATCH] Remove check of model state in DurationCalculation We no longer need this check, because it was only relevant with Ember Data. Additionally it was a cause of a travis-ci/travis-ci#1992, because of wrong syntax: unless someFunction false or true will only check for a result of someFunction with an argument being the result of `or` expression. --- assets/scripts/app/models/extensions.coffee | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/assets/scripts/app/models/extensions.coffee b/assets/scripts/app/models/extensions.coffee index 013da4f5..c94679b9 100644 --- a/assets/scripts/app/models/extensions.coffee +++ b/assets/scripts/app/models/extensions.coffee @@ -7,6 +7,6 @@ Travis.DurationCalculations = Ember.Mixin.create ).property('_duration', 'finishedAt', 'startedAt') updateTimes: -> - unless ['rootState.loaded.reloading', 'rootState.loading'].contains @get('stateManager.currentState.path') or @get('isFinished') + unless @get('isFinished') @notifyPropertyChange '_duration' @notifyPropertyChange 'finished_at'