Tickets :: [#4951] new_bookmark bad url in ./templates/browse.php

Summary	new_bookmark bad url in ./templates/browse.php
Queue	Trean
Type	Bug
State	Resolved
Priority	1. Low
Owners	chuck (at) horde (dot) org
Requester	pehrle (at) ac-reims (dot) fr
Created	01/30/2007 (6862 days ago)
Due	01/30/2007 (6862 days ago)
Updated	02/04/2007 (6857 days ago)
Assigned	01/30/2007 (6862 days ago)
Resolved	02/04/2007 (6857 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

02/04/2007 11:39:11 PM	Chuck Hagenbuch	State ⇒ Resolved

01/30/2007 02:48:23 PM	Chuck Hagenbuch	Comment #2 Assigned to Chuck Hagenbuch State ⇒ Assigned	Reply to this comment
I'm just about done with a revamp of trean's UI that gets rid of a lot of this javascript period.

01/30/2007 02:35:34 PM	pehrle (at) ac-reims (dot) fr	Comment #1 State ⇒ Unconfirmed Priority ⇒ 1. Low Type ⇒ Bug Summary ⇒ new_bookmark bad url in ./templates/browse.php Due ⇒ 01/30/2007 Queue ⇒ Trean	Reply to this comment
When a user click on new_bookmark javascript link generated by ./templates/browse.php the url is : if (action == "new_bookmark" \|\| (document.bookmarks != null && document.bookmarks.other_actions.value == "new_bookmark")) { window.location = "/horde3/trean/add.php?f=157&Horde=43a6a8c15196ce0440a6ac199b9eb4df"; but ..&Hord... gives login horde page (see http://bugs.horde.org/ticket/?id=4948) I try to modify ./templates/browse.php to - window.location = "<?php echo Horde::applicationUrl(Util::addParameter('add.php', 'f', Util::nonInputVar('folderId')), true); ?>"; + window.location = "<?php echo Horde::applicationUrl(Util::addParameter('add.php', 'f', Util::nonInputVar('folderId'))); ?>"; and it works fine.