Permalink
Browse files

the 1.1.7 fix broke the formats functionality by not allow periods in…

… the action
  • Loading branch information...
1 parent a8bf39c commit aa04eb927daa429a3bdc3a4d4e1d221c823e63b3 unknown committed Jan 26, 2012
Showing with 1 addition and 1 deletion.
  1. +1 −1 wheels/dispatch/request.cfm
@@ -379,7 +379,7 @@
// filter out illegal characters from the controller and action arguments
arguments.params.controller = ReReplace(arguments.params.controller, "[^0-9A-Za-z-_]", "", "all");
- arguments.params.action = ReReplace(arguments.params.action, "[^0-9A-Za-z-_]", "", "all");
+ arguments.params.action = ReReplace(arguments.params.action, "[^0-9A-Za-z-_\.]", "", "all");
// convert controller to upperCamelCase and action to normal camelCase
arguments.params.controller = ListSetAt(arguments.params.controller, ListLen(arguments.params.controller, "."), REReplace(ListLast(arguments.params.controller, "."), "(^|-)([a-z])", "\u\2", "all"), ".");

1 comment on commit aa04eb9

@rip747
rip747 commented on aa04eb9 May 5, 2012

i am not seeing this in any of the test coverage that we have. can you give me a specific example of when this failing?

Please sign in to comment.