Drop-down combobox to select a repository browser all had the same title.

See http://www.nabble.com/Possible-bug--Showing-%22Associated-Mantis-Website%22-in-scm-repository-browser-tt22786295.html

git-svn-id: https://hudson.dev.java.net/svn/hudson/trunk/hudson/main@16754 71c3de6d-444a-0410-be80-ed276b4c234a

core/src/main/resources/lib/hudson/listScmBrowsers.jelly

@@ -32,7 +32,7 @@ THE SOFTWARE.
     <f:dropdownListBlock value="auto" title="(${%Auto})" />
     <j:set var="currentBrowser" value="${scm.browser}"/>
     <j:forEach var="descriptor" items="${scmd.browserDescriptors}" varStatus="loop">
-      <f:dropdownListBlock value="${loop.index}" title="${d.displayName}" selected="${currentBrowser.descriptor==descriptor}">
+      <f:dropdownListBlock value="${loop.index}" title="${descriptor.displayName}" selected="${currentBrowser.descriptor==descriptor}">
         <j:set var="browser" value="${currentBrowser.descriptor==descriptor ? currentBrowser : null}"/><!-- for compatibility -->
         <j:set var="instance" value="${browser}" />
         <tr><td>

test/src/test/java/lib/hudson/ListScmBrowsersTest.java

+package lib.hudson;
+
+import com.gargoylesoftware.htmlunit.html.HtmlOption;
+import com.gargoylesoftware.htmlunit.html.HtmlPage;
+import com.gargoylesoftware.htmlunit.html.HtmlSelect;
+import hudson.model.Item;
+import org.jvnet.hudson.test.HudsonTestCase;
+import org.xml.sax.SAXException;
+
+import java.util.HashSet;
+import java.util.List;
+import java.util.Set;
+import java.io.IOException;
+
+/**
+ * @author Kohsuke Kawaguchi
+ */
+public class ListScmBrowsersTest extends HudsonTestCase {
+    public void testSelectBoxesUnique() throws Exception {
+        check(createFreeStyleProject());
+    }
+
+    public void testSelectBoxesUnique2() throws Exception {
+        check(createMavenProject());
+    }
+
+    public void testSelectBoxesUnique3() throws Exception {
+        check(createMatrixProject());
+    }
+
+    private void check(Item p) throws IOException, SAXException {
+        HtmlPage page = new WebClient().getPage(p, "configure");
+        List<HtmlSelect> selects = (List<HtmlSelect>)page.selectNodes("//select");
+        assertTrue(selects.size()>0);
+        for (HtmlSelect select : selects) {
+            Set<String> title = new HashSet<String>();
+            for(HtmlOption o : select.getOptions()) {
+                assertTrue("Duplicate entry: "+o.asText(),title.add(o.asText()));
+            }
+        }
+    }
+}