aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/lib/active_record/relation/query_methods.rb
Commit message (Collapse)AuthorAgeFilesLines
...
* removing a conditional that is not usedAaron Patterson2010-09-281-2/+0
|
* avoid calling to_sql when we canAaron Patterson2010-09-281-1/+1
|
* make sure we use the engine assigned to the table when quotingAaron Patterson2010-09-271-1/+1
|
* Three performance improvements:Neeraj Singh2010-09-271-10/+9
| | | | | | | | * for simple cases like User.last and User.order('name desc').last no need to perform Array#join operation. * Instead of performing String#blank? do Array#empty? * no need to create variable relation
* adds to the API guidelines the proper spelling of ArelXavier Noria2010-09-191-2/+2
|
* @join_values is passed in, so we should use the parameterAaron Patterson2010-09-151-1/+1
|
* creating a new array is required. who knew? :-(Aaron Patterson2010-09-091-1/+1
|
* no need for Array.wrap, also avoid array creationAaron Patterson2010-09-091-5/+4
|
* make apply_modules run fasterNeeraj Singh2010-09-091-3/+5
|
* eliminate present? for fewer method callsAaron Patterson2010-09-081-10/+2
|
* returning arrays lets us avoid Array.wrapAaron Patterson2010-09-081-4/+4
|
* removing unnecessary conditional testAaron Patterson2010-09-081-4/+4
|
* switch to blank? to avoid method callsAaron Patterson2010-09-081-6/+6
|
* refactor to use faster empty?Aaron Patterson2010-09-081-1/+3
|
* drying up joins()Aaron Patterson2010-09-081-4/+2
|
* reduce method calls, use reject + blank? instead of select + present?Aaron Patterson2010-09-081-3/+3
|
* removing useless code, cleaning variable namesAaron Patterson2010-09-071-6/+2
|
* select should raise error when no block or no parameter is passedAaron Patterson2010-09-071-3/+3
|
* select does not need a *argsAaron Patterson2010-09-071-2/+2
|
* cleaning up confusing logicAaron Patterson2010-09-061-3/+6
|
* Make scoped reorder override previous applied ordersSantiago Pastorino2010-09-051-2/+5
| | | | [5528 state:committed]
* Make all the Relation finder methods consistentPratik Naik2010-08-311-32/+67
|
* Merge remote branch 'miloops/rails_master_fixes'Pratik Naik2010-08-311-15/+26
|\
| * Avoid calling build_where is no argument is given.Emilio Tagua2010-08-311-1/+1
| |
| * Improved conditionals usage to prevent calling methods.Emilio Tagua2010-08-311-8/+19
| |
| * Don't test conditional in iteration if it is not present just clone it.Emilio Tagua2010-08-311-9/+9
| |
* | Remove default values for Relation#limit/offset/from/create_withPratik Naik2010-08-311-4/+4
|/
* use blank? instead of present?, mark SQL literals as SQL literalsAaron Patterson2010-08-251-5/+5
| | | | | | Conflicts: activerecord/lib/active_record/relation/query_methods.rb
* no need to send on a public methodAaron Patterson2010-08-251-1/+1
|
* refactor select { is_a? } to grep()Aaron Patterson2010-08-251-1/+1
|
* sql literal strings should be marked as sql literal stringsAaron Patterson2010-08-231-1/+3
|
* 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
|