From:  withmiranda@gmail.com ("sharonmiranda")
Date:  30 Apr 2018 01:38:11 Hong Kong Time
Newsgroup:  news.php.net/php.pear.bugs
Subject:  

[PEAR-BUG] Bug #20976 [Com]: Making static method non static - isError

NNTP-Posting-Host:  null

Edit report at http://pear.php.net/bugs/bug.php?id=20976&edit=1

 ID:               20976
 Comment by:       sharonmiranda
 Reported By:      withmiranda at gmail dot com
 Summary:          Making static method non static - isError
 Status:           Open
 Type:             Bug
 Package:          Mail_Queue
 Operating System: Linux CentOS
 Package Version:  1.2.7
 PHP Version:      5.4.45
 New Comment:

mariom, can you please provide instruction as to how to apply your patch
code?


Previous Comments:
------------------------------------------------------------------------

[2017-01-11 02:28:33] mariom

Hi!

I get few emails that patch link does not work anymore. As patch is
short I'll past it in comment:

diff --git fudge/general/www.inc fudge/general/www.inc
index 8a79fa1..b89fe29 100755
--- fudge/general/www.inc
+++ fudge/general/www.inc
@@ -112,10 +112,10 @@ function valid_email($e)

     // dot character cannot be the first or last character in the
local-part
 	// and it cannot appear two or more times consecutively
-	if (preg_match('/^[a-zA-Z]+(([\'\,\.\- ][a-zA-Z
])?[a-zA-Z]*)*\s+<(['.$local.']+(\.['.$local.']+)*@(((?:[\da-zA-Z]|[\da-zA-Z][\'-\w]*[\da-zA-Z])\.)+[a-zA-Z]{2,7}))>$/',
$e)) {
+	if (preg_match('/^[a-zA-Z]+(([\'\,\.\- ][a-zA-Z
])?[a-zA-Z]*)*\s+<(['.$local.']+(\.['.$local.']+)*@(((?:[\da-zA-Z]|[\da-zA-Z][\'\-\w]*[\da-zA-Z])\.)+[a-zA-Z]{2,7}))>$/',
$e)) {
         // matches email with display name, for example, 'Someone
'
         return TRUE;
-    } else if
(preg_match('/^(['.$local.']+(\.['.$local.']+)*@(((?:[\da-zA-Z]|[\da-zA-Z][\'-\w]*[\da-zA-Z])\.)+[a-zA-Z]{2,7}))$/',
$e)) {
+    } else if
(preg_match('/^(['.$local.']+(\.['.$local.']+)*@(((?:[\da-zA-Z]|[\da-zA-Z][\'\-\w]*[\da-zA-Z])\.)+[a-zA-Z]{2,7}))$/',
$e)) {
         // matches normal email address
         return TRUE;
     } else {

------------------------------------------------------------------------

[2016-05-11 12:34:09] cody

It is still broken, patch-link doesnt work (anymore?) !

------------------------------------------------------------------------

[2016-03-17 09:01:24] artur

link for patch dose't work ((

------------------------------------------------------------------------

[2015-10-29 11:14:52] mariom

Added #patch bug:20976;patch:iserror.patch;revision:1446117292;.

------------------------------------------------------------------------

[2015-10-29 11:13:01] mariom

Description:
------------
Hi,

Mail_Queue is broken, as it tries to make static method PEAR::isError()
nonstatic on line 590 in Mail_Queue.php

Just adding the keyword "static" fixes the problem.


Test script:
---------------