Completed
Push — master ( 300f7c...ac8da7 )
by Joschi
06:40
created

RepositoryGeneratorService::generate()   A

Complexity

Conditions 2
Paths 2

Size

Total Lines 17
Code Lines 5

Duplication

Lines 0
Ratio 0 %

Importance

Changes 1
Bugs 0 Features 0
Metric Value
c 1
b 0
f 0
dl 0
loc 17
rs 9.4285
cc 2
eloc 5
nc 2
nop 1
1
<?php
2
3
/**
4
 * apparat-dev
5
 *
6
 * @category    Apparat
7
 * @package     Apparat\Dev
8
 * @subpackage  Apparat\Dev\Application
9
 * @author      Joschi Kuphal <[email protected]> / @jkphl
10
 * @copyright   Copyright © 2016 Joschi Kuphal <[email protected]> / @jkphl
11
 * @license     http://opensource.org/licenses/MIT The MIT License (MIT)
12
 */
13
14
/***********************************************************************************
15
 *  The MIT License (MIT)
16
 *
17
 *  Copyright © 2016 Joschi Kuphal <[email protected]> / @jkphl
18
 *
19
 *  Permission is hereby granted, free of charge, to any person obtaining a copy of
20
 *  this software and associated documentation files (the "Software"), to deal in
21
 *  the Software without restriction, including without limitation the rights to
22
 *  use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of
23
 *  the Software, and to permit persons to whom the Software is furnished to do so,
24
 *  subject to the following conditions:
25
 *
26
 *  The above copyright notice and this permission notice shall be included in all
27
 *  copies or substantial portions of the Software.
28
 *
29
 *  THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
30
 *  IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS
31
 *  FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR
32
 *  COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER
33
 *  IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
34
 *  CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
35
 ***********************************************************************************/
36
37
namespace Apparat\Dev\Application\Service;
38
39
use Apparat\Dev\Application\Contract\ObjectFactoryInterface;
40
use Apparat\Kernel\Ports\Kernel;
41
use Apparat\Object\Domain\Repository\RepositoryInterface;
42
43
/**
44
 * Repository builder service
45
 *
46
 * @package Apparat\Dev
47
 * @subpackage Apparat\Dev\Application
48
 */
49
class RepositoryGeneratorService
50
{
51
    /**
52
     * Object factory
53
     *
54
     * @var ObjectFactoryInterface
55
     */
56
    protected $objectFactory;
57
58
    /**
59
     * Constructor
60
     *
61
     * @param ObjectFactoryInterface $objectFactory Object factory
62
     * @param RepositoryInterface $repository Repository instance
63
     */
64
    public function __construct(ObjectFactoryInterface $objectFactory, RepositoryInterface $repository)
65
    {
66
        $this->objectFactory = $objectFactory;
67
        $this->objectFactory->setRepository($repository);
68
    }
69
70
    /**
71
     * Build the repository by configuration
72
     *
73
     * @param array $config Object configuration
74
     */
75
    public function generate(array $config)
76
    {
77
        // Instantiate a repository iterator
78
        /** @var RepositoryIterator $repositoryIterator */
79
        $repositoryIterator = Kernel::create(RepositoryIterator::class, [$config]);
80
81
        $objectId = 0;
82
83
        // Iterate through the object stack
84
        /**
85
         * @var \DateTime $creationDate Creation date
86
         * @var array $objectConfig Object configuration
87
         */
88
        foreach ($repositoryIterator as $creationDate => $objectConfig) {
89
            $this->objectFactory->create($creationDate, ++$objectId, $objectConfig);
0 ignored issues
show
Documentation introduced by
$creationDate is of type integer|string, but the function expects a object<DateTime>.

It seems like the type of the argument is not accepted by the function/method which you are calling.

In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug.

We suggest to add an explicit type cast like in the following example:

function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
Loading history...
90
        }
91
    }
92
}