From 382f6efbefba038c54f66fa231a9e4741e6ecc5e Mon Sep 17 00:00:00 2001
From: Jan Schneider <jan@horde.org>
Date: Wed, 25 Aug 2010 12:35:53 +0200
Subject: [PATCH] Fix url escaping and popup code.

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

diff --git a/mnemo/templates/notepad_list.php b/mnemo/templates/notepad_list.php
index 2454f9f23..95d3d7017 100644
--- a/mnemo/templates/notepad_list.php
+++ b/mnemo/templates/notepad_list.php
@@ -26,7 +26,7 @@
    <td><?php echo htmlspecialchars($notepad->get('name')) ?></td>
    <td><a href="<?php echo Horde_Util::addParameter($edit_url_base, 'n', $notepad->getName()) ?>" title="<?php echo _("Edit") ?>"><?php echo $edit_img ?></a></td>
 <?php if (empty($conf['share']['no_sharing'])): ?>
-   <td><a onclick="return !popup(this.href);" href="<?php echo Horde_Util::addParameter($perms_url_base, 'share', $notepad->getName()) ?>" target="_blank" title="<?php echo _("Change Permissions") ?>"><?php echo $perms_img ?></a></td>
+   <td><?php echo $perms_url_base->add('share', $notepad->getName())->link(array('target' => '_blank', 'title' => _("Change Permissions"), 'onclick' => Horde::popupJs($perms_url_base, array('params' => array('share' => $notepad->getName()), 'urlencode' => true)) . 'return false;')) . $perms_img . '</a>' ?></td>
 <?php endif; ?>
    <td><a href="<?php echo Horde_Util::addParameter($delete_url_base, 'n', $notepad->getName()) ?>" title="<?php echo _("Delete") ?>"><?php echo $delete_img ?></a></td>
   </tr>
-- 
2.11.0