Use initAction() instead initScope()

initScope() for actions is deprecated in Ocata.[1]
This patch use initAction() instead initScope().
New scope is now provided as the second argument of perform().

[1]:https://review.openstack.org/#/c/345145/

Change-Id: Ie96f24fe7a892da29e4e2d80cfb100d2f3105c49
This commit is contained in:
Kenji Ishii 2016-12-02 10:57:16 +09:00
parent de53f37654
commit cb3eb2e16b
7 changed files with 20 additions and 25 deletions

View File

@ -41,14 +41,13 @@
function createService( function createService(
$location, policy, actionResult, gettext, $qExtensions, wizardModalService, toast, model, events, resourceType, createWorkflow $location, policy, actionResult, gettext, $qExtensions, wizardModalService, toast, model, events, resourceType, createWorkflow
) { ) {
var scope; var scope;
var message = { var message = {
success: gettext('Container %s was successfully created.') success: gettext('Container %s was successfully created.')
}; };
var service = { var service = {
initScope: initScope, initAction: initAction,
perform: perform, perform: perform,
allowed: allowed allowed: allowed
}; };
@ -57,15 +56,13 @@
////////////// //////////////
function initScope($scope) { function initAction() {
scope = $scope;
scope.workflow = createWorkflow;
scope.model = model;
scope.$on('$destroy', function() {
});
} }
function perform(selected) { function perform(selected, newScope) {
scope = newScope;
scope.workflow = createWorkflow;
scope.model = model;
scope.model.init(); scope.model.init();
// for creation according to selected item // for creation according to selected item
scope.selected = selected; scope.selected = selected;

View File

@ -51,7 +51,7 @@
successEvent: events.DELETE_SUCCESS successEvent: events.DELETE_SUCCESS
}; };
var service = { var service = {
initScope: initScope, initAction: initAction,
allowed: allowed, allowed: allowed,
perform: perform perform: perform
}; };
@ -61,10 +61,7 @@
////////////// //////////////
// include this function in your service function initAction() {
// if you plan to emit events to the parent controller
function initScope($scope) {
scope = $scope;
} }
function allowed() { function allowed() {
@ -72,7 +69,8 @@
} }
// delete selected resource objects // delete selected resource objects
function perform(selected) { function perform(selected, newScope) {
scope = newScope;
var selected = angular.isArray(selected) ? selected : [selected]; var selected = angular.isArray(selected) ? selected : [selected];
context.labels = labelize(selected.length); context.labels = labelize(selected.length);
return $qExtensions.allSettled(selected.map(checkPermission)).then(afterCheck); return $qExtensions.allSettled(selected.map(checkPermission)).then(afterCheck);

View File

@ -39,7 +39,7 @@
}; };
var service = { var service = {
initScope: initScope, initAction: initAction,
allowed: allowed, allowed: allowed,
perform: perform perform: perform
}; };
@ -50,7 +50,7 @@
// include this function in your service // include this function in your service
// if you plan to emit events to the parent controller // if you plan to emit events to the parent controller
function initScope() { function initAction() {
} }
function allowed() { function allowed() {

View File

@ -40,7 +40,7 @@
}; };
var service = { var service = {
initScope: initScope, initAction: initAction,
allowed: allowed, allowed: allowed,
perform: perform perform: perform
}; };
@ -51,7 +51,7 @@
// include this function in your service // include this function in your service
// if you plan to emit events to the parent controller // if you plan to emit events to the parent controller
function initScope() { function initAction() {
} }
function allowed() { function allowed() {

View File

@ -40,7 +40,7 @@
}; };
var service = { var service = {
initScope: initScope, initAction: initAction,
allowed: allowed, allowed: allowed,
perform: perform perform: perform
}; };
@ -51,7 +51,7 @@
// include this function in your service // include this function in your service
// if you plan to emit events to the parent controller // if you plan to emit events to the parent controller
function initScope() { function initAction() {
} }
function allowed() { function allowed() {

View File

@ -40,7 +40,7 @@
}; };
var service = { var service = {
initScope: initScope, initAction: initAction,
allowed: allowed, allowed: allowed,
perform: perform perform: perform
}; };
@ -51,7 +51,7 @@
// include this function in your service // include this function in your service
// if you plan to emit events to the parent controller // if you plan to emit events to the parent controller
function initScope() { function initAction() {
} }
function allowed() { function allowed() {

View File

@ -40,7 +40,7 @@
}; };
var service = { var service = {
initScope: initScope, initAction: initAction,
allowed: allowed, allowed: allowed,
perform: perform perform: perform
}; };
@ -51,7 +51,7 @@
// include this function in your service // include this function in your service
// if you plan to emit events to the parent controller // if you plan to emit events to the parent controller
function initScope() { function initAction() {
} }
function allowed() { function allowed() {