use page object to store/use staticAppConfig #1056

Merged
merged 2 commits into from Apr 2, 2013

Conversation

Projects
None yet
2 participants
Contributor

drewfish commented Apr 2, 2013

Reworking of PR #1051 to hang staticAppConfig off of "page" object (instead of dispatcher).

lib/app/autoload/mojito-client.client.js
@@ -338,6 +338,9 @@ YUI.add('mojito-client', function(Y, NAME) {
// pass globalHookhandler to addons that may want to use hooks
globalHookHandler: globalHookHandler
};
+
+ this.page.staticAppConfig = config.appConfig;
@caridy

caridy Apr 2, 2013

Collaborator

appConfig is a local variable pointing to this.appConfig.

@drewfish

drewfish Apr 2, 2013

Contributor

D'oh.. missed that, will use that instead.

Collaborator

caridy commented Apr 2, 2013

Lets add a couple of tests and merge this.

+1

drewfish added a commit that referenced this pull request Apr 2, 2013

Merge pull request #1056 from drewfish/dispage
use page object to store/use staticAppConfig

@drewfish drewfish merged commit c40415a into YahooArchive:develop Apr 2, 2013

1 check passed

default The Travis build passed
Details

@drewfish drewfish deleted the drewfish:dispage branch Apr 2, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment