Preferences change event listeners model fix

Former-commit-id: 5c3a21fc

plugins/org.jkiss.dbeaver.core/src/org/jkiss/dbeaver/registry/DataFormatterProfile.java

@@ -17,6 +17,7 @@
  */
 package org.jkiss.dbeaver.registry;
 
+import org.jkiss.dbeaver.model.DBPPreferenceListener;
 import org.jkiss.dbeaver.model.DBPPreferenceStore;
 import org.jkiss.dbeaver.model.DBPPropertyDescriptor;
 import org.jkiss.dbeaver.model.data.DBDDataFormatter;
@@ -33,7 +34,7 @@ import java.util.Map;
 /**
  * DataFormatterProfile
  */
-public class DataFormatterProfile implements DBDDataFormatterProfile {
+public class DataFormatterProfile implements DBDDataFormatterProfile, DBPPreferenceListener {
 
     private static final String PROP_LANGUAGE = "dataformat.profile.language"; //$NON-NLS-1$
     private static final String PROP_COUNTRY = "dataformat.profile.country"; //$NON-NLS-1$
@@ -51,6 +52,8 @@ public class DataFormatterProfile implements DBDDataFormatterProfile {
         this.name = profileName;
         this.store = store;
         loadProfile();
+
+        store.addPropertyChangeListener(this);
     }
 
     private void loadProfile()
@@ -226,4 +229,12 @@ public class DataFormatterProfile implements DBDDataFormatterProfile {
             }
         }
     }
+
+    @Override
+    public void preferenceChange(PreferenceChangeEvent event) {
+        if (event.getProperty() != null && event.getProperty().startsWith(DATAFORMAT_PREFIX)) {
+            // Reload this profile
+            loadProfile();
+        }
+    }
 }

plugins/org.jkiss.dbeaver.core/src/org/jkiss/dbeaver/ui/preferences/PrefPageDataFormat.java

@@ -361,7 +361,6 @@ public class PrefPageDataFormat extends TargetPrefPage
         } catch (Exception e) {
             log.warn(e);
         }
-        PrefUtils.savePreferenceStore(store);
     }
 
     @Override

plugins/org.jkiss.dbeaver.model/src/org/jkiss/dbeaver/model/impl/preferences/BundlePreferenceStore.java

@@ -222,12 +222,17 @@ public class BundlePreferenceStore extends AbstractPreferenceStore {
 
     @Override
     public void setValue(String name, String value) {
+        String oldValue = getString(name);
+        if (CommonUtils.equalObjects(oldValue, value)) {
+            return;
+        }
         if (getDefaultString(name).equals(value)) {
             props.remove(name);
         } else {
             props.put(name, value);
         }
         dirty = true;
+        firePropertyChangeEvent(name, oldValue, value);
     }
 
     @Override