Changes have been made in Git (develop):

commit 958d00d4fdbe5221594c6c574ff9a1988d223bbf
Author: Jan Schneider <jan@horde.org>
Date:   Tue Jun 26 11:59:54 2012 +0200

     [mms] Fix patch for filtering on combination fields (Bug #11197).

  ingo/docs/CHANGES |    1 +
  ingo/package.xml  |    2 ++
  2 files changed, 3 insertions(+), 0 deletions(-)

http://git.horde.org/horde-git/-/commit/958d00d4fdbe5221594c6c574ff9a1988d223bbf

Changes have been made in Git (master):

commit 958d00d4fdbe5221594c6c574ff9a1988d223bbf
Author: Jan Schneider <jan@horde.org>
Date:   Tue Jun 26 11:59:54 2012 +0200

     [mms] Fix patch for filtering on combination fields (Bug #11197).

  ingo/docs/CHANGES |    1 +
  ingo/package.xml  |    2 ++
  2 files changed, 3 insertions(+), 0 deletions(-)

http://git.horde.org/horde-git/-/commit/958d00d4fdbe5221594c6c574ff9a1988d223bbf

It is fixed. Sorry for delay.

Can somebody *please* test those (both!) patches and report back if 
this is fixed. We won't release a new version without confirmation.

You didn't apply BOTH patches.

2012-06-08T15:12:38+02:00 DEBUG: HORDE [ingo] SQL  (0,0025s)
        SELECT * FROM ingo_rules WHERE rule_owner = 'rvuser' ORDER BY 
rule_order [pid 3666 on line 803 of 
"/usr/share/php/Horde/Db/Adapter/Base.php"]
2012-06-08T15:12:38+02:00 WARN: HORDE [ingo] PHP ERROR: array_merge(): 
Argument #2 is not an array [pid 3666 on line 626 of 
"/usr/share/php/Horde/Imap/Client/Search/Query.php"]
  7. Ingo_Script_Imap->perform() /var/www/horde/ingo/lib/Api.php:108
  8. Horde_Imap_Client_Search_Query->andSearch() 
/var/www/horde/ingo/lib/Script/Imap.php:236
2012-06-08T15:12:38+02:00 DEBUG: HORDE [ingo]  1. 
IMP_Mailbox->filter() /var/www/horde/imp/mailbox.php:214
  7. Ingo_Script_Imap->perform() /var/www/horde/ingo/lib/Api.php:108
  8. Horde_Imap_Client_Search_Query->andSearch() 
/var/www/horde/ingo/lib/Script/Imap.php:236
2012-06-08T15:12:39+02:00 DEBUG: HORDE [ingo] Filteraktivität: Die 
Nachricht "rhsh rfesrh" von "<123@456.78>" wurde zum Ordner "test" 
verschoben. [pid 3666 on line 27 of 
"/usr/share/php/Horde/Core/Notification/Handler/Decorator/Hordelog.php"]
2012-06-08T15:12:39+02:00 DEBUG: HORDE [ingo] Filteraktivität: Die 
Nachricht "123" von "<123@456.78>" wurde zum Ordner "test" verschoben. 
[pid 3666 on line 27 of 
"/usr/share/php/Horde/Core/Notification/Handler/Decorator/Hordelog.php"]
2012-06-08T15:12:39+02:00 DEBUG: HORDE [ingo] Filteraktivität: Die 
Nachricht "test" von "<123@456.78>" wurde zum Ordner "test" 
verschoben. [pid 3666 on line 27 of 
"/usr/share/php/Horde/Core/Notification/Handler/Decorator/Hordelog.php"]

Not solved!

I would like to test but I don't know how to get/apply patches. Also 
the installation is pear installation. If anybody can provide me help 
on how to do so, I am willing to test and give feedback on whether the 
problem disappeared.

If this is such high priority, did anybody actually test the committed 
fixes and can confirm whether the issue is fixed now or not?

Changes have been made in Git (develop):

commit 460d3766089df6b20f0ec47d51ea27a0ef654fc7
Author: Michael M Slusarz <slusarz@horde.org>
Date:   Mon Jun 4 14:23:26 2012 -0600

     Bug #11197: Another BC fix

  ingo/lib/Script/Imap.php |    2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)

http://git.horde.org/horde-git/-/commit/460d3766089df6b20f0ec47d51ea27a0ef654fc7

Changes have been made in Git (master):

commit 460d3766089df6b20f0ec47d51ea27a0ef654fc7
Author: Michael M Slusarz <slusarz@horde.org>
Date:   Mon Jun 4 14:23:26 2012 -0600

     Bug #11197: Another BC fix

  ingo/lib/Script/Imap.php |    2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)

http://git.horde.org/horde-git/-/commit/460d3766089df6b20f0ec47d51ea27a0ef654fc7

I guess line 159 of ingo/lib/Script/Imap.php need a fix as well from:
$query->andSearch($or_ob);
to
$query->andSearch(array($or_ob));

Hi,
I had the same issue and I have applied the fix on 
ingo/lib/Script/Imap.php however I am now getting the following:

HORDE: [ingo] PHP ERROR: array_merge(): Argument #2 is not an array 
[pid xxx on line 626 of 
"/usr/share/pear/Horde/Imap/Client/Search/Query.php"]

Changes have been made in Git (develop):

commit 958daacad7e1b85c539a9f5fc00b9fade123c645
Author: Michael M Slusarz <slusarz@horde.org>
Date:   Thu May 31 12:00:42 2012 -0600

     Make this call BC (Bug #11197)

  ingo/lib/Script/Imap.php |    2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)

http://git.horde.org/horde-git/-/commit/958daacad7e1b85c539a9f5fc00b9fade123c645

This is exactly what would be expected given the error message 
reported.  Since the specific AND/OR rules are not correctly added to 
the base query object, the base query object would default to an ALL 
search.

Forgot to describe how il is broken:

Filtering rules are as follows:
if match X-headder1 then move to directory dir1 stop
if match X-headder2 then move to directory dir2 stop
...

No matter if headder is matched or not, ALL messages are moved to the 
directory dir1

Also, ingo version is 2.0.8

Hope this helps

Hello,

I just upgraded to the latest groupware edition et filtering is now 
broken and I believe il has to do with this bug. My installation was 
up to date last friday, up and running smoothly and filtering have 
been running fine for months.

My filter backend is SQL.
The filter rules look for specific custom headder and move message to 
different folders accordingly. All the rules are simple, i.e. only one 
headder is checked, if matched, move message then stop.

Here is what I get in the syslog when filter is applied. Hope it will help

May 31 14:57:39 scommuns HORDE: [ingo] PHP ERROR: array_merge(): 
Argument #2 is not an array [pid 30898 on line 626 of 
"/usr/share/pear/Horde/Imap/Client/Search/Query.php"]
May 31 14:57:39 scommuns HORDE: [ingo] PHP ERROR: array_merge(): 
Argument #2 is not an array [pid 30898 on line 626 of 
"/usr/share/pear/Horde/Imap/Client/Search/Query.php"]
May 31 14:57:39 scommuns HORDE: [ingo] PHP ERROR: array_merge(): 
Argument #2 is not an array [pid 30898 on line 626 of 
"/usr/share/pear/Horde/Imap/Client/Search/Query.php"]
May 31 14:57:40 scommuns HORDE: [ingo] PHP ERROR: array_merge(): 
Argument #2 is not an array [pid 30898 on line 626 of 
"/usr/share/pear/Horde/Imap/Client/Search/Query.php"]
May 31 14:57:40 scommuns HORDE: [ingo] PHP ERROR: array_merge(): 
Argument #2 is not an array [pid 30898 on line 626 of 
"/usr/share/pear/Horde/Imap/Client/Search/Query.php"]
May 31 14:57:40 scommuns HORDE: [ingo] PHP ERROR: array_merge(): 
Argument #2 is not an array [pid 30898 on line 626 of 
"/usr/share/pear/Horde/Imap/Client/Search/Query.php"]
May 31 14:57:40 scommuns HORDE: [ingo] PHP ERROR: array_merge(): 
Argument #2 is not an array [pid 30898 on line 626 of 
"/usr/share/pear/Horde/Imap/Client/Search/Query.php"]
May 31 14:57:40 scommuns HORDE: [ingo] PHP ERROR: array_merge(): 
Argument #2 is not an array [pid 30898 on line 626 of 
"/usr/share/pear/Horde/Imap/Client/Search/Query.php"]

Or maybe this helps, since this code works with Imap_Client v2 but not v1.

Changes have been made in Git (master):

commit 958daacad7e1b85c539a9f5fc00b9fade123c645
Author: Michael M Slusarz <slusarz@horde.org>
Date:   Thu May 31 12:00:42 2012 -0600

     Make this call BC (Bug #11197)

  ingo/lib/Script/Imap.php |    2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)

http://git.horde.org/horde-git/-/commit/958daacad7e1b85c539a9f5fc00b9fade123c645

I'm going to revert then.  I don't have time to look at this right now.

This seems to have done more bad than good.

Changes have been made in Git (develop):

commit 3a1776beec29f6b5cd8aaa213f2c93b04d4d7682
Author: Michael M Slusarz <slusarz@horde.org>
Date:   Thu May 24 12:26:17 2012 -0600

     [mms] Fix IMAP filtering on combination fields (Bug #11197).

  ingo/docs/CHANGES        |    1 +
  ingo/lib/Script/Imap.php |   48 
+++++++++++++++++++++++-----------------------
  ingo/package.xml         |    2 +
  3 files changed, 27 insertions(+), 24 deletions(-)

http://git.horde.org/horde-git/-/commit/3a1776beec29f6b5cd8aaa213f2c93b04d4d7682

Does this fix?

Changes have been made in Git (master):

commit 3a1776beec29f6b5cd8aaa213f2c93b04d4d7682
Author: Michael M Slusarz <slusarz@horde.org>
Date:   Thu May 24 12:26:17 2012 -0600

     [mms] Fix IMAP filtering on combination fields (Bug #11197).

  ingo/docs/CHANGES        |    1 +
  ingo/lib/Script/Imap.php |   48 
+++++++++++++++++++++++-----------------------
  ingo/package.xml         |    2 +
  3 files changed, 27 insertions(+), 24 deletions(-)

http://git.horde.org/horde-git/-/commit/3a1776beec29f6b5cd8aaa213f2c93b04d4d7682

See attached file imp-debug-log,  it was generated while processing 
the filter for 4 items on inbox, that were moved by the filter to 
"Nueva" folder.

To further debug this issue, we need details of the IMP -> IMAP/POP 
communication.

To enable debugging, see instructions contained in 
imp/config/backends.php (the 'debug' config parameter).

Debugging should not be enabled on a production server,   Attach/post 
only the portion of the log that directly deals with the problem 
reported (it may be simplest to clear the log file and then perform 
the event that causes the error).

We are using default settings provided in ingo/config/backends.php, so 
imap is the only backend enabled.
In the other hand in config/conf.php we have setted 
$conf[storage][driver]=SQL, as you can see all settings are by default.

You need to provide much more information, most importantly, what 
filter backend you use, and how the generated script looks like, if any.

Using the following fields in the definition of a filter the result of 
the evaluation is always "true", and therefore the filter action 
affects all messages in inbox:  Destination (To,Cc,Bcc,etc.)  -   
Source (From,Reply-to,etc.)  -  Participant (From,To,etc.)