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

diff --git a/src/be/nikiroo/fanfix/library/BasicLibrary.java b/src/be/nikiroo/fanfix/library/BasicLibrary.java
index c558384..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);
 	}
 
 	/**