Test Failed
Pull Request — 4.2 (#140)
by David
04:46
created

TDBMService::getConnectionUniqueId()   A

Complexity

Conditions 1
Paths 1

Size

Total Lines 4
Code Lines 2

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
dl 0
loc 4
rs 10
c 0
b 0
f 0
cc 1
eloc 2
nc 1
nop 0
1
<?php
2
3
/*
4
 Copyright (C) 2006-2016 David Négrier - THE CODING MACHINE
5
6
This program is free software; you can redistribute it and/or modify
7
it under the terms of the GNU General Public License as published by
8
the Free Software Foundation; either version 2 of the License, or
9
(at your option) any later version.
10
11
This program is distributed in the hope that it will be useful,
12
but WITHOUT ANY WARRANTY; without even the implied warranty of
13
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
14
GNU General Public License for more details.
15
16
You should have received a copy of the GNU General Public License
17
along with this program; if not, write to the Free Software
18
Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
19
*/
20
21
namespace Mouf\Database\TDBM;
22
23
use Doctrine\Common\Cache\Cache;
24
use Doctrine\Common\Cache\VoidCache;
25
use Doctrine\DBAL\Connection;
26
use Doctrine\DBAL\Schema\Column;
27
use Doctrine\DBAL\Schema\ForeignKeyConstraint;
28
use Doctrine\DBAL\Schema\Schema;
29
use Doctrine\DBAL\Schema\Table;
30
use Doctrine\DBAL\Types\Type;
31
use Mouf\Database\MagicQuery;
32
use Mouf\Database\SchemaAnalyzer\SchemaAnalyzer;
33
use Mouf\Database\TDBM\QueryFactory\FindObjectsFromSqlQueryFactory;
34
use Mouf\Database\TDBM\QueryFactory\FindObjectsQueryFactory;
35
use Mouf\Database\TDBM\Utils\NamingStrategyInterface;
36
use Mouf\Database\TDBM\Utils\TDBMDaoGenerator;
37
use Phlib\Logger\LevelFilter;
38
use Psr\Log\LoggerInterface;
39
use Psr\Log\LogLevel;
40
use Psr\Log\NullLogger;
41
42
/**
43
 * The TDBMService class is the main TDBM class. It provides methods to retrieve TDBMObject instances
44
 * from the database.
45
 *
46
 * @author David Negrier
47
 * @ExtendedAction {"name":"Generate DAOs", "url":"tdbmadmin/", "default":false}
48
 */
49
class TDBMService
50
{
51
    const MODE_CURSOR = 1;
52
    const MODE_ARRAY = 2;
53
54
    /**
55
     * The database connection.
56
     *
57
     * @var Connection
58
     */
59
    private $connection;
60
61
    /**
62
     * @var SchemaAnalyzer
63
     */
64
    private $schemaAnalyzer;
65
66
    /**
67
     * @var MagicQuery
68
     */
69
    private $magicQuery;
70
71
    /**
72
     * @var TDBMSchemaAnalyzer
73
     */
74
    private $tdbmSchemaAnalyzer;
75
76
    /**
77
     * @var string
78
     */
79
    private $cachePrefix;
80
81
    /**
82
     * Cache of table of primary keys.
83
     * Primary keys are stored by tables, as an array of column.
84
     * For instance $primary_key['my_table'][0] will return the first column of the primary key of table 'my_table'.
85
     *
86
     * @var string[]
87
     */
88
    private $primaryKeysColumns;
89
90
    /**
91
     * Service storing objects in memory.
92
     * Access is done by table name and then by primary key.
93
     * If the primary key is split on several columns, access is done by an array of columns, serialized.
94
     *
95
     * @var StandardObjectStorage|WeakrefObjectStorage
96
     */
97
    private $objectStorage;
98
99
    /**
100
     * The fetch mode of the result sets returned by `getObjects`.
101
     * Can be one of: TDBMObjectArray::MODE_CURSOR or TDBMObjectArray::MODE_ARRAY or TDBMObjectArray::MODE_COMPATIBLE_ARRAY.
102
     *
103
     * In 'MODE_ARRAY' mode (default), the result is an array. Use this mode by default (unless the list returned is very big).
104
     * In 'MODE_CURSOR' mode, the result is a Generator which is an iterable collection that can be scanned only once (only one "foreach") on it,
105
     * and it cannot be accessed via key. Use this mode for large datasets processed by batch.
106
     * In 'MODE_COMPATIBLE_ARRAY' mode, the result is an old TDBMObjectArray (used up to TDBM 3.2).
107
     * You can access the array by key, or using foreach, several times.
108
     *
109
     * @var int
110
     */
111
    private $mode = self::MODE_ARRAY;
112
113
    /**
114
     * Table of new objects not yet inserted in database or objects modified that must be saved.
115
     *
116
     * @var \SplObjectStorage of DbRow objects
117
     */
118
    private $toSaveObjects;
119
120
    /**
121
     * A cache service to be used.
122
     *
123
     * @var Cache|null
124
     */
125
    private $cache;
126
127
    /**
128
     * Map associating a table name to a fully qualified Bean class name.
129
     *
130
     * @var array
131
     */
132
    private $tableToBeanMap = [];
133
134
    /**
135
     * @var \ReflectionClass[]
136
     */
137
    private $reflectionClassCache = array();
138
139
    /**
140
     * @var LoggerInterface
141
     */
142
    private $rootLogger;
143
144
    /**
145
     * @var LevelFilter|NullLogger
146
     */
147
    private $logger;
148
149
    /**
150
     * @var OrderByAnalyzer
151
     */
152
    private $orderByAnalyzer;
153
154
    /**
155
     * @var string
156
     */
157
    private $beanNamespace;
158
159
    /**
160
     * @var NamingStrategyInterface
161
     */
162
    private $namingStrategy;
163
    /**
164
     * @var ConfigurationInterface
165
     */
166
    private $configuration;
167
168
    /**
169
     * @param ConfigurationInterface $configuration The configuration object
170
     */
171
    public function __construct(ConfigurationInterface $configuration)
172
    {
173
        if (extension_loaded('weakref')) {
174
            $this->objectStorage = new WeakrefObjectStorage();
175
        } else {
176
            $this->objectStorage = new StandardObjectStorage();
177
        }
178
        $this->connection = $configuration->getConnection();
179
        $this->cache = $configuration->getCache();
180
        $this->schemaAnalyzer = $configuration->getSchemaAnalyzer();
181
182
        $this->magicQuery = new MagicQuery($this->connection, $this->cache, $this->schemaAnalyzer);
183
184
        $this->tdbmSchemaAnalyzer = new TDBMSchemaAnalyzer($this->connection, $this->cache, $this->schemaAnalyzer);
185
        $this->cachePrefix = $this->tdbmSchemaAnalyzer->getCachePrefix();
186
187
        $this->toSaveObjects = new \SplObjectStorage();
188
        $logger = $configuration->getLogger();
189
        if ($logger === null) {
190
            $this->logger = new NullLogger();
191
            $this->rootLogger = new NullLogger();
192
        } else {
193
            $this->rootLogger = $logger;
194
            $this->setLogLevel(LogLevel::WARNING);
195
        }
196
        $this->orderByAnalyzer = new OrderByAnalyzer($this->cache, $this->cachePrefix);
197
        $this->beanNamespace = $configuration->getBeanNamespace();
198
        $this->namingStrategy = $configuration->getNamingStrategy();
199
        $this->configuration = $configuration;
200
    }
201
202
    /**
203
     * Returns the object used to connect to the database.
204
     *
205
     * @return Connection
206
     */
207
    public function getConnection(): Connection
208
    {
209
        return $this->connection;
210
    }
211
212
    /**
213
     * Sets the default fetch mode of the result sets returned by `findObjects`.
214
     * Can be one of: TDBMObjectArray::MODE_CURSOR or TDBMObjectArray::MODE_ARRAY.
215
     *
216
     * In 'MODE_ARRAY' mode (default), the result is a ResultIterator object that behaves like an array. Use this mode by default (unless the list returned is very big).
217
     * In 'MODE_CURSOR' mode, the result is a ResultIterator object. If you scan it many times (by calling several time a foreach loop), the query will be run
218
     * several times. In cursor mode, you cannot access the result set by key. Use this mode for large datasets processed by batch.
219
     *
220
     * @param int $mode
221
     *
222
     * @return $this
223
     *
224
     * @throws TDBMException
225
     */
226
    public function setFetchMode($mode)
227
    {
228
        if ($mode !== self::MODE_CURSOR && $mode !== self::MODE_ARRAY) {
229
            throw new TDBMException("Unknown fetch mode: '".$this->mode."'");
230
        }
231
        $this->mode = $mode;
232
233
        return $this;
234
    }
235
236
    /**
237
     * Removes the given object from database.
238
     * This cannot be called on an object that is not attached to this TDBMService
239
     * (will throw a TDBMInvalidOperationException).
240
     *
241
     * @param AbstractTDBMObject $object the object to delete
242
     *
243
     * @throws TDBMException
244
     * @throws TDBMInvalidOperationException
245
     */
246
    public function delete(AbstractTDBMObject $object)
247
    {
248
        switch ($object->_getStatus()) {
249
            case TDBMObjectStateEnum::STATE_DELETED:
250
                // Nothing to do, object already deleted.
251
                return;
252
            case TDBMObjectStateEnum::STATE_DETACHED:
253
                throw new TDBMInvalidOperationException('Cannot delete a detached object');
254
            case TDBMObjectStateEnum::STATE_NEW:
255
                $this->deleteManyToManyRelationships($object);
256
                foreach ($object->_getDbRows() as $dbRow) {
257
                    $this->removeFromToSaveObjectList($dbRow);
258
                }
259
                break;
260
            case TDBMObjectStateEnum::STATE_DIRTY:
261
                foreach ($object->_getDbRows() as $dbRow) {
262
                    $this->removeFromToSaveObjectList($dbRow);
263
                }
264
                // And continue deleting...
0 ignored issues
show
Unused Code Comprehensibility introduced by
43% of this comment could be valid code. Did you maybe forget this after debugging?

Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it.

The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production.

This check looks for comments that seem to be mostly valid code and reports them.

Loading history...
265
            case TDBMObjectStateEnum::STATE_NOT_LOADED:
266
            case TDBMObjectStateEnum::STATE_LOADED:
267
                $this->deleteManyToManyRelationships($object);
268
                // Let's delete db rows, in reverse order.
269
                foreach (array_reverse($object->_getDbRows()) as $dbRow) {
270
                    $tableName = $dbRow->_getDbTableName();
271
                    $primaryKeys = $dbRow->_getPrimaryKeys();
272
                    $this->connection->delete($tableName, $primaryKeys);
273
                    $this->objectStorage->remove($dbRow->_getDbTableName(), $this->getObjectHash($primaryKeys));
274
                }
275
                break;
276
            // @codeCoverageIgnoreStart
277
            default:
278
                throw new TDBMInvalidOperationException('Unexpected status for bean');
279
            // @codeCoverageIgnoreEnd
280
        }
281
282
        $object->_setStatus(TDBMObjectStateEnum::STATE_DELETED);
283
    }
284
285
    /**
286
     * Removes all many to many relationships for this object.
287
     *
288
     * @param AbstractTDBMObject $object
289
     */
290
    private function deleteManyToManyRelationships(AbstractTDBMObject $object)
291
    {
292
        foreach ($object->_getDbRows() as $tableName => $dbRow) {
293
            $pivotTables = $this->tdbmSchemaAnalyzer->getPivotTableLinkedToTable($tableName);
294
            foreach ($pivotTables as $pivotTable) {
295
                $remoteBeans = $object->_getRelationships($pivotTable);
296
                foreach ($remoteBeans as $remoteBean) {
297
                    $object->_removeRelationship($pivotTable, $remoteBean);
298
                }
299
            }
300
        }
301
        $this->persistManyToManyRelationships($object);
302
    }
303
304
    /**
305
     * This function removes the given object from the database. It will also remove all objects relied to the one given
306
     * by parameter before all.
307
     *
308
     * Notice: if the object has a multiple primary key, the function will not work.
309
     *
310
     * @param AbstractTDBMObject $objToDelete
311
     */
312
    public function deleteCascade(AbstractTDBMObject $objToDelete)
313
    {
314
        $this->deleteAllConstraintWithThisObject($objToDelete);
315
        $this->delete($objToDelete);
316
    }
317
318
    /**
319
     * This function is used only in TDBMService (private function)
320
     * It will call deleteCascade function foreach object relied with a foreign key to the object given by parameter.
321
     *
322
     * @param AbstractTDBMObject $obj
323
     */
324
    private function deleteAllConstraintWithThisObject(AbstractTDBMObject $obj)
325
    {
326
        $dbRows = $obj->_getDbRows();
327
        foreach ($dbRows as $dbRow) {
328
            $tableName = $dbRow->_getDbTableName();
329
            $pks = array_values($dbRow->_getPrimaryKeys());
330
            if (!empty($pks)) {
331
                $incomingFks = $this->tdbmSchemaAnalyzer->getIncomingForeignKeys($tableName);
332
333
                foreach ($incomingFks as $incomingFk) {
334
                    $filter = array_combine($incomingFk->getLocalColumns(), $pks);
335
336
                    $results = $this->findObjects($incomingFk->getLocalTableName(), $filter);
337
338
                    foreach ($results as $bean) {
339
                        $this->deleteCascade($bean);
340
                    }
341
                }
342
            }
343
        }
344
    }
345
346
    /**
347
     * This function performs a save() of all the objects that have been modified.
348
     */
349
    public function completeSave()
350
    {
351
        foreach ($this->toSaveObjects as $dbRow) {
352
            $this->save($dbRow->getTDBMObject());
353
        }
354
    }
355
356
    /**
357
     * Takes in input a filter_bag (which can be about anything from a string to an array of TDBMObjects... see above from documentation),
358
     * and gives back a proper Filter object.
359
     *
360
     * @param mixed $filter_bag
361
     * @param int   $counter
362
     *
363
     * @return array First item: filter string, second item: parameters
364
     *
365
     * @throws TDBMException
366
     */
367
    public function buildFilterFromFilterBag($filter_bag, $counter = 1)
368
    {
369
        if ($filter_bag === null) {
370
            return ['', []];
371
        } elseif (is_string($filter_bag)) {
372
            return [$filter_bag, []];
373
        } elseif (is_array($filter_bag)) {
374
            $sqlParts = [];
375
            $parameters = [];
376
            foreach ($filter_bag as $column => $value) {
377
                if (is_int($column)) {
378
                    list($subSqlPart, $subParameters) = $this->buildFilterFromFilterBag($value, $counter);
379
                    $sqlParts[] = $subSqlPart;
380
                    $parameters += $subParameters;
381
                } else {
382
                    $paramName = 'tdbmparam'.$counter;
383
                    if (is_array($value)) {
384
                        $sqlParts[] = $this->connection->quoteIdentifier($column).' IN :'.$paramName;
385
                    } else {
386
                        $sqlParts[] = $this->connection->quoteIdentifier($column).' = :'.$paramName;
387
                    }
388
                    $parameters[$paramName] = $value;
389
                    ++$counter;
390
                }
391
            }
392
393
            return [implode(' AND ', $sqlParts), $parameters];
394
        } elseif ($filter_bag instanceof AbstractTDBMObject) {
395
            $sqlParts = [];
396
            $parameters = [];
397
            $dbRows = $filter_bag->_getDbRows();
398
            $dbRow = reset($dbRows);
399
            $primaryKeys = $dbRow->_getPrimaryKeys();
400
401
            foreach ($primaryKeys as $column => $value) {
402
                $paramName = 'tdbmparam'.$counter;
403
                $sqlParts[] = $this->connection->quoteIdentifier($dbRow->_getDbTableName()).'.'.$this->connection->quoteIdentifier($column).' = :'.$paramName;
404
                $parameters[$paramName] = $value;
405
                ++$counter;
406
            }
407
408
            return [implode(' AND ', $sqlParts), $parameters];
409
        } elseif ($filter_bag instanceof \Iterator) {
410
            return $this->buildFilterFromFilterBag(iterator_to_array($filter_bag), $counter);
411
        } else {
412
            throw new TDBMException('Error in filter. An object has been passed that is neither a SQL string, nor an array, nor a bean, nor null.');
413
        }
414
    }
415
416
    /**
417
     * @param string $table
418
     *
419
     * @return string[]
420
     */
421
    public function getPrimaryKeyColumns($table)
422
    {
423
        if (!isset($this->primaryKeysColumns[$table])) {
424
            $this->primaryKeysColumns[$table] = $this->tdbmSchemaAnalyzer->getSchema()->getTable($table)->getPrimaryKeyColumns();
425
426
            // TODO TDBM4: See if we need to improve error reporting if table name does not exist.
427
428
            /*$arr = array();
429
            foreach ($this->connection->getPrimaryKey($table) as $col) {
430
                $arr[] = $col->name;
431
            }
432
            // The primaryKeysColumns contains only the column's name, not the DB_Column object.
433
            $this->primaryKeysColumns[$table] = $arr;
434
            if (empty($this->primaryKeysColumns[$table]))
435
            {
436
                // Unable to find primary key.... this is an error
437
                // Let's try to be precise in error reporting. Let's try to find the table.
438
                $tables = $this->connection->checkTableExist($table);
439
                if ($tables === true)
440
                throw new TDBMException("Could not find table primary key for table '$table'. Please define a primary key for this table.");
441
                elseif ($tables !== null) {
442
                    if (count($tables)==1)
443
                    $str = "Could not find table '$table'. Maybe you meant this table: '".$tables[0]."'";
444
                    else
445
                    $str = "Could not find table '$table'. Maybe you meant one of those tables: '".implode("', '",$tables)."'";
446
                    throw new TDBMException($str);
447
                }
448
            }*/
449
        }
450
451
        return $this->primaryKeysColumns[$table];
452
    }
453
454
    /**
455
     * This is an internal function, you should not use it in your application.
456
     * This is used internally by TDBM to add an object to the object cache.
457
     *
458
     * @param DbRow $dbRow
459
     */
460
    public function _addToCache(DbRow $dbRow)
461
    {
462
        $primaryKey = $this->getPrimaryKeysForObjectFromDbRow($dbRow);
463
        $hash = $this->getObjectHash($primaryKey);
464
        $this->objectStorage->set($dbRow->_getDbTableName(), $hash, $dbRow);
465
    }
466
467
    /**
468
     * This is an internal function, you should not use it in your application.
469
     * This is used internally by TDBM to remove the object from the list of objects that have been
470
     * created/updated but not saved yet.
471
     *
472
     * @param DbRow $myObject
473
     */
474
    private function removeFromToSaveObjectList(DbRow $myObject)
475
    {
476
        unset($this->toSaveObjects[$myObject]);
477
    }
478
479
    /**
480
     * This is an internal function, you should not use it in your application.
481
     * This is used internally by TDBM to add an object to the list of objects that have been
482
     * created/updated but not saved yet.
483
     *
484
     * @param DbRow $myObject
485
     */
486
    public function _addToToSaveObjectList(DbRow $myObject)
487
    {
488
        $this->toSaveObjects[$myObject] = true;
489
    }
490
491
    /**
492
     * Generates all the daos and beans.
493
     *
494
     * @param string $composerFile        If it's set, location of custom Composer file. Relative to project root
495
     *
496
     * @return \string[] the list of tables (key) and bean name (value)
497
     */
498
    public function generateAllDaosAndBeans($composerFile = null)
499
    {
500
        // Purge cache before generating anything.
501
        $this->cache->deleteAll();
0 ignored issues
show
Bug introduced by
The method deleteAll() does not exist on Doctrine\Common\Cache\Cache. Did you maybe mean delete()?

This check marks calls to methods that do not seem to exist on an object.

This is most likely the result of a method being renamed without all references to it being renamed likewise.

Loading history...
502
503
        $tdbmDaoGenerator = new TDBMDaoGenerator($this->configuration, $this->tdbmSchemaAnalyzer);
504
        if (null !== $composerFile) {
505
            $tdbmDaoGenerator->setComposerFile(__DIR__.'/../../../../../../../'.$composerFile);
0 ignored issues
show
Bug introduced by
The method setComposerFile() does not seem to exist on object<Mouf\Database\TDBM\Utils\TDBMDaoGenerator>.

This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces.

This is most likely a typographical error or the method has been renamed.

Loading history...
506
        }
507
508
        $tdbmDaoGenerator->generateAllDaosAndBeans();
509
    }
510
511
    /**
512
     * Returns the fully qualified class name of the bean associated with table $tableName.
513
     *
514
     *
515
     * @param string $tableName
516
     *
517
     * @return string
518
     */
519
    public function getBeanClassName(string $tableName) : string
520
    {
521
        if (isset($this->tableToBeanMap[$tableName])) {
522
            return $this->tableToBeanMap[$tableName];
523
        } else {
524
            $className = $this->beanNamespace.'\\'.$this->namingStrategy->getBeanClassName($tableName);
525
526
            if (!class_exists($className)) {
527
                throw new TDBMInvalidArgumentException(sprintf('Could not find class "%s". Does table "%s" exist? If yes, consider regenerating the DAOs and beans.', $className, $tableName));
528
            }
529
530
            $this->tableToBeanMap[$tableName] = $className;
531
            return $className;
532
        }
533
    }
534
535
    /**
536
     * Saves $object by INSERTing or UPDAT(E)ing it in the database.
537
     *
538
     * @param AbstractTDBMObject $object
539
     *
540
     * @throws TDBMException
541
     */
542
    public function save(AbstractTDBMObject $object)
543
    {
544
        $status = $object->_getStatus();
545
546
        if ($status === null) {
547
            throw new TDBMException(sprintf('Your bean for class %s has no status. It is likely that you overloaded the __construct method and forgot to call parent::__construct.', get_class($object)));
548
        }
549
550
        // Let's attach this object if it is in detached state.
551
        if ($status === TDBMObjectStateEnum::STATE_DETACHED) {
552
            $object->_attach($this);
553
            $status = $object->_getStatus();
554
        }
555
556
        if ($status === TDBMObjectStateEnum::STATE_NEW) {
557
            $dbRows = $object->_getDbRows();
558
559
            $unindexedPrimaryKeys = array();
560
561
            foreach ($dbRows as $dbRow) {
562
                if ($dbRow->_getStatus() == TDBMObjectStateEnum::STATE_SAVING) {
563
                    throw TDBMCyclicReferenceException::createCyclicReference($dbRow->_getDbTableName(), $object);
564
                }
565
                $dbRow->_setStatus(TDBMObjectStateEnum::STATE_SAVING);
566
                $tableName = $dbRow->_getDbTableName();
567
568
                $schema = $this->tdbmSchemaAnalyzer->getSchema();
569
                $tableDescriptor = $schema->getTable($tableName);
570
571
                $primaryKeyColumns = $this->getPrimaryKeyColumns($tableName);
572
573
                $references = $dbRow->_getReferences();
574
575
                // Let's save all references in NEW or DETACHED state (we need their primary key)
576
                foreach ($references as $fkName => $reference) {
577
                    if ($reference !== null) {
578
                        $refStatus = $reference->_getStatus();
579
                        if ($refStatus === TDBMObjectStateEnum::STATE_NEW || $refStatus === TDBMObjectStateEnum::STATE_DETACHED) {
580
                            try {
581
                                $this->save($reference);
582
                            } catch (TDBMCyclicReferenceException $e) {
583
                                throw TDBMCyclicReferenceException::extendCyclicReference($e, $dbRow->_getDbTableName(), $object, $fkName);
584
                            }
585
                        }
586
                    }
587
                }
588
589
                if (empty($unindexedPrimaryKeys)) {
590
                    $primaryKeys = $this->getPrimaryKeysForObjectFromDbRow($dbRow);
591
                } else {
592
                    // First insert, the children must have the same primary key as the parent.
593
                    $primaryKeys = $this->_getPrimaryKeysFromIndexedPrimaryKeys($tableName, $unindexedPrimaryKeys);
594
                    $dbRow->_setPrimaryKeys($primaryKeys);
595
                }
596
597
                $dbRowData = $dbRow->_getDbRow();
598
599
                // Let's see if the columns for primary key have been set before inserting.
600
                // We assume that if one of the value of the PK has been set, the PK is set.
601
                $isPkSet = !empty($primaryKeys);
602
603
                /*if (!$isPkSet) {
604
                    // if there is no autoincrement and no pk set, let's go in error.
605
                    $isAutoIncrement = true;
606
607
                    foreach ($primaryKeyColumns as $pkColumnName) {
608
                        $pkColumn = $tableDescriptor->getColumn($pkColumnName);
609
                        if (!$pkColumn->getAutoincrement()) {
610
                            $isAutoIncrement = false;
611
                        }
612
                    }
613
614
                    if (!$isAutoIncrement) {
615
                        $msg = "Error! You did not set the primary key(s) for the new object of type '$tableName'. The primary key is not set to 'autoincrement' so you must either set the primary key in the object or modify the DB model to create an primary key with auto-increment.";
616
                        throw new TDBMException($msg);
617
                    }
618
619
                }*/
620
621
                $types = [];
622
                $escapedDbRowData = [];
623
624 View Code Duplication
                foreach ($dbRowData as $columnName => $value) {
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated across your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
625
                    $columnDescriptor = $tableDescriptor->getColumn($columnName);
626
                    $types[] = $columnDescriptor->getType();
627
                    $escapedDbRowData[$this->connection->quoteIdentifier($columnName)] = $value;
628
                }
629
630
                $this->connection->insert($tableName, $escapedDbRowData, $types);
631
632
                if (!$isPkSet && count($primaryKeyColumns) == 1) {
633
                    $id = $this->connection->lastInsertId();
634
                    $pkColumn = $primaryKeyColumns[0];
635
                    // lastInsertId returns a string but the column type is usually a int. Let's convert it back to the correct type.
636
                    $id = $tableDescriptor->getColumn($pkColumn)->getType()->convertToPHPValue($id, $this->getConnection()->getDatabasePlatform());
637
                    $primaryKeys[$pkColumn] = $id;
638
                }
639
640
                // TODO: change this to some private magic accessor in future
641
                $dbRow->_setPrimaryKeys($primaryKeys);
642
                $unindexedPrimaryKeys = array_values($primaryKeys);
643
644
                /*
645
                 * When attached, on "save", we check if the column updated is part of a primary key
646
                 * If this is part of a primary key, we call the _update_id method that updates the id in the list of known objects.
647
                 * This method should first verify that the id is not already used (and is not auto-incremented)
648
                 *
649
                 * In the object, the key is stored in an array of  (column => value), that can be directly used to update the record.
650
                 *
651
                 *
652
                 */
653
654
                /*try {
0 ignored issues
show
Unused Code Comprehensibility introduced by
42% of this comment could be valid code. Did you maybe forget this after debugging?

Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it.

The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production.

This check looks for comments that seem to be mostly valid code and reports them.

Loading history...
655
                    $this->db_connection->exec($sql);
656
                } catch (TDBMException $e) {
657
                    $this->db_onerror = true;
658
659
                    // Strange..... if we do not have the line below, bad inserts are not catched.
660
                    // It seems that destructors are called before the registered shutdown function (PHP >=5.0.5)
661
                    //if ($this->tdbmService->isProgramExiting())
662
                    //	trigger_error("program exiting");
663
                    trigger_error($e->getMessage(), E_USER_ERROR);
664
665
                    if (!$this->tdbmService->isProgramExiting())
666
                        throw $e;
667
                    else
668
                    {
669
                        trigger_error($e->getMessage(), E_USER_ERROR);
670
                    }
671
                }*/
672
673
                // Let's remove this object from the $new_objects static table.
674
                $this->removeFromToSaveObjectList($dbRow);
675
676
                // TODO: change this behaviour to something more sensible performance-wise
677
                // Maybe a setting to trigger this globally?
678
                //$this->status = TDBMObjectStateEnum::STATE_NOT_LOADED;
0 ignored issues
show
Unused Code Comprehensibility introduced by
40% of this comment could be valid code. Did you maybe forget this after debugging?

Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it.

The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production.

This check looks for comments that seem to be mostly valid code and reports them.

Loading history...
679
                //$this->db_modified_state = false;
0 ignored issues
show
Unused Code Comprehensibility introduced by
50% of this comment could be valid code. Did you maybe forget this after debugging?

Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it.

The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production.

This check looks for comments that seem to be mostly valid code and reports them.

Loading history...
680
                //$dbRow = array();
0 ignored issues
show
Unused Code Comprehensibility introduced by
63% of this comment could be valid code. Did you maybe forget this after debugging?

Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it.

The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production.

This check looks for comments that seem to be mostly valid code and reports them.

Loading history...
681
682
                // Let's add this object to the list of objects in cache.
683
                $this->_addToCache($dbRow);
684
            }
685
686
            $object->_setStatus(TDBMObjectStateEnum::STATE_LOADED);
687
        } elseif ($status === TDBMObjectStateEnum::STATE_DIRTY) {
688
            $dbRows = $object->_getDbRows();
689
690
            foreach ($dbRows as $dbRow) {
691
                $references = $dbRow->_getReferences();
692
693
                // Let's save all references in NEW state (we need their primary key)
694
                foreach ($references as $fkName => $reference) {
695
                    if ($reference !== null && $reference->_getStatus() === TDBMObjectStateEnum::STATE_NEW) {
696
                        $this->save($reference);
697
                    }
698
                }
699
700
                // Let's first get the primary keys
701
                $tableName = $dbRow->_getDbTableName();
702
                $dbRowData = $dbRow->_getDbRow();
703
704
                $schema = $this->tdbmSchemaAnalyzer->getSchema();
705
                $tableDescriptor = $schema->getTable($tableName);
706
707
                $primaryKeys = $dbRow->_getPrimaryKeys();
708
709
                $types = [];
710
                $escapedDbRowData = [];
711
                $escapedPrimaryKeys = [];
712
713 View Code Duplication
                foreach ($dbRowData as $columnName => $value) {
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated across your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
714
                    $columnDescriptor = $tableDescriptor->getColumn($columnName);
715
                    $types[] = $columnDescriptor->getType();
716
                    $escapedDbRowData[$this->connection->quoteIdentifier($columnName)] = $value;
717
                }
718 View Code Duplication
                foreach ($primaryKeys as $columnName => $value) {
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated across your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
719
                    $columnDescriptor = $tableDescriptor->getColumn($columnName);
720
                    $types[] = $columnDescriptor->getType();
721
                    $escapedPrimaryKeys[$this->connection->quoteIdentifier($columnName)] = $value;
722
                }
723
724
                $this->connection->update($tableName, $escapedDbRowData, $escapedPrimaryKeys, $types);
725
726
                // Let's check if the primary key has been updated...
727
                $needsUpdatePk = false;
728
                foreach ($primaryKeys as $column => $value) {
729
                    if (!isset($dbRowData[$column]) || $dbRowData[$column] != $value) {
730
                        $needsUpdatePk = true;
731
                        break;
732
                    }
733
                }
734
                if ($needsUpdatePk) {
735
                    $this->objectStorage->remove($tableName, $this->getObjectHash($primaryKeys));
736
                    $newPrimaryKeys = $this->getPrimaryKeysForObjectFromDbRow($dbRow);
737
                    $dbRow->_setPrimaryKeys($newPrimaryKeys);
738
                    $this->objectStorage->set($tableName, $this->getObjectHash($primaryKeys), $dbRow);
739
                }
740
741
                // Let's remove this object from the list of objects to save.
742
                $this->removeFromToSaveObjectList($dbRow);
743
            }
744
745
            $object->_setStatus(TDBMObjectStateEnum::STATE_LOADED);
746
        } elseif ($status === TDBMObjectStateEnum::STATE_DELETED) {
747
            throw new TDBMInvalidOperationException('This object has been deleted. It cannot be saved.');
748
        }
749
750
        // Finally, let's save all the many to many relationships to this bean.
751
        $this->persistManyToManyRelationships($object);
752
    }
753
754
    private function persistManyToManyRelationships(AbstractTDBMObject $object)
755
    {
756
        foreach ($object->_getCachedRelationships() as $pivotTableName => $storage) {
757
            $tableDescriptor = $this->tdbmSchemaAnalyzer->getSchema()->getTable($pivotTableName);
758
            list($localFk, $remoteFk) = $this->getPivotTableForeignKeys($pivotTableName, $object);
759
760
            $toRemoveFromStorage = [];
761
762
            foreach ($storage as $remoteBean) {
763
                /* @var $remoteBean AbstractTDBMObject */
764
                $statusArr = $storage[$remoteBean];
765
                $status = $statusArr['status'];
766
                $reverse = $statusArr['reverse'];
767
                if ($reverse) {
768
                    continue;
769
                }
770
771
                if ($status === 'new') {
772
                    $remoteBeanStatus = $remoteBean->_getStatus();
773
                    if ($remoteBeanStatus === TDBMObjectStateEnum::STATE_NEW || $remoteBeanStatus === TDBMObjectStateEnum::STATE_DETACHED) {
774
                        // Let's save remote bean if needed.
775
                        $this->save($remoteBean);
776
                    }
777
778
                    $filters = $this->getPivotFilters($object, $remoteBean, $localFk, $remoteFk);
779
780
                    $types = [];
781
                    $escapedFilters = [];
782
783 View Code Duplication
                    foreach ($filters as $columnName => $value) {
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated across your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
784
                        $columnDescriptor = $tableDescriptor->getColumn($columnName);
785
                        $types[] = $columnDescriptor->getType();
786
                        $escapedFilters[$this->connection->quoteIdentifier($columnName)] = $value;
787
                    }
788
789
                    $this->connection->insert($pivotTableName, $escapedFilters, $types);
790
791
                    // Finally, let's mark relationships as saved.
792
                    $statusArr['status'] = 'loaded';
793
                    $storage[$remoteBean] = $statusArr;
794
                    $remoteStorage = $remoteBean->_getCachedRelationships()[$pivotTableName];
795
                    $remoteStatusArr = $remoteStorage[$object];
796
                    $remoteStatusArr['status'] = 'loaded';
797
                    $remoteStorage[$object] = $remoteStatusArr;
798
                } elseif ($status === 'delete') {
799
                    $filters = $this->getPivotFilters($object, $remoteBean, $localFk, $remoteFk);
800
801
                    $types = [];
802
803
                    foreach ($filters as $columnName => $value) {
804
                        $columnDescriptor = $tableDescriptor->getColumn($columnName);
805
                        $types[] = $columnDescriptor->getType();
806
                    }
807
808
                    $this->connection->delete($pivotTableName, $filters, $types);
809
810
                    // Finally, let's remove relationships completely from bean.
811
                    $toRemoveFromStorage[] = $remoteBean;
812
813
                    $remoteBean->_getCachedRelationships()[$pivotTableName]->detach($object);
814
                }
815
            }
816
817
            // Note: due to https://bugs.php.net/bug.php?id=65629, we cannot delete an element inside a foreach loop on a SplStorageObject.
818
            // Therefore, we cache elements in the $toRemoveFromStorage to remove them at a later stage.
819
            foreach ($toRemoveFromStorage as $remoteBean) {
820
                $storage->detach($remoteBean);
821
            }
822
        }
823
    }
824
825
    private function getPivotFilters(AbstractTDBMObject $localBean, AbstractTDBMObject $remoteBean, ForeignKeyConstraint $localFk, ForeignKeyConstraint $remoteFk)
826
    {
827
        $localBeanPk = $this->getPrimaryKeyValues($localBean);
828
        $remoteBeanPk = $this->getPrimaryKeyValues($remoteBean);
829
        $localColumns = $localFk->getLocalColumns();
830
        $remoteColumns = $remoteFk->getLocalColumns();
831
832
        $localFilters = array_combine($localColumns, $localBeanPk);
833
        $remoteFilters = array_combine($remoteColumns, $remoteBeanPk);
834
835
        return array_merge($localFilters, $remoteFilters);
836
    }
837
838
    /**
839
     * Returns the "values" of the primary key.
840
     * This returns the primary key from the $primaryKey attribute, not the one stored in the columns.
841
     *
842
     * @param AbstractTDBMObject $bean
843
     *
844
     * @return array numerically indexed array of values
845
     */
846
    private function getPrimaryKeyValues(AbstractTDBMObject $bean)
847
    {
848
        $dbRows = $bean->_getDbRows();
849
        $dbRow = reset($dbRows);
850
851
        return array_values($dbRow->_getPrimaryKeys());
852
    }
853
854
    /**
855
     * Returns a unique hash used to store the object based on its primary key.
856
     * If the array contains only one value, then the value is returned.
857
     * Otherwise, a hash representing the array is returned.
858
     *
859
     * @param array $primaryKeys An array of columns => values forming the primary key
860
     *
861
     * @return string
862
     */
863
    public function getObjectHash(array $primaryKeys)
864
    {
865
        if (count($primaryKeys) === 1) {
866
            return reset($primaryKeys);
867
        } else {
868
            ksort($primaryKeys);
869
870
            return md5(json_encode($primaryKeys));
871
        }
872
    }
873
874
    /**
875
     * Returns an array of primary keys from the object.
876
     * The primary keys are extracted from the object columns and not from the primary keys stored in the
877
     * $primaryKeys variable of the object.
878
     *
879
     * @param DbRow $dbRow
880
     *
881
     * @return array Returns an array of column => value
882
     */
883
    public function getPrimaryKeysForObjectFromDbRow(DbRow $dbRow)
884
    {
885
        $table = $dbRow->_getDbTableName();
886
        $dbRowData = $dbRow->_getDbRow();
887
888
        return $this->_getPrimaryKeysFromObjectData($table, $dbRowData);
889
    }
890
891
    /**
892
     * Returns an array of primary keys for the given row.
893
     * The primary keys are extracted from the object columns.
894
     *
895
     * @param $table
896
     * @param array $columns
897
     *
898
     * @return array
899
     */
900
    public function _getPrimaryKeysFromObjectData($table, array $columns)
901
    {
902
        $primaryKeyColumns = $this->getPrimaryKeyColumns($table);
903
        $values = array();
904
        foreach ($primaryKeyColumns as $column) {
0 ignored issues
show
Bug introduced by
The expression $primaryKeyColumns of type string|array is not guaranteed to be traversable. How about adding an additional type check?

There are different options of fixing this problem.

  1. If you want to be on the safe side, you can add an additional type-check:

    $collection = json_decode($data, true);
    if ( ! is_array($collection)) {
        throw new \RuntimeException('$collection must be an array.');
    }
    
    foreach ($collection as $item) { /** ... */ }
    
  2. If you are sure that the expression is traversable, you might want to add a doc comment cast to improve IDE auto-completion and static analysis:

    /** @var array $collection */
    $collection = json_decode($data, true);
    
    foreach ($collection as $item) { /** .. */ }
    
  3. Mark the issue as a false-positive: Just hover the remove button, in the top-right corner of this issue for more options.

Loading history...
905
            if (isset($columns[$column])) {
906
                $values[$column] = $columns[$column];
907
            }
908
        }
909
910
        return $values;
911
    }
912
913
    /**
914
     * Attaches $object to this TDBMService.
915
     * The $object must be in DETACHED state and will pass in NEW state.
916
     *
917
     * @param AbstractTDBMObject $object
918
     *
919
     * @throws TDBMInvalidOperationException
920
     */
921
    public function attach(AbstractTDBMObject $object)
922
    {
923
        $object->_attach($this);
924
    }
925
926
    /**
927
     * Returns an associative array (column => value) for the primary keys from the table name and an
928
     * indexed array of primary key values.
929
     *
930
     * @param string $tableName
931
     * @param array  $indexedPrimaryKeys
932
     */
933
    public function _getPrimaryKeysFromIndexedPrimaryKeys($tableName, array $indexedPrimaryKeys)
934
    {
935
        $primaryKeyColumns = $this->tdbmSchemaAnalyzer->getSchema()->getTable($tableName)->getPrimaryKeyColumns();
936
937
        if (count($primaryKeyColumns) !== count($indexedPrimaryKeys)) {
938
            throw new TDBMException(sprintf('Wrong number of columns passed for primary key. Expected %s columns for table "%s",
939
			got %s instead.', count($primaryKeyColumns), $tableName, count($indexedPrimaryKeys)));
940
        }
941
942
        return array_combine($primaryKeyColumns, $indexedPrimaryKeys);
943
    }
944
945
    /**
946
     * Return the list of tables (from child to parent) joining the tables passed in parameter.
947
     * Tables must be in a single line of inheritance. The method will find missing tables.
948
     *
949
     * Algorithm: one of those tables is the ultimate child. From this child, by recursively getting the parent,
950
     * we must be able to find all other tables.
951
     *
952
     * @param string[] $tables
953
     *
954
     * @return string[]
955
     */
956
    public function _getLinkBetweenInheritedTables(array $tables)
957
    {
958
        sort($tables);
959
960
        return $this->fromCache($this->cachePrefix.'_linkbetweeninheritedtables_'.implode('__split__', $tables),
961
            function () use ($tables) {
962
                return $this->_getLinkBetweenInheritedTablesWithoutCache($tables);
963
            });
964
    }
965
966
    /**
967
     * Return the list of tables (from child to parent) joining the tables passed in parameter.
968
     * Tables must be in a single line of inheritance. The method will find missing tables.
969
     *
970
     * Algorithm: one of those tables is the ultimate child. From this child, by recursively getting the parent,
971
     * we must be able to find all other tables.
972
     *
973
     * @param string[] $tables
974
     *
975
     * @return string[]
976
     */
977
    private function _getLinkBetweenInheritedTablesWithoutCache(array $tables)
978
    {
979
        $schemaAnalyzer = $this->schemaAnalyzer;
980
981
        foreach ($tables as $currentTable) {
982
            $allParents = [$currentTable];
983
            while ($currentFk = $schemaAnalyzer->getParentRelationship($currentTable)) {
984
                $currentTable = $currentFk->getForeignTableName();
985
                $allParents[] = $currentTable;
986
            }
987
988
            // Now, does the $allParents contain all the tables we want?
989
            $notFoundTables = array_diff($tables, $allParents);
990
            if (empty($notFoundTables)) {
991
                // We have a winner!
992
                return $allParents;
993
            }
994
        }
995
996
        throw TDBMInheritanceException::create($tables);
997
    }
998
999
    /**
1000
     * Returns the list of tables related to this table (via a parent or child inheritance relationship).
1001
     *
1002
     * @param string $table
1003
     *
1004
     * @return string[]
1005
     */
1006
    public function _getRelatedTablesByInheritance($table)
1007
    {
1008
        return $this->fromCache($this->cachePrefix.'_relatedtables_'.$table, function () use ($table) {
1009
            return $this->_getRelatedTablesByInheritanceWithoutCache($table);
1010
        });
1011
    }
1012
1013
    /**
1014
     * Returns the list of tables related to this table (via a parent or child inheritance relationship).
1015
     *
1016
     * @param string $table
1017
     *
1018
     * @return string[]
1019
     */
1020
    private function _getRelatedTablesByInheritanceWithoutCache($table)
1021
    {
1022
        $schemaAnalyzer = $this->schemaAnalyzer;
1023
1024
        // Let's scan the parent tables
1025
        $currentTable = $table;
1026
1027
        $parentTables = [];
1028
1029
        // Get parent relationship
1030
        while ($currentFk = $schemaAnalyzer->getParentRelationship($currentTable)) {
1031
            $currentTable = $currentFk->getForeignTableName();
1032
            $parentTables[] = $currentTable;
1033
        }
1034
1035
        // Let's recurse in children
1036
        $childrenTables = $this->exploreChildrenTablesRelationships($schemaAnalyzer, $table);
1037
1038
        return array_merge(array_reverse($parentTables), $childrenTables);
1039
    }
1040
1041
    /**
1042
     * Explore all the children and descendant of $table and returns ForeignKeyConstraints on those.
1043
     *
1044
     * @param string $table
1045
     *
1046
     * @return string[]
1047
     */
1048
    private function exploreChildrenTablesRelationships(SchemaAnalyzer $schemaAnalyzer, $table)
1049
    {
1050
        $tables = [$table];
1051
        $keys = $schemaAnalyzer->getChildrenRelationships($table);
1052
1053
        foreach ($keys as $key) {
1054
            $tables = array_merge($tables, $this->exploreChildrenTablesRelationships($schemaAnalyzer, $key->getLocalTableName()));
1055
        }
1056
1057
        return $tables;
1058
    }
1059
1060
    /**
1061
     * Casts a foreign key into SQL, assuming table name is used with no alias.
1062
     * The returned value does contain only one table. For instance:.
1063
     *
1064
     * " LEFT JOIN table2 ON table1.id = table2.table1_id"
1065
     *
1066
     * @param ForeignKeyConstraint $fk
1067
     * @param bool                 $leftTableIsLocal
1068
     *
1069
     * @return string
1070
     */
1071
    /*private function foreignKeyToSql(ForeignKeyConstraint $fk, $leftTableIsLocal) {
0 ignored issues
show
Unused Code Comprehensibility introduced by
59% of this comment could be valid code. Did you maybe forget this after debugging?

Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it.

The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production.

This check looks for comments that seem to be mostly valid code and reports them.

Loading history...
1072
        $onClauses = [];
1073
        $foreignTableName = $this->connection->quoteIdentifier($fk->getForeignTableName());
1074
        $foreignColumns = $fk->getForeignColumns();
1075
        $localTableName = $this->connection->quoteIdentifier($fk->getLocalTableName());
1076
        $localColumns = $fk->getLocalColumns();
1077
        $columnCount = count($localTableName);
1078
1079
        for ($i = 0; $i < $columnCount; $i++) {
1080
            $onClauses[] = sprintf("%s.%s = %s.%s",
1081
                $localTableName,
1082
                $this->connection->quoteIdentifier($localColumns[$i]),
1083
                $foreignColumns,
1084
                $this->connection->quoteIdentifier($foreignColumns[$i])
1085
                );
1086
        }
1087
1088
        $onClause = implode(' AND ', $onClauses);
1089
1090
        if ($leftTableIsLocal) {
1091
            return sprintf(" LEFT JOIN %s ON (%s)", $foreignTableName, $onClause);
1092
        } else {
1093
            return sprintf(" LEFT JOIN %s ON (%s)", $localTableName, $onClause);
1094
        }
1095
    }*/
1096
1097
    /**
1098
     * Returns a `ResultIterator` object representing filtered records of "$mainTable" .
1099
     *
1100
     * The findObjects method should be the most used query method in TDBM if you want to query the database for objects.
1101
     * (Note: if you want to query the database for an object by its primary key, use the findObjectByPk method).
1102
     *
1103
     * The findObjects method takes in parameter:
1104
     * 	- mainTable: the kind of bean you want to retrieve. In TDBM, a bean matches a database row, so the
1105
     * 			`$mainTable` parameter should be the name of an existing table in database.
1106
     *  - filter: The filter is a filter bag. It is what you use to filter your request (the WHERE part in SQL).
1107
     *          It can be a string (SQL Where clause), or even a bean or an associative array (key = column to filter, value = value to find)
1108
     *  - parameters: The parameters used in the filter. If you pass a SQL string as a filter, be sure to avoid
1109
     *          concatenating parameters in the string (this leads to SQL injection and also to poor caching performance).
1110
     *          Instead, please consider passing parameters (see documentation for more details).
1111
     *  - additionalTablesFetch: An array of SQL tables names. The beans related to those tables will be fetched along
1112
     *          the main table. This is useful to avoid hitting the database with numerous subqueries.
1113
     *  - mode: The fetch mode of the result. See `setFetchMode()` method for more details.
1114
     *
1115
     * The `findObjects` method will return a `ResultIterator`. A `ResultIterator` is an object that behaves as an array
1116
     * (in ARRAY mode) at least. It can be iterated using a `foreach` loop.
1117
     *
1118
     * Finally, if filter_bag is null, the whole table is returned.
1119
     *
1120
     * @param string                       $mainTable             The name of the table queried
1121
     * @param string|array|null            $filter                The SQL filters to apply to the query (the WHERE part). Columns from tables different from $mainTable must be prefixed by the table name (in the form: table.column)
1122
     * @param array                        $parameters
1123
     * @param string|UncheckedOrderBy|null $orderString           The ORDER BY part of the query. Columns from tables different from $mainTable must be prefixed by the table name (in the form: table.column)
1124
     * @param array                        $additionalTablesFetch
1125
     * @param int                          $mode
1126
     * @param string                       $className             Optional: The name of the class to instantiate. This class must extend the TDBMObject class. If none is specified, a TDBMObject instance will be returned
1127
     *
1128
     * @return ResultIterator An object representing an array of results
1129
     *
1130
     * @throws TDBMException
1131
     */
1132
    public function findObjects(string $mainTable, $filter = null, array $parameters = array(), $orderString = null, array $additionalTablesFetch = array(), $mode = null, string $className = null)
1133
    {
1134
        // $mainTable is not secured in MagicJoin, let's add a bit of security to avoid SQL injection.
1135
        if (!preg_match('/^[a-zA-Z_][a-zA-Z0-9_]*$/', $mainTable)) {
1136
            throw new TDBMException(sprintf("Invalid table name: '%s'", $mainTable));
1137
        }
1138
1139
        $mode = $mode ?: $this->mode;
1140
1141
        list($filterString, $additionalParameters) = $this->buildFilterFromFilterBag($filter);
1142
1143
        $parameters = array_merge($parameters, $additionalParameters);
1144
1145
        $queryFactory = new FindObjectsQueryFactory($mainTable, $additionalTablesFetch, $filterString, $orderString, $this, $this->tdbmSchemaAnalyzer->getSchema(), $this->orderByAnalyzer);
1146
1147
        return new ResultIterator($queryFactory, $parameters, $this->objectStorage, $className, $this, $this->magicQuery, $mode, $this->logger);
1148
    }
1149
1150
    /**
1151
     * @param string                       $mainTable   The name of the table queried
1152
     * @param string                       $from        The from sql statement
1153
     * @param string|array|null            $filter      The SQL filters to apply to the query (the WHERE part). All columns must be prefixed by the table name (in the form: table.column)
1154
     * @param array                        $parameters
1155
     * @param string|UncheckedOrderBy|null $orderString The ORDER BY part of the query. All columns must be prefixed by the table name (in the form: table.column)
1156
     * @param int                          $mode
1157
     * @param string                       $className   Optional: The name of the class to instantiate. This class must extend the TDBMObject class. If none is specified, a TDBMObject instance will be returned
1158
     *
1159
     * @return ResultIterator An object representing an array of results
1160
     *
1161
     * @throws TDBMException
1162
     */
1163
    public function findObjectsFromSql(string $mainTable, string $from, $filter = null, array $parameters = array(), $orderString = null, $mode = null, string $className = null)
1164
    {
1165
        // $mainTable is not secured in MagicJoin, let's add a bit of security to avoid SQL injection.
1166
        if (!preg_match('/^[a-zA-Z_][a-zA-Z0-9_]*$/', $mainTable)) {
1167
            throw new TDBMException(sprintf("Invalid table name: '%s'", $mainTable));
1168
        }
1169
1170
        $mode = $mode ?: $this->mode;
1171
1172
        list($filterString, $additionalParameters) = $this->buildFilterFromFilterBag($filter);
1173
1174
        $parameters = array_merge($parameters, $additionalParameters);
1175
1176
        $queryFactory = new FindObjectsFromSqlQueryFactory($mainTable, $from, $filterString, $orderString, $this, $this->tdbmSchemaAnalyzer->getSchema(), $this->orderByAnalyzer, $this->schemaAnalyzer, $this->cache, $this->cachePrefix);
0 ignored issues
show
Bug introduced by
It seems like $this->cache can be null; however, __construct() does not accept null, maybe add an additional type check?

Unless you are absolutely sure that the expression can never be null because of other conditions, we strongly recommend to add an additional type check to your code:

/** @return stdClass|null */
function mayReturnNull() { }

function doesNotAcceptNull(stdClass $x) { }

// With potential error.
function withoutCheck() {
    $x = mayReturnNull();
    doesNotAcceptNull($x); // Potential error here.
}

// Safe - Alternative 1
function withCheck1() {
    $x = mayReturnNull();
    if ( ! $x instanceof stdClass) {
        throw new \LogicException('$x must be defined.');
    }
    doesNotAcceptNull($x);
}

// Safe - Alternative 2
function withCheck2() {
    $x = mayReturnNull();
    if ($x instanceof stdClass) {
        doesNotAcceptNull($x);
    }
}
Loading history...
1177
1178
        return new ResultIterator($queryFactory, $parameters, $this->objectStorage, $className, $this, $this->magicQuery, $mode, $this->logger);
1179
    }
1180
1181
    /**
1182
     * @param $table
1183
     * @param array  $primaryKeys
1184
     * @param array  $additionalTablesFetch
1185
     * @param bool   $lazy                  Whether to perform lazy loading on this object or not
1186
     * @param string $className
1187
     *
1188
     * @return AbstractTDBMObject
1189
     *
1190
     * @throws TDBMException
1191
     */
1192
    public function findObjectByPk(string $table, array $primaryKeys, array $additionalTablesFetch = array(), bool $lazy = false, string $className = null)
1193
    {
1194
        $primaryKeys = $this->_getPrimaryKeysFromObjectData($table, $primaryKeys);
1195
        $hash = $this->getObjectHash($primaryKeys);
1196
1197
        if ($this->objectStorage->has($table, $hash)) {
1198
            $dbRow = $this->objectStorage->get($table, $hash);
1199
            $bean = $dbRow->getTDBMObject();
1200
            if ($className !== null && !is_a($bean, $className)) {
1201
                throw new TDBMException("TDBM cannot create a bean of class '".$className."'. The requested object was already loaded and its class is '".get_class($bean)."'");
1202
            }
1203
1204
            return $bean;
1205
        }
1206
1207
        // Are we performing lazy fetching?
1208
        if ($lazy === true) {
1209
            // Can we perform lazy fetching?
1210
            $tables = $this->_getRelatedTablesByInheritance($table);
1211
            // Only allowed if no inheritance.
1212
            if (count($tables) === 1) {
1213
                if ($className === null) {
1214
                    try {
1215
                        $className = $this->getBeanClassName($table);
1216
                    } catch (TDBMInvalidArgumentException $e) {
1217
                        $className = TDBMObject::class;
1218
                    }
1219
                }
1220
1221
                // Let's construct the bean
1222
                if (!isset($this->reflectionClassCache[$className])) {
1223
                    $this->reflectionClassCache[$className] = new \ReflectionClass($className);
1224
                }
1225
                // Let's bypass the constructor when creating the bean!
1226
                $bean = $this->reflectionClassCache[$className]->newInstanceWithoutConstructor();
1227
                /* @var $bean AbstractTDBMObject */
1228
                $bean->_constructLazy($table, $primaryKeys, $this);
1229
1230
                return $bean;
1231
            }
1232
        }
1233
1234
        // Did not find the object in cache? Let's query it!
1235
        return $this->findObjectOrFail($table, $primaryKeys, [], $additionalTablesFetch, $className);
1236
    }
1237
1238
    /**
1239
     * Returns a unique bean (or null) according to the filters passed in parameter.
1240
     *
1241
     * @param string            $mainTable             The name of the table queried
1242
     * @param string|array|null $filter                The SQL filters to apply to the query (the WHERE part). All columns must be prefixed by the table name (in the form: table.column)
1243
     * @param array             $parameters
1244
     * @param array             $additionalTablesFetch
1245
     * @param string            $className             Optional: The name of the class to instantiate. This class must extend the TDBMObject class. If none is specified, a TDBMObject instance will be returned
1246
     *
1247
     * @return AbstractTDBMObject|null The object we want, or null if no object matches the filters
1248
     *
1249
     * @throws TDBMException
1250
     */
1251 View Code Duplication
    public function findObject(string $mainTable, $filter = null, array $parameters = array(), array $additionalTablesFetch = array(), string $className = null)
0 ignored issues
show
Duplication introduced by
This method seems to be duplicated in your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
1252
    {
1253
        $objects = $this->findObjects($mainTable, $filter, $parameters, null, $additionalTablesFetch, self::MODE_ARRAY, $className);
1254
        $page = $objects->take(0, 2);
1255
        $count = $page->count();
1256
        if ($count > 1) {
1257
            throw new DuplicateRowException("Error while querying an object for table '$mainTable': More than 1 row have been returned, but we should have received at most one.");
1258
        } elseif ($count === 0) {
1259
            return;
1260
        }
1261
1262
        return $page[0];
1263
    }
1264
1265
    /**
1266
     * Returns a unique bean (or null) according to the filters passed in parameter.
1267
     *
1268
     * @param string            $mainTable  The name of the table queried
1269
     * @param string            $from       The from sql statement
1270
     * @param string|array|null $filter     The SQL filters to apply to the query (the WHERE part). All columns must be prefixed by the table name (in the form: table.column)
1271
     * @param array             $parameters
1272
     * @param string            $className  Optional: The name of the class to instantiate. This class must extend the TDBMObject class. If none is specified, a TDBMObject instance will be returned
1273
     *
1274
     * @return AbstractTDBMObject|null The object we want, or null if no object matches the filters
1275
     *
1276
     * @throws TDBMException
1277
     */
1278 View Code Duplication
    public function findObjectFromSql($mainTable, $from, $filter = null, array $parameters = array(), $className = null)
0 ignored issues
show
Duplication introduced by
This method seems to be duplicated in your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
1279
    {
1280
        $objects = $this->findObjectsFromSql($mainTable, $from, $filter, $parameters, null, self::MODE_ARRAY, $className);
1281
        $page = $objects->take(0, 2);
1282
        $count = $page->count();
1283
        if ($count > 1) {
1284
            throw new DuplicateRowException("Error while querying an object for table '$mainTable': More than 1 row have been returned, but we should have received at most one.");
1285
        } elseif ($count === 0) {
1286
            return;
1287
        }
1288
1289
        return $page[0];
1290
    }
1291
1292
    /**
1293
     * Returns a unique bean according to the filters passed in parameter.
1294
     * Throws a NoBeanFoundException if no bean was found for the filter passed in parameter.
1295
     *
1296
     * @param string            $mainTable             The name of the table queried
1297
     * @param string|array|null $filter                The SQL filters to apply to the query (the WHERE part). All columns must be prefixed by the table name (in the form: table.column)
1298
     * @param array             $parameters
1299
     * @param array             $additionalTablesFetch
1300
     * @param string            $className             Optional: The name of the class to instantiate. This class must extend the TDBMObject class. If none is specified, a TDBMObject instance will be returned
1301
     *
1302
     * @return AbstractTDBMObject The object we want
1303
     *
1304
     * @throws TDBMException
1305
     */
1306
    public function findObjectOrFail(string $mainTable, $filter = null, array $parameters = array(), array $additionalTablesFetch = array(), string $className = null)
1307
    {
1308
        $bean = $this->findObject($mainTable, $filter, $parameters, $additionalTablesFetch, $className);
1309
        if ($bean === null) {
1310
            throw new NoBeanFoundException("No result found for query on table '".$mainTable."'");
1311
        }
1312
1313
        return $bean;
1314
    }
1315
1316
    /**
1317
     * @param array $beanData An array of data: array<table, array<column, value>>
1318
     *
1319
     * @return array an array with first item = class name, second item = table name and third item = list of tables needed
1320
     *
1321
     * @throws TDBMInheritanceException
1322
     */
1323
    public function _getClassNameFromBeanData(array $beanData)
1324
    {
1325
        if (count($beanData) === 1) {
1326
            $tableName = array_keys($beanData)[0];
1327
            $allTables = [$tableName];
1328
        } else {
1329
            $tables = [];
1330
            foreach ($beanData as $table => $row) {
1331
                $primaryKeyColumns = $this->getPrimaryKeyColumns($table);
1332
                $pkSet = false;
1333
                foreach ($primaryKeyColumns as $columnName) {
0 ignored issues
show
Bug introduced by
The expression $primaryKeyColumns of type string|array is not guaranteed to be traversable. How about adding an additional type check?

There are different options of fixing this problem.

  1. If you want to be on the safe side, you can add an additional type-check:

    $collection = json_decode($data, true);
    if ( ! is_array($collection)) {
        throw new \RuntimeException('$collection must be an array.');
    }
    
    foreach ($collection as $item) { /** ... */ }
    
  2. If you are sure that the expression is traversable, you might want to add a doc comment cast to improve IDE auto-completion and static analysis:

    /** @var array $collection */
    $collection = json_decode($data, true);
    
    foreach ($collection as $item) { /** .. */ }
    
  3. Mark the issue as a false-positive: Just hover the remove button, in the top-right corner of this issue for more options.

Loading history...
1334
                    if ($row[$columnName] !== null) {
1335
                        $pkSet = true;
1336
                        break;
1337
                    }
1338
                }
1339
                if ($pkSet) {
1340
                    $tables[] = $table;
1341
                }
1342
            }
1343
1344
            // $tables contains the tables for this bean. Let's view the top most part of the hierarchy
1345
            try {
1346
                $allTables = $this->_getLinkBetweenInheritedTables($tables);
1347
            } catch (TDBMInheritanceException $e) {
1348
                throw TDBMInheritanceException::extendException($e, $this, $beanData);
1349
            }
1350
            $tableName = $allTables[0];
1351
        }
1352
1353
        // Only one table in this bean. Life is sweat, let's look at its type:
1354
        try {
1355
            $className = $this->getBeanClassName($tableName);
1356
        } catch (TDBMInvalidArgumentException $e) {
1357
            $className = 'Mouf\\Database\\TDBM\\TDBMObject';
1358
        }
1359
1360
        return [$className, $tableName, $allTables];
1361
    }
1362
1363
    /**
1364
     * Returns an item from cache or computes it using $closure and puts it in cache.
1365
     *
1366
     * @param string   $key
1367
     * @param callable $closure
1368
     *
1369
     * @return mixed
1370
     */
1371 View Code Duplication
    private function fromCache(string $key, callable $closure)
0 ignored issues
show
Duplication introduced by
This method seems to be duplicated in your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
1372
    {
1373
        $item = $this->cache->fetch($key);
1374
        if ($item === false) {
1375
            $item = $closure();
1376
            $this->cache->save($key, $item);
1377
        }
1378
1379
        return $item;
1380
    }
1381
1382
    /**
1383
     * Returns the foreign key object.
1384
     *
1385
     * @param string $table
1386
     * @param string $fkName
1387
     *
1388
     * @return ForeignKeyConstraint
1389
     */
1390
    public function _getForeignKeyByName(string $table, string $fkName)
1391
    {
1392
        return $this->tdbmSchemaAnalyzer->getSchema()->getTable($table)->getForeignKey($fkName);
1393
    }
1394
1395
    /**
1396
     * @param $pivotTableName
1397
     * @param AbstractTDBMObject $bean
1398
     *
1399
     * @return AbstractTDBMObject[]
1400
     */
1401
    public function _getRelatedBeans(string $pivotTableName, AbstractTDBMObject $bean)
1402
    {
1403
        list($localFk, $remoteFk) = $this->getPivotTableForeignKeys($pivotTableName, $bean);
1404
        /* @var $localFk ForeignKeyConstraint */
1405
        /* @var $remoteFk ForeignKeyConstraint */
1406
        $remoteTable = $remoteFk->getForeignTableName();
1407
1408
        $primaryKeys = $this->getPrimaryKeyValues($bean);
1409
        $columnNames = array_map(function ($name) use ($pivotTableName) {
1410
            return $pivotTableName.'.'.$name;
1411
        }, $localFk->getLocalColumns());
1412
1413
        $filter = array_combine($columnNames, $primaryKeys);
1414
1415
        return $this->findObjects($remoteTable, $filter);
1416
    }
1417
1418
    /**
1419
     * @param $pivotTableName
1420
     * @param AbstractTDBMObject $bean The LOCAL bean
1421
     *
1422
     * @return ForeignKeyConstraint[] First item: the LOCAL bean, second item: the REMOTE bean
1423
     *
1424
     * @throws TDBMException
1425
     */
1426
    private function getPivotTableForeignKeys(string $pivotTableName, AbstractTDBMObject $bean)
1427
    {
1428
        $fks = array_values($this->tdbmSchemaAnalyzer->getSchema()->getTable($pivotTableName)->getForeignKeys());
1429
        $table1 = $fks[0]->getForeignTableName();
1430
        $table2 = $fks[1]->getForeignTableName();
1431
1432
        $beanTables = array_map(function (DbRow $dbRow) {
1433
            return $dbRow->_getDbTableName();
1434
        }, $bean->_getDbRows());
1435
1436
        if (in_array($table1, $beanTables)) {
1437
            return [$fks[0], $fks[1]];
1438
        } elseif (in_array($table2, $beanTables)) {
1439
            return [$fks[1], $fks[0]];
1440
        } else {
1441
            throw new TDBMException("Unexpected bean type in getPivotTableForeignKeys. Awaiting beans from table {$table1} and {$table2} for pivot table {$pivotTableName}");
1442
        }
1443
    }
1444
1445
    /**
1446
     * Returns a list of pivot tables linked to $bean.
1447
     *
1448
     * @param AbstractTDBMObject $bean
1449
     *
1450
     * @return string[]
1451
     */
1452
    public function _getPivotTablesLinkedToBean(AbstractTDBMObject $bean)
1453
    {
1454
        $junctionTables = [];
1455
        $allJunctionTables = $this->schemaAnalyzer->detectJunctionTables(true);
1456
        foreach ($bean->_getDbRows() as $dbRow) {
1457
            foreach ($allJunctionTables as $table) {
1458
                // There are exactly 2 FKs since this is a pivot table.
1459
                $fks = array_values($table->getForeignKeys());
1460
1461
                if ($fks[0]->getForeignTableName() === $dbRow->_getDbTableName() || $fks[1]->getForeignTableName() === $dbRow->_getDbTableName()) {
1462
                    $junctionTables[] = $table->getName();
1463
                }
1464
            }
1465
        }
1466
1467
        return $junctionTables;
1468
    }
1469
1470
    /**
1471
     * Array of types for tables.
1472
     * Key: table name
1473
     * Value: array of types indexed by column.
1474
     *
1475
     * @var array[]
1476
     */
1477
    private $typesForTable = [];
1478
1479
    /**
1480
     * @internal
1481
     *
1482
     * @param string $tableName
1483
     *
1484
     * @return Type[]
1485
     */
1486
    public function _getColumnTypesForTable(string $tableName)
1487
    {
1488
        if (!isset($typesForTable[$tableName])) {
0 ignored issues
show
Bug introduced by
The variable $typesForTable seems only to be defined at a later point. As such the call to isset() seems to always evaluate to false.

This check marks calls to isset(...) or empty(...) that are found before the variable itself is defined. These will always have the same result.

This is likely the result of code being shifted around. Consider removing these calls.

Loading history...
1489
            $columns = $this->tdbmSchemaAnalyzer->getSchema()->getTable($tableName)->getColumns();
1490
            $typesForTable[$tableName] = array_map(function (Column $column) {
0 ignored issues
show
Coding Style Comprehensibility introduced by
$typesForTable was never initialized. Although not strictly required by PHP, it is generally a good practice to add $typesForTable = array(); before regardless.

Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code.

Let’s take a look at an example:

foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}

As you can see in this example, the array $myArray is initialized the first time when the foreach loop is entered. You can also see that the value of the bar key is only written conditionally; thus, its value might result from a previous iteration.

This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization $myArray = array() either outside or inside the foreach loop.

Loading history...
1491
                return $column->getType();
1492
            }, $columns);
1493
        }
1494
1495
        return $typesForTable[$tableName];
1496
    }
1497
1498
    /**
1499
     * Sets the minimum log level.
1500
     * $level must be one of Psr\Log\LogLevel::xxx.
1501
     *
1502
     * Defaults to LogLevel::WARNING
1503
     *
1504
     * @param string $level
1505
     */
1506
    public function setLogLevel(string $level)
1507
    {
1508
        $this->logger = new LevelFilter($this->rootLogger, $level);
1509
    }
1510
}
1511