@yogiswara/honcho-editor-ui 3.4.15 → 3.4.16

package/dist/hooks/editor/useHonchoEditorBulk.d.ts

@@ -61,6 +61,7 @@ export declare function useHonchoEditorBulk(controller: Controller, eventID: str
     presets: Preset[];
     selectedBulkPreset: string;
     activePreset: Preset | null;
+    presetSummary: string;
     handleSelectBulkPreset: (presetId: string) => void;
     clearSelection: () => void;
     selectAllImages: () => void;

package/dist/hooks/editor/useHonchoEditorBulk.js

@@ -162,38 +162,55 @@ export function useHonchoEditorBulk(controller, eventID, firebaseUid) {
         return getCurrentAdjustmentsForCopy(selectedIds, currentBatch);
     }, [selectedIds, currentBatch]);
     // Calculate active preset based on preset_id from history (not value comparison)
-    const activePreset = useMemo(() => {
+    const areAdjustmentsEqual = (adj1, adj2) => {
+        // The fix is to use this more specific type for our keys array
+        const keys = [
+            'tempScore', 'tintScore', 'saturationScore', 'vibranceScore',
+            'exposureScore', 'contrastScore', 'highlightsScore', 'shadowsScore',
+            'whitesScore', 'blacksScore', 'clarityScore', 'sharpnessScore'
+        ];
+        for (const key of keys) {
+            // Now, TypeScript knows `key` can't be 'preset_id', so adj1[key] will be a number.
+            if (Math.round(adj1[key] || 0) !== Math.round(adj2[key] || 0)) {
+                return false;
+            }
+        }
+        return true;
+    };
+    const { activePreset, presetSummary } = useMemo(() => {
         if (selectedIds.length === 0) {
-            return null;
+            return { activePreset: null, presetSummary: 'NO_SELECTION' };
         }
-        // Get preset_id for all selected images from current batch
-        const selectedPresetIds = selectedIds.map(imageId => {
-            const adjustment = currentBatch.allImages[imageId];
-            return adjustment?.preset_id;
-        });
-        // Get the first preset_id to compare against
-        const firstPresetId = selectedPresetIds[0];
-        // Check if all selected images have the same preset_id
-        const allSamePresetId = selectedPresetIds.every(presetId => presetId === firstPresetId);
-        if (!allSamePresetId || !firstPresetId) {
-            // Either not all images have the same preset_id, or no preset_id at all
-            return null;
+        const presetIds = selectedIds.map(id => currentBatch.allImages[id]?.preset_id);
+        const uniquePresetIds = new Set(presetIds);
+        // Case 1: All selected images have NO preset.
+        if (uniquePresetIds.size === 1 && uniquePresetIds.has(undefined)) {
+            return { activePreset: null, presetSummary: 'NO_SELECTION' };
+        }
+        // Case 2: A mix of presets, or some with a preset and some without.
+        if (uniquePresetIds.size > 1) {
+            return { activePreset: null, presetSummary: 'MULTIPLE_PRESETS' };
+        }
+        // Case 3: All have the SAME preset ID. Now we must verify the adjustments.
+        const singlePresetId = presetIds[0];
+        if (typeof singlePresetId !== 'string') {
+            return { activePreset: null, presetSummary: 'MULTIPLE_PRESETS' };
+        }
+        const matchingPreset = presets.find(p => p.id === singlePresetId);
+        if (!matchingPreset) {
+            return { activePreset: null, presetSummary: 'MULTIPLE_PRESETS' };
+        }
+        // THE FINAL VERIFICATION
+        const firstImageAdjustments = currentBatch.allImages[selectedIds[0]];
+        const presetAdjustments = presetToAdjustmentState(matchingPreset);
+        if (firstImageAdjustments && areAdjustmentsEqual(firstImageAdjustments, presetAdjustments)) {
+            // ID and adjustments match perfectly.
+            return { activePreset: matchingPreset, presetSummary: 'SINGLE' };
+        }
+        else {
+            // This is a custom adjustment! The ID is there but the values don't match.
+            return { activePreset: null, presetSummary: 'NO_SELECTION' };
         }
-        // Find the preset that matches the common preset_id
-        const matchingPreset = presets.find(preset => preset.id === firstPresetId);
-        log.debug({
-            selectedIds,
-            selectedPresetIds,
-            firstPresetId,
-            allSamePresetId,
-            matchingPreset: matchingPreset?.name || 'none',
-            selectedImageAdjustments: selectedIds.map(id => ({
-                imageId: id,
-                preset_id: currentBatch.allImages[id]?.preset_id,
-                hasAdjustment: !!currentBatch.allImages[id]
-            }))
-        }, '[useHonchoEditorBulk] 🎨 Active preset calculation');
-        return matchingPreset || null;
     }, [selectedIds, currentBatch.allImages, presets]);
     // Update selectedBulkPreset when activePreset changes
     useEffect(() => {
@@ -458,6 +475,7 @@ export function useHonchoEditorBulk(controller, eventID, firebaseUid) {
         presets,
         selectedBulkPreset,
         activePreset,
+        presetSummary,
         handleSelectBulkPreset,
         clearSelection,
         selectAllImages,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yogiswara/honcho-editor-ui",
-  "version": "3.4.15",
+  "version": "3.4.16",
   "description": "A complete UI component library for the Honcho photo editor.",
   "main": "./dist/index.js",
   "module": "./dist/index.mjs",