Compare commits
1 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| 0567d9817f |
+3
-1
@@ -4,7 +4,9 @@
|
|||||||
|
|
||||||
### Bug Fixes
|
### Bug Fixes
|
||||||
|
|
||||||
- **Fixed deleting a wish with an owner assigned** (Issue #152): Resolved foreign key constraint error when removing a set from the wishlist that had an owner assigned
|
- **Fixed "Reset to Defaults" blanking all settings instead of restoring them** (Issue #149a, branch `bugfix/issue-149a`): "Reset to Defaults" was clearing all fields to empty/false instead of populating them with their actual default values
|
||||||
|
- `resetToDefaults()` now reads from `window.DEFAULT_CONFIG` and restores each field to its proper default, matching the same logic used on initial page load
|
||||||
|
- **Fixed deleting a wish with an owner assigned** (Issue #152, branch `bugfix/issue-152`): Resolved foreign key constraint error when removing a set from the wishlist that had an owner assigned
|
||||||
- Wish owners are now deleted before the wish itself, respecting the FK constraint
|
- Wish owners are now deleted before the wish itself, respecting the FK constraint
|
||||||
|
|
||||||
## 1.4
|
## 1.4
|
||||||
|
|||||||
@@ -285,18 +285,21 @@ function saveLiveConfiguration() {
|
|||||||
function resetToDefaults() {
|
function resetToDefaults() {
|
||||||
console.log('Resetting to defaults');
|
console.log('Resetting to defaults');
|
||||||
|
|
||||||
// Reset all form inputs
|
Object.keys(window.DEFAULT_CONFIG).forEach(varName => {
|
||||||
document.querySelectorAll('.config-toggle, .config-number, .config-text').forEach(input => {
|
const defaultValue = window.DEFAULT_CONFIG[varName];
|
||||||
if (input.type === 'checkbox') {
|
|
||||||
input.checked = false;
|
|
||||||
} else {
|
|
||||||
input.value = '';
|
|
||||||
}
|
|
||||||
});
|
|
||||||
|
|
||||||
// Update badges
|
const toggle = document.getElementById(varName);
|
||||||
Object.keys(window.CURRENT_CONFIG).forEach(varName => {
|
if (toggle && toggle.type === 'checkbox') {
|
||||||
updateConfigBadge(varName, null);
|
toggle.checked = defaultValue === true;
|
||||||
|
}
|
||||||
|
|
||||||
|
document.querySelectorAll(`input[data-var="${varName}"]:not(.config-static)`).forEach(input => {
|
||||||
|
if (input.type !== 'checkbox') {
|
||||||
|
input.value = defaultValue !== null && defaultValue !== undefined ? defaultValue : '';
|
||||||
|
}
|
||||||
|
});
|
||||||
|
|
||||||
|
updateConfigBadge(varName, defaultValue);
|
||||||
});
|
});
|
||||||
|
|
||||||
// Show status message
|
// Show status message
|
||||||
|
|||||||
Reference in New Issue
Block a user