Lorna,

you should consider putting the parse_str() logic into the request object, rather than a controller. You are dealing with details of how request works (namely, the raw request data), thus you are coupling your controller to the request more tightly than you'd have to. Plus, by putting that logic into the request, it's easier to reuse.

Stefan