[Tickets #4782] ldap driver not searching for correct ingo boundar

[bugs@bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/?id=4782
-----------------------------------------------------------------------
 Ticket             | 4782
 Created By         | tansy at pacific.net.sg
 Summary            | ldap driver not searching for correct ingo boundar
 Queue              | Ingo
 Version            | 1.1.2
 Type               | Bug
 State              | Unconfirmed
 Priority           | 2. Medium
 Owners             | 
-----------------------------------------------------------------------


tansy at pacific.net.sg (2006-12-14 02:43) wrote:

Hi,

Not sure if I am doing something wrong, but it seems that ldap.php is not
detecting the Ingo filter boundaries correctly.

In ingo/lib/Driver/ldap.php, there are two lines:

if (strpos($value, "##INGO\n") !== false) {

However, on examination of the filtering scripts generated by Ingo, and
the screen captures on http://www.horde.org/ingo/screenshots/, it seems
that there is no such line: ##INGO.

This causes the "Deactivate Script" and "Show Active Script" buttons in
Ingo to fail.

The strpos() line should be searching for lines like "# maildrop script"
(depending on the filter used) instead?