aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib
Commit message (Collapse)AuthorAgeFilesLines
...
| * | Merge pull request #15061 from tgxworld/remove_unnecessary_includeMatthew Draper2014-05-291-6/+0
| |\ \ | | | | | | | | Remove unnecessary include for integration tests.
| | * | Remove unnecessary include for integration tests.Guo Xiang Tan2014-05-111-6/+0
| | | |
| * | | Add missing `:param` option from the docs for Mapper#match [ci skip]Zachary Scott2014-05-291-0/+6
| | | | | | | | | | | | | | | | Option discovered by @zackperdue in #14741, implemented in #5581.
* | | | "controllers" should be a valid path nameAaron Patterson2014-05-291-1/+1
| | | |
* | | | controllers with slash names are also not supported, so we can reuse the messageAaron Patterson2014-05-281-6/+2
| | | |
* | | | only validate controllersAaron Patterson2014-05-281-15/+17
| | | |
* | | | golf down a bitAaron Patterson2014-05-281-8/+4
| | | |
* | | | only error handling between controller and action is the sameAaron Patterson2014-05-281-18/+16
| | | |
* | | | move nil check to a method that yields to a block if the value is not nilAaron Patterson2014-05-281-17/+16
| | | |
* | | | translate action / controller to the desired objectAaron Patterson2014-05-281-12/+11
| | | |
* | | | only one nil check on the action variableAaron Patterson2014-05-281-10/+8
| | | |
* | | | change to case / when on typesAaron Patterson2014-05-281-8/+7
| | | |
* | | | only do one nil check against the controllerAaron Patterson2014-05-281-9/+9
| | | |
* | | | extract controller and action parsing to a methodAaron Patterson2014-05-281-19/+26
| | | |
* | | | we don't need the call to presence. that is my present, to you!Aaron Patterson2014-05-281-1/+1
| | | |
* | | | swtich to returning early if to responds to callAaron Patterson2014-05-281-35/+32
| | | |
* | | | return early if we have a valid controller nameAaron Patterson2014-05-281-5/+5
| | | |
* | | | trade 2 is_a? checks for a nil checkAaron Patterson2014-05-281-2/+4
| | | |
* | | | invert logic to remove nil? and exclude? checks (use ruby rather than AS ↵Aaron Patterson2014-05-281-6/+6
| | | | | | | | | | | | | | | | when possible
* | | | fewer blank? callsAaron Patterson2014-05-281-4/+4
| | | |
* | | | reduce action.blank? callsAaron Patterson2014-05-281-4/+6
| | | |
* | | | reduce blank? checksAaron Patterson2014-05-281-4/+9
| | | |
* | | | extract controller checks to methodsAaron Patterson2014-05-281-19/+26
| | | |
* | | | set defaults at the top so we can avoid the ||= testAaron Patterson2014-05-281-6/+9
|/ / /
* | | Deprecate all *_filter callbacks in favor of *_action callbacksRafael Mendonça França2014-05-271-20/+24
| | | | | | | | | | | | | | | This is the continuation of the work started at 9d62e04838f01f5589fa50b0baa480d60c815e2c
* | | rm dead codeAaron Patterson2014-05-271-5/+0
| | |
* | | PARAMETERS_KEY is only used in the request, so move the constant thereAaron Patterson2014-05-272-4/+4
| | |
* | | Merge branch 'constraints'Aaron Patterson2014-05-2710-73/+79
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * constraints: rm reset_parameters because we automatically do it from 9ca4839a move path_parameter encoding check to the request object dispatcher doesn't need `call` anymore call `serve` with the request on dispatchers constraints class does not need the request class anymore give all endpoints a superclass skip the build business if the stack is empty stop hardcoding path_parameters and get it from the request we do not need to cache rack_app a redirect is not a dispatcher by definition, so eliminate test push is_a check up to where the Constraints object is allocated pass the request object to the application pass a request to `matches?` so we can avoid creating excess requests nothing is passed to `rack_app` anymore, so rm the params one fewer is_a check Constraints#app should never return another Constraints object, so switch to if statement eliminate dispatcher is_a checks push is_a?(Dispatcher) check in to one place Always construct route objects with Constraint objects Conflicts: actionpack/lib/action_controller/metal.rb
| * | | rm reset_parameters because we automatically do it from 9ca4839aAaron Patterson2014-05-272-6/+0
| | | |
| * | | move path_parameter encoding check to the request objectAaron Patterson2014-05-273-18/+13
| | | |
| * | | dispatcher doesn't need `call` anymoreAaron Patterson2014-05-271-4/+0
| | | |
| * | | call `serve` with the request on dispatchersAaron Patterson2014-05-271-9/+10
| | | |
| * | | constraints class does not need the request class anymoreAaron Patterson2014-05-271-4/+4
| | | |
| * | | give all endpoints a superclassAaron Patterson2014-05-274-22/+40
| | | |
| * | | skip the build business if the stack is emptyAaron Patterson2014-05-261-2/+6
| | | |
| * | | stop hardcoding path_parameters and get it from the requestAaron Patterson2014-05-261-1/+2
| | | |
| * | | we do not need to cache rack_appAaron Patterson2014-05-261-9/+3
| | | |
| * | | a redirect is not a dispatcher by definition, so eliminate testAaron Patterson2014-05-261-1/+1
| | | |
| * | | push is_a check up to where the Constraints object is allocatedAaron Patterson2014-05-262-6/+11
| | | |
| * | | pass the request object to the applicationAaron Patterson2014-05-252-4/+3
| | | |
| * | | pass a request to `matches?` so we can avoid creating excess requestsAaron Patterson2014-05-252-7/+6
| | | |
| * | | nothing is passed to `rack_app` anymore, so rm the paramsAaron Patterson2014-05-251-1/+1
| | | |
| * | | one fewer is_a checkAaron Patterson2014-05-251-4/+3
| | | |
| * | | Constraints#app should never return another Constraints object, so switch to ↵Aaron Patterson2014-05-251-1/+1
| | | | | | | | | | | | | | | | if statement
| * | | eliminate dispatcher is_a checksAaron Patterson2014-05-241-8/+14
| | | |
| * | | push is_a?(Dispatcher) check in to one placeAaron Patterson2014-05-243-14/+11
| | | |
| * | | Always construct route objects with Constraint objectsAaron Patterson2014-05-243-9/+7
| | | |
* | | | Merge pull request #15321 from ↵Santiago Pastorino2014-05-271-1/+4
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | tgxworld/only_find_routes_as_heads_for_head_request Call get_routes_as_head only on HEAD requests.
| * | | | Call get_routes_as_head only on HEAD requests.Guo Xiang Tan2014-05-251-1/+4
| | |_|/ | |/| |
* | | | Escape user input before showing in the page.Rafael Mendonça França2014-05-261-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | This is not a security issue since this page is not present in production and user have to type something in the field but is better to escape the input.
generated by cgit v1.2.3 (git 2.25.1) at 2024-11-16 01:01:01 +0000