Fix setting the oofstate property.

This is used to indicate availability of the *feature* not the status of the message.
horde · Nov 24, 2017 · e8ecdb9 · e8ecdb9
1 parent 0d60315
commit e8ecdb9
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/lib/Horde/Core/ActiveSync/Driver.php b/lib/Horde/Core/ActiveSync/Driver.php
@@ -2453,13 +2453,16 @@ public function getSettings(array $settings, $device)
             switch ($key) {
             case 'oof':
                 if (!$vacation = $this->_connector->filters_getVacation()) {
-                    return array('oof' => array('status' => Horde_ActiveSync_Request_Settings::STATUS_UNAVAILABLE));
+                    $res['oof'] = array(
+                        'status' => Horde_ActiveSync_Request_Settings::STATUS_SUCCESS,
+                        'oofstate' =>  Horde_ActiveSync_Request_Settings::OOF_STATE_DISABLED,
+                        'oofmsgs' => array()
+                    );
+                    break;
                 }
                 $res['oof'] = array(
                     'status' => Horde_ActiveSync_Request_Settings::STATUS_SUCCESS,
-                    'oofstate' => ($vacation['disabled']
-                        ? Horde_ActiveSync_Request_Settings::OOF_STATE_DISABLED
-                        : Horde_ActiveSync_Request_Settings::OOF_STATE_ENABLED),
+                    'oofstate' => Horde_ActiveSync_Request_Settings::OOF_STATE_ENABLED,
                     'oofmsgs' => array()
                 );
                 // If we have start/end it's a timebased vacation in EAS-speak.