Opened 17 years ago

Closed 14 years ago

#266 closed enhancement (fixed)

[PATCH] PEAR::Net_IMAP needs patching for use with PHP 5

Reported by: rjl Owned by: rjl
Priority: normal Milestone: 1.0.3
Component: PHP scripts Version: 1.0.0
Severity: normal Keywords: php 5 imap net_imap pear
Cc:

Description

While this isn't a Maia bug, it affects Maia users wishing to use PHP 5 and IMAP authentication. The bug and its solution are described here.

While this is likely to be fixed in future versions of PEAR::Net_IMAP, it plagues the current version (1.0.3), so a one-line patch for the IMAPProtocol.php file is all that's required to get it working with PHP 5. We should provide that patch with Maia until such time as a new release of PEAR::Net_IMAP is issued.

Furthermore, we can detect this condition within configtest.php--the combination of PHP version >= 5 and PEAR::Net_IMAP version == 1.0.3--and warn users to apply the supplied patch.

Attachments (1)

IMAPProtocol.php.patch (404 bytes) - added by rjl 17 years ago.
Patch file for IMAPProtocol.php from PEAR::Net_IMAP 1.0.3

Download all attachments as: .zip

Change History (6)

Changed 17 years ago by rjl

Patch file for IMAPProtocol.php from PEAR::Net_IMAP 1.0.3

comment:1 Changed 17 years ago by dmorton

  • Resolution set to fixed
  • Status changed from new to closed

[952], [953] Pointing users to this page to get patch.

comment:2 Changed 17 years ago by rjl

  • Summary changed from PEAR::Net_IMAP needs patching for use with PHP 5 to [PATCH] PEAR::Net_IMAP needs patching for use with PHP 5

comment:3 Changed 16 years ago by dmorton

  • patch set to 0

I also get an error when logging in, undefined variable $command on line 2707. By changing this to $token, the error goes away.

This can be a show stopper if debugging is turned on, as it causes a popup window and stops the redirect on xlogin.php.

comment:4 Changed 15 years ago by dmorton

  • Milestone changed from 1.0.1 to 1.0.3
  • Resolution fixed deleted
  • Status changed from closed to reopened

That last comment doesn't make sense?

Anyway, reopening this ticket because the url for the fix is wrong in the patches... it needs to go to www.maiamailguard.com now.

comment:5 Changed 14 years ago by rjl

  • Resolution set to fixed
  • Status changed from reopened to closed

URLs corrected in [1236].

Note: See TracTickets for help on using tickets.