From 3de63f9cc8879bc0e471c48ce462d87ace02f6d0 Mon Sep 17 00:00:00 2001
From: espadonne <espadonne@outlook.com>
Date: Thu, 2 Apr 2026 20:44:30 -0400
Subject: [PATCH] log takes_imported from TakesImportModal
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

TakesImportModal bypasses handleImportTake entirely — it calls
addTrack directly via context. The onImport prop was passed but
never used, so takes_imported was never logged. Removed the
dead onImport prop and added logging directly in the modal.
---
 components/audio/DAW/Multitrack/MultitrackEditor.js | 2 +-
 components/audio/DAW/Multitrack/TakesImportModal.js | 6 +++++-
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/components/audio/DAW/Multitrack/MultitrackEditor.js b/components/audio/DAW/Multitrack/MultitrackEditor.js
index 2f08fff..f2d6b64 100644
--- a/components/audio/DAW/Multitrack/MultitrackEditor.js
+++ b/components/audio/DAW/Multitrack/MultitrackEditor.js
@@ -1212,7 +1212,7 @@ export default function MultitrackEditor({ availableTakes: propTakes = [], logOp
         show={showTakesModal}
         onHide={() => setShowTakesModal(false)}
         takes={availableTakes}
-        onImport={handleImportTake}
+        logOperation={logOperation}
       />
 
       {/* Clip Effects Modal */}
diff --git a/components/audio/DAW/Multitrack/TakesImportModal.js b/components/audio/DAW/Multitrack/TakesImportModal.js
index cef343a..d5410b1 100644
--- a/components/audio/DAW/Multitrack/TakesImportModal.js
+++ b/components/audio/DAW/Multitrack/TakesImportModal.js
@@ -7,7 +7,7 @@ import { FaFileAudio, FaClock, FaMusic } from 'react-icons/fa';
 import { useMultitrack } from '../../../../contexts/MultitrackContext';
 import { getAudioProcessor } from './AudioProcessor';
 
-export default function TakesImportModal({ show, onHide, takes = [] }) {
+export default function TakesImportModal({ show, onHide, takes = [], logOperation = null }) {
   const { addTrack, updateTrack } = useMultitrack();
   const [selectedTake, setSelectedTake] = useState(null);
   const [trackName, setTrackName] = useState('');
@@ -127,6 +127,10 @@ export default function TakesImportModal({ show, onHide, takes = [] }) {
 
       console.log('🎵 TakesImportModal: Track created immediately with ID:', newTrack.id);
 
+      if (logOperation) {
+        logOperation('takes_imported', { takeName: selectedTake.name || trackName });
+      }
+
       // Close modal immediately - track is already visible
       setSelectedTake(null);
       setTrackName('');