From: Michael M Slusarz <slusarz@curecanti.org>
Date: Thu, 4 Mar 2010 06:43:08 +0000 (-0700)
Subject: Notification: Remove Horde_Mobile dependency
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=1f6ab08a970905c2deb36aae2f2dc5eeaae52723;p=horde.git

Notification: Remove Horde_Mobile dependency
---

diff --git a/framework/Notification/lib/Horde/Notification/Listener/Status.php b/framework/Notification/lib/Horde/Notification/Listener/Status.php
index 28274d4d3..e7b9bc104 100644
--- a/framework/Notification/lib/Horde/Notification/Listener/Status.php
+++ b/framework/Notification/lib/Horde/Notification/Listener/Status.php
@@ -27,10 +27,7 @@ class Horde_Notification_Listener_Status extends Horde_Notification_Listener
      * message stack.
      *
      * @param array $events   The list of events to handle.
-     * @param array $options  An array of options:
-     * <pre>
-     * 'mobile' - (Horde_Mobile) The mobile object to send status lines to.
-     * </pre>
+     * @param array $options  An array of options. Not used.
      */
     public function notify($events, $options = array())
     {
@@ -38,13 +35,6 @@ class Horde_Notification_Listener_Status extends Horde_Notification_Listener
             return;
         }
 
-        if (!empty($options['mobile']) && class_exists('Horde_Mobile_Text')) {
-            foreach ($events as $event) {
-                $options['mobile']->add(new Horde_Mobile_Text(strip_tags($event)));
-            }
-            return;
-        }
-
         echo '<ul class="notices">';
 
         foreach ($events as $event) {
diff --git a/framework/Notification/package.xml b/framework/Notification/package.xml
index 8f2bed062..e3cafaf7b 100644
--- a/framework/Notification/package.xml
+++ b/framework/Notification/package.xml
@@ -113,10 +113,6 @@ http://pear.php.net/dtd/package-2.0.xsd">
     <channel>pear.horde.org</channel>
    </package>
    <package>
-    <name>Mobile</name>
-    <channel>pear.horde.org</channel>
-   </package>
-   <package>
     <name>Nls</name>
     <channel>pear.horde.org</channel>
    </package>
diff --git a/framework/Notification/test/Horde/Notification/Class/Notification/Listener/StatusTest.php b/framework/Notification/test/Horde/Notification/Class/Notification/Listener/StatusTest.php
index 71576b92d..6a2177621 100644
--- a/framework/Notification/test/Horde/Notification/Class/Notification/Listener/StatusTest.php
+++ b/framework/Notification/test/Horde/Notification/Class/Notification/Listener/StatusTest.php
@@ -60,19 +60,4 @@ class Horde_Notification_Class_Notification_Listener_StatusTest extends PHPUnit_
         $listener->notify($messages);
     }
 
-    public function testMethodNotifyHasMobileOutputIfTheMobileOptionWasSet()
-    {
-        if (!class_exists('Horde_Mobile')) {
-            $this->markTestSkipped('The Horde_Mobile package is not installed!');
-        }
-        $mobile = $this->getMock('Horde_Mobile', array(), array(), '', false, false);
-        $mobile->expects($this->once())
-            ->method('add')
-            ->with($this->isInstanceOf('Horde_Mobile_Text'));
-        $listener = new Horde_Notification_Listener_Status();
-        $event = new Horde_Notification_Event('<b>test</b>');
-        $messages = array($event);
-        $listener->notify($messages, array('mobile' => $mobile));
-    }
-
 }