Bug 79860 - Tools -> Find Messages... can't find Arabic text if it's in the body but works properly if it's in the subject
Summary: Tools -> Find Messages... can't find Arabic text if it's in the body but work...
Status: RESOLVED FIXED
Alias: None
Product: kmail
Classification: Unmaintained
Component: search (show other bugs)
Version: unspecified
Platform: Mandrake RPMs Linux
: NOR normal
Target Milestone: ---
Assignee: kdepim bugs
URL:
Keywords:
: 80633 (view as bug list)
Depends on:
Blocks:
 
Reported: 2004-04-18 05:13 UTC by Munzir Taha
Modified: 2007-09-14 12:17 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Munzir Taha 2004-04-18 05:13:26 UTC
Version:            (using KDE KDE 3.2.0)
Installed from:    Mandrake RPMs
OS:          Linux

Tools -> Find Messages... can't find Arabic text if it's in the body but works properly if it's in the subject. This is very wired.

It's very important for the Arabic community to fix this bug as soon as possible. We can't find/search any Arabic text that appear in the body of the messsage.
Comment 1 Till Adam 2004-06-28 23:41:53 UTC
CVS commit by tilladam: 

Use KMMessage::bodyToUnicode() instead of KMMessage::bodyDecoded() which, 
unlike KMMsgPart::bodyDecoded() does content transfer decoding and not
charset/codec decoding

This fixes searching in non-latin1 mail bodies for those mails, at least,
which don't claim they have quotedprintable and are in reality 7/8 bit.

Muunzir, thanks for the bug report, can you confirm this fixes it?

CCMAIL: 79860-done@bugs.kde.org


  M +1 -1      kmsearchpattern.cpp   1.68


--- kdepim/kmail/kmsearchpattern.cpp  #1.67:1.68
@@ -295,5 +295,5 @@ bool KMSearchRuleString::matches( const 
     logContents = false;
   } else if ( field() == "<body>" ) {
-    msgContents = msg->bodyDecoded();
+    msgContents = msg->bodyToUnicode();
     logContents = false;
   } else if ( field() == "<any header>" ) {


Comment 2 Munzir Taha Obeid 2004-06-30 01:06:09 UTC
I am sorry I can't test now. Any one else?
Thanks a lot for fixing this annoying bug.
Comment 3 Ahmad Okda 2004-06-30 01:56:04 UTC
i can try to test it but i don't know how to try it 
if anyone can show me how to apply the fix or install it i can try to try it and give feedback
Comment 4 Till Adam 2004-07-13 16:19:29 UTC
*** Bug 80633 has been marked as a duplicate of this bug. ***
Comment 5 Dmitri Drozdov 2004-07-16 07:16:34 UTC
Search of the Cyrillic text works properly now in headers and in message body. My thanks to Till Adam! :-)