fix(striker-ui-api): hoist and enhance join logic

striker-ui-api/src/lib/join.ts

@@ -0,0 +1,15 @@
+const join = (
+  elements: string[] | undefined,
+  { beforeReturn, elementWrapper = '', separator = '' }: JoinOptions,
+) => {
+  const joinSeparator = `${elementWrapper}${separator}${elementWrapper}`;
+
+  const toReturn =
+    elements instanceof Array && elements.length > 0
+      ? `${elementWrapper}${elements.join(joinSeparator)}${elementWrapper}`
+      : undefined;
+
+  return typeof beforeReturn === 'function' ? beforeReturn(toReturn) : toReturn;
+};
+
+export default join;

striker-ui-api/src/lib/request_handlers/files/buildQueryFilesDetail.ts

@@ -1,11 +1,14 @@
-const buildQueryFilesDetail = ({ filesUUID }: { filesUUID?: string[] }) => {
+import join from '../../join';
-  let condFilesUUID = '';
 
-  if (filesUUID instanceof Array && filesUUID.length > 0) {
+const buildQueryFilesDetail = ({ filesUUID }: { filesUUID?: string[] }) => {
-    const catFilesUUID = `'${filesUUID.join("', '")}'`;
+  const condFilesUUID = join(filesUUID, {
+    beforeReturn: (toReturn) =>
+      toReturn ? `AND fil.file_uuid IN (${toReturn})` : '',
+    elementWrapper: "'",
+    separator: ', ',
+  });
 
-    condFilesUUID = `AND fil.file_uuid IN (${catFilesUUID})`;
+  console.log(`condFilesUUID=[${condFilesUUID}]`);
-  }
 
   return `
     SELECT

striker-ui-api/src/types/JoinOptions.d.ts

@@ -0,0 +1,5 @@
+type JoinOptions = {
+  beforeReturn?: (toReturn?: string) => string;
+  elementWrapper?: string;
+  separator?: string;
+};