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.
Problem/Motivation
Token hides the menu_link field by default for entity view and form displays, but is using a deprecated key to do this.
Source: http://cgit.drupalcode.org/drupal/tree/core/lib/Drupal/Core/Entity/Entit...
Proposed resolution
Use the hidden region instead of the hidden type.
Remaining tasks
Review the patch.
User interface changes
None.
API changes
None.
Data model changes
None.
Comment | File | Size | Author |
---|---|---|---|
token-deprecated-calls.patch | 541 bytes | samuel.mortenson | |
Comments
Comment #2
samuel.mortensonThis should work, but I see that the region is still defaulting to content.
Comment #3
samuel.mortensonNevermind, I think this is working!
Comment #4
BerdirThanks, is there an easy way to test this?
Comment #5
samuel.mortensonHm, I'm not sure about the best way, but an easy way is to confirm that the "Menu Link" field is hidden on all the Entity form/view displays it's attached to.
Comment #6
BerdirI was thinking that this would be an API change if it's deprecated.
But that's not the point, the whole thing is bogus. If it shouldn't be shown then we shouldn't define display options in the first place. Just remove all of that and it should be fine.
Then we also don't need test coverage IMHO :)
Comment #7
samuel.mortensonThis change happened in dev without updating this issue: http://cgit.drupalcode.org/token/tree/token.module#n681