Summary | More than one entry returned while obtaining Free/Busy URL |
Queue | Kronolith |
Queue Version | HEAD |
Type | Bug |
State | Resolved |
Priority | 1. Low |
Owners | |
Requester | flgomes (at) fazenda (dot) sp (dot) gov (dot) br |
Created | 02/10/2006 (7106 days ago) |
Due | |
Updated | 02/21/2006 (7095 days ago) |
Assigned | 02/10/2006 (7106 days ago) |
Resolved | 02/21/2006 (7095 days ago) |
Github Issue Link | |
Github Pull Request | |
Milestone | |
Patch | No |
State ⇒ Resolved
documentation to flgoms (at) uol (dot) com (dot) br ? It's my
personal e-mail.
New Attachment: Driver.php.patch
because I already committed most of your original patch.
documentation to flgoms (at) uol (dot) com (dot) br ? It's my
personal e-mail.
because I already committed most of your original patch.
New Attachment: new_kronolith_turba_2.patch
breaks any other search and committed the rest. Let me know if
everything works fine now.
$strict['mail']. As far as I could understand that hash key should not
be mapped/translated. I fear it won't work if that key get translated.
Unless I send 'mail' instead of 'email' in api.php:1207 wich will only
work with ldap since the others backends have different key mapping.
To solve that I can translate that key before send it to makeSearch():
$strict_fields[$this->map[$strict_field]] = true;
Then, that key will be translated again in makeSearch(). I attached a
new version of the patch with that change.
What do you suggest?
breaks any other search and committed the rest. Let me know if
everything works fine now.
New Attachment: new_kronolith_turba.patch
I just added a new patch (new_kronolith_turba.patch), it is updated
for Kronolith RC3 and Turba RC2.
And I made the changes you suggested. Please check them and tell me
if they are ok.
That patch was only tested with LDAP backend.
Thank you.
only anyway, it doesn't make much sense to specify the fields that
need to be strict in the parameters. The parameter should be a boolean
instead and getField() should pass 'email' as a hardcoded strict field
then.
Version ⇒ HEAD
patch must be against HEAD or at least Kronolith 2.1-RC3 and Turba
2.1-RC2.
New Attachment: Kronolith_Turba.patch
files for both Turba and Kronolith.
State ⇒ Feedback
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ More than one entry returned while obtaining Free/Busy URL
Queue ⇒ Kronolith
New Attachment: Patch.tar.gz
State ⇒ Unconfirmed
using the default Turba configuration wich makes it use the 'LIKE'
operand (mail = *email@domain.foo*, for LDAP). Doing that, the results
returned may be more than one entry, causing Kronolith to discard all
of them. The correct behavior should search for exact matches (mail =
email@domain.foo), which would return only one entry if found.
I attached one patch with changes in Turba's (2.0.5) lib/api.php and
lib/Driver.php and
Kronolith's (2.0.6) lib/Kronolith.php that enforces strict search for
the 'email' field.
That patch was only tested using LDAP backend and it should be checked
for more experienced Horde programmers to ensure if it is not breaking
any compatibility.