Tickets :: [#10128] Horde_Imap_Client_Utf7ConvertTest::testAmpersandConversion fails

6.0.0-beta1

7/15/25

Summary	Horde_Imap_Client_Utf7ConvertTest::testAmpersandConversion fails
Queue	Horde Framework Packages
Queue Version	Git master
Type	Bug
State	Duplicate
Priority	1. Low
Owners	slusarz (at) horde (dot) org
Requester	wrobel (at) horde (dot) org
Created	05/26/2011 (5164 days ago)
Due
Updated	09/06/2017 (2869 days ago)
Assigned
Resolved	05/26/2011 (5164 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

09/06/2017 08:18:14 AM	Git Commit	Comment #6	Reply to this comment
Changes have been made in Git (master): commit acd883609ea578d22c36fab94f99018011ea4d11 Author: Justin Patrin <justinpatrin@php.net> Date: Thu Mar 15 15:04:50 2007 +0000 ~~Request #10128~~ use CSS instead of the depracetd align attribute for images. Thanks to ritzmo (Moritz Venn) for the patch git-svn-id: https://svn.php.net/repository/pear/packages/Text_Wiki/trunk@231923 c90b9560-bf6c-de11-be94-00142212c4b1 Text/Wiki/Render/Xhtml/Image.php \| 26 +++++++++++++++----------- 1 file changed, 15 insertions(+), 11 deletions(-) http://github.com/horde/horde/commit/acd883609ea578d22c36fab94f99018011ea4d11

05/26/2011 08:50:05 PM	Michael Slusarz	Comment #5 State ⇒ Duplicate	Reply to this comment
Not sure if the [:alnum:] is not catching the 'é' character properly. Hm... 'é'? Which 'é'? $orig = 'Envoy&AOk-'; &AOK- -> é when UTF7-IMAP decoded. I answered my own question... [:alnum:] won't work properly since it is dependent on the current C locale. So I'm reverting my previous fix and will just have to live with the fact that a mailbox name such as "Foo&Bar-Baz" will not work properly for now. So marking as Duplicate of ~~Bug #10093~~.

05/26/2011 08:37:10 PM	Gunnar Wrobel	Comment #4	Reply to this comment
Not sure if the [:alnum:] is not catching the 'é' character properly. Hm... 'é'? Which 'é'? $orig = 'Envoy&AOk-'; https://github.com/horde/horde/blob/master/framework/Imap_Client/test/Horde/Imap/Client/Utf7ConvertTest.php#L84

05/26/2011 04:21:36 PM	Michael Slusarz	Comment #3	Reply to this comment
You could look to see if this line is not working correctly for you (line 185 in Utf7imap.php): !preg_match('/[[:alnum:][:punct:]]/u', self::Utf7ImapToUtf8($val[0]))) { Not sure if the [:alnum:] is not catching the 'é' character properly. Not sure why it wouldn't, though, since IIRC PCRE is bundled with PHP so this shouldn't be a local dependency issue.

05/26/2011 02:49:49 PM	Michael Slusarz	Comment #2	Reply to this comment
Runs fine here - I don't see any errors.

05/26/2011 08:59:36 AM	Gunnar Wrobel	Assigned to Michael Slusarz

05/26/2011 08:59:16 AM	Gunnar Wrobel	Comment #1 Priority ⇒ 1. Low Type ⇒ Bug Summary ⇒ Horde_Imap_Client_Utf7ConvertTest::testAmpersandConversion fails Queue ⇒ Horde Framework Packages Milestone ⇒ Patch ⇒ No State ⇒ Unconfirmed	Reply to this comment
1) Horde_Imap_Client_Utf7ConvertTest::testAmpersandConversion Failed asserting that two strings are equal. --- Expected +++ Actual @@ @@ -Envoy&AOk- +Envoy&-AOk- /horde/framework/Imap_Client/test/Horde/Imap/Client/Utf7ConvertTest.php:89 I see this locally and within the CI server. I'd like to add the package to ci.horde.org as well.