Tickets :: [#12680] adding Organizer as attendee needs own fb-url in address book

6.0.0-alpha10

5/14/25

Summary	adding Organizer as attendee needs own fb-url in address book
Queue	Kronolith
Queue Version	Git master
Type	Enhancement
State	Assigned
Priority	1. Low
Owners	jan (at) horde (dot) org
Requester	thpo+horde (at) dotrc (dot) de
Created	09/13/2013 (4261 days ago)
Due
Updated	01/28/2016 (3394 days ago)
Assigned	01/28/2016 (3394 days ago)
Resolved
Milestone	5.0.0
Patch	No

01/28/2016 03:06:30 PM	Jan Schneider	Assigned to Jan Schneider State ⇒ Assigned Milestone ⇒ 5.0.0 Patch ⇒ No

09/16/2013 08:48:28 AM	Jan Schneider	Comment #7	Reply to this comment
This approach is completely wrong. Instead of querying the FB URL for the current user internally, the FB information can (and has to) be generated directly, on the fly. See attendees.php on how this works in the basic view.

09/15/2013 02:08:06 PM	Michael Rubinsky	Comment #6	Reply to this comment
The idea seems sound to me, but you should refactor that so that you only query the identity system if we know we need it. I.e., Check $url first, and then get identity and do the comparisons etc...

09/15/2013 10:35:00 AM	thpo+horde (at) dotrc (dot) de	Comment #5 New Attachment: owner-fb-url[2].patch	Reply to this comment
So how about this patch? It first checks for a given FB-URL of the owner, and only if there was none, it sets the default. Sorry, wrong patch again. :-(

09/15/2013 10:32:31 AM	thpo+horde (at) dotrc (dot) de	Comment #4 New Attachment: owner-fb-url[1].patch	Reply to this comment
So how about this patch? It first checks for a given FB-URL of the owner, and only if there was none, it sets the default.

09/13/2013 08:03:49 PM	Michael Rubinsky	Comment #3 State ⇒ Feedback	Reply to this comment
This patch assumes that the user wants to use his/her Kronolith calendar as the canonical source for FB information. Most likely, yes, but we never enforced that restriction before.

09/13/2013 09:53:31 AM	thpo+horde (at) dotrc (dot) de	Comment #2	Reply to this comment
Sorry, I attached the wrong patch: strcmp should check "=== 0" instead of "!== 0"

09/13/2013 09:46:29 AM	thpo+horde (at) dotrc (dot) de	Comment #1 Priority ⇒ 1. Low State ⇒ New New Attachment: owner-fb-url.patch Patch ⇒ Yes Milestone ⇒ Queue ⇒ Kronolith Summary ⇒ adding Organizer as attendee needs own fb-url in address book Type ⇒ Enhancement	Reply to this comment
When one schedules an event in kronolith and adds attendees, a free/busy column is added for the organizer as well. But this infomation is retrieved by searching the address-book for an entry with email = Kronolith.conf.email, that has a correct fb-URL set. The attached patch skips this search and sets the correkt fb-URL for the organizer directly.