From eb96eab513101783a750c13419de6bdce3d2fa4c Mon Sep 17 00:00:00 2001
From: James Moger <james.moger@gitblit.com>
Date: Tue, 20 Dec 2011 19:36:56 -0500
Subject: [PATCH] Renamed GitblitUserService, sendMail method, sendmail.groovy, and RepositoryModel.mailingLists

---
 src/com/gitblit/wicket/pages/EditRepositoryPage.java |   30 ++++++++++++++++++++----------
 1 files changed, 20 insertions(+), 10 deletions(-)

diff --git a/src/com/gitblit/wicket/pages/EditRepositoryPage.java b/src/com/gitblit/wicket/pages/EditRepositoryPage.java
index 492addc..f595053 100644
--- a/src/com/gitblit/wicket/pages/EditRepositoryPage.java
+++ b/src/com/gitblit/wicket/pages/EditRepositoryPage.java
@@ -19,9 +19,11 @@
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.Collections;
+import java.util.HashSet;
 import java.util.Iterator;
 import java.util.List;
 import java.util.Map;
+import java.util.Set;
 
 import org.apache.wicket.PageParameters;
 import org.apache.wicket.extensions.markup.html.form.palette.Palette;
@@ -55,7 +57,7 @@
 
 	private boolean isAdmin;
 
-	private IModel<String> mailRecipients;
+	private IModel<String> mailingLists;
 
 	public EditRepositoryPage() {
 		// create constructor
@@ -118,7 +120,8 @@
 		}
 		final Palette<String> preReceivePalette = new Palette<String>("preReceiveScripts",
 				new ListModel<String>(preReceiveScripts), new CollectionModel<String>(GitBlit
-						.self().getAvailableScripts()), new ChoiceRenderer<String>("", ""), 12, true);
+						.self().getAvailableScripts()), new ChoiceRenderer<String>("", ""), 12,
+				true);
 
 		// post-receive palette
 		if (repositoryModel.postReceiveScripts != null) {
@@ -126,7 +129,8 @@
 		}
 		final Palette<String> postReceivePalette = new Palette<String>("postReceiveScripts",
 				new ListModel<String>(postReceiveScripts), new CollectionModel<String>(GitBlit
-						.self().getAvailableScripts()), new ChoiceRenderer<String>("", ""), 12, true);
+						.self().getAvailableScripts()), new ChoiceRenderer<String>("", ""), 12,
+				true);
 
 		CompoundPropertyModel<RepositoryModel> model = new CompoundPropertyModel<RepositoryModel>(
 				repositoryModel);
@@ -191,11 +195,17 @@
 						}
 					}
 
-					// set mail recipients
-					String ml = mailRecipients.getObject();
+					// set mailing lists
+					String ml = mailingLists.getObject();
 					if (!StringUtils.isEmpty(ml)) {
-						List<String> list = StringUtils.getStringsFromValue(ml.trim(), " ");
-						repositoryModel.mailRecipients = list;
+						Set<String> list = new HashSet<String>();
+						for (String address : ml.split("(,|\\s)")) {
+							if (StringUtils.isEmpty(address)) {
+								continue;
+							}
+							list.add(address.toLowerCase());
+						}
+						repositoryModel.mailingLists = new ArrayList<String>(list);
 					}
 
 					// pre-receive scripts
@@ -275,9 +285,9 @@
 		form.add(new CheckBox("showReadme"));
 		form.add(new CheckBox("skipSizeCalculation"));
 		form.add(new CheckBox("skipSummaryMetrics"));
-		mailRecipients = new Model<String>(repositoryModel.mailRecipients == null ? ""
-				: StringUtils.flattenStrings(repositoryModel.mailRecipients, " "));
-		form.add(new TextField<String>("mailRecipients", mailRecipients));
+		mailingLists = new Model<String>(repositoryModel.mailingLists == null ? ""
+				: StringUtils.flattenStrings(repositoryModel.mailingLists, " "));
+		form.add(new TextField<String>("mailingLists", mailingLists));
 		form.add(usersPalette);
 		form.add(teamsPalette);
 		form.add(federationSetsPalette);

--
Gitblit v1.9.1