Works for me. Can please someone, who has no administrator rights (this might distort my results), please try that as well?

I get the exact same result, nothing happens...

This seems to be a problem with the Javascript support in Internet Explorer. I'll have a look at it later. Please use Mozilla if you need to urgently update maintainers.

Does this problem still persist?

Yes. Removing someone works. Adding someone does not. (Tested with IE 6.0, XP SP2RC2)

JS in package-maintainers.js is a little bit confusing.
Can I suggest rename form from "form" to "editmaintainers" and also add name attribute for search form in site header?

Another issue can be (unlikely, but still) in shortened prototype for JS Option object.

+++ package-maintainers.js	2005-03-10 
- item = new Option(name + " (" + handle + ", " + role + ")", value);
+ item = new Option(name + " (" + handle + ", " + role + ")", value, false, true);

In either case - can you supply string from your address bar when the page with "Done" link is displayed?

Here are two patches, which can solve the problem
Patch for pearweb\public_html\admin\package-maintainers.php

--- package-maintainers_old.php	2005-02-19 02:34:52.000000000 +0200
+++ package-maintainers.php	2005-03-13 15:14:06.000000000 +0200
@@ -110,13 +110,13 @@
     echo '  </td>';
 
     echo '  <td>';
-    echo '  <input type="submit" onClick="addMaintainer(); return false" name="add" value="Add as" />';
+    echo '  <input type="button" onClick="addMaintainer();" name="add" value="Add as" />';
     echo '  <select name="role" size="1">';
     echo '    <option value="lead">lead</option>';
     echo '    <option value="developer">developer</option>';
     echo '    <option value="helper">helper</option>';
     echo '  </select><br /><br />';
-    echo '  <input type="submit" onClick="removeMaintainer(); return false" name="remove" value="Remove" />';
+    echo '  <input type="button" onClick="removeMaintainer();" name="remove" value="Remove" />';
     echo '  </td>';
 
     echo '  <td>';

Here is the second part.
For pearweb\public_html\javascript\package-maintainers.js

--- package-maintainers_old.js	2005-02-19 02:34:58.000000000 +0200
+++ package-maintainers.js	2005-03-13 15:36:00.000000000 +0200
@@ -36,7 +36,7 @@
             role = getRole();
             handle = document.form.accounts.options[i].value;
             value = handle + "||" + role;
-            item = new Option(name + " (" + handle + ", " + role + ")", value);
+            item = new Option(name + " (" + handle + ", " + role + ")", value, false, true);
             document.form['maintainers[]'].options[document.form['maintainers[]'].length] = item;
         }
     }
@@ -61,6 +61,7 @@
     for (i = 0; i < document.form['maintainers[]'].length; i++) {
         field = document.form['maintainers[]'].options[i].selected = true;
     }
+    return true;
 }
 
 function activateAdd() {

Martin, have you received the patches I mailed you?

Please provide a link to the patch.

Doing so will help for other people to test your patch and/or apply if everything works fine.

Thanks for it btw :)

--Pierre

Here you are.

public_html/javascript/package-maintainers.js
http://www.megaupload.com/?d=045XQJH3

public_html/admin/package-maintainers.php
http://www.megaupload.com/?d=04Z0SSHM

This bug has been fixed in CVS.

In case this was a documentation problem, the fix will show up at the
end of next Sunday (CET) on pear.php.net.

In case this was a pear.php.net website problem, the change will show
up on the website in short time.
 
Thank you for the report, and for helping us make PEAR better.