On `src/Plugin/search_api/backend/SearchApiSolrBackend.php` function `deleteAllIndexItems` we are not checking that the index is read-only.
In this file from `search_api` module, `search_api/src/Entity/Server.php`, we check the read-only property. See `deleteAllIndexItems`.
Add code to check the read-only property in `Drupal\search_api_solr\Plugin\search_api\backend::deleteAllIndexItems`, as in `Drupal\search_api\Entity\Server::deleteAllIndexItems`.
```
if ($index->isReadOnly()) {
$vars = [
'%index' => $index->label(),
];
$this->getLogger()->warning('Trying to delete items from index %index which is marked as read-only.', $vars);
return;
}
```
Active
4.3
Code
Thanks for the patch. But as you mentioned, the log entry is generated in the Server class which then doesn't call the method on the backend anymore. The function is not meant to be called directly.
If we should harden the code if called directly, we need to do it for all methods, for example deleteItems().