6.0.0-beta1
7/26/25

[#3449] More than one entry returned while obtaining Free/Busy URL
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

History
02/21/2006 04:13:47 PM Chuck Hagenbuch Comment #11
State ⇒ Resolved
Reply to this comment
Here you are.
Committed, thanks.
BTW, can you please change my e-mail address in the
documentation to flgoms (at) uol (dot) com (dot) br   ? It's my
personal e-mail.
That's the only email address I see anywhere in Turba's source.
02/21/2006 12:17:49 PM flgomes (at) fazenda (dot) sp (dot) gov (dot) br Comment #10
New Attachment: Driver.php.patch Download
Reply to this comment
That sounds good. But please make a patch against current CVS HEAD,
because I already committed most of your original patch.
Here you are. BTW, can you please change my e-mail address in the 
documentation to flgoms (at) uol (dot) com (dot) br   ? It's my 
personal e-mail.
02/21/2006 12:08:16 PM Jan Schneider Comment #9 Reply to this comment
That sounds good. But please make a patch against current CVS HEAD, 
because I already committed most of your original patch.
02/21/2006 11:58:02 AM flgomes (at) fazenda (dot) sp (dot) gov (dot) br Comment #8
New Attachment: new_kronolith_turba_2.patch Download
Reply to this comment
Was the change to makeSearch() intended? I left it out because it
breaks any other search and committed the rest. Let me know if
everything works fine now.
Yes, it was intended because I'm sendind $strict['email'], not 
$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?
02/20/2006 10:18:00 PM Jan Schneider Comment #7 Reply to this comment
Was the change to makeSearch() intended? I left it out because it 
breaks any other search and committed the rest. Let me know if 
everything works fine now.
02/20/2006 09:45:00 PM Jan Schneider Deleted Original Message
 
02/20/2006 09:44:49 PM Jan Schneider Deleted Original Message
 
02/20/2006 09:11:11 PM flgomes (at) fazenda (dot) sp (dot) gov (dot) br Comment #6
New Attachment: new_kronolith_turba.patch Download
Reply to this comment
Hi Jan,



   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.
02/16/2006 01:55:53 PM Jan Schneider Comment #5 Reply to this comment
And a suggestion: As getField() allows searching for email address 
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.
02/16/2006 01:52:12 PM Jan Schneider Comment #4
Version ⇒ HEAD
Reply to this comment
Sorry, I missed this when looking at the patch the first time, but the 
patch must be against HEAD or at least Kronolith 2.1-RC3 and Turba 
2.1-RC2.
02/10/2006 06:50:53 PM flgomes (at) fazenda (dot) sp (dot) gov (dot) br Comment #3
New Attachment: Kronolith_Turba.patch
Reply to this comment
Can you create a single patch file?
There it goes. Although I'm not sure if it will help since there are 
files for both Turba and Kronolith.
02/10/2006 06:00:02 PM Jan Schneider Comment #2
State ⇒ Feedback
Reply to this comment
Can you create a single patch file?
02/10/2006 12:17:47 PM flgomes (at) fazenda (dot) sp (dot) gov (dot) br Comment #1
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
Reply to this comment
While trying to obtain Free/Busy URLs, Kronolith searchs the backends 
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.

Saved Queries