X-Git-Url: http://git.nikiroo.be/?a=blobdiff_plain;f=src%2Fbe%2Fnikiroo%2Ffanfix%2Freader%2Fui%2FGuiReader.java;h=9f2fd3dad1056c6ba4635a70856e2983f330fad0;hb=31e28683c108bb5903864c58b0cc06bfa258fbfc;hp=519a507ea7d0fc1d080d3c04f602736a26afb45a;hpb=c3b229a10b147a2ca104a13ad0b43e49549b4ed9;p=nikiroo-utils.git

diff --git a/src/be/nikiroo/fanfix/reader/ui/GuiReader.java b/src/be/nikiroo/fanfix/reader/ui/GuiReader.java
index 519a507..9f2fd3d 100644
--- a/src/be/nikiroo/fanfix/reader/ui/GuiReader.java
+++ b/src/be/nikiroo/fanfix/reader/ui/GuiReader.java
@@ -2,6 +2,8 @@ package be.nikiroo.fanfix.reader.ui;
 
 import java.awt.Desktop;
 import java.awt.EventQueue;
+import java.awt.event.WindowAdapter;
+import java.awt.event.WindowEvent;
 import java.io.File;
 import java.io.IOException;
 import java.net.URISyntaxException;
@@ -79,14 +81,14 @@ class GuiReader extends BasicReader {
 	}
 
 	@Override
-	public void read() throws IOException {
+	public void read(boolean sync) throws IOException {
 		MetaData meta = getMeta();
 
 		if (meta == null) {
 			throw new IOException("No story to read");
 		}
 
-		read(meta.getLuid(), null);
+		read(meta.getLuid(), sync, null);
 	}
 
 	/**
@@ -107,6 +109,7 @@ class GuiReader extends BasicReader {
 		// TODO: improve presentation of update message
 		final VersionCheck updates = VersionCheck.check();
 		StringBuilder builder = new StringBuilder();
+		final Boolean[] done = new Boolean[] { false };
 
 		final JEditorPane updateMessage = new JEditorPane("text/html", "");
 		if (updates.isNewVersionAvailable()) {
@@ -162,21 +165,49 @@ class GuiReader extends BasicReader {
 					}
 				}
 
-				new GuiReaderFrame(GuiReader.this, typeFinal).setVisible(true);
+				try {
+					GuiReaderFrame gui = new GuiReaderFrame(GuiReader.this,
+							typeFinal);
+					gui.addWindowListener(new WindowAdapter() {
+						@Override
+						public void windowClosing(WindowEvent e) {
+							super.windowClosing(e);
+							done[0] = true;
+						}
+					});
+
+					gui.setVisible(true);
+				} catch (Exception e) {
+					Instance.getTraceHandler().error(e);
+					done[0] = true;
+				}
 			}
 		});
+
+		// This action must be synchronous, so wait until the frame is closed
+		while (!done[0]) {
+			try {
+				Thread.sleep(100);
+			} catch (InterruptedException e) {
+			}
+		}
 	}
 
 	@Override
-	public void start(File target, String program) throws IOException {
-		if (program == null) {
+	public void start(File target, String program, boolean sync)
+			throws IOException {
+
+		boolean handled = false;
+		if (program == null && !sync) {
 			try {
 				Desktop.getDesktop().browse(target.toURI());
+				handled = true;
 			} catch (UnsupportedOperationException e) {
-				super.start(target, program);
 			}
-		} else {
-			super.start(target, program);
+		}
+
+		if (!handled) {
+			super.start(target, program, sync);
 		}
 	}
 
@@ -218,9 +249,14 @@ class GuiReader extends BasicReader {
 	/**
 	 * "Open" the given {@link Story}. It usually involves starting an external
 	 * program adapted to the given file type.
+	 * <p>
+	 * Asynchronous method.
 	 * 
 	 * @param luid
 	 *            the luid of the {@link Story} to open
+	 * @param sync
+	 *            execute the process synchronously (wait until it is terminated
+	 *            before returning)
 	 * @param pg
 	 *            the optional progress (we may need to prepare the
 	 *            {@link Story} for reading
@@ -228,13 +264,13 @@ class GuiReader extends BasicReader {
 	 * @throws IOException
 	 *             in case of I/O errors
 	 */
-	void read(String luid, Progress pg) throws IOException {
+	void read(String luid, boolean sync, Progress pg) throws IOException {
 		File file = cacheLib.getFile(luid, pg);
 
 		// TODO: show a special page for the chapter?
 		// We could also implement an internal viewer, both for image and
 		// non-image documents
-		openExternal(getLibrary().getInfo(luid), file);
+		openExternal(getLibrary().getInfo(luid), file, sync);
 	}
 
 	/**
@@ -257,4 +293,38 @@ class GuiReader extends BasicReader {
 			Instance.getTraceHandler().error(e);
 		}
 	}
+
+	/**
+	 * Change the title of the given {@link Story}.
+	 * 
+	 * @param luid
+	 *            the luid of the {@link Story} to change
+	 * @param newTitle
+	 *            the new title
+	 */
+	void changeTitle(String luid, String newTitle) {
+		try {
+			cacheLib.changeTitle(luid, newTitle, null);
+		} catch (IOException e) {
+			Instance.getTraceHandler().error(e);
+		}
+	}
+
+	/**
+	 * Change the author of the given {@link Story}.
+	 * <p>
+	 * The author can be a new one, it needs not exist before hand.
+	 * 
+	 * @param luid
+	 *            the luid of the {@link Story} to change
+	 * @param newAuthor
+	 *            the new author
+	 */
+	void changeAuthor(String luid, String newAuthor) {
+		try {
+			cacheLib.changeAuthor(luid, newAuthor, null);
+		} catch (IOException e) {
+			Instance.getTraceHandler().error(e);
+		}
+	}
 }