Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | | | 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 | |
| | | | | ||||||
| * | | | fix AR::AttributeMethods::Dirty :nodoc: [ci skip] | Francesco Rodriguez | 2012-09-21 | 1 | -4/+4 | |
| | | | |