aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | Better Error handling when parsing database.yamlGaurish Sharma2012-11-201-0/+4
|/ / / / / | | | | | | | | | | | | | | | | | | | | Provides a better error message incase the database.yaml has some errors.
* | | | | Merge branch 'url-for-params'Carlos Antonio da Silva2012-11-193-2/+16
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix error when using a non-hash query argument named "params" in `url_for`. Before: url_for(params: "") # => undefined method `reject!' for "":String After: url_for(params: "") # => http://www.example.com?params=` Closes #8233
| * | | | | Correct the use of params options when given to url_forCarlos Antonio da Silva2012-11-193-10/+16
| | | | | | | | | | | | | | | | | | | | | | | | Merge url for tests and add changelog entry for #8233.
| * | | | | Fix issue with params in url_fortumayun2012-11-192-1/+9
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | With a "params" argument, the following error is raised: undefined method `reject!` for "":String
* | | | | Refactor request tests, remove #with_set methodCarlos Antonio da Silva2012-11-191-8/+3
| | | | |
* | | | | Be a bit less conservative with mysql in adapterCarlos Antonio da Silva2012-11-191-1/+1
| |_|_|/ |/| | |
* | | | Merge pull request #8212 from senny/8197_make_render_in_spacer_template_workJosé Valim2012-11-193-2/+18
|\ \ \ \ | | | | | | | | | | render every partial with a new `PartialRenderer`.
| * | | | render every partial with a new `PartialRenderer`.Yves Senn2012-11-193-2/+18
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | This resolves issues when rendering nested partials. Previously the `PartialRenderer` was reused which led to situations where the state of the renderer was reset. Closes #8197
* | | | Merge pull request #8044 from nateberkopec/block_does_not_set_parts_orderRafael Mendonça França2012-11-194-46/+31
|\ \ \ \ | | | | | | | | | | Explicit multipart messages respect :parts_order
| * | | | Explicit multipart messages respect :parts_orderNate Berkopec2012-11-194-46/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As issue #7978, the order in which ActionMailer sends multipart messages could be unintentionally overwritten if a block is passed to the mail method. This changes the mail method such that :parts_order is always respected, regardless of whether a block is passed to mail.
* | | | | Add UpgradeSignatureToEncryptionCookieStore to upgrading_ruby_on_rails guideSantiago Pastorino2012-11-191-0/+2
| | | | |
* | | | | Fix UpgradeSignatureToEncryptionCookieStore docSantiago Pastorino2012-11-191-9/+4
|/ / / /
* | | | Remove not needed begin..end from AR#saveCarlos Antonio da Silva2012-11-191-5/+3
| | | |
* | | | Initialize accessors to remove some warnings in Ruby 2.0Carlos Antonio da Silva2012-11-192-7/+4
| | | |
* | | | Merge pull request #8262 from senny/8229_correct_bundler_require_in_enginesRafael Mendonça França2012-11-192-3/+8
|\ \ \ \ | | | | | | | | | | require bundler groups to include rake-tasks in engines
| * | | | require bundler groups to include rake-tasks in enginesYves Senn2012-11-182-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If you generate a full engine, this will include rake tasks from your gem under the `app` namespace. For example if you have a dependency on `rspec-rails` in your engine's `gemspec`. You will get the task `app:spec` Closes #8229
| * | | | cleanup, remove whitspace from railties CHANGELOGYves Senn2012-11-181-2/+2
| | | | |
* | | | | Merge pull request #8256 from nalwayaabhishek/masterCarlos Antonio da Silva2012-11-191-3/+3
|\ \ \ \ \ | | | | | | | | | | | | Displaying error message if validation fail in examples (p tag was missing)
| * | | | | displaying error message(p tag was missing) and adding output as comment Abhishek Nalwaya2012-11-191-3/+3
| | | | | |
* | | | | | Refactor config abort message a bit for dbconsoleCarlos Antonio da Silva2012-11-182-8/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove not used variable. Remove config attr_reader causing warning. Fix indent in runner file.
* | | | | | Fix rails db command with sqlite3 databaseCarlos Antonio da Silva2012-11-182-5/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When using sqlite3 it was attempting to find the database file based on Rails.root, the problem is that Rails.root is not always present because we try to first manually load "config/database.yml" instead of loading the entire app, to make "rails db" faster. This means that when we're in the root path of the app, calling "rails db" won't allow us to use Rails.root, making the command fail for sqlite3 with the error: ./rails/commands/dbconsole.rb:62:in `start': undefined method `root' for Rails:Module (NoMethodError) The fix is to simply not pass any dir string to File.expand_path, which will make it use the current directory of the process as base, or the root path of the app, which is what we want. When we are in any other subdirectory, calling "rails db" should work just fine, because "config/database.yml" won't be found, thus "rails db" will fallback to loading the app, making Rails.root available. Closes #8257.
* | | | | | Merge pull request #3023 from Tho85/preserve_sti_typeRafael Mendonça França2012-11-183-2/+31
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | AR::Base.becomes should not change the STI type Conflicts: activerecord/CHANGELOG.md
| * | | | | | AR::Base.becomes should not change the STI typeThomas Hollstegge2012-11-173-2/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | If you want to change the STI type too, use AR::Base.becomes! instead
* | | | | | | Merge pull request #8266 from CvX/index_renameRafael Mendonça França2012-11-183-0/+23
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Add rename_index to change_table.
| * | | | | | | Add rename_index to change_table.Jarek Radosz2012-11-193-0/+23
|/ / / / / / /
* | | | | | | Merge branch 'deprecate-pending'Carlos Antonio da Silva2012-11-184-2/+24
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Properly deprecate #pending from AS::TestCase. This has been previously removed from master, and is now back with a deprecation instead, to avoid people having tests breaking when upgrading an app. Please check #4575 for more background.
| * | | | | | | Properly deprecate #pending from AS::TestCaseCarlos Antonio da Silva2012-11-183-8/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Check https://github.com/rails/rails/pull/4575#issuecomment-5765575.
| * | | | | | | Revert "Merge pull request #4575 from carlosantoniodasilva/remove-test-pending"Carlos Antonio da Silva2012-11-183-2/+22
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 1620df78dff527b4fa3f7b204fa05d1b630aae17, reversing changes made to 2d000328dfc0d4b297fb4bdcebc9af6c2fb559dc. Conflicts: activesupport/CHANGELOG.md activesupport/lib/active_support/test_case.rb
* | | | | | | Ensure ordering to make the test pass with postgresqlCarlos Antonio da Silva2012-11-181-4/+5
| | | | | | |
* | | | | | | Merge pull request #8258 from kommen/eager_loading_with_select_test2Carlos Antonio da Silva2012-11-181-0/+13
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | Add test to ensure preloading works as expected with "group", "select" and "includes".
| * | | | | | Add test to ensure preloading works as expected with "select" and "includes".Dieter Komendera2012-11-181-0/+13
| | |/ / / / | |/| | | | | | | | | | | | | | | | This didn't work in rails 3.1. See #2303 for more information and original pull request.
* | | | | | Merge pull request #8260 from sauliusg/improved_deprecation_noticeRafael Mendonça França2012-11-181-4/+6
|\ \ \ \ \ \ | | | | | | | | | | | | | | Point user to strong_parameters as the new protection model [ci skip]
| * | | | | | Point user to strong_parameters as the new protection model [ci skip]Saulius Grigaliunas2012-11-181-4/+6
| |/ / / / /
* | | | | | Merge pull request #8259 from senny/make_queued_message_autoloadableRafael Mendonça França2012-11-181-0/+1
|\ \ \ \ \ \ | |/ / / / / |/| | | | | make ActionMailer::QueuedMessage autoloadable
| * | | | | make ActionMailer::QueuedMessage autoloadableYves Senn2012-11-181-0/+1
|/ / / / /
* | | | | Upgrade mail dependency to 2.5.2Mikel Lindsaar2012-11-181-1/+1
| | | | |
* | | | | Remove not used require and some useless test commentsCarlos Antonio da Silva2012-11-172-14/+3
| | | | |
* | | | | Bump mysql gem version to the newly 2.9.0, fix build.Carlos Antonio da Silva2012-11-172-2/+2
| | | | |
* | | | | Upgrade mail dependency to 2.5.0Mikel Lindsaar2012-11-181-1/+1
| | | | |
* | | | | Initialize #min_cost to avoid warning in Ruby 2.0Carlos Antonio da Silva2012-11-171-2/+3
| | | | |
* | | | | Fix typo in module name and make #in_time_zone privateCarlos Antonio da Silva2012-11-173-3/+5
| | | | |
* | | | | Extract #in_time_zone helper method duplication to a moduleCarlos Antonio da Silva2012-11-173-24/+17
| | | | |
* | | | | Simplify helper method in attribute methods testCarlos Antonio da Silva2012-11-171-3/+1
| | | | |
* | | | | Remove return guardCarlos Antonio da Silva2012-11-172-3/+1
|/ / / /
* | | | Merge pull request #8251 from alexeymuranov/deprecate-removed-fixture-methodsCarlos Antonio da Silva2012-11-171-0/+6
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | Add a deprecation before removing ActiveRecord::Fixtures::find_table_name Removed in e0ef0936193491689724880599ae26a8f5c2b5a6
| * | | | Add a deprecation before removing find_table_nameAlexey Muranov2012-11-171-0/+6
| | | | | | | | | | | | | | | The `ActiveRecord::Fixtures::find_table_name` method was removed from "master" almost a year ago, but it was never deprecated. Here it comes back, more dead than alive.
* | | | | Merge pull request #8249 from steveklabnik/add_two_oh_to_travisCarlos Antonio da Silva2012-11-171-0/+4
|\ \ \ \ \ | | | | | | | | | | | | Add Ruby 2.0 to .travis.yml.
| * | | | | Add Ruby 2.0 to .travis.yml.Steve Klabnik2012-11-171-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | @joshk said it's okay. :)
* | | | | | Merge pull request #8248 from zenspider/masterCarlos Antonio da Silva2012-11-172-5/+8
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | Removed extra call to #diff in #assert_recognizes
| * | | | | Please use assert_equal instead of diff directlyRyan Davis2012-11-171-1/+1
| | | | | |
generated by cgit v1.2.3 (git 2.25.1) at 2024-07-07 15:56:43 +0000