From: David ‘Bombe’ Roden <bombe@pterodactylus.net>
Date: Fri, 23 Sep 2011 14:54:13 +0000 (+0200)
Subject: Return the image the given image was swapped with.
X-Git-Tag: 0.7^2~2^2~21
X-Git-Url: https://git.pterodactylus.net/?a=commitdiff_plain;h=7ea94de6b3101ad2825b45c97d1d4893d65843f2;p=Sone.git

Return the image the given image was swapped with.
---

diff --git a/src/main/java/net/pterodactylus/sone/data/Album.java b/src/main/java/net/pterodactylus/sone/data/Album.java
index 56bddb2..cc32f88 100644
--- a/src/main/java/net/pterodactylus/sone/data/Album.java
+++ b/src/main/java/net/pterodactylus/sone/data/Album.java
@@ -214,15 +214,18 @@ public class Album implements Fingerprintable {
 	 *
 	 * @param image
 	 *            The image to move up
+	 * @return The image that the given image swapped the place with, or
+	 *         <code>null</code> if the image did not change its place
 	 */
-	public void moveImageUp(Image image) {
+	public Image moveImageUp(Image image) {
 		Validation.begin().isNotNull("Image", image).check().isEqual("Image Album", image.getAlbum(), this).isEqual("Album Owner", image.getAlbum().getSone(), sone).check();
 		int oldIndex = imageIds.indexOf(image.getId());
 		if (oldIndex <= 0) {
-			return;
+			return null;
 		}
 		imageIds.remove(image.getId());
 		imageIds.add(oldIndex - 1, image.getId());
+		return images.get(imageIds.get(oldIndex));
 	}
 
 	/**
@@ -231,15 +234,18 @@ public class Album implements Fingerprintable {
 	 *
 	 * @param image
 	 *            The image to move down
+	 * @return The image that the given image swapped the place with, or
+	 *         <code>null</code> if the image did not change its place
 	 */
-	public void moveImageDown(Image image) {
+	public Image moveImageDown(Image image) {
 		Validation.begin().isNotNull("Image", image).check().isEqual("Image Album", image.getAlbum(), this).isEqual("Album Owner", image.getAlbum().getSone(), sone).check();
 		int oldIndex = imageIds.indexOf(image.getId());
 		if ((oldIndex == -1) || (oldIndex >= (imageIds.size() - 1))) {
-			return;
+			return null;
 		}
 		imageIds.remove(image.getId());
 		imageIds.add(oldIndex + 1, image.getId());
+		return images.get(imageIds.get(oldIndex));
 	}
 
 	/**