Pagination with standard CI urls + query strings

[eluser]skunkbad[/eluser]
I like to use query strings for searches. The reason for this is that I don't have the annoying browser messages when traveling backwards through history when POST is used. It was pretty easy to extend the Pagination class, and I wouldn't have even needed to extend the class if the $suffix class member was applied to ALL of the pagination links, but it isn't.

If it was, then I could have done something like this in my controller:

Code:

if( ! empty( $_SERVER["QUERY_STRING"] ) )
{
$config['suffix'] =  '?' . $_SERVER["QUERY_STRING"];
}

What do you think? Is there any good reason why the suffix shouldn't be appended to ALL of the links?