aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack
Commit message (Collapse)AuthorAgeFilesLines
* Partially revert deprecation of *_filterRafael Mendonça França2014-06-033-36/+9
| | | | | | | | | | | We are going to deprecate only on Rails 5 to make easier plugin maintainers support different Rails versions. Right now we are only discouraging their usage. This reverts commit 6c5f43bab8206747a8591435b2aa0ff7051ad3de. Conflicts: actionpack/CHANGELOG.md
* Merge pull request #15349 from tgxworld/remove_duplicated_method_callRafael Mendonça França2014-06-033-7/+3
|\ | | | | Remove duplicated HashWithIndifferentAccess#with_indifferent_access.
| * Remove duplicated HashWithIndifferentAccess#with_indifferent_access.Guo Xiang Tan2014-05-263-7/+3
| |
* | oops! :bomb:Aaron Patterson2014-06-032-92/+0
| |
* | Routes specifying 'to:' must be a string that contains a "#" or a rackAaron Patterson2014-06-036-27/+149
| | | | | | | | | | application. Use of a symbol should be replaced with `action: symbol`. Use of a string without a "#" should be replaced with `controller: string`.
* | use the factory method to construct the mappingAaron Patterson2014-06-031-1/+1
| |
* | add tests for mixing :to and controller / actionAaron Patterson2014-06-031-0/+48
| |
* | push some options munging to an alternate constructorAaron Patterson2014-06-031-13/+19
| |
* | add tests for nested lambda constraintsAaron Patterson2014-06-031-0/+27
| |
* | move options_constraints tests next to each otherAaron Patterson2014-06-031-2/+2
| |
* | only do is_a checks on `formatted` onceAaron Patterson2014-06-031-9/+5
| |
* | push options_constraints processing upAaron Patterson2014-06-031-22/+24
| | | | | | | | | | this lets us reduce is_a checks on the options_constraints and push "callable constraints" verification to the right place.
* | remove options as an ivarAaron Patterson2014-06-031-6/+8
| |
* | only do Regexp === option onceAaron Patterson2014-06-031-3/+4
| |
* | return early from add_request_methodAaron Patterson2014-06-031-11/+11
| |
* | only loop over `options` once (hopefully)Aaron Patterson2014-06-031-13/+11
| |
* | only loop through constraints onceAaron Patterson2014-06-031-19/+15
| |
* | change defaults allocation to a one-linerAaron Patterson2014-06-031-3/+2
| |
* | no longer need the scope ivarAaron Patterson2014-06-031-2/+1
| |
* | only look up scope[:module] onceAaron Patterson2014-06-031-6/+6
| |
* | remove IGNORE_OPTIONSAaron Patterson2014-06-031-4/+8
| |
* | :only is never used in Mapping, so rm the keyAaron Patterson2014-06-031-1/+2
| |
* | remove :defaults from the IGNORE_OPTIONS listAaron Patterson2014-06-031-4/+4
| |
* | remove :constraints from IGNORE_OPTIONSAaron Patterson2014-06-031-4/+5
| |
* | disconnect normalize_defaults! from options[:constraints]Aaron Patterson2014-06-031-6/+6
| |
* | disconnect options and scope from the `blocks` methodAaron Patterson2014-06-031-7/+8
| |
* | disconnect the constraints method from the options and scope hashesAaron Patterson2014-06-031-12/+15
| |
* | :on is removed from options before the Mapping is instantiated, so remove it ↵Aaron Patterson2014-06-031-1/+1
| | | | | | | | from IGNORE_OPTIONS
* | pull up via extraction and remove it from options / IGNORE_OPTIONSAaron Patterson2014-06-031-8/+6
| |
* | always make :via a listAaron Patterson2014-06-031-3/+2
| |
* | always pull out a via variable and simplify logicAaron Patterson2014-06-031-12/+15
| |
* | test with an empty viaAaron Patterson2014-06-031-0/+8
| |
* | add a test for missing "via" parameterAaron Patterson2014-06-032-1/+9
| |
* | remove :as and :anchor from IGNORE_OPTIONSAaron Patterson2014-06-031-3/+5
| |
* | remove another value from IGNORE_OPTIONSAaron Patterson2014-06-031-2/+2
| |
* | shorten up IGNORE_OPTIONSAaron Patterson2014-06-031-2/+2
| | | | | | | | | | | | since we are now passing the format value around, we can remove it from the options hash, which means we don't need to consult as many values from IGNORE_OPTIONS
* | only look up the format option from the hash onceAaron Patterson2014-06-031-16/+18
| |
* | Feature requests should be made on the mailing list, not submitted toZachary Scott2014-06-011-1/+6
| | | | | | | | the issue tracker. See also #15455 [ci skip]
* | Missing period from AP/CHANGELOG [ci skip]Zachary Scott2014-05-311-1/+1
| |
* | Mapping never actually uses @set, so rmAaron Patterson2014-05-292-4/+4
| |
* | no reason to make a Mapper object if the path is blankAaron Patterson2014-05-291-1/+2
| |
* | no more is_a checks on instantiationAaron Patterson2014-05-292-13/+4
| |
* | Path::Pattern is instantiated internally, so make the contructor require a ↵Aaron Patterson2014-05-295-46/+49
| | | | | | | | strexp object
* | Strexp#names is only used in a test, so rmAaron Patterson2014-05-292-36/+0
| |
* | pass the parsed path from mapper to the StrexpAaron Patterson2014-05-292-5/+9
| |
* | add an alternate constructor to Strexp that takes a stringAaron Patterson2014-05-296-39/+41
| |
* | ask the strexp for the astAaron Patterson2014-05-292-3/+7
| |
* | remove dead codeAaron Patterson2014-05-291-1/+0
| |
* | disconnect path from the instanceAaron Patterson2014-05-291-6/+6
| |
* | reuse the ast we already madeAaron Patterson2014-05-291-9/+10
| |