WildDragonLLC/dragonflight
1
0
Fork 0
Code Issues 18 Pull requests 3 Projects Releases Packages Wiki Activity Actions

Fix recorders.html: rename handlers to avoid api.js shadowing (infinite recursion), fix resolution→recording_resolution

Browse source
This commit is contained in:
Zac Gaetano 2026-05-16 00:48:40 -04:00
parent 6f7a446aa5
commit ed52dfcafb
1 changed files with 13 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

23
services/web-ui/public/recorders.html
View file

@ -544,7 +544,7 @@
</div>
<div class="modal-footer">
<button class="btn btn-secondary" onclick="closeCreateModal()">Cancel</button>
<button class="btn btn-primary" onclick="createRecorder()">Create</button>
<button class="btn btn-primary" onclick="handleCreateRecorder()">Create</button>
</div>
</div>
</div>
@ -690,7 +690,7 @@
</div>
<div class="recorder-meta-item">
<span class="recorder-meta-label">Resolution</span>
<span class="recorder-meta-value">${recorder.resolution || 'Native'}</span>
<span class="recorder-meta-value">${recorder.recording_resolution || 'Native'}</span>
</div>
${isRecording ? `
<div class="recorder-meta-item">
@ -701,11 +701,11 @@
</div>
<div class="recorder-actions">
${isRecording ? `
<button class="btn btn-sm btn-secondary" onclick="stopRecorder('${recorder.id}')">⏹ Stop</button>
<button class="btn btn-sm btn-secondary" onclick="handleStopRecorder('${recorder.id}')">⏹ Stop</button>
` : `
<button class="btn btn-sm btn-primary" onclick="startRecorder('${recorder.id}')">⏺ Start</button>
<button class="btn btn-sm btn-primary" onclick="handleStartRecorder('${recorder.id}')">⏺ Start</button>
`}
<button class="btn btn-sm btn-secondary" onclick="deleteRecorder('${recorder.id}')">🗑 Delete</button>
<button class="btn btn-sm btn-secondary" onclick="handleDeleteRecorder('${recorder.id}')">🗑 Delete</button>
</div>
</div>
`;
@ -792,9 +792,12 @@
// ============================================================
// RECORDER OPERATIONS
// Named handleCreateRecorder / handleStartRecorder / handleStopRecorder /
// handleDeleteRecorder to avoid shadowing the api.js globals of the
// same name (which would cause infinite recursion).
// ============================================================
async function createRecorder() {
async function handleCreateRecorder() {
const name = document.getElementById('recorderName').value.trim();
const sourceType = document.getElementById('sourceType').value;
@ -819,7 +822,7 @@
source_type: sourceType,
source_config: sourceConfig,
recording_codec: document.getElementById('recordingCodec').value,
resolution: document.getElementById('resolution').value,
recording_resolution: document.getElementById('resolution').value,
project_id: document.getElementById('projectSelect').value,
};
@ -844,7 +847,7 @@
}
}
async function startRecorder(id) {
async function handleStartRecorder(id) {
try {
const result = await startRecorder(id);
if (result.success) {
@ -861,7 +864,7 @@
}
}
async function stopRecorder(id) {
async function handleStopRecorder(id) {
try {
const result = await stopRecorder(id);
if (result.success) {
@ -877,7 +880,7 @@
}
}
async function deleteRecorder(id) {
async function handleDeleteRecorder(id) {
if (!confirm('Are you sure you want to delete this recorder?')) return;
try {