rick/LEDMatrix
1
0
Fork 0
mirror of https://github.com/ChuckBuilds/LEDMatrix.git synced 2026-04-10 21:03:01 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: Update upload button onclick when reindexing custom feed rows

Browse Source 
Fix removeCustomFeedRow to update button onclick handlers that reference
file input IDs with _logo_<index> when rows are reindexed after deletion.

Previously, after deleting a row, the upload button's onclick still referenced
the old file input ID, causing the upload functionality to fail.

Now properly updates:
- getElementById('..._logo_<num>') patterns in onclick handlers
- Other _logo_<num> patterns in button onclick strings
- Function parameter indices in onclick handlers

This ensures upload buttons continue to work correctly after row deletion.
This commit is contained in:
Chuck
2026-01-08 12:31:26 -05:00
parent c80c23cd08
commit 0eb457fbc3
1 changed files with 23 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

23
web_interface/templates/v3/base.html
View File

@@ -4920,6 +4920,29 @@
input.setAttribute('onchange', onchange.replace(/,\s*\d+\s*,/, `, ${index},`));
}
}
// Update button onclick handlers that reference file input IDs with _logo_<num>
// Check for buttons (not just inputs) and update onclick if it contains _logo_ references
if (input.tagName === 'BUTTON') {
const onclick = input.getAttribute('onclick');
if (onclick) {
let updatedOnclick = onclick;
// Replace getElementById('..._logo_<num>') with getElementById('..._logo_<newIndex>')
updatedOnclick = updatedOnclick.replace(
/getElementById\(['"]([^'"]*_logo_)\d+['"]\)/g,
`getElementById('$1${index}')`
);
// Also handle patterns like _logo_<num> in other contexts
updatedOnclick = updatedOnclick.replace(
/(['"])([^'"]*_logo_)\d+(['"])/g,
`$1$2${index}$3`
);
// Update function call parameters (handleCustomFeedLogoUpload, removeCustomFeedRow, etc.)
updatedOnclick = updatedOnclick.replace(/,\s*\d+\s*,/g, `, ${index},`);
if (updatedOnclick !== onclick) {
input.setAttribute('onclick', updatedOnclick);
}
}
}
});
});
}