Don’t use albums in abstract image builders.
authorDavid ‘Bombe’ Roden <bombe@pterodactylus.net>
Sat, 12 Oct 2013 13:12:45 +0000 (15:12 +0200)
committerDavid ‘Bombe’ Roden <bombe@pterodactylus.net>
Fri, 28 Feb 2014 21:17:43 +0000 (22:17 +0100)
src/main/java/net/pterodactylus/sone/data/impl/AbstractImageBuilder.java
src/main/java/net/pterodactylus/sone/data/impl/DefaultImageBuilder.java

index 0803084..1692bee 100644 (file)
  */
 package net.pterodactylus.sone.data.impl;
 
-import static com.google.common.base.Preconditions.checkNotNull;
 import static com.google.common.base.Preconditions.checkState;
 import static java.util.UUID.randomUUID;
 
-import net.pterodactylus.sone.data.Album;
 import net.pterodactylus.sone.data.Sone;
 import net.pterodactylus.sone.database.ImageBuilder;
 
@@ -32,8 +30,6 @@ import net.pterodactylus.sone.database.ImageBuilder;
  */
 public abstract class AbstractImageBuilder implements ImageBuilder {
 
-       protected final Album album;
-
        /** Whether to create an album with a random ID. */
        protected boolean randomId;
 
@@ -46,10 +42,6 @@ public abstract class AbstractImageBuilder implements ImageBuilder {
        protected int width;
        protected int height;
 
-       public AbstractImageBuilder(Album album) {
-               this.album = checkNotNull(album, "album must not be null");
-       }
-
        @Override
        public ImageBuilder randomId() {
                randomId = true;
index 7f7f53f..29cab0c 100644 (file)
@@ -28,8 +28,10 @@ import net.pterodactylus.sone.database.ImageBuilder;
  */
 public class DefaultImageBuilder extends AbstractImageBuilder {
 
+       protected final Album album;
+
        public DefaultImageBuilder(Album album) {
-               super(album);
+               this.album = album;
        }
 
        @Override