improving config change detection on switches

src/common/config/public/ClientConfig.ts

@@ -2,8 +2,7 @@
 import 'reflect-metadata';
 import {SortingMethods} from '../../entities/SortingMethods';
 import {UserRoles} from '../../entities/UserDTO';
-import {SubConfigClass} from 'typeconfig/src/decorators/class/SubConfigClass';
+import {ConfigProperty, SubConfigClass} from 'typeconfig/common';
-import {ConfigProperty} from 'typeconfig/src/decorators/property/ConfigPropoerty';
 
 
 export module ClientConfig {
@@ -155,7 +154,6 @@ export module ClientConfig {
   @SubConfigClass()
   export class Config {
 
-
     @ConfigProperty({volatile: true})
     upTime: string;
     @ConfigProperty({volatile: true})

src/common/entities/UserDTO.ts

@@ -6,8 +6,7 @@ export enum UserRoles {
   Guest = 2,
   User = 3,
   Admin = 4,
-  Developer = 5,
+  Developer = 5
-
 }
 
 export interface UserDTO {

src/frontend/app/ui/settings/_abstract/settings-entry/settings-entry.component.ts

@@ -156,7 +156,10 @@ export class SettingsEntryComponent implements ControlValueAccessor, Validator,
   }
 
   validate(control: FormControl): ValidationErrors {
-    if (!this.required || (this.state && this.state.value && this.state.value !== '')) {
+    if (!this.required || (this.state &&
+      typeof this.state.value !== 'undefined' &&
+      this.state.value !== null &&
+      this.state.value !== '')) {
       return null;
     }
     return {required: true};

src/frontend/app/ui/settings/other/other.settings.component.html

@@ -14,8 +14,7 @@
         name="Threading"
         description="Runs directory scanning and thumbnail generation (only for Jimp) in a different thread."
         i18n-description i18n-name
-        [ngModel]="states.Server.enabled"
+        [ngModel]="states.Server.enabled">
-        required="true">
       </app-settings-entry>
 
 
@@ -37,32 +36,28 @@
         name="Scroll based thumbnail generation"
         description="Those thumbnails get higher priority that are visible on the screen."
         i18n-description i18n-name
-        [ngModel]="states.Client.enableOnScrollThumbnailPrioritising"
+        [ngModel]="states.Client.enableOnScrollThumbnailPrioritising">
-        required="true">
       </app-settings-entry>
 
       <app-settings-entry
         name="Lazy image rendering"
         description="Shows only the required amount of photos at once. Renders more if page bottom is reached."
         i18n-description i18n-name
-        [ngModel]="states.Client.enableOnScrollRendering"
+        [ngModel]="states.Client.enableOnScrollRendering">
-        required="true">
       </app-settings-entry>
 
       <app-settings-entry
         name="Cache"
         description="Caches directory contents and search results for better performance."
         i18n-description i18n-name
-        [ngModel]="states.Client.enableCache"
+        [ngModel]="states.Client.enableCache">
-        required="true">
       </app-settings-entry>
 
       <app-settings-entry
         name="Caption first naming"
         description="Show the caption (IPTC 120) tags from the EXIF data instead of the filenames."
         i18n-description i18n-name
-        [ngModel]="states.Client.captionFirstNaming"
+        [ngModel]="states.Client.captionFirstNaming">
-        required="true">
       </app-settings-entry>
 
 

src/frontend/app/ui/settings/photo/photo.settings.component.html

@@ -39,8 +39,7 @@
         name="Converting"
         description="Downsizes photos for faster preview loading. (Zooming in to the photo loads the original)."
         i18n-description i18n-name
-        [ngModel]="states.client.Converting.enabled"
+        [ngModel]="states.client.Converting.enabled">
-        required="true">
       </app-settings-entry>
 
       <app-settings-entry
@@ -49,8 +48,7 @@
         i18n-description i18n-name
         [ngModel]="states.server.Converting.onTheFly"
         [simplifiedMode]="simplifiedMode"
-        [disabled]="!states.client.Converting.enabled.value"
+        [disabled]="!states.client.Converting.enabled.value">
-        required="true">
       </app-settings-entry>
 
       <app-settings-entry

src/frontend/app/ui/settings/search/search.settings.component.html

@@ -32,16 +32,14 @@
           name="Autocomplete"
           description="Show hints while typing search query."
           i18n-description i18n-name
-          [ngModel]="states.AutoComplete.enabled"
+          [ngModel]="states.AutoComplete.enabled">
-          required="true">
         </app-settings-entry>
 
         <app-settings-entry
           name="Instant search"
           description="Enables showing search results, while typing search query."
           i18n-description i18n-name
-          [ngModel]="states.instantSearchEnabled"
+          [ngModel]="states.instantSearchEnabled">
-          required="true">
         </app-settings-entry>
 
 

src/frontend/app/ui/settings/thumbnail/thumbnail.settings.component.html

@@ -12,8 +12,7 @@
         description="High quality may be slow. Especially with Jimp."
         i18n-description i18n-name
         [ngModel]="states.server.qualityPriority"
-        [simplifiedMode]="simplifiedMode"
+        [simplifiedMode]="simplifiedMode">
-        required="true">
       </app-settings-entry>