Re: sThreadResponse error with SVN trunk

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 7/29/07, James Pooton <james@xxxxxxxxxxxxxxxxxx> wrote:
> Hello,
>
> I checked out the latest SVN trunk today and all looks good except for a seemingly
> random error that comes up during mail "searches":
>
> Category:  PHP
> Message:  Undefined variable: sThreadResponse
> FILE:  /var/www/html/mail.moonshineridge.com/squirrelmail/functions/imap_messages.php
> LINE:  398
>
> I'm using Cyrus and it has thread sorting, etc.  But even so, this looks more like
> a script problem where the variable hasn't been set.
>
> I'm on PHP 5.2 FWIW...
>
> Anyone else seeing this?

Not me, but this should fix it:

Index: functions/imap_messages.php
===================================================================
--- functions/imap_messages.php (revision 12567)
+++ functions/imap_messages.php (working copy)
@@ -335,6 +335,7 @@
     } elseif ($response == 'BAD') {
         sqm_trigger_imap_error('SQM_IMAP_NO_THREAD',$query,
$response, $message);
     }
+    $sThreadResponse = '';
     if (isset($sRead[0])) {
         for ($i=0,$iCnt=count($sRead);$i<$iCnt;++$i) {
             if (preg_match("/^\* THREAD (.+)$/", $sRead[$i], $aMatch)) {
@@ -342,8 +343,6 @@
                 break;
             }
         }
-    } else {
-        $sThreadResponse = "";
     }
     unset($sRead);

Thanks,
paul

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/
--
squirrelmail-users mailing list
Posting Guidelines: http://www.squirrelmail.org/wiki/MailingListPostingGuidelines
List Address: squirrelmail-users@xxxxxxxxxxxxxxxxxxxxx
List Archives: http://news.gmane.org/thread.php?group=gmane.mail.squirrelmail.user
List Archives:  http://sourceforge.net/mailarchive/forum.php?forum_id=2995
List Info: https://lists.sourceforge.net/lists/listinfo/squirrelmail-users

[Index of Archives]     [Video For Linux]     [Yosemite News]     [Yosemite Photos]     [gtk]     [KDE]     [Cyrus SASL]     [Gimp on Windows]     [Steve's Art]     [Webcams]

  Powered by Linux