From df513e2ad330d85e3f88efad89ec4345de13a100 Mon Sep 17 00:00:00 2001
From: Jan Schneider <jan@horde.org>
Date: Tue, 6 Jul 2010 16:26:38 +0200
Subject: [PATCH] Render alarm list through Horde_View.

---
 horde/admin/alarms.php                     | 37 ++++++++++++++++--------------
 horde/templates/admin/alarms/list.html.php | 17 ++++++++++++++
 2 files changed, 37 insertions(+), 17 deletions(-)
 create mode 100644 horde/templates/admin/alarms/list.html.php

diff --git a/horde/admin/alarms.php b/horde/admin/alarms.php
index fab67d5fb..a6f50e2e0 100644
--- a/horde/admin/alarms.php
+++ b/horde/admin/alarms.php
@@ -108,31 +108,34 @@ if ($id) {
     }
 }
 
+$view = new Horde_View(array('templatePath' => HORDE_TEMPLATES . '/admin/alarms'));
+new Horde_View_Helper_Text($view);
+
 try {
     $alarms = $horde_alarm->globalAlarms();
+    $url = Horde::url('alarms.php');
+    foreach ($alarms as &$alarm) {
+        $url->add('alarm', $alarm['id']);
+        $alarm['edit_link'] = $url->link()
+            . htmlspecialchars($alarm['title'])
+            . '</a>';
+        $alarm['delete_link'] = $url->copy()
+            ->add('delete', 1)
+            ->link(array('title' => sprintf(_("Delete \"%s\""), $alarm['title']),
+                         'onclick' => 'return confirm(\'' . addslashes(sprintf(_("Are you sure you want to delete '%s'?"), $alarm['title'])) . '\')'))
+            . Horde::img('delete.png')
+            . '</a>';
+    }
+    $view->alarms = $alarms;
 } catch (Horde_Alarm_Exception $e) {
-    $alarms = $e;
+    $view->alarms = array();
+    $view->error = sprintf(_("Listing alarms failed: %s"), $e->getMessage());
 }
 
 $title = _("Alarms");
 require HORDE_TEMPLATES . '/common-header.inc';
 require HORDE_TEMPLATES . '/admin/menu.inc';
-
-echo '<h1 class="header">' . _("Current Alarms");
-if ($alarms instanceof Exception) {
-    echo '</h1><p class="headerbox"><em>' . sprintf(_("Listing alarms failed: %s"), $alarms->getMessage()) . '</em></p>';
-} else {
-    echo ' (' . count($alarms) . ')</h1>';
-    echo '<ul class="headerbox linedRow">';
-    foreach ($alarms as $alarm_details) {
-        $url = Horde_Util::addParameter(Horde::url('alarms.php'), 'alarm', $alarm_details['id']);
-        echo '<li>' . Horde::link(Horde_Util::addParameter($url, 'delete', 1), sprintf(_("Delete \"%s\""), $alarm_details['title']), '', '', 'return confirm(\'' . addslashes(sprintf(_("Are you sure you want to delete '%s'?"), $alarm_details['title'])) . '\')') . Horde::img('delete.png') . '</a> ' . Horde::link($url) . htmlspecialchars($alarm_details['title']) . '</a></li>';
-    }
-    echo '</ul>';
-}
-
-echo '<br />';
-
+echo $view->render('list');
 $form->renderActive();
 
 require HORDE_TEMPLATES . '/common-footer.inc';
diff --git a/horde/templates/admin/alarms/list.html.php b/horde/templates/admin/alarms/list.html.php
new file mode 100644
index 000000000..302c7490b
--- /dev/null
+++ b/horde/templates/admin/alarms/list.html.php
@@ -0,0 +1,17 @@
+<h1 class="header">
+  <?php echo _("Current Alarms") ?>
+  (<?php echo count($this->alarms) ?>)
+</h1>
+<?php if ($this->error): ?>
+<p class="headerbox"><em><?php echo $this-h($this->error) ?></em></p>
+<?php else: ?>
+<ul class="headerbox linedRow">
+  <?php foreach ($this->alarms as $alarm): ?>
+  <li>
+    <?php echo $alarm['delete_link'] ?>
+    <?php echo $alarm['edit_link'] ?>
+  </li>
+  <?php endforeach; ?>
+</ul>
+<?php endif; ?>
+<br />
-- 
2.11.0