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
Follow-up to #1768526: NodeFormController::validate() calls buildEntity() twice: This patch documented EntityFormInterface::validate() wrong.
Proposed resolution
Change documentation to reflect current code.
Remaining tasks
User interface changes
API changes
-
Comment | File | Size | Author |
---|---|---|---|
#1 | d8_EFI_validate_fix.patch | 1.07 KB | fago |
Comments
Comment #1
fagoAttached patch changes docs to reflect reality: It's an entity that is returned. But actually most config entities return nothing. So we could alter the docs of content entity only, but I think it might be worthwhile for some config entities to also validate based on the entity objects - in which case it should work the same way as for content entities. Thus I think it's best to adjust the general docs to return the object if built.
Comment #2
mgiffordNeeds re-roll.
Comment #13
quietone CreditAttribution: quietone at PreviousNext commentedEntityFormInterface::validate() was removed in #2453175: Remove EntityFormInterface::validate() and stop using button-level validation by default in entity forms.
Therefore, closing as outdated. If this is incorrect reopen the issue, by setting the status to 'Active', and add a comment explaining what still needs to be done.
Thanks!