aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/lib/active_record/relation
Commit message (Collapse)AuthorAgeFilesLines
...
* TypoOscar Del Ben2012-07-161-1/+0
|
* Add documentation for query_methods bang methodsOscar Del Ben2012-07-161-1/+20
|
* Add nodoc to create_with_valueOscar Del Ben2012-07-161-1/+1
| | | | | Reason: all *_value methods are defined dynamically and so don't appear in the documentation.
* Add documentation for arel and build_arelOscar Del Ben2012-07-161-0/+2
|
* Add documentation for create_withOscar Del Ben2012-07-161-1/+19
|
* Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-07-151-2/+39
|\
| * Add docs for having, lock and readonlyOscar Del Ben2012-07-071-0/+12
| |
| * Add doc for joins and improve includes docOscar Del Ben2012-07-071-2/+17
| |
| * Add docs for eager_laod and preloadOscar Del Ben2012-07-071-0/+10
| |
* | fix association :extend optionJon Leighton2012-07-131-1/+1
| |
* | support relations created with a table aliasJon Leighton2012-07-131-1/+1
|/
* minor text change [ci skip]Vijay Dev2012-07-071-1/+1
|
* Revert "Add nodoc to relation methods"Vijay Dev2012-07-071-22/+24
| | | | | | This reverts commit c47a698d5d497340d4e349257522212173865838. Reason: Let's revert pending further discussions
* Add nodoc to relation methodsOscar Del Ben2012-07-061-24/+22
|
* Add order docsOscar Del Ben2012-07-051-0/+10
|
* Add group documentationOscar Del Ben2012-07-051-0/+12
|
* Add documentation for includesOscar Del Ben2012-07-051-0/+11
|
* Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-06-301-5/+7
|\ | | | | | | | | Conflicts: activemodel/lib/active_model/errors.rb
| * update AR::FinderMethods#exists?(false) example [ci skip]Francesco Rodriguez2012-06-281-5/+7
| |
* | fixing typo in from documentationSubba Rao Pasupuleti2012-06-281-2/+2
| |
* | Use args.flatten! in query methods when applicableCarlos Antonio da Silva2012-06-251-7/+12
| | | | | | | | | | Try to use more destructive methods on *args when applicable, to avoid creating new objects.
* | Merge pull request #6544 from flexoid/issue-6431Piotr Sarnacki2012-06-251-6/+6
|\ \ | | | | | | exists?(false) returns false
| * | exists?(false) returns falseEgor Lynko2012-06-251-6/+6
| | | | | | | | | | | | `FinderMethods#exists?` finder method now returns *false* with the *false* argument
* | | Ensure Arel columns are typecasted properly when grouping with calculationCarlos Antonio da Silva2012-06-251-6/+6
|/ / | | | | | | Fix build issue with postgresql.
* / Stop assuming strings for grouped calculationsErnie Miller2012-06-241-6/+17
|/ | | | | | | | | | | | | | | Execute_grouped_calculation is one of those places where ActiveRecord forgets that it has ARel underpinnings, and assumes that the values provided to group_values are strings. This artificially hobbles otherwise functional code. This patch stops assuming that incoming values respond to to_sym, stops using string interpolation for table aliases on objects that support aliasing, and stops unnecessarily joining group_values on the relation. Additionally, it calls to_sql, if available, on objects sent to column_alias_for, in order to get a more reasonable alias string than a non-string's default to_str method.
* Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-06-221-3/+15
|\
| * fixes a few mistakes in api docs [ci skip]Vijay Dev2012-06-221-4/+2
| |
| * Add documentation for ActiveRecord::QueryMethods#reverse_order [ci skip]Florent Guilleux2012-06-161-0/+3
| |
| * Add documentation for ActiveRecord::QueryMethods#offset [ci skip]Ulises Cabanillas2012-06-161-0/+7
| |
| * Add documentation for ActiveRecord::QueryMethods#limit [ci skip]Florent Guilleux2012-06-161-3/+7
| |
* | Add changelog entry and guide updates for pluck with multiple columnsCarlos Antonio da Silva2012-06-221-0/+1
| |
* | Refactor pluck with multiple columnsCarlos Antonio da Silva2012-06-221-26/+13
| | | | | | | | | | | | | | | | | | | | | | | | Ensure it works with mix of symbols and strings, and with a select clause possibly containing more than one column. Also remove support for pluck with an array of columns, in favor of passing the list of attributes: Model.pluck(:a, :b) See comments: https://github.com/rails/rails/pull/6500#issuecomment-6030292
* | ActiveRecord#pluck now accepts multiple columnsjeroeningen2012-06-221-15/+35
| |
* | disallow mutating a relation once loadedJon Leighton2012-06-221-18/+20
| |
* | Extract conditional to a method to avoid duplicationRafael Mendonça França2012-06-191-15/+19
| | | | | | | | Also use if/else block to not use short circuit return
* | handle joins/includes correctly for pluck and calculation.Andrey Deryabin2012-06-191-1/+6
| | | | | | | | Fix #5990
* | Removing composed_of from ActiveRecord.Steve Klabnik2012-06-181-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This feature adds a lot of complication to ActiveRecord for dubious value. Let's talk about what it does currently: class Customer < ActiveRecord::Base composed_of :balance, :class_name => "Money", :mapping => %w(balance amount) end Instead, you can do something like this: def balance @balance ||= Money.new(value, currency) end def balance=(balance) self[:value] = balance.value self[:currency] = balance.currency @balance = balance end Since that's fairly easy code to write, and doesn't need anything extra from the framework, if you use composed_of today, you'll have to add accessors/mutators like that. Closes #1436 Closes #2084 Closes #3807
* | Documentation for where and where! methods on relations.David Paschich2012-06-181-0/+92
|/ | | | Based on examples seen in the Rails test suite. [ci skip]
* Address ORA-00911 errors because of the heading underscore.Yasuo Honda2012-06-111-1/+1
|
* Fix that #exists? raises ThrowResult when called with an empty limitedBen Woosley2012-06-101-0/+2
| | | | | | | | | reflection. ActiveRecord::FinderMethods#construct_limited_ids_condition will raise ThrowResult if the limited reflection comes back empty. The other callers of #construct_limited_ids_condition handle this exception (more specifically, the callers of construct_relation_for*), but #exists? didn't until now.
* Add test to column alias in `exists?` SQL.Rafael Mendonça França2012-06-101-1/+1
| | | | | | | | This behavior was added in be4ecdcc87984e9421ff5d5c90d33f475e0fbc01. Closes #1139. Fixes #2553, #1141, #1623 and #2062.
* Minimal change to query generation of exists? that makes SQLServer and ↵iaddict2012-06-101-1/+1
| | | | | | | others happy that do not work without a column alias. Conflicts: activerecord/lib/active_record/relation/finder_methods.rb
* Fix #6635. We should call Scoping methods, before calling Array methods.kennyj2012-06-101-5/+5
|
* Remove conditional committed by accidentSantiago Pastorino2012-06-091-1/+1
|
* Use each_with_object instead of each hereSantiago Pastorino2012-06-091-1/+1
|
* modulize AR::NullRelationAkira Matsuda2012-06-011-1/+1
| | | | now we can invoke previously added scope extension methods
* Merge branch 'master-sec'Aaron Patterson2012-05-311-1/+1
|\ | | | | | | | | | | * master-sec: Strip [nil] from parameters hash. Thanks to Ben Murphy for reporting this! predicate builder should not recurse for determining where columns. Thanks to Ben Murphy for reporting this
| * predicate builder should not recurse for determining where columns.Aaron Patterson2012-05-301-1/+1
| | | | | | | | | | | | Thanks to Ben Murphy for reporting this CVE-2012-2661
* | Fix wrong example for count method with block [ci skip]Carlos Antonio da Silva2012-05-311-1/+1
| | | | | | | | Thanks @splattael.
* | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-05-303-22/+32
|\ \