Bug #2028

in function cmdCapability() missplaced curly brace

Details

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)

Comments

[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;
}