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.
#1946404: Convert forms in field_ui.admin.inc to the new form interface removed 90% of the Field UI 'admin' section of hook_entity_bundle_info(), leaving only 'real path', which is used almost exclusively by Field UI confirmation forms and the "Field List" admin reports page.
At this point it can be automatically derived, no more need for module developers to specify it manually.
Comment | File | Size | Author |
---|---|---|---|
#10 | field-admin-path-1982088-10.patch | 14.74 KB | tim.plunkett |
#10 | interdiff.txt | 2.31 KB | tim.plunkett |
#5 | real-path-1982088-5.patch | 14.8 KB | tim.plunkett |
#5 | interdiff.txt | 2.17 KB | tim.plunkett |
#3 | real-path-1982088-3.patch | 15.16 KB | tim.plunkett |
Comments
Comment #1
tim.plunkettThis path actually has nothing to do with the Field UI, it's about the Entity itself.
Comment #2
andypostAwesome! Also we need to unify uri() entity_uri mess
Comment #3
tim.plunkettYeah, this has nothing to do with entities, just entity types.
When this was decoupled from entity_get_info(), there was no longer any need to access config files directly.
Also fixing user and hook.
Comment #5
tim.plunkettI forgot about hook_field_extra_fields and its ilk. Reverting the entity_load_multiple bits.
Comment #6
andyposta lot of inconsistency for follow-ups
Comment #7
ParisLiakos CreditAttribution: ParisLiakos commentedawesome cleanup!
what happens here if there is no bundle_prefix, besides $bundle being undefined?
Comment #8
ParisLiakos CreditAttribution: ParisLiakos commentedyea, apparently i missed the $bundle argument:)
Comment #9
webchickThis will need a re-roll for #1982984: Create Drupal::entityManager for improved DX.
Apart from that, this looks like a great clean-up to me.
Comment #10
tim.plunkettReroll!
Comment #11
ParisLiakos CreditAttribution: ParisLiakos commentedlooks good:)
Comment #12
alexpottCommitted 50ae329 and pushed to 8.x. Thanks!
Comment #13
tim.plunkettUpdated this change notice https://drupal.org/node/1982084/revisions/view/2667850/2675638