Fixes storing of the correct value

src/main/java/org/jabref/gui/maintable/PersistenceVisualStateTable.java

@@ -2,14 +2,12 @@
 
 import java.util.stream.Collectors;
 
-import javafx.collections.ListChangeListener;
-import javafx.scene.control.TableColumn;
+import javafx.beans.InvalidationListener;
 
 import org.jabref.preferences.JabRefPreferences;
 
 /**
- * Keep track of changes made to the columns, like reordering or resizing.
- *
+ * Keep track of changes made to the columns (reordering, resorting, resizing).
  */
 public class PersistenceVisualStateTable {
 
@@ -20,21 +18,15 @@ public PersistenceVisualStateTable(final MainTable mainTable, JabRefPreferences
         this.mainTable = mainTable;
         this.preferences = preferences;
 
-        mainTable.getColumns().addListener(this::onColumnsChanged);
-        mainTable.getSortOrder().addListener(this::onColumnsChanged);
-        mainTable.getColumns().forEach(col -> col.widthProperty().addListener(obs -> updateColumnPreferences()));
-        mainTable.getColumns().forEach(col -> col.sortTypeProperty().addListener(obs -> updateColumnPreferences()));
-    }
-
-    private void onColumnsChanged(ListChangeListener.Change<? extends TableColumn<BibEntryTableViewModel, ?>> change) {
-        boolean changed = false;
-        while (change.next()) {
-            changed = true;
-        }
+        mainTable.getColumns().addListener((InvalidationListener) obs -> updateColumnPreferences());
+        mainTable.getSortOrder().addListener((InvalidationListener) obs -> updateColumnPreferences());
 
-        if (changed) {
-            updateColumnPreferences();
-        }
+        // As we store the ColumnModels of the MainTable, we need to add the listener to the ColumnModel properties,
+        // since the value is bound to the model after the listener to the column itself is called.
+        mainTable.getColumns().forEach(col ->
+                ((MainTableColumn<?>) col).getModel().widthProperty().addListener(obs -> updateColumnPreferences()));
+        mainTable.getColumns().forEach(col ->
+                ((MainTableColumn<?>) col).getModel().sortTypeProperty().addListener(obs -> updateColumnPreferences()));
     }
 
     /**