Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Revert "Key the attributes hash with symbols" | Jon Leighton | 2012-10-12 | 4 | -14/+10 |
| | | | | | | | | | | | | This reverts commit 86c3dfbd47cb96af02daaa655963292b1a1b110e. Conflicts: activerecord/lib/active_record/attribute_methods/read.rb Reason: whilst this increased performance, it also presents a DoS risk via memory exhaustion if users were allowing user input to dictate the arguments of read/write_attribute. I will investigate alternative ways to cut down on string allocations here. | ||||
* | Cleanup trailing whitespaces | dfens | 2012-10-12 | 5 | -5/+5 |
| | |||||
* | Eager autoload Preloader classes | John Firebaugh | 2012-10-10 | 1 | -10/+14 |
| | | | | | Without eager autoloading, these would be autoloaded only when #preloader_for is called, which is too late in threaded applications. | ||||
* | Remove the leading :: constant qualifier in the ActiveRecord::Fixtures ↵ | Jeremy Kemper | 2012-10-10 | 1 | -1/+1 |
| | | | | deprecation message | ||||
* | Merge pull request #7887 from senny/remove_unused_requires_in_ar_tests | Vijay Dev | 2012-10-10 | 4 | -4/+0 |
|\ | | | | | remove duplicated require statements in AR test cases | ||||
| * | remove duplicated require statements in AR test cases | Yves Senn | 2012-10-09 | 4 | -4/+0 |
| | | |||||
* | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-10-11 | 7 | -24/+39 |
|\ \ | | | | | | | | | | | | | | | | Conflicts: activerecord/lib/active_record/persistence.rb railties/lib/rails/generators/rails/resource_route/resource_route_generator.rb | ||||
| * | | copy edits [ci skip] | Vijay Dev | 2012-10-11 | 1 | -2/+2 |
| | | | |||||
| * | | Added clarity to update_column(s) | Adam Haymond | 2012-10-10 | 1 | -2/+2 |
| | | | |||||
| * | | gradually moving documentation to new hash syntax | AvnerCohen | 2012-10-10 | 2 | -2/+2 |
| | | | |||||
| * | | Fixed unclosing tag | AvnerCohen | 2012-10-09 | 1 | -1/+1 |
| | | | |||||
| * | | Fix typo: 'this also mean' -> 'this also means' | Jeffrey Hardy | 2012-10-08 | 1 | -1/+1 |
| | | | |||||
| * | | Fix missing typewriter tag | Lincoln Lee | 2012-10-08 | 1 | -1/+1 |
| | | | |||||
| * | | Add CollectionAssociation#destroy to ActiveRecord::Association::ClassMethods ↵ | Samuel Cochran | 2012-10-05 | 1 | -3/+15 |
| | | | | | | | | | | | | code docs | ||||
| * | | fix example in Migration docs [ci skip] | Francesco Rodriguez | 2012-09-30 | 1 | -1/+1 |
| | | | |||||
| * | | add change_table transformation to Migration docs [ci skip] | Francesco Rodriguez | 2012-09-30 | 1 | -16/+19 |
| | | | |||||
* | | | Merge pull request #7859 from ernie/fix-collection-associations-with-select | Aaron Patterson | 2012-10-09 | 2 | -1/+9 |
|\ \ \ | |_|/ |/| | | Fix has_many assocation w/select load after create | ||||
| * | | Fix has_many assocation w/select load after create | Ernie Miller | 2012-10-05 | 2 | -1/+9 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If you create a new record via a collection association proxy that has not loaded its target, and which selects additional attributes through the association, then when the proxy loads its target, it will inadvertently trigger an ActiveModel::MissingAttributeError during attribute writing when CollectionAssociation#merge_target_lists attempts to do its thing, since the newly loaded records will possess attributes the created record does not. This error also raises a bogus/confusing deprecation warning when accessing the association in Rails 3.2.x, so cherry-pick would be appreciated! | ||||
| * | | Revert "Use flat_map { } instead of map {}.flatten" | Santiago Pastorino | 2012-10-05 | 9 | -13/+13 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit abf8de85519141496a6773310964ec03f6106f3f. We should take a deeper look to those cases flat_map doesn't do deep flattening. irb(main):002:0> [[[1,3], [1,2]]].map{|i| i}.flatten => [1, 3, 1, 2] irb(main):003:0> [[[1,3], [1,2]]].flat_map{|i| i} => [[1, 3], [1, 2]] | ||||
| * | | Use flat_map { } instead of map {}.flatten | Santiago Pastorino | 2012-10-05 | 9 | -13/+13 |
| | | | |||||
* | | | remove unused `expand_range_bind_variables` method | Yves Senn | 2012-10-08 | 1 | -17/+0 |
| | | | | | | | | | | | | this method was not used, not documented and not tested. | ||||
* | | | Should use app.paths instead of specific path. | kennyj | 2012-10-08 | 1 | -1/+2 |
| | | | |||||
* | | | Add CHANGELOG entry for "Fixtures" -> "FixtureSet" | Alexey Muranov | 2012-10-07 | 1 | -1/+13 |
| | | | |||||
* | | | Move/rename files to follow naming conventions | Alexey Muranov | 2012-10-07 | 3 | -1/+1 |
| | | | |||||
* | | | Deprecate "Fixtures" constant | Alexey Muranov | 2012-10-07 | 1 | -0/+8 |
| | | | |||||
* | | | Rename "Fixtures" class to "FixtureSet" | Alexey Muranov | 2012-10-07 | 11 | -58/+58 |
| | | | | | | | | | | | | Rename `ActiveRecord::Fixtures` class to `ActiveRecord::FixtureSet`. Instances of this class normally hold a collection of fixtures (records) loaded either from a single YAML file, or from a file and a folder with the same name. This change make the class name singular and makes the class easier to distinguish from the modules like `ActiveRecord::TestFixtures`, which operates on multiple fixture sets, or `DelegatingFixtures`, `::Fixtures`, etc., and from the class `ActiveRecord::Fixture`, which corresponds to a single fixture. | ||||
* | | | Remove unneeded require | Rafael Mendonça França | 2012-10-06 | 1 | -1/+0 |
| | | | |||||
* | | | Move multiparameter attributes related tests to its own file | Rafael Mendonça França | 2012-10-06 | 2 | -337/+351 |
| | | | |||||
* | | | PostgreSQL, quote table names when fetching the primary key. Closes #5920 | Yves Senn | 2012-10-05 | 3 | -1/+10 |
|/ / | |||||
* | | Update activerecord/lib/active_record/persistence.rb | Adam Haymond | 2012-10-04 | 1 | -2/+2 |
| | | | | | | - Changed the comments documentation for the update_column(s) methods to add a little bit of clarity | ||||
* | | Fix CHANGELOG entry [ci skip] | Rafael Mendonça França | 2012-10-04 | 1 | -4/+4 |
| | | |||||
* | | Count returns 0 without querying if parent is not saved | Francesco Rodriguez | 2012-10-03 | 5 | -0/+37 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Patches `CollectionAssociation#count` to return 0 without querying if the parent record is new. Consider the following code: class Account has_many :dossiers end class Dossier belongs_to :account end a = Account.new a.dossiers.build # before patch a.dossiers.count # SELECT COUNT(*) FROM "dossiers" WHERE "dossiers"."account_id" IS NULL # => 0 # after a.dosiers.count # fires without sql query # => 0 Fixes #1856. | ||||
* | | Refactor | Santiago Pastorino | 2012-10-03 | 1 | -5/+4 |
| | | |||||
* | | fix warning: method redefined | kennyj | 2012-10-02 | 2 | -2/+2 |
| | | |||||
* | | Merge pull request #7822 from lulalala/reset-counter-cache-for-has-many-through | Rafael Mendonça França | 2012-10-02 | 5 | -4/+28 |
|\ \ | | | | | | | Fix reset_counters crashing on has_many :through associations. | ||||
| * | | Fix reset_counters() crashing on has_many :through associations. | lulalala | 2012-10-02 | 5 | -4/+28 |
| | | | | | | | | | | | | | | | The counter column name in the intermediate model need to be access via the through reflection. | ||||
* | | | Merge pull request #7708 from bdurand/optimize_log_subscribers | Rafael Mendonça França | 2012-10-01 | 1 | -4/+6 |
|\ \ \ | |/ / |/| | | Optimize log subscribers to check if the log level is sufficient | ||||
| * | | Optimize log subscribers to check if the log level is sufficient before ↵ | Brian Durand | 2012-09-30 | 1 | -4/+6 |
| | | | | | | | | | | | | performing an operations. | ||||
* | | | small refactoring of build_relation in uniqueness | Angelo Capilleri | 2012-09-29 | 1 | -5/+3 |
| | | | | | | | | | | | | | | | reflection init as 'if' stantment. column is always the same expression and depends from the changing of attributes | ||||
* | | | Change query pattern case insensitive | Yasuo Honda | 2012-09-29 | 1 | -1/+1 |
| | | | | | | | | | | | | because Oracle adapter uses upper case attribute/column name. | ||||
* | | | Fix syntax error with no US-ASCII char | Rafael Mendonça França | 2012-09-28 | 1 | -1/+1 |
| |/ |/| | |||||
* | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-09-28 | 17 | -186/+161 |
|\ \ | | | | | | | | | | | | | Conflicts: actionpack/lib/action_view/helpers/asset_tag_helper.rb | ||||
| * | | remove added indentation from the last commit [ci skip] | Francesco Rodriguez | 2012-09-22 | 1 | -1/+1 |
| | | | |||||
| * | | update AR::Validations documentation [ci skip] | Francesco Rodriguez | 2012-09-22 | 5 | -28/+28 |
| | | | |||||
| * | | nodoc AR::Railtie [ci skip] | Francesco Rodriguez | 2012-09-21 | 2 | -4/+4 |
| | | | |||||
| * | | update AR::Scoping documentation [ci skip] | Francesco Rodriguez | 2012-09-21 | 2 | -91/+77 |
| | | | |||||
| * | | add :nodoc: directive to AR::Fixtures::File [ci skip] | Francesco Rodriguez | 2012-09-21 | 1 | -1/+1 |
| | | | |||||
| * | | fix AR::Coders::YAMLColumn nodoc [ci skip] | Francesco Rodriguez | 2012-09-21 | 1 | -4/+2 |
| | | | |||||
| * | | update AR/attribute_methods documentation [ci skip] | Francesco Rodriguez | 2012-09-21 | 4 | -26/+36 |
| | | | |||||
| * | | Better docs for overriding inheretance column | Steve Klabnik | 2012-09-22 | 1 | -5/+2 |
| | | |