Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | | | | | `Http::Headers` directly modifies the passed environment. | Yves Senn | 2013-03-15 | 2 | -4/+4 | |
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The env hash passed to `Http::Headers#new` must be in env format. Also be aware that the passed hash is modified directly. docs and test-cases for setting headers/env in functional tests. Follow up to #9700. | |||||
* | | | | | Handle conditional get in live requests - this will prevent error when using ↵ | Bernard Potocki | 2013-03-14 | 1 | -0/+4 | |
| | | | | | | | | | | | | | | | | | | | | stale on live streams(issue #9636) | |||||
* | | | | | Add extra clarifying line in docs. [ci skip] | Steve Klabnik | 2013-03-13 | 1 | -0/+1 | |
| | | | | | ||||||
* | | | | | Fix docs: response -> request. | Steve Klabnik | 2013-03-13 | 1 | -2/+2 | |
| | | | | | | | | | | | | | | | | | | | | Even though I read it carefully, my brain tricked me. :cry: | |||||
* | | | | | Merge pull request #9698 from garethrees/request_documentation | Steve Klabnik | 2013-03-13 | 1 | -0/+16 | |
|\ \ \ \ \ | | | | | | | | | | | | | Add documentation to ActionDispatch::Request | |||||
| * | | | | | Add documentation to ActionDispatch::Request | Gareth Rees | 2013-03-13 | 1 | -0/+16 | |
| | | | | | | ||||||
* | | | | | | `Http::Headers` respects dotted env vars, symbols, headers with numbers. | Yves Senn | 2013-03-13 | 1 | -11/+6 | |
| | | | | | | ||||||
* | | | | | | allow headers and env to be passed in `IntegrationTest`. | Yves Senn | 2013-03-13 | 2 | -35/+49 | |
| | | | | | | | | | | | | | | | | | | | | | | | | Closes #6513. | |||||
* | | | | | | refactor, `Http::Headers` stores headers in env notation | Yves Senn | 2013-03-13 | 1 | -17/+21 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also: cleanup, use consistent syntax for `Http::Header` and test. | |||||
* | | | | | | Http::Headers respects headers that are not prefixed with HTTP_ | Yves Senn | 2013-03-13 | 1 | -1/+13 | |
| | | | | | | ||||||
* | | | | | | Change from each to each_value in http/parameters since we don't use key | Vipul A M | 2013-03-13 | 1 | -1/+1 | |
|/ / / / / | ||||||
* | | | | | Skip fetching path if the iteration is going to be skipped | Carlos Antonio da Silva | 2013-03-11 | 1 | -2/+2 | |
| | | | | | ||||||
* | | | | | Merge pull request #9626 from dasch/dasch/instrument-strong-params | José Valim | 2013-03-09 | 2 | -1/+7 | |
|\ \ \ \ \ | |_|_|/ / |/| | | | | Use AS::Notifications to instrument Strong Params | |||||
| * | | | | Use the instrumentation framework to instrument Strong Params | Daniel Schierbeck | 2013-03-07 | 2 | -1/+7 | |
| | | | | | ||||||
* | | | | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2013-03-10 | 1 | -1/+1 | |
|\ \ \ \ \ | ||||||
| * | | | | | Update capture_helper.rb | Jess Brown | 2013-03-06 | 1 | -1/+1 | |
| | |/ / / | |/| | | | | | | | | if there's content for the right column, then we need the two-column class, if not the one-column | |||||
* | | | | | ensure response.stream is closed | Sam Ruby | 2013-03-09 | 1 | -0/+1 | |
| | | | | | ||||||
* | | | | | Fix incorrectly appended square brackets to a multiple select box | Olek Janiszewski | 2013-03-08 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If an explicit name has been given and it already ends with "[]" Before: select(:category, [], {}, multiple: true, name: "post[category][]") # => <select name="post[category][][]" ...> After: select(:category, [], {}, multiple: true, name: "post[category][]") # => <select name="post[category][]" ...> | |||||
* | | | | | Fix hash spaces and use 1.9 style hash [ci skip] | Carlos Antonio da Silva | 2013-03-07 | 1 | -1/+1 | |
| | | | | | ||||||
* | | | | | Merge pull request #9464 from jcoyne/assert_template_file | Rafael Mendonça França | 2013-03-07 | 3 | -2/+18 | |
|\ \ \ \ \ | | | | | | | | | | | | | Allow use of assert_template with the :file option. | |||||
| * | | | | | Allow use of assert_template with the :file option. | Justin Coyne | 2013-03-01 | 3 | -2/+18 | |
| | | | | | | | | | | | | | | | | | | | | | | | | This worked in Rails 3.2, but was a regression in 4.0.0.beta1 | |||||
* | | | | | | Merge pull request #9586 from robertomiranda/useless_tr | Rafael Mendonça França | 2013-03-06 | 1 | -2/+2 | |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | Change useless tr to gsub | |||||
| * | | | | | | Change useless tr to gsub | robertomiranda | 2013-03-06 | 1 | -2/+2 | |
| | |_|/ / / | |/| | | | | ||||||
* / | | | | | Typo in comment to ActionView::Resolver#find_all | RKushnir | 2013-03-06 | 1 | -1/+1 | |
|/ / / / / | ||||||
* | / / / | Use gsub intead of tr when the to_str is longer than the from_str | robertomiranda | 2013-03-05 | 1 | -1/+1 | |
| |/ / / |/| | | | ||||||
* | | | | change useless gsub to tr | robertomiranda | 2013-03-05 | 4 | -5/+5 | |
| | | | | ||||||
* | | | | Merge pull request #9561 from ↵ | Carlos Antonio da Silva | 2013-03-05 | 1 | -1/+2 | |
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | inossidabile/explicit_match_via_requirement_exception_message Routing match `via` option requirement exception message made more explicit | |||||
| * | | | | Routing match `via` option requirement exception message made more explicit | Boris Staal | 2013-03-05 | 1 | -1/+2 | |
| | |_|/ | |/| | | ||||||
* / | | | change useless gsub to delete | Benjamin Sullivan | 2013-03-05 | 1 | -1/+1 | |
|/ / / | ||||||
* | | | Fix formats on xhr requests when HTTP_ACCEPT is empty string | maximerety | 2013-03-04 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | Fix ActionDispatch::Request#formats on xhr requests when HTTP_ACCEPT header is empty string. About issue #7774, same fix as in commit bebb02f but for xhr requests. | |||||
* | | | Use custom visitor class for optimized url helpers | Andrew White | 2013-03-03 | 3 | -9/+13 | |
|/ / | | | | | | | | | | | | | | | Rather than trying to use gsub to remove the optional route segments, which will fail with nested optional segments, use a custom visitor class that returns a empty string for group nodes. Closes #9524 | |||||
* | | Merge pull request #9366 from killthekitten/fix-content_for-regression | Rafael Mendonça França | 2013-02-28 | 1 | -1/+1 | |
|\ \ | | | | | | | Fix for #9360 content_for regression | |||||
| * | | Fix #9360 regression in content_for | Nikolay Shebanov | 2013-02-25 | 1 | -1/+1 | |
| | | | ||||||
* | | | Remove redundant method | Andrew White | 2013-02-28 | 1 | -8/+0 | |
| | | | ||||||
* | | | `format: true` does not override existing format constraints. | Yves Senn | 2013-02-27 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | | Closes #9466. Passing `format: true` used to override the constraints: { format: /json/ } with `/.+/`. This patch only sets the format if there is no constraint present. | |||||
* | | | Improve docs for AbsC::Rendering | José Valim | 2013-02-27 | 1 | -3/+11 | |
| | | | ||||||
* | | | Merge pull request #9452 from frodsan/remove_private_attribute_warning | Steve Klabnik | 2013-02-26 | 1 | -1/+3 | |
|\ \ \ | | | | | | | | | remove private attribute? warning | |||||
| * | | | remove private attribute? warning | Francesco Rodriguez | 2013-02-26 | 1 | -1/+3 | |
| | | | | ||||||
* | | | | allow non-String default params in the router. | Yves Senn | 2013-02-26 | 1 | -0/+2 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes #9435. Skip valid encoding checks for non-String parameters that come from the matched route's defaults. | |||||
* | | | | the router allows String contraints. | Yves Senn | 2013-02-26 | 1 | -11/+17 | |
| | | | | | | | | | | | | | | | | Closes #9432. | |||||
* | | | | Merge pull request #9434 from zires/dev | Rafael Mendonça França | 2013-02-26 | 1 | -1/+1 | |
|\ \ \ \ | |/ / / |/| | | | Journey::Path::Pattern#new raise more meaningful exception message. | |||||
| * | | | Make ActionDispatch::Journey::Path::Pattern#new raise more meaningful ↵ | zires | 2013-02-27 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | exception message. | |||||
* | | | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2013-02-26 | 2 | -2/+2 | |
|\ \ \ \ | ||||||
| * | | | | Typo fix. | Carson McDonald | 2013-02-21 | 1 | -1/+1 | |
| | | | | | ||||||
| * | | | | improve grammar describing ActionDispatch::Cookies::CookieJar#delete | Weston Platter | 2013-02-19 | 1 | -1/+1 | |
| | | | | | ||||||
| * | | | | improve grammar describing ActionDispatch::Cookies::CookieJar#delete | Weston Platter | 2013-02-19 | 1 | -1/+1 | |
| | | | | | ||||||
* | | | | | Allow both a path and an option on root in config/routes | Sam Ruby | 2013-02-26 | 1 | -2/+9 | |
| |/ / / |/| | | | ||||||
* | | | | Preparing for 4.0.0.beta1 release | David Heinemeier Hansson | 2013-02-25 | 1 | -1/+1 | |
| | | | | ||||||
* | | | | Merge pull request #8530 from dasch/dependency-trackers | David Heinemeier Hansson | 2013-02-25 | 2 | -44/+98 | |
|\ \ \ \ | |_|_|/ |/| | | | Custom dependency trackers for the template digestor | |||||
| * | | | Register trackers on extensions, not handlers | Daniel Schierbeck | 2013-02-20 | 1 | -6/+11 | |
| | | | |