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

6.0.0-beta6

4/1/26

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	5/26/11 (5424 days ago)
Due
Updated	9/6/17 (3129 days ago)
Assigned
Resolved	5/26/11 (5424 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.