Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Not compatible with new Sencha Cmd #22

Open
Reggino opened this issue Oct 31, 2012 · 4 comments
Open

Not compatible with new Sencha Cmd #22

Reggino opened this issue Oct 31, 2012 · 4 comments

Comments

@Reggino
Copy link

Reggino commented Oct 31, 2012

Thanks for your great work!

When building a project that contains extjs-boxselect using the new Sencha Cmd [http://www.sencha.com/blog/all-new-sencha-cmd/] , the build fails:

[INFO ] Processing class inheritance graph
[INFO ] Processing instantiation refereces to classes and aliases
[INFO ] Processing source dependencies
[ERROR] Circular requires reference :
.\development\app\app.js ->
.\development\app\controller\ObjectSelect.js ->
.\development\app\form\field\MultipleObjectComboBox.js ->
.\development\ext-addons\form\field\BoxSelect.js ->
.\development\ext-addons\form\field\BoxSelect.js

This is caused by the declaration of both 'Ext.ux.form.field.BoxSelect' and 'Ext.ux.layout.component.field.BoxSelectField' in the same file: sencha cmd can not handle that. Splitting the file in parts should fix this issue.

@jasontwill
Copy link

Confirmed. Separating the 2 declarations into files works. Make sure you put the BoxSelectField before BoxSelect.

@amackay11
Copy link

Since we are using 'Ext.ux.layout.component.field.BoxSelectField' and 'Ext.ux.form.field.BoxSelect' I'm unsure how to split the files. Perhaps this is more of a Sencha CMD question but an example would be nice. Would I copy the files into the Ext.ux directory structure or my own structure boxselect/src/...? Thanks.

@gabeattack
Copy link

I've submitted a pull request #24. I tried to remember the changes I made previously to get this to work so please let me know if you have any issues and I'll try to install CMD3 on my current computer.

@Reggino
Copy link
Author

Reggino commented Feb 26, 2013

This issue is fixed in pull request #27 too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants