Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | assuming there is only one column, we can simplify the type cast loop | Aaron Patterson | 2012-05-16 | 1 | -7/+6 | |
| | ||||||
* | MySQL returns "SUM(DISTINCT(credit_limit))" as the column name unless | Aaron Patterson | 2012-05-16 | 1 | -4/+11 | |
| | | | | | an alias is provided. Without the alias, the column cannot be found and properly typecast. | |||||
* | Fixes issue where SQL fragments prevented type casting based on column type. | Erich Menge | 2012-05-16 | 1 | -8/+3 | |
| | ||||||
* | Expand and improve #pluck docs | Jeremy Kemper | 2012-05-15 | 1 | -6/+27 | |
| | ||||||
* | Restore support for Model.pluck('sql fragment') | Jeremy Kemper | 2012-05-15 | 1 | -1/+2 | |
| | ||||||
* | Lowercase. | Ben Pickles | 2012-05-01 | 1 | -1/+1 | |
| | ||||||
* | Add ActiveRecord::Base#ids | twinturbo | 2012-04-30 | 1 | -0/+10 | |
| | ||||||
* | extract deprecated #calculate code | Jon Leighton | 2012-04-13 | 1 | -83/+25 | |
| | ||||||
* | use a hash to store relation values | Jon Leighton | 2012-04-13 | 1 | -5/+5 | |
| | ||||||
* | Missing generated query for pluck method. | Juanjo Bazán | 2012-03-27 | 1 | -1/+1 | |
| | | | (and changed the use of true for a more database agnostic example). | |||||
* | Merge pull request #4984 from raghunadhd/change_orderhash | Vijay Dev | 2012-03-03 | 1 | -1/+1 | |
|\ | | | | | Refactored code | |||||
| * | Refactored code | Raghunadh | 2012-02-10 | 1 | -1/+1 | |
| | | ||||||
* | | use bind values for join columns | Aaron Patterson | 2012-02-27 | 1 | -3/+4 | |
| | | ||||||
* | | typecast columns based on the returned types | Aaron Patterson | 2012-02-09 | 1 | -2/+14 | |
|/ | ||||||
* | AR::Relation#pluck: improve to work with joins | Bogdan Gusiev | 2012-02-08 | 1 | -0/+3 | |
| | ||||||
* | rewrites a couple of alternations in regexps as character classes | Xavier Noria | 2012-01-30 | 1 | -1/+1 | |
| | | | | | Character classes are the specific regexp construct to express alternation of individual characters. | |||||
* | Delete unnecessary line that was necessary before I changed the code and it ↵ | Jon Leighton | 2011-12-22 | 1 | -1/+0 | |
| | | | | became no longer necessary. | |||||
* | Make ActiveRecord::Relation#pluck work with serialized attributes | Jon Leighton | 2011-12-22 | 1 | -4/+4 | |
| | ||||||
* | ActiveRecord::Relation#pluck method | Bogdan Gusiev | 2011-11-30 | 1 | -0/+17 | |
| | ||||||
* | Make it the responsibility of the connection to hold onto an ARel visitor ↵ | Jon Leighton | 2011-08-08 | 1 | -2/+2 | |
| | | | | for generating SQL. This improves the code architecture generally, and solves some problems with marshalling. Adapter authors please take note: you now need to define an Adapter.visitor_for method, but it degrades gracefully with a deprecation warning for now. | |||||
* | Fixed failing query when performing calculation with having based on select. | Dmitriy Kiriyenko | 2011-07-27 | 1 | -0/+1 | |
| | ||||||
* | Fixed AR::Relation#sum compatibility with Array#sum | Bogdan Gusiev | 2011-07-05 | 1 | -2/+6 | |
| | | | | | In order make Relation behavior closer to Array Made Relation#sum to accept block and delegate it to Array#sum | |||||
* | Apply the default scope earlier when doing calculations. Fixes #1682. | Jon Leighton | 2011-06-23 | 1 | -5/+11 | |
| | ||||||
* | Ensure that calculations properly override the select value even if it's set ↵ | Jon Leighton | 2011-05-31 | 1 | -2/+2 | |
| | | | | in the default scope. Fixes #1395. | |||||
* | Remove extra white spaces on ActiveRecord docs. | Sebastian Martinez | 2011-05-23 | 1 | -4/+4 | |
| | ||||||
* | Bug fixes: | Fadzril Muhamad & Joseph Palermo | 2011-05-12 | 1 | -5/+4 | |
| | | | | | | - If doing a count on a relation that has an :include and a :join, it does a distinct even though it should not. - When doing a count on a relation that has an :include, it always falls back to a old style left join when performing the count. Looks like it was broken here: https://github.com/rails/rails/commit/b9599502c9e738a5a1513e75d08f8d40ed408265 | |||||
* | Ensure that ORDER BY clauses from default scopes are not included in ↵ | Jon Leighton | 2011-05-08 | 1 | -1/+1 | |
| | | | | calculation queries. Fixes #406. | |||||
* | Use Arel to build subquery. Adapt tests to changed fixtures. | John Mileham | 2011-03-24 | 1 | -13/+12 | |
| | ||||||
* | Change behavior of count(:limit => x, :offset => y) to limit/offset before ↵ | John Mileham | 2011-03-03 | 1 | -14/+26 | |
| | | | | counting. | |||||
* | removing limits and offsets from COUNT queries unless both are specified. ↵ | Aaron Patterson | 2011-02-25 | 1 | -1/+13 | |
| | | | | [#6268 state:resolved] | |||||
* | The type_cast_calculated_value method will trust DB types before casting to ↵ | Ken Collins | 2011-02-03 | 1 | -1/+1 | |
| | | | | | | | | a BigDecimal. [#6365 state:committed] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | |||||
* | Always return decimal average of integer fields | Raimonds Simanovskis | 2011-01-10 | 1 | -9/+5 | |
| | | | | In previous version if database adapter (e.g. SQLite and Oracle) returned non-String calculated values then type_cast_using_column converted decimal average value of intefer field to integer value. Now operation parameter is always checked to decide which conversion of calculated value should be done. | |||||
* | ActiveRecord::Relation#primary_key should return a string, just like ↵ | Jon Leighton | 2011-01-03 | 1 | -1/+1 | |
| | | | | ActiveRecord::Base.primary_key does. | |||||
* | Rename AssociationReflection#primary_key_name to foreign_key, since the ↵ | Jon Leighton | 2010-12-31 | 1 | -1/+1 | |
| | | | | options key which it relates to is :foreign_key | |||||
* | just grep the AST for OuterJoin nodes rather than converting the tree to SQL | Aaron Patterson | 2010-11-30 | 1 | -1/+1 | |
| | ||||||
* | reduce method calls on arel | Aaron Patterson | 2010-11-30 | 1 | -1/+1 | |
| | ||||||
* | refactor to reduce method calls | Aaron Patterson | 2010-11-16 | 1 | -10/+19 | |
| | ||||||
* | Fix ActiveRecord calculations when grouped by multiple fields | Alexandru Catighera | 2010-11-16 | 1 | -14/+18 | |
| | ||||||
* | we should always cast the value based on the column | Aaron Patterson | 2010-10-11 | 1 | -1/+1 | |
| | ||||||
* | Honor distinct option when used with count operation after group clause ↵ | Marcelo Giorgi | 2010-10-11 | 1 | -4/+8 | |
| | | | | [#5721 state:resolved] | |||||
* | [#5441 state:resolved] refactoring code to determine aggregate column | Aaron Patterson | 2010-09-30 | 1 | -11/+14 | |
| | ||||||
* | refactoring to remove crazy logic | Aaron Patterson | 2010-09-28 | 1 | -1/+5 | |
| | ||||||
* | fisting the postgresql tests | Aaron Patterson | 2010-09-28 | 1 | -1/+2 | |
| | ||||||
* | avoid calling to_sql when we can | Aaron Patterson | 2010-09-28 | 1 | -3/+3 | |
| | ||||||
* | removing an inject + merge in favor of Hash#[] | Aaron Patterson | 2010-09-28 | 1 | -1/+1 | |
| | ||||||
* | Goodbye inject, hello map. | Emilio Tagua | 2010-09-22 | 1 | -7/+5 | |
| | | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | |||||
* | return is not needed here | Neeraj Singh | 2010-09-09 | 1 | -2/+2 | |
| | ||||||
* | remove unnecessary call | Neeraj Singh | 2010-09-09 | 1 | -1/+0 | |
| | | | | if operation is count then column_name will never be blank | |||||
* | attributes should be constructed with table objects | Aaron Patterson | 2010-09-07 | 1 | -1/+1 | |
| | ||||||
* | attributes should be associated with tables | Aaron Patterson | 2010-09-07 | 1 | -1/+1 | |
| |