Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
theme_filter_tips() takes an argument $extra that is never used and can be safely removed.
I've searched through all files in core, and it's only invoked by Filter module itself.
Comment | File | Size | Author |
---|---|---|---|
#7 | drupal.theme-filter-tips.patch | 3.79 KB | sun |
#6 | drupal.theme-filter-tips.patch | 3.79 KB | sun |
#4 | drupal.theme-filter-tips.patch | 3.68 KB | sun |
#2 | drupal.theme-filter-tips.patch | 3.32 KB | sun |
drupal.theme-filter-tips.patch | 2.12 KB | sun | |
Comments
Comment #1
webchickIf this really has no use, we should remove it from the theme function declaration in hook_theme().
Comment #2
sunBoy, I needed to lookup multiple functions to grasp what's actually passed there in $tips.... so I added an example.
Comment #3
Dries CreditAttribution: Dries commentedNote that $extra is now an unused variable. It is, after all, declared. It looks like the the "More information about formatting options" gets lost now?
Comment #4
sunGood catch, but only true for filter_admin_format_form(). Fixed the appending of the more link there, which was not displayed until now, and also did not use the corresponding theme function...
Please note that we most probably want to refactor the whole processing and theming of filter tips (and the ($extra) more link) anyway, but let's discuss this in a new issue and not this one, please. Code clean-up should come first.
Comment #5
Dries CreditAttribution: Dries commentedI think this is a good patch, although the @code example tends to raise questions (e.g. what is ID for?). I'm not mandating we add more documentation, so I'll commit this after someone else has reviewed this.
Comment #6
sunI agree that ID should be explained as well. Hopefully ready to go now.
Comment #7
sunFixed typo "intented".
Comment #8
Dries CreditAttribution: Dries commentedCommitted it to CVS HEAD. The extra documentation pushed it over the threshold. ;-)