From d979c985f60aeec96bbcdff6a0a8e4a878ad0672 Mon Sep 17 00:00:00 2001
From: Darin Kelkhoff <darin.kelkhoff@gmail.com>
Date: Tue, 3 Oct 2023 08:31:40 -0500
Subject: [PATCH] Add sort (by summary) of processes - for stability (and just
 to be a little nicer)

---
 .../actions/GenerateOpenApiSpecAction.java    | 23 +++++++++++++++++--
 1 file changed, 21 insertions(+), 2 deletions(-)

diff --git a/qqq-middleware-api/src/main/java/com/kingsrook/qqq/api/actions/GenerateOpenApiSpecAction.java b/qqq-middleware-api/src/main/java/com/kingsrook/qqq/api/actions/GenerateOpenApiSpecAction.java
index e6d53c19..53617063 100644
--- a/qqq-middleware-api/src/main/java/com/kingsrook/qqq/api/actions/GenerateOpenApiSpecAction.java
+++ b/qqq-middleware-api/src/main/java/com/kingsrook/qqq/api/actions/GenerateOpenApiSpecAction.java
@@ -859,6 +859,9 @@ public class GenerateOpenApiSpecAction extends AbstractQActionFunction<GenerateO
 
          apiProcessMetaDataList.add(Pair.of(apiProcessMetaData, processMetaData));
       }
+
+      apiProcessMetaDataList.sort(Comparator.comparing(apiProcessMetaDataQProcessMetaDataPair -> getProcessSummary(apiProcessMetaDataQProcessMetaDataPair.getA(), apiProcessMetaDataQProcessMetaDataPair.getB())));
+
       return (apiProcessMetaDataList);
    }
 
@@ -885,7 +888,7 @@ public class GenerateOpenApiSpecAction extends AbstractQActionFunction<GenerateO
       Method methodForProcess = new Method()
          .withOperationId(apiProcessMetaData.getApiProcessName())
          .withTags(tags)
-         .withSummary(ObjectUtils.requireConditionElse(apiProcessMetaData.getSummary(), StringUtils::hasContent, processMetaData.getLabel()))
+         .withSummary(getProcessSummary(apiProcessMetaData, processMetaData))
          .withDescription(description)
          .withSecurity(getSecurity(apiInstanceMetaData, processMetaData.getName()));
 
@@ -1018,6 +1021,16 @@ public class GenerateOpenApiSpecAction extends AbstractQActionFunction<GenerateO
 
 
 
+   /*******************************************************************************
+    **
+    *******************************************************************************/
+   private static String getProcessSummary(ApiProcessMetaData apiProcessMetaData, QProcessMetaData processMetaData)
+   {
+      return ObjectUtils.requireConditionElse(apiProcessMetaData.getSummary(), StringUtils::hasContent, processMetaData.getLabel());
+   }
+
+
+
    /*******************************************************************************
     **
     *******************************************************************************/
@@ -1029,7 +1042,7 @@ public class GenerateOpenApiSpecAction extends AbstractQActionFunction<GenerateO
       Method methodForProcess = new Method()
          .withOperationId("getStatusFor" + StringUtils.ucFirst(apiProcessMetaData.getApiProcessName()))
          .withTags(tags)
-         .withSummary("Get Status of Job: " + ObjectUtils.requireConditionElse(apiProcessMetaData.getSummary(), StringUtils::hasContent, processMetaData.getLabel()))
+         .withSummary("Get Status of Job: " + getProcessSummary(apiProcessMetaData, processMetaData))
          .withDescription("Get the status for a previous asynchronous call to the process named " + processMetaData.getLabel())
          .withSecurity(getSecurity(apiInstanceMetaData, processMetaData.getName()));
 
@@ -1158,6 +1171,12 @@ public class GenerateOpenApiSpecAction extends AbstractQActionFunction<GenerateO
 
          apiProcessMetaDataList.add(Pair.of(apiProcessMetaData, processMetaData));
       }
+
+      /////////////////////////////////////////////////////////////////////
+      // sort by process summary (for stability, and just to be better) //
+      /////////////////////////////////////////////////////////////////////
+      apiProcessMetaDataList.sort(Comparator.comparing(apiProcessMetaDataQProcessMetaDataPair -> getProcessSummary(apiProcessMetaDataQProcessMetaDataPair.getA(), apiProcessMetaDataQProcessMetaDataPair.getB())));
+
       return (apiProcessMetaDataList);
    }