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
SqlContentEntityStorage::invokeHook
calls $this->invokeFieldMethod
for presave. The $this->invokeFieldMethod
method lives in ContentEntityStorageBase
and I think this invokeHook override should as well.
Proposed resolution
Move up the method.
Remaining tasks
User interface changes
API changes
Alternative storage engines begin to work :)
Comment | File | Size | Author |
---|---|---|---|
presave.patch | 1.48 KB | chx | |
Comments
Comment #1
jibranTags
Comment #2
jibranIt has nothing to do with SQL so moving it makes sense.
Comment #3
andypostI does, "preSave" hook should be executed for all entities not only sql-storage based
Comment #4
alexpottThis issue is a normal bug fix, and doesn't include any disruptive changes, so it is allowed per https://www.drupal.org/core/beta-changes. Committed 6c91bb7 and pushed to 8.0.x. Thanks!