[deprecations] Remove the need to use beforeObserver in job-log component
This commit is contained in:
parent
21f25cd2bf
commit
13d2eba016
|
@ -3,36 +3,24 @@ import Ember from 'ember';
|
||||||
export default Ember.Component.extend({
|
export default Ember.Component.extend({
|
||||||
logBinding: 'job.log',
|
logBinding: 'job.log',
|
||||||
|
|
||||||
didInsertElement() {
|
didReceiveAttrs: function(options) {
|
||||||
Ember.run.scheduleOnce('afterRender', this, 'setupLog');
|
this._super(...arguments);
|
||||||
},
|
|
||||||
|
|
||||||
logDidChange: function() {
|
if(options.oldAttrs && options.oldAttrs.job) {
|
||||||
return this.setupLog();
|
this.teardownLog(options.oldAttrs.job.value);
|
||||||
}.observes('log'),
|
}
|
||||||
|
|
||||||
logWillChange: function() {
|
if(options.newAttrs && options.newAttrs.job) {
|
||||||
return this.teardownLog();
|
this.setupLog(options.newAttrs.job.value);
|
||||||
}.observesBefore('log'),
|
|
||||||
|
|
||||||
willDestroyElement() {
|
|
||||||
Ember.run.scheduleOnce('afterRender', this, 'teardownLog');
|
|
||||||
},
|
|
||||||
|
|
||||||
teardownLog() {
|
|
||||||
var job;
|
|
||||||
job = this.get('job');
|
|
||||||
if (job) {
|
|
||||||
return job.unsubscribe();
|
|
||||||
}
|
}
|
||||||
},
|
},
|
||||||
|
|
||||||
setupLog() {
|
teardownLog(job) {
|
||||||
var job;
|
job.unsubscribe();
|
||||||
job = this.get('job');
|
},
|
||||||
if (job) {
|
|
||||||
|
setupLog(job) {
|
||||||
job.get('log').fetch();
|
job.get('log').fetch();
|
||||||
return job.subscribe();
|
job.subscribe();
|
||||||
}
|
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
|
|
Loading…
Reference in New Issue
Block a user