SF.net SVN: postfixadmin:[1484] trunk

[<chr...@us...>]

Revision: 1484
          http://sourceforge.net/p/postfixadmin/code/1484
Author:   christian_boltz
Date:     2013-06-23 15:07:17 +0000 (Sun, 23 Jun 2013)
Log Message:
-----------
MailboxHandler.php:
- make login() a non-static function
- login: use $this->db_table and $this->id_field instead of hardcoded names

users/login.php, xmlrpc.php:
- adopt to now non-static MailboxHandler->login()

Modified Paths:
--------------
    trunk/model/MailboxHandler.php
    trunk/users/login.php
    trunk/xmlrpc.php

Modified: trunk/model/MailboxHandler.php
===================================================================
--- trunk/model/MailboxHandler.php	2013-06-23 15:04:05 UTC (rev 1483)
+++ trunk/model/MailboxHandler.php	2013-06-23 15:07:17 UTC (rev 1484)
@@ -442,12 +442,12 @@
      * @param string $password
      * @return boolean true on successful login (i.e. password matches etc)
      */
-    public static function login($username, $password) {
+    public function login($username, $password) {
         $username = escape_string($username);
 
-        $table_mailbox = table_by_key('mailbox');
+        $table = table_by_key($this->db_table);
         $active = db_get_boolean(True);
-        $query = "SELECT password FROM $table_mailbox WHERE username='$username' AND active='$active'";
+        $query = "SELECT password FROM $table WHERE " . $this->id_field . "='$username' AND active='$active'";
 
         $result = db_query ($query);
         if ($result['rows'] == 1) {

Modified: trunk/users/login.php
===================================================================
--- trunk/users/login.php	2013-06-23 15:04:05 UTC (rev 1483)
+++ trunk/users/login.php	2013-06-23 15:07:17 UTC (rev 1484)
@@ -43,7 +43,8 @@
    $fUsername = escape_string ($_POST['fUsername']);
    $fPassword = escape_string ($_POST['fPassword']);
 
-   if(MailboxHandler::login($_POST['fUsername'], $_POST['fPassword'])) {
+   $h = new MailboxHandler();
+   if($h->login($_POST['fUsername'], $_POST['fPassword'])) {
       session_regenerate_id();
       $_SESSION['sessid'] = array();
       $_SESSION['sessid']['roles'] = array();

Modified: trunk/xmlrpc.php
===================================================================
--- trunk/xmlrpc.php	2013-06-23 15:04:05 UTC (rev 1483)
+++ trunk/xmlrpc.php	2013-06-23 15:07:17 UTC (rev 1484)
@@ -45,7 +45,9 @@
  * @return boolean true on success, else false.
  */
 function login($username, $password) {
-    if(MailboxHandler::login($username, $password)) {
+
+    $h = new MailboxHandler();
+    if($h::login($username, $password)) {
         session_regenerate_id();
         $_SESSION['authenticated'] = true;
         $_SESSION['username'] = $username;

This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.