aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/lib/active_record/relation/query_methods.rb
Commit message (Collapse)AuthorAgeFilesLines
* we should wrap strings as sql literalsAaron Patterson2010-08-191-1/+1
|
* Ensure we can nest include calls [#5285 state:resolved]Neeraj Singh2010-08-121-1/+1
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* subtracting blank strings is slightly faster than blank?Aaron Patterson2010-08-111-3/+1
|
* avoiding tap saves us timeAaron Patterson2010-08-111-2/+6
|
* unless Array#empty? is faster than if Array#present?Aaron Patterson2010-08-111-1/+1
|
* avoid passing lists of lists to the group clauseAaron Patterson2010-08-041-1/+1
|
* order query is already a string, no need to to_sAaron Patterson2010-08-031-2/+2
|
* reverting where() functionality, adding a testAaron Patterson2010-07-301-4/+4
|
* fewer method calls, using faster methods when possibleAaron Patterson2010-07-301-5/+5
|
* reducing function calls and using faster methods for testingAaron Patterson2010-07-301-9/+3
|
* got the ternary backwards. somehow the sqlite tests passed!Aaron Patterson2010-07-291-1/+1
|
* PERF: avoiding splat args and reducing function callsAaron Patterson2010-07-291-8/+5
|
* Shouldn't have committed this! Revert "avoid calling to_sql twice"Aaron Patterson2010-07-281-7/+4
| | | | This reverts commit 1c030a3c3c61d6d6262785bf67e1d8f44da87ea5.
* avoid calling to_sql twiceAaron Patterson2010-07-281-4/+7
|
* Revert "Style fixes"Pratik Naik2010-07-151-20/+20
| | | | This reverts commit 3c300b31219f2af3ecd46ef22b04e5c5548db899.
* Style fixeswycats2010-07-141-20/+20
|
* Style fixesPratik Naik2010-07-141-20/+20
|
* Add scoping and unscoped as the syntax to replace the old with_scope and ↵José Valim2010-06-291-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | with_exclusive_scope. A few examples: * with_scope now should be scoping: Before: Comment.with_scope(:find => { :conditions => { :post_id => 1 } }) do Comment.first #=> SELECT * FROM comments WHERE post_id = 1 end After: Comment.where(:post_id => 1).scoping do Comment.first #=> SELECT * FROM comments WHERE post_id = 1 end * with_exclusive_scope now should be unscoped: class Post < ActiveRecord::Base default_scope :published => true end Post.all #=> SELECT * FROM posts WHERE published = true Before: Post.with_exclusive_scope do Post.all #=> SELECT * FROM posts end After: Post.unscoped do Post.all #=> SELECT * FROM posts end Notice you can also use unscoped without a block and it will return an anonymous scope with default_scope values: Post.unscoped.all #=> SELECT * FROM posts
* Remove unneeded rejectSantiago Pastorino2010-06-281-8/+1
|
* Refactor: metaprogramming here it's confusing and make use of tapSantiago Pastorino2010-06-261-64/+73
| | | | Signed-off-by: Jeremy Kemper <jeremy@bitsweat.net>
* Removes useless flattenSantiago Pastorino2010-06-261-4/+9
|
* Create a little bit less objects in ARel.José Valim2010-06-261-6/+9
|
* blocks removed from all the ActiveRelation query_methods, extend method ↵Santiago Pastorino2010-06-261-8/+4
| | | | | | added instead Signed-off-by: José Valim <jose.valim@gmail.com>
* reorder method added to ActiveRelationSantiago Pastorino2010-06-261-0/+8
| | | | | | [#4972 state:committed] Signed-off-by: José Valim <jose.valim@gmail.com>
* Support for multiple selects addedNeeraj Singh and Santiago Pastorino2010-06-251-4/+7
| | | | [#4841 state:committed]
* Makes a build_select for the select part of build_arelSantiago Pastorino2010-06-251-10/+13
|
* Metaprogramming not needed hereSantiago Pastorino2010-06-251-11/+9
|
* Only yield block if given.Emilio Tagua2010-06-241-2/+2
|
* Fix order method to accept relation attributes.Emilio Tagua2010-06-221-1/+1
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Don't waste time building relations if there are no values presents. [#4860 ↵Emilio Tagua2010-06-211-4/+4
| | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* Refactor build_arel: move joins out and simplify havings. [#4860 state:resolved]Emilio Tagua2010-06-201-46/+46
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Special treatement for Relation#select { with block }Pratik Naik2010-06-021-1/+14
|
* To allow proper fisting of stack traceSantiago Pastorino2010-05-141-3/+3
| | | | Signed-off-by: Jeremy Kemper <jeremy@bitsweat.net>
* Fix unintuitive behavior with multiple order and group clausesErnie Miller2010-05-061-6/+2
| | | | | | [#4545 state:committed] Signed-off-by: Jeremy Kemper <jeremy@bitsweat.net>
* Fix eager loading of associations causing table name collisionsErnie Miller2010-04-281-31/+42
| | | | | | [#4463 state:committed] Signed-off-by: Jeremy Kemper <jeremy@bitsweat.net>
* Build PredicateBuilder object only when neededPratik Naik2010-04-211-3/+1
|
* Fix evals missing contextJeremy Kemper2010-04-101-3/+3
|
* Avoid deprecated String#to_a by using Array.wrap(...) instead of Array(...)Jeremy Kemper2010-04-101-1/+2
|
* Add Relation extensionsPratik Naik2010-04-021-4/+21
|
* adds missing requires for Object#blank? and Object#present?Xavier Noria2010-03-281-0/+2
|
* From and lock should be defined to be consistent with other ivars. Limit and ↵Emilio Tagua2010-03-221-14/+4
| | | | | | | | offset are always defined, no need to test that. [#4253 state:committed] Signed-off-by: Jeremy Kemper <jeremy@bitsweat.net>
* kill where / having method redefine warnings. [#4225 state:resolved]Aaron Patterson2010-03-181-0/+1
| | | | Signed-off-by: wycats <wycats@gmail.com>
* removing spawn from SpawnMethodsAaron Patterson2010-03-101-6/+6
| | | | Signed-off-by: wycats <wycats@gmail.com>
* refactoring build_whereAaron Patterson2010-03-061-7/+7
| | | | Signed-off-by: wycats <wycats@gmail.com>
* avoid @lock_value not initialized warningSantiago Pastorino2010-02-221-1/+1
|
* Clear up some ivar warningsJeremy Kemper2010-01-271-3/+13
|
* Simplify finder method definitionsPratik Naik2010-01-211-13/+11
|
* Supplying Arel::SqlLiteral is much fasterPratik Naik2010-01-211-5/+12
|
* Use quoted_table_name with arel.from() if no from values explicitly ↵Pratik Naik2010-01-211-2/+4
| | | | supplied. Arel seems to be spending a lot of time figuring out the FROM value otherwise.
* Always use table.* in the finder query unless specifiedPratik Naik2010-01-211-1/+1
|