Don't delete renewed licenses when deleting orders.

As things currently are, when a order is deleted e.g from /admin/commerce/orders, this also deletes associated license. This is not a good behavior if the license is renewable and was renewed, because it can have many orders associated to it...

What about :

@@ -68,13 +69,42 @@ function commerce_license_commerce_order_item_delete(EntityInterface $entity) {
 
+  /** @var \Drupal\commerce_license\Entity\LicenseInterface $license */
   $license = $entity->license->entity;
-  $license->delete();
+
+  // Previous conditions will validate in case of renewed licenses,
+  // because $entity->getOrder() already returns null !
+  // Only act if the license was not renewed yet.
+  if ($license->getRenewedTime()) {
+    return;
+  }

This seems to work fine. We may also add in the same block a message only for admins, such as ?

+    \Drupal::messenger()->addWarning(
+      t('This order is associated to a renewed license (Id : @id), you may check if you wish to delete that license or not.', [
+        "@id" => $license->id(),
+      ])
+    );