Critical fix: sanitize_settings now merges with existing settings to prevent data loss from standalone forms (Logs tab)

This commit is contained in:
Forkless
2026-05-11 21:57:24 +02:00
parent 66f8f75bf4
commit 61fe8b626d
+5 -1
View File
@@ -464,7 +464,11 @@ class Settings {
* @return array<string,mixed>
*/
public function sanitize_settings( array $input ): array {
$clean = [];
// Merge with existing settings so standalone forms (e.g., Logs tab)
// don't wipe unrelated fields.
$existing = get_option( 'piperless_settings', [] );
$input = array_merge( $existing, $input );
$clean = [];
$clean['piper_binary'] = sanitize_text_field( $input['piper_binary'] ?? '' );
$clean['models_directory'] = sanitize_text_field( $input['models_directory'] ?? '' );