Remove AlbumBuilderFactory interface.
authorDavid ‘Bombe’ Roden <bombe@pterodactylus.net>
Tue, 15 Oct 2013 20:14:34 +0000 (22:14 +0200)
committerDavid ‘Bombe’ Roden <bombe@pterodactylus.net>
Fri, 28 Feb 2014 21:25:27 +0000 (22:25 +0100)
src/main/java/net/pterodactylus/sone/data/Album.java
src/main/java/net/pterodactylus/sone/data/Sone.java
src/main/java/net/pterodactylus/sone/database/AlbumBuilderFactory.java [deleted file]
src/main/java/net/pterodactylus/sone/web/CreateAlbumPage.java

index 7b245f9..a173e19 100644 (file)
@@ -26,7 +26,7 @@ import java.util.Comparator;
 import java.util.List;
 import javax.annotation.Nonnull;
 
-import net.pterodactylus.sone.database.AlbumBuilderFactory;
+import net.pterodactylus.sone.database.AlbumBuilder;
 import net.pterodactylus.sone.database.ImageBuilder;
 
 import com.google.common.base.Function;
@@ -40,7 +40,7 @@ import com.google.common.collect.ImmutableList;
  *
  * @author <a href="mailto:bombe@pterodactylus.net">David ‘Bombe’ Roden</a>
  */
-public interface Album extends Identified, Fingerprintable, AlbumBuilderFactory {
+public interface Album extends Identified, Fingerprintable {
 
        /** Compares two {@link Album}s by {@link #getTitle()}. */
        Comparator<Album> TITLE_COMPARATOR = new Comparator<Album>() {
@@ -167,6 +167,8 @@ public interface Album extends Identified, Fingerprintable, AlbumBuilderFactory
         */
        String getDescription();
 
+       AlbumBuilder newAlbumBuilder() throws IllegalStateException;
+
        ImageBuilder newImageBuilder() throws IllegalStateException;
 
        /**
index 41fdcc1..f1d9ffe 100644 (file)
@@ -28,7 +28,7 @@ import java.util.List;
 import java.util.Set;
 
 import net.pterodactylus.sone.core.Options;
-import net.pterodactylus.sone.database.AlbumBuilderFactory;
+import net.pterodactylus.sone.database.AlbumBuilder;
 import net.pterodactylus.sone.database.PostBuilder;
 import net.pterodactylus.sone.database.PostBuilderFactory;
 import net.pterodactylus.sone.freenet.wot.Identity;
@@ -46,7 +46,7 @@ import com.google.common.primitives.Ints;
  *
  * @author <a href="mailto:bombe@pterodactylus.net">David ‘Bombe’ Roden</a>
  */
-public interface Sone extends Identified, Fingerprintable, PostBuilderFactory, AlbumBuilderFactory, Comparable<Sone> {
+public interface Sone extends Identified, Fingerprintable, PostBuilderFactory, Comparable<Sone> {
 
        /**
         * Enumeration for the possible states of a {@link Sone}.
@@ -549,6 +549,8 @@ public interface Sone extends Identified, Fingerprintable, PostBuilderFactory, A
        /* TODO - remove this method again, maybe add an option provider */
        void setOptions(Options options);
 
+       AlbumBuilder newAlbumBuilder() throws IllegalStateException;
+
        PostBuilder newPostBuilder();
 
 }
diff --git a/src/main/java/net/pterodactylus/sone/database/AlbumBuilderFactory.java b/src/main/java/net/pterodactylus/sone/database/AlbumBuilderFactory.java
deleted file mode 100644 (file)
index e7a537d..0000000
+++ /dev/null
@@ -1,34 +0,0 @@
-/*
- * Sone - AlbumBuilderFactory.java - Copyright © 2013 David Roden
- *
- * This program is free software: you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program.  If not, see <http://www.gnu.org/licenses/>.
- */
-
-package net.pterodactylus.sone.database;
-
-/**
- * Factory for {@link AlbumBuilder}s.
- *
- * @author <a href="mailto:bombe@pterodactylus.net">David ‘Bombe’ Roden</a>
- */
-public interface AlbumBuilderFactory {
-
-       /**
-        * Creates a new album builder.
-        *
-        * @return A new album builder
-        */
-       AlbumBuilder newAlbumBuilder();
-
-}
index 30e1e60..97c0e3b 100644 (file)
@@ -21,7 +21,7 @@ import static net.pterodactylus.sone.text.TextFilter.filter;
 
 import net.pterodactylus.sone.data.Album;
 import net.pterodactylus.sone.data.Sone;
-import net.pterodactylus.sone.database.AlbumBuilderFactory;
+import net.pterodactylus.sone.database.AlbumBuilder;
 import net.pterodactylus.sone.web.page.FreenetRequest;
 import net.pterodactylus.util.template.Template;
 import net.pterodactylus.util.template.TemplateContext;
@@ -71,8 +71,8 @@ public class CreateAlbumPage extends SoneTemplatePage {
        private Album createAlbum(FreenetRequest request, String name) {
                Sone currentSone = getCurrentSone(request.getToadletContext());
                String parentId = request.getHttpRequest().getPartAsStringFailsafe("parent", 36);
-               AlbumBuilderFactory parent = parentId.equals("") ? currentSone : webInterface.getCore().getAlbum(parentId).get();
-               Album album = parent.newAlbumBuilder().build();
+               AlbumBuilder albumBuilder = parentId.equals("") ? currentSone.newAlbumBuilder() : webInterface.getCore().getAlbum(parentId).get().newAlbumBuilder();
+               Album album = albumBuilder.build();
                return setTitleAndDescription(request, name, album);
        }