aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/lib/active_record/relation
Commit message (Collapse)AuthorAgeFilesLines
...
* 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-052-3/+15
| | | | [5528 state:committed]
* order should always be concatenated.Neeraj Singh2010-09-051-2/+1
| | | | | | | | | | | | | | | | | | | | | | order that is declared first has highest priority in all cases. Here are some examples. Car.order('name desc').find(:first, :order => 'id').name Car.named_scope_with_order.named_scope_with_another_order Car.order('id DESC').scoping do Car.find(:first, :order => 'id asc') end No special treatment to with_scope or scoping. Also note that if default_scope declares an order then the order declared in default_scope has the highest priority unless with_exclusive_scope is used. Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* Use new finders syntax in docs.Emilio Tagua2010-09-012-18/+18
|
* 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
| |
* | Change relation merging to always append select, group and order valuesPratik Naik2010-08-311-7/+10
|/
* PERF: Avoid two method callsSantiago Pastorino2010-08-281-1/+1
|
* Merge remote branch 'docrails/master'Xavier Noria2010-08-261-8/+6
|\
| * minor formatting fixesNeeraj Singh2010-08-251-8/+6
| |
* | 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
|
* refactoring to remove duplicate logicAaron Patterson2010-08-181-4/+3
|
* call to present? is not necessaryAaron Patterson2010-08-181-2/+1
|
* Deletes trailing whitespaces (over text files only find * -type f -exec sed ↵Santiago Pastorino2010-08-143-34/+34
| | | | 's/[ \t]*$//' -i {} \;)
* Removing most of the symbol to proc usage in Active RecordPrem Sichanugrist2010-08-131-1/+1
| | | | This will hopefully make Active Record run a bit more faster.
* avoid direct use of arel constantsAaron Patterson2010-08-131-2/+4
|
* Ensure we can nest include calls [#5285 state:resolved]Neeraj Singh2010-08-122-2/+8
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* subtracting blank strings is slightly faster than blank?Aaron Patterson2010-08-111-3/+1
|
* avoid multiple hash lookupsAaron Patterson2010-08-111-2/+6
|
* 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
|
* stop using private methodsAaron Patterson2010-08-111-2/+2
|
* adds missing requires for Object#tryXavier Noria2010-08-091-0/+1
|
* avoid passing lists of lists to the group clauseAaron Patterson2010-08-041-1/+1
|
* call to_a since we are not passing anything to all()Aaron Patterson2010-08-041-1/+1
|
* fisting indentationAaron Patterson2010-08-041-3/+3
|
* do not pass AR objects to ARelAaron Patterson2010-08-041-0/+2
|
* avoid passing AR::Base objects to Arel when we canAaron Patterson2010-08-031-0/+2
|
* order query is already a string, no need to to_sAaron Patterson2010-08-031-2/+2
|
* ensuring that description does not exceed 100 columnsNeeraj Singh2010-08-022-36/+66
|
* 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
|
* Fix the #using_limitable_reflections? helper to work correctly by not ↵Ken Collins2010-07-081-1/+1
| | | | | | examining the length of an array which contains false/true, hence always passing. [#4869 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>