fix: remove StorableList and StorableModel

src/main/java/viewtify/model/Preferences.java

@@ -37,7 +37,7 @@ public abstract class Preferences implements Storable<Preferences>, Extensible {
     public final Preference<String> name = initialize("");
 
     /** The grouping container. */
-    Storable container;
+    protected Storable container;
 
     /**
      * Hide constructor.
@@ -208,6 +208,16 @@ public static <P extends Preferences> List<P> all(Class<P> type) {
         return CACHE.computeIfAbsent(type, PreferencesList::new);
     }
 
+    /**
+     * Save the specified user preferences.
+     */
+    public static <P extends Preferences> void store(Class<P> type) {
+        CACHE.computeIfPresent(type, (x, list) -> {
+            list.store();
+            return list;
+        });
+    }
+
     /**
      * Preference value.
      */

src/main/java/viewtify/model/PreferencesList.java

@@ -9,22 +9,27 @@
  */
 package viewtify.model;
 
-import java.util.concurrent.CopyOnWriteArrayList;
+import java.util.ArrayList;
+
+import com.sun.javafx.collections.ObservableListWrapper;
 
 import kiss.Managed;
 import kiss.Storable;
 import viewtify.Viewtify;
 
-@SuppressWarnings("serial")
-final class PreferencesList<E extends Preferences> extends CopyOnWriteArrayList<E> implements Storable<PreferencesList<E>> {
+final class PreferencesList<E extends Preferences> extends ObservableListWrapper<E> implements Storable<PreferencesList<E>> {
 
     /** The model id. */
     private final String id;
 
+    private boolean restoring;
+
     /**
      * Hide constructor.
      */
     PreferencesList(Class<E> type) {
+        super(new ArrayList());
+
         Managed annotation = type.getAnnotation(Managed.class);
         if (annotation == null) {
             id = type.getName();
@@ -52,11 +57,73 @@ final class PreferencesList<E extends Preferences> extends CopyOnWriteArrayList<
         });
     }
 
+    /**
+     * {@inheritDoc}
+     */
+    @Override
+    public PreferencesList<E> restore() {
+        try {
+            restoring = true;
+            return Storable.super.restore();
+        } finally {
+            restoring = false;
+        }
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    @Override
+    public PreferencesList<E> store() {
+        if (!restoring) {
+            Storable.super.store();
+        }
+        return this;
+    }
+
     /**
      * {@inheritDoc}
      */
     @Override
     public final String locate() {
         return Viewtify.UserPreference.exact().file(id + ".json").path();
     }
+
+    /**
+     * {@inheritDoc}
+     */
+    @Override
+    protected void doAdd(int index, E element) {
+        super.doAdd(index, element);
+        store();
+
+        if (element != null && element.name.is("") && element.getClass().getSimpleName().equals("Task")) {
+            new Error().printStackTrace();
+        }
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    @Override
+    protected E doSet(int index, E element) {
+        E e = super.doSet(index, element);
+        store();
+
+        if (element != null && element.name.is("") && element.getClass().getSimpleName().equals("Task")) {
+            new Error().printStackTrace();
+        }
+
+        return e;
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    @Override
+    protected E doRemove(int index) {
+        E e = super.doRemove(index);
+        store();
+        return e;
+    }
 }

src/main/java/viewtify/ui/helper/CollectableHelper.java

@@ -139,6 +139,10 @@ default Self items(Iterable<E> items) {
      * @return Chainable API.
      */
     default Self items(List<E> items) {
+        if (items instanceof ObservableList) {
+            return items((ObservableList<E>) items);
+        }
+
         if (items == null || items.isEmpty()) {
             modifyItemUISafely(list -> {
                 list.clear();