- Issue created by @justin2pin
- Merge request !17Add setNeedsSave() method definition in EntityNeedsSaveInterface. β (Merged) created by justin2pin
- Status changed to RTBC
almost 2 years ago 11:32am 8 May 2023 - π§π¬Bulgaria pfrenssen Sofia
This looks good. It is a possible backwards compatibility break though. Any custom or contrib module that uses
EntityNeedsSaveInterface
will throw fatal errors if they do not yet implement this method. In practice this might not be a big problem though since we offerEntityNeedsSaveTrait
which already includes the method. I did a search on Drupal Code Search for contrib modules that use this interface and as far as I could see this trait is used universally. - First commit to issue fork.
- last update
about 1 year ago 39 pass -
Berdir β
committed c043170f on 8.x-1.x authored by
justin2pin β
Issue #3336752: EntityNeedsSaveInterface should include a method...
-
Berdir β
committed c043170f on 8.x-1.x authored by
justin2pin β
- Status changed to Fixed
about 1 year ago 11:49pm 12 January 2024 - π¨πSwitzerland berdir Switzerland
Yeah, I guess this is probably OK.
Automatically closed - issue fixed for 2 weeks with no activity.