The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.I am reporting http://drupal.org/update/modules/6/7 to moderators because:
There is no mention of the removal of the theme key table_select_header_cell nor how to convert the code.
Comments
Comment #1
jhodgdonCan you provide any more specifics on that -- where was this theme key before, and actually, what do you mean by "theme key"?
Comment #2
Anonymous (not verified) commentedhttp://api.drupal.org/api/drupal/includes%21theme.inc/function/theme_tab...
"Theme key" is mentioned in the error. It is the hook you pass to the theme() API. http://api.drupal.org/api/drupal/includes%21theme.inc/function/theme/7
Comment #3
Anonymous (not verified) commentedFYI, this was phased out in favor of FAPI #type of tableselect.
Comment #4
jhodgdonOK, I think you're saying that in D6 you could call theme('table_select_header_cell'), but in D7, you can't. That's because the tableselect theming has changed. In D6 it was rather piecemeal, and in D7, it is unified as theme('tableselect') and you don't have to do something special with the headers.
And I agree that it is not documented on
http://drupal.org/update/themes/6/7
or
http://drupal.org/update/modules/6/7
So... We need a change notice here. Someone needs to find the issue where this was introduced, and write up a change notice.
Comment #5
jhodgdonRE #3 - right, but this change was apparently not documented on the change notices page or with a change notice node.
Comment #6
star-szrSimilar to #1632338: Change notice missing for profile_load_profile()…
Issue found via
git log -S "table_select_header_cell":#768490: theme_table_select_header_cell() is not really a theme function
I think this should be added to http://drupal.org/update/themes/6/7 or http://drupal.org/update/modules/6/7 as @jhodgdon mentioned :)