Skip to content

Commit

Permalink
#131 Replace the inline controller declaration for the job report modal
Browse files Browse the repository at this point in the history
  • Loading branch information
nadouani committed Jun 28, 2017
1 parent 1e858e4 commit 7be046c
Showing 1 changed file with 15 additions and 10 deletions.
25 changes: 15 additions & 10 deletions ui/app/scripts/controllers/case/CaseObservablesCtrl.js
Original file line number Diff line number Diff line change
Expand Up @@ -642,18 +642,16 @@

var modalInstance = $uibModal.open({
templateUrl: 'views/partials/observables/list/job-report-dialog.html',
controller: function($uibModalInstance, report, observable) {
this.report = report;
this.observable = observable;
this.close = function() {
$uibModalInstance.dismiss();
}
},
controller: 'JobReportModalCtrl',
controllerAs: '$vm',
size: 'max',
resolve: {
report: report,
observable: observable
report: function() {
return report
},
observable: function() {
return observable;
}
}
});
})
Expand All @@ -662,6 +660,13 @@
})
}
}
);
)
.controller('JobReportModalCtrl', function($uibModalInstance, report, observable) {
this.report = report;
this.observable = observable;
this.close = function() {
$uibModalInstance.dismiss();
}
});

})();

0 comments on commit 7be046c

Please sign in to comment.