Bug #103 [Fbk]: Edit Maintainers broken

Edit report at http://pear.php.net/bugs/bug.php?id=103&edit=1 ID: 103 Updated by: pierre@dotgeek.org Reported By: thesaur at php dot net Status: Feedback Type: Bug Package: Web Site Operating System: Irrelevant PHP Version: Irrelevant Assigned To: mj New Comment: 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 Previous Comments: ------------------------------------------------------------------------ [2005-05-25 03:48:10] techtonik at php dot net Martin, have you received the patches I mailed you? ------------------------------------------------------------------------ [2005-03-31 06:50:03] techtonik at php dot net 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() { ------------------------------------------------------------------------ [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) ------------------------------------------------------------------------ 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