aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/lib/active_record/relation
Commit message (Collapse)AuthorAgeFilesLines
...
* reorder removed in favor of except(:order).orderSantiago Pastorino2010-10-112-20/+2
|
* avoid creating objects when we canAaron Patterson2010-10-032-12/+6
|
* build_where should be privateAaron Patterson2010-09-301-2/+2
|
* [#5441 state:resolved] refactoring code to determine aggregate columnAaron Patterson2010-09-301-11/+14
|
* removing more useless code! yay!Aaron Patterson2010-09-281-6/+2
|
* SqlLiteral is a string, so we can dry up these conditionalsAaron Patterson2010-09-281-4/+3
|
* shorten up or sql literal creation statementsAaron Patterson2010-09-281-4/+2
|
* we only care about arrays and stringsAaron Patterson2010-09-281-1/+1
|
* dry up calls to arel.join()Aaron Patterson2010-09-281-5/+4
|
* removing unused lasgnsAaron Patterson2010-09-281-4/+4
|
* refactoring to remove crazy logicAaron Patterson2010-09-282-9/+6
|
* performance improvement based on discussion at ↵Neeraj Singh2010-09-281-9/+3
| | | | | | http://github.com/rails/rails/commit/fbd1d306b95cc2efb6422e12d26d5818a3a42343 Credit goes to all the participants in the discussion
* removing a conditional that is not usedAaron Patterson2010-09-281-2/+0
|
* fisting the postgresql testsAaron Patterson2010-09-281-1/+2
|
* avoid calling to_sql when we canAaron Patterson2010-09-282-4/+4
|
* removing an inject + merge in favor of Hash#[]Aaron 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
* Goodbye inject, hello map.Emilio Tagua2010-09-221-7/+5
| | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* un-nodocs AR::BatchesXavier Noria2010-09-211-1/+1
|
* 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
|
* passing the quoted id to arel if the object has a quoted idAaron Patterson2010-09-101-1/+3
|
* converting an inject to a map + Hash[]Aaron Patterson2010-09-101-1/+1
|
* avoid method_missing and reduce method callsAaron Patterson2010-09-101-1/+1
|
* intersection between a relation and an array works in both directionsMarcelo Giorgi2010-09-101-0/+1
| | | | Signed-off-by: Mikel Lindsaar <raasdnil@gmail.com>
* Refactor finder conditions look up and assigment for apply_finder_conditions.Emilio Tagua2010-09-091-13/+7
|
* Refactor Relation#only. No need to go through every option and test it.Emilio Tagua2010-09-091-8/+6
|
* No need to go through every option and test it, just don't add skipped ones.Emilio Tagua2010-09-091-4/+4
|
* return is not needed hereNeeraj Singh2010-09-091-2/+2
|
* remove unnecessary callNeeraj Singh2010-09-091-1/+0
| | | | if operation is count then column_name will never be blank
* 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
|
* attributes should be constructed with table objectsAaron Patterson2010-09-071-1/+1
|
* attributes should be associated with tablesAaron Patterson2010-09-071-1/+1
|
* 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-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
|