RecentFileService: split out persistence methods

Author: ctrueden

src/main/java/org/scijava/io/DefaultRecentFileService.java

@@ -111,7 +111,7 @@ public void add(final String path) {
 		recentFiles.add(path);
 
 		// persist the updated list
-		prefService.putList(recentFiles, RECENT_FILES_KEY);
+		saveList();
 
 		if (present) {
 			// path already present; update linked module info
@@ -135,7 +135,7 @@ public boolean remove(final String path) {
 		final boolean success = recentFiles.remove(path);
 
 		// persist the updated list
-		prefService.putList(recentFiles, RECENT_FILES_KEY);
+		saveList();
 
 		// remove linked module info
 		final ModuleInfo info = recentModules.remove(path);
@@ -164,7 +164,7 @@ public List<String> getRecentFiles() {
 
 	@Override
 	public void initialize() {
-		recentFiles = prefService.getList(RECENT_FILES_KEY);
+		loadList();
 		recentModules = new HashMap<String, ModuleInfo>();
 		for (final String path : recentFiles) {
 			recentModules.put(path, createInfo(path));
@@ -183,6 +183,16 @@ protected void onEvent(final IOEvent event) {
 
 	// -- Helper methods --
 
+	/** Loads the list of recent files from persistent storage. */
+	private void loadList() {
+		recentFiles = prefService.getList(RECENT_FILES_KEY);
+	}
+
+	/** Saves the list of recent files to persistent storage. */
+	private void saveList() {
+		prefService.putList(recentFiles, RECENT_FILES_KEY);
+	}
+
 	/** Creates a {@link ModuleInfo} to reopen data at the given path. */
 	private ModuleInfo createInfo(final String path) {
 		// CTR FIXME: Avoid circular (and compile-time-unsafe) dependency between