Merge pull request #3728 from FinnStutzenstein/stylistic-remarks
Fixed stylistic remarks from #3682 (fixes #3709)
This commit is contained in:
commit
331eb9fc22
@ -566,14 +566,14 @@ angular.module('OpenSlidesApp.core', [
|
|||||||
//
|
//
|
||||||
// It is possible to provide a scope, that is merged into the surrounding scope.
|
// It is possible to provide a scope, that is merged into the surrounding scope.
|
||||||
// You can override functions or values of the surrounding scope by providing them:
|
// You can override functions or values of the surrounding scope by providing them:
|
||||||
// { Id: 'hookId', template: '<button ng-click="customFn()">click me</button>',
|
// { id: 'hookId', template: '<button ng-click="customOrOverwritten()">click me</button>',
|
||||||
// scope: {
|
// scope: {
|
||||||
// customOrOverwritten: function () { /*Do something */ },
|
// customOrOverwritten: function () { /*Do something */ },
|
||||||
// },
|
// },
|
||||||
// }
|
// }
|
||||||
// Or you provide a function that returns an object of functions/values to overwrite to
|
// Or you provide a function that returns an object of functions/values to overwrite to
|
||||||
// get access to the scope merged in:
|
// get access to the scope merged in:
|
||||||
// { Id: 'hookId', template: '<button ng-click="customFn()">click me</button>',
|
// { id: 'hookId', template: '<button ng-click="customFn()">click me</button>',
|
||||||
// scope: function (scope) {
|
// scope: function (scope) {
|
||||||
// return {
|
// return {
|
||||||
// customOrOverwritten: function () {
|
// customOrOverwritten: function () {
|
||||||
@ -592,15 +592,15 @@ angular.module('OpenSlidesApp.core', [
|
|||||||
hooks: hooks,
|
hooks: hooks,
|
||||||
registerHook: function (hook) {
|
registerHook: function (hook) {
|
||||||
// Deprecated: Set the new style 'id', if 'Id' is given.
|
// Deprecated: Set the new style 'id', if 'Id' is given.
|
||||||
if (hook.id === undefined) {
|
if (hook.id === void 0) {
|
||||||
hook.id = hook.Id;
|
hook.id = hook.Id;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (hooks[hook.id] === undefined) {
|
if (hooks[hook.id] === void 0) {
|
||||||
hooks[hook.id] = [];
|
hooks[hook.id] = [];
|
||||||
}
|
}
|
||||||
// set override default
|
// set override default
|
||||||
if (hook.override === undefined) {
|
if (hook.override === void 0) {
|
||||||
hook.override = !!(hook.template || hook.templateUrl);
|
hook.override = !!(hook.template || hook.templateUrl);
|
||||||
}
|
}
|
||||||
hooks[hook.id].push(hook);
|
hooks[hook.id].push(hook);
|
||||||
|
Loading…
Reference in New Issue
Block a user