ge-aerospace/shopdb
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix auto-populate: use attr() instead of data() for reliable reading

Browse Source 
jQuery's data() caches values; attr() reads directly from DOM

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
cproudlock
2025-12-08 15:21:17 -05:00
parent db797cb60f
commit 810dda29a8
1 changed files with 5 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
addprinter.asp
View File

@@ -328,10 +328,11 @@
var machineOption = $('#machineid option:selected'); var machineOption = $('#machineid option:selected');
var modelOption = $('#modelid option:selected'); var modelOption = $('#modelid option:selected');
var vendor = modelOption.data('vendor') || ''; // Use attr() instead of data() for reliable reading
var model = modelOption.data('model') || ''; var vendor = modelOption.attr('data-vendor') || '';
var machineAlias = machineOption.data('alias') || ''; var model = modelOption.attr('data-model') || '';
var machineNumber = machineOption.data('number') || ''; var machineAlias = machineOption.attr('data-alias') || '';
var machineNumber = machineOption.attr('data-number') || '';
// Only generate if we have model AND (machine OR CSF name) // Only generate if we have model AND (machine OR CSF name)
if (!vendor || (!csfName && !machineAlias && !machineNumber)) { if (!vendor || (!csfName && !machineAlias && !machineNumber)) {