aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * update ActiveModel::Dirty documentationFrancesco Rodriguez2012-05-311-24/+35
| |
| * Fix typo [ci skip]Erich Menge2012-05-301-1/+1
| |
* | Revert some ActiveSupport::Callbacks changes.Rafael Mendonça França2012-06-011-3/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commits 911a0859ac065aa8e8834ac985353d659c7c7b65 and 30b31f51af6f7094c4a27b086755fc66c368d6fa. Reason: these changes make the Active Model tests fail randomly. Some examples: http://travis-ci.org/#!/rails/rails/jobs/1498992 http://travis-ci.org/#!/rails/rails/jobs/1496948 http://travis-ci.org/#!/rails/rails/jobs/1489985 This script was used to reproduce these breaks: https://gist.github.com/f6828a03ee4d40bffbc3 200 times, 0 failures
* | Allow to use mounted helpers in ActionView::TestCasePiotr Sarnacki2012-06-014-1/+24
| | | | | | | | | | Similarly to 6525002, this allows to use routes helpers for mounted helpers, but this time in ActionView::TestCase
* | minor comment revisionXavier Noria2012-06-011-1/+1
| |
* | Merge pull request #6588 from nbibler/polymorphic_to_modelJosé Valim2012-06-012-2/+34
| | | | | | | | Correct the use of to_model in polymorphic routing
* | Sync CHANGLOG with the 3-2-stable branchRafael Mendonça França2012-06-015-4/+91
| |
* | Include routes.mounted_helpers into integration testsPiotr Sarnacki2012-06-014-3/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In integration tests, you might want to use helpers from engines that you mounted in your application. It's not hard to add it by yourself, but it's unneeded boilerplate. mounted_helpers are now included by default. That means that given engine mounted like: mount Foo::Engine => "/foo", :as => "foo" you will be able to use paths from this engine in tests this way: foo.root_path #=> "/foo" (closes #6573)
* | Merge pull request #6586 from amatsuda/generator_blank_lineRafael Mendonça França2012-06-012-2/+2
|\ \ | | | | | | remove unneeded blank line from !namespeced? generated controllers
| * | remove unneeded blank line from !namespeced? controller generatorsAkira Matsuda2012-06-012-2/+2
|/ /
* | Merge pull request #6583 from amatsuda/rake_dsl_19Piotr Sarnacki2012-06-011-1/+1
|\ \ | | | | | | Revert "Only include Rake::DSL if it's defined."
| * | Revert "Only include Rake::DSL if it's defined."Akira Matsuda2012-06-011-1/+1
| | | | | | | | | | | | | | | | | | This reverts commit 82c3aca17e78d25f217702e530586673f2a219d7. Reason: Ruby 1.9.3 is shipped with Rake > 0.9
* | | Merge pull request #6584 from amatsuda/ar_explain_inspect_readabilityXavier Noria2012-06-011-1/+7
|\ \ \ | | | | | | | | improve readability of AR explain result
| * | | improve readability of AR explain resultAkira Matsuda2012-06-011-1/+7
|/ / /
* | | Merge pull request #6579 from amatsuda/null_relation_modulizeJosé Valim2012-05-313-11/+15
|\ \ \ | |/ / |/| | modulize AR::NullRelation
| * | unused method argumentsAkira Matsuda2012-06-011-7/+6
| | |
| * | modulize AR::NullRelationAkira Matsuda2012-06-013-4/+9
|/ / | | | | | | now we can invoke previously added scope extension methods
* | Rake::DSL should always be availableJeremy Kemper2012-05-311-1/+1
| |
* | Merge branch 'master-sec'Aaron Patterson2012-05-315-4/+65
|\ \ | | | | | | | | | | | | | | | * master-sec: Strip [nil] from parameters hash. Thanks to Ben Murphy for reporting this! predicate builder should not recurse for determining where columns. Thanks to Ben Murphy for reporting this
| * | Strip [nil] from parameters hash.Aaron Patterson2012-05-302-1/+28
| | | | | | | | | | | | | | | | | | Thanks to Ben Murphy for reporting this! CVE-2012-2660
| * | predicate builder should not recurse for determining where columns.Aaron Patterson2012-05-303-3/+37
| | | | | | | | | | | | | | | | | | Thanks to Ben Murphy for reporting this CVE-2012-2661
* | | Explicitly require rake so its DSL is availableJeremy Kemper2012-05-311-0/+1
| | |
* | | Rake is in 1.9 stdlib, no need for gem depJeremy Kemper2012-05-311-4/+2
| | |
* | | Psych is always available on 1.9Jeremy Kemper2012-05-312-29/+23
| | |
* | | Merge pull request #6571 from ↵Rafael Mendonça França2012-05-313-7/+50
|\ \ \ | | | | | | | | | | | | | | | | jeremyf/extract-force-ssl-filter-logic-to-separate-method Extracted redirect logic from ActionController::Force::ClassMethods.forc...
| * | | Extracted redirect logic from ActionController::Force::ClassMethods.force_sslJeremy Friesen2012-05-313-7/+50
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Prior to this patch the existing .force_ssl method handles both defining the filter and handling the logic for performing the redirect. With this patch the logic for redirecting to the HTTPS protocol is separated from the filter logic that determines if a redirect should occur. By separating the two levels of behavior, an instance method for ActionController (i.e. #force_ssl_redirect) is exposed and available for more granular SSL enforcement. Cleaned up indentation.
* | | | Reduce warning messages when running Active Support testsAndrew White2012-05-312-2/+8
| | | | | | | | | | | | | | | | Eliminate the warnings generated by redefining methods and constants.
* | | | Merge pull request #6566 from arunagw/add-javascripts-to-rake-stats-taskRafael Mendonça França2012-05-312-2/+10
|\ \ \ \ | |/ / / |/| | | Add code statistics for Javascript and CoffeeScript files
| * | | Add code statistics for Javascript andArun Agrawal2012-05-312-2/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | CoffeeScript files to `rake stats` task Orignal PR was #2270 Thanks to @nfm
* | | | Fix changelog example for count method with block [ci skip]Carlos Antonio da Silva2012-05-311-1/+1
| | | | | | | | | | | | | | | | Missed that too, thanks again @splattael.
* | | | Fix wrong example for count method with block [ci skip]Carlos Antonio da Silva2012-05-311-1/+1
| | | | | | | | | | | | | | | | Thanks @splattael.
* | | | Simplify link_to using content_tagCarlos Antonio da Silva2012-05-312-12/+26
| | | | | | | | | | | | | | | | Add some tests for link_to with blocks and escaping content.
* | | | Make link_to arguments explicitCarlos Antonio da Silva2012-05-311-8/+4
| | | |
* | | | Make button_to arguments explicit and refactor a bitCarlos Antonio da Silva2012-05-311-10/+6
| | | | | | | | | | | | | | | | Prefer Hash#[]= over Hash#merge when setting a value.
* | | | Simplify logic to initialize valid conditions in RouteSetCarlos Antonio da Silva2012-05-311-7/+3
|/ / / | | | | | | | | | | | | | | | Remove :to_sym call from public_instance_methods iteration, as such methods in Ruby 1.9 already return symbols. Initialize valid conditions with controller/action instead of setting them afterwards.
* | | Work around undiagnosed bug that's draining a relation's bind_valuesJeremy Kemper2012-05-311-0/+1
| | |
* | | Fix backward compatibility with stored Hash values. Wrap coders to convert ↵Jeremy Kemper2012-05-303-15/+51
| | | | | | | | | | | | serialized values to indifferent access.
* | | Set RACK_ENV to nil in the dbconsole testRafael Mendonça França2012-05-301-0/+1
| | | | | | | | | | | | This will fix the travis-ci build
* | | Fix app_generator_test with the changes introduced atRafael Mendonça França2012-05-301-1/+1
| | | | | | | | | | | | f4d7af67ffc90f2542afa50c7579fc83ea4f45f2
* | | Merge pull request #6560 from martinoconnor/generator_fixRafael Mendonça França2012-05-301-1/+1
|\ \ \ | | | | | | | | Changed symbol platform to platforms for the commented out call to gem 'therubyracer'.
| * | | Changed symbol platform to platforms for the commented out call to gem ↵Martin O'Connor2012-05-301-1/+1
|/ / / | | | | | | | | | | | | | | | 'therubyracer'. Dependency.rb expects the symbol to be named :platforms as opposed to platform. RubyMine's inspections indicate that the symbol should be named :platforms.
* | | Merge pull request #6276 from samoli/fix_console_argumentsRafael Mendonça França2012-05-307-132/+182
|\ \ \ | | | | | | | | Fix various bugs with console arguments and improve test coverage
| * | | Remove support for rails server RAILS_ENV=env-nameSam Oliver2012-05-302-7/+2
| | | |
| * | | Fix various bugs with console arguments.Sam Oliver2012-05-305-125/+180
|/ / / | | | | | | | | | Allow hyphens in environment names again.
* | | Merge pull request #6477 from steveklabnik/close_discovered_pg_connectionRafael Mendonça França2012-05-301-1/+2
|\ \ \ | | | | | | | | Properly discover a connection is closed in postgresql_adapter
| * | | Properly discover a connection is closed in postgresql_adapterSteve Klabnik2012-05-241-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | PQstatus doesn't properly test if future operations will succeed. A PQping function is added to libpq in PostgreSQL 9.1, but if we rely on it, everyone on earlier versions of Postgres is out of luck, and the pg gem wouldn't have the 'fix' until the next release. Thanks to @cbrecabarren and @ged for handling all the dirty details. Closes #3392.
* | | | Merge pull request #6556 from lest/patch-1Rafael Mendonça França2012-05-304-5/+53
|\ \ \ \ | | | | | | | | | | accept a block in button_to helper
| * | | | accept a block in button_to helperSergey Nartimov2012-05-303-3/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make possible to use a block in button_to helper if button text is hard to fit into the name parameter, e.g.: <%= button_to [:make_happy, @user] do %> Make happy <strong><%= @user.name %></strong> <% end %> # => "<form method="post" action="/users/1/make_happy" class="button_to"> # <div> # <button type="submit"> # Make happy <strong>Name</strong> # </button> # </div> # </form>"
| * | | | no need to pass an empty block to button_to helperSergey Nartimov2012-05-301-2/+2
| | | | |
* | | | | Merge pull request #6554 from erichmenge/remove-json-variablePiotr Sarnacki2012-05-304-14/+7
|\ \ \ \ \ | |/ / / / |/| | | | Remove deprecated ActiveSupport::JSON::Variable.