ecomfe/er

title没有放到ActionContext中

Closed this issue · 10 comments

这种向前兼容真累

fixed #114 带来一个坑啊,组里各种反馈model里边的title字段取不到,可以加个er_或者action_的前缀吗?

估计是有的 team 就是期望 model.get('title') 拿到当前 Action 标题才会这么改吧……但是还是觉得这个比较坑啊。

可以在config里用documentTitle代替title,这样就不影响title这个字段了

在 2014年7月29日,下午12:08,GU Yiling notifications@github.com 写道:

估计是有的 team 就是期望 model.get('title') 拿到当前 Action 标题才会这么改吧……但是还是觉得这个比较坑啊。


Reply to this email directly or view it on GitHub.

https://github.com/ecomfe/er/blob/master/src/controller.js#L407
documentTitle 代替 title 的时候,这里似乎已经强行把 args.title 赋值为 undefined 了吧?

是哦,这里是个BUG……

现在的情况是,有些项目,就是要用model.get('title')获得Action的标题,但是被Model里的后续字段给覆盖了?

现在的情况是我们的项目一般没有action.titlemodel.get('title')获得的是undefined而不是业务数据,所以提议加个前缀,现在项目里边都把title的字段改掉了

不应该啊……从Action的流程来看,是先把actionContext放进model中,再调用model.load()的,你们的title这个字段是在什么时机放进model的?在load里不应该会被覆盖啊

在url中传了title字段,actionContext.args.title就会被actionConfig.title覆盖

哦原来是来自URL的字段,我了解了,那么我去看看怎么修复……

放在3.1.0-dev分支上