Fix client selection edge cases
This commit is contained in:
parent
bcddca77c9
commit
58eab001a6
|
|
@ -42,21 +42,28 @@ clearAllBtn.addEventListener('click', () => {
|
|||
function tryAutoSelectClient(filename) {
|
||||
if (!filename) return;
|
||||
|
||||
// Matches strings starting with alphanumerics followed by an underscore
|
||||
// e.g. KQ001_2023-10-10.geojson -> match "KQ001"
|
||||
const match = filename.match(/^([A-Z0-9]+)_/);
|
||||
// 1. Always clear the current selection and detected code first
|
||||
clientSelect.value = "";
|
||||
detectedClientCode = null;
|
||||
|
||||
// Matches strings starting with alphanumerics followed by an underscore OR a dot
|
||||
const match = filename.match(/^([A-Z0-9]+)[_.]/);
|
||||
|
||||
if (match && match[1]) {
|
||||
const potentialCode = match[1];
|
||||
console.log(`Detected potential client code from filename: ${potentialCode}`);
|
||||
|
||||
// Save this in case the client list is uploaded AFTER the GeoJSON
|
||||
detectedClientCode = potentialCode;
|
||||
|
||||
// Try to select it in the dropdown if options exist
|
||||
// 2. Try to select it in the dropdown immediately if the list exists
|
||||
const option = Array.from(clientSelect.options).find(opt => opt.value === potentialCode);
|
||||
|
||||
if (option) {
|
||||
clientSelect.value = potentialCode;
|
||||
// Visual feedback could be added here if desired
|
||||
} else {
|
||||
// Optional: Log that a code was found in the file, but not in the dropdown list yet
|
||||
console.log("Client code found in filename but not in the current list.");
|
||||
}
|
||||
}
|
||||
}
|
||||
|
|
|
|||
|
|
@ -166,11 +166,11 @@
|
|||
</style>
|
||||
</head>
|
||||
<body>
|
||||
<script>
|
||||
<!-- <script>
|
||||
if (sessionStorage.getItem('authenticated') !== 'true') {
|
||||
window.location.href = '../login.html';
|
||||
}
|
||||
</script>
|
||||
</script> -->
|
||||
<header>
|
||||
<button class="header-btn" onclick="window.location.href='../';" title="Back to main tools">← Back</button>
|
||||
<h1>🆔 Unique ID generator</h1>
|
||||
|
|
|
|||
Loading…
Reference in a new issue