aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | Remove ruby/shimSantiago Pastorino2012-05-112-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | require 'action_view' is now much faster
* | | | | | | Copy literal route constraints to defaults - fixes #3571 and #6224.Andrew White2012-05-114-3/+69
| | | | | | |
* | | | | | | Don't ignore nil positional arguments for url helpers - fixes #6196.Andrew White2012-05-102-1/+36
| | | | | | |
* | | | | | | Refactor the handling of default_url_options in integration testsAndrew White2012-05-102-7/+104
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit improves the handling of default_url_options in integration tests by making behave closer to how a real application operates. Specifically the following issues have been addressed: * Options specified in routes.rb are used (fixes #546) * Options specified in controllers are used * Request parameters are recalled correctly * Tests can override default_url_options directly
* | | | | | | remove duplicate usage of Rack::Utils.status_code in ↵Rafael Magana2012-05-101-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | LogSubscriber#process_action
* | | | | | | removing key argument from run_callbacks - fix buildFrancesco Rodriguez2012-05-101-1/+1
| |_|_|/ / / |/| | | | |
* | | | | | Refactor Generator class to not rely on in-place editing the controllerAndrew White2012-05-092-11/+16
| | | | | |
* | | | | | Fix bug when url_for changes controller.Nikita Beloglazov2012-05-092-2/+11
| | | | | |
* | | | | | fix a couple of formatting issues [ci skip]Vijay Dev2012-05-091-2/+4
| | | | | |
* | | | | | update some examples in asset tag helper docs [ci skip]Vijay Dev2012-05-091-15/+15
| | | | | |
* | | | | | remove redundant and stray line [ci skip]Vijay Dev2012-05-091-2/+0
| |/ / / / |/| | | |
* | | | | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-05-085-105/+180
|\ \ \ \ \
| * | | | | Fix NumberHelper options wrapping to prevent verbatim blocks being rendered ↵Mark J. Titorenko2012-05-081-92/+161
| | | | | | | | | | | | | | | | | | | | | | | | instead of line continuations. While I'm at it, wrap long comment lines consistently.
| * | | | | Add example for image_submit_tag with confirmation. [ci skip]Edward Tsech2012-05-081-0/+3
| | | | | |
| * | | | | Add annotaion for class_eval in AbstractController#helper_method [ci skip]Edward Tsech2012-05-081-3/+3
| | | | | |
| * | | | | Improve readability of metaprogramming annotations at AbstractController ↵Edward Tsech2012-05-081-10/+10
| | | | | | | | | | | | | | | | | | | | | | | | callbacks.
| * | | | | Added reference to I18n.localize, as discussed here:Ernesto Tagwerker2012-05-041-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | https://github.com/rails/rails/issues/4027
* | | | | | Add failing test re #3436 which demonstrates content_type is not respected ↵Kunal Shah2012-05-081-0/+21
| | | | | | | | | | | | | | | | | | | | | | | | when using the :head method/shortcut
* | | | | | If content_type is explicitly passed to the :head method use the value or ↵Kunal Shah2012-05-081-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | fallback
* | | | | | Add passing tests for generating URLs with nested SCRIPT_NAMEsJeremy Kemper2012-05-072-3/+17
| | | | | |
* | | | | | Merge pull request #5368 from andhapp/remove-max-staleJosé Valim2012-05-071-4/+4
|\ \ \ \ \ \ | | | | | | | | | | | | | | Minor test improvement
| * | | | | | Removed max-stale from the tests since it's a request cache-control ↵Anuj Dutta2012-03-101-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | directive, just for clarity sake.
* | | | | | | Merge pull request #6182 from shaliko/change_unless_else_into_if_elseJosé Valim2012-05-071-3/+3
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Change unless + else into if + else
| * | | | | | | Change unless + else into if + elseShaliko Usubov2012-05-061-3/+3
| | | | | | | |
* | | | | | | | Fix that optimized named routes should also work as singleton methods on the ↵Jeremy Kemper2012-05-062-1/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | url_helpers module
* | | | | | | | doc edits [ci skip]Vijay Dev2012-05-061-3/+2
| | | | | | | |
* | | | | | | | Merge pull request #5924 from cjolly/signed-cookies-docsVijay Dev2012-05-061-3/+5
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | Improve signed cookies documentation
| * | | | | | | Improve signed cookies documentationChad Jolly2012-04-211-3/+5
| | | | | | | |
* | | | | | | | Fix typo in submit_tag helper documentationAlex Soulim2012-05-061-1/+1
| | | | | | | |
* | | | | | | | Update actionpack/CHANGELOG.mdJosé Valim2012-05-061-1/+5
| | | | | | | |
* | | | | | | | remove .new from raise ActionController::UnknownFormatSteven Soroka2012-05-061-1/+1
| | | | | | | |
* | | | | | | | added an integration test that checks ActionController::UnknownFormat ↵Steven Soroka2012-05-061-0/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | renders 406 :not_acceptable
* | | | | | | | Raise a rescuable exception when Rails doesn't know what to do with the ↵Steven Soroka2012-05-064-15/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | format, rather than responding with a head :not_acceptable (406)
* | | | | | | | check checkboxes with array of strings as :checked optionVasiliy Ermolovich2012-05-052-1/+10
| |_|_|_|/ / / |/| | | | | |
* | | | | | | Try to convert object passed to debug_hash to hashPiotr Sarnacki2012-05-051-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | SessionStore was recently changed to delegate to hash object instead of inherit from it. Since we don't want to extend SessionStore with every method implemented in Hash, it's better to just convert any object passed to debug_hash (which is also better as we don't require to pass Hash instance there, it can be any object that can be converted to Hash).
* | | | | | | need to dup the default options so that mutations will not impact usAaron Patterson2012-05-041-1/+1
| | | | | | |
* | | | | | | make sure the superclass matches so load order does not matterAaron Patterson2012-05-042-2/+2
| | | | | | |
* | | | | | | Use Hash#fetch instead of has_key? checkCarlos Antonio da Silva2012-05-041-1/+1
| | | | | | |
* | | | | | | Merge pull request #4445 from nragaz/role_based_params_wrappingJosé Valim2012-05-042-4/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | specify a role for identifying accessible attributes when wrapping params
* | | | | | | Improve assert_template layout checkingAlexey Vakhov2012-05-042-3/+30
| | | | | | |
* | | | | | | Fix assert_template :layout => nil assertionAlexey Vakhov2012-05-042-2/+17
| | | | | | |
* | | | | | | Fix assert_template assertion with :layout optionAlexey Vakhov2012-05-043-1/+13
| | | | | | |
* | | | | | | Merge pull request #6160 from ↵Aaron Patterson2012-05-043-5/+13
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | carlosantoniodasilva/resource-route-canonical-action Force given path to http methods in mapper to skip canonical action checking
| * | | | | | | Force given path to http methods in mapper to skip canonical action checkingCarlos Antonio da Silva2012-05-043-5/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes the following scenario: resources :contacts do post 'new', action: 'new', on: :collection, as: :new end Where the /new path is not generated because it's considered a canonical action, part of the normal resource actions: new_contacts POST /contacts(.:format) contacts#new Fixes #2999
* | | | | | | | fix doc [ci skip]Vijay Dev2012-05-041-1/+1
| | | | | | | |
* | | | | | | | session creation methods to a moduleAaron Patterson2012-05-043-24/+11
| | | | | | | |
* | | | | | | | Merge pull request #6154 from shaliko/patch-1Vijay Dev2012-05-041-1/+6
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | Update documentation for button_to.
| * | | | | | | Update documentation for button_to.Shalva Usubov2012-05-041-1/+6
| | |_|/ / / / | |/| | | | |
* | | | | | | update invalid partial error message as per ↵Vijay Dev2012-05-041-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | 04202a3f8d66c0675eac41fbf63309f4be356e81
* | | | | | | Merge pull request #6159 from marcandre/fix_bad_partial_error_messVijay Dev2012-05-041-2/+2
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Adjust error message for partial names starting with an uppercase letter...