@primocaredentgroup/elettromedicali 0.1.0 → 0.1.1

package/src/component/ticketTriggers.ts

@@ -123,7 +123,7 @@ export const createTrigger = mutation({
             v.literal('specific_user'),
             v.literal('specific_supplier')
           ),
-          userId: v.optional(v.id('user_profiles')),
+          userId: v.optional(v.string()),
           supplierId: v.optional(v.id('suppliers')),
         })),
         message: v.string(),
@@ -264,7 +264,7 @@ export const updateTrigger = mutation({
             v.literal('specific_user'),
             v.literal('specific_supplier')
           ),
-          userId: v.optional(v.id('user_profiles')),
+          userId: v.optional(v.string()),
           supplierId: v.optional(v.id('suppliers')),
         })),
         message: v.string(),
@@ -377,7 +377,20 @@ export const reorderTriggers = mutation({
 
 export const executeTriggers = mutation({
   args: {
-    ticketId: v.id('maintenance_tasks'),
+    ticketId: v.string(),
+    ticketData: v.object({
+      status: v.string(),
+      priority: v.optional(v.string()),
+      supplierId: v.optional(v.string()),
+      clinicId: v.optional(v.string()),
+      customFields: v.optional(v.any()),
+      notes: v.optional(v.string()),
+    }),
+    deviceData: v.optional(v.object({
+      category: v.optional(v.string()),
+      brand: v.optional(v.string()),
+      model: v.optional(v.string()),
+    })),
     triggerOn: v.union(
       v.literal('create'),
       v.literal('status_change'),
@@ -386,16 +399,11 @@ export const executeTriggers = mutation({
       v.literal('sla_breach')
     ),
     oldStatus: v.optional(v.string()),
+    clinicRegion: v.optional(v.string()),
   },
   handler: async (ctx, args) => {
-    const ticket = await ctx.db.get(args.ticketId);
-    if (!ticket) {
-      throw new Error('Ticket not found');
-    }
-
-    const device = ticket.deviceId ? await ctx.db.get(ticket.deviceId) : null;
-
-    const clinic = await ctx.db.get(ticket.clinicId);
+    const ticket = args.ticketData;
+    const device = args.deviceData || null;
 
     const triggers = await ctx.db
       .query('ticket_triggers')
@@ -413,65 +421,39 @@ export const executeTriggers = mutation({
           result = value === args.triggerOn;
           break;
         case 'status':
-          const statusValues = Array.isArray(value) ? value : [value];
-          result = statusValues.includes(ticket.status);
+          result = (Array.isArray(value) ? value : [value]).includes(ticket.status);
           break;
         case 'category':
-          const categoryValues = Array.isArray(value) ? value : [value];
-          result = device ? categoryValues.includes(device.category) : false;
+          result = device ? (Array.isArray(value) ? value : [value]).includes(device.category) : false;
           break;
         case 'brand':
-          const brandValues = Array.isArray(value) ? value : [value];
-          result = device ? brandValues.includes(device.brand) : false;
+          result = device ? (Array.isArray(value) ? value : [value]).includes(device.brand) : false;
           break;
         case 'model':
-          const modelValues = Array.isArray(value) ? value : [value];
-          result = device ? modelValues.includes(device.model) : false;
+          result = device ? (Array.isArray(value) ? value : [value]).includes(device.model) : false;
           break;
         case 'region':
-          const regionValues = Array.isArray(value) ? value : [value];
-          const clinicRegion = (clinic as any)?.region;
-          result = clinicRegion ? regionValues.includes(clinicRegion) : false;
+          result = args.clinicRegion ? (Array.isArray(value) ? value : [value]).includes(args.clinicRegion) : false;
           break;
         case 'priority':
-          const priorityValues = Array.isArray(value) ? value : [value];
-          result = priorityValues.includes((ticket as any).priority);
+          result = (Array.isArray(value) ? value : [value]).includes(ticket.priority);
           break;
-        case 'custom_field':
+        case 'custom_field': {
           const { customFieldId, customFieldOperator } = condition;
-          if (!customFieldId) {
-            result = true;
-            break;
-          }
+          if (!customFieldId) { result = true; break; }
           const fieldValue = ticket.customFields?.[customFieldId];
-          const operator = customFieldOperator || 'equals';
-
-          switch (operator) {
-            case 'equals':
-              result = fieldValue === value;
-              break;
-            case 'not_equals':
-              result = fieldValue !== value;
-              break;
-            case 'contains':
-              result = fieldValue && String(fieldValue).includes(String(value));
-              break;
-            case 'greater_than':
-              result = fieldValue && Number(fieldValue) > Number(value);
-              break;
-            case 'less_than':
-              result = fieldValue && Number(fieldValue) < Number(value);
-              break;
-            case 'is_empty':
-              result = fieldValue === undefined || fieldValue === null || fieldValue === '';
-              break;
-            case 'is_not_empty':
-              result = fieldValue !== undefined && fieldValue !== null && fieldValue !== '';
-              break;
-            default:
-              result = true;
+          switch (customFieldOperator || 'equals') {
+            case 'equals': result = fieldValue === value; break;
+            case 'not_equals': result = fieldValue !== value; break;
+            case 'contains': result = fieldValue && String(fieldValue).includes(String(value)); break;
+            case 'greater_than': result = fieldValue && Number(fieldValue) > Number(value); break;
+            case 'less_than': result = fieldValue && Number(fieldValue) < Number(value); break;
+            case 'is_empty': result = fieldValue === undefined || fieldValue === null || fieldValue === ''; break;
+            case 'is_not_empty': result = fieldValue !== undefined && fieldValue !== null && fieldValue !== ''; break;
+            default: result = true;
           }
           break;
+        }
         default:
           result = true;
       }
@@ -484,206 +466,141 @@ export const executeTriggers = mutation({
       if (conditions.length === 1) return evaluateCondition(conditions[0]);
 
       const results = conditions.map(c => evaluateCondition(c));
-
       const andGroups: boolean[][] = [[]];
       let currentGroup = 0;
 
       for (let i = 0; i < conditions.length; i++) {
         andGroups[currentGroup].push(results[i]);
-
-        if (i < conditions.length - 1) {
-          const nextOp = conditions[i].nextOperator || 'AND';
-          if (nextOp === 'OR') {
-            currentGroup++;
-            andGroups[currentGroup] = [];
-          }
+        if (i < conditions.length - 1 && (conditions[i].nextOperator || 'AND') === 'OR') {
+          currentGroup++;
+          andGroups[currentGroup] = [];
         }
       }
 
-      const groupResults = andGroups.map(group => group.every(r => r));
-      return groupResults.some(r => r);
+      return andGroups.map(group => group.every(r => r)).some(r => r);
     };
 
+    const allActions: Array<{
+      triggerId: string;
+      triggerName: string;
+      actions: any;
+    }> = [];
+
+    let currentStatus = ticket.status;
+    let currentNotes = ticket.notes || '';
+
     for (const trigger of sortedTriggers) {
       const triggerConditions = trigger.conditions as any;
 
-      if (triggerConditions.triggerOn && triggerConditions.triggerOn !== args.triggerOn) {
-        continue;
-      }
+      if (triggerConditions.triggerOn && triggerConditions.triggerOn !== args.triggerOn) continue;
 
-      if (triggerConditions.conditionsList && triggerConditions.conditionsList.length > 0) {
-        const shouldExecute = evaluateConditionsWithOperators(triggerConditions.conditionsList);
+      let shouldExecute = false;
 
-        if (!shouldExecute) {
-          continue;
-        }
+      if (triggerConditions.conditionsList && triggerConditions.conditionsList.length > 0) {
+        shouldExecute = evaluateConditionsWithOperators(triggerConditions.conditionsList);
       } else {
-        if (trigger.conditions.triggerOn !== args.triggerOn) {
-          continue;
-        }
+        if (trigger.conditions.triggerOn !== args.triggerOn) continue;
 
         const conditionLogic = trigger.conditions.conditionLogic || 'AND';
         const conditionResults: boolean[] = [];
 
         let categoryMatch = true;
         if (!trigger.conditions.applyToAllCategories && device) {
-          if (!trigger.conditions.categories || !trigger.conditions.categories.includes(device.category)) {
-            categoryMatch = false;
-          }
+          if (!trigger.conditions.categories || !device.category || !trigger.conditions.categories.includes(device.category)) categoryMatch = false;
         }
         conditionResults.push(categoryMatch);
 
-        let brandMatch = true;
-        if (triggerConditions.brands && triggerConditions.brands.length > 0 && device) {
-          const brandOperator = triggerConditions.brandOperator || 'is';
+        if (triggerConditions.brands?.length > 0 && device) {
           const brandInList = triggerConditions.brands.includes(device.brand);
+          conditionResults.push((triggerConditions.brandOperator || 'is') === 'is' ? brandInList : !brandInList);
+        } else conditionResults.push(true);
 
-          if (brandOperator === 'is') {
-            brandMatch = brandInList;
-          } else {
-            brandMatch = !brandInList;
-          }
-        }
-        conditionResults.push(brandMatch);
-
-        let modelMatch = true;
-        if (triggerConditions.models && triggerConditions.models.length > 0 && device) {
-          const modelOperator = triggerConditions.modelOperator || 'is';
+        if (triggerConditions.models?.length > 0 && device) {
           const modelInList = triggerConditions.models.includes(device.model);
-
-          if (modelOperator === 'is') {
-            modelMatch = modelInList;
-          } else {
-            modelMatch = !modelInList;
-          }
-        }
-        conditionResults.push(modelMatch);
+          conditionResults.push((triggerConditions.modelOperator || 'is') === 'is' ? modelInList : !modelInList);
+        } else conditionResults.push(true);
 
         let regionMatch = true;
         if (trigger.conditions.applyToAllRegions === false) {
-          const clinicRegion = (clinic as any)?.region;
-
-          if (!clinicRegion || !trigger.conditions.regions || !trigger.conditions.regions.includes(clinicRegion)) {
-            regionMatch = false;
-          }
+          if (!args.clinicRegion || !trigger.conditions.regions?.includes(args.clinicRegion)) regionMatch = false;
         }
         conditionResults.push(regionMatch);
 
-        let statusMatch = true;
-        if (trigger.conditions.statuses && trigger.conditions.statuses.length > 0) {
-          const statusOperator = trigger.conditions.statusOperator || 'is';
-          const statusInList = trigger.conditions.statuses.includes(ticket.status);
-
-          if (statusOperator === 'is') {
-            statusMatch = statusInList;
-          } else {
-            statusMatch = !statusInList;
-          }
-        }
-        conditionResults.push(statusMatch);
-
-        let customFieldsMatch = true;
-        if (trigger.conditions.customFieldConditions && trigger.conditions.customFieldConditions.length > 0) {
-          for (const condition of trigger.conditions.customFieldConditions) {
-            const fieldValue = ticket.customFields?.[condition.fieldId];
-            let conditionMet = false;
-
-            switch (condition.operator) {
-              case 'equals':
-                conditionMet = fieldValue === condition.value;
-                break;
-              case 'not_equals':
-                conditionMet = fieldValue !== condition.value;
-                break;
-              case 'contains':
-                conditionMet = fieldValue && String(fieldValue).includes(String(condition.value));
-                break;
-              case 'greater_than':
-                conditionMet = fieldValue && Number(fieldValue) > Number(condition.value);
-                break;
-              case 'less_than':
-                conditionMet = fieldValue && Number(fieldValue) < Number(condition.value);
-                break;
-              case 'is_empty':
-                conditionMet = fieldValue === undefined || fieldValue === null || fieldValue === '';
-                break;
-              case 'is_not_empty':
-                conditionMet = fieldValue !== undefined && fieldValue !== null && fieldValue !== '';
-                break;
-            }
-
-            if (!conditionMet) {
-              customFieldsMatch = false;
-              break;
+        const statuses = trigger.conditions.statuses;
+        if (statuses && statuses.length > 0) {
+          const statusInList = statuses.includes(currentStatus);
+          conditionResults.push((trigger.conditions.statusOperator || 'is') === 'is' ? statusInList : !statusInList);
+        } else conditionResults.push(true);
+
+        const cfConditions = trigger.conditions.customFieldConditions;
+        if (cfConditions && cfConditions.length > 0) {
+          let cfMatch = true;
+          for (const cond of cfConditions) {
+            const fv = ticket.customFields?.[cond.fieldId];
+            let met = false;
+            switch (cond.operator) {
+              case 'equals': met = fv === cond.value; break;
+              case 'not_equals': met = fv !== cond.value; break;
+              case 'contains': met = fv && String(fv).includes(String(cond.value)); break;
+              case 'greater_than': met = fv && Number(fv) > Number(cond.value); break;
+              case 'less_than': met = fv && Number(fv) < Number(cond.value); break;
+              case 'is_empty': met = fv === undefined || fv === null || fv === ''; break;
+              case 'is_not_empty': met = fv !== undefined && fv !== null && fv !== ''; break;
             }
+            if (!met) { cfMatch = false; break; }
           }
-        }
-        conditionResults.push(customFieldsMatch);
+          conditionResults.push(cfMatch);
+        } else conditionResults.push(true);
 
-        let shouldExecute = false;
-        if (conditionLogic === 'OR') {
-          shouldExecute = conditionResults.some(result => result === true);
-        } else {
-          shouldExecute = conditionResults.every(result => result === true);
-        }
-
-        if (!shouldExecute) {
-          continue;
-        }
+        shouldExecute = conditionLogic === 'OR'
+          ? conditionResults.some(r => r)
+          : conditionResults.every(r => r);
       }
 
-      const updates: any = {};
-
-      const actions = trigger.actions as any;
-      if (actions.setPriority) {
-        updates.priority = actions.setPriority;
-      }
+      if (!shouldExecute) continue;
 
-      if (trigger.actions.changeStatus) {
-        updates.status = trigger.actions.changeStatus;
-      }
+      const triggerActions = trigger.actions as any;
+      const updates: any = {};
 
-      if (trigger.actions.assignSupplier) {
-        updates.supplierId = trigger.actions.assignSupplier;
-      }
+      if (triggerActions.setPriority) updates.priority = triggerActions.setPriority;
+      if (triggerActions.changeStatus) { updates.status = triggerActions.changeStatus; currentStatus = updates.status; }
+      if (triggerActions.assignSupplier) updates.supplierId = triggerActions.assignSupplier;
 
-      if (actions.applySlaRule && actions.assignSupplier) {
-        const priority = actions.setPriority || (ticket as any).priority || 'medium';
+      if (triggerActions.applySlaRule && triggerActions.assignSupplier) {
+        const priority = triggerActions.setPriority || ticket.priority || 'medium';
         const slaRule = await ctx.db
           .query('sla_rules')
           .withIndex('by_supplierId_priority', (q: any) =>
-            q.eq('supplierId', actions.assignSupplier).eq('priority', priority)
+            q.eq('supplierId', triggerActions.assignSupplier).eq('priority', priority)
           )
           .first();
-
-        if (slaRule) {
-          const slaDeadline = Date.now() + (slaRule.resolutionTimeHours * 60 * 60 * 1000);
-          updates.slaDeadline = slaDeadline;
-        }
-      } else if (trigger.actions.setSlaHours) {
-        const slaDeadline = Date.now() + (trigger.actions.setSlaHours * 60 * 60 * 1000);
-        updates.slaDeadline = slaDeadline;
+        if (slaRule) updates.slaDeadline = Date.now() + (slaRule.resolutionTimeHours * 60 * 60 * 1000);
+      } else if (triggerActions.setSlaHours) {
+        updates.slaDeadline = Date.now() + (triggerActions.setSlaHours * 60 * 60 * 1000);
       }
 
-      if (trigger.actions.addNote) {
-        const currentNotes = ticket.notes || '';
-        const newNote = `[Trigger: ${trigger.name}] ${trigger.actions.addNote}`;
-        updates.notes = currentNotes ? `${currentNotes}\n\n${newNote}` : newNote;
+      if (triggerActions.addNote) {
+        const newNote = `[Trigger: ${trigger.name}] ${triggerActions.addNote}`;
+        currentNotes = currentNotes ? `${currentNotes}\n\n${newNote}` : newNote;
+        updates.notes = currentNotes;
       }
 
       if (Object.keys(updates).length > 0) {
-        updates.updated_at = Date.now();
-        await ctx.db.patch(args.ticketId, updates);
+        allActions.push({
+          triggerId: trigger._id,
+          triggerName: trigger.name,
+          actions: updates,
+        });
 
         try {
-          const actionDescriptions: string[] = [];
-          if (updates.status) actionDescriptions.push(`Stato → ${updates.status}`);
+          const desc: string[] = [];
+          if (updates.status) desc.push(`Stato → ${updates.status}`);
           if (updates.supplierId) {
             const supplier = await ctx.db.get(updates.supplierId);
-            actionDescriptions.push(`Assegnato a ${(supplier as any)?.name || 'fornitore'}`);
+            desc.push(`Assegnato a ${(supplier as any)?.name || 'fornitore'}`);
           }
-          if (updates.priority) actionDescriptions.push(`Priorità → ${updates.priority}`);
-          if (updates.slaDeadline) actionDescriptions.push(`SLA impostato`);
+          if (updates.priority) desc.push(`Priorità → ${updates.priority}`);
+          if (updates.slaDeadline) desc.push(`SLA impostato`);
 
           await ctx.runMutation(api.ticketHistory.recordHistoryEvent, {
             ticketId: args.ticketId,
@@ -692,26 +609,24 @@ export const executeTriggers = mutation({
             triggerId: trigger._id,
             oldValue: updates.status ? ticket.status : undefined,
             newValue: updates.status,
-            oldSupplierId: updates.supplierId ? ticket.supplierId : undefined,
-            newSupplierId: updates.supplierId,
-            notes: actionDescriptions.join(', '),
+            notes: desc.join(', '),
             isSystemAction: true,
           });
         } catch (error) {
           console.error('Error recording trigger history:', error);
         }
       }
-
     }
 
-    return { success: true };
+    return { success: true, actions: allActions };
   },
 });
 
 export const checkAttachmentRequired = query({
   args: {
     deviceId: v.optional(v.id('devices')),
-    clinicId: v.optional(v.id('clinics')),
+    clinicId: v.optional(v.string()),
+    clinicRegion: v.optional(v.string()),
     status: v.optional(v.string()),
     priority: v.optional(v.union(v.literal('low'), v.literal('medium'), v.literal('high'))),
     customFields: v.optional(v.any()),
@@ -732,7 +647,6 @@ export const checkAttachmentRequired = query({
       }
 
       let device = null;
-      let clinic = null;
 
       try {
         if (args.deviceId) {
@@ -742,14 +656,6 @@ export const checkAttachmentRequired = query({
         console.error('Error fetching device:', e);
       }
 
-      try {
-        if (args.clinicId) {
-          clinic = await ctx.db.get(args.clinicId);
-        }
-      } catch (e) {
-        console.error('Error fetching clinic:', e);
-      }
-
     const evaluateCondition = (condition: any): boolean => {
       const { type, negated, value } = condition;
       let result = false;
@@ -776,8 +682,7 @@ export const checkAttachmentRequired = query({
           break;
         case 'region':
           const regionValues = Array.isArray(value) ? value : [value];
-          const clinicRegion = (clinic as any)?.region;
-          result = clinicRegion ? regionValues.includes(clinicRegion) : false;
+          result = args.clinicRegion ? regionValues.includes(args.clinicRegion) : false;
           break;
         case 'priority':
           const priorityValues = Array.isArray(value) ? value : [value];
@@ -886,8 +791,7 @@ export const checkAttachmentRequired = query({
 
         let regionMatch = true;
         if (triggerConditions.applyToAllRegions === false) {
-          const clinicRegion = (clinic as any)?.region;
-          if (!clinicRegion || !triggerConditions.regions || !triggerConditions.regions.includes(clinicRegion)) {
+          if (!args.clinicRegion || !triggerConditions.regions || !triggerConditions.regions.includes(args.clinicRegion)) {
             regionMatch = false;
           }
         }