From add9e5b7affba54732289d1208853ab705c0c891 Mon Sep 17 00:00:00 2001
From: David Goodwin <david@palepurple.co.uk>
Date: Wed, 19 Feb 2014 11:49:26 +0000
Subject: [PATCH] fix xmlrpc function call (non-static method being called
 statically); fix method docblock which is read by Zend_XmlRpc_Server etc

git-svn-id: https://svn.code.sf.net/p/postfixadmin/code/trunk@1643 a1433add-5e2c-0410-b055-b7f2511e0802
---
 xmlrpc.php | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/xmlrpc.php b/xmlrpc.php
index 0b4bc524..372bd76e 100644
--- a/xmlrpc.php
+++ b/xmlrpc.php
@@ -47,7 +47,7 @@ $server = new Zend_XmlRpc_Server();
 function login($username, $password) {
 
     $h = new MailboxHandler();
-    if($h::login($username, $password)) {
+    if($h->login($username, $password)) {
         session_regenerate_id();
         $_SESSION['authenticated'] = true;
         $_SESSION['sessid'] = array();
@@ -129,11 +129,11 @@ class VacationProxy {
     /**
      * @param string $subject
      * @param string $body
-     * @param string $reply_type
      * @param string $interval_time
      * @param string $activeFrom
      * @param string $activeUntil
      * @return boolean true on success.
+     * Whatiis @replyType?? for
      */
     public function setAway($subject, $body, $interval_time = 0, $activeFrom = '2000-01-01', $activeUntil = '2099-12-31') {
         $vh = new VacationHandler($_SESSION['sessid']['username']);
@@ -174,8 +174,12 @@ class AliasProxy {
             return false; # invalid parameter
         }
 
-        if (!$ah->set($values)) return false;
-        return $ah->store();
+        if (!$ah->set($values)) {
+            //error_log('ah->set failed' . print_r($values, true));
+            return false;
+        }
+        $store = $ah->store();
+        return $store;
     }
 
     /**