starred/stf-DeviceFarmer
1
0
Fork 0
mirror of https://github.com/DeviceFarmer/stf.git synced 2026-04-18 16:13:24 +02:00
Code Issues Packages Projects Releases Wiki Activity

Don't expose install() to scope directly because it shouldn't be called.

Browse Source
This commit is contained in:
Simo Kinnunen
2014-09-05 13:27:55 +09:00
parent ede65d518c
commit eb73140465
1 changed files with 9 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
res/app/control-panes/dashboard/install/install-controller.js
View File

@@ -43,6 +43,13 @@ module.exports = function InstallCtrl(
}
}
function install(installation) {
return $scope.control.install(installation)
.progressed(function (result) {
installation.update(50 + result.progress / 2, result.lastData)
})
}
$scope.accordionOpen = true
$scope.clear = function () {
@@ -59,7 +66,7 @@ module.exports = function InstallCtrl(
.then(function (uploadResult) {
installation.update(uploadResult.progress / 2, uploadResult.lastData)
installation.manifest = uploadResult.body
return $scope.install(installation)
return install(installation)
})
.then(function() {
installation.okay('installed')
@@ -88,7 +95,7 @@ module.exports = function InstallCtrl(
.then(function(res) {
if (res.data.success) {
installation.manifest = res.data.manifest
return $scope.install(installation)
return install(installation)
}
else {
throw new Error('Unable to retrieve manifest')
@@ -103,13 +110,6 @@ module.exports = function InstallCtrl(
})
}
$scope.install = function (installation) {
return $scope.control.install(installation)
.progressed(function (result) {
installation.update(50 + result.progress / 2, result.lastData)
})
}
$scope.uninstall = function (packageName) {
// TODO: After clicking uninstall accordion opens
return $scope.control.uninstall(packageName)