Closed (won't fix)
Project:
Drupal core
Version:
8.6.x-dev
Component:
entity system
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
12 Sep 2014 at 18:16 UTC
Updated:
26 Apr 2019 at 18:37 UTC
Jump to comment: Most recent
Comments
Comment #1
berdirdelete on the storage is already multiple , the only difference to the function is that that accepts ids and then loads them before calling delete() on the storage.
Not sure if Entity::deleteMultiple() makes sense, that doesn't follow the create/load pattern of existing methods, as it doesn't return entity objects, so we don't gain much by putting it through much.
If you have the storage object, you can already do this $storage->delete($storage->loadMultiple($ids)). We could also consider to add a deleteByIds() or make delete() accept both but that's usually a bad design.
Comment #8
mikelutzI think there's been enough time for anyone who wishes to voice opposition to Berdir. Closing in favor of #3051069: Remove usage of deprecated entity_delete_multiple in core and trigger an error