CS fixes + more comments

pull/18/head
Aleksander Machniak 13 years ago
parent 5354c5dac7
commit 3e386efeee

@ -83,13 +83,12 @@ class rcube_db
return new $class($db_dsnw, $db_dsnr, $pconn); return new $class($db_dsnw, $db_dsnr, $pconn);
} }
/** /**
* Object constructor * Object constructor
* *
* @param string $db_dsnw DSN for read/write operations * @param string $db_dsnw DSN for read/write operations
* @param string $db_dsnr Optional DSN for read only operations * @param string $db_dsnr Optional DSN for read only operations
* @param bool $pconn Enables persistent connections * @param bool $pconn Enables persistent connections
*/ */
public function __construct($db_dsnw, $db_dsnr = '', $pconn = false) public function __construct($db_dsnw, $db_dsnr = '', $pconn = false)
{ {
@ -108,7 +107,9 @@ class rcube_db
$this->init(); $this->init();
} }
/**
* Initialization of the object with driver specific code
*/
protected function init() protected function init()
{ {
// To be used by driver classes // To be used by driver classes
@ -159,15 +160,34 @@ class rcube_db
return $dbh; return $dbh;
} }
/**
* Driver-specific preparation of database connection
*
* @param array $dsn DSN for DB connections
*/
protected function conn_prepare($dsn) protected function conn_prepare($dsn)
{ {
} }
/**
* Driver-specific configuration of database connection
*
* @param array $dsn DSN for DB connections
* @param PDO $dbh Connection handler
*/
protected function conn_configure($dsn, $dbh) protected function conn_configure($dsn, $dbh)
{ {
} }
/**
* Driver-specific database character set setting
*
* @param string $charset Character set name
*/
protected function set_charset($charset)
{
$this->query("SET NAMES 'utf8'");
}
/** /**
* Connect to appropiate database depending on the operation * Connect to appropiate database depending on the operation
@ -215,7 +235,6 @@ class rcube_db
} }
} }
/** /**
* Activate/deactivate debug mode * Activate/deactivate debug mode
* *
@ -226,13 +245,6 @@ class rcube_db
$this->options['debug_mode'] = $dbg; $this->options['debug_mode'] = $dbg;
} }
protected function set_charset($charset)
{
$this->query("SET NAMES 'utf8'");
}
/** /**
* Getter for error state * Getter for error state
* *
@ -243,7 +255,6 @@ class rcube_db
return $this->db_error ? $this->db_error_msg : false; return $this->db_error ? $this->db_error_msg : false;
} }
/** /**
* Connection state checker * Connection state checker
* *
@ -254,7 +265,6 @@ class rcube_db
return !is_object($this->dbh) ? false : $this->db_connected; return !is_object($this->dbh) ? false : $this->db_connected;
} }
/** /**
* Is database replication configured? * Is database replication configured?
* This returns true if dsnw != dsnr * This returns true if dsnw != dsnr
@ -264,7 +274,6 @@ class rcube_db
return !empty($this->db_dsnr) && $this->db_dsnw != $this->db_dsnr; return !empty($this->db_dsnr) && $this->db_dsnw != $this->db_dsnr;
} }
/** /**
* Execute a SQL query * Execute a SQL query
* *
@ -286,7 +295,6 @@ class rcube_db
return $this->_query($query, 0, 0, $params); return $this->_query($query, 0, 0, $params);
} }
/** /**
* Execute a SQL query with limits * Execute a SQL query with limits
* *
@ -307,7 +315,6 @@ class rcube_db
return $this->_query($query, $offset, $numrows, $params); return $this->_query($query, $offset, $numrows, $params);
} }
/** /**
* Execute a SQL query with limits * Execute a SQL query with limits
* *
@ -369,7 +376,6 @@ class rcube_db
return $this->_add_result($query); return $this->_add_result($query);
} }
/** /**
* Get number of affected rows for the last query * Get number of affected rows for the last query
* *
@ -385,7 +391,6 @@ class rcube_db
return 0; return 0;
} }
/** /**
* Get last inserted record ID * Get last inserted record ID
* For Postgres databases, a sequence name is required * For Postgres databases, a sequence name is required
@ -410,7 +415,6 @@ class rcube_db
return $id; return $id;
} }
/** /**
* Get an associative array for one row * Get an associative array for one row
* If no query handle is specified, the last query will be taken as reference * If no query handle is specified, the last query will be taken as reference
@ -425,7 +429,6 @@ class rcube_db
return $this->_fetch_row($result, PDO::FETCH_ASSOC); return $this->_fetch_row($result, PDO::FETCH_ASSOC);
} }
/** /**
* Get an index array for one row * Get an index array for one row
* If no query handle is specified, the last query will be taken as reference * If no query handle is specified, the last query will be taken as reference
@ -440,7 +443,6 @@ class rcube_db
return $this->_fetch_row($result, PDO::FETCH_NUM); return $this->_fetch_row($result, PDO::FETCH_NUM);
} }
/** /**
* Get col values for a result row * Get col values for a result row
* *
@ -458,7 +460,6 @@ class rcube_db
return $result->fetch($mode); return $result->fetch($mode);
} }
/** /**
* Adds LIMIT,OFFSET clauses to the query * Adds LIMIT,OFFSET clauses to the query
* *
@ -476,7 +477,6 @@ class rcube_db
return $query; return $query;
} }
/** /**
* Returns list of tables in a database * Returns list of tables in a database
* *
@ -499,7 +499,6 @@ class rcube_db
return $this->tables; return $this->tables;
} }
/** /**
* Returns list of columns in database table * Returns list of columns in database table
* *
@ -519,7 +518,6 @@ class rcube_db
return array(); return array();
} }
/** /**
* Formats input so it can be safely used in a query * Formats input so it can be safely used in a query
* *
@ -552,7 +550,6 @@ class rcube_db
return 'NULL'; return 'NULL';
} }
/** /**
* Quotes a string so it can be safely used as a table or column name * Quotes a string so it can be safely used as a table or column name
* *
@ -567,7 +564,6 @@ class rcube_db
return $this->quote_identifier($str); return $this->quote_identifier($str);
} }
/** /**
* Quotes a string so it can be safely used as a table or column name * Quotes a string so it can be safely used as a table or column name
* *
@ -589,7 +585,6 @@ class rcube_db
return implode($name, '.'); return implode($name, '.');
} }
/** /**
* Return SQL function for current time and date * Return SQL function for current time and date
* *
@ -600,7 +595,6 @@ class rcube_db
return "now()"; return "now()";
} }
/** /**
* Return list of elements for use with SQL's IN clause * Return list of elements for use with SQL's IN clause
* *
@ -622,7 +616,6 @@ class rcube_db
return implode(',', $arr); return implode(',', $arr);
} }
/** /**
* Return SQL statement to convert a field value into a unix timestamp * Return SQL statement to convert a field value into a unix timestamp
* *
@ -639,7 +632,6 @@ class rcube_db
return "UNIX_TIMESTAMP($field)"; return "UNIX_TIMESTAMP($field)";
} }
/** /**
* Return SQL statement to convert from a unix timestamp * Return SQL statement to convert from a unix timestamp
* *
@ -652,7 +644,6 @@ class rcube_db
return date("'Y-m-d H:i:s'", $timestamp); return date("'Y-m-d H:i:s'", $timestamp);
} }
/** /**
* Return SQL statement for case insensitive LIKE * Return SQL statement for case insensitive LIKE
* *
@ -666,7 +657,6 @@ class rcube_db
return $this->quote_identifier($column).' LIKE '.$this->quote($value); return $this->quote_identifier($column).' LIKE '.$this->quote($value);
} }
/** /**
* Abstract SQL statement for value concatenation * Abstract SQL statement for value concatenation
* *
@ -682,7 +672,6 @@ class rcube_db
return '(' . join(' || ', $args) . ')'; return '(' . join(' || ', $args) . ')';
} }
/** /**
* Encodes non-UTF-8 characters in string/array/object (recursive) * Encodes non-UTF-8 characters in string/array/object (recursive)
* *
@ -708,7 +697,6 @@ class rcube_db
return utf8_encode($input); return utf8_encode($input);
} }
/** /**
* Decodes encoded UTF-8 string/object/array (recursive) * Decodes encoded UTF-8 string/object/array (recursive)
* *
@ -734,7 +722,6 @@ class rcube_db
return utf8_decode($input); return utf8_decode($input);
} }
/** /**
* Adds a query result and returns a handle ID * Adds a query result and returns a handle ID
* *
@ -751,7 +738,6 @@ class rcube_db
return $res_id; return $res_id;
} }
/** /**
* Resolves a given handle ID and returns the according query handle * Resolves a given handle ID and returns the according query handle
* If no ID is specified, the last resource handle will be returned * If no ID is specified, the last resource handle will be returned
@ -773,7 +759,6 @@ class rcube_db
return false; return false;
} }
/** /**
* Return correct name for a specific database table * Return correct name for a specific database table
* *
@ -795,7 +780,6 @@ class rcube_db
return $table; return $table;
} }
/** /**
* Return correct name for a specific database sequence * Return correct name for a specific database sequence
* (used for Postgres only) * (used for Postgres only)
@ -818,9 +802,12 @@ class rcube_db
return $sequence; return $sequence;
} }
/** /**
* MDB2 DSN string parser * MDB2 DSN string parser
*
* @param string $sequence Secuence name
*
* @return array DSN parameters
*/ */
public static function parse_dsn($dsn) public static function parse_dsn($dsn)
{ {
@ -939,7 +926,11 @@ class rcube_db
} }
/** /**
* Returns PDO DSN string from DSN array (parse_dsn() result) * Returns PDO DSN string from DSN array
*
* @param array $dsn DSN parameters
*
* @return string DSN string
*/ */
protected function dsn_string($dsn) protected function dsn_string($dsn)
{ {
@ -966,7 +957,11 @@ class rcube_db
} }
/** /**
* Returns PDO driver options array from DSN array (parse_dsn() result) * Returns driver-specific connection options
*
* @param array $dsn DSN parameters
*
* @return array Connection options
*/ */
protected function dsn_options($dsn) protected function dsn_options($dsn)
{ {

Loading…
Cancel
Save