# HG changeset patch
# User Alain Mazy <am@orthanc.team>
# Date 1740048854 -3600
# Node ID 77004b16d7fe8e8a66a3653139428ca92e42743d
# Parent  cfddeec356c509da249f199754ed7070af6800d6
Now returning results when e.g, ordering instances against a metadata they don't have

diff -r cfddeec356c5 -r 77004b16d7fe Framework/Plugins/ISqlLookupFormatter.cpp
--- a/Framework/Plugins/ISqlLookupFormatter.cpp	Fri Feb 14 15:36:41 2025 +0100
+++ b/Framework/Plugins/ISqlLookupFormatter.cpp	Thu Feb 20 11:54:14 2025 +0100
@@ -440,26 +440,26 @@
 
     if (tagLevel == requestLevel)
     {
-      target = " LEFT JOIN " + tagTable + " " + orderArg + " ON " + orderArg + ".id = " + FormatLevel(requestLevel) +
+      target = " FULL JOIN " + tagTable + " " + orderArg + " ON " + orderArg + ".id = " + FormatLevel(requestLevel) +
                 ".internalId AND " + tagFilter;
     }
     else if (static_cast<int32_t>(requestLevel) - static_cast<int32_t>(tagLevel) == 1)
     {
       target = " INNER JOIN Resources " + orderArg + "parent ON " + orderArg + "parent.internalId = " + FormatLevel(requestLevel) + ".parentId "
-               " LEFT JOIN " + tagTable + " " + orderArg + " ON " + orderArg + ".id = " + orderArg + "parent.internalId AND " + tagFilter;
+               " FULL JOIN " + tagTable + " " + orderArg + " ON " + orderArg + ".id = " + orderArg + "parent.internalId AND " + tagFilter;
     }
     else if (static_cast<int32_t>(requestLevel) - static_cast<int32_t>(tagLevel) == 2)
     {
       target = " INNER JOIN Resources " + orderArg + "parent ON " + orderArg + "parent.internalId = " + FormatLevel(requestLevel) + ".parentId "
                " INNER JOIN Resources " + orderArg + "grandparent ON " + orderArg + "grandparent.internalId = " + orderArg + "parent.parentId "
-               " LEFT JOIN " + tagTable + " " + orderArg + " ON " + orderArg + ".id = " + orderArg + "grandparent.internalId AND " + tagFilter;
+               " FULL JOIN " + tagTable + " " + orderArg + " ON " + orderArg + ".id = " + orderArg + "grandparent.internalId AND " + tagFilter;
     }
     else if (static_cast<int32_t>(requestLevel) - static_cast<int32_t>(tagLevel) == 3)
     {
       target = " INNER JOIN Resources " + orderArg + "parent ON " + orderArg + "parent.internalId = " + FormatLevel(requestLevel) + ".parentId "
                " INNER JOIN Resources " + orderArg + "grandparent ON " + orderArg + "grandparent.internalId = " + orderArg + "parent.parentId "
                " INNER JOIN Resources " + orderArg + "grandgrandparent ON " + orderArg + "grandgrandparent.internalId = " + orderArg + "grandparent.parentId "
-               " LEFT JOIN " + tagTable + " " + orderArg + " ON " + orderArg + ".id = " + orderArg + "grandgrandparent.internalId AND " + tagFilter;
+               " FULL JOIN " + tagTable + " " + orderArg + " ON " + orderArg + ".id = " + orderArg + "grandgrandparent.internalId AND " + tagFilter;
     }
   }
 #endif
@@ -473,7 +473,7 @@
   {
     std::string arg = "order" + boost::lexical_cast<std::string>(index);
 
-    target = " INNER JOIN Metadata " + arg + " ON " + arg + ".id = " + FormatLevel(requestLevel) +
+    target = " FULL JOIN Metadata " + arg + " ON " + arg + ".id = " + FormatLevel(requestLevel) +
              ".internalId AND " + arg + ".type = " +
              boost::lexical_cast<std::string>(metadata);
   }
diff -r cfddeec356c5 -r 77004b16d7fe PostgreSQL/NEWS
--- a/PostgreSQL/NEWS	Fri Feb 14 15:36:41 2025 +0100
+++ b/PostgreSQL/NEWS	Thu Feb 20 11:54:14 2025 +0100
@@ -22,6 +22,7 @@
 * Fixed high memory usage due to caching of too many
   prepared SQL statement when using since & limit.
 * Removed duplicate comparison in find SQL queries.
+* Now returning results when e.g, ordering instances against a metadata they don't have.