DBTNG: Tracker module

And here we are.

This looks good. Looking at the tracker module test, we should have good text coverage for these changes. The one exception is that the tracker module tests don't test the paging part. For extra points, update the tracker tests to use a pager?

I've committed the patch in #1 to CVS. Thanks DamZ. We can talk more about tests and slave queries so not marking this 'fixed' yet.

Ok, there was two mistakes:

- the object returned by the call to ->extend() should be used when calling ->execute()... if not, the extension as no effect at all
- tracker doesn't support tablesort (no header is associated with a sql column) which made tablesort.inc throw away some notices... a fix for which I sneaked in the attached patch.

I don't see the tag 'node_access' on the listing queries.

Ahem - still no 'node_access' tag. Lets not leave HEAD in an insecure state for months at a time.

- Added the tag
- Re-ordered the query a bit, early extended and chained more calls..
- Removed tablesort extender, we don't need it..

moshe suggested to remove the tests, no other page is testing the paging (except of the PagerDefault tests, obviously..).

New patch.

Ah, node access API thanks you.

Committed to CVS HEAD. Thanks, Berdir.