X-Git-Url: http://git.nikiroo.be/?a=blobdiff_plain;f=src%2Fbe%2Fnikiroo%2Ffanfix%2Flibrary%2FBasicLibrary.java;h=b92e5b078b01047e4e40375ec401f47b4f6bfe73;hb=1f2a7d5fe14f98b5dd762207eeaff90494fe102e;hp=099859dcfaffed075f30b3d716b790f644a43a09;hpb=9892fdaafb368227aa9630ab3abe6ed10bb1d001;p=nikiroo-utils.git

diff --git a/src/be/nikiroo/fanfix/library/BasicLibrary.java b/src/be/nikiroo/fanfix/library/BasicLibrary.java
index 099859d..b92e5b0 100644
--- a/src/be/nikiroo/fanfix/library/BasicLibrary.java
+++ b/src/be/nikiroo/fanfix/library/BasicLibrary.java
@@ -122,6 +122,13 @@ abstract public class BasicLibrary {
 	 */
 	public abstract Image getCover(String luid) throws IOException;
 
+	// TODO: ensure it is the main used interface
+	public synchronized MetaResultList getList(Progress pg) throws IOException {
+		return new MetaResultList(getMetas(pg));
+	}
+	
+	//TODO: make something for (normal and custom) not-story covers
+	
 	/**
 	 * Return the cover image associated to this source.
 	 * <p>
@@ -245,6 +252,8 @@ abstract public class BasicLibrary {
 	/**
 	 * Return the list of stories (represented by their {@link MetaData}, which
 	 * <b>MAY</b> not have the cover included).
+	 * <p>
+	 * The returned list <b>MUST</b> be a copy, not the original one.
 	 * 
 	 * @param pg
 	 *            the optional {@link Progress}
@@ -563,31 +572,29 @@ abstract public class BasicLibrary {
 	}
 
 	/**
-	 * List all the stories of the given source type in the {@link BasicLibrary}
-	 * , or all the stories if NULL is passed as a type.
+	 * List all the stories of the given source type in the {@link BasicLibrary} ,
+	 * or all the stories if NULL is passed as a type.
 	 * <p>
 	 * Cover images not included.
 	 * 
-	 * @param type
-	 *            the type of story to retrieve, or NULL for all
+	 * @param source the type of story to retrieve, or NULL for all
 	 * 
 	 * @return the stories
 	 * 
-	 * @throws IOException
-	 *             in case of IOException
+	 * @throws IOException in case of IOException
+	 * 
+	 * @deprecated use {@link BasicLibrary#getList(Progress)} and
+	 *             {@link MetaResultList#filter(List, List, List)}
 	 */
-	public synchronized List<MetaData> getListBySource(String type)
-			throws IOException {
-		List<MetaData> list = new ArrayList<MetaData>();
-		for (MetaData meta : getMetas(null)) {
-			String storyType = meta.getSource();
-			if (type == null || type.equalsIgnoreCase(storyType)) {
-				list.add(meta);
-			}
+	@Deprecated
+	public synchronized List<MetaData> getListBySource(String source) throws IOException {
+		List<String> sources = null;
+		if (source != null) {
+			sources = new ArrayList<String>();
+			sources.add(source);
 		}
 
-		Collections.sort(list);
-		return list;
+		return getList(null).filter(sources, null, null);
 	}
 
 	/**
@@ -603,19 +610,18 @@ abstract public class BasicLibrary {
 	 * 
 	 * @throws IOException
 	 *             in case of IOException
+	 *             
+	 *             @deprecated use {@link BasicLibrary#getList(Progress)} and
+	 *             {@link MetaResultList#filter(List, List, List)}
 	 */
-	public synchronized List<MetaData> getListByAuthor(String author)
-			throws IOException {
-		List<MetaData> list = new ArrayList<MetaData>();
-		for (MetaData meta : getMetas(null)) {
-			String storyAuthor = meta.getAuthor();
-			if (author == null || author.equalsIgnoreCase(storyAuthor)) {
-				list.add(meta);
-			}
+	public synchronized List<MetaData> getListByAuthor(String author) throws IOException {
+		List<String> authors = null;
+		if (author != null) {
+			authors = new ArrayList<String>();
+			authors.add(author);
 		}
 
-		Collections.sort(list);
-		return list;
+		return getList(null).filter(null, authors, null);
 	}
 
 	/**
@@ -730,10 +736,8 @@ abstract public class BasicLibrary {
 		} catch (IOException e) {
 			// We should not have not-supported files in the
 			// library
-			Instance.getTraceHandler().error(
-					new IOException(String.format(
-							"Cannot load file of type '%s' from library: %s",
-							meta.getType(), file), e));
+			Instance.getInstance().getTraceHandler().error(new IOException(
+					String.format("Cannot load file of type '%s' from library: %s", meta.getType(), file), e));
 		} finally {
 			pgProcess.done();
 			pg.done();
@@ -892,8 +896,7 @@ abstract public class BasicLibrary {
 	public synchronized Story save(Story story, String luid, Progress pg)
 			throws IOException {
 
-		Instance.getTraceHandler().trace(
-				this.getClass().getSimpleName() + ": saving story " + luid);
+		Instance.getInstance().getTraceHandler().trace(this.getClass().getSimpleName() + ": saving story " + luid);
 
 		// Do not change the original metadata, but change the original story
 		MetaData meta = story.getMeta().clone();
@@ -913,9 +916,8 @@ abstract public class BasicLibrary {
 
 		updateInfo(story.getMeta());
 
-		Instance.getTraceHandler().trace(
-				this.getClass().getSimpleName() + ": story saved (" + luid
-						+ ")");
+		Instance.getInstance().getTraceHandler()
+				.trace(this.getClass().getSimpleName() + ": story saved (" + luid + ")");
 
 		return story;
 	}
@@ -930,14 +932,13 @@ abstract public class BasicLibrary {
 	 *             in case of I/O error
 	 */
 	public synchronized void delete(String luid) throws IOException {
-		Instance.getTraceHandler().trace(
-				this.getClass().getSimpleName() + ": deleting story " + luid);
+		Instance.getInstance().getTraceHandler().trace(this.getClass().getSimpleName() + ": deleting story " + luid);
 
 		doDelete(luid);
 		invalidateInfo(luid);
 
-		Instance.getTraceHandler().trace(
-				this.getClass().getSimpleName() + ": story deleted (" + luid
+		Instance.getInstance().getTraceHandler()
+				.trace(this.getClass().getSimpleName() + ": story deleted (" + luid
 						+ ")");
 	}