Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Task
Convert the following theme functions to use the new table #type:
Module | Theme function name | Where in Code | What is it really? |
---|---|---|---|
forum | theme_forum_list | template | |
forum | theme_forum_topic_list | template |
Comments
Comment #1
larowlan.
Comment #2
jibranTagging.
Comment #3
neochief CreditAttribution: neochief commentedIt looks like these functions dont exist anymore and listing was replaced by views.
Comment #4
zhgenti CreditAttribution: zhgenti commentedComment #5
joelpittetRe-opening this because it has a tablesort_init call in there and a twig template file that shouldn't be needed.
The manually built header is calling a private function that also doesn't need to exist called '_theme_table_cell'
Comment #6
joelpittetAlso this issue is holding up #1939008: Convert theme_table() to Twig a bit with an @todo.
Comment #7
joelpittetOk scrap that idea. #type table is kinda slow from a DX (@see #1876718: Allow modules to inject columns into tables more easily) and performance prospective(@see #1938926-18: Convert simpletest theme tables to table #type. Just going to change the forum bit that I need for #theme table and leave this closed.