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.
Currently Panels uses isset()
for determining whether a content type should be at the top level.
Consequently setting 'top level' to FALSE makes the content type appear at top level. It's illogical and goes against code and examples in Fieldable Panel Panes.
Instead of fixing this in Fieldable Panel Panes how about using empty()
instead?
The following patch implements this change.
Comment | File | Size | Author |
---|---|---|---|
#1 | panels-top-level-empty-1836428-1.patch | 1.08 KB | kasperg |
Comments
Comment #1
kasperg CreditAttribution: kasperg commentedAnd here's the patch.
Comment #2
merlinofchaos CreditAttribution: merlinofchaos commentedI agree with this (though it's easy tow ork around by setting the value to NULL instead of FALSE).
Comment #3
merlinofchaos CreditAttribution: merlinofchaos commentedCommitted and pushed.