After #2871910: Standardize on DrupalDefaultEntityController, following guidelines in the base class is the next step.
In special it is recommended to move away from conditions parameter on load(), so #2897078: Operation controller does not cache when custom conditions are used was in a way a step back.
Let's get rid of custom conditions in favour if using EntityFieldQuery class.
This may involve at least changing operation controller to remove the 3 custom conditions.
Comments
Comment #2
drummOther than
versioncontrolmodule itself, I think #3137267: Move away from conditions argument in VersioncontrolGitItem::determineSourceItemRevisionID() is the last use of these in Drupal.org’s codebase.