X-Git-Url: http://git.nikiroo.be/?a=blobdiff_plain;f=src%2Fbe%2Fnikiroo%2Ffanfix%2FInstance.java;h=74897491f7a0b93c60893b828fac0ec82ef5111a;hb=7d4c3830e2128253dbb0212d1afb737aa28e033b;hp=b6849c279dd298f3b1d1f139367c613a87d9c980;hpb=f83510cf13d55c8700de01ee06ae752da1d102b4;p=nikiroo-utils.git

diff --git a/src/be/nikiroo/fanfix/Instance.java b/src/be/nikiroo/fanfix/Instance.java
index b6849c2..7489749 100644
--- a/src/be/nikiroo/fanfix/Instance.java
+++ b/src/be/nikiroo/fanfix/Instance.java
@@ -42,7 +42,42 @@ public class Instance {
 	private static TraceHandler tracer;
 	private static TempFiles tempFiles;
 
-	static {
+	private static boolean init;
+
+	/**
+	 * Initialise the instance -- if already initialised, nothing will happen.
+	 * <p>
+	 * Before calling this method, you may call
+	 * {@link Bundles#setDirectory(String)} if wanted.
+	 */
+	static public void init() {
+		init(false);
+	}
+
+	/**
+	 * Initialise the instance -- if already initialised, nothing will happen
+	 * unless you pass TRUE to <tt>force</tt>.
+	 * <p>
+	 * Before calling this method, you may call
+	 * {@link Bundles#setDirectory(String)} if wanted.
+	 * <p>
+	 * Note: forcing the initialisation can be dangerous, so make sure to only
+	 * make it under controlled circumstances -- for instance, at the start of
+	 * the program, you could call {@link Instance#init()}, change some settings
+	 * because you want to force those settings (it will also forbid users to
+	 * change them!) and then call {@link Instance#init(boolean)} with
+	 * <tt>force</tt> set to TRUE.
+	 * 
+	 * @param force
+	 *            force the initialisation even if already initialised
+	 */
+	static public void init(boolean force) {
+		if (init && !force) {
+			return;
+		}
+
+		init = true;
+
 		// Before we can configure it:
 		Boolean debug = checkEnv("DEBUG");
 		boolean trace = debug != null && debug;
@@ -58,7 +93,7 @@ public class Instance {
 		createConfigs(configDir, false);
 
 		// Proxy support
-		Proxy.use(Instance.getConfig().getString(Config.USE_PROXY));
+		Proxy.use(Instance.getConfig().getString(Config.NETWORK_PROXY));
 
 		// update tracer:
 		if (debug == null) {
@@ -73,17 +108,16 @@ public class Instance {
 		lib = createDefaultLibrary(remoteDir);
 
 		// create cache and TMP
-		Image.setTemporaryFilesRoot(new File(configDir, "tmp.images"));
-		File tmp = getFile(Config.CACHE_DIR);
-		if (tmp == null) {
-			// Could have used: System.getProperty("java.io.tmpdir")
-			tmp = new File(configDir, "tmp");
+		File tmp = getFile(Config.CACHE_DIR, new File(configDir, "tmp"));
+		if (!tmp.isAbsolute()) {
+			tmp = new File(configDir, tmp.getPath());
 		}
-		String ua = config.getString(Config.USER_AGENT);
+		Image.setTemporaryFilesRoot(new File(tmp.getParent(), "tmp.images"));
+
+		String ua = config.getString(Config.NETWORK_USER_AGENT, "");
 		try {
-			int hours = config.getInteger(Config.CACHE_MAX_TIME_CHANGING, -1);
-			int hoursLarge = config
-					.getInteger(Config.CACHE_MAX_TIME_STABLE, -1);
+			int hours = config.getInteger(Config.CACHE_MAX_TIME_CHANGING, 0);
+			int hoursLarge = config.getInteger(Config.CACHE_MAX_TIME_STABLE, 0);
 			cache = new DataLoader(tmp, ua, hours, hoursLarge);
 		} catch (IOException e) {
 			tracer.error(new IOException(
@@ -94,18 +128,12 @@ public class Instance {
 		cache.setTraceHandler(tracer);
 
 		// readerTmp / coverDir
-		readerTmp = getFile(UiConfig.CACHE_DIR_LOCAL_READER);
-		if (readerTmp == null) {
-			readerTmp = new File(configDir, "tmp-reader");
-		}
+		readerTmp = getFile(UiConfig.CACHE_DIR_LOCAL_READER, new File(
+				configDir, "tmp-reader"));
 
-		coverDir = getFile(Config.DEFAULT_COVERS_DIR);
-		if (coverDir != null && !coverDir.exists()) {
-			tracer.error(new IOException(
-					"The 'default covers' directory does not exists: "
-							+ coverDir));
-			coverDir = null;
-		}
+		coverDir = getFile(Config.DEFAULT_COVERS_DIR, new File(configDir,
+				"covers"));
+		coverDir.mkdirs();
 
 		try {
 			tempFiles = new TempFiles("fanfix");
@@ -297,7 +325,7 @@ public class Instance {
 	 */
 	public static boolean isVersionCheckNeeded() {
 		try {
-			long wait = config.getInteger(Config.UPDATE_INTERVAL, 1) * 24 * 60
+			long wait = config.getInteger(Config.NETWORK_UPDATE_INTERVAL, 0) * 24 * 60
 					* 60 * 1000;
 			if (wait >= 0) {
 				String lastUpString = IOUtils.readSmallFile(new File(configDir,
@@ -429,9 +457,9 @@ public class Instance {
 
 		String remoteLib = config.getString(Config.DEFAULT_LIBRARY);
 		if (remoteLib == null || remoteLib.trim().isEmpty()) {
-			String libDir = System.getProperty("fanfix.libdir");
+			String libDir = System.getenv("BOOKS_DIR");
 			if (libDir == null || libDir.isEmpty()) {
-				libDir = config.getString(Config.LIBRARY_DIR);
+				libDir = config.getString(Config.LIBRARY_DIR, "$HOME/Books");
 			}
 			try {
 				lib = new LocalLibrary(getFile(libDir));
@@ -441,6 +469,7 @@ public class Instance {
 								+ getFile(libDir), e));
 			}
 		} else {
+			Exception ex = null;
 			int pos = remoteLib.lastIndexOf(":");
 			if (pos >= 0) {
 				String port = remoteLib.substring(pos + 1).trim();
@@ -459,13 +488,14 @@ public class Instance {
 								lib);
 
 					} catch (Exception e) {
+						ex = e;
 					}
 				}
 			}
 
 			if (lib == null) {
 				tracer.error(new IOException(
-						"Cannot create remote library for: " + remoteLib));
+						"Cannot create remote library for: " + remoteLib, ex));
 			}
 		}
 
@@ -477,8 +507,13 @@ public class Instance {
 	 * 
 	 * @return the path
 	 */
-	private static File getFile(Config id) {
-		return getFile(config.getString(id));
+	private static File getFile(Config id, File def) {
+		String path = config.getString(id);
+		if (path != null && path.isEmpty()) {
+			path = def.getPath();
+		}
+
+		return getFile(path);
 	}
 
 	/**
@@ -486,8 +521,13 @@ public class Instance {
 	 * 
 	 * @return the path
 	 */
-	private static File getFile(UiConfig id) {
-		return getFile(uiconfig.getString(id));
+	private static File getFile(UiConfig id, File def) {
+		String path = uiconfig.getString(id);
+		if (path != null && path.isEmpty()) {
+			path = def.getPath();
+		}
+
+		return getFile(path);
 	}
 
 	/**
@@ -510,16 +550,28 @@ public class Instance {
 	}
 
 	/**
-	 * Return the home directory from the system properties.
+	 * Return the home directory from the environment (FANFIX_DIR) or the system
+	 * properties.
+	 * <p>
+	 * The environment variable is tested first. Then, the custom property
+	 * "fanfix.home" is tried, followed by the usual "user.home" then
+	 * "java.io.tmp" if nothing else is found.
 	 * 
 	 * @return the home
 	 */
 	private static String getHome() {
-		String home = System.getProperty("fanfix.home");
+		String home = System.getenv("FANFIX_DIR");
 		if (home != null && new File(home).isFile()) {
 			home = null;
 		}
 
+		if (home == null || home.trim().isEmpty()) {
+			home = System.getProperty("fanfix.home");
+			if (home != null && new File(home).isFile()) {
+				home = null;
+			}
+		}
+
 		if (home == null || home.trim().isEmpty()) {
 			home = System.getProperty("user.home");
 			if (!new File(home).isDirectory()) {