aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/lib
Commit message (Collapse)AuthorAgeFilesLines
...
* | string IDs are now quoted correctly [#5064 state:resolved]Will St. Clair + Neeraj Singh2010-07-131-1/+1
| | | | | | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* | enable AS::JSON.encode to encode AR::Relation by providing `as_json` methodMislav Marohnić2010-07-131-1/+3
| | | | | | | | | | | | [#5073 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* | Tidying up a bit, so update_attribute is not called twice on touch.José Valim2010-07-132-25/+17
| |
* | with this fix touch method - does not call validations - doest not call ↵Neeraj Singh2010-07-132-9/+8
| | | | | | | | | | | | | | | | callbacks - updates updated_at/on along with attribute if attribute is provided - marks udpated_at/on and attribute as NOT changed [#2520 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* | our method is modifying the original array, so refactor to use destructive ↵Aaron Patterson2010-07-121-5/+4
| | | | | | | | methods
* | PostgreSQLAdapter#select_raw removing dead codeAaron Patterson2010-07-121-3/+0
| |
* | PostgreSQLAdapter#select_raw use each_with_index to avoid multiple array lookupsAaron Patterson2010-07-121-5/+5
| |
* | PostgreSQLAdapter#select_raw fields and results are empty even if ntuples is 0Aaron Patterson2010-07-121-25/+21
| |
* | refactor PostgreSQLAdapter#result_as_array to create fewer Range objectsAaron Patterson2010-07-121-3/+3
| |
* | cleaning up PostgreSQLAdapter#selectAaron Patterson2010-07-121-8/+2
| |
* | these strings do not need to be frozenAaron Patterson2010-07-121-2/+2
| |
* | use Hash#each to avoid a second hash lookupAaron Patterson2010-07-121-4/+2
| |
* | Just add connection management middleware if running in a concurrent ↵José Valim2010-07-121-5/+8
| | | | | | | | environment.
* | Remove old rubygems require.José Valim2010-07-121-7/+1
| |
* | reducing funcalls, cleaning up primary key methods for sqlite adapterAaron Patterson2010-07-101-6/+11
|/
* Merge remote branch 'docrails/master'Xavier Noria2010-07-091-2/+7
|\
| * copy-edits some docsXavier Noria2010-07-081-6/+5
| |
| * Merge branch 'master' of github.com:lifo/docrailsNeeraj Singh2010-07-086-15/+28
| |\
| * | clarifying how to create non standard primary keyNeeraj Singh2010-07-081-1/+7
| | |
* | | Refactor previous commits a bit.José Valim2010-07-081-7/+10
| | |
* | | Return from ActiveRecord::Base#attributes= unless value is a hash [#4070 ↵David Trasbo2010-07-081-1/+1
| | | | | | | | | | | | | | | | | | state:committed] Signed-off-by: José Valim <jose.valim@gmail.com>
* | | reset_counter should work with non-traditional belongs_to and polymorphic ↵Neeraj Singh2010-07-081-3/+9
| | | | | | | | | | | | | | | | | | | | | | | | belongs_to [#4984 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* | | Improve a bit the code in latest commits.José Valim2010-07-082-20/+16
| | |
* | | This patch changes update_attribute implementatino so:Neeraj Singh2010-07-082-7/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - it will only save the attribute it has been asked to save and not all dirty attributes - it does not invoke callbacks - it does change updated_at/on Signed-off-by: José Valim <jose.valim@gmail.com>
* | | consolidating updated_at and updated_onNeeraj Singh2010-07-081-9/+16
| | | | | | | | | | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* | | Eager loading :through associations will join the :source model if there are ↵Grant Ammons2010-07-081-1/+6
| | | | | | | | | | | | | | | | | | :conditions. [#2362 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* | | 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>
* | | removed an old unused method in AR which removed readonly attributesJosh Kalderimis2010-07-081-9/+0
| | | | | | | | | | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* | | removing unused methodNeeraj Singh2010-07-081-5/+0
| | | | | | | | | | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* | | moving parse_sqlite_config to the sqlite3_connection method (where it ↵Aaron Patterson2010-07-082-20/+11
| | | | | | | | | | | | | | | | | | belongs) [#5071 state:resolved] Signed-off-by: Jeremy Kemper <jeremy@bitsweat.net>
* | | removing useless code. [#5070 state:resolved]Aaron Patterson2010-07-081-2/+1
| | | | | | | | | | | | Signed-off-by: Jeremy Kemper <jeremy@bitsweat.net>
* | | fix ActiveRecord `destroy_all` so it returns destroyed recordsMislav Marohnić2010-07-082-5/+5
| | | | | | | | | | | | Signed-off-by: Jeremy Kemper <jeremy@bitsweat.net>
* | | mass_assignment_security moved from AR to AMo, and minor test cleanupJosh Kalderimis2010-07-085-214/+1
| | | | | | | | | | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* | | minor changes to mass assignment security patch to bring it in line with ↵Josh Kalderimis2010-07-084-118/+99
| | | | | | | | | | | | | | | | | | rails standards Signed-off-by: José Valim <jose.valim@gmail.com>
* | | Mass assignment security refactoringEric Chapweske2010-07-085-133/+243
| |/ |/| | | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* | Merge remote branch 'docrails/master'Xavier Noria2010-07-072-2/+8
|\|
| * added more info about << operation in associationsNeeraj Singh2010-07-061-0/+4
| |
| * clarifying the comments regarding base_class declarationNeeraj Singh2010-07-011-2/+4
| |
* | documents automatic management of join models in hmt associations, in ↵Xavier Noria2010-07-061-4/+11
| | | | | | | | particular the gotcha that deletion is direct
* | Define a convention for descendants and subclasses.José Valim2010-07-051-3/+3
| | | | | | | | | | | | The former should be symmetric with ancestors and include all children. However, it should not include self since ancestors + descendants should not have duplicated. The latter is symmetric to superclass in the sense it only includes direct children. By adopting a convention, we expect to have less conflict with other frameworks, as Datamapper. For this moment, to ensure ActiveModel::Validations can be used with Datamapper, we should always call ActiveSupport::DescendantsTracker.descendants(self) internally instead of self.descendants avoiding conflicts.
* | Avoid calls to Rails::Application since this is not the official API.José Valim2010-07-011-1/+1
| | | | | | | | Your application should *always* reference your application const (as Blog::Application) and Rails.application should be used just internally.
* | Don't remove scheduled destroys when loading an association. [#4642 ↵James Le Cuirot2010-07-011-1/+6
| | | | | | | | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* | Make relation a private method.José Valim2010-07-012-6/+7
|/
* Merge remote branch 'rails/master'Xavier Noria2010-06-3012-74/+165
|\
| * Support any mysql-like adapterJeremy Kemper2010-06-301-7/+7
| |
| * AssociationCollection#create_by_*, find_or_create_by_* work properly now. ↵Aaron Patterson2010-06-291-0/+11
| | | | | | | | | | | | [#1108 state:resolved] Signed-off-by: Jeremy Kemper <jeremy@bitsweat.net>
| * migrations.rb requires active_support/core_ext/module/aliasing [#5008 ↵Tekin2010-06-301-0/+1
| | | | | | | | | | | | state:committed] Signed-off-by: Xavier Noria <fxn@hashref.com>
| * Fixed error when removing an index from a table name values, which is a ↵J. Pablo Fernández2010-06-291-1/+1
| | | | | | | | | | | | reserved word, with test. Signed-off-by: José Valim <jose.valim@gmail.com>
| * No need to check if the attribute exists (this is the same behavior as in ↵José Valim2010-06-291-3/+1
| | | | | | | | 2.3) [#4994 state:resolved] and [#5003 state:resolved]
| * array subtraction should be faster than iterating over all the elementsNeeraj Singh2010-06-291-1/+1
| | | | | | | | Signed-off-by: José Valim <jose.valim@gmail.com>