minor fixes for serversidefilter-1.5-1.2.7-RC1

View: New views
1 Messages — Rating Filter:   Alert me  

minor fixes for serversidefilter-1.5-1.2.7-RC1

by Jan-41 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Hello list, hello Paul and Tony

I also discovered the bug number 1956733 commited by tuxick
and tracked it down in options.php
170,171c167
<   $recipies[$_POST['rule']]['field'] = $fields[$_POST['field']];
//multilanguage problem
<   $recipies[$_POST['rule']]['field'] = $_POST['field'];
---
>   $recipies[$_POST['rule']]['field'] = $fields[$_POST['field']];

This patch seems also a bit buggy because it overrides the first if
instruction and the line 170 should therefore be commented. Nevertheless,
the problem is the $fields[$_POST['field']] value which leads to wrong
procmailrc-files for non-english versions.

I stumbled over the bug number 1763209 as well, which made problems for
using regexes.
I changed the following in recipie_functions.php:
178,184c177
< if ($matching == "Matches Regex"){ //if regexes are used
< $ssfield = $recipies[$i]['field']; //don't addslashes!
< }
< else
<  $ssfield = addslashes($recipies[$i]['field']);
<
<          $ssstring =$recipies[$i]['string'];
---
>          $ssstring = addslashes($recipies[$i]['string']);


And I had something similar like bug number 1586696.
I changed the case instruction in backend.php to:

172c170
<       case "Body" :
---
>       case _("Body") :
176c174
<       case "Any Header" :
---
>       case _("Any Header") :
180c178
<       case "Anywhere" :
---
>       case _("Anywhere") :
which solved it.

Btw. is this case _( something php special?

The last change prevents some problems if the maildir variable is set to
something with a ending slash:

backend.php

146c146,147
< $folder = '${MAILDIR}'.$folder;
---
>
>    $folder = '$MAILDIR'.$folder;

I like to mark some of my mails already as read with the formail function.
I only hacked this feature into it for mboxes, if anyone on the lists like
this feature I can delivery an configurable version for both mailbox
formats.

Just to get the idea, here is the diff from the backend.php:
151,154d151
<
<    if ($action == "mr")
<            $folder ="| formail -I \"Status: R\" >>".$folder;
<





-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
-----
squirrelmail-plugins mailing list
Posting guidelines: http://squirrelmail.org/postingguidelines
List address: squirrelmail-plugins@...
List archives: http://news.gmane.org/gmane.mail.squirrelmail.plugins
List info (subscribe/unsubscribe/change options): https://lists.sourceforge.net/lists/listinfo/squirrelmail-plugins
LightInTheBox - Buy quality products at wholesale price