search

lanrion / weixin_rails_middleware

微信集成 ruby weixin_rails_middleware for integration weixin.
MIT License
408 stars 107 forks source link

Update weixin_controller.rb #56

Closed mzdz closed 7 years ago

mzdz commented 7 years ago

rails 5.1 去掉了 before_filter, 所以把 before_filter 改为 before_action

fixed #51

lanrion commented 7 years ago

这样会导致不兼容Rails 3。

greatghoul commented 7 years ago

能不能判断下版本，根据不同的版本使用不同的方法。

lanrion commented 7 years ago

是的，Rails.version.

mzdz commented 7 years ago

能不能区分版本呢, 比如 rails < 5.0 就用 此 gem 的 1.0 版本 , rails > 5.1 就用 此 gem 的 2.0 版本?

lanrion commented 7 years ago

@mzdz 还没到这种情况吧，可以在代码中判断使用哪个before_*方法来处理。

windy commented 7 years ago

同样的问题, 建议是可以大胆使用 before_action, 然后在 readme 中为 3.0 专门固定一个版本说明就可以了. 很多 gem 都这样用.

lanrion commented 7 years ago

看来去除 before_filter大势所趋。

lanrion commented 7 years ago

@mzdz 麻烦更新一下commit信息：deprecated before_filter 。 合并之后，我再单独发一个版本。