Notice: Undefined variable: rows in scheduler_list() (line 441 of /var/www/staging/7/html/sites/all/modules/scheduler/scheduler.module).
The attached patch will fix that, as well as update the code comment for the function for the new location of the tab.
Comments
Comment #1
jonathan1055 commentedTested and this works.
Richard, do you know if there are any extra settings when returning the $build array with 'table' to re-create the tablesort headers like we had in D6. I'm sure this functionality must be in D7 and there should be a neat way of doing it with the new return value.
Comment #2
jonathan1055 commentedI've also worked out a new db_query for that sql. Change
to
Attached is a screen grab showing that is produces the same result. Makes sense for you to add it to your patch, instead of me making another on top.
It does not solve the tablesort yet, though.[edit: I found the tablesort extender, as added it above, very nice and easy]
Comment #3
sillygwailoUpdating my patch with Jonathan's query changes.
Comment #4
jonathan1055 commentedI've just found out that you can add many fields from one table in one go. Use 'fields' instead of 'addField'
For example: $query->fields('s', array('publish_on', 'unpublish_on'));
Comment #5
sillygwailoLet's go with that.
Comment #6
sillygwailoThe following patch should add back the pager too. I tested this with 2 nodes and setting the pager to 1 (one) instead of 50 (fifty) as I have it in the patch.
Comment #7
sillygwailoComment #8
eric-alexander schaefer commentedWorks.
http://drupal.org/cvs?commit=489422