From d04eb76abb48e3c72271500b15c85ad057360817 Mon Sep 17 00:00:00 2001 From: David Grudl Date: Tue, 2 Jul 2013 18:42:55 +0200 Subject: [PATCH] typos & whitespace --- dibi/bridges/Nette/DibiNette20Extension.php | 1 - dibi/bridges/Nette/DibiNette21Extension.php | 1 - dibi/bridges/Nette/DibiNettePanel.php | 8 - dibi/dibi.php | 55 +-- dibi/drivers/DibiFirebirdDriver.php | 94 ++--- dibi/drivers/DibiMsSql2005Driver.php | 64 +--- dibi/drivers/DibiMsSql2005Reflector.php | 5 - dibi/drivers/DibiMsSqlDriver.php | 60 +-- dibi/drivers/DibiMsSqlReflector.php | 6 - dibi/drivers/DibiMySqlDriver.php | 103 ++--- dibi/drivers/DibiMySqlReflector.php | 5 - dibi/drivers/DibiMySqliDriver.php | 89 ++--- dibi/drivers/DibiOdbcDriver.php | 80 ++-- dibi/drivers/DibiOracleDriver.php | 69 +--- dibi/drivers/DibiPdoDriver.php | 185 ++++----- dibi/drivers/DibiPostgreDriver.php | 110 ++---- dibi/drivers/DibiSqlite3Driver.php | 68 +--- dibi/drivers/DibiSqliteDriver.php | 65 +--- dibi/drivers/DibiSqliteReflector.php | 5 - dibi/libs/DibiConnection.php | 57 +-- dibi/libs/DibiDataSource.php | 31 +- dibi/libs/DibiDatabaseInfo.php | 60 --- dibi/libs/DibiDateTime.php | 8 - dibi/libs/DibiEvent.php | 3 - dibi/libs/DibiException.php | 13 - dibi/libs/DibiFileLogger.php | 7 +- dibi/libs/DibiFirePhpLogger.php | 4 - dibi/libs/DibiFluent.php | 41 +- dibi/libs/DibiHashMap.php | 5 - dibi/libs/DibiLiteral.php | 2 - dibi/libs/DibiObject.php | 21 +- dibi/libs/DibiResult.php | 68 ++-- dibi/libs/DibiResultIterator.php | 7 - dibi/libs/DibiRow.php | 17 +- dibi/libs/DibiTranslator.php | 405 ++++++++++---------- dibi/libs/interfaces.php | 15 +- examples/Nette/Debugger.php | 13 - examples/Nette/license.txt | 2 - examples/connecting-to-databases.php | 15 - examples/database-reflection.php | 3 - examples/dumping-sql-and-result-set.php | 1 - examples/nette-debug-and-exceptions.php | 1 - examples/query-language-and-conditions.php | 4 - examples/query-language-basic-examples.php | 6 - examples/result-set-data-types.php | 1 - examples/using-datetime.php | 2 - examples/using-extension-methods.php | 1 - examples/using-fluent-syntax.php | 6 - examples/using-limit-and-offset.php | 2 - examples/using-logger.php | 1 - examples/using-profiler.php | 2 +- examples/using-substitutions.php | 8 - license.md | 2 - tests/DibiFluent.cloning.phpt | 3 - 54 files changed, 612 insertions(+), 1298 deletions(-) diff --git a/dibi/bridges/Nette/DibiNette20Extension.php b/dibi/bridges/Nette/DibiNette20Extension.php index 1045571c..ae657b3b 100644 --- a/dibi/bridges/Nette/DibiNette20Extension.php +++ b/dibi/bridges/Nette/DibiNette20Extension.php @@ -10,7 +10,6 @@ */ - /** * Dibi extension for Nette Framework 2.0. Creates 'connection' service. * diff --git a/dibi/bridges/Nette/DibiNette21Extension.php b/dibi/bridges/Nette/DibiNette21Extension.php index f471582e..a109586c 100644 --- a/dibi/bridges/Nette/DibiNette21Extension.php +++ b/dibi/bridges/Nette/DibiNette21Extension.php @@ -10,7 +10,6 @@ */ - /** * Dibi extension for Nette Framework 2.1. Creates 'connection' service. * diff --git a/dibi/bridges/Nette/DibiNettePanel.php b/dibi/bridges/Nette/DibiNettePanel.php index 3a030a15..93ca65de 100644 --- a/dibi/bridges/Nette/DibiNettePanel.php +++ b/dibi/bridges/Nette/DibiNettePanel.php @@ -10,13 +10,11 @@ */ - if (interface_exists('Nette\Diagnostics\IBarPanel')) { class_alias('Nette\Diagnostics\IBarPanel', 'IBarPanel'); } - /** * Dibi panel for Nette\Diagnostics. * @@ -38,7 +36,6 @@ class DibiNettePanel extends DibiObject implements IBarPanel private $events = array(); - public function __construct($explain = TRUE, $filter = NULL) { $this->filter = $filter ? (int) $filter : DibiEvent::QUERY; @@ -46,7 +43,6 @@ class DibiNettePanel extends DibiObject implements IBarPanel } - public function register(DibiConnection $connection) { if (is_callable('Nette\Diagnostics\Debugger::enable') && !class_exists('NDebugger')) { @@ -70,7 +66,6 @@ class DibiNettePanel extends DibiObject implements IBarPanel } - /** * After event notification. * @return void @@ -84,7 +79,6 @@ class DibiNettePanel extends DibiObject implements IBarPanel } - /** * Returns blue-screen custom tab. * @return mixed @@ -100,7 +94,6 @@ class DibiNettePanel extends DibiObject implements IBarPanel } - /** * Returns HTML code for custom tab. (Nette\Diagnostics\IBarPanel) * @return mixed @@ -118,7 +111,6 @@ class DibiNettePanel extends DibiObject implements IBarPanel } - /** * Returns HTML code for custom panel. (Nette\Diagnostics\IBarPanel) * @return mixed diff --git a/dibi/dibi.php b/dibi/dibi.php index c33cc038..9659f0c7 100644 --- a/dibi/dibi.php +++ b/dibi/dibi.php @@ -20,7 +20,6 @@ if (version_compare(PHP_VERSION, '5.2.0', '<')) { @set_magic_quotes_runtime(FALSE); // intentionally @ - require_once dirname(__FILE__) . '/libs/interfaces.php'; require_once dirname(__FILE__) . '/libs/DibiDateTime.php'; require_once dirname(__FILE__) . '/libs/DibiObject.php'; @@ -43,9 +42,6 @@ if (interface_exists('Nette\Diagnostics\IBarPanel') || interface_exists('IBarPan } - - - /** * Interface for database drivers. * @@ -112,7 +108,6 @@ class dibi public static $defaultDriver = 'mysql'; - /** * Static class - cannot be instantiated. */ @@ -122,11 +117,9 @@ class dibi } - /********************* connections handling ****************d*g**/ - /** * Creates a new DibiConnection object and connects it to specified database. * @param mixed connection parameters @@ -140,7 +133,6 @@ class dibi } - /** * Disconnects from database (doesn't destroy DibiConnection object). * @return void @@ -151,7 +143,6 @@ class dibi } - /** * Returns TRUE when connection was established. * @return bool @@ -162,7 +153,6 @@ class dibi } - /** * Retrieve active connection. * @param string connection registy name @@ -187,7 +177,6 @@ class dibi } - /** * Sets connection. * @param DibiConnection @@ -199,7 +188,6 @@ class dibi } - /** * Change active connection. * @param string connection registy name @@ -212,11 +200,9 @@ class dibi } - /********************* monostate for active connection ****************d*g**/ - /** * Generates and executes SQL query - Monostate for DibiConnection::query(). * @param array|mixed one or more arguments @@ -230,7 +216,6 @@ class dibi } - /** * Executes the SQL query - Monostate for DibiConnection::nativeQuery(). * @param string SQL statement. @@ -242,7 +227,6 @@ class dibi } - /** * Generates and prints SQL query - Monostate for DibiConnection::test(). * @param array|mixed one or more arguments @@ -255,7 +239,6 @@ class dibi } - /** * Generates and returns SQL query as DibiDataSource - Monostate for DibiConnection::test(). * @param array|mixed one or more arguments @@ -268,7 +251,6 @@ class dibi } - /** * Executes SQL query and fetch result - Monostate for DibiConnection::query() & fetch(). * @param array|mixed one or more arguments @@ -282,7 +264,6 @@ class dibi } - /** * Executes SQL query and fetch results - Monostate for DibiConnection::query() & fetchAll(). * @param array|mixed one or more arguments @@ -296,7 +277,6 @@ class dibi } - /** * Executes SQL query and fetch first column - Monostate for DibiConnection::query() & fetchSingle(). * @param array|mixed one or more arguments @@ -310,7 +290,6 @@ class dibi } - /** * Executes SQL query and fetch pairs - Monostate for DibiConnection::query() & fetchPairs(). * @param array|mixed one or more arguments @@ -324,7 +303,6 @@ class dibi } - /** * Gets the number of affected rows. * Monostate for DibiConnection::getAffectedRows() @@ -337,7 +315,6 @@ class dibi } - /** * Gets the number of affected rows. Alias for getAffectedRows(). * @return int number of rows @@ -349,7 +326,6 @@ class dibi } - /** * Retrieves the ID generated for an AUTO_INCREMENT column by the previous INSERT query. * Monostate for DibiConnection::getInsertId() @@ -363,7 +339,6 @@ class dibi } - /** * Retrieves the ID generated for an AUTO_INCREMENT column. Alias for getInsertId(). * @param string optional sequence name @@ -376,7 +351,6 @@ class dibi } - /** * Begins a transaction - Monostate for DibiConnection::begin(). * @param string optional savepoint name @@ -389,7 +363,6 @@ class dibi } - /** * Commits statements in a transaction - Monostate for DibiConnection::commit($savepoint = NULL). * @param string optional savepoint name @@ -402,7 +375,6 @@ class dibi } - /** * Rollback changes in a transaction - Monostate for DibiConnection::rollback(). * @param string optional savepoint name @@ -415,7 +387,6 @@ class dibi } - /** * Gets a information about the current database - Monostate for DibiConnection::getDatabaseInfo(). * @return DibiDatabaseInfo @@ -426,7 +397,6 @@ class dibi } - /** * Import SQL dump from file - extreme fast! * @param string filename @@ -438,24 +408,21 @@ class dibi } - /** * Replacement for majority of dibi::methods() in future. */ public static function __callStatic($name, $args) { //if ($name = 'select', 'update', ...') { - // return self::command()->$name($args); + // return self::command()->$name($args); //} return call_user_func_array(array(self::getConnection(), $name), $args); } - /********************* fluent SQL builders ****************d*g**/ - /** * @return DibiFluent */ @@ -465,7 +432,6 @@ class dibi } - /** * @param string column name * @return DibiFluent @@ -477,7 +443,6 @@ class dibi } - /** * @param string table * @param array @@ -489,7 +454,6 @@ class dibi } - /** * @param string table * @param array @@ -501,7 +465,6 @@ class dibi } - /** * @param string table * @return DibiFluent @@ -512,11 +475,9 @@ class dibi } - /********************* data types ****************d*g**/ - /** * @return DibiDateTime */ @@ -527,7 +488,6 @@ class dibi } - /** * @deprecated */ @@ -538,11 +498,9 @@ class dibi } - /********************* substitutions ****************d*g**/ - /** * Returns substitution hashmap - Monostate for DibiConnection::getSubstitutes(). * @return DibiHashMap @@ -553,7 +511,6 @@ class dibi } - /** @deprecated */ public static function addSubst($expr, $subst) { @@ -562,7 +519,6 @@ class dibi } - /** @deprecated */ public static function removeSubst($expr) { @@ -578,7 +534,6 @@ class dibi } - /** @deprecated */ public static function setSubstFallback($callback) { @@ -587,11 +542,9 @@ class dibi } - /********************* misc tools ****************d*g**/ - /** * Prints out a syntax highlighted version of the SQL command or DibiResult. * @param string|DibiResult @@ -605,7 +558,9 @@ class dibi $sql->dump(); } else { - if ($sql === NULL) $sql = self::$sql; + if ($sql === NULL) { + $sql = self::$sql; + } static $keywords1 = 'SELECT|(?:ON\s+DUPLICATE\s+KEY)?UPDATE|INSERT(?:\s+INTO)?|REPLACE(?:\s+INTO)?|DELETE|CALL|UNION|FROM|WHERE|HAVING|GROUP\s+BY|ORDER\s+BY|LIMIT|OFFSET|SET|VALUES|LEFT\s+JOIN|INNER\s+JOIN|TRUNCATE'; static $keywords2 = 'ALL|DISTINCT|DISTINCTROW|IGNORE|AS|USING|ON|AND|OR|IN|IS|NOT|NULL|LIKE|RLIKE|REGEXP|TRUE|FALSE'; @@ -643,7 +598,6 @@ class dibi } - private static function highlightCallback($matches) { if (!empty($matches[1])) { // comment @@ -661,7 +615,6 @@ class dibi } - private static function cliHighlightCallback($matches) { if (!empty($matches[1])) { // comment diff --git a/dibi/drivers/DibiFirebirdDriver.php b/dibi/drivers/DibiFirebirdDriver.php index 69a18dae..9ddd0cc6 100644 --- a/dibi/drivers/DibiFirebirdDriver.php +++ b/dibi/drivers/DibiFirebirdDriver.php @@ -56,13 +56,12 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Connects to a database. * @return void * @throws DibiException */ - public function connect(array &$config) + public function connect(array & $config) { DibiConnection::alias($config, 'database', 'db'); @@ -71,11 +70,13 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } else { // default values - if (!isset($config['username'])) $config['username'] = ini_get('ibase.default_password'); - if (!isset($config['password'])) $config['password'] = ini_get('ibase.default_user'); - if (!isset($config['database'])) $config['database'] = ini_get('ibase.default_db'); - if (!isset($config['charset'])) $config['charset'] = ini_get('ibase.default_charset'); - if (!isset($config['buffers'])) $config['buffers'] = 0; + $config += array( + 'username' => ini_get('ibase.default_password'), + 'password' => ini_get('ibase.default_user'), + 'database' => ini_get('ibase.default_db'), + 'charset' => ini_get('ibase.default_charset'), + 'buffers' => 0, + ); DibiDriverException::tryError(); if (empty($config['persistent'])) { @@ -95,7 +96,6 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Disconnects from a database. * @return void @@ -106,7 +106,6 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Executes the SQL query. * @param string SQL statement. @@ -138,7 +137,6 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Gets the number of affected rows by the last INSERT, UPDATE or DELETE query. * @return int|FALSE number of rows or FALSE on error @@ -149,7 +147,6 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Retrieves the ID generated for an AUTO_INCREMENT column by the previous INSERT query. * @param string generator name @@ -161,7 +158,6 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Begins a transaction (if supported). * @param string optional savepoint name @@ -178,7 +174,6 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Commits statements in a transaction. * @param string optional savepoint name @@ -199,7 +194,6 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Rollback changes in a transaction. * @param string optional savepoint name @@ -220,7 +214,6 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Is in transaction? * @return bool @@ -231,7 +224,6 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Returns the connection resource. * @return resource @@ -242,7 +234,6 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Returns the connection reflector. * @return IDibiReflector @@ -253,7 +244,6 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Result set driver factory. * @param resource @@ -267,11 +257,9 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /********************* SQL ********************/ - /** * Encodes data for use in a SQL statement. * @param mixed value @@ -282,29 +270,28 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD public function escape($value, $type) { switch ($type) { - case dibi::TEXT: - case dibi::BINARY: - return "'" . str_replace("'", "''", $value) . "'"; + case dibi::TEXT: + case dibi::BINARY: + return "'" . str_replace("'", "''", $value) . "'"; - case dibi::IDENTIFIER: - return $value; + case dibi::IDENTIFIER: + return $value; - case dibi::BOOL: - return $value ? 1 : 0; + case dibi::BOOL: + return $value ? 1 : 0; - case dibi::DATE: - return $value instanceof DateTime ? $value->format("'Y-m-d'") : date("'Y-m-d'", $value); + case dibi::DATE: + return $value instanceof DateTime ? $value->format("'Y-m-d'") : date("'Y-m-d'", $value); - case dibi::DATETIME: - return $value instanceof DateTime ? $value->format("'Y-m-d H:i:s'") : date("'Y-m-d H:i:s'", $value); + case dibi::DATETIME: + return $value instanceof DateTime ? $value->format("'Y-m-d H:i:s'") : date("'Y-m-d H:i:s'", $value); - default: - throw new InvalidArgumentException('Unsupported type.'); + default: + throw new InvalidArgumentException('Unsupported type.'); } } - /** * Encodes string for use in a LIKE statement. * @param string @@ -317,7 +304,6 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Decodes data from result set. * @param string value @@ -334,28 +320,22 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Injects LIMIT/OFFSET to the SQL query. - * @param string &$sql The SQL query that will be modified. - * @param int $limit - * @param int $offset * @return void */ - public function applyLimit(&$sql, $limit, $offset) + public function applyLimit(& $sql, $limit, $offset) { - if ($limit < 0 && $offset < 1) return; - - // see http://scott.yang.id.au/2004/01/limit-in-select-statements-in-firebird/ - $sql = 'SELECT FIRST ' . (int) $limit . ($offset > 0 ? ' SKIP ' . (int) $offset : '') . ' * FROM (' . $sql . ')'; + if ($limit >= 0 && $offset > 0) { + // see http://scott.yang.id.au/2004/01/limit-in-select-statements-in-firebird/ + $sql = 'SELECT FIRST ' . (int) $limit . ($offset > 0 ? ' SKIP ' . (int) $offset : '') . ' * FROM (' . $sql . ')'; + } } - /********************* result set ********************/ - /** * Automatically frees the resources allocated for this result set. * @return void @@ -366,7 +346,6 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Returns the number of rows in a result set. * @return int @@ -377,7 +356,6 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Fetches the row at current position and moves the internal cursor to the next position. * @param bool TRUE for associative array, FALSE for numeric @@ -402,7 +380,6 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Moves cursor position without fetching row. * @param int the 0-based cursor pos to seek to @@ -415,7 +392,6 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Frees the resources allocated for this result set. * @return void @@ -427,7 +403,6 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Returns the result set resource. * @return mysqli_result @@ -439,7 +414,6 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Returns metadata for all columns in a result set. * @return array @@ -461,11 +435,9 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /********************* IDibiReflector ********************/ - /** * Returns list of tables. * @return array @@ -489,7 +461,6 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Returns metadata for all columns in a table. * @param string @@ -545,7 +516,6 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Returns metadata for all indexes in a table (the constraints are included). * @param string @@ -580,7 +550,6 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Returns metadata for all foreign keys in a table. * @param string @@ -611,7 +580,6 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Returns list of indices in given table (the constraints are not listed). * @param string @@ -634,7 +602,6 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Returns list of constraints in given table. * @param string @@ -659,7 +626,6 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Returns metadata for all triggers in a table or database. * (Only if user has permissions on ALTER TABLE, INSERT/UPDATE/DELETE record in table) @@ -709,7 +675,6 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Returns list of triggers for given table. * (Only if user has permissions on ALTER TABLE, INSERT/UPDATE/DELETE record in table) @@ -732,7 +697,6 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Returns metadata from stored procedures and their input and output parameters. * @param string @@ -786,7 +750,6 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Returns list of stored procedures. * @return array @@ -805,7 +768,6 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Returns list of generators. * @return array @@ -825,7 +787,6 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - /** * Returns list of user defined functions (UDF). * @return array @@ -847,8 +808,6 @@ class DibiFirebirdDriver extends DibiObject implements IDibiDriver, IDibiResultD } - - /** * Database procedure exception. * @@ -875,7 +834,6 @@ class DibiProcedureException extends DibiException } - /** * Gets the exception severity. * @return string diff --git a/dibi/drivers/DibiMsSql2005Driver.php b/dibi/drivers/DibiMsSql2005Driver.php index d502dee3..a6903bf5 100644 --- a/dibi/drivers/DibiMsSql2005Driver.php +++ b/dibi/drivers/DibiMsSql2005Driver.php @@ -44,7 +44,6 @@ class DibiMsSql2005Driver extends DibiObject implements IDibiDriver, IDibiResult private $affectedRows = FALSE; - /** * @throws DibiNotSupportedException */ @@ -56,13 +55,12 @@ class DibiMsSql2005Driver extends DibiObject implements IDibiDriver, IDibiResult } - /** * Connects to a database. * @return void * @throws DibiException */ - public function connect(array &$config) + public function connect(array & $config) { DibiConnection::alias($config, 'options|UID', 'username'); DibiConnection::alias($config, 'options|PWD', 'password'); @@ -74,7 +72,9 @@ class DibiMsSql2005Driver extends DibiObject implements IDibiDriver, IDibiResult } else { // Default values - if (!isset($config['options']['CharacterSet'])) $config['options']['CharacterSet'] = 'UTF-8'; + if (!isset($config['options']['CharacterSet'])) { + $config['options']['CharacterSet'] = 'UTF-8'; + } $this->connection = sqlsrv_connect($config['host'], (array) $config['options']); } @@ -86,7 +86,6 @@ class DibiMsSql2005Driver extends DibiObject implements IDibiDriver, IDibiResult } - /** * Disconnects from a database. * @return void @@ -97,7 +96,6 @@ class DibiMsSql2005Driver extends DibiObject implements IDibiDriver, IDibiResult } - /** * Executes the SQL query. * @param string SQL statement. @@ -120,7 +118,6 @@ class DibiMsSql2005Driver extends DibiObject implements IDibiDriver, IDibiResult } - /** * Gets the number of affected rows by the last INSERT, UPDATE or DELETE query. * @return int|FALSE number of rows or FALSE on error @@ -131,7 +128,6 @@ class DibiMsSql2005Driver extends DibiObject implements IDibiDriver, IDibiResult } - /** * Retrieves the ID generated for an AUTO_INCREMENT column by the previous INSERT query. * @return int|FALSE int on success or FALSE on failure @@ -147,7 +143,6 @@ class DibiMsSql2005Driver extends DibiObject implements IDibiDriver, IDibiResult } - /** * Begins a transaction (if supported). * @param string optional savepoint name @@ -160,7 +155,6 @@ class DibiMsSql2005Driver extends DibiObject implements IDibiDriver, IDibiResult } - /** * Commits statements in a transaction. * @param string optional savepoint name @@ -173,7 +167,6 @@ class DibiMsSql2005Driver extends DibiObject implements IDibiDriver, IDibiResult } - /** * Rollback changes in a transaction. * @param string optional savepoint name @@ -186,7 +179,6 @@ class DibiMsSql2005Driver extends DibiObject implements IDibiDriver, IDibiResult } - /** * Returns the connection resource. * @return mixed @@ -197,7 +189,6 @@ class DibiMsSql2005Driver extends DibiObject implements IDibiDriver, IDibiResult } - /** * Returns the connection reflector. * @return IDibiReflector @@ -208,7 +199,6 @@ class DibiMsSql2005Driver extends DibiObject implements IDibiDriver, IDibiResult } - /** * Result set driver factory. * @param resource @@ -222,11 +212,9 @@ class DibiMsSql2005Driver extends DibiObject implements IDibiDriver, IDibiResult } - /********************* SQL ****************d*g**/ - /** * Encodes data for use in a SQL statement. * @param mixed value @@ -237,30 +225,29 @@ class DibiMsSql2005Driver extends DibiObject implements IDibiDriver, IDibiResult public function escape($value, $type) { switch ($type) { - case dibi::TEXT: - case dibi::BINARY: - return "'" . str_replace("'", "''", $value) . "'"; + case dibi::TEXT: + case dibi::BINARY: + return "'" . str_replace("'", "''", $value) . "'"; - case dibi::IDENTIFIER: - // @see http://msdn.microsoft.com/en-us/library/ms176027.aspx - return '[' . str_replace(']', ']]', $value) . ']'; + case dibi::IDENTIFIER: + // @see http://msdn.microsoft.com/en-us/library/ms176027.aspx + return '[' . str_replace(']', ']]', $value) . ']'; - case dibi::BOOL: - return $value ? 1 : 0; + case dibi::BOOL: + return $value ? 1 : 0; - case dibi::DATE: - return $value instanceof DateTime ? $value->format("'Y-m-d'") : date("'Y-m-d'", $value); + case dibi::DATE: + return $value instanceof DateTime ? $value->format("'Y-m-d'") : date("'Y-m-d'", $value); - case dibi::DATETIME: - return $value instanceof DateTime ? $value->format("'Y-m-d H:i:s'") : date("'Y-m-d H:i:s'", $value); + case dibi::DATETIME: + return $value instanceof DateTime ? $value->format("'Y-m-d H:i:s'") : date("'Y-m-d H:i:s'", $value); - default: - throw new InvalidArgumentException('Unsupported type.'); + default: + throw new InvalidArgumentException('Unsupported type.'); } } - /** * Encodes string for use in a LIKE statement. * @param string @@ -274,7 +261,6 @@ class DibiMsSql2005Driver extends DibiObject implements IDibiDriver, IDibiResult } - /** * Decodes data from result set. * @param string value @@ -291,15 +277,11 @@ class DibiMsSql2005Driver extends DibiObject implements IDibiDriver, IDibiResult } - /** * Injects LIMIT/OFFSET to the SQL query. - * @param string &$sql The SQL query that will be modified. - * @param int $limit - * @param int $offset * @return void */ - public function applyLimit(&$sql, $limit, $offset) + public function applyLimit(& $sql, $limit, $offset) { // offset support is missing if ($limit >= 0) { @@ -312,11 +294,9 @@ class DibiMsSql2005Driver extends DibiObject implements IDibiDriver, IDibiResult } - /********************* result set ****************d*g**/ - /** * Automatically frees the resources allocated for this result set. * @return void @@ -327,7 +307,6 @@ class DibiMsSql2005Driver extends DibiObject implements IDibiDriver, IDibiResult } - /** * Returns the number of rows in a result set. * @return int @@ -338,7 +317,6 @@ class DibiMsSql2005Driver extends DibiObject implements IDibiDriver, IDibiResult } - /** * Fetches the row at current position and moves the internal cursor to the next position. * @param bool TRUE for associative array, FALSE for numeric @@ -350,7 +328,6 @@ class DibiMsSql2005Driver extends DibiObject implements IDibiDriver, IDibiResult } - /** * Moves cursor position without fetching row. * @param int the 0-based cursor pos to seek to @@ -362,7 +339,6 @@ class DibiMsSql2005Driver extends DibiObject implements IDibiDriver, IDibiResult } - /** * Frees the resources allocated for this result set. * @return void @@ -374,7 +350,6 @@ class DibiMsSql2005Driver extends DibiObject implements IDibiDriver, IDibiResult } - /** * Returns metadata for all columns in a result set. * @return array @@ -393,7 +368,6 @@ class DibiMsSql2005Driver extends DibiObject implements IDibiDriver, IDibiResult } - /** * Returns the result set resource. * @return mixed diff --git a/dibi/drivers/DibiMsSql2005Reflector.php b/dibi/drivers/DibiMsSql2005Reflector.php index 2a19cfc2..f7175199 100644 --- a/dibi/drivers/DibiMsSql2005Reflector.php +++ b/dibi/drivers/DibiMsSql2005Reflector.php @@ -23,14 +23,12 @@ class DibiMsSql2005Reflector extends DibiObject implements IDibiReflector private $driver; - public function __construct(IDibiDriver $driver) { $this->driver = $driver; } - /** * Returns list of tables. * @return array @@ -49,7 +47,6 @@ class DibiMsSql2005Reflector extends DibiObject implements IDibiReflector } - /** * Returns metadata for all columns in a table. * @param string @@ -102,7 +99,6 @@ class DibiMsSql2005Reflector extends DibiObject implements IDibiReflector } - /** * Returns metadata for all indexes in a table. * @param string @@ -128,7 +124,6 @@ class DibiMsSql2005Reflector extends DibiObject implements IDibiReflector } - /** * Returns metadata for all foreign keys in a table. * @param string diff --git a/dibi/drivers/DibiMsSqlDriver.php b/dibi/drivers/DibiMsSqlDriver.php index 55c40316..1a5c90b9 100644 --- a/dibi/drivers/DibiMsSqlDriver.php +++ b/dibi/drivers/DibiMsSqlDriver.php @@ -38,7 +38,6 @@ class DibiMsSqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv private $autoFree = TRUE; - /** * @throws DibiNotSupportedException */ @@ -50,13 +49,12 @@ class DibiMsSqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Connects to a database. * @return void * @throws DibiException */ - public function connect(array &$config) + public function connect(array & $config) { if (isset($config['resource'])) { $this->connection = $config['resource']; @@ -76,7 +74,6 @@ class DibiMsSqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Disconnects from a database. * @return void @@ -87,7 +84,6 @@ class DibiMsSqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Executes the SQL query. * @param string SQL statement. @@ -107,7 +103,6 @@ class DibiMsSqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Gets the number of affected rows by the last INSERT, UPDATE or DELETE query. * @return int|FALSE number of rows or FALSE on error @@ -118,7 +113,6 @@ class DibiMsSqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Retrieves the ID generated for an AUTO_INCREMENT column by the previous INSERT query. * @return int|FALSE int on success or FALSE on failure @@ -134,7 +128,6 @@ class DibiMsSqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Begins a transaction (if supported). * @param string optional savepoint name @@ -147,7 +140,6 @@ class DibiMsSqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Commits statements in a transaction. * @param string optional savepoint name @@ -160,7 +152,6 @@ class DibiMsSqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Rollback changes in a transaction. * @param string optional savepoint name @@ -173,7 +164,6 @@ class DibiMsSqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Returns the connection resource. * @return mixed @@ -184,7 +174,6 @@ class DibiMsSqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Returns the connection reflector. * @return IDibiReflector @@ -195,7 +184,6 @@ class DibiMsSqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Result set driver factory. * @param resource @@ -209,11 +197,9 @@ class DibiMsSqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /********************* SQL ****************d*g**/ - /** * Encodes data for use in a SQL statement. * @param mixed value @@ -224,30 +210,29 @@ class DibiMsSqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv public function escape($value, $type) { switch ($type) { - case dibi::TEXT: - case dibi::BINARY: - return "'" . str_replace("'", "''", $value) . "'"; + case dibi::TEXT: + case dibi::BINARY: + return "'" . str_replace("'", "''", $value) . "'"; - case dibi::IDENTIFIER: - // @see http://msdn.microsoft.com/en-us/library/ms176027.aspx - return '[' . str_replace(array('[', ']'), array('[[', ']]'), $value) . ']'; + case dibi::IDENTIFIER: + // @see http://msdn.microsoft.com/en-us/library/ms176027.aspx + return '[' . str_replace(array('[', ']'), array('[[', ']]'), $value) . ']'; - case dibi::BOOL: - return $value ? 1 : 0; + case dibi::BOOL: + return $value ? 1 : 0; - case dibi::DATE: - return $value instanceof DateTime ? $value->format("'Y-m-d'") : date("'Y-m-d'", $value); + case dibi::DATE: + return $value instanceof DateTime ? $value->format("'Y-m-d'") : date("'Y-m-d'", $value); - case dibi::DATETIME: - return $value instanceof DateTime ? $value->format("'Y-m-d H:i:s'") : date("'Y-m-d H:i:s'", $value); + case dibi::DATETIME: + return $value instanceof DateTime ? $value->format("'Y-m-d H:i:s'") : date("'Y-m-d H:i:s'", $value); - default: - throw new InvalidArgumentException('Unsupported type.'); + default: + throw new InvalidArgumentException('Unsupported type.'); } } - /** * Encodes string for use in a LIKE statement. * @param string @@ -261,7 +246,6 @@ class DibiMsSqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Decodes data from result set. * @param string value @@ -278,15 +262,11 @@ class DibiMsSqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Injects LIMIT/OFFSET to the SQL query. - * @param string &$sql The SQL query that will be modified. - * @param int $limit - * @param int $offset * @return void */ - public function applyLimit(&$sql, $limit, $offset) + public function applyLimit(& $sql, $limit, $offset) { // offset support is missing if ($limit >= 0) { @@ -299,11 +279,9 @@ class DibiMsSqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /********************* result set ****************d*g**/ - /** * Automatically frees the resources allocated for this result set. * @return void @@ -314,7 +292,6 @@ class DibiMsSqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Returns the number of rows in a result set. * @return int @@ -325,7 +302,6 @@ class DibiMsSqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Fetches the row at current position and moves the internal cursor to the next position. * @param bool TRUE for associative array, FALSE for numeric @@ -337,7 +313,6 @@ class DibiMsSqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Moves cursor position without fetching row. * @param int the 0-based cursor pos to seek to @@ -349,7 +324,6 @@ class DibiMsSqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Frees the resources allocated for this result set. * @return void @@ -361,7 +335,6 @@ class DibiMsSqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Returns metadata for all columns in a result set. * @return array @@ -383,7 +356,6 @@ class DibiMsSqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Returns the result set resource. * @return mixed diff --git a/dibi/drivers/DibiMsSqlReflector.php b/dibi/drivers/DibiMsSqlReflector.php index 4b853054..f9391525 100644 --- a/dibi/drivers/DibiMsSqlReflector.php +++ b/dibi/drivers/DibiMsSqlReflector.php @@ -25,14 +25,12 @@ class DibiMsSqlReflector extends DibiObject implements IDibiReflector private $driver; - public function __construct(IDibiDriver $driver) { $this->driver = $driver; } - /** * Returns list of tables. * @return array @@ -54,7 +52,6 @@ class DibiMsSqlReflector extends DibiObject implements IDibiReflector } - /** * Returns count of rows in a table * @param string @@ -87,7 +84,6 @@ class DibiMsSqlReflector extends DibiObject implements IDibiReflector } - /** * Returns metadata for all columns in a table. * @param string @@ -138,7 +134,6 @@ class DibiMsSqlReflector extends DibiObject implements IDibiReflector } - /** * Returns metadata for all indexes in a table. * @param string @@ -180,7 +175,6 @@ class DibiMsSqlReflector extends DibiObject implements IDibiReflector } - /** * Returns metadata for all foreign keys in a table. * @param string diff --git a/dibi/drivers/DibiMySqlDriver.php b/dibi/drivers/DibiMySqlDriver.php index 01b96706..1cd933ec 100644 --- a/dibi/drivers/DibiMySqlDriver.php +++ b/dibi/drivers/DibiMySqlDriver.php @@ -53,7 +53,6 @@ class DibiMySqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv private $buffered; - /** * @throws DibiNotSupportedException */ @@ -65,13 +64,12 @@ class DibiMySqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Connects to a database. * @return void * @throws DibiException */ - public function connect(array &$config) + public function connect(array & $config) { if (isset($config['resource'])) { $this->connection = $config['resource']; @@ -79,17 +77,21 @@ class DibiMySqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } else { // default values DibiConnection::alias($config, 'flags', 'options'); - if (!isset($config['charset'])) $config['charset'] = 'utf8'; - if (!isset($config['timezone'])) $config['timezone'] = date('P'); - if (!isset($config['username'])) $config['username'] = ini_get('mysql.default_user'); - if (!isset($config['password'])) $config['password'] = ini_get('mysql.default_password'); + $config += array( + 'charset' => 'utf8', + 'timezone' => date('P'), + 'username' => ini_get('mysql.default_user'), + 'password' => ini_get('mysql.default_password'), + ); if (!isset($config['host'])) { $host = ini_get('mysql.default_host'); if ($host) { $config['host'] = $host; $config['port'] = ini_get('mysql.default_port'); } else { - if (!isset($config['socket'])) $config['socket'] = ini_get('mysql.default_socket'); + if (!isset($config['socket'])) { + $config['socket'] = ini_get('mysql.default_socket'); + } $config['host'] = NULL; } } @@ -140,7 +142,6 @@ class DibiMySqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Disconnects from a database. * @return void @@ -151,7 +152,6 @@ class DibiMySqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Executes the SQL query. * @param string SQL statement. @@ -175,7 +175,6 @@ class DibiMySqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Retrieves information about the most recently executed query. * @return array @@ -184,7 +183,9 @@ class DibiMySqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv { $res = array(); preg_match_all('#(.+?): +(\d+) *#', mysql_info($this->connection), $matches, PREG_SET_ORDER); - if (preg_last_error()) throw new DibiPcreException; + if (preg_last_error()) { + throw new DibiPcreException; + } foreach ($matches as $m) { $res[$m[1]] = (int) $m[2]; @@ -193,7 +194,6 @@ class DibiMySqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Gets the number of affected rows by the last INSERT, UPDATE or DELETE query. * @return int|FALSE number of rows or FALSE on error @@ -204,7 +204,6 @@ class DibiMySqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Retrieves the ID generated for an AUTO_INCREMENT column by the previous INSERT query. * @return int|FALSE int on success or FALSE on failure @@ -215,7 +214,6 @@ class DibiMySqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Begins a transaction (if supported). * @param string optional savepoint name @@ -228,7 +226,6 @@ class DibiMySqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Commits statements in a transaction. * @param string optional savepoint name @@ -241,7 +238,6 @@ class DibiMySqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Rollback changes in a transaction. * @param string optional savepoint name @@ -254,7 +250,6 @@ class DibiMySqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Returns the connection resource. * @return mixed @@ -265,7 +260,6 @@ class DibiMySqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Returns the connection reflector. * @return IDibiReflector @@ -276,7 +270,6 @@ class DibiMySqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Result set driver factory. * @param resource @@ -290,11 +283,9 @@ class DibiMySqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /********************* SQL ****************d*g**/ - /** * Encodes data for use in a SQL statement. * @param mixed value @@ -305,38 +296,37 @@ class DibiMySqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv public function escape($value, $type) { switch ($type) { - case dibi::TEXT: - if (!is_resource($this->connection)) { - throw new DibiException('Lost connection to server.'); - } - return "'" . mysql_real_escape_string($value, $this->connection) . "'"; + case dibi::TEXT: + if (!is_resource($this->connection)) { + throw new DibiException('Lost connection to server.'); + } + return "'" . mysql_real_escape_string($value, $this->connection) . "'"; - case dibi::BINARY: - if (!is_resource($this->connection)) { - throw new DibiException('Lost connection to server.'); - } - return "_binary'" . mysql_real_escape_string($value, $this->connection) . "'"; + case dibi::BINARY: + if (!is_resource($this->connection)) { + throw new DibiException('Lost connection to server.'); + } + return "_binary'" . mysql_real_escape_string($value, $this->connection) . "'"; - case dibi::IDENTIFIER: - // @see http://dev.mysql.com/doc/refman/5.0/en/identifiers.html - return '`' . str_replace('`', '``', $value) . '`'; + case dibi::IDENTIFIER: + // @see http://dev.mysql.com/doc/refman/5.0/en/identifiers.html + return '`' . str_replace('`', '``', $value) . '`'; - case dibi::BOOL: - return $value ? 1 : 0; + case dibi::BOOL: + return $value ? 1 : 0; - case dibi::DATE: - return $value instanceof DateTime ? $value->format("'Y-m-d'") : date("'Y-m-d'", $value); + case dibi::DATE: + return $value instanceof DateTime ? $value->format("'Y-m-d'") : date("'Y-m-d'", $value); - case dibi::DATETIME: - return $value instanceof DateTime ? $value->format("'Y-m-d H:i:s'") : date("'Y-m-d H:i:s'", $value); + case dibi::DATETIME: + return $value instanceof DateTime ? $value->format("'Y-m-d H:i:s'") : date("'Y-m-d H:i:s'", $value); - default: - throw new InvalidArgumentException('Unsupported type.'); + default: + throw new InvalidArgumentException('Unsupported type.'); } } - /** * Encodes string for use in a LIKE statement. * @param string @@ -350,7 +340,6 @@ class DibiMySqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Decodes data from result set. * @param string value @@ -367,29 +356,23 @@ class DibiMySqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Injects LIMIT/OFFSET to the SQL query. - * @param string &$sql The SQL query that will be modified. - * @param int $limit - * @param int $offset * @return void */ - public function applyLimit(&$sql, $limit, $offset) + public function applyLimit(& $sql, $limit, $offset) { - if ($limit < 0 && $offset < 1) return; - - // see http://dev.mysql.com/doc/refman/5.0/en/select.html - $sql .= ' LIMIT ' . ($limit < 0 ? '18446744073709551615' : (int) $limit) - . ($offset > 0 ? ' OFFSET ' . (int) $offset : ''); + if ($limit >= 0 || $offset > 0) { + // see http://dev.mysql.com/doc/refman/5.0/en/select.html + $sql .= ' LIMIT ' . ($limit < 0 ? '18446744073709551615' : (int) $limit) + . ($offset > 0 ? ' OFFSET ' . (int) $offset : ''); + } } - /********************* result set ****************d*g**/ - /** * Automatically frees the resources allocated for this result set. * @return void @@ -400,7 +383,6 @@ class DibiMySqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Returns the number of rows in a result set. * @return int @@ -414,7 +396,6 @@ class DibiMySqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Fetches the row at current position and moves the internal cursor to the next position. * @param bool TRUE for associative array, FALSE for numeric @@ -426,7 +407,6 @@ class DibiMySqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Moves cursor position without fetching row. * @param int the 0-based cursor pos to seek to @@ -443,7 +423,6 @@ class DibiMySqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Frees the resources allocated for this result set. * @return void @@ -455,7 +434,6 @@ class DibiMySqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Returns metadata for all columns in a result set. * @return array @@ -478,7 +456,6 @@ class DibiMySqlDriver extends DibiObject implements IDibiDriver, IDibiResultDriv } - /** * Returns the result set resource. * @return mixed diff --git a/dibi/drivers/DibiMySqlReflector.php b/dibi/drivers/DibiMySqlReflector.php index c8353a3c..1788e5e1 100644 --- a/dibi/drivers/DibiMySqlReflector.php +++ b/dibi/drivers/DibiMySqlReflector.php @@ -23,14 +23,12 @@ class DibiMySqlReflector extends DibiObject implements IDibiReflector private $driver; - public function __construct(IDibiDriver $driver) { $this->driver = $driver; } - /** * Returns list of tables. * @return array @@ -54,7 +52,6 @@ class DibiMySqlReflector extends DibiObject implements IDibiReflector } - /** * Returns metadata for all columns in a table. * @param string @@ -88,7 +85,6 @@ class DibiMySqlReflector extends DibiObject implements IDibiReflector } - /** * Returns metadata for all indexes in a table. * @param string @@ -115,7 +111,6 @@ class DibiMySqlReflector extends DibiObject implements IDibiReflector } - /** * Returns metadata for all foreign keys in a table. * @param string diff --git a/dibi/drivers/DibiMySqliDriver.php b/dibi/drivers/DibiMySqliDriver.php index 2c9f4e6a..f6698649 100644 --- a/dibi/drivers/DibiMySqliDriver.php +++ b/dibi/drivers/DibiMySqliDriver.php @@ -54,7 +54,6 @@ class DibiMySqliDriver extends DibiObject implements IDibiDriver, IDibiResultDri private $buffered; - /** * @throws DibiNotSupportedException */ @@ -66,13 +65,12 @@ class DibiMySqliDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Connects to a database. * @return void * @throws DibiException */ - public function connect(array &$config) + public function connect(array & $config) { mysqli_report(MYSQLI_REPORT_OFF); if (isset($config['resource'])) { @@ -80,12 +78,14 @@ class DibiMySqliDriver extends DibiObject implements IDibiDriver, IDibiResultDri } else { // default values - if (!isset($config['charset'])) $config['charset'] = 'utf8'; - if (!isset($config['timezone'])) $config['timezone'] = date('P'); - if (!isset($config['username'])) $config['username'] = ini_get('mysqli.default_user'); - if (!isset($config['password'])) $config['password'] = ini_get('mysqli.default_pw'); - if (!isset($config['socket'])) $config['socket'] = ini_get('mysqli.default_socket'); - if (!isset($config['port'])) $config['port'] = NULL; + $config += array( + 'charset' => 'utf8', + 'timezone' => date('P'), + 'username' => ini_get('mysqli.default_user'), + 'password' => ini_get('mysqli.default_pw'), + 'socket' => ini_get('mysqli.default_socket'), + 'port' => NULL, + ); if (!isset($config['host'])) { $host = ini_get('mysqli.default_host'); if ($host) { @@ -141,7 +141,6 @@ class DibiMySqliDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Disconnects from a database. * @return void @@ -152,7 +151,6 @@ class DibiMySqliDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Executes the SQL query. * @param string SQL statement. @@ -172,7 +170,6 @@ class DibiMySqliDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Retrieves information about the most recently executed query. * @return array @@ -181,7 +178,9 @@ class DibiMySqliDriver extends DibiObject implements IDibiDriver, IDibiResultDri { $res = array(); preg_match_all('#(.+?): +(\d+) *#', mysqli_info($this->connection), $matches, PREG_SET_ORDER); - if (preg_last_error()) throw new DibiPcreException; + if (preg_last_error()) { + throw new DibiPcreException; + } foreach ($matches as $m) { $res[$m[1]] = (int) $m[2]; @@ -190,7 +189,6 @@ class DibiMySqliDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Gets the number of affected rows by the last INSERT, UPDATE or DELETE query. * @return int|FALSE number of rows or FALSE on error @@ -201,7 +199,6 @@ class DibiMySqliDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Retrieves the ID generated for an AUTO_INCREMENT column by the previous INSERT query. * @return int|FALSE int on success or FALSE on failure @@ -212,7 +209,6 @@ class DibiMySqliDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Begins a transaction (if supported). * @param string optional savepoint name @@ -225,7 +221,6 @@ class DibiMySqliDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Commits statements in a transaction. * @param string optional savepoint name @@ -238,7 +233,6 @@ class DibiMySqliDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Rollback changes in a transaction. * @param string optional savepoint name @@ -251,7 +245,6 @@ class DibiMySqliDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Returns the connection resource. * @return mysqli @@ -262,7 +255,6 @@ class DibiMySqliDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Returns the connection reflector. * @return IDibiReflector @@ -273,7 +265,6 @@ class DibiMySqliDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Result set driver factory. * @param mysqli_result @@ -287,11 +278,9 @@ class DibiMySqliDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /********************* SQL ****************d*g**/ - /** * Encodes data for use in a SQL statement. * @param mixed value @@ -302,31 +291,30 @@ class DibiMySqliDriver extends DibiObject implements IDibiDriver, IDibiResultDri public function escape($value, $type) { switch ($type) { - case dibi::TEXT: - return "'" . mysqli_real_escape_string($this->connection, $value) . "'"; + case dibi::TEXT: + return "'" . mysqli_real_escape_string($this->connection, $value) . "'"; - case dibi::BINARY: - return "_binary'" . mysqli_real_escape_string($this->connection, $value) . "'"; + case dibi::BINARY: + return "_binary'" . mysqli_real_escape_string($this->connection, $value) . "'"; - case dibi::IDENTIFIER: - return '`' . str_replace('`', '``', $value) . '`'; + case dibi::IDENTIFIER: + return '`' . str_replace('`', '``', $value) . '`'; - case dibi::BOOL: - return $value ? 1 : 0; + case dibi::BOOL: + return $value ? 1 : 0; - case dibi::DATE: - return $value instanceof DateTime ? $value->format("'Y-m-d'") : date("'Y-m-d'", $value); + case dibi::DATE: + return $value instanceof DateTime ? $value->format("'Y-m-d'") : date("'Y-m-d'", $value); - case dibi::DATETIME: - return $value instanceof DateTime ? $value->format("'Y-m-d H:i:s'") : date("'Y-m-d H:i:s'", $value); + case dibi::DATETIME: + return $value instanceof DateTime ? $value->format("'Y-m-d H:i:s'") : date("'Y-m-d H:i:s'", $value); - default: - throw new InvalidArgumentException('Unsupported type.'); + default: + throw new InvalidArgumentException('Unsupported type.'); } } - /** * Encodes string for use in a LIKE statement. * @param string @@ -340,7 +328,6 @@ class DibiMySqliDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Decodes data from result set. * @param string value @@ -357,29 +344,23 @@ class DibiMySqliDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Injects LIMIT/OFFSET to the SQL query. - * @param string &$sql The SQL query that will be modified. - * @param int $limit - * @param int $offset * @return void */ - public function applyLimit(&$sql, $limit, $offset) + public function applyLimit(& $sql, $limit, $offset) { - if ($limit < 0 && $offset < 1) return; - - // see http://dev.mysql.com/doc/refman/5.0/en/select.html - $sql .= ' LIMIT ' . ($limit < 0 ? '18446744073709551615' : (int) $limit) - . ($offset > 0 ? ' OFFSET ' . (int) $offset : ''); + if ($limit >= 0 || $offset > 0) { + // see http://dev.mysql.com/doc/refman/5.0/en/select.html + $sql .= ' LIMIT ' . ($limit < 0 ? '18446744073709551615' : (int) $limit) + . ($offset > 0 ? ' OFFSET ' . (int) $offset : ''); + } } - /********************* result set ****************d*g**/ - /** * Automatically frees the resources allocated for this result set. * @return void @@ -390,7 +371,6 @@ class DibiMySqliDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Returns the number of rows in a result set. * @return int @@ -404,7 +384,6 @@ class DibiMySqliDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Fetches the row at current position and moves the internal cursor to the next position. * @param bool TRUE for associative array, FALSE for numeric @@ -416,7 +395,6 @@ class DibiMySqliDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Moves cursor position without fetching row. * @param int the 0-based cursor pos to seek to @@ -432,7 +410,6 @@ class DibiMySqliDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Frees the resources allocated for this result set. * @return void @@ -444,7 +421,6 @@ class DibiMySqliDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Returns metadata for all columns in a result set. * @return array @@ -478,7 +454,6 @@ class DibiMySqliDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Returns the result set resource. * @return mysqli_result diff --git a/dibi/drivers/DibiOdbcDriver.php b/dibi/drivers/DibiOdbcDriver.php index fc6e40b6..70d7cb7c 100644 --- a/dibi/drivers/DibiOdbcDriver.php +++ b/dibi/drivers/DibiOdbcDriver.php @@ -42,7 +42,6 @@ class DibiOdbcDriver extends DibiObject implements IDibiDriver, IDibiResultDrive private $row = 0; - /** * @throws DibiNotSupportedException */ @@ -54,21 +53,22 @@ class DibiOdbcDriver extends DibiObject implements IDibiDriver, IDibiResultDrive } - /** * Connects to a database. * @return void * @throws DibiException */ - public function connect(array &$config) + public function connect(array & $config) { if (isset($config['resource'])) { $this->connection = $config['resource']; } else { // default values - if (!isset($config['username'])) $config['username'] = ini_get('odbc.default_user'); - if (!isset($config['password'])) $config['password'] = ini_get('odbc.default_pw'); - if (!isset($config['dsn'])) $config['dsn'] = ini_get('odbc.default_db'); + $config += array( + 'username' => ini_get('odbc.default_user'), + 'password' => ini_get('odbc.default_pw'), + 'dsn' => ini_get('odbc.default_db'), + ); if (empty($config['persistent'])) { $this->connection = @odbc_connect($config['dsn'], $config['username'], $config['password']); // intentionally @ @@ -83,7 +83,6 @@ class DibiOdbcDriver extends DibiObject implements IDibiDriver, IDibiResultDrive } - /** * Disconnects from a database. * @return void @@ -94,7 +93,6 @@ class DibiOdbcDriver extends DibiObject implements IDibiDriver, IDibiResultDrive } - /** * Executes the SQL query. * @param string SQL statement. @@ -116,7 +114,6 @@ class DibiOdbcDriver extends DibiObject implements IDibiDriver, IDibiResultDrive } - /** * Gets the number of affected rows by the last INSERT, UPDATE or DELETE query. * @return int|FALSE number of rows or FALSE on error @@ -127,7 +124,6 @@ class DibiOdbcDriver extends DibiObject implements IDibiDriver, IDibiResultDrive } - /** * Retrieves the ID generated for an AUTO_INCREMENT column by the previous INSERT query. * @return int|FALSE int on success or FALSE on failure @@ -138,7 +134,6 @@ class DibiOdbcDriver extends DibiObject implements IDibiDriver, IDibiResultDrive } - /** * Begins a transaction (if supported). * @param string optional savepoint name @@ -153,7 +148,6 @@ class DibiOdbcDriver extends DibiObject implements IDibiDriver, IDibiResultDrive } - /** * Commits statements in a transaction. * @param string optional savepoint name @@ -169,7 +163,6 @@ class DibiOdbcDriver extends DibiObject implements IDibiDriver, IDibiResultDrive } - /** * Rollback changes in a transaction. * @param string optional savepoint name @@ -185,7 +178,6 @@ class DibiOdbcDriver extends DibiObject implements IDibiDriver, IDibiResultDrive } - /** * Is in transaction? * @return bool @@ -196,7 +188,6 @@ class DibiOdbcDriver extends DibiObject implements IDibiDriver, IDibiResultDrive } - /** * Returns the connection resource. * @return mixed @@ -207,7 +198,6 @@ class DibiOdbcDriver extends DibiObject implements IDibiDriver, IDibiResultDrive } - /** * Returns the connection reflector. * @return IDibiReflector @@ -218,7 +208,6 @@ class DibiOdbcDriver extends DibiObject implements IDibiDriver, IDibiResultDrive } - /** * Result set driver factory. * @param resource @@ -232,11 +221,9 @@ class DibiOdbcDriver extends DibiObject implements IDibiDriver, IDibiResultDrive } - /********************* SQL ****************d*g**/ - /** * Encodes data for use in a SQL statement. * @param mixed value @@ -247,29 +234,28 @@ class DibiOdbcDriver extends DibiObject implements IDibiDriver, IDibiResultDrive public function escape($value, $type) { switch ($type) { - case dibi::TEXT: - case dibi::BINARY: - return "'" . str_replace("'", "''", $value) . "'"; + case dibi::TEXT: + case dibi::BINARY: + return "'" . str_replace("'", "''", $value) . "'"; - case dibi::IDENTIFIER: - return '[' . str_replace(array('[', ']'), array('[[', ']]'), $value) . ']'; + case dibi::IDENTIFIER: + return '[' . str_replace(array('[', ']'), array('[[', ']]'), $value) . ']'; - case dibi::BOOL: - return $value ? 1 : 0; + case dibi::BOOL: + return $value ? 1 : 0; - case dibi::DATE: - return $value instanceof DateTime ? $value->format("#m/d/Y#") : date("#m/d/Y#", $value); + case dibi::DATE: + return $value instanceof DateTime ? $value->format("#m/d/Y#") : date("#m/d/Y#", $value); - case dibi::DATETIME: - return $value instanceof DateTime ? $value->format("#m/d/Y H:i:s#") : date("#m/d/Y H:i:s#", $value); + case dibi::DATETIME: + return $value instanceof DateTime ? $value->format("#m/d/Y H:i:s#") : date("#m/d/Y H:i:s#", $value); - default: - throw new InvalidArgumentException('Unsupported type.'); + default: + throw new InvalidArgumentException('Unsupported type.'); } } - /** * Encodes string for use in a LIKE statement. * @param string @@ -283,7 +269,6 @@ class DibiOdbcDriver extends DibiObject implements IDibiDriver, IDibiResultDrive } - /** * Decodes data from result set. * @param string value @@ -300,30 +285,26 @@ class DibiOdbcDriver extends DibiObject implements IDibiDriver, IDibiResultDrive } - /** * Injects LIMIT/OFFSET to the SQL query. - * @param string &$sql The SQL query that will be modified. - * @param int $limit - * @param int $offset * @return void */ - public function applyLimit(&$sql, $limit, $offset) + public function applyLimit(& $sql, $limit, $offset) { // offset support is missing if ($limit >= 0) { $sql = 'SELECT TOP ' . (int) $limit . ' * FROM (' . $sql . ')'; } - if ($offset) throw new DibiNotSupportedException('Offset is not implemented in driver odbc.'); + if ($offset) { + throw new DibiNotSupportedException('Offset is not implemented in driver odbc.'); + } } - /********************* result set ****************d*g**/ - /** * Automatically frees the resources allocated for this result set. * @return void @@ -334,7 +315,6 @@ class DibiOdbcDriver extends DibiObject implements IDibiDriver, IDibiResultDrive } - /** * Returns the number of rows in a result set. * @return int @@ -346,7 +326,6 @@ class DibiOdbcDriver extends DibiObject implements IDibiDriver, IDibiResultDrive } - /** * Fetches the row at current position and moves the internal cursor to the next position. * @param bool TRUE for associative array, FALSE for numeric @@ -358,7 +337,9 @@ class DibiOdbcDriver extends DibiObject implements IDibiDriver, IDibiResultDrive return odbc_fetch_array($this->resultSet, ++$this->row); } else { $set = $this->resultSet; - if (!odbc_fetch_row($set, ++$this->row)) return FALSE; + if (!odbc_fetch_row($set, ++$this->row)) { + return FALSE; + } $count = odbc_num_fields($set); $cols = array(); for ($i = 1; $i <= $count; $i++) $cols[] = odbc_result($set, $i); @@ -367,7 +348,6 @@ class DibiOdbcDriver extends DibiObject implements IDibiDriver, IDibiResultDrive } - /** * Moves cursor position without fetching row. * @param int the 0-based cursor pos to seek to @@ -380,7 +360,6 @@ class DibiOdbcDriver extends DibiObject implements IDibiDriver, IDibiResultDrive } - /** * Frees the resources allocated for this result set. * @return void @@ -392,7 +371,6 @@ class DibiOdbcDriver extends DibiObject implements IDibiDriver, IDibiResultDrive } - /** * Returns metadata for all columns in a result set. * @return array @@ -413,7 +391,6 @@ class DibiOdbcDriver extends DibiObject implements IDibiDriver, IDibiResultDrive } - /** * Returns the result set resource. * @return mixed @@ -425,11 +402,9 @@ class DibiOdbcDriver extends DibiObject implements IDibiDriver, IDibiResultDrive } - /********************* IDibiReflector ****************d*g**/ - /** * Returns list of tables. * @return array @@ -451,7 +426,6 @@ class DibiOdbcDriver extends DibiObject implements IDibiDriver, IDibiResultDrive } - /** * Returns metadata for all columns in a table. * @param string @@ -478,7 +452,6 @@ class DibiOdbcDriver extends DibiObject implements IDibiDriver, IDibiResultDrive } - /** * Returns metadata for all indexes in a table. * @param string @@ -490,7 +463,6 @@ class DibiOdbcDriver extends DibiObject implements IDibiDriver, IDibiResultDrive } - /** * Returns metadata for all foreign keys in a table. * @param string diff --git a/dibi/drivers/DibiOracleDriver.php b/dibi/drivers/DibiOracleDriver.php index 93d6603b..41e9069e 100644 --- a/dibi/drivers/DibiOracleDriver.php +++ b/dibi/drivers/DibiOracleDriver.php @@ -45,7 +45,6 @@ class DibiOracleDriver extends DibiObject implements IDibiDriver, IDibiResultDri private $fmtDate, $fmtDateTime; - /** * @throws DibiNotSupportedException */ @@ -57,13 +56,12 @@ class DibiOracleDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Connects to a database. * @return void * @throws DibiException */ - public function connect(array &$config) + public function connect(array & $config) { $foo = & $config['charset']; $this->fmtDate = isset($config['formatDate']) ? $config['formatDate'] : 'U'; @@ -84,7 +82,6 @@ class DibiOracleDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Disconnects from a database. * @return void @@ -95,7 +92,6 @@ class DibiOracleDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Executes the SQL query. * @param string SQL statement. @@ -121,7 +117,6 @@ class DibiOracleDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Gets the number of affected rows by the last INSERT, UPDATE or DELETE query. * @return int|FALSE number of rows or FALSE on error @@ -132,7 +127,6 @@ class DibiOracleDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Retrieves the ID generated for an AUTO_INCREMENT column by the previous INSERT query. * @return int|FALSE int on success or FALSE on failure @@ -144,7 +138,6 @@ class DibiOracleDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Begins a transaction (if supported). * @param string optional savepoint name @@ -156,7 +149,6 @@ class DibiOracleDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Commits statements in a transaction. * @param string optional savepoint name @@ -173,7 +165,6 @@ class DibiOracleDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Rollback changes in a transaction. * @param string optional savepoint name @@ -190,7 +181,6 @@ class DibiOracleDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Returns the connection resource. * @return mixed @@ -201,7 +191,6 @@ class DibiOracleDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Returns the connection reflector. * @return IDibiReflector @@ -212,7 +201,6 @@ class DibiOracleDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Result set driver factory. * @param resource @@ -226,11 +214,9 @@ class DibiOracleDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /********************* SQL ****************d*g**/ - /** * Encodes data for use in a SQL statement. * @param mixed value @@ -241,30 +227,29 @@ class DibiOracleDriver extends DibiObject implements IDibiDriver, IDibiResultDri public function escape($value, $type) { switch ($type) { - case dibi::TEXT: - case dibi::BINARY: - return "'" . str_replace("'", "''", $value) . "'"; // TODO: not tested + case dibi::TEXT: + case dibi::BINARY: + return "'" . str_replace("'", "''", $value) . "'"; // TODO: not tested - case dibi::IDENTIFIER: - // @see http://download.oracle.com/docs/cd/B10500_01/server.920/a96540/sql_elements9a.htm - return '"' . str_replace('"', '""', $value) . '"'; + case dibi::IDENTIFIER: + // @see http://download.oracle.com/docs/cd/B10500_01/server.920/a96540/sql_elements9a.htm + return '"' . str_replace('"', '""', $value) . '"'; - case dibi::BOOL: - return $value ? 1 : 0; + case dibi::BOOL: + return $value ? 1 : 0; - case dibi::DATE: - return $value instanceof DateTime ? $value->format($this->fmtDate) : date($this->fmtDate, $value); + case dibi::DATE: + return $value instanceof DateTime ? $value->format($this->fmtDate) : date($this->fmtDate, $value); - case dibi::DATETIME: - return $value instanceof DateTime ? $value->format($this->fmtDateTime) : date($this->fmtDateTime, $value); + case dibi::DATETIME: + return $value instanceof DateTime ? $value->format($this->fmtDateTime) : date($this->fmtDateTime, $value); - default: - throw new InvalidArgumentException('Unsupported type.'); + default: + throw new InvalidArgumentException('Unsupported type.'); } } - /** * Encodes string for use in a LIKE statement. * @param string @@ -279,7 +264,6 @@ class DibiOracleDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Decodes data from result set. * @param string value @@ -296,19 +280,17 @@ class DibiOracleDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Injects LIMIT/OFFSET to the SQL query. - * @param string &$sql The SQL query that will be modified. - * @param int $limit - * @param int $offset * @return void */ - public function applyLimit(&$sql, $limit, $offset) + public function applyLimit(& $sql, $limit, $offset) { if ($offset > 0) { // see http://www.oracle.com/technology/oramag/oracle/06-sep/o56asktom.html - $sql = 'SELECT * FROM (SELECT t.*, ROWNUM AS "__rnum" FROM (' . $sql . ') t ' . ($limit >= 0 ? 'WHERE ROWNUM <= ' . ((int) $offset + (int) $limit) : '') . ') WHERE "__rnum" > '. (int) $offset; + $sql = 'SELECT * FROM (SELECT t.*, ROWNUM AS "__rnum" FROM (' . $sql . ') t ' + . ($limit >= 0 ? 'WHERE ROWNUM <= ' . ((int) $offset + (int) $limit) : '') + . ') WHERE "__rnum" > '. (int) $offset; } elseif ($limit >= 0) { $sql = 'SELECT * FROM (' . $sql . ') WHERE ROWNUM <= ' . (int) $limit; @@ -316,11 +298,9 @@ class DibiOracleDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /********************* result set ****************d*g**/ - /** * Automatically frees the resources allocated for this result set. * @return void @@ -331,7 +311,6 @@ class DibiOracleDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Returns the number of rows in a result set. * @return int @@ -342,7 +321,6 @@ class DibiOracleDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Fetches the row at current position and moves the internal cursor to the next position. * @param bool TRUE for associative array, FALSE for numeric @@ -354,7 +332,6 @@ class DibiOracleDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Moves cursor position without fetching row. * @param int the 0-based cursor pos to seek to @@ -366,7 +343,6 @@ class DibiOracleDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Frees the resources allocated for this result set. * @return void @@ -378,7 +354,6 @@ class DibiOracleDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Returns metadata for all columns in a result set. * @return array @@ -399,7 +374,6 @@ class DibiOracleDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Returns the result set resource. * @return mixed @@ -411,11 +385,9 @@ class DibiOracleDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /********************* IDibiReflector ****************d*g**/ - /** * Returns list of tables. * @return array @@ -436,7 +408,6 @@ class DibiOracleDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Returns metadata for all columns in a table. * @param string @@ -448,7 +419,6 @@ class DibiOracleDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Returns metadata for all indexes in a table. * @param string @@ -460,7 +430,6 @@ class DibiOracleDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Returns metadata for all foreign keys in a table. * @param string diff --git a/dibi/drivers/DibiPdoDriver.php b/dibi/drivers/DibiPdoDriver.php index 03634cfa..be2dc3ab 100644 --- a/dibi/drivers/DibiPdoDriver.php +++ b/dibi/drivers/DibiPdoDriver.php @@ -43,7 +43,6 @@ class DibiPdoDriver extends DibiObject implements IDibiDriver, IDibiResultDriver private $driverName; - /** * @throws DibiNotSupportedException */ @@ -55,13 +54,12 @@ class DibiPdoDriver extends DibiObject implements IDibiDriver, IDibiResultDriver } - /** * Connects to a database. * @return void * @throws DibiException */ - public function connect(array &$config) + public function connect(array & $config) { $foo = & $config['dsn']; $foo = & $config['options']; @@ -85,7 +83,6 @@ class DibiPdoDriver extends DibiObject implements IDibiDriver, IDibiResultDriver } - /** * Disconnects from a database. * @return void @@ -96,7 +93,6 @@ class DibiPdoDriver extends DibiObject implements IDibiDriver, IDibiResultDriver } - /** * Executes the SQL query. * @param string SQL statement. @@ -131,7 +127,6 @@ class DibiPdoDriver extends DibiObject implements IDibiDriver, IDibiResultDriver } - /** * Gets the number of affected rows by the last INSERT, UPDATE or DELETE query. * @return int|FALSE number of rows or FALSE on error @@ -142,7 +137,6 @@ class DibiPdoDriver extends DibiObject implements IDibiDriver, IDibiResultDriver } - /** * Retrieves the ID generated for an AUTO_INCREMENT column by the previous INSERT query. * @return int|FALSE int on success or FALSE on failure @@ -153,7 +147,6 @@ class DibiPdoDriver extends DibiObject implements IDibiDriver, IDibiResultDriver } - /** * Begins a transaction (if supported). * @param string optional savepoint name @@ -169,7 +162,6 @@ class DibiPdoDriver extends DibiObject implements IDibiDriver, IDibiResultDriver } - /** * Commits statements in a transaction. * @param string optional savepoint name @@ -185,7 +177,6 @@ class DibiPdoDriver extends DibiObject implements IDibiDriver, IDibiResultDriver } - /** * Rollback changes in a transaction. * @param string optional savepoint name @@ -201,7 +192,6 @@ class DibiPdoDriver extends DibiObject implements IDibiDriver, IDibiResultDriver } - /** * Returns the connection resource. * @return PDO @@ -212,7 +202,6 @@ class DibiPdoDriver extends DibiObject implements IDibiDriver, IDibiResultDriver } - /** * Returns the connection reflector. * @return IDibiReflector @@ -220,20 +209,19 @@ class DibiPdoDriver extends DibiObject implements IDibiDriver, IDibiResultDriver public function getReflector() { switch ($this->driverName) { - case 'mysql': - return new DibiMySqlReflector($this); + case 'mysql': + return new DibiMySqlReflector($this); - case 'sqlite': - case 'sqlite2': - return new DibiSqliteReflector($this); + case 'sqlite': + case 'sqlite2': + return new DibiSqliteReflector($this); - default: - throw new DibiNotSupportedException; + default: + throw new DibiNotSupportedException; } } - /** * Result set driver factory. * @param PDOStatement @@ -247,11 +235,9 @@ class DibiPdoDriver extends DibiObject implements IDibiDriver, IDibiResultDriver } - /********************* SQL ****************d*g**/ - /** * Encodes data for use in a SQL statement. * @param mixed value @@ -262,52 +248,51 @@ class DibiPdoDriver extends DibiObject implements IDibiDriver, IDibiResultDriver public function escape($value, $type) { switch ($type) { - case dibi::TEXT: - return $this->connection->quote($value, PDO::PARAM_STR); + case dibi::TEXT: + return $this->connection->quote($value, PDO::PARAM_STR); - case dibi::BINARY: - return $this->connection->quote($value, PDO::PARAM_LOB); + case dibi::BINARY: + return $this->connection->quote($value, PDO::PARAM_LOB); - case dibi::IDENTIFIER: - switch ($this->driverName) { - case 'mysql': - return '`' . str_replace('`', '``', $value) . '`'; + case dibi::IDENTIFIER: + switch ($this->driverName) { + case 'mysql': + return '`' . str_replace('`', '``', $value) . '`'; - case 'pgsql': - return '"' . str_replace('"', '""', $value) . '"'; + case 'pgsql': + return '"' . str_replace('"', '""', $value) . '"'; - case 'sqlite': - case 'sqlite2': - return '[' . strtr($value, '[]', ' ') . ']'; + case 'sqlite': + case 'sqlite2': + return '[' . strtr($value, '[]', ' ') . ']'; - case 'odbc': - case 'oci': // TODO: not tested - case 'mssql': - return '[' . str_replace(array('[', ']'), array('[[', ']]'), $value) . ']'; + case 'odbc': + case 'oci': // TODO: not tested + case 'mssql': + return '[' . str_replace(array('[', ']'), array('[[', ']]'), $value) . ']'; - case 'sqlsrv': - return '[' . str_replace(']', ']]', $value) . ']'; + case 'sqlsrv': + return '[' . str_replace(']', ']]', $value) . ']'; + + default: + return $value; + } + + case dibi::BOOL: + return $this->connection->quote($value, PDO::PARAM_BOOL); + + case dibi::DATE: + return $value instanceof DateTime ? $value->format("'Y-m-d'") : date("'Y-m-d'", $value); + + case dibi::DATETIME: + return $value instanceof DateTime ? $value->format("'Y-m-d H:i:s'") : date("'Y-m-d H:i:s'", $value); default: - return $value; - } - - case dibi::BOOL: - return $this->connection->quote($value, PDO::PARAM_BOOL); - - case dibi::DATE: - return $value instanceof DateTime ? $value->format("'Y-m-d'") : date("'Y-m-d'", $value); - - case dibi::DATETIME: - return $value instanceof DateTime ? $value->format("'Y-m-d H:i:s'") : date("'Y-m-d H:i:s'", $value); - - default: - throw new InvalidArgumentException('Unsupported type.'); + throw new InvalidArgumentException('Unsupported type.'); } } - /** * Encodes string for use in a LIKE statement. * @param string @@ -320,7 +305,6 @@ class DibiPdoDriver extends DibiObject implements IDibiDriver, IDibiResultDriver } - /** * Decodes data from result set. * @param string value @@ -337,62 +321,64 @@ class DibiPdoDriver extends DibiObject implements IDibiDriver, IDibiResultDriver } - /** * Injects LIMIT/OFFSET to the SQL query. - * @param string &$sql The SQL query that will be modified. - * @param int $limit - * @param int $offset * @return void */ - public function applyLimit(&$sql, $limit, $offset) + public function applyLimit(& $sql, $limit, $offset) { - if ($limit < 0 && $offset < 1) return; + if ($limit < 0 && $offset < 1) { + return; + } switch ($this->driverName) { - case 'mysql': - $sql .= ' LIMIT ' . ($limit < 0 ? '18446744073709551615' : (int) $limit) - . ($offset > 0 ? ' OFFSET ' . (int) $offset : ''); - break; - - case 'pgsql': - if ($limit >= 0) $sql .= ' LIMIT ' . (int) $limit; - if ($offset > 0) $sql .= ' OFFSET ' . (int) $offset; - break; - - case 'sqlite': - case 'sqlite2': - $sql .= ' LIMIT ' . $limit . ($offset > 0 ? ' OFFSET ' . (int) $offset : ''); - break; - - case 'oci': - if ($offset > 0) { - $sql = 'SELECT * FROM (SELECT t.*, ROWNUM AS "__rnum" FROM (' . $sql . ') t ' . ($limit >= 0 ? 'WHERE ROWNUM <= ' . ((int) $offset + (int) $limit) : '') . ') WHERE "__rnum" > '. (int) $offset; - } elseif ($limit >= 0) { - $sql = 'SELECT * FROM (' . $sql . ') WHERE ROWNUM <= ' . (int) $limit; - } - break; - - case 'odbc': - case 'mssql': - case 'sqlsrv': - if ($offset < 1) { - $sql = 'SELECT TOP ' . (int) $limit . ' * FROM (' . $sql . ') t'; + case 'mysql': + $sql .= ' LIMIT ' . ($limit < 0 ? '18446744073709551615' : (int) $limit) + . ($offset > 0 ? ' OFFSET ' . (int) $offset : ''); break; - } - // intentionally break omitted - default: - throw new DibiNotSupportedException('PDO or driver does not support applying limit or offset.'); + case 'pgsql': + if ($limit >= 0) { + $sql .= ' LIMIT ' . (int) $limit; + } + if ($offset > 0) { + $sql .= ' OFFSET ' . (int) $offset; + } + break; + + case 'sqlite': + case 'sqlite2': + $sql .= ' LIMIT ' . $limit . ($offset > 0 ? ' OFFSET ' . (int) $offset : ''); + break; + + case 'oci': + if ($offset > 0) { + $sql = 'SELECT * FROM (SELECT t.*, ROWNUM AS "__rnum" FROM (' . $sql . ') t ' + . ($limit >= 0 ? 'WHERE ROWNUM <= ' . ((int) $offset + (int) $limit) : '') + . ') WHERE "__rnum" > '. (int) $offset; + } elseif ($limit >= 0) { + $sql = 'SELECT * FROM (' . $sql . ') WHERE ROWNUM <= ' . (int) $limit; + } + break; + + case 'odbc': + case 'mssql': + case 'sqlsrv': + if ($offset < 1) { + $sql = 'SELECT TOP ' . (int) $limit . ' * FROM (' . $sql . ') t'; + break; + } + // intentionally break omitted + + default: + throw new DibiNotSupportedException('PDO or driver does not support applying limit or offset.'); } } - /********************* result set ****************d*g**/ - /** * Returns the number of rows in a result set. * @return int @@ -403,7 +389,6 @@ class DibiPdoDriver extends DibiObject implements IDibiDriver, IDibiResultDriver } - /** * Fetches the row at current position and moves the internal cursor to the next position. * @param bool TRUE for associative array, FALSE for numeric @@ -415,7 +400,6 @@ class DibiPdoDriver extends DibiObject implements IDibiDriver, IDibiResultDriver } - /** * Moves cursor position without fetching row. * @param int the 0-based cursor pos to seek to @@ -427,7 +411,6 @@ class DibiPdoDriver extends DibiObject implements IDibiDriver, IDibiResultDriver } - /** * Frees the resources allocated for this result set. * @return void @@ -438,7 +421,6 @@ class DibiPdoDriver extends DibiObject implements IDibiDriver, IDibiResultDriver } - /** * Returns metadata for all columns in a result set. * @return array @@ -472,7 +454,6 @@ class DibiPdoDriver extends DibiObject implements IDibiDriver, IDibiResultDriver } - /** * Returns the result set resource. * @return PDOStatement diff --git a/dibi/drivers/DibiPostgreDriver.php b/dibi/drivers/DibiPostgreDriver.php index ebf25ca0..c4740ad6 100644 --- a/dibi/drivers/DibiPostgreDriver.php +++ b/dibi/drivers/DibiPostgreDriver.php @@ -43,7 +43,6 @@ class DibiPostgreDriver extends DibiObject implements IDibiDriver, IDibiResultDr private $escMethod = FALSE; - /** * @throws DibiNotSupportedException */ @@ -55,19 +54,20 @@ class DibiPostgreDriver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Connects to a database. * @return void * @throws DibiException */ - public function connect(array &$config) + public function connect(array & $config) { if (isset($config['resource'])) { $this->connection = $config['resource']; } else { - if (!isset($config['charset'])) $config['charset'] = 'utf8'; + $config += array( + 'charset' => 'utf8', + ); if (isset($config['string'])) { $string = $config['string']; } else { @@ -75,7 +75,9 @@ class DibiPostgreDriver extends DibiObject implements IDibiDriver, IDibiResultDr DibiConnection::alias($config, 'user', 'username'); DibiConnection::alias($config, 'dbname', 'database'); foreach (array('host','hostaddr','port','dbname','user','password','connect_timeout','options','sslmode','service') as $key) { - if (isset($config[$key])) $string .= $key . '=' . $config[$key] . ' '; + if (isset($config[$key])) { + $string .= $key . '=' . $config[$key] . ' '; + } } } @@ -110,7 +112,6 @@ class DibiPostgreDriver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Disconnects from a database. * @return void @@ -121,7 +122,6 @@ class DibiPostgreDriver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Executes the SQL query. * @param string SQL statement. @@ -145,7 +145,6 @@ class DibiPostgreDriver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Gets the number of affected rows by the last INSERT, UPDATE or DELETE query. * @return int|FALSE number of rows or FALSE on error @@ -156,7 +155,6 @@ class DibiPostgreDriver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Retrieves the ID generated for an AUTO_INCREMENT column by the previous INSERT query. * @return int|FALSE int on success or FALSE on failure @@ -170,14 +168,15 @@ class DibiPostgreDriver extends DibiObject implements IDibiDriver, IDibiResultDr $res = $this->query("SELECT CURRVAL('$sequence')"); } - if (!$res) return FALSE; + if (!$res) { + return FALSE; + } $row = $res->fetch(FALSE); return is_array($row) ? $row[0] : FALSE; } - /** * Begins a transaction (if supported). * @param string optional savepoint name @@ -190,7 +189,6 @@ class DibiPostgreDriver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Commits statements in a transaction. * @param string optional savepoint name @@ -203,7 +201,6 @@ class DibiPostgreDriver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Rollback changes in a transaction. * @param string optional savepoint name @@ -216,7 +213,6 @@ class DibiPostgreDriver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Is in transaction? * @return bool @@ -227,7 +223,6 @@ class DibiPostgreDriver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Returns the connection resource. * @return mixed @@ -238,7 +233,6 @@ class DibiPostgreDriver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Returns the connection reflector. * @return IDibiReflector @@ -249,7 +243,6 @@ class DibiPostgreDriver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Result set driver factory. * @param resource @@ -263,11 +256,9 @@ class DibiPostgreDriver extends DibiObject implements IDibiDriver, IDibiResultDr } - /********************* SQL ****************d*g**/ - /** * Encodes data for use in a SQL statement. * @param mixed value @@ -278,46 +269,45 @@ class DibiPostgreDriver extends DibiObject implements IDibiDriver, IDibiResultDr public function escape($value, $type) { switch ($type) { - case dibi::TEXT: - if ($this->escMethod) { - if (!is_resource($this->connection)) { - throw new DibiException('Lost connection to server.'); + case dibi::TEXT: + if ($this->escMethod) { + if (!is_resource($this->connection)) { + throw new DibiException('Lost connection to server.'); + } + return "'" . pg_escape_string($this->connection, $value) . "'"; + } else { + return "'" . pg_escape_string($value) . "'"; } - return "'" . pg_escape_string($this->connection, $value) . "'"; - } else { - return "'" . pg_escape_string($value) . "'"; - } - case dibi::BINARY: - if ($this->escMethod) { - if (!is_resource($this->connection)) { - throw new DibiException('Lost connection to server.'); + case dibi::BINARY: + if ($this->escMethod) { + if (!is_resource($this->connection)) { + throw new DibiException('Lost connection to server.'); + } + return "'" . pg_escape_bytea($this->connection, $value) . "'"; + } else { + return "'" . pg_escape_bytea($value) . "'"; } - return "'" . pg_escape_bytea($this->connection, $value) . "'"; - } else { - return "'" . pg_escape_bytea($value) . "'"; - } - case dibi::IDENTIFIER: - // @see http://www.postgresql.org/docs/8.2/static/sql-syntax-lexical.html#SQL-SYNTAX-IDENTIFIERS - return '"' . str_replace('"', '""', $value) . '"'; + case dibi::IDENTIFIER: + // @see http://www.postgresql.org/docs/8.2/static/sql-syntax-lexical.html#SQL-SYNTAX-IDENTIFIERS + return '"' . str_replace('"', '""', $value) . '"'; - case dibi::BOOL: - return $value ? 'TRUE' : 'FALSE'; + case dibi::BOOL: + return $value ? 'TRUE' : 'FALSE'; - case dibi::DATE: - return $value instanceof DateTime ? $value->format("'Y-m-d'") : date("'Y-m-d'", $value); + case dibi::DATE: + return $value instanceof DateTime ? $value->format("'Y-m-d'") : date("'Y-m-d'", $value); - case dibi::DATETIME: - return $value instanceof DateTime ? $value->format("'Y-m-d H:i:s'") : date("'Y-m-d H:i:s'", $value); + case dibi::DATETIME: + return $value instanceof DateTime ? $value->format("'Y-m-d H:i:s'") : date("'Y-m-d H:i:s'", $value); - default: - throw new InvalidArgumentException('Unsupported type.'); + default: + throw new InvalidArgumentException('Unsupported type.'); } } - /** * Encodes string for use in a LIKE statement. * @param string @@ -337,7 +327,6 @@ class DibiPostgreDriver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Decodes data from result set. * @param string value @@ -354,29 +343,25 @@ class DibiPostgreDriver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Injects LIMIT/OFFSET to the SQL query. - * @param string &$sql The SQL query that will be modified. - * @param int $limit - * @param int $offset * @return void */ - public function applyLimit(&$sql, $limit, $offset) + public function applyLimit(& $sql, $limit, $offset) { - if ($limit >= 0) + if ($limit >= 0) { $sql .= ' LIMIT ' . (int) $limit; + } - if ($offset > 0) + if ($offset > 0) { $sql .= ' OFFSET ' . (int) $offset; + } } - /********************* result set ****************d*g**/ - /** * Automatically frees the resources allocated for this result set. * @return void @@ -387,7 +372,6 @@ class DibiPostgreDriver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Returns the number of rows in a result set. * @return int @@ -398,7 +382,6 @@ class DibiPostgreDriver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Fetches the row at current position and moves the internal cursor to the next position. * @param bool TRUE for associative array, FALSE for numeric @@ -410,7 +393,6 @@ class DibiPostgreDriver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Moves cursor position without fetching row. * @param int the 0-based cursor pos to seek to @@ -422,7 +404,6 @@ class DibiPostgreDriver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Frees the resources allocated for this result set. * @return void @@ -434,7 +415,6 @@ class DibiPostgreDriver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Returns metadata for all columns in a result set. * @return array @@ -457,7 +437,6 @@ class DibiPostgreDriver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Returns the result set resource. * @return mixed @@ -469,11 +448,9 @@ class DibiPostgreDriver extends DibiObject implements IDibiDriver, IDibiResultDr } - /********************* IDibiReflector ****************d*g**/ - /** * Returns list of tables. * @return array @@ -502,7 +479,6 @@ class DibiPostgreDriver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Returns metadata for all columns in a table. * @param string @@ -543,7 +519,6 @@ class DibiPostgreDriver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Returns metadata for all indexes in a table. * @param string @@ -585,7 +560,6 @@ class DibiPostgreDriver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Returns metadata for all foreign keys in a table. * @param string diff --git a/dibi/drivers/DibiSqlite3Driver.php b/dibi/drivers/DibiSqlite3Driver.php index f4e96b18..dcde878b 100644 --- a/dibi/drivers/DibiSqlite3Driver.php +++ b/dibi/drivers/DibiSqlite3Driver.php @@ -46,7 +46,6 @@ class DibiSqlite3Driver extends DibiObject implements IDibiDriver, IDibiResultDr private $dbcharset, $charset; - /** * @throws DibiNotSupportedException */ @@ -58,13 +57,12 @@ class DibiSqlite3Driver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Connects to a database. * @return void * @throws DibiException */ - public function connect(array &$config) + public function connect(array & $config) { DibiConnection::alias($config, 'database', 'file'); $this->fmtDate = isset($config['formatDate']) ? $config['formatDate'] : 'U'; @@ -93,7 +91,6 @@ class DibiSqlite3Driver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Disconnects from a database. * @return void @@ -104,7 +101,6 @@ class DibiSqlite3Driver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Executes the SQL query. * @param string SQL statement. @@ -127,7 +123,6 @@ class DibiSqlite3Driver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Gets the number of affected rows by the last INSERT, UPDATE or DELETE query. * @return int|FALSE number of rows or FALSE on error @@ -138,7 +133,6 @@ class DibiSqlite3Driver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Retrieves the ID generated for an AUTO_INCREMENT column by the previous INSERT query. * @return int|FALSE int on success or FALSE on failure @@ -149,7 +143,6 @@ class DibiSqlite3Driver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Begins a transaction (if supported). * @param string optional savepoint name @@ -162,7 +155,6 @@ class DibiSqlite3Driver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Commits statements in a transaction. * @param string optional savepoint name @@ -175,7 +167,6 @@ class DibiSqlite3Driver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Rollback changes in a transaction. * @param string optional savepoint name @@ -188,7 +179,6 @@ class DibiSqlite3Driver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Returns the connection resource. * @return mixed @@ -199,7 +189,6 @@ class DibiSqlite3Driver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Returns the connection reflector. * @return IDibiReflector @@ -210,7 +199,6 @@ class DibiSqlite3Driver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Result set driver factory. * @param SQLite3Result @@ -224,11 +212,9 @@ class DibiSqlite3Driver extends DibiObject implements IDibiDriver, IDibiResultDr } - /********************* SQL ****************d*g**/ - /** * Encodes data for use in a SQL statement. * @param mixed value @@ -239,31 +225,30 @@ class DibiSqlite3Driver extends DibiObject implements IDibiDriver, IDibiResultDr public function escape($value, $type) { switch ($type) { - case dibi::TEXT: - return "'" . $this->connection->escapeString($value) . "'"; + case dibi::TEXT: + return "'" . $this->connection->escapeString($value) . "'"; - case dibi::BINARY: - return "X'" . bin2hex((string) $value) . "'"; + case dibi::BINARY: + return "X'" . bin2hex((string) $value) . "'"; - case dibi::IDENTIFIER: - return '[' . strtr($value, '[]', ' ') . ']'; + case dibi::IDENTIFIER: + return '[' . strtr($value, '[]', ' ') . ']'; - case dibi::BOOL: - return $value ? 1 : 0; + case dibi::BOOL: + return $value ? 1 : 0; - case dibi::DATE: - return $value instanceof DateTime ? $value->format($this->fmtDate) : date($this->fmtDate, $value); + case dibi::DATE: + return $value instanceof DateTime ? $value->format($this->fmtDate) : date($this->fmtDate, $value); - case dibi::DATETIME: - return $value instanceof DateTime ? $value->format($this->fmtDateTime) : date($this->fmtDateTime, $value); + case dibi::DATETIME: + return $value instanceof DateTime ? $value->format($this->fmtDateTime) : date($this->fmtDateTime, $value); - default: - throw new InvalidArgumentException('Unsupported type.'); + default: + throw new InvalidArgumentException('Unsupported type.'); } } - /** * Encodes string for use in a LIKE statement. * @param string @@ -277,7 +262,6 @@ class DibiSqlite3Driver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Decodes data from result set. * @param string value @@ -294,26 +278,21 @@ class DibiSqlite3Driver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Injects LIMIT/OFFSET to the SQL query. - * @param string &$sql The SQL query that will be modified. - * @param int $limit - * @param int $offset * @return void */ - public function applyLimit(&$sql, $limit, $offset) + public function applyLimit(& $sql, $limit, $offset) { - if ($limit < 0 && $offset < 1) return; - $sql .= ' LIMIT ' . $limit . ($offset > 0 ? ' OFFSET ' . (int) $offset : ''); + if ($limit >= 0 || $offset > 0) { + $sql .= ' LIMIT ' . (int) $limit . ($offset > 0 ? ' OFFSET ' . (int) $offset : ''); + } } - /********************* result set ****************d*g**/ - /** * Automatically frees the resources allocated for this result set. * @return void @@ -324,7 +303,6 @@ class DibiSqlite3Driver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Returns the number of rows in a result set. * @return int @@ -336,7 +314,6 @@ class DibiSqlite3Driver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Fetches the row at current position and moves the internal cursor to the next position. * @param bool TRUE for associative array, FALSE for numeric @@ -360,7 +337,6 @@ class DibiSqlite3Driver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Moves cursor position without fetching row. * @param int the 0-based cursor pos to seek to @@ -373,7 +349,6 @@ class DibiSqlite3Driver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Frees the resources allocated for this result set. * @return void @@ -385,7 +360,6 @@ class DibiSqlite3Driver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Returns metadata for all columns in a result set. * @return array @@ -407,7 +381,6 @@ class DibiSqlite3Driver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Returns the result set resource. * @return mixed @@ -419,11 +392,9 @@ class DibiSqlite3Driver extends DibiObject implements IDibiDriver, IDibiResultDr } - /********************* user defined functions ****************d*g**/ - /** * Registers an user defined function for use in SQL statements. * @param string function name @@ -437,7 +408,6 @@ class DibiSqlite3Driver extends DibiObject implements IDibiDriver, IDibiResultDr } - /** * Registers an aggregating user defined function for use in SQL statements. * @param string function name diff --git a/dibi/drivers/DibiSqliteDriver.php b/dibi/drivers/DibiSqliteDriver.php index 131df44f..b2306d3b 100644 --- a/dibi/drivers/DibiSqliteDriver.php +++ b/dibi/drivers/DibiSqliteDriver.php @@ -48,7 +48,6 @@ class DibiSqliteDriver extends DibiObject implements IDibiDriver, IDibiResultDri private $dbcharset, $charset; - /** * @throws DibiNotSupportedException */ @@ -60,13 +59,12 @@ class DibiSqliteDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Connects to a database. * @return void * @throws DibiException */ - public function connect(array &$config) + public function connect(array & $config) { DibiConnection::alias($config, 'database', 'file'); $this->fmtDate = isset($config['formatDate']) ? $config['formatDate'] : 'U'; @@ -95,7 +93,6 @@ class DibiSqliteDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Disconnects from a database. * @return void @@ -106,7 +103,6 @@ class DibiSqliteDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Executes the SQL query. * @param string SQL statement. @@ -134,7 +130,6 @@ class DibiSqliteDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Gets the number of affected rows by the last INSERT, UPDATE or DELETE query. * @return int|FALSE number of rows or FALSE on error @@ -145,7 +140,6 @@ class DibiSqliteDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Retrieves the ID generated for an AUTO_INCREMENT column by the previous INSERT query. * @return int|FALSE int on success or FALSE on failure @@ -156,7 +150,6 @@ class DibiSqliteDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Begins a transaction (if supported). * @param string optional savepoint name @@ -169,7 +162,6 @@ class DibiSqliteDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Commits statements in a transaction. * @param string optional savepoint name @@ -182,7 +174,6 @@ class DibiSqliteDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Rollback changes in a transaction. * @param string optional savepoint name @@ -195,7 +186,6 @@ class DibiSqliteDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Returns the connection resource. * @return mixed @@ -206,7 +196,6 @@ class DibiSqliteDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Returns the connection reflector. * @return IDibiReflector @@ -217,7 +206,6 @@ class DibiSqliteDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Result set driver factory. * @param resource @@ -231,11 +219,9 @@ class DibiSqliteDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /********************* SQL ****************d*g**/ - /** * Encodes data for use in a SQL statement. * @param mixed value @@ -246,29 +232,28 @@ class DibiSqliteDriver extends DibiObject implements IDibiDriver, IDibiResultDri public function escape($value, $type) { switch ($type) { - case dibi::TEXT: - case dibi::BINARY: - return "'" . sqlite_escape_string($value) . "'"; + case dibi::TEXT: + case dibi::BINARY: + return "'" . sqlite_escape_string($value) . "'"; - case dibi::IDENTIFIER: - return '[' . strtr($value, '[]', ' ') . ']'; + case dibi::IDENTIFIER: + return '[' . strtr($value, '[]', ' ') . ']'; - case dibi::BOOL: - return $value ? 1 : 0; + case dibi::BOOL: + return $value ? 1 : 0; - case dibi::DATE: - return $value instanceof DateTime ? $value->format($this->fmtDate) : date($this->fmtDate, $value); + case dibi::DATE: + return $value instanceof DateTime ? $value->format($this->fmtDate) : date($this->fmtDate, $value); - case dibi::DATETIME: - return $value instanceof DateTime ? $value->format($this->fmtDateTime) : date($this->fmtDateTime, $value); + case dibi::DATETIME: + return $value instanceof DateTime ? $value->format($this->fmtDateTime) : date($this->fmtDateTime, $value); - default: - throw new InvalidArgumentException('Unsupported type.'); + default: + throw new InvalidArgumentException('Unsupported type.'); } } - /** * Encodes string for use in a LIKE statement. * @param string @@ -281,7 +266,6 @@ class DibiSqliteDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Decodes data from result set. * @param string value @@ -298,26 +282,21 @@ class DibiSqliteDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Injects LIMIT/OFFSET to the SQL query. - * @param string &$sql The SQL query that will be modified. - * @param int $limit - * @param int $offset * @return void */ - public function applyLimit(&$sql, $limit, $offset) + public function applyLimit(& $sql, $limit, $offset) { - if ($limit < 0 && $offset < 1) return; - $sql .= ' LIMIT ' . $limit . ($offset > 0 ? ' OFFSET ' . (int) $offset : ''); + if ($limit >= 0 || $offset > 0) { + $sql .= ' LIMIT ' . (int) $limit . ($offset > 0 ? ' OFFSET ' . (int) $offset : ''); + } } - /********************* result set ****************d*g**/ - /** * Returns the number of rows in a result set. * @return int @@ -331,7 +310,6 @@ class DibiSqliteDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Fetches the row at current position and moves the internal cursor to the next position. * @param bool TRUE for associative array, FALSE for numeric @@ -355,7 +333,6 @@ class DibiSqliteDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Moves cursor position without fetching row. * @param int the 0-based cursor pos to seek to @@ -371,7 +348,6 @@ class DibiSqliteDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Frees the resources allocated for this result set. * @return void @@ -382,7 +358,6 @@ class DibiSqliteDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Returns metadata for all columns in a result set. * @return array @@ -405,7 +380,6 @@ class DibiSqliteDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Returns the result set resource. * @return mixed @@ -416,11 +390,9 @@ class DibiSqliteDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /********************* user defined functions ****************d*g**/ - /** * Registers an user defined function for use in SQL statements. * @param string function name @@ -434,7 +406,6 @@ class DibiSqliteDriver extends DibiObject implements IDibiDriver, IDibiResultDri } - /** * Registers an aggregating user defined function for use in SQL statements. * @param string function name diff --git a/dibi/drivers/DibiSqliteReflector.php b/dibi/drivers/DibiSqliteReflector.php index abf9ee68..396a6299 100644 --- a/dibi/drivers/DibiSqliteReflector.php +++ b/dibi/drivers/DibiSqliteReflector.php @@ -23,14 +23,12 @@ class DibiSqliteReflector extends DibiObject implements IDibiReflector private $driver; - public function __construct(IDibiDriver $driver) { $this->driver = $driver; } - /** * Returns list of tables. * @return array @@ -51,7 +49,6 @@ class DibiSqliteReflector extends DibiObject implements IDibiReflector } - /** * Returns metadata for all columns in a table. * @param string @@ -87,7 +84,6 @@ class DibiSqliteReflector extends DibiObject implements IDibiReflector } - /** * Returns metadata for all indexes in a table. * @param string @@ -139,7 +135,6 @@ class DibiSqliteReflector extends DibiObject implements IDibiReflector } - /** * Returns metadata for all foreign keys in a table. * @param string diff --git a/dibi/libs/DibiConnection.php b/dibi/libs/DibiConnection.php index 8b124bca..e8d78ead 100644 --- a/dibi/libs/DibiConnection.php +++ b/dibi/libs/DibiConnection.php @@ -10,7 +10,6 @@ */ - /** * dibi connection. * @@ -45,7 +44,6 @@ class DibiConnection extends DibiObject private $substitutes; - /** * Connection options: (see driver-specific options too) * - lazy (bool) => if TRUE, connection will be established only when required @@ -139,7 +137,6 @@ class DibiConnection extends DibiObject } - /** * Automatically frees the resources allocated for this result set. * @return void @@ -151,7 +148,6 @@ class DibiConnection extends DibiObject } - /** * Connects to a database. * @return void @@ -171,7 +167,6 @@ class DibiConnection extends DibiObject } - /** * Disconnects from a database. * @return void @@ -183,7 +178,6 @@ class DibiConnection extends DibiObject } - /** * Returns TRUE when connection was established. * @return bool @@ -194,7 +188,6 @@ class DibiConnection extends DibiObject } - /** * Returns configuration variable. If no $key is passed, returns the entire array. * @see self::__construct @@ -216,7 +209,6 @@ class DibiConnection extends DibiObject } - /** * Apply configuration alias or default values. * @param array connect configuration @@ -224,10 +216,12 @@ class DibiConnection extends DibiObject * @param string alias key * @return void */ - public static function alias(&$config, $key, $alias) + public static function alias(& $config, $key, $alias) { $foo = & $config; - foreach (explode('|', $key) as $key) $foo = & $foo[$key]; + foreach (explode('|', $key) as $key) { + $foo = & $foo[$key]; + } if (!isset($foo) && isset($config[$alias])) { $foo = $config[$alias]; @@ -236,7 +230,6 @@ class DibiConnection extends DibiObject } - /** * Returns the driver and connects to a database in lazy mode. * @return IDibiDriver @@ -248,7 +241,6 @@ class DibiConnection extends DibiObject } - /** * Generates (translates) and executes SQL query. * @param array|mixed one or more arguments @@ -262,7 +254,6 @@ class DibiConnection extends DibiObject } - /** * Generates SQL query. * @param array|mixed one or more arguments @@ -276,7 +267,6 @@ class DibiConnection extends DibiObject } - /** * Generates and prints SQL query. * @param array|mixed one or more arguments @@ -300,7 +290,6 @@ class DibiConnection extends DibiObject } - /** * Generates (translates) and returns SQL query as DibiDataSource. * @param array|mixed one or more arguments @@ -314,7 +303,6 @@ class DibiConnection extends DibiObject } - /** * Generates SQL query. * @param array @@ -327,7 +315,6 @@ class DibiConnection extends DibiObject } - /** * Executes the SQL query. * @param string SQL statement. @@ -359,7 +346,6 @@ class DibiConnection extends DibiObject } - /** * Gets the number of affected rows by the last INSERT, UPDATE or DELETE query. * @return int number of rows @@ -369,12 +355,13 @@ class DibiConnection extends DibiObject { $this->connected || $this->connect(); $rows = $this->driver->getAffectedRows(); - if (!is_int($rows) || $rows < 0) throw new DibiException('Cannot retrieve number of affected rows.'); + if (!is_int($rows) || $rows < 0) { + throw new DibiException('Cannot retrieve number of affected rows.'); + } return $rows; } - /** * Gets the number of affected rows. Alias for getAffectedRows(). * @return int number of rows @@ -386,7 +373,6 @@ class DibiConnection extends DibiObject } - /** * Retrieves the ID generated for an AUTO_INCREMENT column by the previous INSERT query. * @param string optional sequence name @@ -397,12 +383,13 @@ class DibiConnection extends DibiObject { $this->connected || $this->connect(); $id = $this->driver->getInsertId($sequence); - if ($id < 1) throw new DibiException('Cannot retrieve last generated ID.'); + if ($id < 1) { + throw new DibiException('Cannot retrieve last generated ID.'); + } return (int) $id; } - /** * Retrieves the ID generated for an AUTO_INCREMENT column. Alias for getInsertId(). * @param string optional sequence name @@ -415,7 +402,6 @@ class DibiConnection extends DibiObject } - /** * Begins a transaction (if supported). * @param string optional savepoint name @@ -436,7 +422,6 @@ class DibiConnection extends DibiObject } - /** * Commits statements in a transaction. * @param string optional savepoint name @@ -457,7 +442,6 @@ class DibiConnection extends DibiObject } - /** * Rollback changes in a transaction. * @param string optional savepoint name @@ -478,7 +462,6 @@ class DibiConnection extends DibiObject } - /** * Result set factory. * @param IDibiResultDriver @@ -492,11 +475,9 @@ class DibiConnection extends DibiObject } - /********************* fluent SQL builders ****************d*g**/ - /** * @return DibiFluent */ @@ -506,7 +487,6 @@ class DibiConnection extends DibiObject } - /** * @param string column name * @return DibiFluent @@ -518,7 +498,6 @@ class DibiConnection extends DibiObject } - /** * @param string table * @param array @@ -533,7 +512,6 @@ class DibiConnection extends DibiObject } - /** * @param string table * @param array @@ -551,7 +529,6 @@ class DibiConnection extends DibiObject } - /** * @param string table * @return DibiFluent @@ -562,11 +539,9 @@ class DibiConnection extends DibiObject } - /********************* substitutions ****************d*g**/ - /** * Returns substitution hashmap. * @return DibiHashMap @@ -577,7 +552,6 @@ class DibiConnection extends DibiObject } - /** * Provides substitution. * @return string @@ -588,7 +562,6 @@ class DibiConnection extends DibiObject } - /** * Substitution callback. */ @@ -598,11 +571,9 @@ class DibiConnection extends DibiObject } - /********************* shortcuts ****************d*g**/ - /** * Executes SQL query and fetch result - shortcut for query() & fetch(). * @param array|mixed one or more arguments @@ -616,7 +587,6 @@ class DibiConnection extends DibiObject } - /** * Executes SQL query and fetch results - shortcut for query() & fetchAll(). * @param array|mixed one or more arguments @@ -630,7 +600,6 @@ class DibiConnection extends DibiObject } - /** * Executes SQL query and fetch first column - shortcut for query() & fetchSingle(). * @param array|mixed one or more arguments @@ -644,7 +613,6 @@ class DibiConnection extends DibiObject } - /** * Executes SQL query and fetch pairs - shortcut for query() & fetchPairs(). * @param array|mixed one or more arguments @@ -658,11 +626,9 @@ class DibiConnection extends DibiObject } - /********************* misc ****************d*g**/ - /** * Import SQL dump from file - extreme fast! * @param string filename @@ -698,7 +664,6 @@ class DibiConnection extends DibiObject } - /** * Gets a information about the current database. * @return DibiDatabaseInfo @@ -710,7 +675,6 @@ class DibiConnection extends DibiObject } - /** * Prevents unserialization. */ @@ -720,7 +684,6 @@ class DibiConnection extends DibiObject } - /** * Prevents serialization. */ diff --git a/dibi/libs/DibiDataSource.php b/dibi/libs/DibiDataSource.php index c8d3613b..413db2d4 100644 --- a/dibi/libs/DibiDataSource.php +++ b/dibi/libs/DibiDataSource.php @@ -10,7 +10,6 @@ */ - /** * Default implementation of IDataSource for dibi. * @@ -55,7 +54,6 @@ class DibiDataSource extends DibiObject implements IDataSource private $limit; - /** * @param string SQL command or table or view name, as data source * @param DibiConnection connection @@ -71,12 +69,11 @@ class DibiDataSource extends DibiObject implements IDataSource } - /** * Selects columns to query. * @param string|array column name or array of column names * @param string column alias - * @return DibiDataSource provides a fluent interface + * @return self */ public function select($col, $as = NULL) { @@ -90,11 +87,10 @@ class DibiDataSource extends DibiObject implements IDataSource } - /** * Adds conditions to query. * @param mixed conditions - * @return DibiDataSource provides a fluent interface + * @return self */ public function where($cond) { @@ -109,12 +105,11 @@ class DibiDataSource extends DibiObject implements IDataSource } - /** * Selects columns to order by. * @param string|array column name or array of column names * @param string sorting direction - * @return DibiDataSource provides a fluent interface + * @return self */ public function orderBy($row, $sorting = 'ASC') { @@ -128,12 +123,11 @@ class DibiDataSource extends DibiObject implements IDataSource } - /** * Limits number of rows. * @param int limit * @param int offset - * @return DibiDataSource provides a fluent interface + * @return self */ public function applyLimit($limit, $offset = NULL) { @@ -144,7 +138,6 @@ class DibiDataSource extends DibiObject implements IDataSource } - /** * Returns the dibi connection. * @return DibiConnection @@ -155,11 +148,9 @@ class DibiDataSource extends DibiObject implements IDataSource } - /********************* executing ****************d*g**/ - /** * Returns (and queries) DibiResult. * @return DibiResult @@ -173,7 +164,6 @@ class DibiDataSource extends DibiObject implements IDataSource } - /** * @return DibiResultIterator */ @@ -183,7 +173,6 @@ class DibiDataSource extends DibiObject implements IDataSource } - /** * Generates, executes SQL query and fetches the single row. * @return DibiRow|FALSE array on success, FALSE if no next record @@ -194,7 +183,6 @@ class DibiDataSource extends DibiObject implements IDataSource } - /** * Like fetch(), but returns only first field. * @return mixed value on success, FALSE if no next record @@ -205,7 +193,6 @@ class DibiDataSource extends DibiObject implements IDataSource } - /** * Fetches all records from table. * @return array @@ -216,7 +203,6 @@ class DibiDataSource extends DibiObject implements IDataSource } - /** * Fetches all records from table and returns associative tree. * @param string associative descriptor @@ -228,7 +214,6 @@ class DibiDataSource extends DibiObject implements IDataSource } - /** * Fetches all records from table like $key => $value pairs. * @param string associative key @@ -241,7 +226,6 @@ class DibiDataSource extends DibiObject implements IDataSource } - /** * Discards the internal cache. * @return void @@ -252,11 +236,9 @@ class DibiDataSource extends DibiObject implements IDataSource } - /********************* exporting ****************d*g**/ - /** * Returns this data source wrapped in DibiFluent object. * @return DibiFluent @@ -267,7 +249,6 @@ class DibiDataSource extends DibiObject implements IDataSource } - /** * Returns this data source wrapped in DibiDataSource object. * @return DibiDataSource @@ -278,7 +259,6 @@ class DibiDataSource extends DibiObject implements IDataSource } - /** * Returns SQL query. * @return string @@ -299,11 +279,9 @@ FROM %SQL', $this->sql, ' } - /********************* counting ****************d*g**/ - /** * Returns the number of rows in a given data source. * @return int @@ -321,7 +299,6 @@ FROM %SQL', $this->sql, ' } - /** * Returns the number of rows in a given data source. * @return int diff --git a/dibi/libs/DibiDatabaseInfo.php b/dibi/libs/DibiDatabaseInfo.php index 6da52509..3e0351a1 100644 --- a/dibi/libs/DibiDatabaseInfo.php +++ b/dibi/libs/DibiDatabaseInfo.php @@ -10,7 +10,6 @@ */ - /** * Reflection metadata class for a database. * @@ -33,7 +32,6 @@ class DibiDatabaseInfo extends DibiObject private $tables; - public function __construct(IDibiReflector $reflector, $name) { $this->reflector = $reflector; @@ -41,7 +39,6 @@ class DibiDatabaseInfo extends DibiObject } - /** * @return string */ @@ -51,7 +48,6 @@ class DibiDatabaseInfo extends DibiObject } - /** * @return array of DibiTableInfo */ @@ -62,7 +58,6 @@ class DibiDatabaseInfo extends DibiObject } - /** * @return array of string */ @@ -77,7 +72,6 @@ class DibiDatabaseInfo extends DibiObject } - /** * @param string * @return DibiTableInfo @@ -95,7 +89,6 @@ class DibiDatabaseInfo extends DibiObject } - /** * @param string * @return bool @@ -107,7 +100,6 @@ class DibiDatabaseInfo extends DibiObject } - /** * @return void */ @@ -124,8 +116,6 @@ class DibiDatabaseInfo extends DibiObject } - - /** * Reflection metadata class for a database table. * @@ -164,7 +154,6 @@ class DibiTableInfo extends DibiObject private $primaryKey; - public function __construct(IDibiReflector $reflector, array $info) { $this->reflector = $reflector; @@ -173,7 +162,6 @@ class DibiTableInfo extends DibiObject } - /** * @return string */ @@ -183,7 +171,6 @@ class DibiTableInfo extends DibiObject } - /** * @return bool */ @@ -193,7 +180,6 @@ class DibiTableInfo extends DibiObject } - /** * @return array of DibiColumnInfo */ @@ -204,7 +190,6 @@ class DibiTableInfo extends DibiObject } - /** * @return array of string */ @@ -219,7 +204,6 @@ class DibiTableInfo extends DibiObject } - /** * @param string * @return DibiColumnInfo @@ -237,7 +221,6 @@ class DibiTableInfo extends DibiObject } - /** * @param string * @return bool @@ -249,7 +232,6 @@ class DibiTableInfo extends DibiObject } - /** * @return array of DibiForeignKeyInfo */ @@ -260,7 +242,6 @@ class DibiTableInfo extends DibiObject } - /** * @return array of DibiIndexInfo */ @@ -271,7 +252,6 @@ class DibiTableInfo extends DibiObject } - /** * @return DibiIndexInfo */ @@ -282,7 +262,6 @@ class DibiTableInfo extends DibiObject } - /** * @return void */ @@ -297,7 +276,6 @@ class DibiTableInfo extends DibiObject } - /** * @return void */ @@ -319,7 +297,6 @@ class DibiTableInfo extends DibiObject } - /** * @return void */ @@ -331,8 +308,6 @@ class DibiTableInfo extends DibiObject } - - /** * Reflection metadata class for a result set. * @@ -354,14 +329,12 @@ class DibiResultInfo extends DibiObject private $names; - public function __construct(IDibiResultDriver $driver) { $this->driver = $driver; } - /** * @return array of DibiColumnInfo */ @@ -372,7 +345,6 @@ class DibiResultInfo extends DibiObject } - /** * @param bool * @return array of string @@ -388,7 +360,6 @@ class DibiResultInfo extends DibiObject } - /** * @param string * @return DibiColumnInfo @@ -406,7 +377,6 @@ class DibiResultInfo extends DibiObject } - /** * @param string * @return bool @@ -418,7 +388,6 @@ class DibiResultInfo extends DibiObject } - /** * @return void */ @@ -436,8 +405,6 @@ class DibiResultInfo extends DibiObject } - - /** * Reflection metadata class for a table or result set column. * @@ -467,7 +434,6 @@ class DibiColumnInfo extends DibiObject private $info; - public function __construct(IDibiReflector $reflector = NULL, array $info) { $this->reflector = $reflector; @@ -475,7 +441,6 @@ class DibiColumnInfo extends DibiObject } - /** * @return string */ @@ -485,7 +450,6 @@ class DibiColumnInfo extends DibiObject } - /** * @return string */ @@ -495,7 +459,6 @@ class DibiColumnInfo extends DibiObject } - /** * @return bool */ @@ -505,7 +468,6 @@ class DibiColumnInfo extends DibiObject } - /** * @return DibiTableInfo */ @@ -518,7 +480,6 @@ class DibiColumnInfo extends DibiObject } - /** * @return string */ @@ -528,7 +489,6 @@ class DibiColumnInfo extends DibiObject } - /** * @return string */ @@ -538,7 +498,6 @@ class DibiColumnInfo extends DibiObject } - /** * @return mixed */ @@ -548,7 +507,6 @@ class DibiColumnInfo extends DibiObject } - /** * @return int */ @@ -558,7 +516,6 @@ class DibiColumnInfo extends DibiObject } - /** * @return bool */ @@ -568,7 +525,6 @@ class DibiColumnInfo extends DibiObject } - /** * @return bool */ @@ -578,7 +534,6 @@ class DibiColumnInfo extends DibiObject } - /** * @return bool */ @@ -588,7 +543,6 @@ class DibiColumnInfo extends DibiObject } - /** * @return mixed */ @@ -598,7 +552,6 @@ class DibiColumnInfo extends DibiObject } - /** * @param string * @return mixed @@ -609,7 +562,6 @@ class DibiColumnInfo extends DibiObject } - /** * Heuristic type detection. * @param string @@ -639,7 +591,6 @@ class DibiColumnInfo extends DibiObject } - /** * @internal */ @@ -654,8 +605,6 @@ class DibiColumnInfo extends DibiObject } - - /** * Reflection metadata class for a foreign key. * @@ -675,7 +624,6 @@ class DibiForeignKeyInfo extends DibiObject private $references; - public function __construct($name, array $references) { $this->name = $name; @@ -683,7 +631,6 @@ class DibiForeignKeyInfo extends DibiObject } - /** * @return string */ @@ -693,7 +640,6 @@ class DibiForeignKeyInfo extends DibiObject } - /** * @return array */ @@ -705,8 +651,6 @@ class DibiForeignKeyInfo extends DibiObject } - - /** * Reflection metadata class for a index or primary key. * @@ -730,7 +674,6 @@ class DibiIndexInfo extends DibiObject } - /** * @return string */ @@ -740,7 +683,6 @@ class DibiIndexInfo extends DibiObject } - /** * @return array */ @@ -750,7 +692,6 @@ class DibiIndexInfo extends DibiObject } - /** * @return bool */ @@ -760,7 +701,6 @@ class DibiIndexInfo extends DibiObject } - /** * @return bool */ diff --git a/dibi/libs/DibiDateTime.php b/dibi/libs/DibiDateTime.php index 0a4362e4..df75552a 100644 --- a/dibi/libs/DibiDateTime.php +++ b/dibi/libs/DibiDateTime.php @@ -10,7 +10,6 @@ */ - /** * DateTime with serialization and timestamp support for PHP 5.2. * @@ -33,7 +32,6 @@ class DibiDateTime extends DateTime } - public function modifyClone($modify = '') { $dolly = clone($this); @@ -41,7 +39,6 @@ class DibiDateTime extends DateTime } - public function modify($modify) { parent::modify($modify); @@ -49,7 +46,6 @@ class DibiDateTime extends DateTime } - public function __sleep() { $this->fix = array($this->format('Y-m-d H:i:s'), $this->getTimezone()->getName()); @@ -57,7 +53,6 @@ class DibiDateTime extends DateTime } - public function __wakeup() { $this->__construct($this->fix[0], new DateTimeZone($this->fix[1])); @@ -65,21 +60,18 @@ class DibiDateTime extends DateTime } - public function getTimestamp() { return (int) $this->format('U'); } - public function setTimestamp($timestamp) { return $this->__construct(date('Y-m-d H:i:s', $timestamp), new DateTimeZone($this->getTimezone()->getName())); // getTimeZone() crashes in PHP 5.2.6 } - public function __toString() { return $this->format('Y-m-d H:i:s'); diff --git a/dibi/libs/DibiEvent.php b/dibi/libs/DibiEvent.php index 9c5e6091..1e4b9e06 100644 --- a/dibi/libs/DibiEvent.php +++ b/dibi/libs/DibiEvent.php @@ -10,7 +10,6 @@ */ - /** * Profiler & logger event. * @@ -54,7 +53,6 @@ class DibiEvent public $source; - public function __construct(DibiConnection $connection, $type, $sql = NULL) { $this->connection = $connection; @@ -85,7 +83,6 @@ class DibiEvent } - public function done($result = NULL) { $this->result = $result; diff --git a/dibi/libs/DibiException.php b/dibi/libs/DibiException.php index b4c44035..a62853f3 100644 --- a/dibi/libs/DibiException.php +++ b/dibi/libs/DibiException.php @@ -10,7 +10,6 @@ */ - /** * dibi common exception. * @@ -36,7 +35,6 @@ class DibiException extends Exception } - /** * @return string The SQL passed to the constructor */ @@ -46,7 +44,6 @@ class DibiException extends Exception } - /** * @return string string represenation of exception with SQL command */ @@ -58,8 +55,6 @@ class DibiException extends Exception } - - /** * database server exception. * @@ -72,12 +67,10 @@ class DibiDriverException extends DibiException /********************* error catching ****************d*g**/ - /** @var string */ private static $errorMsg; - /** * Starts catching potential errors/warnings. * @return void @@ -89,7 +82,6 @@ class DibiDriverException extends DibiException } - /** * Returns catched error/warning message. * @param string catched message @@ -104,7 +96,6 @@ class DibiDriverException extends DibiException } - /** * Internal error handler. Do not call directly. * @internal @@ -124,8 +115,6 @@ class DibiDriverException extends DibiException } - - /** * PCRE exception. * @@ -149,7 +138,6 @@ class DibiPcreException extends Exception { } - /** * @package dibi */ @@ -157,7 +145,6 @@ class DibiNotImplementedException extends DibiException {} - /** * @package dibi */ diff --git a/dibi/libs/DibiFileLogger.php b/dibi/libs/DibiFileLogger.php index 49be5d90..d77e03c0 100644 --- a/dibi/libs/DibiFileLogger.php +++ b/dibi/libs/DibiFileLogger.php @@ -10,7 +10,6 @@ */ - /** * dibi file logger. * @@ -26,7 +25,6 @@ class DibiFileLogger extends DibiObject public $filter; - public function __construct($file, $filter = NULL) { $this->file = $file; @@ -34,7 +32,6 @@ class DibiFileLogger extends DibiObject } - /** * After event notification. * @return void @@ -46,7 +43,9 @@ class DibiFileLogger extends DibiObject } $handle = fopen($this->file, 'a'); - if (!$handle) return; // or throw exception? + if (!$handle) { + return; // or throw exception? + } flock($handle, LOCK_EX); if ($event->result instanceof Exception) { diff --git a/dibi/libs/DibiFirePhpLogger.php b/dibi/libs/DibiFirePhpLogger.php index 18dfd3c8..bf10ad74 100644 --- a/dibi/libs/DibiFirePhpLogger.php +++ b/dibi/libs/DibiFirePhpLogger.php @@ -10,7 +10,6 @@ */ - /** * dibi FirePHP logger. * @@ -38,7 +37,6 @@ class DibiFirePhpLogger extends DibiObject private static $fireTable = array(array('Time', 'SQL Statement', 'Rows', 'Connection')); - /** * @return bool */ @@ -48,14 +46,12 @@ class DibiFirePhpLogger extends DibiObject } - public function __construct($filter = NULL) { $this->filter = $filter ? (int) $filter : DibiEvent::QUERY; } - /** * After event notification. * @return void diff --git a/dibi/libs/DibiFluent.php b/dibi/libs/DibiFluent.php index 07887cb8..1d63a9e3 100644 --- a/dibi/libs/DibiFluent.php +++ b/dibi/libs/DibiFluent.php @@ -10,7 +10,6 @@ */ - /** * dibi SQL builder via fluent interfaces. EXPERIMENTAL! * @@ -101,7 +100,6 @@ class DibiFluent extends DibiObject implements IDataSource private static $normalizer; - /** * @param DibiConnection */ @@ -115,12 +113,11 @@ class DibiFluent extends DibiObject implements IDataSource } - /** * Appends new argument to the clause. * @param string clause name * @param array arguments - * @return DibiFluent provides a fluent interface + * @return self */ public function __call($clause, $args) { @@ -205,11 +202,10 @@ class DibiFluent extends DibiObject implements IDataSource } - /** * Switch to a clause. * @param string clause name - * @return DibiFluent provides a fluent interface + * @return self */ public function clause($clause, $remove = FALSE) { @@ -227,11 +223,10 @@ class DibiFluent extends DibiObject implements IDataSource } - /** * Removes a clause. * @param string clause name - * @return DibiFluent provides a fluent interface + * @return self */ public function removeClause($clause) { @@ -240,12 +235,11 @@ class DibiFluent extends DibiObject implements IDataSource } - /** * Change a SQL flag. * @param string flag name * @param bool value - * @return DibiFluent provides a fluent interface + * @return self */ public function setFlag($flag, $value = TRUE) { @@ -259,7 +253,6 @@ class DibiFluent extends DibiObject implements IDataSource } - /** * Is a flag set? * @param string flag name @@ -271,7 +264,6 @@ class DibiFluent extends DibiObject implements IDataSource } - /** * Returns SQL command. * @return string @@ -282,7 +274,6 @@ class DibiFluent extends DibiObject implements IDataSource } - /** * Returns the dibi connection. * @return DibiConnection @@ -293,12 +284,11 @@ class DibiFluent extends DibiObject implements IDataSource } - /** * Adds DibiResult setup. * @param string method * @param mixed args - * @return DibiFluent provides a fluent interface + * @return self */ public function setupResult($method) { @@ -307,11 +297,9 @@ class DibiFluent extends DibiObject implements IDataSource } - /********************* executing ****************d*g**/ - /** * Generates and executes SQL query. * @param mixed what to return? @@ -325,7 +313,6 @@ class DibiFluent extends DibiObject implements IDataSource } - /** * Generates, executes SQL query and fetches the single row. * @return DibiRow|FALSE array on success, FALSE if no next record @@ -340,7 +327,6 @@ class DibiFluent extends DibiObject implements IDataSource } - /** * Like fetch(), but returns only first field. * @return mixed value on success, FALSE if no next record @@ -355,7 +341,6 @@ class DibiFluent extends DibiObject implements IDataSource } - /** * Fetches all records from table. * @param int offset @@ -368,7 +353,6 @@ class DibiFluent extends DibiObject implements IDataSource } - /** * Fetches all records from table and returns associative tree. * @param string associative descriptor @@ -380,7 +364,6 @@ class DibiFluent extends DibiObject implements IDataSource } - /** * Fetches all records from table like $key => $value pairs. * @param string associative key @@ -393,7 +376,6 @@ class DibiFluent extends DibiObject implements IDataSource } - /** * Required by the IteratorAggregate interface. * @param int offset @@ -406,7 +388,6 @@ class DibiFluent extends DibiObject implements IDataSource } - /** * Generates and prints SQL query or it's part. * @param string clause name @@ -418,7 +399,6 @@ class DibiFluent extends DibiObject implements IDataSource } - /** * @return int */ @@ -430,7 +410,6 @@ class DibiFluent extends DibiObject implements IDataSource } - /** * @return DibiResult */ @@ -444,11 +423,9 @@ class DibiFluent extends DibiObject implements IDataSource } - /********************* exporting ****************d*g**/ - /** * @return DibiDataSource */ @@ -458,7 +435,6 @@ class DibiFluent extends DibiObject implements IDataSource } - /** * Returns SQL query. * @return string @@ -473,7 +449,6 @@ class DibiFluent extends DibiObject implements IDataSource } - /** * Generates parameters for DibiTranslator. * @param string clause name @@ -499,7 +474,9 @@ class DibiFluent extends DibiObject implements IDataSource if ($clause === $this->command && $this->flags) { $args[] = implode(' ', array_keys($this->flags)); } - foreach ($statement as $arg) $args[] = $arg; + foreach ($statement as $arg) { + $args[] = $arg; + } } } @@ -507,7 +484,6 @@ class DibiFluent extends DibiObject implements IDataSource } - /** * Format camelCase clause name to UPPER CASE. * @param string @@ -524,7 +500,6 @@ class DibiFluent extends DibiObject implements IDataSource } - public function __clone() { // remove references diff --git a/dibi/libs/DibiHashMap.php b/dibi/libs/DibiHashMap.php index 3ecfbf38..4dd7b98b 100644 --- a/dibi/libs/DibiHashMap.php +++ b/dibi/libs/DibiHashMap.php @@ -10,7 +10,6 @@ */ - /** * Lazy cached storage. * @@ -29,7 +28,6 @@ abstract class DibiHashMapBase } - public function setCallback($callback) { if (!is_callable($callback)) { @@ -40,7 +38,6 @@ abstract class DibiHashMapBase } - public function getCallback() { return $this->callback; @@ -49,7 +46,6 @@ abstract class DibiHashMapBase } - /** * Lazy cached storage. * @@ -68,7 +64,6 @@ final class DibiHashMap extends DibiHashMapBase } - public function __get($nm) { if ($nm == '') { diff --git a/dibi/libs/DibiLiteral.php b/dibi/libs/DibiLiteral.php index 94d91179..ee10d04a 100644 --- a/dibi/libs/DibiLiteral.php +++ b/dibi/libs/DibiLiteral.php @@ -10,7 +10,6 @@ */ - /** * SQL literal value. * @@ -28,7 +27,6 @@ class DibiLiteral extends DibiObject } - /** * @return string */ diff --git a/dibi/libs/DibiObject.php b/dibi/libs/DibiObject.php index d9541ea2..ad42de18 100644 --- a/dibi/libs/DibiObject.php +++ b/dibi/libs/DibiObject.php @@ -10,7 +10,6 @@ */ - /** * DibiObject is the ultimate ancestor of all instantiable classes. * @@ -58,7 +57,6 @@ abstract class DibiObject private static $extMethods; - /** * Returns the name of the class of this object. * @return string @@ -69,7 +67,6 @@ abstract class DibiObject } - /** * Access to reflection. * @return \ReflectionObject @@ -80,7 +77,6 @@ abstract class DibiObject } - /** * Call to undefined method. * @param string method name @@ -125,7 +121,6 @@ abstract class DibiObject } - /** * Call to undefined static method. * @param string method name (in lower case!) @@ -140,7 +135,6 @@ abstract class DibiObject } - /** * Adding method to class. * @param string method name @@ -158,7 +152,9 @@ abstract class DibiObject self::$extMethods[$pair[1]][''] = NULL; } } - if ($name === NULL) return NULL; + if ($name === NULL) { + return NULL; + } } $name = strtolower($name); @@ -202,14 +198,13 @@ abstract class DibiObject } - /** * Returns property value. Do not call directly. * @param string property name * @return mixed property value * @throws \LogicException if the property is not defined. */ - public function &__get($name) + public function & __get($name) { $class = get_class($this); @@ -222,8 +217,8 @@ abstract class DibiObject $m = 'get' . $name; if (self::hasAccessor($class, $m)) { // ampersands: - // - uses &__get() because declaration should be forward compatible (e.g. with Nette\Web\Html) - // - doesn't call &$this->$m because user could bypass property setter by: $x = & $obj->property; $x = 'new value'; + // - uses & __get() because declaration should be forward compatible (e.g. with Nette\Web\Html) + // - doesn't call & $this->$m because user could bypass property setter by: $x = & $obj->property; $x = 'new value'; $val = $this->$m(); return $val; } @@ -239,7 +234,6 @@ abstract class DibiObject } - /** * Sets value of a property. Do not call directly. * @param string property name @@ -274,7 +268,6 @@ abstract class DibiObject } - /** * Is property defined? * @param string property name @@ -287,7 +280,6 @@ abstract class DibiObject } - /** * Access to undeclared property. * @param string property name @@ -301,7 +293,6 @@ abstract class DibiObject } - /** * Has property an accessor? * @param string class name diff --git a/dibi/libs/DibiResult.php b/dibi/libs/DibiResult.php index b43ebb86..8683f009 100644 --- a/dibi/libs/DibiResult.php +++ b/dibi/libs/DibiResult.php @@ -10,7 +10,6 @@ */ - /** * dibi result set. * @@ -61,7 +60,6 @@ class DibiResult extends DibiObject implements IDataSource private $formats = array(); - /** * @param IDibiResultDriver */ @@ -72,7 +70,6 @@ class DibiResult extends DibiObject implements IDataSource } - /** * @deprecated */ @@ -82,7 +79,6 @@ class DibiResult extends DibiObject implements IDataSource } - /** * Frees the resources allocated for this result set. * @return void @@ -96,7 +92,6 @@ class DibiResult extends DibiObject implements IDataSource } - /** * Safe access to property $driver. * @return IDibiResultDriver @@ -112,11 +107,9 @@ class DibiResult extends DibiObject implements IDataSource } - /********************* rows ****************d*g**/ - /** * Moves cursor position without fetching row. * @param int the 0-based cursor pos to seek to @@ -129,7 +122,6 @@ class DibiResult extends DibiObject implements IDataSource } - /** * Required by the Countable interface. * @return int @@ -140,7 +132,6 @@ class DibiResult extends DibiObject implements IDataSource } - /** * Returns the number of rows in a result set. * @return int @@ -151,7 +142,6 @@ class DibiResult extends DibiObject implements IDataSource } - /** * Returns the number of rows in a result set. Alias for getRowCount(). * @deprecated @@ -163,7 +153,6 @@ class DibiResult extends DibiObject implements IDataSource } - /** * Required by the IteratorAggregate interface. * @return DibiResultIterator @@ -177,15 +166,13 @@ class DibiResult extends DibiObject implements IDataSource } - /********************* fetching rows ****************d*g**/ - /** * Set fetched object class. This class should extend the DibiRow class. * @param string - * @return DibiResult provides a fluent interface + * @return self */ public function setRowClass($class) { @@ -194,7 +181,6 @@ class DibiResult extends DibiObject implements IDataSource } - /** * Returns fetched object class name. * @return string @@ -205,11 +191,10 @@ class DibiResult extends DibiObject implements IDataSource } - /** * Set a factory to create fetched object instances. These should extend the DibiRow class. * @param callback - * @return DibiResult provides a fluent interface + * @return self */ public function setRowFactory($callback) { @@ -218,7 +203,6 @@ class DibiResult extends DibiObject implements IDataSource } - /** * Fetches the row at current position, process optional type conversion. * and moves the internal cursor to the next position @@ -241,7 +225,6 @@ class DibiResult extends DibiObject implements IDataSource } - /** * Like fetch(), but returns only first field. * @return mixed value on success, FALSE if no next record @@ -258,7 +241,6 @@ class DibiResult extends DibiObject implements IDataSource } - /** * Fetches all records from table. * @param int offset @@ -270,11 +252,15 @@ class DibiResult extends DibiObject implements IDataSource $limit = $limit === NULL ? -1 : (int) $limit; $this->seek((int) $offset); $row = $this->fetch(); - if (!$row) return array(); // empty result set + if (!$row) { + return array(); // empty result set + } $data = array(); do { - if ($limit === 0) break; + if ($limit === 0) { + break; + } $limit--; $data[] = $row; } while ($row = $this->fetch()); @@ -283,7 +269,6 @@ class DibiResult extends DibiObject implements IDataSource } - /** * Fetches all records from table and returns associative tree. * Examples: @@ -303,7 +288,9 @@ class DibiResult extends DibiObject implements IDataSource $this->seek(0); $row = $this->fetch(); - if (!$row) return array(); // empty result set + if (!$row) { + return array(); // empty result set + } $data = NULL; $assoc = preg_split('#(\[\]|->|=|\|)#', $assoc, NULL, PREG_SPLIT_DELIM_CAPTURE | PREG_SPLIT_NO_EMPTY); @@ -362,7 +349,6 @@ class DibiResult extends DibiObject implements IDataSource } - /** * @deprecated */ @@ -370,7 +356,9 @@ class DibiResult extends DibiObject implements IDataSource { $this->seek(0); $row = $this->fetch(); - if (!$row) return array(); // empty result set + if (!$row) { + return array(); // empty result set + } $data = NULL; $assoc = explode(',', $assoc); @@ -435,7 +423,6 @@ class DibiResult extends DibiObject implements IDataSource } - /** * Fetches all records from table like $key => $value pairs. * @param string associative key @@ -447,7 +434,9 @@ class DibiResult extends DibiObject implements IDataSource { $this->seek(0); $row = $this->fetch(); - if (!$row) return array(); // empty result set + if (!$row) { + return array(); // empty result set + } $data = array(); @@ -493,11 +482,9 @@ class DibiResult extends DibiObject implements IDataSource } - /********************* column types ****************d*g**/ - /** * Autodetect column types. * @return void @@ -513,7 +500,6 @@ class DibiResult extends DibiObject implements IDataSource } - /** * Converts values to specified type and format. * @param array @@ -561,12 +547,11 @@ class DibiResult extends DibiObject implements IDataSource } - /** * Define column type. * @param string column * @param string type (use constant Dibi::*) - * @return DibiResult provides a fluent interface + * @return self */ final public function setType($col, $type) { @@ -575,7 +560,6 @@ class DibiResult extends DibiObject implements IDataSource } - /** * Returns column type. * @return string @@ -586,12 +570,11 @@ class DibiResult extends DibiObject implements IDataSource } - /** * Sets data format. * @param string type (use constant Dibi::*) * @param string format - * @return DibiResult provides a fluent interface + * @return self */ final public function setFormat($type, $format) { @@ -600,7 +583,6 @@ class DibiResult extends DibiObject implements IDataSource } - /** * Returns data format. * @return string @@ -611,11 +593,9 @@ class DibiResult extends DibiObject implements IDataSource } - /********************* meta info ****************d*g**/ - /** * Returns a meta information about the current result set. * @return DibiResultInfo @@ -629,7 +609,6 @@ class DibiResult extends DibiObject implements IDataSource } - /** * @deprecated */ @@ -639,7 +618,6 @@ class DibiResult extends DibiObject implements IDataSource } - /** @deprecated */ public function getColumnNames($fullNames = FALSE) { @@ -648,11 +626,9 @@ class DibiResult extends DibiObject implements IDataSource } - /********************* misc tools ****************d*g**/ - /** * Displays complete result set as HTML or text table for debug purposes. * @return void @@ -668,7 +644,7 @@ class DibiResult extends DibiObject implements IDataSource if ($i === 0) { foreach ($row as $col => $foo) { $len = mb_strlen($col); - if ($len > $maxLen) $maxLen = $len; + $maxLen = max($len, $maxLen); } } @@ -687,7 +663,9 @@ class DibiResult extends DibiObject implements IDataSource $i++; } - if ($i === 0) echo "empty result set\n"; + if ($i === 0) { + echo "empty result set\n"; + } echo "\n"; } else { diff --git a/dibi/libs/DibiResultIterator.php b/dibi/libs/DibiResultIterator.php index 8331cd2f..4caf7522 100644 --- a/dibi/libs/DibiResultIterator.php +++ b/dibi/libs/DibiResultIterator.php @@ -10,7 +10,6 @@ */ - /** * External result set iterator. * @@ -47,7 +46,6 @@ class DibiResultIterator implements Iterator, Countable } - /** * Rewinds the iterator to the first element. * @return void @@ -60,7 +58,6 @@ class DibiResultIterator implements Iterator, Countable } - /** * Returns the key of the current element. * @return mixed @@ -71,7 +68,6 @@ class DibiResultIterator implements Iterator, Countable } - /** * Returns the current element. * @return mixed @@ -82,7 +78,6 @@ class DibiResultIterator implements Iterator, Countable } - /** * Moves forward to next element. * @return void @@ -94,7 +89,6 @@ class DibiResultIterator implements Iterator, Countable } - /** * Checks if there is a current element after calls to rewind() or next(). * @return bool @@ -105,7 +99,6 @@ class DibiResultIterator implements Iterator, Countable } - /** * Required by the Countable interface. * @return int diff --git a/dibi/libs/DibiRow.php b/dibi/libs/DibiRow.php index 5ea30430..19a8bc6d 100644 --- a/dibi/libs/DibiRow.php +++ b/dibi/libs/DibiRow.php @@ -10,7 +10,6 @@ */ - /** * Result set single row. * @@ -22,18 +21,18 @@ class DibiRow implements ArrayAccess, IteratorAggregate, Countable public function __construct($arr) { - foreach ($arr as $k => $v) $this->$k = $v; + foreach ($arr as $k => $v) { + $this->$k = $v; + } } - public function toArray() { return (array) $this; } - /** * Converts value to DateTime object. * @param string key @@ -53,7 +52,6 @@ class DibiRow implements ArrayAccess, IteratorAggregate, Countable } - /** * Converts value to UNIX timestamp. * @param string key @@ -69,7 +67,6 @@ class DibiRow implements ArrayAccess, IteratorAggregate, Countable } - /** * Converts value to boolean. * @param string key @@ -82,7 +79,6 @@ class DibiRow implements ArrayAccess, IteratorAggregate, Countable } - /** @deprecated */ public function asDate($key, $format = NULL) { @@ -95,46 +91,39 @@ class DibiRow implements ArrayAccess, IteratorAggregate, Countable } - /********************* interfaces ArrayAccess, Countable & IteratorAggregate ****************d*g**/ - final public function count() { return count((array) $this); } - final public function getIterator() { return new ArrayIterator($this); } - final public function offsetSet($nm, $val) { $this->$nm = $val; } - final public function offsetGet($nm) { return $this->$nm; } - final public function offsetExists($nm) { return isset($this->$nm); } - final public function offsetUnset($nm) { unset($this->$nm); diff --git a/dibi/libs/DibiTranslator.php b/dibi/libs/DibiTranslator.php index d5a7ec34..51d5dc1f 100644 --- a/dibi/libs/DibiTranslator.php +++ b/dibi/libs/DibiTranslator.php @@ -10,7 +10,6 @@ */ - /** * dibi SQL translator. * @@ -53,14 +52,12 @@ final class DibiTranslator extends DibiObject private $identifiers; - public function __construct(DibiConnection $connection) { $this->connection = $connection; } - /** * Generates SQL. * @param array @@ -94,8 +91,7 @@ final class DibiTranslator extends DibiObject // iterate $sql = array(); - while ($cursor < count($this->args)) - { + while ($cursor < count($this->args)) { $arg = $this->args[$cursor]; $cursor++; @@ -126,7 +122,9 @@ final class DibiTranslator extends DibiObject array($this, 'cb'), substr($arg, $toSkip) ); - if (preg_last_error()) throw new DibiPcreException; + if (preg_last_error()) { + throw new DibiPcreException; + } } continue; } @@ -148,7 +146,9 @@ final class DibiTranslator extends DibiObject $commandIns = $commandIns === 'INSERT' || $commandIns === 'REPLAC'; $sql[] = $this->formatValue($arg, $commandIns ? 'v' : 'a'); } else { - if ($lastArr === $cursor - 1) $sql[] = ','; + if ($lastArr === $cursor - 1) { + $sql[] = ','; + } $sql[] = $this->formatValue($arg, $commandIns ? 'l' : 'a'); } $lastArr = $cursor; @@ -161,7 +161,9 @@ final class DibiTranslator extends DibiObject } // while - if ($comment) $sql[] = "*/"; + if ($comment) { + $sql[] = "*/"; + } $sql = implode(' ', $sql); @@ -178,7 +180,6 @@ final class DibiTranslator extends DibiObject } - /** * Apply modifier to single value. * @param mixed @@ -199,131 +200,131 @@ final class DibiTranslator extends DibiObject if (is_array($value)) { $vx = $kx = array(); switch ($modifier) { - case 'and': - case 'or': // key=val AND key IS NULL AND ... - if (empty($value)) { - return '1=1'; - } + case 'and': + case 'or': // key=val AND key IS NULL AND ... + if (empty($value)) { + return '1=1'; + } - foreach ($value as $k => $v) { - if (is_string($k)) { - $pair = explode('%', $k, 2); // split into identifier & modifier - $k = $this->identifiers->{$pair[0]} . ' '; - if (!isset($pair[1])) { - $v = $this->formatValue($v, FALSE); - $vx[] = $k . ($v === 'NULL' ? 'IS ' : '= ') . $v; + foreach ($value as $k => $v) { + if (is_string($k)) { + $pair = explode('%', $k, 2); // split into identifier & modifier + $k = $this->identifiers->{$pair[0]} . ' '; + if (!isset($pair[1])) { + $v = $this->formatValue($v, FALSE); + $vx[] = $k . ($v === 'NULL' ? 'IS ' : '= ') . $v; - } elseif ($pair[1] === 'ex') { // TODO: this will be removed - $vx[] = $k . $this->formatValue($v, 'ex'); + } elseif ($pair[1] === 'ex') { // TODO: this will be removed + $vx[] = $k . $this->formatValue($v, 'ex'); - } else { - $v = $this->formatValue($v, $pair[1]); - if ($pair[1] === 'l' || $pair[1] === 'in') { - $op = 'IN '; - } elseif (strpos($pair[1], 'like') !== FALSE) { - $op = 'LIKE '; - } elseif ($v === 'NULL') { - $op = 'IS '; } else { - $op = '= '; + $v = $this->formatValue($v, $pair[1]); + if ($pair[1] === 'l' || $pair[1] === 'in') { + $op = 'IN '; + } elseif (strpos($pair[1], 'like') !== FALSE) { + $op = 'LIKE '; + } elseif ($v === 'NULL') { + $op = 'IS '; + } else { + $op = '= '; + } + $vx[] = $k . $op . $v; } - $vx[] = $k . $op . $v; + + } else { + $vx[] = $this->formatValue($v, 'ex'); } - - } else { - $vx[] = $this->formatValue($v, 'ex'); } - } - return '(' . implode(') ' . strtoupper($modifier) . ' (', $vx) . ')'; + return '(' . implode(') ' . strtoupper($modifier) . ' (', $vx) . ')'; - case 'n': // key, key, ... identifier names - foreach ($value as $k => $v) { - if (is_string($k)) { - $vx[] = $this->identifiers->$k . (empty($v) ? '' : ' AS ' . $this->identifiers->$v); - } else { - $pair = explode('%', $v, 2); // split into identifier & modifier - $vx[] = $this->identifiers->{$pair[0]}; + case 'n': // key, key, ... identifier names + foreach ($value as $k => $v) { + if (is_string($k)) { + $vx[] = $this->identifiers->$k . (empty($v) ? '' : ' AS ' . $this->identifiers->$v); + } else { + $pair = explode('%', $v, 2); // split into identifier & modifier + $vx[] = $this->identifiers->{$pair[0]}; + } } - } - return implode(', ', $vx); + return implode(', ', $vx); - case 'a': // key=val, key=val, ... - foreach ($value as $k => $v) { - $pair = explode('%', $k, 2); // split into identifier & modifier - $vx[] = $this->identifiers->{$pair[0]} . '=' - . $this->formatValue($v, isset($pair[1]) ? $pair[1] : (is_array($v) ? 'ex' : FALSE)); - } - return implode(', ', $vx); + case 'a': // key=val, key=val, ... + foreach ($value as $k => $v) { + $pair = explode('%', $k, 2); // split into identifier & modifier + $vx[] = $this->identifiers->{$pair[0]} . '=' + . $this->formatValue($v, isset($pair[1]) ? $pair[1] : (is_array($v) ? 'ex' : FALSE)); + } + return implode(', ', $vx); - case 'in':// replaces scalar %in modifier! - case 'l': // (val, val, ...) - foreach ($value as $k => $v) { - $pair = explode('%', $k, 2); // split into identifier & modifier - $vx[] = $this->formatValue($v, isset($pair[1]) ? $pair[1] : (is_array($v) ? 'ex' : FALSE)); - } - return '(' . (($vx || $modifier === 'l') ? implode(', ', $vx) : 'NULL') . ')'; + case 'in':// replaces scalar %in modifier! + case 'l': // (val, val, ...) + foreach ($value as $k => $v) { + $pair = explode('%', $k, 2); // split into identifier & modifier + $vx[] = $this->formatValue($v, isset($pair[1]) ? $pair[1] : (is_array($v) ? 'ex' : FALSE)); + } + return '(' . (($vx || $modifier === 'l') ? implode(', ', $vx) : 'NULL') . ')'; - case 'v': // (key, key, ...) VALUES (val, val, ...) - foreach ($value as $k => $v) { - $pair = explode('%', $k, 2); // split into identifier & modifier - $kx[] = $this->identifiers->{$pair[0]}; - $vx[] = $this->formatValue($v, isset($pair[1]) ? $pair[1] : (is_array($v) ? 'ex' : FALSE)); - } - return '(' . implode(', ', $kx) . ') VALUES (' . implode(', ', $vx) . ')'; + case 'v': // (key, key, ...) VALUES (val, val, ...) + foreach ($value as $k => $v) { + $pair = explode('%', $k, 2); // split into identifier & modifier + $kx[] = $this->identifiers->{$pair[0]}; + $vx[] = $this->formatValue($v, isset($pair[1]) ? $pair[1] : (is_array($v) ? 'ex' : FALSE)); + } + return '(' . implode(', ', $kx) . ') VALUES (' . implode(', ', $vx) . ')'; - case 'm': // (key, key, ...) VALUES (val, val, ...), (val, val, ...), ... - foreach ($value as $k => $v) { - if (is_array($v)) { - if (isset($proto)) { - if ($proto !== array_keys($v)) { - $this->hasError = TRUE; - return '**Multi-insert array "' . $k . '" is different.**'; + case 'm': // (key, key, ...) VALUES (val, val, ...), (val, val, ...), ... + foreach ($value as $k => $v) { + if (is_array($v)) { + if (isset($proto)) { + if ($proto !== array_keys($v)) { + $this->hasError = TRUE; + return '**Multi-insert array "' . $k . '" is different.**'; + } + } else { + $proto = array_keys($v); } } else { - $proto = array_keys($v); + $this->hasError = TRUE; + return '**Unexpected type ' . gettype($v) . '**'; + } + + $pair = explode('%', $k, 2); // split into identifier & modifier + $kx[] = $this->identifiers->{$pair[0]}; + foreach ($v as $k2 => $v2) { + $vx[$k2][] = $this->formatValue($v2, isset($pair[1]) ? $pair[1] : (is_array($v2) ? 'ex' : FALSE)); } - } else { - $this->hasError = TRUE; - return '**Unexpected type ' . gettype($v) . '**'; } - - $pair = explode('%', $k, 2); // split into identifier & modifier - $kx[] = $this->identifiers->{$pair[0]}; - foreach ($v as $k2 => $v2) { - $vx[$k2][] = $this->formatValue($v2, isset($pair[1]) ? $pair[1] : (is_array($v2) ? 'ex' : FALSE)); + foreach ($vx as $k => $v) { + $vx[$k] = '(' . implode(', ', $v) . ')'; } - } - foreach ($vx as $k => $v) { - $vx[$k] = '(' . implode(', ', $v) . ')'; - } - return '(' . implode(', ', $kx) . ') VALUES ' . implode(', ', $vx); + return '(' . implode(', ', $kx) . ') VALUES ' . implode(', ', $vx); - case 'by': // key ASC, key DESC - foreach ($value as $k => $v) { - if (is_array($v)) { - $vx[] = $this->formatValue($v, 'ex'); - } elseif (is_string($k)) { - $v = (is_string($v) && strncasecmp($v, 'd', 1)) || $v > 0 ? 'ASC' : 'DESC'; - $vx[] = $this->identifiers->$k . ' ' . $v; - } else { - $vx[] = $this->identifiers->$v; + case 'by': // key ASC, key DESC + foreach ($value as $k => $v) { + if (is_array($v)) { + $vx[] = $this->formatValue($v, 'ex'); + } elseif (is_string($k)) { + $v = (is_string($v) && strncasecmp($v, 'd', 1)) || $v > 0 ? 'ASC' : 'DESC'; + $vx[] = $this->identifiers->$k . ' ' . $v; + } else { + $vx[] = $this->identifiers->$v; + } } - } - return implode(', ', $vx); + return implode(', ', $vx); - case 'ex': - case 'sql': - $translator = new self($this->connection); - return $translator->translate($value); + case 'ex': + case 'sql': + $translator = new self($this->connection); + return $translator->translate($value); - default: // value, value, value - all with the same modifier - foreach ($value as $v) { - $vx[] = $this->formatValue($v, $modifier); - } - return implode(', ', $vx); + default: // value, value, value - all with the same modifier + foreach ($value as $v) { + $vx[] = $this->formatValue($v, $modifier); + } + return implode(', ', $vx); } } @@ -336,94 +337,98 @@ final class DibiTranslator extends DibiObject } switch ($modifier) { - case 's': // string - case 'bin':// binary - case 'b': // boolean - return $value === NULL ? 'NULL' : $this->driver->escape($value, $modifier); + case 's': // string + case 'bin':// binary + case 'b': // boolean + return $value === NULL ? 'NULL' : $this->driver->escape($value, $modifier); - case 'sN': // string or NULL - case 'sn': - return $value == '' ? 'NULL' : $this->driver->escape($value, dibi::TEXT); // notice two equal signs + case 'sN': // string or NULL + case 'sn': + return $value == '' ? 'NULL' : $this->driver->escape($value, dibi::TEXT); // notice two equal signs - case 'iN': // signed int or NULL - case 'in': // deprecated - if ($value == '') $value = NULL; - // intentionally break omitted - - case 'i': // signed int - case 'u': // unsigned int, ignored - // support for long numbers - keep them unchanged - if (is_string($value) && preg_match('#[+-]?\d++(e\d+)?\z#A', $value)) { - return $value; - } else { - return $value === NULL ? 'NULL' : (string) (int) ($value + 0); - } - - case 'f': // float - // support for extreme numbers - keep them unchanged - if (is_string($value) && is_numeric($value) && strpos($value, 'x') === FALSE) { - return $value; // something like -9E-005 is accepted by SQL, HEX values are not - } else { - return $value === NULL ? 'NULL' : rtrim(rtrim(number_format($value + 0, 10, '.', ''), '0'), '.'); - } - - case 'd': // date - case 't': // datetime - if ($value === NULL) { - return 'NULL'; - } else { - if (is_numeric($value)) { - $value = (int) $value; // timestamp - - } elseif (is_string($value)) { - $value = new DateTime($value); + case 'iN': // signed int or NULL + case 'in': // deprecated + if ($value == '') { + $value = NULL; } - return $this->driver->escape($value, $modifier); - } + // intentionally break omitted - case 'by': - case 'n': // identifier name - return $this->identifiers->$value; + case 'i': // signed int + case 'u': // unsigned int, ignored + // support for long numbers - keep them unchanged + if (is_string($value) && preg_match('#[+-]?\d++(e\d+)?\z#A', $value)) { + return $value; + } else { + return $value === NULL ? 'NULL' : (string) (int) ($value + 0); + } - case 'ex': - case 'sql': // preserve as dibi-SQL (TODO: leave only %ex) - $value = (string) $value; - // speed-up - is regexp required? - $toSkip = strcspn($value, '`[\'":'); - if (strlen($value) !== $toSkip) { - $value = substr($value, 0, $toSkip) - . preg_replace_callback( - '/(?=[`[\'":])(?:`(.+?)`|\[(.+?)\]|(\')((?:\'\'|[^\'])*)\'|(")((?:""|[^"])*)"|(\'|")|:(\S*?:)([a-zA-Z0-9._]?))/s', - array($this, 'cb'), - substr($value, $toSkip) - ); - if (preg_last_error()) throw new DibiPcreException; - } - return $value; + case 'f': // float + // support for extreme numbers - keep them unchanged + if (is_string($value) && is_numeric($value) && strpos($value, 'x') === FALSE) { + return $value; // something like -9E-005 is accepted by SQL, HEX values are not + } else { + return $value === NULL ? 'NULL' : rtrim(rtrim(number_format($value + 0, 10, '.', ''), '0'), '.'); + } - case 'SQL': // preserve as real SQL (TODO: rename to %sql) - return (string) $value; + case 'd': // date + case 't': // datetime + if ($value === NULL) { + return 'NULL'; + } else { + if (is_numeric($value)) { + $value = (int) $value; // timestamp - case 'like~': // LIKE string% - return $this->driver->escapeLike($value, 1); + } elseif (is_string($value)) { + $value = new DateTime($value); + } + return $this->driver->escape($value, $modifier); + } - case '~like': // LIKE %string - return $this->driver->escapeLike($value, -1); + case 'by': + case 'n': // identifier name + return $this->identifiers->$value; - case '~like~': // LIKE %string% - return $this->driver->escapeLike($value, 0); + case 'ex': + case 'sql': // preserve as dibi-SQL (TODO: leave only %ex) + $value = (string) $value; + // speed-up - is regexp required? + $toSkip = strcspn($value, '`[\'":'); + if (strlen($value) !== $toSkip) { + $value = substr($value, 0, $toSkip) + . preg_replace_callback( + '/(?=[`[\'":])(?:`(.+?)`|\[(.+?)\]|(\')((?:\'\'|[^\'])*)\'|(")((?:""|[^"])*)"|(\'|")|:(\S*?:)([a-zA-Z0-9._]?))/s', + array($this, 'cb'), + substr($value, $toSkip) + ); + if (preg_last_error()) { + throw new DibiPcreException; + } + } + return $value; - case 'and': - case 'or': - case 'a': - case 'l': - case 'v': - $this->hasError = TRUE; - return '**Unexpected type ' . gettype($value) . '**'; + case 'SQL': // preserve as real SQL (TODO: rename to %sql) + return (string) $value; - default: - $this->hasError = TRUE; - return "**Unknown or invalid modifier %$modifier**"; + case 'like~': // LIKE string% + return $this->driver->escapeLike($value, 1); + + case '~like': // LIKE %string + return $this->driver->escapeLike($value, -1); + + case '~like~': // LIKE %string% + return $this->driver->escapeLike($value, 0); + + case 'and': + case 'or': + case 'a': + case 'l': + case 'v': + $this->hasError = TRUE; + return '**Unexpected type ' . gettype($value) . '**'; + + default: + $this->hasError = TRUE; + return "**Unknown or invalid modifier %$modifier**"; } } @@ -457,7 +462,6 @@ final class DibiTranslator extends DibiObject } - /** * PREG callback from translate() or formatValue(). * @param array @@ -536,12 +540,16 @@ final class DibiTranslator extends DibiObject return ''; } elseif ($mod === 'lmt') { // apply limit - if ($this->args[$cursor] !== NULL) $this->limit = (int) $this->args[$cursor]; + if ($this->args[$cursor] !== NULL) { + $this->limit = (int) $this->args[$cursor]; + } $cursor++; return ''; } elseif ($mod === 'ofs') { // apply offset - if ($this->args[$cursor] !== NULL) $this->offset = (int) $this->args[$cursor]; + if ($this->args[$cursor] !== NULL) { + $this->offset = (int) $this->args[$cursor]; + } $cursor++; return ''; @@ -551,21 +559,23 @@ final class DibiTranslator extends DibiObject } } - if ($this->comment) return '...'; + if ($this->comment) { + return '...'; + } - if ($matches[1]) // SQL identifiers: `ident` + if ($matches[1]) { // SQL identifiers: `ident` return $this->identifiers->{$matches[1]}; - if ($matches[2]) // SQL identifiers: [ident] + } elseif ($matches[2]) { // SQL identifiers: [ident] return $this->identifiers->{$matches[2]}; - if ($matches[3]) // SQL strings: '...' + } elseif ($matches[3]) { // SQL strings: '...' return $this->driver->escape( str_replace("''", "'", $matches[4]), dibi::TEXT); - if ($matches[5]) // SQL strings: "..." + } elseif ($matches[5]) { // SQL strings: "..." return $this->driver->escape( str_replace('""', '"', $matches[6]), dibi::TEXT); - if ($matches[7]) { // string quote + } elseif ($matches[7]) { // string quote $this->hasError = TRUE; return '**Alone quote**'; } @@ -580,7 +590,6 @@ final class DibiTranslator extends DibiObject } - /** * Apply substitutions to indentifier and delimites it. * @param string indentifier @@ -592,7 +601,9 @@ final class DibiTranslator extends DibiObject $value = $this->connection->substitute($value); $parts = explode('.', $value); foreach ($parts as & $v) { - if ($v !== '*') $v = $this->driver->escape($v, dibi::IDENTIFIER); + if ($v !== '*') { + $v = $this->driver->escape($v, dibi::IDENTIFIER); + } } return implode('.', $parts); } diff --git a/dibi/libs/interfaces.php b/dibi/libs/interfaces.php index 79c4d01f..20880d76 100644 --- a/dibi/libs/interfaces.php +++ b/dibi/libs/interfaces.php @@ -10,7 +10,6 @@ */ - /** * Provides an interface between a dataset and data-aware components. * @package dibi @@ -22,7 +21,6 @@ interface IDataSource extends Countable, IteratorAggregate } - /** * dibi driver interface. * @package dibi @@ -36,7 +34,7 @@ interface IDibiDriver * @return void * @throws DibiException */ - function connect(array &$config); + function connect(array & $config); /** * Disconnects from a database. @@ -120,19 +118,13 @@ interface IDibiDriver /** * Injects LIMIT/OFFSET to the SQL query. - * @param string &$sql The SQL query that will be modified. - * @param int $limit - * @param int $offset * @return void */ - function applyLimit(&$sql, $limit, $offset); + function applyLimit(& $sql, $limit, $offset); } - - - /** * dibi result set driver interface. * @package dibi @@ -193,9 +185,6 @@ interface IDibiResultDriver } - - - /** * dibi driver reflection. * diff --git a/examples/Nette/Debugger.php b/examples/Nette/Debugger.php index 9e314309..9ffe168b 100644 --- a/examples/Nette/Debugger.php +++ b/examples/Nette/Debugger.php @@ -23,7 +23,6 @@ render(){$obLevel=ob_get_level();$panels=array();foreach($this->panels as$id=>$panel){try{$panels[]=array('id'=>preg_replace('#[^a-z0-9]+#i','-',$id),'tab'=>$tab=(string)$panel->getTab(),'panel'=>$tab?(string)$panel->getPanel():NULL);}catch(Exception$e){$panels[]=array('id'=>"error-$id",'tab'=>"Error: $id",'panel'=>nl2br(htmlSpecialChars((string)$e)));while(ob_get_level()>$obLevel){ob_end_clean();}}}?> - @@ -130,7 +129,6 @@ htmlspecialchars('; caused by '.get_class($ex).' '.$ex->getMessage().($ex->getCo -
getMessage())
- @@ -165,7 +162,6 @@ htmlspecialchars($ex->getMessage())?>

- @@ -180,7 +176,6 @@ htmlSpecialChars($panel['tab'])?> ▼ - getTrace();$expanded=NULL?> getFile(),$expandPath)===0){foreach($stack as$key=>$row){if(isset($row['file'])&&strpos($row['file'],$expandPath)!==0){$expanded=$key;break;}}}?> @@ -196,7 +191,6 @@ self::highlightFile($ex->getFile(),$ex->getLine(),15,isset($ex->context)?$ex->co -
@@ -251,7 +245,6 @@ self::highlightFile($row['file'],$row['line'])?>
- context)&&is_array($ex->context)):?>

Variables ►

@@ -272,7 +265,6 @@ as$k=>$v){echo'$',htmlspecialchars($k),'',NDebugHelpers::clicka
'?> - @@ -288,7 +280,6 @@ htmlSpecialChars($panel['tab'])?> ► -

Environment ►

@@ -336,7 +327,6 @@ as$k=>$v)echo'',htmlspecialchars($k),'',NDebugHelpers::clickabl
-

HTTP request ►

@@ -371,7 +361,6 @@ foreach($GLOBALS[$name]as$k=>$v)echo'',htmlspecialchars($k),'',
-

HTTP response ►

@@ -388,7 +377,6 @@ htmlspecialchars($s),'
';?>
-
@@ -401,7 +389,6 @@ htmlSpecialChars($panel['tab'])?> ▼ - \n"; - // table reflection $table = $database->getTable('products'); @@ -40,7 +38,6 @@ foreach ($table->getColumns() as $column) { echo "\n"; - echo "Indexes"; echo "