Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | documents the contract for the argument of AD::Routing::RouteSet#url_for | Xavier Noria | 2012-04-05 | 1 | -0/+1 |
| | |||||
* | Fix url_for when options is nil | Rafael Mendonça França | 2012-04-05 | 1 | -1/+1 |
| | | | | | RouteSet#url_for was modifying the options hash that belong to the caller | ||||
* | don't pass unnecessary argument | Sergey Nartimov | 2012-03-02 | 1 | -2/+2 |
| | |||||
* | Optimize url helpers. | Sergey Nartimov + José Valim | 2012-03-02 | 1 | -9/+8 |
| | |||||
* | Optimize path helpers. | José Valim | 2012-03-02 | 1 | -11/+54 |
| | |||||
* | Fix the assert_recognizes test method so that it works when there are | Matt Fawcett | 2012-02-24 | 1 | -0/+2 |
| | | | | constraints on the querystring. Issue #2781 | ||||
* | Avoid inspecting the whole route set, closes #1525 | José Valim | 2012-02-23 | 1 | -0/+6 |
| | |||||
* | Remove reference to rails_legacy_mapper, which isn't compatible with 3.2 #5022 | johndouthat | 2012-02-18 | 1 | -2/+1 |
| | |||||
* | Fix GH #4720. Routing problem with nested namespace and already camelized ↵ | kennyj | 2012-02-10 | 1 | -0/+5 |
| | | | | controller option. | ||||
* | Fixed force_ssl redirects to include original query params | Ryan McGeary | 2012-02-07 | 1 | -0/+1 |
| | | | | | | | | | | `ActionController.force_ssl` redirects http URLs to their https equivalent; however, when a URL contains a query string, the resulting redirect lacked the original query string. Conflicts: actionpack/lib/action_controller/metal/force_ssl.rb | ||||
* | Fix url_for method's behavior when it is called with :controller option ↵ | kennyj | 2012-02-07 | 1 | -1/+1 |
| | | | | | | which starts with "/" from multiple nested controller. Closes #3864 | ||||
* | Merge pull request #3775 from karevn/master | Aaron Patterson | 2012-01-24 | 1 | -1/+2 |
|\ | | | | | Please pull my changes - they fix a rare problem with tests framework | ||||
| * | Fix: when using subdomains and constraints, request params were not passed ↵ | karevn | 2011-11-28 | 1 | -1/+2 |
| | | | | | | | | to constraints callback | ||||
* | | Added custom regexps to ASTs that have literal nodes on either side of | Aaron Patterson | 2012-01-23 | 1 | -1/+20 |
| | | | | | | | | symbol nodes. Fixes #4585 | ||||
* | | We should finalize the routes as soon as possible and not wait for the ↵ | José Valim | 2012-01-06 | 1 | -2/+0 |
| | | | | | | | | controller to be loaded. | ||||
* | | Clean up routes inclusion and add some comments for the next soul that ↵ | José Valim | 2012-01-03 | 1 | -30/+37 |
| | | | | | | | | decides to adventure on this code. | ||||
* | | be explicit about where helpers are installed | Aaron Patterson | 2011-12-30 | 1 | -1/+1 |
| | | |||||
* | | stop using __send__ and just module eval in the extensions | Aaron Patterson | 2011-12-29 | 1 | -3/+2 |
| | | |||||
* | | we know the classes will be a list, so *tell* it to respond to each | Aaron Patterson | 2011-12-29 | 1 | -3/+3 |
| | | | | | | | | rather than casting | ||||
* | | don't need the begin / end | Aaron Patterson | 2011-12-29 | 1 | -21/+17 |
| | | |||||
* | | avoid extra method calls by just defining the delegate | Aaron Patterson | 2011-12-29 | 1 | -2/+3 |
| | | |||||
* | | don't use instance eval, just reference variables so we don't have to | Aaron Patterson | 2011-12-29 | 1 | -10/+11 |
| | | | | | | | | worry about "inspect" marshalling | ||||
* | | removing dead code. | Aaron Patterson | 2011-12-29 | 1 | -12/+3 |
| | | |||||
* | | decouple initialize from clear!. Initialize ivars in initialize, clear | Aaron Patterson | 2011-12-29 | 1 | -5/+5 |
| | | | | | | | | ivars in clear! | ||||
* | | modules don't have any instance methods | Aaron Patterson | 2011-12-29 | 1 | -3/+1 |
| | | |||||
* | | deprecate String#encoding_aware? and remove its usage | Sergey Nartimov | 2011-12-24 | 1 | -1/+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. | ||||
* | | Named Routes shouldn't override existing ones (currently route recognition ↵ | Andy Jeffries | 2011-12-05 | 1 | -1/+1 |
|/ | | | | goes with the earliest match, named routes use the latest match) | ||||
* | Fix typo in Dispatcher#controller documentation | Aviv Ben-Yosef | 2011-11-01 | 1 | -1/+1 |
| | |||||
* | Leave escaping up to Journey | Jeremy Kemper | 2011-10-13 | 1 | -4/+3 |
| | |||||
* | fix require | Aaron Patterson | 2011-10-06 | 1 | -1/+1 |
| | |||||
* | Fix named routes modifying arguments | Pawel Pierzchala | 2011-09-22 | 1 | -3/+4 |
| | |||||
* | removing backwards compatibility module | Aaron Patterson | 2011-09-12 | 1 | -5/+4 |
| | |||||
* | unfactor the Route class to private factory methods | Aaron Patterson | 2011-09-12 | 1 | -5/+42 |
| | |||||
* | reduce dependencies of external objects in the Route class | Aaron Patterson | 2011-09-12 | 1 | -3/+3 |
| | |||||
* | delete unused code, pass path explicitly to journey | Aaron Patterson | 2011-09-12 | 1 | -1/+2 |
| | |||||
* | all routes can be stored in the Journey Routes object | Aaron Patterson | 2011-09-12 | 1 | -5/+4 |
| | |||||
* | clear! does not need to be called from initialize | Aaron Patterson | 2011-09-12 | 1 | -4/+3 |
| | |||||
* | reuse the route collection and formatter by clearing them | Aaron Patterson | 2011-09-12 | 1 | -6/+10 |
| | |||||
* | Instantiate each part of our routing system: | Aaron Patterson | 2011-09-12 | 1 | -7/+8 |
| | | | | | | * A collection of Routes * The Router, which consults the collection of routes * A formatter that consults the collection of routes | ||||
* | stop freezing the routes | Aaron Patterson | 2011-09-12 | 1 | -1/+0 |
| | |||||
* | stop using a hash for parameterizing | Aaron Patterson | 2011-09-12 | 1 | -11/+9 |
| | |||||
* | Switching rack-mount to journey. | Aaron Patterson | 2011-09-12 | 1 | -1/+2 |
| | |||||
* | TODO fix explicitly loading exceptations, autoload removed | Vishnu Atrai | 2011-07-11 | 1 | -0/+1 |
| | |||||
* | prepend the assets route instead of appending, closes #436 | José Valim | 2011-05-08 | 1 | -3/+6 |
| | |||||
* | Use #remove_possible_method instead here | Sebastian Martinez | 2011-05-04 | 1 | -2/+3 |
| | |||||
* | raise an error if the old router draw method is used, along with a message ↵ | Josh Kalderimis | 2011-05-03 | 1 | -0/+5 |
| | | | | advising them to either upgrade their routes or add rails_legacy_mapper to their Gemfile | ||||
* | Explicitly define main_app proxy | Piotr Sarnacki | 2011-04-25 | 1 | -2/+1 |
| | |||||
* | stop being clever and just call methods on the Route object | Aaron Patterson | 2011-03-08 | 1 | -1/+1 |
| | |||||
* | Raise ArgumentError if route name is invalid [#6517 state:resolved] | Andrew White | 2011-03-06 | 1 | -0/+2 |
| |