projects
/
Sone.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Ensure that an image is always set as album image, unless the album is empty.
[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
20862a4
..
89823a8
100644
(file)
--- a/
src/main/java/net/pterodactylus/sone/data/Album.java
+++ b/
src/main/java/net/pterodactylus/sone/data/Album.java
@@
-18,7
+18,9
@@
package net.pterodactylus.sone.data;
import java.util.ArrayList;
package net.pterodactylus.sone.data;
import java.util.ArrayList;
+import java.util.LinkedHashMap;
import java.util.List;
import java.util.List;
+import java.util.Map;
import java.util.UUID;
import net.pterodactylus.util.validation.Validation;
import java.util.UUID;
import net.pterodactylus.util.validation.Validation;
@@
-40,7
+42,7
@@
public class Album implements Fingerprintable {
private final List<Album> albums = new ArrayList<Album>();
/** The images in this album. */
private final List<Album> albums = new ArrayList<Album>();
/** The images in this album. */
- private final
List<Image> images = new ArrayList<
Image>();
+ private final
Map<String, Image> images = new LinkedHashMap<String,
Image>();
/** The parent album. */
private Album parent;
/** The parent album. */
private Album parent;
@@
-51,8
+53,8
@@
public class Album implements Fingerprintable {
/** The description of this album. */
private String description;
/** The description of this album. */
private String description;
- /** The
index
of the album picture. */
- private
int albumImage = -1
;
+ /** The
ID
of the album picture. */
+ private
String albumImage
;
/**
* Creates a new album with a random ID.
/**
* Creates a new album with a random ID.
@@
-149,7
+151,7
@@
public class Album implements Fingerprintable {
* @return The images in this album
*/
public List<Image> getImages() {
* @return The images in this album
*/
public List<Image> getImages() {
- return new ArrayList<Image>(images);
+ return new ArrayList<Image>(images
.values()
);
}
/**
}
/**
@@
-164,8
+166,11
@@
public class Album implements Fingerprintable {
image.getAlbum().removeImage(image);
}
image.setAlbum(this);
image.getAlbum().removeImage(image);
}
image.setAlbum(this);
- if (!images.contains(image)) {
- images.add(image);
+ if (images.isEmpty()) {
+ albumImage = image.getId();
+ }
+ if (!images.containsKey(image.getId())) {
+ images.put(image.getId(), image);
}
}
}
}
@@
-178,6
+183,13
@@
public class Album implements Fingerprintable {
public void removeImage(Image image) {
Validation.begin().isNotNull("Image", image).check().isEqual("Image Owner", image.getSone(), sone).check();
images.remove(image);
public void removeImage(Image image) {
Validation.begin().isNotNull("Image", image).check().isEqual("Image Owner", image.getSone(), sone).check();
images.remove(image);
+ if (image.getId().equals(albumImage)) {
+ if (images.isEmpty()) {
+ albumImage = null;
+ } else {
+ albumImage = images.values().iterator().next().getId();
+ }
+ }
}
/**
}
/**
@@
-187,13
+199,25
@@
public class Album implements Fingerprintable {
* @return The image to show when this album is listed
*/
public Image getAlbumImage() {
* @return The image to show when this album is listed
*/
public Image getAlbumImage() {
- if (albumImage ==
-1
) {
+ if (albumImage ==
null
) {
return null;
}
return images.get(albumImage);
}
/**
return null;
}
return images.get(albumImage);
}
/**
+ * Sets the ID of the album image.
+ *
+ * @param id
+ * The ID of the album image
+ * @return This album
+ */
+ public Album setAlbumImage(String id) {
+ this.albumImage = id;
+ return this;
+ }
+
+ /**
* Returns whether this album contains any other albums or images.
*
* @return {@code true} if this album is empty, {@code false} otherwise
* Returns whether this album contains any other albums or images.
*
* @return {@code true} if this album is empty, {@code false} otherwise
@@
-303,7
+327,7
@@
public class Album implements Fingerprintable {
/* add images. */
fingerprint.append("Images(");
/* add images. */
fingerprint.append("Images(");
- for (Image image : images) {
+ for (Image image : images
.values()
) {
if (image.isInserted()) {
fingerprint.append(image.getFingerprint());
}
if (image.isInserted()) {
fingerprint.append(image.getFingerprint());
}