now local settings modal load correctly the database yaml config

Author: RafaelGB

dist/main.js

@@ -110,7 +110,7 @@ export class DatabaseView extends TextFileView implements HoverParent {
     async initDatabase(): Promise<void> {
       LOGGER.info(`=>initDatabase ${this.file.path}`);
       this.diskConfig = new DatabaseInfo(this.file);
-      await this.diskConfig.initDatabaseconfigYaml();
+      await this.diskConfig.initDatabaseconfigYaml(this.plugin.settings.local_settings);
       const columns = await obtainColumnsFromFolder(this.diskConfig.yaml.columns);
       const metatadaColumns = await obtainMetadataColumns();
       columns.push(...metatadaColumns);

src/DatabaseView.tsx

@@ -10,7 +10,7 @@ interface GlobalSettings {
   logger_level_info: string;
 }
 
-interface LocalSettings {
+export interface LocalSettings {
   enable_show_state: boolean;
 }
 export interface DatabaseSettings {
@@ -109,7 +109,7 @@ export class SettingsModal extends Modal {
 
       modalEl.addClass('database-settings-modal');
 
-      this.settingsManager.constructUI(contentEl, this.view.file.basename, true);
+      this.settingsManager.constructUI(contentEl, this.view.file.basename, true, this.view);
     }
 
     onClose() {

src/Settings.ts

@@ -1,4 +1,5 @@
 import { RowType } from "cdm/RowTypeModel"
+import { LocalSettings } from "Settings"
 
 /** database column */
 export type DatabaseColumn = {
@@ -22,5 +23,5 @@ export type DatabaseYaml = {
     columns: Record<string, DatabaseColumn>
     /** database local configuration 
      * TODO typing*/
-    config?: Record<string, any>
+    config?: LocalSettings
 }

src/cdm/DatabaseModel.ts

@@ -7,6 +7,7 @@ import { LOGGER } from 'services/Logger';
 import { VaultManagerDB } from 'services/FileManagerService';
 import { convertDatabaseYamlToParsedString, hasFrontmatterKey } from 'parsers/DatabaseParser';
 import { NoteContentAction } from 'cdm/FolderModel';
+import { LocalSettings } from 'Settings';
 
 export default class DatabaseInfo {
     private file: TFile;
@@ -20,7 +21,7 @@ export default class DatabaseInfo {
      * @param file 
      * @returns 
      */
-    async initDatabaseconfigYaml():Promise<void> {
+    async initDatabaseconfigYaml(default_local_settings:LocalSettings):Promise<void> {
         LOGGER.info(`=>initDatabaseconfigYaml`,`file:${this.file.path}`);
         const databaseRaw = await VaultManagerDB.obtainContentFromTfile(this.file);
         if (!databaseRaw || !hasFrontmatterKey(databaseRaw))  throw new Error('No frontmatter found');
@@ -32,6 +33,8 @@ export default class DatabaseInfo {
         }
         const frontmatterRaw = match[1];
         this.yaml = parseYaml(frontmatterRaw);
+        // Add default config in case of does not exist on file yet
+        if(!this.yaml.config) this.yaml.config = default_local_settings;
         LOGGER.info(`<=initDatabaseconfigYaml`);
     }
     /**
@@ -109,10 +112,8 @@ export default class DatabaseInfo {
         await this.saveOnDisk();
     }
 
-    async updateConfig(config:Record<string,any>):Promise<void>{
-        for (let key in config) {
-            this.yaml.config[key] = config[key];
-        }
+    async updateConfig<K extends keyof LocalSettings>(key: K, value: LocalSettings[K]): Promise<void> {
+        this.yaml.config[key] = value;
         await this.saveOnDisk();
-    }
+      }
 }

src/services/DatabaseInfo.ts

@@ -1,11 +1,12 @@
+import { DatabaseView } from "DatabaseView";
 import { Setting } from "obsidian";
 import { LOGGER } from "services/Logger";
 import { SettingsManager } from "Settings";
 import { AbstractSettingsHandler } from "settings/handlers/AbstractSettingHandler";
 
 export class LoggerLevelInfoDropDownHandler extends AbstractSettingsHandler {
     settingTitle: string = 'Enable debug mode';
-    handle(settingsManager: SettingsManager, containerEl: HTMLElement, local: boolean): [string, string][] {
+    handle(settingsManager: SettingsManager, containerEl: HTMLElement, local: boolean, view?: DatabaseView): [string, string][] {
         if(!local && settingsManager.plugin.settings.global_settings.enable_debug_mode){
             new Setting(containerEl)
             .setName('Select level info of logs')
@@ -31,7 +32,7 @@ export class LoggerLevelInfoDropDownHandler extends AbstractSettingsHandler {
             });
         }
         if (this.nextHandler) {
-            return this.nextHandler.handle(settingsManager, containerEl, local);
+            return this.nextHandler.handle(settingsManager, containerEl, local, view);
         }
         return this.listOfErrors;
     }

src/settings/handlers/developer/LoggerLevelInfoDropDownHandler.ts

@@ -2,9 +2,10 @@ import { LOGGER } from "services/Logger";
 import { SettingsManager } from "Settings";
 import { add_toggle } from "settings/SettingsComponents";
 import { AbstractSettingsHandler } from "settings/handlers/AbstractSettingHandler";
+import { DatabaseView } from "DatabaseView";
 export class LoggerToggleHandler extends AbstractSettingsHandler {
     settingTitle: string = 'Enable developer settings. Logger will be enabled';
-    handle(settingsManager: SettingsManager, containerEl: HTMLElement, local: boolean): [string, string][] {
+    handle(settingsManager: SettingsManager, containerEl: HTMLElement, local: boolean, view?: DatabaseView): [string, string][] {
         // pass if modal opened from local settings
         if(!local){
             const logger_togle_promise = async (value: boolean): Promise<void> => {
@@ -30,7 +31,7 @@ export class LoggerToggleHandler extends AbstractSettingsHandler {
         }
         // Check next handler
         if (this.nextHandler) {
-            return this.nextHandler.handle(settingsManager, containerEl, local);
+            return this.nextHandler.handle(settingsManager, containerEl, local, view);
         }
 
         return this.listOfErrors;

src/settings/handlers/developer/LoggerToggleHandler.ts

@@ -11,7 +11,7 @@ export class TableStateToggleHandler extends AbstractSettingsHandler {
                 // Check context to define correct promise
                 if(local){
                     // Persist value
-                    view .diskConfig.updateConfig({enable_show_state: value});
+                    view .diskConfig.updateConfig('enable_show_state', value);
                 }else{
                     // switch table state on/off
                     const update_local_settings = settingsManager.plugin.settings.local_settings;
@@ -22,17 +22,18 @@ export class TableStateToggleHandler extends AbstractSettingsHandler {
                     });
                 }
             }
+
             add_toggle(
                 containerEl,
                 this.settingTitle, 
                 "This will show/hide properties of the table on the bottom of the view",
-                settingsManager.plugin.settings.local_settings.enable_show_state, 
+                local ? view.diskConfig.yaml.config.enable_show_state : settingsManager.plugin.settings.local_settings.enable_show_state, 
                 table_state_togle_promise
             );
         }
         // Check next handler
         if (this.nextHandler) {
-            return this.nextHandler.handle(settingsManager, containerEl, local);
+            return this.nextHandler.handle(settingsManager, containerEl, local, view);
         }
         return this.listOfErrors;
     }