Req #8176 [Asn->Csd]: do not mail in active developers
| From: | cellog@php.net | 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