#4693 Remove unnecessary line in the code - Ruby on Rails

Type	To find
responsible:me	tickets assigned to you
tagged:"@high"	tickets tagged @high
milestone:next	tickets in the upcoming milestone
state:invalid	tickets with the state invalid
created:"last week"	tickets created last week
sort:number, importance, updated	tickets sorted by #, importance or updated
Combine keywords for powerful searching.
Use advanced searching »

This project is archived and is in readonly mode.

#4693 ✓invalid

Remove unnecessary line in the code

Reported by Anil Wadghule | May 25th, 2010 @ 07:49 PM

Looks like rack_test.rb has a line which is not needed and probably there by mistake. Attaching patch in the comment.

You flagged this item as spam.
Anil Wadghule May 25th, 2010 @ 07:51 PM
- no changes were found...
- remove_unnecessary_code.diff 828 Bytes
You flagged this item as spam.
Anil Wadghule May 25th, 2010 @ 08:33 PM
I came to know that request.request_parameters does params parsing. It forces normalize_params which does not happen on new as as you don't need to do (expensive) param parsing sometimes. It would error if params were parsed and the body was not rewound. (thanks @raggi for info)

This ticket can be marked as invalid.
Rohit Arondekar June 15th, 2010 @ 01:47 PM
- State changed from “new” to “invalid”