Req #8176 [Asn->Csd]: do not mail in active developers

From: Date: Thu, 19 Oct 2006 02:21:32 +0000
Subject: Req #8176 [Asn->Csd]: do not mail in active developers
References: 1  Groups: php.pear.webmaster 
Request: Send a blank email to pear-webmaster+get-5327@lists.php.net to get a copy of this message
Edit report at http://pear.php.net/bugs/bug.php?id=8176&edit=1 ID: 8176 Updated by: cellog@php.net Reported By: smith at pooteeweet dot org -Status: Assigned +Status: Closed Id: 8176 Type: Feature/Change Request Package: Web Site Operating System: irrelevant PHP Version: Irrelevant Assigned To: davidc New Comment: This bug has been fixed in CVS. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better. Previous Comments: ------------------------------------------------------------------------ [2006-10-18 16:58:15] davidc at php dot net http://dev.agoraproduction.com/pear/pearweb/bugs/8176/ I have put the fix in cvs diff -u there. This is against the latest cvs up -dP I had. So explanation: File: include/Damblan/Trackback.php ----> This function has the function getMaintainers() and I have added a new param set by default to false which is called $activeOnly. What you pass to this, will be automatically changing the call to maintainer::get and pass a third parameter which will be true or false. include/pear-database.php maintainer::get() ----> this function will execute the query on the database, thus I have made it that if you pass the new third parameter true to it, then it will check only the active set to 1. public_html/trackback/trackback.php ----> I added true to the $trackback->getMaintainers() function in order to only get maintainers that are set to true. This $trackback variable is Damblan_Trackback instance, thus changed Damblan_Trackback::getMaintainers(true) so that it calls pear-database::maintainer::get('id', true, true) making pear-database::maintainer::get() query only the active lead. If this is not clear just look at the patches, should be rather simple and straightforward. ------------------------------------------------------------------------ [2006-07-10 14:45:36] smith at pooteeweet dot org Description: ------------ it would be nice if developers that are marked as "in active" for a given package no longer receive email notifications for trackbacks and bugs (and anything else we mail maintainers for) for this package. ------------------------------------------------------------------------ -- Edit this bug report at http://pear.php.net/bugs/bug.php?id=8176&edit=1

« previous php.pear.webmaster (#5327) next »