Skip to content

Commit

Permalink
Revert: 32a975e BIG CHANGE: move checkTimeLimits from setReady to onS…
Browse files Browse the repository at this point in the history
…uccessfulLoad to ensure it triggers before setReady. Reason: onSuccessfulLoad is not propagated to tool level, so the event is lost when hook is changed
  • Loading branch information
angiehjort committed Apr 12, 2018
1 parent c1f4ddd commit b0138ed
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/base/tool.js
Expand Up @@ -67,9 +67,9 @@ const ToolModel = Model.extend({
validate_func(c);
},

onSuccessfullLoad() {
this.checkTimeLimits();
this._super();
setReady(arg) {
if (arg !== false) this.checkTimeLimits();
this._super(arg);
},


Expand Down

0 comments on commit b0138ed

Please sign in to comment.