These are programming errors, throw invalidArgumentException here.

diff --git a/framework/ActiveSync/lib/Horde/ActiveSync/Driver/Base.php b/framework/ActiveSync/lib/Horde/ActiveSync/Driver/Base.php
index 8293fd4..31083e9 100644 (file)
--- a/framework/ActiveSync/lib/Horde/ActiveSync/Driver/Base.php
+++ b/framework/ActiveSync/lib/Horde/ActiveSync/Driver/Base.php
@@ -129,7 +129,7 @@ abstract class Horde_ActiveSync_Driver_Base
         if (empty($params['state_basic']) ||
             !($params['state_basic'] instanceof Horde_ActiveSync_State_Base)) {
 
-            throw new Horde_ActiveSync_Exception('Missing required state object');
+            throw new InvalidArgumentException('Missing required state object');
         }
 
         /* Create a stub if we don't have a useable logger. */

diff --git a/framework/ActiveSync/lib/Horde/ActiveSync/Driver/Horde.php b/framework/ActiveSync/lib/Horde/ActiveSync/Driver/Horde.php
index 3422086..5aab09e 100644 (file)
--- a/framework/ActiveSync/lib/Horde/ActiveSync/Driver/Horde.php
+++ b/framework/ActiveSync/lib/Horde/ActiveSync/Driver/Horde.php
@@ -51,7 +51,7 @@ class Horde_ActiveSync_Driver_Horde extends Horde_ActiveSync_Driver_Base
     {
         parent::__construct($params);
         if (empty($this->_params['connector'])) {
-            throw new Horde_ActiveSync_Exception('Missing required connector object.');
+            throw new InvalidArgumentException('Missing required connector object.');
         }
         $this->_connector = $params['connector'];
     }

diff --git a/framework/ActiveSync/lib/Horde/ActiveSync/Driver/Horde/Connector/Registry.php b/framework/ActiveSync/lib/Horde/ActiveSync/Driver/Horde/Connector/Registry.php
index 0de7599..327b285 100644 (file)
--- a/framework/ActiveSync/lib/Horde/ActiveSync/Driver/Horde/Connector/Registry.php
+++ b/framework/ActiveSync/lib/Horde/ActiveSync/Driver/Horde/Connector/Registry.php
@@ -33,7 +33,7 @@ class Horde_ActiveSync_Driver_Horde_Connector_Registry
     public function __construct($params = array())
     {
         if (empty($params['registry'])) {
-            throw new Horde_ActiveSync_Exception('Missing required Horde_Registry object.');
+            throw new InvalidArgumentException('Missing required Horde_Registry object.');
         }
 
         $this->_registry = $params['registry'];