Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.Firstly, thanks for the excellent module, very useful!
I've been exploring an issue with adding first/last classes and found that tabs_pre_render_tabset was being called twice.
It no longer needs to be manually called as it is called by drupal_render
The attached patch removes the manual call to tabs_pre_render_tabset and saves the double processing time on form load.
DT
| Comment | File | Size | Author |
|---|---|---|---|
| cck_fieldgroup_tabs.module.patch | 679 bytes | davidwhthomas |
Comments
Comment #1
davidwhthomas commentedSee http://api.drupal.org/api/function/drupal_render/6 for more info
In D5 that bug may have been an issue but is no longer so.
DT
Comment #2
nedjoI reviewed the code and believe the these lines are still needed.
The issue is that, if a theme callback is used, child elements are not themselves passed to drupal_render() and so don't get #pre_render callbacks triggered.