- 10 1月, 2014 1 次提交
-
-
由 Yves Senn 提交于
Refs #13658
-
- 04 12月, 2013 1 次提交
-
-
由 Lauro Caetano 提交于
Closes #7441
-
- 15 11月, 2013 1 次提交
-
-
由 Yves Senn 提交于
-
- 21 10月, 2013 1 次提交
-
-
由 Shimpei Makimoto 提交于
-
- 16 10月, 2013 1 次提交
-
-
由 Rafael Mendonça França 提交于
Conflicts: activerecord/lib/active_record/relation/finder_methods.rb
-
- 15 10月, 2013 2 次提交
-
-
由 Aaron Patterson 提交于
-
由 Aaron Patterson 提交于
-
- 14 10月, 2013 5 次提交
-
-
由 Aaron Patterson 提交于
-
由 Aaron Patterson 提交于
I don't really like passing the block, but this seems easiest for now
-
由 Aaron Patterson 提交于
-
由 Aaron Patterson 提交于
-
由 Aaron Patterson 提交于
-
- 11 10月, 2013 1 次提交
-
-
由 Aaron Patterson 提交于
-
- 08 10月, 2013 1 次提交
-
-
由 Edo Balvers 提交于
-
- 19 8月, 2013 1 次提交
-
-
由 Xavier Noria 提交于
This fixes a regression. The documentation said in its introduction paragraph that the method returns truthy/falsy, but then below it was said that if there were no arguments you'd get `true` or `false`. Also when the argument is exactly `false` a singleton is documented to be returned. The method was not returning the singletons so it didn't conform to those special cases. The best solution here seems to be to just return singletons in all cases. This solution is backwards compatible. Also, the contract has been revised because it has no sense that the predicate varies that way depending on the input. I bet the previous contract was just an accident, not something mixed on purpose. Conflicts: activerecord/lib/active_record/relation/finder_methods.rb activerecord/test/cases/finder_test.rb
-
- 10 7月, 2013 2 次提交
-
-
由 Aaron Patterson 提交于
-
由 Aaron Patterson 提交于
-
- 06 7月, 2013 1 次提交
-
-
由 Vipul A M 提交于
Perf ref: https://gist.github.com/vipulnsward/8209749201dfdd678c97
-
- 28 6月, 2013 2 次提交
-
-
由 Carlos Antonio da Silva 提交于
In 94924dc3 the internal default_scope implementation has changed making it simpler to follow, meaning that the old usage of with_default_scope has been removed. With that, order_values was the same argument for both calls to find_first_with_limit, so remove it and use the existent attribute for the sake of clarity/simplification.
-
由 Jon Leighton 提交于
The previous implementation was necessary in order to support stuff like: class Post < ActiveRecord::Base default_scope where(published: true) scope :ordered, order("created_at") end If we didn't evaluate the default scope at the last possible moment before sending the SQL to the database, it would become impossible to do: Post.unscoped.ordered This is because the default scope would already be bound up in the "ordered" scope, and therefore wouldn't be removed by the "Post.unscoped" part. In 4.0, we have deprecated all "eager" forms of scopes. So now you must write: class Post < ActiveRecord::Base default_scope { where(published: true) } scope :ordered, -> { order("created_at") } end This prevents the default scope getting bound up inside the "ordered" scope, which means we can now have a simpler/better/more natural implementation of default scoping. A knock on effect is that some things that didn't work properly now do. For example it was previously impossible to use #except to remove a part of the default scope, since the default scope was evaluated after the call to #except.
-
- 14 6月, 2013 1 次提交
-
-
由 Vijay Dev 提交于
-
- 13 6月, 2013 1 次提交
-
-
由 Aaron Patterson 提交于
-
- 10 6月, 2013 2 次提交
-
-
由 Dmitry Polushkin 提交于
-
由 Dmitry Polushkin 提交于
-
- 09 6月, 2013 3 次提交
-
-
由 Thiago Pinto 提交于
-
由 Thiago Pinto 提交于
-
由 Thiago Pinto 提交于
-
- 11 5月, 2013 1 次提交
-
-
由 Ben Woosley 提交于
Rather than raising ThrowResult when construct_limited_ids_conditions comes up empty, set the relation to NullRelation and rely on its results. This will help avoid errors like 2fcafee2, because in most cases NullRelation will do the right thing. Minor bonus is avoiding the use of exceptions for flow control.
-
- 10 5月, 2013 8 次提交
-
-
由 Ben Woosley 提交于
-
由 Ben Woosley 提交于
In #apply_join_dependency, we can apply the #where in-place because relation is always a new object. Thanks to the #except we call at the top of the method.
-
由 Ben Woosley 提交于
-
由 Ben Woosley 提交于
-
由 Ben Woosley 提交于
-
由 Ben Woosley 提交于
Move the except(:select) inside the construct_limited_ids_condition method to pair it closely with its motivation.
-
由 Ben Woosley 提交于
Reject blank order_values within #columns_for_distinct, as the orders aren't used at all on non-postgres adapters.
-
由 Ben Woosley 提交于
The combination of a :uniq => true association and the #distinct call in #construct_limited_ids_condition combine to create invalid SQL, because we're explicitly selecting DISTINCT, and also sending #distinct on to AREL, via the relation#distinct_value. Rather than build a select distinct clause in #construct_limited_ids_condition, I set #distinct! and pass just the columns into the select statement. This requires introducing a #columns_for_distinct method to return the select columns but not the statement itself.
-
- 03 4月, 2013 1 次提交
-
-
由 Steve Klabnik 提交于
-
- 02 4月, 2013 1 次提交
-
-
由 wangjohn 提交于
inverse_of option. I've also refactored the code for raising a RecordNotFound exception when searching for records with ids.
-
- 27 3月, 2013 1 次提交
-
-
由 Prathamesh Sonpatki 提交于
-
- 18 3月, 2013 1 次提交
-
-
由 Carson McDonald 提交于
[ci skip]
-