Filtering fails with some fields

Using the following fields in the definition of a filter the

Tue, 15 May 2012 17:21:10 +0000

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.)

You need to provide much more information, most importantly,

Tue, 15 May 2012 17:35:32 +0000

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

> You need to provide much more information, most importantl

Tue, 15 May 2012 17:54:15 +0000

> You need to provide much more information, most importantly, what > filter backend you use, and how the generated script looks like, if > any. 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.

To further debug this issue, we need details of the IMP -> I

Tue, 15 May 2012 18:30:07 +0000

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).

See attached file imp-debug-log, it was generated while pro

Wed, 16 May 2012 14:30:25 +0000

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.

Changes have been made in Git (master): commit 3a1776beec29

Thu, 24 May 2012 18:26:45 +0000

Changes have been made in Git (master): commit 3a1776beec29f6b5cd8aaa213f2c93b04d4d7682 Author: Michael M Slusarz 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?

Thu, 24 May 2012 18:29:51 +0000

Does this fix?

Changes have been made in Git (develop): commit 3a1776beec2

Tue, 29 May 2012 17:43:13 +0000

Changes have been made in Git (develop): commit 3a1776beec29f6b5cd8aaa213f2c93b04d4d7682 Author: Michael M Slusarz 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

This seems to have done more bad than good.

Thu, 31 May 2012 10:27:55 +0000

This seems to have done more bad than good.

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

Thu, 31 May 2012 17:38:26 +0000

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

Changes have been made in Git (master): commit 958daacad7e1

Thu, 31 May 2012 18:00:53 +0000

Changes have been made in Git (master): commit 958daacad7e1b85c539a9f5fc00b9fade123c645 Author: Michael M Slusarz 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

Or maybe this helps, since this code works with Imap_Client

Thu, 31 May 2012 18:02:09 +0000

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

Hello, I just upgraded to the latest groupware edition et

Thu, 31 May 2012 19:11:25 +0000

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"]

Forgot to describe how il is broken: Filtering rules are

Fri, 01 Jun 2012 13:58:01 +0000

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

> No matter if headder is matched or not, ALL messages are m

Fri, 01 Jun 2012 16:46:25 +0000

> No matter if headder is matched or not, ALL messages are moved to the > directory dir1 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.

Changes have been made in Git (develop): commit 958daacad7e

Mon, 04 Jun 2012 15:14:48 +0000

Changes have been made in Git (develop): commit 958daacad7e1b85c539a9f5fc00b9fade123c645 Author: Michael M Slusarz 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

Hi, I had the same issue and I have applied the fix on ingo

Mon, 04 Jun 2012 16:53:32 +0000

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 > 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 guess line 159 of ingo/lib/Script/Imap.php need a fix as w

Mon, 04 Jun 2012 17:11:04 +0000

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 >> 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 >

Changes have been made in Git (master): commit 460d3766089d

Mon, 04 Jun 2012 20:23:44 +0000

Changes have been made in Git (master): commit 460d3766089df6b20f0ec47d51ea27a0ef654fc7 Author: Michael M Slusarz 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 (develop): commit 460d3766089

Tue, 05 Jun 2012 19:49:48 +0000

Changes have been made in Git (develop): commit 460d3766089df6b20f0ec47d51ea27a0ef654fc7 Author: Michael M Slusarz 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

If this is such high priority, did anybody actually test the

Wed, 06 Jun 2012 09:34:25 +0000

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

> If this is such high priority, did anybody actually test t

Wed, 06 Jun 2012 14:22:37 +0000

> If this is such high priority, did anybody actually test the > committed fixes and can confirm whether the issue is fixed now or not? 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.

2012-06-08T15:12:38+02:00 DEBUG: HORDE [ingo] SQL (0,0025s)

Fri, 08 Jun 2012 13:16:11 +0000

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!

> Not solved! You didn't apply BOTH patches.

Fri, 08 Jun 2012 23:15:18 +0000

> Not solved! You didn't apply BOTH patches.

Can somebody please test those (both!) patches and report

Sat, 16 Jun 2012 17:44:53 +0000

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

It is fixed. Sorry for delay.

Mon, 18 Jun 2012 06:36:09 +0000

It is fixed. Sorry for delay.

Changes have been made in Git (master): commit 958d00d4fdbe

Tue, 26 Jun 2012 10:04:32 +0000

Changes have been made in Git (master): commit 958d00d4fdbe5221594c6c574ff9a1988d223bbf Author: Jan Schneider 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 (develop): commit 958d00d4fdb

Tue, 26 Jun 2012 14:16:31 +0000

Changes have been made in Git (develop): commit 958d00d4fdbe5221594c6c574ff9a1988d223bbf Author: Jan Schneider 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