- Issue created by @duaelfr
- Merge request !49Issue #3513777: Less complexity and more performances in FocalPointManager::getCropEntity() → (Open) created by duaelfr
In \Drupal\focal_point\FocalPointManager::getCropEntity(), we use \Drupal\crop\Entity\Crop::cropExists() before calling \Drupal\crop\Entity\Crop::findCrop() or creating a new Crop entity.
This can cause a point of failure when cropExists returns TRUE and findCrop returns NULL (this can happen during migrations or content deployment for example) and it adds an unneeded database request.
Only use the \Drupal\crop\Entity\Crop::findCrop() method and test its result.
Active
2.0
Other Code