Closed (fixed)
Project:
Scheduler
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
25 Jan 2011 at 22:50 UTC
Updated:
9 Feb 2011 at 20:00 UTC
Jump to comment: Most recent file
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