Edit other fields in profiles page bug
Summary
具体是profile页面的,修改除了手机号之外别的字段的时候提交,会检查有没有验证码字段而报错。 实际因为没修改手机号,所以不需要检查验证码。
这个在重构之前检查了的,重构的时候有一行代码换了位置导致的
Steps to reproduce
在profiles页面修改除了手机号外的任何字段提交。会复现,如下图:
Possible fixes
这个在重构之前检查了的,重构的时候有一行代码换了位置导致的。所以需要将这行代码
return if params[:user][:phone] == current_user.phone
从jh/app/controllers/jh/profiles_controller.rb
移动到
jh/app/controllers/concerns/check_phone_and_code.rb
因为在此操作之前要执行 merge_area_code_and_phone_with_user
方法来合并区号和手机号参数,再检查手机号是否改变
我创建了 !208 (merged) 来修复此bug,除此之外 !206 (closed) 为为registrations 和 profiles 页面添加的测试的代码。 @prajnamas @icbd CC @qianzhangxa
由 Qian Zhang (Gary) 编辑于