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

定义新的model后,路由切换时报app.model: namespace should be unique #22

Closed
YehanZhou opened this issue Jan 5, 2017 · 14 comments

Comments

@YehanZhou
Copy link
Contributor

YehanZhou commented Jan 5, 2017

定义新的model后,路由切换时报错 app.model: namespace should be unique

model:
models

路由配置:
router

报错截图:
error

@xianaldo
Copy link

xianaldo commented Jan 5, 2017

遇到一样的问题

@zuiidea
Copy link
Owner

zuiidea commented Jan 5, 2017

依次檢查:

  • src/index.js有沒有已經定義了該model(即app.model()
  • src/models文件夾下的文件的namespace是否重複定義

@YehanZhou
Copy link
Contributor Author

YehanZhou commented Jan 5, 2017

检查了这两项,都没有问题。
会不会是路由跳转的时候,由于又require了一次model,导致namespace又定义一遍,所以重复了?

@zuiidea
Copy link
Owner

zuiidea commented Jan 5, 2017

不會啊,第一次跳轉到該路由時,僅加載一次該組件的js,此時是第一次require該組件的model,整個項目衹會require一次model/markets
試試注釋該路由,會不會報錯
方便的話,提交到你的分支,我幫你檢查一下

@YehanZhou
Copy link
Contributor Author

YehanZhou commented Jan 6, 2017

我刚下载了这个分支上的代码,不加任何改动就会报错。
你试一下,先跳到【仪表盘】,再跳到【用户管理】,再又跳回到【仪表盘】,就会报错。
应该就是第二次跳某个带model的路由时,就会报app.model: namespace should be unique。

@zuiidea
Copy link
Owner

zuiidea commented Jan 7, 2017

這兩天出去玩了,沒帶電腦😅

gif

我這邊暫時不能復現

@taven-liu
Copy link

taven-liu commented Jan 10, 2017

出现同样问题了

@zuiidea
Copy link
Owner

zuiidea commented Jan 10, 2017

@taven-liu能看一下你的新增代码吗

@taven-liu
Copy link

{ path: 't1', name: 't1', getComponent (nextState, cb) { require.ensure([], require => { app.model(require('./modules/aTest/models/t1')) cb(null, {component: require('./modules/aTest/containers/t1Container')}) }) } },

@xianaldo
Copy link

xianaldo commented Jan 10, 2017 via email

@zuiidea
Copy link
Owner

zuiidea commented Jan 10, 2017

dvajs/dva#465

可能是dva版本的原因,暂时先把dva 切换到 1.1 版本吧,后面我解决一下

@zuiidea
Copy link
Owner

zuiidea commented Jan 17, 2017

已解决 ff2005d

@xiaosansiji
Copy link

@zuiidea @YehanZhou @xianaldo

dvajs/dva#533 官方针对1.2的解决办法,最好还是搞下吧,要不升级到1.2后还会有问题

@zuiidea
Copy link
Owner

zuiidea commented Feb 3, 2017

@xiaosansiji ,现在升级没问题了啊,ff2005d

@zuiidea zuiidea closed this as completed Feb 7, 2017
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

5 participants