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.
I found a need to do this and thought it might be handy to add
Comment | File | Size | Author |
---|---|---|---|
#4 | masquerade-865750.patch | 2.16 KB | realityloop |
#3 | masquerade-865750.patch | 1.39 KB | realityloop |
#1 | masquerade-865750.patch | 1.79 KB | realityloop |
Comments
Comment #1
realityloopPatch attached
Comment #2
deekayen CreditAttribution: deekayen commentedFirst, don't abbreviate variables with this module or any other. I thought at first you were checking to see if the site was defaulting to English before deciding whether or not to display a search box.
Since there are only two options, I'd think you could use 0 or 1. The variable itself isn't re-purposed anywhere else, so make it describe what it does, "masquerade_block_search_visibility".
I personally think radio buttons would be better here. Do you have some plans for other options in this box or a reason to need to select more than one?
You tried to stab a kitten with your patch by changing an unrelated string, which would affect translations which have already been submitted. That's a separate issue so the translation maintainers can be notified.
Comment #3
realityloopThanks for the feedback, revised patch attached.
Comment #4
realityloopLast patch didn't include uninstall for added variable, please test this one instead.
Comment #5
apadernoI am closing this feature request, as Drupal 6 is no longer supported. Please re-open it if you would like the feature implemented in the Drupal 7 or 8 version of the module.