| Commit message (Collapse) | Author | Age | Files | Lines | |
|---|---|---|---|---|---|
| * | Update incorrect backtick usage in RDoc to teletype | T.J. Schuck | 2017-11-22 | 1 | -1/+1 |
| | | | | [ci skip] | ||||
| * | Use tt in doc for ActionPack [ci skip] | Yoshiyuki Hirano | 2017-08-26 | 1 | -1/+1 |
| | | |||||
| * | Use frozen string literal in actionpack/ | Kir Shatrov | 2017-07-29 | 1 | -0/+2 |
| | | |||||
| * | Revert "Merge pull request #29540 from kirs/rubocop-frozen-string" | Matthew Draper | 2017-07-02 | 1 | -1/+0 |
| | | | | | | This reverts commit 3420a14590c0e6915d8b6c242887f74adb4120f9, reversing changes made to afb66a5a598ce4ac74ad84b125a5abf046dcf5aa. | ||||
| * | Enforce frozen string in Rubocop | Kir Shatrov | 2017-07-01 | 1 | -0/+1 |
| | | |||||
| * | Reuse the Parameters#to_h check in the routing helpers | Rafael Mendonça França | 2017-04-18 | 1 | -10/+3 |
| | | | | | | Since this protection is now in Parameters we can use it instead of reimplementing again. | ||||
| * | Merge pull request #28394 from shime/docs-action-dispatch | Xavier Noria | 2017-03-22 | 1 | -2/+2 |
| |\ | | | | | [docs] fix ActionDispatch documentation | ||||
| | * | [docs] fix ActionDispatch documentation | Hrvoje Šimić | 2017-03-13 | 1 | -2/+2 |
| | | | |||||
| * | | Always use original url_for when generating direct routes | Andrew White | 2017-03-17 | 1 | -0/+4 |
| | | | | | | | | | | | | | | | Action View overrides `url_for` in the view context to render paths by default when using `url_for` and this means that direct route helpers don't get the full url when called with the url suffix. To fix this always call the original `url_for`. | ||||
| * | | Add support for calling nested direct routes (#28462) | Andrew White | 2017-03-17 | 1 | -0/+4 |
| |/ | | | | | | | | | | | | | | | | | | | | Not all requirements can be expressed in terms of polymorphic url options so add a `route_for` method that allows calling another direct route (or regular named route) which a set of arguments, e.g: resources :buckets direct :recordable do |recording| route_for(:bucket, recording.bucket) end direct :threadable do |threadable| route_for(:recordable, threadable.parent) end This maintains the context of the original caller, e.g. threadable_path(threadable) # => /buckets/1 threadable_url(threadable) # => http://example.com/buckets/1 | ||||
| * | Privatize unneededly protected methods in Action Pack | Akira Matsuda | 2016-12-24 | 1 | -2/+4 |
| | | |||||
| * | normalizes indentation and whitespace across the project | Xavier Noria | 2016-08-06 | 1 | -12/+12 |
| | | |||||
| * | Add more info to insecure URL generation error | Derek Prior | 2016-04-26 | 1 | -1/+1 |
| | | | | | | | | | I always appreciate having a bit more information as to why something is now an error. We can use this error to tell people why what they were previously doing is insecure and give them hints on how to fix it. Signed-off-by: Kasper Timm Hansen <kaspth@gmail.com> | ||||
| * | Use a URL instead of an URL everywhere | Abhishek Jain | 2016-02-15 | 1 | -1/+1 |
| | | |||||
| * | Prevent ActionController::Parameters from being passed to url_for directly | Jean Boussier | 2015-12-15 | 1 | -1/+4 |
| | | |||||
| * | [ci skip] Change 'an URL' to 'a URL' as URL doesn't have a vowel sound | tanmay3011 | 2015-10-06 | 1 | -1/+1 |
| | | |||||
| * | `url_for` does not modify polymorphic options | Bernerd Schaefer | 2015-09-04 | 1 | -1/+2 |
| | | | | | | | | | | | | | | | | | | The `url_for` methods in `actionpack` and `actionview` now make a copy of the provided options before generating polymorphic paths or URLs. The bug in the previous behavior is most noticeable in a case like: url_options = [:new, :post, param: 'value'] if current_page?(url_options) css_class = "active" end link_to "New Post", url_options, class: css_class | ||||
| * | Make AC::Parameters not inherited from Hash | Prem Sichanugrist | 2015-07-15 | 1 | -0/+4 |
| | | | | | | | | | This is another take at #14384 as we decided to wait until `master` is targeting Rails 5.0. This commit is implementation-complete, as it guarantees that all the public methods on the hash-inherited Parameters are still working (based on test case). We can decide to follow-up later if we want to remove some methods out from Parameters. | ||||
| * | [ci skip] Improve the url_for documentation | Roque Pinel | 2015-07-01 | 1 | -3/+19 |
| | | | | | | | | | | Clarify the `url_for` usage in mailers. Re-add the documentation about `url_for` and Route's path parameters, first introduced by 5c4f1859970d06228a0b67cad6d4486c1526ef2a. This was reported on #15097 and until it is decided to deprecate it or not, I believe the documentation should exist. | ||||
| * | move _generate_paths_by_default to where it is used | Aaron Patterson | 2015-03-01 | 1 | -6/+0 |
| | | | | | | _generate_paths_by_default wasn't used in AD::Routing::UrlFor, so we should be able to move it where it is used in AV::Routing | ||||
| * | let mailer templates generate URLs by default [Xavier Noria, Richard Schneeman] | Xavier Noria | 2014-11-24 | 1 | -0/+6 |
| | | |||||
| * | remove some caching | Aaron Patterson | 2014-07-28 | 1 | -2/+1 |
| | | | | | | this caching doesn't increase performance, but does increase complexity. remove it for now and find better ways to speed up this code. | ||||
| * | pass the route name to define_url_helper | Aaron Patterson | 2014-07-17 | 1 | -1/+3 |
| | | | | | | this allows us to avoid 2 hash allocations per named helper definition, also we can avoid a `merge` and `delete`. | ||||
| * | use the helper method builder to construct helper methods | Aaron Patterson | 2014-05-12 | 1 | -1/+5 |
| | | |||||
| * | undo optimized calls until the builder object is finished | Aaron Patterson | 2014-05-12 | 1 | -1/+1 |
| | | |||||
| * | Take Hash with options inside Array in #url_for | Andrey Ognevsky | 2013-11-15 | 1 | -0/+2 |
| | | |||||
| * | Fix link_to return value | Earl J St Sauver | 2013-09-22 | 1 | -1/+1 |
| | | | | | | | The documentation is showing the link_to method as just returning the contents of the url_for method. It should be returning an "<a>" tag with the correct href set. | ||||
| * | Add script_name option description and example for #url_for options | Ivan Vanyak | 2012-12-17 | 1 | -0/+5 |
| | | |||||
| * | 1.9 hash syntax changes to docs | AvnerCohen | 2012-10-31 | 1 | -15/+15 |
| | | |||||
| * | removed : warning: `*' interpreted as argument prefix | Arun Agrawal | 2012-08-28 | 1 | -1/+1 |
| | | |||||
| * | Don't require action_dispatch in ActionView::UrlHelpers | Piotr Sarnacki | 2012-08-28 | 1 | -0/+2 |
| | | | | | | | | | | | ActionDispatch::Routing::UrlFor was always required in UrlHelpers. This was changed by splitting previous implementation of UrlHelper into 2 modules: ActionView::Helpers::UrlHelper and ActionView::Routing::UrlHelper. The former one keeps only basic implementation of url_for. The latter adds features that allow to use routes and is only required when url_helpers or mounted_helpers are required. | ||||
| * | fix typo in documentation | Anatoly Makarevich | 2012-07-26 | 1 | -1/+1 |
| | | |||||
| * | Removing ==Examples and last blank lines of docs from actionpack | Francesco Rodriguez | 2012-05-15 | 1 | -2/+0 |
| | | |||||
| * | Corrected the name of the module that should be included to get the url helpers. | Anuj Dutta | 2012-05-03 | 1 | -1/+1 |
| | | |||||
| * | Revert "Merge pull request #5750 from ahoward/master" | Santiago Pastorino | 2012-04-08 | 1 | -10/+1 |
| | | | | | | | | This reverts commit 174cf8b22064c3999dfa685014d5b31ac993bb54, reversing changes made to 7ecd6a731bd60665bc6de94095137f0b2c4ada2a. The reverted commit improved the performance in the wrong place, now we have added this 6ddbd1844a6fd6aca2992f5f75c9f605cf89808f improvement. | ||||
| * | Make url_for work when option is nil and default_url_options has | Rafael Mendonça França | 2012-04-05 | 1 | -1/+1 |
| | | | | | stringified keys | ||||
| * | url_for: avoid a nil check. unroll a loop. handle most common case 1st. | ara.t.howard | 2012-04-05 | 1 | -18/+13 |
| | | |||||
| * | carefully and quickly merge url_for options | ara.t.howard | 2012-04-04 | 1 | -1/+17 |
| | | |||||
| * | Optimize url helpers. | Sergey Nartimov + José Valim | 2012-03-02 | 1 | -2/+5 |
| | | |||||
| * | Optimize path helpers. | José Valim | 2012-03-02 | 1 | -0/+5 |
| | | |||||
| * | Clean up routes inclusion and add some comments for the next soul that ↵ | José Valim | 2012-01-03 | 1 | -11/+12 |
| | | | | | decides to adventure on this code. | ||||
| * | just add the writer rather than adding both and removing one | Aaron Patterson | 2011-12-29 | 1 | -2/+1 |
| | | |||||
| * | Move symbolize keys to the inner options as we can assume url_options will ↵ | José Valim | 2011-12-08 | 1 | -1/+1 |
| | | | | | be properly symbolized. | ||||
| * | what's -> that's | Uģis Ozols | 2011-11-25 | 1 | -1/+1 |
| | | |||||
| * | Remove unnecessary comment. | Uģis Ozols | 2011-11-25 | 1 | -1/+1 |
| | | |||||
| * | :subdomain can now be specified with a value of false in url_for, allowing ↵ | Kamil Sobieraj | 2011-10-04 | 1 | -2/+3 |
| | | | | | for subdomain(s) removal from the host during link generation. Closes #2025 | ||||
| * | doc fixes | Vijay Dev | 2011-08-28 | 1 | -4/+8 |
| | | |||||
| * | We actually don't need a reverse_merge! here. dup removed was giving error ↵ | Arun Agrawal | 2011-08-12 | 1 | -1/+1 |
| | | | | | with nil class. | ||||
| * | when calling url_for with a hash, additional (likely unwanted) values (such ↵ | Andrew Kaspick | 2011-08-11 | 1 | -1/+1 |
| | | | | | as :host) would be returned in the hash... calling #dup on the hash prevents this | ||||
| * | Remove extra white spaces on ActionPack docs. | Sebastian Martinez | 2011-05-23 | 1 | -1/+1 |
| | | |||||
 |
index : rails.git | |
| Mirror of official rails repo with custom fixes. | Harald Eilertsen |
| aboutsummaryrefslogtreecommitdiffstats |