vtiger

[libregeek]

Hi all,
I have just tried to fix the issue detailed in http://forums.vtiger.com/viewtopic.php?t=4389
In my analysis I found that it's due to two minor bugs. The first one is in module/squirrelmail-1.4.4/move_messages.php. The problem is that the message(email body) send throught the location header is not "urlencoded". I just changed the following line:
header("Location: index.php?module=Emails&action=Save&fromemail=".$fromemail.
"&subject=".$subject."&idlist=".$tempidlist."&adddate=".$date."&detail=".urlencode($detail));

Please note the urlencode at the end of the line. So now the url will not blow up when it encounters an '@' symbol in the message body. As far these I am successful.

But now I am stucked with the file modules/Email/Save.php
I don't understand why the code starting from Line no 75 is included. IMO $descriptions = explode(",",$_REQUEST['detail']); does all the damage. I can't get any reason why the message body is scanned for any email ids. Is it a requirement ?
if there is any developers are available on YIM/AIM please feel free to contact me.
Once the bug is fixed I am ready to submit the patch.
My YIM screen name : libregeek
My AIM screen name : manilalkm.

regards
Manilal