removed save-on-shutdown, using save-on-change pragma now for data st…

…orage
JanStrauss · Sep 27, 2014 · 0130d7b · 0130d7b
1 parent b5d043c
commit 0130d7b
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 4 deletions.
diff --git a/src/main/java/eu/over9000/skadi/SkadiMain.java b/src/main/java/eu/over9000/skadi/SkadiMain.java
@@ -163,8 +163,6 @@ public void run() {
 				for (final Channel channel : ChannelManager.getInstance().getChannels()) {
 					channel.closeStreamAndChat();
 				}
-				SkadiLogging.log("SAVING DATA..");
-				PersistenceManager.getInstance().saveData();
 
 				if (SkadiMain.this.lockingSocket != null) {
 					SkadiLogging.log("RELEASING LOCK..");

diff --git a/src/main/java/eu/over9000/skadi/channel/ChannelManager.java b/src/main/java/eu/over9000/skadi/channel/ChannelManager.java
@@ -24,6 +24,7 @@
 import java.util.ArrayList;
 import java.util.List;
 
+import eu.over9000.skadi.io.PersistenceManager;
 import eu.over9000.skadi.logging.SkadiLogging;
 import eu.over9000.skadi.util.ChannelDataRetriever;
 import eu.over9000.skadi.util.ChannelURLUtil;
@@ -87,6 +88,7 @@ public boolean addChannel(final String url, final boolean userAdd) {
 
 		if (userAdd) {
 			SkadiLogging.log("added new channel " + fixedUrl);
+			PersistenceManager.getInstance().saveData();
 		}
 
 		return true;
@@ -99,6 +101,9 @@ public void removeChannel(final Channel channel) {
 				listener.removed(channel, this.channels.size());
 			}
 		}
+
+		PersistenceManager.getInstance().saveData();
+
 		SkadiLogging.log("removed channel " + channel.getURL());
 	}
 

diff --git a/src/main/java/eu/over9000/skadi/gui/SettingsDialog.java b/src/main/java/eu/over9000/skadi/gui/SettingsDialog.java
@@ -142,6 +142,9 @@ public void actionPerformed(final ActionEvent e) {
 					        .isSelected();
 
 					SkadiLogging.log("settings updated");
+
+					PersistenceManager.getInstance().saveData();
+
 					SettingsDialog.this.setVisible(false);
 				}
 			});

diff --git a/src/main/java/eu/over9000/skadi/io/PersistenceManager.java b/src/main/java/eu/over9000/skadi/io/PersistenceManager.java
@@ -88,7 +88,7 @@ private PersistenceManager() {
 		}
 	}
 
-	public void saveData() {
+	public synchronized void saveData() {
 		try {
 			final DocumentBuilderFactory documentBuilderFactory = DocumentBuilderFactory.newInstance();
 			final DocumentBuilder documentBuilder = documentBuilderFactory.newDocumentBuilder();
@@ -170,7 +170,7 @@ private void loadChannels(final Element channelsRootElement) {
 
 	}
 
-	public void loadData() {
+	public synchronized void loadData() {
 
 		try {
 			final FileInputStream stream = new FileInputStream(this.dataFile);

diff --git a/src/main/java/eu/over9000/skadi/util/ImportFollowedUtil.java b/src/main/java/eu/over9000/skadi/util/ImportFollowedUtil.java
@@ -32,6 +32,7 @@
 
 import eu.over9000.skadi.channel.ChannelManager;
 import eu.over9000.skadi.gui.ImportDialog;
+import eu.over9000.skadi.io.PersistenceManager;
 import eu.over9000.skadi.logging.SkadiLogging;
 
 /**
@@ -132,6 +133,9 @@ public static String importFollowedChannelsFromTwitch(final String username, fin
 				count++;
 			}
 		}
+
+		PersistenceManager.getInstance().saveData();
+
 		return "Imported " + count + " of " + newChannels.size() + " followed channels.";
 	}
 }