Bug #103 [Com]: Edit Maintainers broken

Edit report at http://pear.php.net/bugs/bug.php?id=103&edit=1 ID: 103 Comment by: techtonik at php dot net Reported By: thesaur at php dot net Status: Assigned Type: Bug Package: Web Site Operating System: Irrelevant PHP Version: Irrelevant Assigned To: mj New Comment: 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() { Previous Comments: ------------------------------------------------------------------------ [2005-03-13 08:55:20] techtonik at php dot net 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>'; ------------------------------------------------------------------------ [2005-03-10 11:22:39] techtonik at php dot net 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? ------------------------------------------------------------------------ [2004-08-13 13:30:07] klaus at capitalfocus dot org Yes. Removing someone works. Adding someone does not. (Tested with IE 6.0, XP SP2RC2) ------------------------------------------------------------------------ [2004-08-13 11:59:58] mj at php dot net Does this problem still persist? ------------------------------------------------------------------------ [2003-10-22 06:26:15] mj at php dot net 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. ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at http://pear.php.net/bugs/bug.php?id=103 -- Edit this bug report at http://pear.php.net/bugs/bug.php?id=103&edit=1