aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Dry-up ActionDispatch::Routing autoloadsGenadi Samokovarov2013-06-041-5/+7
|
* Merge pull request #10767 from jmondo/masterCarlos Antonio da Silva2013-05-291-1/+1
|\ | | | | Use grep instead of select with === in QueryMethods
| * use grep over select for consistency and efficiencyJohn Gesimondo2013-05-281-1/+1
| | | | | | | | pass block directly to grep
* | Merge pull request #10781 from arunagw/run-isolated-testRafael Mendonça França2013-05-284-37/+13
|\ \ | | | | | | Running isolated tests
| * | Running isolated testsArun Agrawal2013-05-284-37/+13
| | | | | | | | | | | | These stopped running don't know when. But these should be running now.
* | | Merge pull request #10766 from kylerippey/minor_schema_migration_refactorRafael Mendonça França2013-05-281-18/+24
|\ \ \ | | | | | | | | Minor refactor of ActiveRecord::SchemaMigration
| * | | Minor refactor of ActiveRecord::SchemaMigration to remove references to ↵Kyle Rippey2013-05-261-18/+24
| | | | | | | | | | | | | | | | Base, override table_exists method, and switch to preferred style for class method definitions.
* | | | Merge pull request #10495 from senny/postgres_test_cleanupRafael Mendonça França2013-05-282-30/+30
|\ \ \ \ | | | | | | | | | | PostgreSQL specific test cleanup
| * | | | cleanup, assert on warning from postgres adapterYves Senn2013-05-081-1/+1
| | | | |
| * | | | cleanup, switch arguments in assert_equal callsYves Senn2013-05-081-29/+29
| | | | |
* | | | | Merge branch 'fix-test_unit-railtie'Rafael Mendonça França2013-05-281-1/+1
|\ \ \ \ \ | |_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | This branch is composed of #10629 and #10708 Closes #10629 Closes #10708 Fixes #10628
| * | | | No need changelog entryRafael Mendonça França2013-05-281-2/+0
| | | | |
| * | | | condition simplifiedOleg Sukhodolsky2013-05-281-1/+1
| | | | |
| * | | | Merge pull request #10629 from stmpjmpr/bug_10628Rafael Mendonça França2013-05-282-1/+3
|/| | | | | | | | | | | | | | | | | | | Fixes bug 10628.
| * | | | Fixes bug 10628.Scott Hill2013-05-152-1/+3
| | | | |
* | | | | Merge pull request #10755 from lukewendling/masterRafael Mendonça França2013-05-281-0/+3
|\ \ \ \ \ | | | | | | | | | | | | Add notice on server boot if binding to 0.0.0.0
| * | | | | add notice to server boot messages if using default 0.0.0.0 bindingLuke Wendling2013-05-251-0/+3
| | | | | |
* | | | | | Merge pull request #10775 from senny/postgres_adapter_test_cleanupRafael Mendonça França2013-05-281-5/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | test cleanup, replace `define_method` and `remove_method` with stubs.
| * | | | | | test cleanup, replace `define_method` and `remove_method` with stubs.Yves Senn2013-05-281-5/+1
|/ / / / / /
* | | | | | Merge pull request #10769 from ↵Rafael Mendonça França2013-05-275-28/+29
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | | | | | | | | | | | | | | senny/10615_join_should_not_return_readonly_records using `#joins` does not imply `readonly = true`.
| * | | | | `implicit_readonly` is being removed in favor of calling `readonly` explicitlyYves Senn2013-05-275-24/+25
| | | | | |
| * | | | | cleanup whitespace in `active_record/relation.rb`.Yves Senn2013-05-271-4/+4
|/ / / / /
* | | | | Merge pull request #10763 from senny/10758_to_sentence_with_blank_elementsCarlos Antonio da Silva2013-05-261-0/+4
|\ \ \ \ \ | |_|_|_|/ |/| | | | test-case for `Array#to_sentence` with `blank?` items.
| * | | | add test-case for `Array#to_sentence with blank items.Yves Senn2013-05-261-0/+4
| |/ / /
* | | | Merge pull request #10762 from vipulnsward/change_compatibilityCarlos Antonio da Silva2013-05-261-3/+3
|\ \ \ \ | | | | | | | | | | compatability => compatibility
| * | | | compatability => compatibilityVipul A M2013-05-261-3/+3
| |/ / /
* | | | Merge pull request #10761 from vipulnsward/fix-typo-1Carlos Antonio da Silva2013-05-264-4/+4
|\ \ \ \ | |/ / / |/| | | Fix some typo in method names, variables
| * | | Fix some typo in method names, variablesVipul A M2013-05-264-4/+4
| | | |
* | | | Add note about decorator loading in Engines guide.Steve Klabnik2013-05-241-0/+26
|/ / / | | | | | | | | | | | | | | | | | | | | | Because decorators aren't referenced by the application, they won't get autoloaded. And because we recommend the pattern, it would be irresponsible to not show how to load them properly. Fixes #10647.
* | | Merge pull request #10222 from senny/update_bundler_section_in_assets_guideRafael Mendonça França2013-05-231-9/+7
|\ \ \ | | | | | | | | | | | | | | | | docs, adjust the application.rb snippets to match the generated file. Closes #10449
| * | | asset guide, match application.rb snippets to the generated file [ci skip]Yves Senn2013-04-151-9/+7
| | | | | | | | | | | | | | | | `if defined?(Bundler)` is no longer used in our current application.rb
* | | | Merge pull request #10713 from senny/10693_fix_primary_key_option_on_has_manyRafael Mendonça França2013-05-234-2/+15
|\ \ \ \ | | | | | | | | | | Fix the `:primary_key` option for `has_many` associations.
| * | | | Fix the `:primary_key` option for `has_many` associations.Yves Senn2013-05-234-2/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When removing records from a `has_many` association it used the `primary_key` defined on the association. Our test suite didn't fail because on all occurences of `:primary_key`, the specified column was available in both tables. This prevented the code from raising an exception but it still behaved badly. I added a test-case to prevent regressions that failed with: ``` 1) Error: HasManyAssociationsTest#test_has_many_assignment_with_custom_primary_key: ActiveRecord::StatementInvalid: SQLite3::SQLException: no such column: essays.first_name: UPDATE "essays" SET "writer_id" = NULL WHERE "essays"."writer_id" = ? AND "essays"."first_name" IS NULL ```
* | | | | Merge branch 'deprecate-validator-setup'Rafael Mendonça França2013-05-238-42/+61
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | Closes #10716
| * | | | deprecate Validator#setup (to get rid of a respond_to call). validators do ↵Nick Sutterer2013-05-238-42/+61
|/ / / / | | | | | | | | | | | | their setup in their constructor now.
* | | | Add CHANGELOG entry for 99860582b2b1c0fc42bf84c52aac57b243d42678Rafael Mendonça França2013-05-221-0/+6
| | | |
* | | | Require only minitest.Rafael Mendonça França2013-05-222-2/+2
| | | | | | | | | | | | | | | | | | | | minitest/autorun required minitest/spec and we are avoiding to require it.
* | | | Merge pull request #10667 from rubys/minitest-autorunRafael Mendonça França2013-05-222-2/+2
|\ \ \ \ | | | | | | | | | | Eliminate minitest warnings
| * | | | Eliminate minitest warningsSam Ruby2013-05-172-2/+2
| | | | | | | | | | | | | | | | | | | | https://github.com/seattlerb/minitest/commit/9a57c520ceac76abfe6105866f8548a94eb357b6#L15R8
* | | | | Merge pull request #10725 from neerajdotname/remove_code_duplicationCarlos Antonio da Silva2013-05-221-2/+3
|\ \ \ \ \ | | | | | | | | | | | | Remove code duplication
| * | | | | remove code duplicationNeeraj Singh2013-05-221-2/+3
|/ / / / /
* | | | | Merge pull request #10714 from tkhr/extract_test_from_base_test.rb_in_ARCarlos Antonio da Silva2013-05-224-45/+54
|\ \ \ \ \ | | | | | | | | | | | | Extract tests code out from AR's base_test.rb
| * | | | | Extract AR::Persistence#becomes's test code out from base_test.rbTakehiro Adachi2013-05-222-13/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The method got extracted out from AR::Base in commit d916c62cfc7c59ab6411407a05b946d3dd7535e9, but the tests never did.
| * | | | | Extract AR::AttributeMethods#attribute_for_inspect's test code out from ↵Takehiro Adachi2013-05-222-8/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | base_test.rb The method itself got extracted out from ActiveRecored::Base in commit ceb33f84933639d3b61aac62e5e71fd087ab65ed, but the test code never did.
| * | | | | Extract AR::Core#inspect and AR::Core.inspect's test code from base_test.rbTakehiro Adachi2013-05-222-24/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The methods got moved to core.rb in commit b2c9ce341a1c907041f55461aefebb0321280cb5, but the tests never did.
* | | | | | Merge pull request #10722 from jgaskins/patch-1José Valim2013-05-221-0/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | Add ActiveModel requirement to application.rb
| * | | | | | Add ActiveModel requirement to application.rbJamie Gaskins2013-05-221-0/+1
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, ActiveModel is only loaded by ActiveRecord. If you skip ActiveRecord, ActiveModel will not be required (or even autoloaded) and including `ActiveModel::Model` into a plain Ruby class will raise `NameError`. To reproduce this: - create a new app with `rails new my_app -O` - create a Ruby class that includes `ActiveModel::Model` in `app/models` - load up a Rails console and try to do anything with the class :-) Since ActionPack relies so heavily on the ActiveModel API, this should probably be considered a dependency of the app. Another possibility would be to make it a dependency of ActionController.
* | | | | | reverse comparison because of strange AS behaviorAaron Patterson2013-05-211-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | http://tenderlovemaking.com/2013/05/21/one-danger-of-freedom-patches.html
* | | | | | just set the default argument, a nil parent should be an errorAaron Patterson2013-05-211-2/+1
| | | | | |
* | | | | | fold the collection rather than multiple assigmentsAaron Patterson2013-05-211-3/+2
| | | | | |
generated by cgit v1.2.3 (git 2.25.1) at 2024-07-08 18:38:27 +0000