Review by Derick ================ [-] I am not sure if the name for the new option constant for addUnorderedParameter() are good - the ezcUrlConfiguration::AGGREGATE_ARGUMENTS one. Perhaps MULTIPLE_ARRAY_ARGUMENTS would work? Or perhaps MULTIPLE_ARGUMENTS | ARRAY_ARGUMENTS would be a viable alternative? I think the latter makes most sense, but is slightly more to type. I think the AGGREGATE_ARGUMENTS is better. [ ] parseQueryString() will most likely have locale issues - needs investigating. [X] In ReST, the following things are equivalent: For example, if the $_SERVER array has these fields: :: and: For example, if the $_SERVER array has these fields:: - Done Additional review by kn ======================= [X] It would be nice to have some more class level documentation for ezcUrlTools, or at least mention it only provides static methods. - Done.