it should probably use a custom $pagevar instead of usurping the 'page' GET parameter, it might be a problem especially when page already used for something else on that page or if another instance of paging existed on the same page - in general all subsystems should use some kind of unique prefix imho.