Improve remote, fix bugs, update nikiroo-utils
[nikiroo-utils.git] / src / be / nikiroo / fanfix / library / RemoteLibrary.java
index 63082ce94fd0a954790865d4a4ddc19fbf6c5ebb..29d2bcda594f353afc882436438d071686db2dc0 100644 (file)
@@ -23,10 +23,6 @@ import be.nikiroo.utils.serial.ConnectActionClient;
 public class RemoteLibrary extends BasicLibrary {
        private String host;
        private int port;
-       private File baseDir;
-
-       private LocalLibrary lib;
-       private List<MetaData> metas;
 
        /**
         * Create a {@link RemoteLibrary} linked to the given server.
@@ -39,11 +35,6 @@ public class RemoteLibrary extends BasicLibrary {
        public RemoteLibrary(String host, int port) {
                this.host = host;
                this.port = port;
-
-               this.baseDir = Instance.getRemoteDir(host);
-               this.baseDir.mkdirs();
-
-               this.lib = new LocalLibrary(baseDir);
        }
 
        @Override
@@ -54,34 +45,13 @@ public class RemoteLibrary extends BasicLibrary {
        @Override
        protected List<MetaData> getMetas(Progress pg) {
                // TODO: progress
+               final List<MetaData> metas = new ArrayList<MetaData>();
+               MetaData[] fromNetwork = this.<MetaData[]> getRemoteObject( //
+                               new Object[] { "GET_METADATA", "*" });
 
-               if (metas == null) {
-                       metas = new ArrayList<MetaData>();
-
-                       try {
-                               new ConnectActionClient(host, port, true) {
-                                       @Override
-                                       public void action(Version serverVersion) throws Exception {
-                                               try {
-                                                       Object rep = send("GET_METADATA *");
-                                                       for (MetaData meta : (MetaData[]) rep) {
-                                                               metas.add(meta);
-                                                       }
-                                               } catch (Exception e) {
-                                                       Instance.syserr(e);
-                                               }
-                                       }
-                               }.connect();
-                       } catch (IOException e) {
-                               Instance.syserr(e);
-                       }
-
-                       List<String> test = new ArrayList<String>();
-                       for (MetaData meta : metas) {
-                               if (test.contains(meta.getLuid())) {
-                                       throw new RuntimeException("wwops");
-                               }
-                               test.add(meta.getLuid());
+               if (fromNetwork != null) {
+                       for (MetaData meta : fromNetwork) {
+                               metas.add(meta);
                        }
                }
 
@@ -89,95 +59,56 @@ public class RemoteLibrary extends BasicLibrary {
        }
 
        @Override
-       public synchronized File getFile(final String luid) {
-               File file = lib.getFile(luid);
-               if (file == null) {
-                       final File[] tmp = new File[1];
-                       try {
-                               new ConnectActionClient(host, port, true) {
-                                       @Override
-                                       public void action(Version serverVersion) throws Exception {
-                                               try {
-                                                       Object rep = send("GET_STORY " + luid);
-                                                       Story story = (Story) rep;
-                                                       if (story != null) {
-                                                               lib.save(story, luid, null);
-                                                               tmp[0] = lib.getFile(luid);
-                                                       }
-                                               } catch (Exception e) {
-                                                       Instance.syserr(e);
-                                               }
-                                       }
-                               }.connect();
-                       } catch (IOException e) {
-                               Instance.syserr(e);
-                       }
-
-                       file = tmp[0];
-               }
-
-               if (file != null) {
-                       MetaData meta = getInfo(luid);
-                       metas.add(meta);
-               }
-
-               return file;
+       public BufferedImage getCover(final String luid) {
+               return this.<BufferedImage> getRemoteObject( //
+                               new Object[] { "GET_COVER", luid });
        }
 
        @Override
-       public BufferedImage getCover(final String luid) {
-               // Retrieve it from the cache if possible:
-               if (lib.getInfo(luid) != null) {
-                       return lib.getCover(luid);
-               }
-
-               final BufferedImage[] result = new BufferedImage[1];
-               try {
-                       new ConnectActionClient(host, port, true) {
-                               @Override
-                               public void action(Version serverVersion) throws Exception {
-                                       try {
-                                               Object rep = send("GET_COVER " + luid);
-                                               result[0] = (BufferedImage) rep;
-                                       } catch (Exception e) {
-                                               Instance.syserr(e);
-                                       }
-                               }
-                       }.connect();
-               } catch (IOException e) {
-                       Instance.syserr(e);
-               }
+       public BufferedImage getSourceCover(final String source) {
+               return this.<BufferedImage> getRemoteObject( //
+                               new Object[] { "GET_SOURCE_COVER", source });
+       }
 
-               return result[0];
+       @Override
+       public synchronized Story getStory(final String luid, Progress pg) {
+               return this.<Story> getRemoteObject(new Object[] { "GET_STORY", luid });
        }
 
        @Override
        protected void clearCache() {
-               metas = null;
-               lib.clearCache();
        }
 
        @Override
        public synchronized Story save(Story story, String luid, Progress pg)
                        throws IOException {
-               throw new java.lang.InternalError(
-                               "No write support allowed on remote Libraries");
+               getRemoteObject(new Object[] { "SAVE_STORY", story, luid });
+
+               // because the meta changed:
+               clearCache();
+               story.setMeta(getInfo(luid));
+
+               return story;
        }
 
        @Override
        public synchronized void delete(String luid) throws IOException {
-               throw new java.lang.InternalError(
-                               "No write support allowed on remote Libraries");
+               getRemoteObject(new Object[] { "DELETE_STORY", luid });
        }
 
        @Override
        public void setSourceCover(String source, String luid) {
+               this.<BufferedImage> getRemoteObject( //
+               new Object[] { "SET_SOURCE_COVER", source, luid });
+       }
+
+       @Override
+       public synchronized File getFile(final String luid, Progress pg) {
                throw new java.lang.InternalError(
-                               "No write support allowed on remote Libraries");
+                               "Operation not supportorted on remote Libraries");
        }
 
-       // All the following methods are only used by Save and Delete in
-       // BasicLibrary:
+       // The following methods are only used by Save and Delete in BasicLibrary:
 
        @Override
        protected int getNextId() {
@@ -195,11 +126,39 @@ public class RemoteLibrary extends BasicLibrary {
        }
 
        /**
-        * Return the backing local library.
+        * Return an object from the server.
+        * 
+        * @param <T>
+        *            the expected type of object
+        * @param command
+        *            the command to send
         * 
-        * @return the library
+        * @return the object or NULL
         */
-       LocalLibrary getLocalLibrary() {
-               return lib;
+       @SuppressWarnings("unchecked")
+       private <T> T getRemoteObject(final Object[] command) {
+               final Object[] result = new Object[1];
+               try {
+                       new ConnectActionClient(host, port, true) {
+                               @Override
+                               public void action(Version serverVersion) throws Exception {
+                                       try {
+                                               Object rep = send(command);
+                                               result[0] = rep;
+                                       } catch (Exception e) {
+                                               Instance.syserr(e);
+                                       }
+                               }
+                       }.connect();
+               } catch (IOException e) {
+                       Instance.syserr(e);
+               }
+
+               try {
+                       return (T) result[0];
+               } catch (Exception e) {
+                       Instance.syserr(e);
+                       return null;
+               }
        }
 }