From 43899681e6260cacb2db7f88c4cedcdd0ad9fc7e Mon Sep 17 00:00:00 2001
From: "Michael J. Rubinsky" <mrubinsk@horde.org>
Date: Tue, 18 May 2010 13:19:03 -0400
Subject: [PATCH] fix horde_url usage, get back the proper behaviour for the
 perms popup

---
 turba/templates/addressbook_list.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/turba/templates/addressbook_list.php b/turba/templates/addressbook_list.php
index 53c0da3a3..a980e6f30 100644
--- a/turba/templates/addressbook_list.php
+++ b/turba/templates/addressbook_list.php
@@ -30,7 +30,7 @@
    <td><?php echo htmlspecialchars($addressbook->get('name')) ?></td>
    <td><a href="<?php echo Horde_Util::addParameter($edit_url_base, 'a', $addressbook->getName()) ?>" title="<?php echo _("Edit") ?>"><?php echo $edit_img ?></a></td>
 <?php if (empty($conf['share']['no_sharing'])): ?>
-   <td><a onclick="<?php echo Horde::popupJs($perms_url_base, array('params' => array('share' => $addressbook->getName()))) ?>return false;" href="<?php echo Horde_Util::addParameter($perms_url_base, 'share', $addressbook->getName()) ?>" target="_blank" title="<?php echo _("Change Permissions") ?>"><?php echo $perms_img ?></a></td>
+   <td><?php echo $perms_url_base->add('share', $addressbook->getName())->link(array('title' => _("Change Permissions"), 'target' => '_blank', 'onclick' => Horde::popupJs($perms_url_base, array('params' => array('share' => $addressbook->getName()), 'urlencode' => true)) . 'return false;')) . $perms_img . '</a>';?>
 <?php endif; ?>
    <td><a href="<?php echo Horde_Util::addParameter($delete_url_base, 'a', $addressbook->getName()) ?>" title="<?php echo _("Delete") ?>"><?php echo $delete_img ?></a></td>
   </tr>
-- 
2.11.0