aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Make sure that javascript_include_tag/stylesheet_link_tag methods inJorge Bejar2012-05-032-2/+8
| | | | sprockets/helpers/rails_helper.rb discard duplicated assets
* Reset the request parameters after a constraints checkAndrew White2012-05-033-0/+25
| | | | | | | | | | | | | | A callable object passed as a constraint for a route may access the request parameters as part of its check. This causes the combined parameters hash to be cached in the environment hash. If the constraint fails then any subsequent access of the request parameters will be against that stale hash. To fix this we delete the cache after every call to `matches?`. This may have a negative performance impact if the contraint wraps a large number of routes as the parameters hash is built by merging GET, POST and path parameters. Fixes #2510. (cherry picked from commit 56030506563352944fed12a6bb4793bb2462094b)
* Merge pull request #6095 from route/assets_precompile_taskJeremy Kemper2012-05-022-0/+12
|\ | | | | Fix that asset precompile didn't respect the index.js convention. Fixes #3993.
| * Added test for assets:precompile for index filesDmitry Vorotilin2012-05-011-0/+9
| |
| * Fix #3993 assets:precompile task does not detect index filesDmitry Vorotilin2012-05-011-0/+3
| |
* | Deprecate remove_column with array as an argumentPiotr Sarnacki2012-05-023-0/+19
|/
* Correcting some confusion. Pago Pago is part of American Samoa, not Samoa.Dave Gerton2012-04-301-2/+4
| | | | | | | | | Further, Samoa and Tokelau jumped across the IDL from Dec 29 to Dec 31, 2011 switching from UTC-11 to UTC+13. American Samoa did not make the change and remains at UTC-11. Pacific/Fakaofo and Pacific/Apia are in TZInfo and documentation about the dateline change is in austalasia at IANA. (cherry picked from commit 1d08ce5f56e45fdee41bb16b2d8d4464bc69bf22)
* Failing test for #6034Piotr Sarnacki2012-04-301-0/+20
|
* fix the Flash middleware loading the session on every request (very ↵Will Bryant2012-04-302-7/+3
| | | | dangerous especially with Rack::Cache), it should only be loaded when the flash method is called
* Add note about using 303 See Other for XHR requests other than GET/POSTAndrew White2012-04-301-0/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | IE since version 6 and recently Chrome and Firefox have started following 302 redirects from XHR requests other than GET/POST using the original request method. This can lead to DELETE requests being redirected amongst other things. Although it doesn't directly affect the Rails framework since it doesn't return a 302 redirect to any non-GET/POST request a note has been added to raise awareness of the issue. Some references: Original article from @technoweenie: http://techno-weenie.net/2011/8/19/ie9-deletes-stuff/ Hacker News discussion of the article: http://news.ycombinator.com/item?id=2903493 WebKit bug report: https://bugs.webkit.org/show_bug.cgi?id=46183 Firefox bug report and changeset: https://bugzilla.mozilla.org/show_bug.cgi?id=598304 https://hg.mozilla.org/mozilla-central/rev/9525d7e2d20d Chrome bug report: http://code.google.com/p/chromium/issues/detail?id=56373 HTTPbis bug report and changeset: http://trac.tools.ietf.org/wg/httpbis/trac/ticket/160 http://trac.tools.ietf.org/wg/httpbis/trac/changeset/1428 Roy T. Fielding's history of the issue: http://ftp.ics.uci.edu/pub/ietf/http/hypermail/1997q3/0611.html Automated browser tests for the issue: http://www.mnot.net/javascript/xmlhttprequest/ Fixes #4144 (cherry picked from commit 24f143789a8989f3bccde14ff28067de25cafd87)
* Merge pull request #5922 from rafaelfranca/deprecate_javascript_helpersJeremy Kemper2012-04-299-18/+40
|\ | | | | Deprecate link_to_function and button_to_function
| * Add release data to Rails 3.2.3Rafael Mendonça França2012-04-305-5/+5
| |
| * Add CHANGELOG entry.Rafael Mendonça França2012-04-301-1/+6
| | | | | | | | | | Closes #5886 Closes #3093
| * Deprecate link_to_function and button_to_function helpersRafael Mendonça França2012-04-302-12/+28
| |
| * Add missing require when helpers are used in isolationRafael Mendonça França2012-04-301-0/+1
|/
* Restore interpolation of path option in redirect routesAndrew White2012-04-292-1/+50
|
* Add missing require from b608cdd64c95d0d16eb98d86562e22f3b01be9e3Andrew White2012-04-291-0/+1
|
* Escape interpolated params when redirecting - fixes #5688Andrew White2012-04-292-1/+34
|
* Merge pull request #6051 from rafaelfranca/fix_buildJosé Valim2012-04-292-3/+3
|\ | | | | Fix the build
| * Remove warning of unused variableRafael Mendonça França2012-04-291-1/+1
| |
| * Fix the build.Rafael Mendonça França2012-04-291-2/+2
|/ | | | | * The method for persisted records in 3-2-branch is 'PUT' * size is generated by default in inputs
* Don't convert params if the request isn't HTML - fixes #5341Andrew White2012-04-292-8/+58
| | | | | | | | (cherry picked from commit 7a80b69e00f68e673c6ceb5cc684aa9196ed3d9f) Conflicts: actionpack/test/controller/test_test.rb
* Lazy load `default_form_builder` if it's passed as a stringPiotr Sarnacki2012-04-284-2/+27
| | | | closes #3341
* Allow Thor 0.15 and 1.0, to be released shortlytomhuda2012-04-281-1/+5
|
* Merge pull request #6038 from arunagw/warning_removed_3-2-stablePiotr Sarnacki2012-04-281-1/+0
|\ | | | | Warning removed 3 2 stable
| * Remove circular require of time/zonesOscar Del Ben2012-04-291-1/+0
|/
* Merge pull request #6022 from sikachu/3-2-stable-record_tag_backportAaron Patterson2012-04-272-2/+8
|\ | | | | Do not mutate options hash
| * Do not mutate options hashCarlos Antonio da Silva2012-04-272-2/+8
|/
* Merge pull request #4528 from j-manu/log-tailer-fixAaron Patterson2012-04-271-3/+7
| | | | Fix for log tailer when the log file doesn't exist.
* fix number_to_human docs [ci skip]Vijay Dev2012-04-261-2/+2
|
* Merge pull request #5971 from carlosantoniodasilva/fix-build-3-2Jeremy Kemper2012-04-241-1/+1
|\ | | | | Add extra order clause to fix failing test on Ruby 1.8.7
| * Add extra order clause to fix failing test on Ruby 1.8.7Carlos Antonio da Silva2012-04-241-1/+1
|/
* Merge pull request #5968 from sikachu/3-2-stable-backportJeremy Kemper2012-04-242-3/+9
|\ | | | | Backport workarounds for Mocha behavior changes.
| * Fix build. It seems that the Mocha's behavior were changed.kennyj2012-04-242-3/+9
|/
* Merge pull request #5946 from sikachu/3-2-stable-fix-merge-conflictSantiago Pastorino2012-04-231-9/+11
|\ | | | | Fix broken test from the earlier merge conflict
| * Fix broken test from the earlier merge conflictPrem Sichanugrist2012-04-231-9/+11
|/ | | | Seriously people, please run the test before submitting pull request.
* Merge pull request #5866 from tiegz/minor_fixes_3-2-stableAaron Patterson2012-04-232-42/+27
|\ | | | | Catch nil.to_sym errors in partial_renderer, and raise ArgumentError instead
| * converting some tests to assert_raises, and DRY'ing retrieve_variable changesTieg Zaharia2012-04-162-47/+23
| | | | | | | | | | | | Conflicts: actionpack/test/template/render_test.rb
| * catch nil.to_sym errors in partial_renderer, and raise ArgumentError insteadTieg Zaharia2012-04-162-1/+10
| |
* | Merge pull request #5919 from joevandyk/rake-dsl-fixJeremy Kemper2012-04-231-1/+1
|\ \ | | | | | | Only include Rake::DSL if it's defined (Rake >= 0.9)
| * | Only include Rake::DSL if it's defined.Joe Van Dyk2012-04-211-1/+1
| | | | | | | | | | | | rake < 0.9 doesn't define Rake::DSL.
* | | Merge pull request #5936 from arunagw/other_readme_fixesVijay Dev2012-04-236-6/+6
|\ \ \ | | | | | | | | Updated other README to point 3-2-stable
| * | | Updated other README to point 3-2-stableArun Agrawal2012-04-236-6/+6
| | | |
* | | | Merge pull request #5935 from arunagw/readme_fixes_3-2-stableVijay Dev2012-04-231-1/+1
|\ \ \ \ | |/ / / |/| | | As ARes is removed from master then 3-2-stable URL should be in README.
| * | | ARes URL should be pointed to 3-2-stable tree not master.Arun Agrawal2012-04-231-1/+1
|/ / /
* | | Merge pull request #5896 from sferik/revert_5861Jeremy Kemper2012-04-212-4/+18
|\ \ \ | |/ / |/| | Revert #5861. Feature-detect which MultiJson API to use.
| * | Use `Object#respond_to?` to determine which MultiJson API to useErik Michaels-Ober2012-04-181-3/+17
| | |
| * | Revert "multi_json is restricted to < 1.3 "Erik Michaels-Ober2012-04-181-1/+1
| | | | | | | | | | | | This reverts commit 9b14e3ff80ee4044cfd89a11effcb5f52eaf888b.
* | | Be sure to correctly fetch PK name from MySQL even if the PK has some custom ↵Akira Matsuda2012-04-192-1/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | option Backports #5900 Conflicts: activerecord/lib/active_record/connection_adapters/abstract_mysql_adapter.rb activerecord/test/cases/adapters/mysql/mysql_adapter_test.rb
* | | Merge pull request #5898 from benedikt/3-2-stableJeremy Kemper2012-04-196-8/+40
|\ \ \ | |/ / |/| | Readds the fix for #5667 and back ports the regression fix from #5718