Remove image from previous album, if set.

[Sone.git] / src / main / java / net / pterodactylus / sone / data / Album.java

diff --git a/src/main/java/net/pterodactylus/sone/data/Album.java b/src/main/java/net/pterodactylus/sone/data/Album.java
index 16c9916..20862a4 100644 (file)
--- a/src/main/java/net/pterodactylus/sone/data/Album.java
+++ b/src/main/java/net/pterodactylus/sone/data/Album.java
@@ -45,8 +45,8 @@ public class Album implements Fingerprintable {
        /** The parent album. */
        private Album parent;
 
-       /** The name of this album. */
-       private String name;
+       /** The title of this album. */
+       private String title;
 
        /** The description of this album. */
        private String description;
@@ -103,7 +103,7 @@ public class Album implements Fingerprintable {
         * @return This album
         */
        public Album setSone(Sone sone) {
-               Validation.begin().isNull("Current Album Owner", this.sone).isNotNull("New Album Owner", sone).check();
+               Validation.begin().isNotNull("New Album Owner", sone).isEither("Old Album Owner", this.sone, null, sone).check();
                this.sone = sone;
                return this;
        }
@@ -124,9 +124,11 @@ public class Album implements Fingerprintable {
         *            The album to add
         */
        public void addAlbum(Album album) {
-               Validation.begin().isNotNull("Album", album).check().isEqual("Album Owner", album.sone, sone).isNull("Album Parent", album.parent).check();
-               albums.add(album);
+               Validation.begin().isNotNull("Album", album).check().isEqual("Album Owner", album.sone, sone).isEither("Old Album Parent", this.parent, null, album.parent).check();
                album.setParent(this);
+               if (!albums.contains(album)) {
+                       albums.add(album);
+               }
        }
 
        /**
@@ -157,8 +159,14 @@ public class Album implements Fingerprintable {
         *            The image to add
         */
        public void addImage(Image image) {
-               Validation.begin().isNotNull("Image", image).check().isEqual("Image Owner", image.getSone(), sone).check();
-               images.add(image);
+               Validation.begin().isNotNull("Image", image).check().isNotNull("Image Owner", image.getSone()).check().isEqual("Image Owner", image.getSone(), sone).check();
+               if (image.getAlbum() != null) {
+                       image.getAlbum().removeImage(image);
+               }
+               image.setAlbum(this);
+               if (!images.contains(image)) {
+                       images.add(image);
+               }
        }
 
        /**
@@ -186,6 +194,15 @@ public class Album implements Fingerprintable {
        }
 
        /**
+        * Returns whether this album contains any other albums or images.
+        *
+        * @return {@code true} if this album is empty, {@code false} otherwise
+        */
+       public boolean isEmpty() {
+               return albums.isEmpty() && images.isEmpty();
+       }
+
+       /**
         * Returns the parent album of this album.
         *
         * @return The parent album of this album, or {@code null} if this album
@@ -214,30 +231,29 @@ public class Album implements Fingerprintable {
         * @return This album
         */
        protected Album removeParent() {
-               Validation.begin().isNotNull("Album Parent", parent).check();
                this.parent = null;
                return this;
        }
 
        /**
-        * Returns the name of this album.
+        * Returns the title of this album.
         *
-        * @return The name of this album
+        * @return The title of this album
         */
-       public String getName() {
-               return name;
+       public String getTitle() {
+               return title;
        }
 
        /**
-        * Sets the name of this album.
+        * Sets the title of this album.
         *
-        * @param name
-        *            The name of this album
+        * @param title
+        *            The title of this album
         * @return This album
         */
-       public Album setName(String name) {
-               Validation.begin().isNotNull("Album Name", name).check();
-               this.name = name;
+       public Album setTitle(String title) {
+               Validation.begin().isNotNull("Album Title", title).check();
+               this.title = title;
                return this;
        }
 
@@ -275,7 +291,7 @@ public class Album implements Fingerprintable {
                StringBuilder fingerprint = new StringBuilder();
                fingerprint.append("Album(");
                fingerprint.append("ID(").append(id).append(')');
-               fingerprint.append("Name(").append(name).append(')');
+               fingerprint.append("Title(").append(title).append(')');
                fingerprint.append("Description(").append(description).append(')');
 
                /* add nested albums. */
@@ -288,7 +304,9 @@ public class Album implements Fingerprintable {
                /* add images. */
                fingerprint.append("Images(");
                for (Image image : images) {
-                       fingerprint.append(image.getFingerprint());
+                       if (image.isInserted()) {
+                               fingerprint.append(image.getFingerprint());
+                       }
                }
                fingerprint.append(')');
 
@@ -304,6 +322,14 @@ public class Album implements Fingerprintable {
         * {@inheritDoc}
         */
        @Override
+       public int hashCode() {
+               return id.hashCode();
+       }
+
+       /**
+        * {@inheritDoc}
+        */
+       @Override
        public boolean equals(Object object) {
                if (!(object instanceof Album)) {
                        return false;