[bugs] [Bug 1152] Changed - Turba cannot handle greek unicode entries correctly from an Active Directory

[bugs at bugs.horde.org]

http://bugs.horde.org/show_bug.cgi?id=1152

*** shadow/1152	Mon Jan 20 10:15:21 2003
--- shadow/1152.tmp.16944	Tue Jan 21 07:00:32 2003
***************
*** 3,10 ****
  Version: 2.1 Unstable
  Platform: PHP Code
  OS/Version: Linux
! Status: NEW   
! Resolution: 
  Severity: normal
  Priority: P2
  Component: Turba
--- 3,10 ----
  Version: 2.1 Unstable
  Platform: PHP Code
  OS/Version: Linux
! Status: RESOLVED   
! Resolution: FIXED
  Severity: normal
  Priority: P2
  Component: Turba
***************
*** 25,27 ****
--- 25,37 ----
  replaced utf8_encoding() and utf8_decoding with mb_convert_encoding() in 
  horde/turba/lib/Drive/ldap.php. This seems to solve the problem for me, 
  although I haven't really search'ed for sideaffects. Any opinions?
+ 
+ ------- Additional Comments From jan at horde.org  01/21/03 07:00 -------
+ The problem was that the utf8 functions from PHP used to en/decode data from the
+ ldap directory were build to handle iso-8859-1 data only. The charset used for
+ Greek is iso-8859-7 though.
+ 
+ If you use the mbstring functions to convert the charset, it's absolutely valid
+ and shouldn't have any sideeffects. What _might_ happen though is that you'll
+ have some users in the future that use a translation in a charset that mbstring
+ can't handle. But those wouldn't have any luck with the old method either.