Bug 79860

Summary:	Tools -> Find Messages... can't find Arabic text if it's in the body but works properly if it's in the subject
Product:	[Unmaintained] kmail	Reporter:	Munzir Taha <munzirtaha>
Component:	search	Assignee:	kdepim bugs <kdepim-bugs>
Status:	RESOLVED FIXED
Severity:	normal	CC:	ddv
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Mandrake RPMs
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:

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! :-)