More CS fixes and comments

Aleksander Machniak

2012-07-04 8c2375a07443231cad32bd4cbd1d9ffbd1aa5087

More CS fixes and comments

 program/include/rcube_db.php

@@ -463,6 +463,11 @@
    /**
     * Adds LIMIT,OFFSET clauses to the query
     *
     * @param string $query   SQL query
     * @param int    $limit   Number of rows
     * @param int    $offset  Offset
     *
     * @return string SQL query
     */
    protected function set_limit($query, $limit = 0, $offset = 0)
    {

 program/include/rcube_db_mssql.php

@@ -31,16 +31,22 @@
 */
class rcube_db_mssql extends rcube_db
{
    /**
     * Driver initialization
     */
    protected function init()
    {
        $this->options['identifier_start'] = '[';
        $this->options['identifier_end'] = ']';
    }

    /**
     * Character setting
     */
    protected function set_charset($charset)
    {
        // UTF-8 is default
    }


    /**
     * Return SQL function for current time and date
@@ -51,7 +57,6 @@
    {
        return "getdate()";
    }


    /**
     * Return SQL statement to convert a field value into a unix timestamp
@@ -69,7 +74,6 @@
        return "DATEDIFF(second, '19700101', $field) + DATEDIFF(second, GETDATE(), GETUTCDATE())";
    }


    /**
     * Abstract SQL statement for value concatenation
     *
@@ -86,28 +90,44 @@
        return '(' . join('+', $args) . ')';
    }


    /**
     * Adds TOP (LIMIT,OFFSET) clause to the query
     *
     * @param string $query   SQL query
     * @param int    $limit   Number of rows
     * @param int    $offset  Offset
     *
     * @return string SQL query
     */
    protected function set_limit($query, $limit = 0, $offset = 0)
    {
        // code from MDB2 package
        if ($limit > 0) {
            $fetch = $offset + $limit;
            return preg_replace('/^([\s(])*SELECT( DISTINCT)?(?!\s*TOP\s*\()/i',
                "\\1SELECT\\2 TOP $fetch", $query);
        $limit  = intval($limit);
        $offset = intval($offset);

        $orderby = stristr($query, 'ORDER BY');
        if ($orderby !== false) {
            $sort  = (stripos($orderby, ' desc') !== false) ? 'desc' : 'asc';
            $order = str_ireplace('ORDER BY', '', $orderby);
            $order = trim(preg_replace('/\bASC\b|\bDESC\b/i', '', $order));
        }

// @TODO: proper OFFSET handling i _fetch_row()
        $query = preg_replace('/^SELECT\s/i', 'SELECT TOP ' . ($limit + $offset) . ' ', $query);

        $query = 'SELECT * FROM (SELECT TOP ' . $limit . ' * FROM (' . $query . ') AS inner_tbl';
        if ($orderby !== false) {
            $query .= ' ORDER BY ' . $order . ' ';
            $query .= (stripos($sort, 'asc') !== false) ? 'DESC' : 'ASC';
        }
        $query .= ') AS outer_tbl';
        if ($orderby !== false) {
            $query .= ' ORDER BY ' . $order . ' ' . $sort;
        }

        return $query;
    }


    /**
     * Returns PDO DSN string from DSN array (parse_dsn() result)
     * Returns PDO DSN string from DSN array
     */
    protected function dsn_string($dsn)
    {

 program/include/rcube_db_mysql.php

@@ -31,13 +31,15 @@
 */
class rcube_db_mysql extends rcube_db
{
    /**
     * Driver initialization/configuration
     */
    protected function init()
    {
        // SQL identifiers quoting
        $this->options['identifier_start'] = '`';
        $this->options['identifier_end'] = '`';
    }


    /**
     * Abstract SQL statement for value concatenation
@@ -55,9 +57,8 @@
        return 'CONCAT(' . join(', ', $args) . ')';
    }


    /**
     * Returns PDO DSN string from DSN array (parse_dsn() result)
     * Returns PDO DSN string from DSN array
     */
    protected function dsn_string($dsn)
    {

 program/include/rcube_db_pgsql.php

@@ -31,7 +31,6 @@
 */
class rcube_db_pgsql extends rcube_db
{

    /**
     * Get last inserted record ID
     * For Postgres databases, a table name is required
@@ -55,7 +54,6 @@
        return $id;
    }


    /**
     * Return SQL statement to convert a field value into a unix timestamp
     *
@@ -71,7 +69,6 @@
    {
        return "EXTRACT (EPOCH FROM $field)";
    }


    /**
     * Return SQL statement for case insensitive LIKE

 program/include/rcube_db_sqlite.php

@@ -31,11 +31,16 @@
 */
class rcube_db_sqlite extends rcube_db
{

    /**
     * Database character set
     */
    protected function set_charset($charset)
    {
    }

    /**
     * Prepare connection
     */
    protected function conn_prepare($dsn)
    {
        // Create database file, required by PDO to exist on connection
@@ -44,6 +49,9 @@
        }
    }

    /**
     * Configure connection, create database if not exists
     */
    protected function conn_configure($dsn, $dbh)
    {
        // we emulate via callback some missing functions
@@ -74,7 +82,6 @@
        }
    }


    /**
     * Callback for sqlite: unix_timestamp()
     */
@@ -94,7 +101,6 @@
        return $ret;
    }


    /**
     * Callback for sqlite: now()
     */
@@ -102,7 +108,6 @@
    {
        return date("Y-m-d H:i:s");
    }


    /**
     * Returns list of tables in database
@@ -125,7 +130,6 @@

        return $this->tables;
    }


    /**
     * Returns list of columns in database table
@@ -161,7 +165,9 @@
        return $columns;
    }


    /**
     * Build DSN string for PDO constructor
     */
    protected function dsn_string($dsn)
    {
        return $dsn['phptype'] . ':' . $dsn['database'];

 program/include/rcube_db_sqlsrv.php

@@ -31,16 +31,22 @@
 */
class rcube_db_sqlsrv extends rcube_db
{
    /**
     * Driver initialization
     */
    protected function init()
    {
        $this->options['identifier_start'] = '[';
        $this->options['identifier_end'] = ']';
    }

    /**
     * Database character set setting
     */
    protected function set_charset($charset)
    {
        // UTF-8 is default
    }


    /**
     * Return SQL function for current time and date
@@ -51,7 +57,6 @@
    {
        return "getdate()";
    }


    /**
     * Return SQL statement to convert a field value into a unix timestamp
@@ -69,7 +74,6 @@
        return "DATEDIFF(second, '19700101', $field) + DATEDIFF(second, GETDATE(), GETUTCDATE())";
    }


    /**
     * Abstract SQL statement for value concatenation
     *
@@ -86,10 +90,14 @@
        return '(' . join('+', $args) . ')';
    }


    /**
     * Adds TOP (LIMIT,OFFSET) clause to the query
     *
     * @param string $query   SQL query
     * @param int    $limit   Number of rows
     * @param int    $offset  Offset
     *
     * @return string SQL query
     */
    protected function set_limit($query, $limit = 0, $offset = 0)
    {
@@ -119,7 +127,7 @@
    }

    /**
     * Returns PDO DSN string from DSN array (parse_dsn() result)
     * Returns PDO DSN string from DSN array
     */
    protected function dsn_string($dsn)
    {