Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Merge pull request #9032 from firmhouse/head-breaks-csrf | Santiago Pastorino | 2013-01-28 | 1 | -2/+2 |
|\ | | | | | Make HEAD work / convert to GET once more | ||||
| * | Added request.head? to forgery protection code | Michiel Sikkes | 2013-01-22 | 1 | -2/+2 |
| | | |||||
* | | Use Encoding::UTF_8 constant :do_not_litter: | Akira Matsuda | 2013-01-28 | 4 | -5/+5 |
| | | |||||
* | | Use already defined Encoding constants rather than creating one-trip Strings | Akira Matsuda | 2013-01-28 | 2 | -2/+2 |
| | | |||||
* | | add fetch to CookieJar | Aaron Patterson | 2013-01-27 | 1 | -0/+4 |
| | | |||||
* | | Merge remote-tracking branch 'docrails/master' | Xavier Noria | 2013-01-26 | 6 | -4/+98 |
|\ \ | | | | | | | | | | | | | | | | Conflicts: actionpack/lib/action_view/helpers/form_options_helper.rb guides/code/getting_started/app/controllers/comments_controller.rb | ||||
| * | | generic pass before merging docrails | Xavier Noria | 2013-01-26 | 4 | -422/+62 |
| | | | |||||
| * | | :scissors: "raise" duplication | Akira Matsuda | 2013-01-26 | 1 | -2/+1 |
| | | | |||||
| * | | Review #translate docs [ci skip] | Carlos Antonio da Silva | 2013-01-20 | 1 | -8/+7 |
| | | | |||||
| * | | Add documentation for abstract controller #translate and #localize method. | Jens Bissinger | 2013-01-20 | 1 | -0/+10 |
| | | | |||||
| * | | Add documentation to ActionDispatch::Response | Matthew Stopa | 2013-01-20 | 1 | -0/+7 |
| | | | |||||
| * | | fix broken examples format in image_tag helper [ci skip] | Francesco Rodriguez | 2013-01-18 | 1 | -0/+2 |
| | | | |||||
| * | | repeating documentation for option form helpers | Thiago Pinto | 2013-01-17 | 1 | -0/+286 |
| | | | |||||
| * | | repeating documentation for date and time form helpers | Thiago Pinto | 2013-01-17 | 1 | -0/+148 |
| | | | |||||
| * | | Fix typo introduced in 0004ca3a | Carlos Antonio da Silva | 2013-01-16 | 1 | -1/+1 |
| | | | | | | | | | | | | [ci skip] | ||||
| * | | More documentation for ActionDispatch::Response | Matthew Stopa | 2013-01-16 | 1 | -1/+3 |
| | | | | | | | | | | | | [ci skip] | ||||
| * | | Document ActionDispatch::Response#body method | Matthew Stopa | 2013-01-16 | 1 | -0/+1 |
| | | | |||||
* | | | Add keys/values methods to TestSession | Carlos Antonio da Silva | 2013-01-25 | 1 | -0/+8 |
| | | | | | | | | | | | | Bring back the same API we have with Request::Session. | ||||
* | | | Integrate Action Pack with Rack 1.5 | Carlos Antonio da Silva | 2013-01-25 | 3 | -4/+22 |
| | | | | | | | | | | | | | | | | | | All ActionPack and Railties tests are passing. Closes #8891. [Carlos Antonio da Silva + Santiago Pastorino] | ||||
* | | | use the helpers list rather than getting the methods from the module | Aaron Patterson | 2013-01-24 | 1 | -1/+1 |
| | | | |||||
* | | | module_eval is not necessary here | Aaron Patterson | 2013-01-24 | 1 | -3/+1 |
| | | | |||||
* | | | don't need to eval everything | Aaron Patterson | 2013-01-24 | 1 | -1/+1 |
| | | | |||||
* | | | ActionDispatch::Http::UploadedFile is a permitted scalar [Closes #9051] | Xavier Noria | 2013-01-23 | 1 | -2/+5 |
| |/ |/| | |||||
* | | Only check for unpermmited parameters if | Rafael Mendonça França | 2013-01-22 | 1 | -1/+1 |
| | | | | | | | | action_on_unpermitted_parameters is present | ||||
* | | avoid creating an object in every call | Xavier Noria | 2013-01-22 | 1 | -1/+2 |
| | | | | | | | | This was a suggestion of @carlosantoniodasilva, thanks! | ||||
* | | Capture block so content won't leak. | José Mota | 2013-01-21 | 2 | -4/+4 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | The [following pull request](https://github.com/rails/rails/pull/8916) fixed the block being passed to the appropriate helper method. However, the content being passed into the block is generating repeated markup on the page due to some weird ERb evaluation. This commit tries to capture the block's generated output so the page isn't flooded with markup. [Rafael França + José Mota] Closes #8936 | ||||
* | | Fix asset_path in mounted engine | Piotr Sarnacki | 2013-01-21 | 1 | -2/+1 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | Historically serving assets from a mountable engine could be achieved by running ActionDispatch::Static as a part of engine middleware stack or to copy assets prefixed with an engine name. After introduction of assets pipeline this is not needed as all of the assets are served or compiled into main application's assets. This commit removes the obsolete line making asset_path always generate paths relative to the root or config.relative_url_root if it's set. (closes #8119) | ||||
* | | Duplicate possible frozen string from route | Andrew White | 2013-01-21 | 1 | -2/+3 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | Ruby 1.9 freezes Hash string keys by default so where a route is defined like this: get 'search' => 'search' then the Mapper will derive the action from the key. This blows up later when the action is added to the parameters hash and the encoding is forced. Closes #3429 | ||||
* | | In Browser Path Matching with Javascript | schneems | 2013-01-20 | 3 | -12/+121 |
| | | | | | | | | | | | | | | | | | | | | | | When debugging routes ,it can sometimes be difficult to understand exactly how the paths are matched. This PR adds a JS based path matching widget to the `/rails/info/routes` output. You can enter in a path, and it will tell you which of the routes that path matches, while preserving order (top match wins). The matching widget in action:  Prior to this PR the only way to check matching paths is via mental math, or typing in a path in the url bar and seeing where it goes. This feature will be an invaluable debugging tool by dramatically decreasing the time needed to check a path match. ATP actionpack | ||||
* | | Refactor grep call to remove .each | Carlos Antonio da Silva | 2013-01-20 | 1 | -1/+1 |
| | | | | | | | | Grep already yields the matching keys to the given block. | ||||
* | | Use 1.9 hash style in docs/comments [ci skip] | Carlos Antonio da Silva | 2013-01-20 | 1 | -3/+3 |
| | | |||||
* | | Removing warning: shadowing outer local variable | Arun Agrawal | 2013-01-20 | 1 | -3/+3 |
| | | |||||
* | | strong parameters filters permitted scalars | Xavier Noria | 2013-01-20 | 1 | -22/+92 |
| | | |||||
* | | Merge pull request #8978 from chrismcg/remove_i18n_symbol_dependency | Aaron Patterson | 2013-01-19 | 1 | -0/+1 |
|\ \ | | | | | | | Remove i18n symbol dependency | ||||
| * | | Don't change the original i18n data | Chris McGrath | 2013-01-17 | 1 | -1/+1 |
| | | | |||||
| * | | Remove i18n symbol dependency | Chris McGrath | 2013-01-17 | 1 | -0/+1 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | date.order is the only key in rails i18n that is required to be a symbol. This patch allows for symbols or strings which means: * No requirement for symbol type in .yml files. A future YAML.safe_load wouldn't need to load symbols * Rails could actually use json rather than yml as the backend | ||||
* | | | Restore and adapt the implementation reverted at | Rafael Mendonça França | 2013-01-19 | 2 | -29/+58 |
| | | | | | | | | | | | | | | | | | | https://github.com/rails/rails/commit/cc1c3c5be061e7572018f734e5239750ab449e3f Now instead of raise, we log by default in development and test | ||||
* | | | Added ability to raise or log on unpermitted params. | Thomas Drake-Brockman | 2013-01-20 | 2 | -6/+34 |
| | | | |||||
* | | | Merge pull request #8977 from Soylent/master | Aaron Patterson | 2013-01-18 | 1 | -1/+1 |
|\ \ \ | | | | | | | | | Resovled issue rails#7774 | ||||
| * | | | Fix ActionDispatch::Request#formats when HTTP_ACCEPT header is empty string | Konstantin Papkovskiy | 2013-01-17 | 1 | -1/+1 |
| | | | | |||||
* | | | | Fix syntax error :grin: | Guillermo Iguaran | 2013-01-18 | 1 | -1/+1 |
| | | | | |||||
* | | | | Remove "Application" section title from routes | schneems | 2013-01-18 | 1 | -6/+13 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This PR standardizes the output of the HTML and console based routes to not include the title for "Application Routes" those that are defined by the application. Instead only routes defined in engines get any special treatment. Based on this conversation: https://github.com/rails/rails/commit/af5c0fd85fce1adb311083dd1ecf96432ee8caa3#commitcomment-2458823 ATP actionpack /cc @carlosantoniodasilva | ||||
* | | | | Revert "Merge pull request #8989 from robertomiranda/use-rails-4-find-by" | Guillermo Iguaran | 2013-01-18 | 3 | -4/+4 |
| | | | | | | | | | | | | | | | | | | | | This reverts commit 637a7d9d357a0f3f725b0548282ca8c5e7d4af4a, reversing changes made to 5937bd02dee112646469848d7fe8a8bfcef5b4c1. | ||||
* | | | | User Rails 4 find_by | robertomiranda | 2013-01-18 | 3 | -4/+4 |
|/ / / | |||||
* / / | Remove useless || operation | Carlos Antonio da Silva | 2013-01-17 | 1 | -2/+1 |
|/ / | |||||
* | | Deprecate direct calls to AC::RecordIdentifier.dom_id and dom_class | Carlos Antonio da Silva | 2013-01-16 | 1 | -5/+17 |
| | | | | | | | | Also add some generic tests to ensure they're properly deprecated. | ||||
* | | Fix date_select :selected option so you can pass it nil | Colin Burn-Murdoch | 2013-01-16 | 1 | -1/+1 |
| | | |||||
* | | Merge pull request #8958 from balexand/strong_parameters_exception_handling | Rafael Mendonça França | 2013-01-15 | 2 | -7/+2 |
|\ \ | | | | | | | Strong parameters exception handling | ||||
| * | | strong parameters exception handling | Brian Alexander | 2013-01-15 | 2 | -7/+2 |
| | | | |||||
* | | | Remove header bloat introduced by BestStandardsSupport middleware | Edward Anderson | 2013-01-15 | 1 | -1/+3 |
|/ / | | | | | | | The same headers were being duplicated on every request. |