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

Deprecated code warnings #9

Open
zzet opened this issue Nov 20, 2013 · 4 comments
Open

Deprecated code warnings #9

zzet opened this issue Nov 20, 2013 · 4 comments

Comments

@zzet
Copy link
Member

zzet commented Nov 20, 2013

DEPRECATION WARNING: Model.scoped is deprecated. Please use Model.all instead. (called from block (2 levels) in <module:UsefullScopes> at /var/lib/jenkins/.rvm/gems/ruby-2.0.0-p247/bundler/gems/usefull_scopes-016c3807f615/lib/usefull_scopes.rb:13)
@melekes
Copy link
Contributor

melekes commented Jul 3, 2014

Should we use where(nil) or all? Choosing where(nil) will allow us to keep compatibility with Rails 3.

@zzet what do you think?

@zzet
Copy link
Member Author

zzet commented Jul 3, 2014

Отказаться от поддержки третьих рельс - или ставить костыль с if Rails.version boa bla bla ? Если ставить вопрос так - то я бы пока ориентировался на третьи рельсы, так как под четвертые - нужно сильно освежать код и лучше это делать в следующей версии.

@numbata
Copy link
Contributor

numbata commented Aug 28, 2014

@zzet @akalyaev Можно запросто отказаться от поддержки третьих рельс выпустив новую мажорную версию. Оставив примечание: для рельс3 gem 'usefull_scopes', '~> rails3.version', для рельс4+ gem 'usefull_scopes'. так, в общем-то, многие поступают :)

@zzet
Copy link
Member Author

zzet commented Aug 28, 2014

@numbata вообще уже можно, да. Только, как я уже говорил, для четвертых, по-хорошему, надо переколбасить код :) Там многое можно сделать "подругому", да и arel заметно обновился.

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

3 participants