aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack
Commit message (Collapse)AuthorAgeFilesLines
...
| | * Convert StrongParameters cache to a hash. This fixes an unboundedRyan Davis2014-06-032-7/+7
| | | | | | | | | | | | | | | | | | memory leak demonstrated on @tenderlove's latest blog post: http://tenderlovemaking.com/2014/06/02/yagni-methods-are-killing-me.html
| * | 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
| | | |
* | | | push splitting "to" up the callstackAaron Patterson2014-06-031-15/+20
|/ / /
* | | 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
| | |