aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | | | add an option to skip jbuilderAaron Patterson2013-10-271-0/+4
| | | | | | | | |
* | | | | | | | | fix variable nameAaron Patterson2013-10-272-3/+3
| | | | | | | | |
* | | | | | | | | add jbuilder web-console and sdoc to the GemEntry listAaron Patterson2013-10-273-29/+33
| | | | | | | | |
* | | | | | | | | fix the class nameAaron Patterson2013-10-271-17/+17
| | | | | | | | |
* | | | | | | | | refactor Gemfile templateAaron Patterson2013-10-271-8/+6
| | | | | | | | |
* | | | | | | | | use the GemfileGem object for more entriesAaron Patterson2013-10-273-56/+53
| | | | | | | | |
* | | | | | | | | add the database gemAaron Patterson2013-10-273-9/+8
| | | | | | | | |
* | | | | | | | | get a list of gems when filling out the gemfileAaron Patterson2013-10-273-13/+42
|/ / / / / / / /
* | | | | | | | set accessors in the set_accessors! methodAaron Patterson2013-10-272-2/+2
| | | | | | | |
* | | | | | | | Merge pull request #12661 from robin850/patch-10Rafael Mendonça França2013-10-271-0/+8
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Add a changelog entry for #12656 [ci skip]
| * | | | | | | | Add a changelog entry for #12656 [ci skip]Robin Dupret2013-10-271-0/+8
| | | | | | | | |
* | | | | | | | | Add a note about ensuring the version is right for the adapter (since you ↵David Heinemeier Hansson2013-10-271-1/+1
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | might well have specified the right gem, but locked it to too low of a version)
* | | | | | | | Merge pull request #12656 from dougcole/fix_strong_parameters_fetchGuillermo Iguaran2013-10-262-1/+14
|\ \ \ \ \ \ \ \ | |_|/ / / / / / |/| | | | | | | don't let StrongParameters mutate the hash with fetch
| * | | | | | | don't mutate hash with fetchDoug Cole2013-10-262-1/+14
|/ / / / / / /
* | | | | | | Merge pull request #12655 from ↵Santiago Pastorino2013-10-261-2/+5
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | rails/ps-remove-surprise-if-in-show-exception-middleware Remove surprise if from show_exception middleware
| * | | | | | | Remove surprise if from show_exception middlewarePrem Sichanugrist2013-10-271-2/+5
|/ / / / / / / | | | | | | | | | | | | | | This increase the readability within the rescue block.
* | | | | | | Merge pull request #12648 from chrisledet/patch-1Rafael Mendonça França2013-10-251-2/+2
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Fixed typo with after_commit docs
| * | | | | | | Fixed typo with after_commit docsChris Ledet2013-10-251-2/+2
|/ / / / / / / | | | | | | | | | | | | | | Fixed syntax error on `after_commit` docs
* | | | | | | Merge pull request #12643 from severin/pg_cast_json_on_writeYves Senn2013-10-253-0/+26
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | cast json values on write to be consistent with reading from the db.
| * | | | | | | cast json values on write to be consistent with reading from the db.Severin Schoepke2013-10-253-0/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | See also commit 5ac2341fab689344991b2a4817bd2bc8b3edac9d
* | | | | | | | prevent `time_zone_aware_attributes` test leak. follow-up to #12633.Yves Senn2013-10-251-21/+21
| | | | | | | |
* | | | | | | | only warn on leaked time zone state instead of rasing an error.Yves Senn2013-10-251-3/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Raising `RuntimeErrors` skips important cleanup code and leads to a lot of subsequent errors. This clutters the test output with a lot of noise.
* | | | | | | | Merge pull request #12633 from senny/no_more_time_zone_leaksYves Senn2013-10-2512-214/+223
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | prevent global timezone state from leaking out of test cases.
| * | | | | | | prevent global timezone state from leaking out of test cases.Yves Senn2013-10-2512-214/+223
| | | | | | | |
* | | | | | | | Merge pull request #12642 from ka8725/fix_usage_for_model_generatorsRafael Mendonça França2013-10-251-1/+1
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Improve usage example for model generator decimal field
| * | | | | | | | Improve usage example for model generatorAndrey Koleshko2013-10-251-1/+1
| | | | | | | | |
* | | | | | | | | Merge pull request #12641 from ↵Rafael Mendonça França2013-10-251-2/+2
|\ \ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | jetthoughts/12640_updated_migrations_doc_to_use_remove_column Fix migration docs to use new remove_column semantics
| * | | | | | | | Fix migration docs to use new remove_column semanticsPaul Nikitochkin2013-10-251-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: #12640
* | | | | | | | | Merge pull request #12490 from senny/store_accessor_hstore_bugYves Senn2013-10-248-14/+127
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | ActiveRecord Store works with PG `hstore` columns
| * | | | | | | | | `ActiveRecord::Store` works together with PG `hstore` columns.Yves Senn2013-10-256-14/+88
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is necessary because as of 5ac2341 `hstore` columns are always stored as `Hash` with `String` keys. `ActiveRecord::Store` expected the attribute to be an instance of `HashWithIndifferentAccess`, which led to the bug.
| * | | | | | | | | `stored_attributes` need to be specific to a subclass.Yves Senn2013-10-253-0/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently they are all stored globally in the same `Hash`. This commit forces the creation of a per-class variable if necessary. The behavior was exposed through the following test-case: ``` 1) Failure: StoreTest#test_all_stored_attributes_are_returned [/Users/senny/Projects/rails/activerecord/test/cases/store_test.rb:151]: --- expected +++ actual @@ -1 +1 @@ -[:color, :homepage, :favorite_food] +[:resolution, :color, :homepage, :favorite_food] ```
| * | | | | | | | | test to verify the `ActiveRecord::Store` behavior with PG's json typeYves Senn2013-10-251-0/+18
|/ / / / / / / / /
* | | | | | | | | Merge pull request #12621 from ↵Rafael Mendonça França2013-10-243-1/+24
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | laurocaetano/fix_has_one_association_with_primary_key_set Save association when primary key is manually set Conflicts: activerecord/CHANGELOG.md
| * | | | | | | | | Save association when primary key is manually setlaurocaetano2013-10-243-1/+24
| | | | | | | | | |
* | | | | | | | | | Merge pull request #12636 from kongregate/dont_rescue_ExceptionsRafael Mendonça França2013-10-241-2/+2
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | Avoid unnecessary catching of Exception instead of StandardError (conver...
| * | | | | | | | | | Avoid unnecessary catching of Exception instead of StandardError (converting ↵stopdropandrew2013-10-241-2/+2
| | |_|/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | Exceptions into StandardErrors)
* | | | | | | | | | Improve the test code for #8422Rafael Mendonça França2013-10-241-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Remove unneeded comments. * Use key? instead of keys.include?.
* | | | | | | | | | Merge pull request #8491 from igagnidz/masterRafael Mendonça França2013-10-241-0/+10
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Issue #8442
| * | | | | | | | | | Added test to ensure that DescendantsTracker does not leak memory on ↵Iuri Gagnidze2012-12-091-0/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | singleton classes
* | | | | | | | | | | Merge pull request #12632 from al2o3cr/fix_race_for_attribute_methodsAaron Patterson2013-10-241-6/+3
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Always check to see if methods exist after calling define_attribute_methods
| * | | | | | | | | | | always check to see if methods exist after calling define_attribute_methodsMatt Jones2013-10-241-6/+3
| | |/ / / / / / / / / | |/| | | | | | | | |
* | | | | | | | | | | Add CHANGELOG entry for #12635 [ci skip]Rafael Mendonça França2013-10-241-0/+4
| | | | | | | | | | |
* | | | | | | | | | | Merge pull request #12635 from mperham/4-0-stableGuillermo Iguaran2013-10-241-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow any version of BCrypt
* | | | | | | | | | | Merge pull request #10471 from andyw8/button_to_paramsRafael Mendonça França2013-10-243-0/+19
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add params option for button_to Conflicts: actionpack/CHANGELOG.md
| * | | | | | | | | | | Add params option for button_toAndy Waite2013-09-183-0/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The parameters are rendered as hidden form fields within the generated form. This is useful for when a record has multiple buttons associated with it, each of which target the same controller method, but which need to submit different attributes.
* | | | | | | | | | | | Merge pull request #12611 from antonio/slice_bang_honor_default_procRafael Mendonça França2013-10-243-0/+24
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | Make slice! honor default hash value/proc
| * | | | | | | | | | | | slice! should not remove default hash value/procAntonio Santos2013-10-243-0/+24
| | |_|_|_|/ / / / / / / | |/| | | | | | | | | |
* | | | | | | | | | | | Improve the CHANGELOG entry [ci skip]Rafael Mendonça França2013-10-241-10/+9
| | | | | | | | | | | |
* | | | | | | | | | | | Merge pull request #12624 from timsly/masterRafael Mendonça França2013-10-241-0/+16
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / / / |/| | | | | | | | | | | update CHANGELOG
| * | | | | | | | | | | update CHANGELOGTima Maslyuchenko2013-10-241-0/+16
| |/ / / / / / / / / /