aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Delegate #model_name method to self.classYuki Nishijima2014-06-222-0/+12
|
* changelog entries for #15866 and #15848Yves Senn2014-06-221-0/+11
| | | | /cc @sgrif
* Merge pull request #15848 from sgrif/sg-remove-deprecationYves Senn2014-06-221-8/+1
|\ | | | | Remove old deprecation warning
| * Remove old deprecation warningSean Griffin2014-06-221-8/+1
| | | | | | | | | | This has been around for a couple of versions now, a `NoMethodError` should suffice at this point.
* | Merge pull request #15865 from sgrif/sg-missing-testYves Senn2014-06-221-0/+20
|\ \ | | | | | | Add missing test cases for `attribute_names` instance method
| * | Add missing test cases for `attribute_names` instance methodSean Griffin2014-06-221-0/+20
| | | | | | | | | | | | | | | There is a class method with the same name that is tested, but not the instance method.
* | | Merge pull request #15866 from sgrif/sg-reload-attributesYves Senn2014-06-223-5/+8
|\ \ \ | |/ / |/| | `reload` should fully reload attributes
| * | `reload` should fully reload attributesSean Griffin2014-06-223-5/+8
|/ / | | | | | | | | | | `reload` is meant to put a record in the same state it would be if you were to do `Post.find(post.id)`. This means we should fully replace the attributes hash.
* | Merge pull request #15860 from aidanharan/docrails-assets-manifest-typoYves Senn2014-06-221-1/+1
|\ \ | |/ |/| [ci skip] Added missing colon in asset manifest JSON
| * Added missing colon in asset manifest JSON.Aidan Haran2014-06-181-1/+1
| |
* | Merge pull request #15859 from zuhao/refactor_activesupport_caching_testYves Senn2014-06-221-14/+13
|\ \ | | | | | | Create with_env helper for tests.
| * | Create with_env helper for tests.Zuhao Wan2014-06-221-14/+13
|/ /
* | Merge pull request #15839 from sgrif/sg-attr-set-nullYves Senn2014-06-226-12/+17
|\ \ | | | | | | Return a null object from `AttributeSet#[]`
| * | Return a null object from `AttributeSet#[]`Sean Griffin2014-06-206-12/+17
| | |
* | | Merge pull request #15849 from sgrif/sg-move-attributesYves Senn2014-06-223-3/+14
|\ \ \ | | | | | | | | Move `attributes` to the `AttributeSet` object.
| * | | Move `attributes` to the `AttributeSet` object.Sean Griffin2014-06-213-3/+14
| | | |
* | | | Merge pull request #15850 from sgrif/sg-missing-testsYves Senn2014-06-221-0/+11
|\ \ \ \ | | | | | | | | | | Add missing test cases for `attribute_method?`
| * | | | Add missing test cases for `attribute_method?`Sean Griffin2014-06-211-0/+11
| |/ / /
* | | | Merge pull request #15856 from zuhao/refactor_activesupport_decoding_testYves Senn2014-06-221-11/+19
|\ \ \ \ | | | | | | | | | | Use with_parse_json_times helper in tests.
| * | | | Use with_parse_json_times helper in tests.Zuhao Wan2014-06-221-11/+19
| |/ / /
* | | | Merge pull request #15857 from zuhao/refactor_activesupport_encoding_testYves Senn2014-06-221-14/+19
|\ \ \ \ | |/ / / |/| | | Avoid hardcoded magic number in test teardown.
| * | | Avoid hardcoded magic number in test teardown.Zuhao Wan2014-06-221-14/+19
|/ / /
* | | :nail_care: wrap the tip from #15624 at 80 chars [ci skip]Zachary Scott2014-06-201-1/+2
| | |
* | | Merge pull request #15624 from aditya-kapoor/add-warnings-on-guidesZachary Scott2014-06-201-0/+2
|\ \ \ | |/ / |/| | [ci skip] Add Tip for change_column_null and change_column_default in guides
| * | [ci skip] Add Tip for change_column_null and change_column_default in guidesAditya Kapoor2014-06-211-0/+2
| | |
* | | Merge pull request #15764 from arthurnn/dependencies_name_errorMatthew Draper2014-06-213-5/+11
|\ \ \ | | | | | | | | Make dependencies.rb add a name to NameError
| * | | Add regression test for NameError#nameArthur Neves2014-06-202-2/+5
| | | |
| * | | Make dependencies.rb add a name to NameErrorArthur Neves2014-06-202-4/+7
|/ / /
* | | docs, configurations method is listed in the docs. [ci skip]Yves Senn2014-06-201-1/+0
| | | | | | | | | | | | | | | The :singleton-method: directive is printed in the docs. This directive is not necessary as the method is documentable just fine.
* | | Merge pull request #15809 from maurogeorge/guides-console-appYves Senn2014-06-201-0/+25
|\ \ \ | | | | | | | | Add to guides app and helper objects on command line
| * | | Add to guides app and helper objects on command lineMauro George2014-06-191-0/+25
| | | | | | | | | | | | | | | | [ci skip]
* | | | Merge pull request #15537 from tgxworld/fix_state_leakMatthew Draper2014-06-204-5/+5
|\ \ \ \ | | | | | | | | | | Fix state leak.
| * | | | Remove redundant code.Guo Xiang Tan2014-06-051-4/+0
| | | | |
| * | | | Prevent state leak.Guo Xiang Tan2014-06-054-1/+5
| | | | |
* | | | | Merge pull request #15824 from bhicks/remove-extra-list-item-elementSean Griffin2014-06-191-1/+1
|\ \ \ \ \ | | | | | | | | | | | | Remove extra list item element in engine guide
| * | | | | Remove extra list item elementBen Hicks2014-06-191-1/+1
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Cleaning up the list items that detail what an engines `--full` option generates. Joining a multiline multi list item into a multiline single list item. [ci skip]
* | | | | add both branches to the only_path conditionalAaron Patterson2014-06-191-6/+6
| | | | |
* | | | | Merge pull request #15728 from sgrif/sg-double-save-hm-tRafael Mendonça França2014-06-193-4/+30
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Don't save through records twice Conflicts: activerecord/CHANGELOG.md activerecord/test/cases/associations/has_many_through_associations_test.rb
| * | | | | Don't save through records twiceSean Griffin2014-06-173-3/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If the through record gets created in an `after_create` hook that is defined before the association is defined (therefore after its `after_create` hook) get saved twice. This ensures that the through records are created only once, regardless of the order of the hooks.
* | | | | | Merge pull request #15823 from ↵Rafael Mendonça França2014-06-191-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | yuki24/better-docmentation-for-dynamic-error-pages ErrorsController shouldn't inherit ApplicationController, but ActionController
| * | | | | | ErrorsController shouldn't inherit ApplicationController, but ↵Yuki Nishijima2014-06-191-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ActionController::Base Inheriting `ApplicationController` often causes an issue as it always has before/after actions. It should encourage to use `ActionController::Base` instead. [ci skip]
* | | | | | | Merge pull request #15747 from sgrif/sg-trolololol-this-is-so-brokenRafael Mendonça França2014-06-193-1/+56
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Always update counter caches in memory when adding records
| * | | | | | | Always update counter caches in memory when adding recordsSean Griffin2014-06-163-1/+56
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Before, calling `size` would only work if it skipped the cache, and would return a different result from the cache, but only if: - The association was previously loaded - Or you called size previously - But only if the size was 0 when you called it This ensures that the counter is appropriately updated in memory.
* | | | | | | | Merge pull request #15800 from sgrif/sg-column-in-calculationRafael Mendonça França2014-06-192-17/+8
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Don't use `Column` for type casting in Relation calculations
| * | | | | | | | Don't use `Column` for type casting in Relation calculationsSean Griffin2014-06-182-17/+8
| | | | | | | | |
* | | | | | | | | Merge pull request #15801 from sgrif/sg-column-defaultsRafael Mendonça França2014-06-191-2/+2
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Don't use column object when calculating type cast defaults
| * | | | | | | | | Don't use column object when calculating type cast defaultsSean Griffin2014-06-181-2/+2
| |/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | Moves towards removing type casting knowledge from the column entirely
* | | | | | | | | Merge pull request #15802 from sgrif/sg-column-quotingRafael Mendonça França2014-06-193-31/+6
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Don't use column object for type casting in `quoting`
| * | | | | | | | | Don't use column object for type casting in `quoting`Sean Griffin2014-06-183-31/+6
| |/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We're never going to be able to use the attribute object here, however, so let's just accept the ugly demeter violation here for now. Remove test cases which were either redundant with other tests in the file, or were actually testing the type objects (which are tested elsewhere)
* | | | | | | | | Merge pull request #15803 from sgrif/sg-column-in-associationsRafael Mendonça França2014-06-193-5/+5
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Don't rely on the column for type casting reflections
generated by cgit v1.2.3 (git 2.25.1) at 2025-01-04 02:17:20 +0000