fix horde_url usage, get back the proper behaviour for the perms popup

diff --git a/turba/templates/addressbook_list.php b/turba/templates/addressbook_list.php
index 53c0da3..a980e6f 100644 (file)
--- 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>