@verdocs/js-sdk 6.4.4 → 6.4.6

package/dist/index.js

@@ -1783,13 +1783,11 @@ function sortFields(fields) {
         if (aPage !== bPage) {
             return aPage - bPage;
         }
-        // NOTE: Logic looks a little strange X vs Y. It's because we go top down,
-        // left to right. But Y coordinates are inverted in PDFs. The reason for
-        // the division is because no human makes perfect templates and frequently
-        // two fields on the "same line" will be slightly offset vertically.
         const divaY = Math.floor(aY / 5);
         const divbY = Math.floor(bY / 5);
         if (divaY !== divbY) {
+            // Unlike the other properties, Y coordinates have 0,0 at the BOTTOM LEFT corner, so
+            // we need to sort descending (b first) on this one.
             return divbY - divaY;
         }
         return aX - bX;
@@ -1801,17 +1799,16 @@ function sortFields(fields) {
  * NOTE: This function mutates the input array.
  */
 function sortDocuments(documents) {
-    return documents.sort((a, b) => 
     // The Date conversion is unnecessary 90% of the time but is safer, and this isn't something
     // we do much of so in reality it has almmost no impact.
-    a.order !== b.order ? a.order - b.order : new Date(a.created_at).getTime() - new Date(b.created_at).getTime());
+    return documents.sort((a, b) => a.order - b.order || new Date(a.created_at).getTime() - new Date(b.created_at).getTime());
 }
 /**
  * Utility function to sort documents by their order, falling back to created_at.
  * NOTE: This function mutates the input array.
  */
 function sortRecipients(recipients) {
-    return recipients?.sort((a, b) => (a.sequence !== b.sequence ? b.sequence - a.sequence : b.order - a.order));
+    return recipients?.sort((a, b) => a.sequence - b.sequence || a.order - b.order);
 }
 
 const canPerformTemplateAction = (profile, action, template) => {