- Fixed #44098, imap_utf8() returns only capital letters

diff --git a/NEWS b/NEWS
index 622beb84189f32fdfbf55c8390990788e9c20499..f041d854e3f7feb20c553661ffaf55e844211c21 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -69,7 +69,8 @@ PHP                                                                        NEWS
 - Fixed bug #47281 ($php_errormsg is limited in size of characters)
   (Oracle Corp.)
 - Fixed bug #44827 (define() allows :: in constant names). (Ilia)
-
+- Fixed bug #44098 (imap_utf8() returns only capital letters).
+  (steffen at dislabs dot de, Pierre)
 
 ?? ??? 20??, PHP 5.3.2
 - Upgraded bundled sqlite to version 3.6.21. (Ilia)

diff --git a/ext/imap/config.m4 b/ext/imap/config.m4
index ec777ce22889ea593ad87e546e8b0a8c11e6d13b..3ad7c107ddac1e489d64edf7221cd79ae045d42d 100644 (file)
--- a/ext/imap/config.m4
+++ b/ext/imap/config.m4
@@ -147,23 +147,23 @@ if test "$PHP_IMAP" != "no"; then
 
     old_CFLAGS=$CFLAGS
     CFLAGS="-I$IMAP_INC_DIR"
-    AC_CACHE_CHECK(for U8T_CANONICAL, ac_cv_u8t_canonical,
+    AC_CACHE_CHECK(for U8T_DECOMPOSE, ac_cv_u8t_canonical,
       AC_TRY_COMPILE([
 #include <c-client.h>
       ],[
          int i = U8T_CANONICAL;
       ],[
-         ac_cv_u8t_canonical=yes
+         ac_cv_u8t_decompose=yes
       ],[
-         ac_cv_u8t_canonical=no
+         ac_cv_u8t_decompose=no
       ])
     )
     CFLAGS=$old_CFLAGS
 
-    if test "$ac_cv_u8t_canonical" = "no" && test "$ac_cv_utf8_mime2text" = "new"; then
+    if test "$ac_cv_u8t_decompose" = "no" && test "$ac_cv_utf8_mime2text" = "new"; then
                AC_MSG_ERROR([utf8_mime2text() has new signature, but U8T_CANONICAL is missing. This should not happen. Check config.log for additional information.])
     fi
-    if test "$ac_cv_u8t_canonical" = "yes" && test "$ac_cv_utf8_mime2text" = "old"; then
+    if test "$ac_cv_u8t_decompose" = "yes" && test "$ac_cv_utf8_mime2text" = "old"; then
                AC_MSG_ERROR([utf8_mime2text() has old signature, but U8T_CANONICAL is present. This should not happen. Check config.log for additional information.])
     fi
 

diff --git a/ext/imap/config.w32 b/ext/imap/config.w32
index beb12f03e854f2003e8931364cec973b56eedd64..5778c34bb4937d9e98bd4c5d9fc7bb673fec2a86 100644 (file)
--- a/ext/imap/config.w32
+++ b/ext/imap/config.w32
@@ -21,6 +21,7 @@ if (PHP_IMAP == "yes") {
                AC_DEFINE('HAVE_NEW_MIME2TEXT', 1, 'Have utf8_mime2text', true);
                AC_DEFINE('HAVE_RFC822_OUTPUT_ADDRESS_LIST', 1, 'Have rfc822_output_address_list', true);
                AC_DEFINE('HAVE_IMAP_MUTF7', 1, 'Have modified utf7 support', true);
+               AC_DEFINE('HAVE_NEW_MIME2TEXT', 1, 'Whether utf8_mime2text() has new signature');
        } else {
                WARNING("imap not enabled; libraries and headers not found");
        }

diff --git a/ext/imap/php_imap.c b/ext/imap/php_imap.c
index aaf1c96b64b565836e5f52ee1e5c1b2ed3350df6..5e47510d484e561c9c3451a5c0e5591f1bb37aa0 100644 (file)
--- a/ext/imap/php_imap.c
+++ b/ext/imap/php_imap.c
@@ -2664,7 +2664,7 @@ PHP_FUNCTION(imap_utf8)
 #ifndef HAVE_NEW_MIME2TEXT
        utf8_mime2text(&src, &dest);
 #else
-       utf8_mime2text(&src, &dest, U8T_CANONICAL);
+       utf8_mime2text(&src, &dest, U8T_DECOMPOSE);
 #endif
        RETVAL_STRINGL(dest.data, dest.size, 1);
        if (dest.data) {

diff --git a/ext/imap/tests/bug44098.phpt b/ext/imap/tests/bug44098.phpt
new file mode 100644 (file)
index 0000000..f758c11
--- /dev/null
+++ b/ext/imap/tests/bug44098.phpt
@@ -0,0 +1,20 @@
+--TEST--
+Bug #44098 (imap_utf8() returns only capital letters)
+--SKIPIF--
+<?php
+        if (!extension_loaded("imap")) { 
+                die("skip imap extension not available");  
+        }
+?>
+--FILE--
+<?php
+$exp = 'Luzon®14 dot CoM';
+$res = imap_utf8('=?iso-8859-1?b?THV6b26uMTQ=?= dot CoM');
+if ($res != $exp) {
+       echo "failed: got <$res>, expected <exp>\n";
+} else {
+       echo "ok";
+}
+?>
+--EXPECT--
+ok