From dd38d8d2b26a8cbede609bb64a1115fb3c6c8b4b Mon Sep 17 00:00:00 2001
From: bach <bachir@figureslibres.io>
Date: Thu, 17 Nov 2022 22:50:20 +0100
Subject: [PATCH] reversed license order newer first, license form selection
 list for product variation

---
 ...tity_form_display.commerce_license.role.default.yml |  8 ++------
 .../custom/materio_commerce/materio_commerce.module    | 10 ++++++----
 2 files changed, 8 insertions(+), 10 deletions(-)

diff --git a/config/sync/core.entity_form_display.commerce_license.role.default.yml b/config/sync/core.entity_form_display.commerce_license.role.default.yml
index 60d3322d..3619f09c 100644
--- a/config/sync/core.entity_form_display.commerce_license.role.default.yml
+++ b/config/sync/core.entity_form_display.commerce_license.role.default.yml
@@ -11,14 +11,10 @@ bundle: role
 mode: default
 content:
   product_variation:
-    type: entity_reference_autocomplete
+    type: options_select
     weight: 0
     region: content
-    settings:
-      match_operator: CONTAINS
-      match_limit: 10
-      size: 60
-      placeholder: ''
+    settings: {  }
     third_party_settings: {  }
   state:
     type: options_select
diff --git a/web/modules/custom/materio_commerce/materio_commerce.module b/web/modules/custom/materio_commerce/materio_commerce.module
index a0625a5d..52bdd55d 100644
--- a/web/modules/custom/materio_commerce/materio_commerce.module
+++ b/web/modules/custom/materio_commerce/materio_commerce.module
@@ -68,7 +68,8 @@ function materio_commerce_form_role_delegation_role_assign_form_alter(&$form, &$
 
         $description = t("This role is granted by a license. It cannot be removed manually.<br/>");
       }
-      foreach ($role_licenses['licenses'] as $license) {
+      $reversed_licenses = array_reverse($role_licenses['licenses']);
+      foreach ($reversed_licenses as $license) {
         $license_state = $license->getState()->getValue()['value'];
 
         $product_variation = $license->getPurchasedEntity();
@@ -94,7 +95,7 @@ function materio_commerce_form_role_delegation_role_assign_form_alter(&$form, &$
           )), $order_url);
         }
 
-        $description .= t("-- @product: @variation | license state: @license_state | granted: @granted | expires: @expires | @license_link | @order_link<br/>", array(
+        $description .= t("-- @product: @variation | license state: <b>@license_state</b> | granted: @granted | expires: @expires | @license_link | @order_link<br/>", array(
           '@product' => $product ? $product->getTitle() : t('no product'),
           '@variation' => $product_variation ? $product_variation->getTitle() : t('no variation'),
           '@license_state' => $license_state,
@@ -158,7 +159,8 @@ function materio_commerce_form_user_form_alter(&$form, &$form_state, $form_id) {
 
         $description = t("This role is granted by a license. It cannot be removed manually.<br/>");
       }
-      foreach ($role_licenses['licenses'] as $license) {
+      $reversed_licenses = array_reverse($role_licenses['licenses']);
+      foreach ($reversed_licenses as $license) {
         $license_state = $license->getState()->getValue()['value'];
 
         $product_variation = $license->getPurchasedEntity();
@@ -184,7 +186,7 @@ function materio_commerce_form_user_form_alter(&$form, &$form_state, $form_id) {
           )), $order_url);
         }
 
-        $description .= t("-- @product: @variation | license state: @license_state | granted: @granted | expires: @expires | @license_link | @order_link<br/>", array(
+        $description .= t("-- @product: @variation | license state: <b>@license_state</b> | granted: @granted | expires: @expires | @license_link | @order_link<br/>", array(
           '@product' => $product ? $product->getTitle() : t('no product'),
           '@variation' => $product_variation ? $product_variation->getTitle() : t('no variation'),
           '@license_state' => $license_state,