Entity type is not checked during delete

When an entity is created, this module checks if the entity type should be added to the index table.

However, when an entity is deleted this module's code is always running without any checks of the entity's type:

This is what happens in our environment:

1) Drupal Commerce decies to refresh the cart:
commerce_cart_order_refresh() called at [.../sites/all/modules/commerce/modules/cart/commerce_cart.module:601]

2) Commerce Fees decides to delete a fee line item from the order:
commerce_fees_commerce_cart_order_refresh()
commerce_line_item_delete_multiple() called at [.../sites/all/modules/commerce_fees/commerce_fees.module:557]

3) The entity is deleted and all module's hooks fires:
call_user_func_array() called at [.../includes/module.inc:895]
apachesolr_entity_delete()

4) Apachesolr sends the delete query to the database without checking the entity type:
apachesolr_remove_entity() called at [.../sites/all/modules/apachesolr/apachesolr.module:2069]
apachesolr_index_delete_entity_from_index() called at [.../sites/all/modules/apachesolr/apachesolr.module:2084]
apachesolr_set_last_index_updated() called at [.../sites/all/modules/apachesolr/apachesolr.index.inc:711]

Proposed solution

Check the entity type like we do when adding entities by adding apachesolr_entity_should_index():

function apachesolr_entity_delete($entity, $entity_type) {
  if (apachesolr_entity_should_index($entity, $entity_type)) {
    list($entity_id) = entity_extract_ids($entity_type, $entity);
    // Get all environments and delete it from their table and index
    $environments = apachesolr_load_all_environments();
    foreach ($environments as $environment) {
      apachesolr_remove_entity($environment['env_id'], $entity_type, $entity_id);
    }
  }
}