Problem/Motivation

There are way too many theme functions / template files in core. Some are similar (if not identical) and should be consolidated.

Proposed resolution

We have two options:

  1. Remove theme_book_admin_table and call theme('table') instead.
  2. If we really do need a separate preprocess function for the admin tables in book module, then call theme('table__book_admin') instead of theme('table') and add all our preprocess magic into template_preprocess_table__book_admin()

Remaining tasks

Decide which of the 2 options above to pursue, and do it.

User interface changes

None.

API changes

Removal of theme_book_admin_table

#1812684: [meta] Consolidate all table templates and add theme_hook_suggestions
#1968982: Convert book theme tables to table #type

CommentFileSizeAuthor
#1 2035905-remove-theme_book_admin_table-1.patch7.93 KBjoelpittet
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet commented
Issue summary: View changes
Status: Active » Needs review
FileSize
2035905-remove-theme_book_admin_table-1.patch7.93 KB

I thought I'd give this a stab to get it started and maybe someone can pick it up from here.

The values aren't showing but most everything else is rendering. May be due to something render element does...

Status: Needs review » Needs work

The last submitted patch, 1: 2035905-remove-theme_book_admin_table-1.patch, failed testing.

joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet commented
Status: Needs work » Closed (duplicate)

Woops, should have checked this is a dupe, will compare above with this #1968982: Convert book theme tables to table #type