Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

LayoutManager not defined #204

Merged
merged 0 commits into from

5 participants

@codymorrison

When creating a new project LayoutManager is not defined in app.js causing an error on .configure(). I setup a call to Backbone.Layout instead which is the shim's export.

Let me know if you have an issue with any of this.

@tbranyen
Owner

hrm wonder why the export is failing, might be because of an older version of LM that i'm packaging?

@codymorrison

Maybe I'm not seeing it but according to the require.js shim it's intended to work this way. https://github.com/tbranyen/backbone-boilerplate/blob/master/vendor/jam/require.config.js#L37-L44

@SBoudrias
Owner

In fact, the exports seems good there: http://requirejs.org/docs/api.html#config-shim . Can you check if by any means app module could be loaded before ../vendor/jam/require.config? (I tested it on my side and it was working correctly with the lastest master branch version)

As a side note, this pull request bring on a problem because it stop referencing LayoutManager as a dependency of app.js (which it still is even if we don't use the direct reference it returns). Plus, by referencing directly Backbone, you made Backbone one of your dependency. Can you fix it?

@tbranyen
Owner

I believe this is an issue with Lo-Dash and how it defines extend. Since these are being attached as static properties. I confirmed the bug locally as well. The underscore build should work fine though.

@khmelevskii

How can I fix this bug? My release and debug version not working

@tbranyen
Owner
@khmelevskii

I upgrade all libs (jam upgrade) and remove lodash. I am using underscore 1.4.4. But error remains:
Uncaught TypeError: Cannot call method 'configure' of undefined
LayoutManager not defined in app.

@jhrabows

My comment may not be directly related but I would like to replace LayoutManager with require text! plugin. I probably should open a separate request ticket for this.

@SBoudrias
Owner

@jhrabows Yeah, this is an unrelated question to the current topic. You'll need to override the fetch function to use text! plugin instead.

If you want more detail, open a question on StackOverflow.

@tbranyen tbranyen merged commit cd1eaa8 into backbone-boilerplate:master
@codymorrison

Just an update that this error still appears when running bbb server:debug or bbb server:release on BBB 0.2.0-alpha-5 on the latest build. Running bbb server seems to resolve the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
This page is out of date. Refresh to see the latest.
Showing with 0 additions and 0 deletions.
Something went wrong with that request. Please try again.