@@ -86,7 +86,7 @@ |
||
| 86 | 86 | * @param string $columnName |
| 87 | 87 | * @param string $value |
| 88 | 88 | */ |
| 89 | - public function __construct($tableName=null, $columnName=null, $value=null) { |
|
| 89 | + public function __construct($tableName = null, $columnName = null, $value = null) { |
|
| 90 | 90 | $this->tableName = $tableName; |
| 91 | 91 | $this->columnName = $columnName; |
| 92 | 92 | $this->value = $value; |
@@ -59,7 +59,7 @@ |
||
| 59 | 59 | * |
| 60 | 60 | * @param FilterInterface $filter |
| 61 | 61 | */ |
| 62 | - public function __construct($filter=null) { |
|
| 62 | + public function __construct($filter = null) { |
|
| 63 | 63 | $this->filter = $filter; |
| 64 | 64 | } |
| 65 | 65 | |
@@ -62,7 +62,7 @@ discard block |
||
| 62 | 62 | * |
| 63 | 63 | * @param string $sqlString |
| 64 | 64 | */ |
| 65 | - public function __construct($sqlString=null) { |
|
| 65 | + public function __construct($sqlString = null) { |
|
| 66 | 66 | $this->sqlString = $sqlString; |
| 67 | 67 | } |
| 68 | 68 | |
@@ -95,32 +95,32 @@ discard block |
||
| 95 | 95 | // First, let's remove all the stuff in quotes: |
| 96 | 96 | |
| 97 | 97 | // Let's remove all the \' found |
| 98 | - $work_str = str_replace("\\'",'',$this->sqlString); |
|
| 98 | + $work_str = str_replace("\\'", '', $this->sqlString); |
|
| 99 | 99 | // Now, let's split the string using ' |
| 100 | 100 | $work_table = explode("'", $work_str); |
| 101 | 101 | |
| 102 | - if (count($work_table)==0) |
|
| 102 | + if (count($work_table) == 0) |
|
| 103 | 103 | return ''; |
| 104 | 104 | |
| 105 | 105 | // if we start with a ', let's remove the first text |
| 106 | - if (strstr($work_str,"'")===0) |
|
| 106 | + if (strstr($work_str, "'") === 0) |
|
| 107 | 107 | array_shift($work_table); |
| 108 | 108 | |
| 109 | - if (count($work_table)==0) |
|
| 109 | + if (count($work_table) == 0) |
|
| 110 | 110 | return ''; |
| 111 | 111 | |
| 112 | 112 | // Now, let's take only the stuff outside the quotes. |
| 113 | 113 | $work_str2 = ''; |
| 114 | 114 | |
| 115 | - $i=0; |
|
| 115 | + $i = 0; |
|
| 116 | 116 | foreach ($work_table as $str_fragment) { |
| 117 | - if (($i % 2) == 0) |
|
| 117 | + if (($i%2) == 0) |
|
| 118 | 118 | $work_str2 .= $str_fragment.' '; |
| 119 | 119 | $i++; |
| 120 | 120 | } |
| 121 | 121 | |
| 122 | 122 | // Now, let's run a regexp to find all the strings matching the pattern xxx.yyy |
| 123 | - preg_match_all('/([a-zA-Z_](?:[a-zA-Z0-9_]*))\.(?:[a-zA-Z_](?:[a-zA-Z0-9_]*))/', $work_str2,$capture_result); |
|
| 123 | + preg_match_all('/([a-zA-Z_](?:[a-zA-Z0-9_]*))\.(?:[a-zA-Z_](?:[a-zA-Z0-9_]*))/', $work_str2, $capture_result); |
|
| 124 | 124 | |
| 125 | 125 | $tables_used = $capture_result[1]; |
| 126 | 126 | // remove doubles: |
@@ -49,7 +49,7 @@ discard block |
||
| 49 | 49 | * |
| 50 | 50 | * @param array<FilterInterface> $filters |
| 51 | 51 | */ |
| 52 | - public function __construct(array $filters=array()) { |
|
| 52 | + public function __construct(array $filters = array()) { |
|
| 53 | 53 | $this->filters = $filters; |
| 54 | 54 | } |
| 55 | 55 | |
@@ -91,7 +91,7 @@ discard block |
||
| 91 | 91 | } |
| 92 | 92 | |
| 93 | 93 | if (count($filters_sql)>0) { |
| 94 | - return '('.implode(' AND ',$filters_sql).')'; |
|
| 94 | + return '('.implode(' AND ', $filters_sql).')'; |
|
| 95 | 95 | } else { |
| 96 | 96 | return ''; |
| 97 | 97 | } |
@@ -114,7 +114,7 @@ discard block |
||
| 114 | 114 | throw new TDBMException("Error in AndFilter: One of the parameters is not a filter."); |
| 115 | 115 | } |
| 116 | 116 | |
| 117 | - $tables = array_merge($tables,$filter->getUsedTables()); |
|
| 117 | + $tables = array_merge($tables, $filter->getUsedTables()); |
|
| 118 | 118 | } |
| 119 | 119 | // Remove tables in double. |
| 120 | 120 | $tables = array_flip(array_flip($tables)); |
@@ -101,7 +101,7 @@ |
||
| 101 | 101 | * @param string $value1 |
| 102 | 102 | * @param string $value2 |
| 103 | 103 | */ |
| 104 | - public function __construct($tableName=null, $columnName=null, $value1=null, $value2=null) { |
|
| 104 | + public function __construct($tableName = null, $columnName = null, $value1 = null, $value2 = null) { |
|
| 105 | 105 | $this->tableName = $tableName; |
| 106 | 106 | $this->columnName = $columnName; |
| 107 | 107 | $this->value1 = $value1; |
@@ -59,7 +59,7 @@ discard block |
||
| 59 | 59 | * |
| 60 | 60 | * @param array<FilterInterface> $filters |
| 61 | 61 | */ |
| 62 | - public function __construct(array $filters=array()) { |
|
| 62 | + public function __construct(array $filters = array()) { |
|
| 63 | 63 | $this->filters = $filters; |
| 64 | 64 | } |
| 65 | 65 | |
@@ -91,7 +91,7 @@ discard block |
||
| 91 | 91 | } |
| 92 | 92 | |
| 93 | 93 | if (count($filters_sql)>0) { |
| 94 | - return '('.implode(' OR ',$filters_sql).')'; |
|
| 94 | + return '('.implode(' OR ', $filters_sql).')'; |
|
| 95 | 95 | } else { |
| 96 | 96 | return ''; |
| 97 | 97 | } |
@@ -114,7 +114,7 @@ discard block |
||
| 114 | 114 | throw new TDBMException("Error in OrFilter: One of the parameters is not a filter."); |
| 115 | 115 | } |
| 116 | 116 | |
| 117 | - $tables = array_merge($tables,$filter->getUsedTables()); |
|
| 117 | + $tables = array_merge($tables, $filter->getUsedTables()); |
|
| 118 | 118 | } |
| 119 | 119 | // Remove tables in double. |
| 120 | 120 | $tables = array_flip(array_flip($tables)); |
@@ -84,7 +84,7 @@ |
||
| 84 | 84 | * @param string $columnName |
| 85 | 85 | * @param string $value |
| 86 | 86 | */ |
| 87 | - public function __construct($tableName=null, $columnName=null, $value=null) { |
|
| 87 | + public function __construct($tableName = null, $columnName = null, $value = null) { |
|
| 88 | 88 | $this->tableName = $tableName; |
| 89 | 89 | $this->columnName = $columnName; |
| 90 | 90 | $this->value = $value; |
@@ -85,7 +85,7 @@ |
||
| 85 | 85 | * @param string $columnName |
| 86 | 86 | * @param string $value |
| 87 | 87 | */ |
| 88 | - public function __construct($tableName=null, $columnName=null, $value=null) { |
|
| 88 | + public function __construct($tableName = null, $columnName = null, $value = null) { |
|
| 89 | 89 | $this->tableName = $tableName; |
| 90 | 90 | $this->columnName = $columnName; |
| 91 | 91 | $this->value = $value; |
@@ -95,7 +95,7 @@ discard block |
||
| 95 | 95 | * @throws TDBMException |
| 96 | 96 | * @throws TDBMInvalidOperationException |
| 97 | 97 | */ |
| 98 | - public function __construct($tableName=null, array $primaryKeys=array(), TDBMService $tdbmService=null) { |
|
| 98 | + public function __construct($tableName = null, array $primaryKeys = array(), TDBMService $tdbmService = null) { |
|
| 99 | 99 | // FIXME: lazy loading should be forbidden on tables with inheritance and dynamic type assignation... |
| 100 | 100 | if (!empty($tableName)) { |
| 101 | 101 | $this->dbRows[$tableName] = new DbRow($this, $tableName, $primaryKeys, $tdbmService); |
@@ -199,7 +199,7 @@ discard block |
||
| 199 | 199 | * $status = TDBMObjectStateEnum::STATE_LOADED when the object is cached in memory. |
| 200 | 200 | * @param string $state |
| 201 | 201 | */ |
| 202 | - public function _setStatus($state){ |
|
| 202 | + public function _setStatus($state) { |
|
| 203 | 203 | $this->status = $state; |
| 204 | 204 | |
| 205 | 205 | // TODO: we might ignore the loaded => dirty state here! dirty status comes from the db_row itself. |
@@ -210,7 +210,7 @@ discard block |
||
| 210 | 210 | |
| 211 | 211 | public function get($var, $tableName = null) { |
| 212 | 212 | if ($tableName === null) { |
| 213 | - if (count($this->dbRows) > 1) { |
|
| 213 | + if (count($this->dbRows)>1) { |
|
| 214 | 214 | throw new TDBMException('This object is based on several tables. You must specify which table you are retrieving data from.'); |
| 215 | 215 | } elseif (count($this->dbRows) === 1) { |
| 216 | 216 | $tableName = array_keys($this->dbRows)[0]; |
@@ -236,7 +236,7 @@ discard block |
||
| 236 | 236 | */ |
| 237 | 237 | public function has($var, $tableName = null) { |
| 238 | 238 | if ($tableName === null) { |
| 239 | - if (count($this->dbRows) > 1) { |
|
| 239 | + if (count($this->dbRows)>1) { |
|
| 240 | 240 | throw new TDBMException('This object is based on several tables. You must specify which table you are retrieving data from.'); |
| 241 | 241 | } elseif (count($this->dbRows) === 1) { |
| 242 | 242 | $tableName = array_keys($this->dbRows)[0]; |
@@ -256,7 +256,7 @@ discard block |
||
| 256 | 256 | |
| 257 | 257 | public function set($var, $value, $tableName = null) { |
| 258 | 258 | if ($tableName === null) { |
| 259 | - if (count($this->dbRows) > 1) { |
|
| 259 | + if (count($this->dbRows)>1) { |
|
| 260 | 260 | throw new TDBMException('This object is based on several tables. You must specify which table you are retrieving data from.'); |
| 261 | 261 | } elseif (count($this->dbRows) === 1) { |
| 262 | 262 | $tableName = array_keys($this->dbRows)[0]; |
@@ -281,7 +281,7 @@ discard block |
||
| 281 | 281 | */ |
| 282 | 282 | public function setRef($foreignKeyName, AbstractTDBMObject $bean, $tableName = null) { |
| 283 | 283 | if ($tableName === null) { |
| 284 | - if (count($this->dbRows) > 1) { |
|
| 284 | + if (count($this->dbRows)>1) { |
|
| 285 | 285 | throw new TDBMException('This object is based on several tables. You must specify which table you are retrieving data from.'); |
| 286 | 286 | } elseif (count($this->dbRows) === 1) { |
| 287 | 287 | $tableName = array_keys($this->dbRows)[0]; |
@@ -306,7 +306,7 @@ discard block |
||
| 306 | 306 | */ |
| 307 | 307 | public function getRef($foreignKeyName, $tableName = null) { |
| 308 | 308 | if ($tableName === null) { |
| 309 | - if (count($this->dbRows) > 1) { |
|
| 309 | + if (count($this->dbRows)>1) { |
|
| 310 | 310 | throw new TDBMException('This object is based on several tables. You must specify which table you are retrieving data from.'); |
| 311 | 311 | } elseif (count($this->dbRows) === 1) { |
| 312 | 312 | $tableName = array_keys($this->dbRows)[0]; |
@@ -421,13 +421,13 @@ discard block |
||
| 421 | 421 | */ |
| 422 | 422 | private function setRelationship($pivotTableName, AbstractTDBMObject $remoteBean, $status) { |
| 423 | 423 | $storage = $this->getRelationshipStorage($pivotTableName); |
| 424 | - $storage->attach($remoteBean, [ 'status' => $status, 'reverse' => false ]); |
|
| 424 | + $storage->attach($remoteBean, ['status' => $status, 'reverse' => false]); |
|
| 425 | 425 | if ($this->status === TDBMObjectStateEnum::STATE_LOADED) { |
| 426 | 426 | $this->_setStatus(TDBMObjectStateEnum::STATE_DIRTY); |
| 427 | 427 | } |
| 428 | 428 | |
| 429 | 429 | $remoteStorage = $remoteBean->getRelationshipStorage($pivotTableName); |
| 430 | - $remoteStorage->attach($this, [ 'status' => $status, 'reverse' => true ]); |
|
| 430 | + $remoteStorage->attach($this, ['status' => $status, 'reverse' => true]); |
|
| 431 | 431 | } |
| 432 | 432 | |
| 433 | 433 | /** |
@@ -496,7 +496,7 @@ discard block |
||
| 496 | 496 | * Implement the unique JsonSerializable method |
| 497 | 497 | * @return array |
| 498 | 498 | */ |
| 499 | - public function jsonSerialize(){ |
|
| 499 | + public function jsonSerialize() { |
|
| 500 | 500 | // FIXME |
| 501 | 501 | $this->_dbLoadIfNotLoaded(); |
| 502 | 502 | return $this->dbRow; |
@@ -526,23 +526,23 @@ discard block |
||
| 526 | 526 | * |
| 527 | 527 | * @return string |
| 528 | 528 | */ |
| 529 | - private function getPrimaryKeyWhereStatement () { |
|
| 529 | + private function getPrimaryKeyWhereStatement() { |
|
| 530 | 530 | // Let's first get the primary keys |
| 531 | 531 | $pk_table = $this->tdbmService->getPrimaryKeyColumns($this->dbTableName); |
| 532 | 532 | // Now for the object_id |
| 533 | 533 | $object_id = $this->TDBMObject_id; |
| 534 | 534 | // If there is only one primary key: |
| 535 | - if (count($pk_table)==1) { |
|
| 535 | + if (count($pk_table) == 1) { |
|
| 536 | 536 | $sql_where = $this->db_connection->escapeDBItem($this->dbTableName).'.'.$this->db_connection->escapeDBItem($pk_table[0])."=".$this->db_connection->quoteSmart($this->TDBMObject_id); |
| 537 | 537 | } else { |
| 538 | 538 | $ids = unserialize($object_id); |
| 539 | - $i=0; |
|
| 539 | + $i = 0; |
|
| 540 | 540 | $sql_where_array = array(); |
| 541 | 541 | foreach ($pk_table as $pk) { |
| 542 | 542 | $sql_where_array[] = $this->db_connection->escapeDBItem($this->dbTableName).'.'.$this->db_connection->escapeDBItem($pk)."=".$this->db_connection->quoteSmart($ids[$i]); |
| 543 | 543 | $i++; |
| 544 | 544 | } |
| 545 | - $sql_where = implode(" AND ",$sql_where_array); |
|
| 545 | + $sql_where = implode(" AND ", $sql_where_array); |
|
| 546 | 546 | } |
| 547 | 547 | return $sql_where; |
| 548 | 548 | } |
@@ -550,7 +550,7 @@ discard block |
||
| 550 | 550 | /** |
| 551 | 551 | * Override the native php clone function for TDBMObjects |
| 552 | 552 | */ |
| 553 | - public function __clone(){ |
|
| 553 | + public function __clone() { |
|
| 554 | 554 | $this->_dbLoadIfNotLoaded(); |
| 555 | 555 | //First lets set the status to new (to enter the save function) |
| 556 | 556 | $this->status = TDBMObjectStateEnum::STATE_NEW; |
@@ -577,7 +577,7 @@ discard block |
||
| 577 | 577 | private function registerTable($tableName) { |
| 578 | 578 | $dbRow = new DbRow($this, $tableName); |
| 579 | 579 | |
| 580 | - if (in_array($this->status, [ TDBMObjectStateEnum::STATE_NOT_LOADED, TDBMObjectStateEnum::STATE_LOADED, TDBMObjectStateEnum::STATE_DIRTY ])) { |
|
| 580 | + if (in_array($this->status, [TDBMObjectStateEnum::STATE_NOT_LOADED, TDBMObjectStateEnum::STATE_LOADED, TDBMObjectStateEnum::STATE_DIRTY])) { |
|
| 581 | 581 | // Let's get the primary key for the new table |
| 582 | 582 | $anotherDbRow = array_values($this->dbRows)[0]; |
| 583 | 583 | /* @var $anotherDbRow DbRow */ |