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.
IF condition always evaluate to true :(
Comment | File | Size | Author |
---|---|---|---|
seven.patch | 826 bytes | droplet | |
Comments
Comment #1
tamanna-freelancer CreditAttribution: tamanna-freelancer commentedI have tested this patch and it works fine.
Comment #2
xjmThanks for the patch! Kinda not sure about the form of this fix, though. I don't think it's backportable, either. See:
I think we need more review on the approach here, at least.
Comment #3
David_Rothstein CreditAttribution: David_Rothstein commentedSo is this issue actually a duplicate? It seems like the first part of the patch is being covered in #997408: $tabs is always set and the second part in #953034: [meta] Themes improperly check renderable arrays when determining visibility...
This will break if you have other blocks in the 'Help' region, or if you're using something other than the Block module to manage your page regions.
Comment #4
bleen CreditAttribution: bleen commentedfixing title ...
Comment #5
xjmClosing as duplicate of #1158090: Separate usage of hide(), render(), and other template features from print statements to enable non-developers to create themes .