Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.1.4

Bug #2028 in function cmdCapability() missplaced curly brace
Submitted: 2004-07-30 18:26 UTC
From: nic at postilion dot org Assigned: damian
Status: Closed Package: Net_IMAP
PHP Version: 4.3.3 OS: any
Roadmaps: (Not assigned)    
Subscription  
Comments Add Comment Add patch


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 50 + 47 = ?

 
 [2004-07-30 18:26 UTC] nic at postilion dot org
Description: ------------ In the function cmdCapability(), you close the if(isset( $ret["PARSED"] ) ) stanza and then do $this->_serverSupportedCapabilities = $ret["PARSED"]["CAPABILITIES"]; which should be done prior to the closing curly brace. Reproduce code: --------------- diff -u Net_IMAP-1.0.1/IMAPProtocol.php Net_IMAP-1.0.1a/IMAPProtocol.php --- Net_IMAP-1.0.1/IMAPProtocol.php 2004-06-26 10:42:31.000000000 -0500 +++ Net_IMAP-1.0.1a/IMAPProtocol.php 2004-07-30 08:23:27.000000000 -0500 @@ -959,9 +959,9 @@ if( strtoupper( substr( $auth_method , 0 ,5 ) ) == "AUTH=" ) $this->_serverAuthMethods[] = substr( $auth_method , 5 ); } + // Keep the capabilities response to use ir later + $this->_serverSupportedCapabilities = $ret["PARSED"]["CAPABILITIES"]; } - // Keep the capabilities response to use ir later - $this->_serverSupportedCapabilities = $ret["PARSED"]["CAPABILITIES"]; return $ret; }

Comments

 [2004-08-05 03:19 UTC] User who submitted this comment has not confirmed identity
If you submitted this note, check your email.If you do not have a message, click here to re-send
MANUAL CONFIRMATION IS NOT POSSIBLE.  Write a message to pear-dev@lists.php.net
to request the confirmation link.  All bugs/comments/patches associated with this

email address will be deleted within 48 hours if the account request is not confirmed!