Quantcast
Channel: Issues for Drupal core
Viewing all articles
Browse latest Browse all 301158

loadUnchanged() should check the id

$
0
0

ContentEntityStorageBase::loadUnchanged() implies that its argument is a valid entity id, but never checks it. Supplying, eg, NULL as an argument can lead to a number of side-effects, such as resetting static cache, calling preload hook for all entities etc, to say nothing about just time it takes. It should return NULL and exit instead.


Viewing all articles
Browse latest Browse all 301158

Trending Articles



<script src="https://jsc.adskeeper.com/r/s/rssing.com.1596347.js" async> </script>