From 9d70913276805a0d7e0508a2aac9f2684d4d0d69 Mon Sep 17 00:00:00 2001
From: =?utf8?q?David=20=E2=80=98Bombe=E2=80=99=20Roden?=
 <bombe@pterodactylus.net>
Date: Tue, 31 Jul 2012 07:09:04 +0200
Subject: [PATCH] Add functions to manage groups of an artist.

---
 .../demoscenemusic/page/admin/EditArtistPage.java  | 29 ++++++++++++++---
 src/main/resources/templates/admin.edit-artist     | 38 ++++++++++++++++++++++
 2 files changed, 63 insertions(+), 4 deletions(-)

diff --git a/src/main/java/net/pterodactylus/demoscenemusic/page/admin/EditArtistPage.java b/src/main/java/net/pterodactylus/demoscenemusic/page/admin/EditArtistPage.java
index 8c141fa..e737e60 100644
--- a/src/main/java/net/pterodactylus/demoscenemusic/page/admin/EditArtistPage.java
+++ b/src/main/java/net/pterodactylus/demoscenemusic/page/admin/EditArtistPage.java
@@ -17,8 +17,11 @@
 
 package net.pterodactylus.demoscenemusic.page.admin;
 
+import java.util.Collection;
+
 import net.pterodactylus.demoscenemusic.core.Core;
 import net.pterodactylus.demoscenemusic.data.Artist;
+import net.pterodactylus.demoscenemusic.data.Group;
 import net.pterodactylus.demoscenemusic.page.ServletRequest;
 import net.pterodactylus.util.database.DatabaseException;
 import net.pterodactylus.util.template.Template;
@@ -59,19 +62,37 @@ public class EditArtistPage extends AdminBasePage {
 	protected void processTemplate(TemplateContext templateContext, ServletRequest request) throws RedirectException {
 		super.processTemplate(templateContext, request);
 		String id = request.getServletRequest().getParameter("id");
-		/* TODO - check for null. */
-		Artist artist;
 		try {
-			artist = getCore().getDataManager().getArtistById(id);
+			Artist artist = getCore().getDataManager().getArtistById(id);
 			/* TODO - check for null. */
 			templateContext.set("artist", artist);
 			if (request.getMethod() == Method.POST) {
 				String name = request.getServletRequest().getParameter("name");
-				artist.setName(name);
+				if ((name != null) && (name.trim().length() > 0)) {
+					artist.setName(name);
+				}
 
 				/* process property changes. */
 				processProperties(request, artist);
 
+				/* check for new group. */
+				if ("true".equals(request.getServletRequest().getParameter("add-group"))) {
+					String groupId = request.getServletRequest().getParameter("group");
+					Group group = getCore().getDataManager().getGroupById(groupId);
+					Collection<Group> groups = artist.getGroups();
+					groups.add(group);
+					artist.setGroups(groups);
+				}
+
+				/* check for deleted groups. */
+				if ("true".equals(request.getServletRequest().getParameter("delete-group"))) {
+					String groupId = request.getServletRequest().getParameter("group");
+					Group group = getCore().getDataManager().getGroupById(groupId);
+					Collection<Group> groups = artist.getGroups();
+					groups.remove(group);
+					artist.setGroups(groups);
+				}
+
 				getCore().getDataManager().saveArtist(artist);
 				throw new RedirectException("admin.edit-artist?id=" + artist.getId());
 			}
diff --git a/src/main/resources/templates/admin.edit-artist b/src/main/resources/templates/admin.edit-artist
index c63d238..77d2d35 100644
--- a/src/main/resources/templates/admin.edit-artist
+++ b/src/main/resources/templates/admin.edit-artist
@@ -26,6 +26,44 @@
 
 </form>
 
+<h2>Groups</h2>
+
+<%foreach artist.groups group>
+
+	<form action="admin.edit-artist" method="post" accept-charset="utf-8">
+
+		<input type="hidden" name="id" value="<%artist.id|html>"/>
+		<input type="hidden" name="group" value="<%group.id|html>"/>
+
+		<div>
+			<%group.name|html> <a href="admin.edit-group?id=<%group.id|html>">edit</a>
+		</div>
+		<button type="submit" name="delete-group" value="true">Delete Group</button>
+
+	</form>
+
+<%foreachelse>
+	<p>This artists does not belong to any groups.</p>
+<%/foreach>
+
+<h3>Add a Group</h3>
+
+<form action="admin.edit-artist" method="post" accept-charset="utf-8">
+
+	<input type="hidden" name="id" value="<%artist.id|html>"/>
+
+	<div>
+		Add Group:
+		<select name="group">
+			<%foreach dataManager.allGroups group>
+				<option value="<%group.id|html>"><%group.name|html></option>
+			<%/foreach>
+		</select>
+	</div>
+	<button type="submit" name="add-group" value="true">Add Group</button>
+
+</form>
+
 <h2>Tracks</h2>
 
 <%foreach artist.tracks track>
-- 
2.7.4