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.In the \core\includes\entity.inc file the docblock contains code examples with syntax errors for the $values variables for the entity_get_display and entity_get_form_display functions, which are gonna be deprecated.
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | interdiff-2-7.txt | 354 bytes | michielnugter |
| #7 | 2755545-7.patch | 681 bytes | michielnugter |
| #2 | 2755545-2.patch | 992 bytes | balagan |
Comments
Comment #2
balagan CreditAttribution: balagan commentedComment #3
balagan CreditAttribution: balagan commentedComment #5
michielnugter CreditAttribution: michielnugter as a volunteer and at Synetic commentedI found out the same problem and came up with the exact same patch. Looks good, set to RTBC.
Comment #7
michielnugter CreditAttribution: michielnugter as a volunteer and at Synetic commentedReroll on the current 8.3.x branch, removed 1 change as it was already fixed in another issue.
Comment #8
michielnugter CreditAttribution: michielnugter as a volunteer and at Synetic commentedReroll on the current 8.3.x branch, removed 1 change as it was already fixed in another issue.
Comment #9
balagan CreditAttribution: balagan commentedPatch looks fine, changing it to RTBC.
Comment #12
xjmI checked with git blame and this was added in #2474151: Mark procedural wrappers in entity.inc as deprecated and has not changed since. The similar docs for the
entity_view_displaywere updated in #2547933: Code example for non deprecated use of entity_get_display is wrong but that issue missed this one.Committed and pushed to 8.3.x and 8.2.x. Thanks!