Skip to content

Commit

Permalink
Allow you to generateActions in subclass
Browse files Browse the repository at this point in the history
  • Loading branch information
goatslacker committed Apr 7, 2015
1 parent bc1a42b commit 06d07e1
Show file tree
Hide file tree
Showing 6 changed files with 11 additions and 5 deletions.
2 changes: 1 addition & 1 deletion dist/alt-browser-with-addons.js
Expand Up @@ -1595,7 +1595,7 @@ var Alt = (function () {
// Instance type methods for injecting alt into your application as context

value: function addActions(name, ActionsClass) {
this.actions[name] = this.createActions(ActionsClass);
this.actions[name] = Array.isArray(ActionsClass) ? this.generateActions.apply(this, ActionsClass) : this.createActions(ActionsClass);
}
},
addStore: {
Expand Down
2 changes: 1 addition & 1 deletion dist/alt-browser.js
Expand Up @@ -1339,7 +1339,7 @@ var Alt = (function () {
// Instance type methods for injecting alt into your application as context

value: function addActions(name, ActionsClass) {
this.actions[name] = this.createActions(ActionsClass);
this.actions[name] = Array.isArray(ActionsClass) ? this.generateActions.apply(this, ActionsClass) : this.createActions(ActionsClass);
}
},
addStore: {
Expand Down
2 changes: 1 addition & 1 deletion dist/alt-with-runtime.js
Expand Up @@ -588,7 +588,7 @@ var Alt = (function () {
// Instance type methods for injecting alt into your application as context

value: function addActions(name, ActionsClass) {
this.actions[name] = this.createActions(ActionsClass);
this.actions[name] = Array.isArray(ActionsClass) ? this.generateActions.apply(this, ActionsClass) : this.createActions(ActionsClass);
}
},
addStore: {
Expand Down
2 changes: 1 addition & 1 deletion dist/alt.js
Expand Up @@ -606,7 +606,7 @@ var Alt = (function () {
// Instance type methods for injecting alt into your application as context

value: function addActions(name, ActionsClass) {
this.actions[name] = this.createActions(ActionsClass);
this.actions[name] = Array.isArray(ActionsClass) ? this.generateActions.apply(this, ActionsClass) : this.createActions(ActionsClass);
}
},
addStore: {
Expand Down
4 changes: 3 additions & 1 deletion src/alt.js
Expand Up @@ -533,7 +533,9 @@ class Alt {
// Instance type methods for injecting alt into your application as context

addActions(name, ActionsClass) {
this.actions[name] = this.createActions(ActionsClass)
this.actions[name] = Array.isArray(ActionsClass)
? this.generateActions.apply(this, ActionsClass)
: this.createActions(ActionsClass)
}

addStore(name, StoreModel, saveStore) {
Expand Down
4 changes: 4 additions & 0 deletions test/index.js
Expand Up @@ -317,6 +317,7 @@ class AltInstance extends Alt {
constructor() {
super()
this.addActions('myActions', MyActions)
this.addActions('fauxActions', ['one', 'two'])
this.addStore('myStore', MyStore)
}
}
Expand Down Expand Up @@ -762,6 +763,9 @@ const tests = {
const myActionsFromInst = altInstance.getActions('myActions')
assert.isObject(myActionsFromInst, 'the actions exist')

const fauxActions = altInstance.getActions('fauxActions')
assert.isFunction(fauxActions.one, 'faux actions were generated')

const myActionsFail = altInstance.getActions('ActionsThatDontExist')
assert.isUndefined(myActionsFail, 'undefined actions')

Expand Down

0 comments on commit 06d07e1

Please sign in to comment.