Tickets :: [#10767] Imple-Calendar-Link wrong

5.3.0-git

2015-01-30

Summary	Imple-Calendar-Link wrong
Queue	Kronolith
Queue Version	3.0.11
Type	Bug
State	Resolved
Priority	1. Low
Owners	mrubinsk (at) horde (dot) org
Requester	michael.groene (at) zel (dot) uni-hannover (dot) de
Created	2011-11-15 (1172 days ago)
Due
Updated	2011-12-06 (1151 days ago)
Assigned	2011-12-04 (1153 days ago)
Resolved	2011-12-04 (1153 days ago)
Milestone
Patch	No

2011-12-06 14:46:58	michael (dot) groene (at) zel (dot) uni-hannover (dot) de	Comment #12	Reply to this comment
Those links should be full urls now, thanks for the report. Thank you for fixing. Works well :)

2011-12-05 14:28:50	Michael Rubinsky	Comment #11	Reply to this comment
Those links should be full urls now, thanks for the report.

2011-12-05 14:26:39	Git Commit	Comment #10	Reply to this comment
Changes have been made in Git for this ticket: Output full urls for edit/delete links in blocks as well. ~~Bug: 10767~~ 1 files changed, 10 insertions(+), 8 deletions(-) http://git.horde.org/horde-git/-/commit/db9b754ebf6586fe4dd9973055abc9fad945bbb2

2011-12-05 06:12:09	michael (dot) groene (at) zel (dot) uni-hannover (dot) de	Comment #9 New Attachment: edit_delete.png	Reply to this comment
Thank you, this works, as expected now. A little issue still left in imple. I made a screenshot, which will describe the most. If the user is logged in horde in the same browser where the imple-calendars are loaded, too, the edit/delete-button are shown. The problem is, that they do not contain the whole uri of hordes website. 404 is the result... The event-link itself will be fine.

2011-12-04 23:14:04	Michael Rubinsky	Comment #8	Reply to this comment
Do you mean the display_cal variable in general, or support viewing any calendar type in a Kronolith block/embedded imple? For the former, it supports any calendar type, the latter will support any calendar type supported by the blocks - pretty much anything except external, from what I can tell.

2011-12-04 22:49:44	Jan Schneider	Comment #7	Reply to this comment
Though it still only supports internal calendars, right?

2011-12-04 22:31:31	Michael Rubinsky	Comment #6 State ⇒ Resolved Taken from Jan Schneider	Reply to this comment
This also affected Kronolith's blocks since the same rendering code is used for the embedded imples. Should be fixed now.

2011-12-04 22:30:23	Git Commit	Comment #5	Reply to this comment
Changes have been made in Git for this ticket: Catch cases where 'internal_' is prepended to the calendar id. When urls are build directly from the results of Kronolith::listCalendars(), the driver names are prepended to the calendar ids. ~~Bug: 10767~~ 3 files changed, 7 insertions(+), 0 deletions(-) http://git.horde.org/horde-git/-/commit/385e6688d92ab6d531213370b7218982e2fa2255

2011-12-04 22:13:28	Michael Rubinsky	Comment #4	Reply to this comment
Eh, nevermind. It's the link from the embedded imple, not the link on the My Calendars page.

2011-12-04 22:11:20	Michael Rubinsky	Comment #3 State ⇒ Feedback	Reply to this comment
I'm not seeing this issue. Is this an internal calendar? The display link for me correctly passes the calendar id without the leading 'internal_'.

2011-11-30 17:49:01	Jan Schneider	Comment #2 State ⇒ Assigned Assigned to Michael Rubinsky Assigned to Jan Schneider	Reply to this comment
We need to implement support for all calendar drivers.

2011-11-15 13:58:40	michael (dot) groene (at) zel (dot) uni-hannover (dot) de	Comment #1 State ⇒ Unconfirmed Patch ⇒ No Milestone ⇒ Queue ⇒ Kronolith Summary ⇒ Imple-Calendar-Link wrong Type ⇒ Bug Priority ⇒ 1. Low	Reply to this comment
I just embedded an calendar using the provided imple-link in calendars pref. The cal is displayed fine, but the link to display the calendar within horde seems to be wrong: https://asdf.zel.uni-hannover.de/kronolith/?display_cal=internal__GtoL9cNCvpOwlmpzvgznUA doesn't work. Cutting the "internal_" makes it work: https://asdf.zel.uni-hannover.de/kronolith/?display_cal=_GtoL9cNCvpOwlmpzvgznUA