These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more
1 | <?php |
||
2 | |||
3 | /** |
||
4 | * TechDivision\Import\Loaders\ArtefactsLoader |
||
5 | * |
||
6 | * NOTICE OF LICENSE |
||
7 | * |
||
8 | * This source file is subject to the Open Software License (OSL 3.0) |
||
9 | * that is available through the world-wide-web at this URL: |
||
10 | * http://opensource.org/licenses/osl-3.0.php |
||
11 | * |
||
12 | * PHP version 5 |
||
13 | * |
||
14 | * @author Tim Wagner <[email protected]> |
||
15 | * @copyright 2019 TechDivision GmbH <[email protected]> |
||
16 | * @license http://opensource.org/licenses/osl-3.0.php Open Software License (OSL 3.0) |
||
17 | * @link https://github.com/techdivision/import |
||
18 | * @link http://www.techdivision.com |
||
19 | */ |
||
20 | |||
21 | namespace TechDivision\Import\Loaders; |
||
22 | |||
23 | use TechDivision\Import\Utils\RegistryKeys; |
||
24 | use TechDivision\Import\Services\RegistryProcessorInterface; |
||
25 | |||
26 | /** |
||
27 | * Loader for import artefacts. |
||
28 | * |
||
29 | * @author Tim Wagner <[email protected]> |
||
30 | * @copyright 2019 TechDivision GmbH <[email protected]> |
||
31 | * @license http://opensource.org/licenses/osl-3.0.php Open Software License (OSL 3.0) |
||
32 | * @link https://github.com/techdivision/import |
||
33 | * @link http://www.techdivision.com |
||
34 | */ |
||
35 | class ArtefactsLoader implements LoaderInterface |
||
36 | { |
||
37 | |||
38 | /** |
||
39 | * The registry processor instance. |
||
40 | * |
||
41 | * @var \TechDivision\Import\Services\RegistryProcessorInterface |
||
42 | */ |
||
43 | protected $registryProcessor; |
||
44 | |||
45 | /** |
||
46 | * Construct that initializes the iterator with the registry processor instance. |
||
47 | * |
||
48 | * @param \TechDivision\Import\Services\RegistryProcessorInterface $registryProcessor The registry processor instance |
||
49 | */ |
||
50 | public function __construct(RegistryProcessorInterface $registryProcessor) |
||
51 | { |
||
52 | $this->registryProcessor = $registryProcessor; |
||
53 | } |
||
54 | |||
55 | /** |
||
56 | * Loads and returns data. |
||
57 | * |
||
58 | * @return \ArrayAccess The array with the data |
||
59 | */ |
||
60 | public function load() |
||
61 | { |
||
62 | |||
63 | // initialize the array for the directories that has to be cleared |
||
64 | $filenames = array(); |
||
65 | |||
66 | // load the actual status |
||
67 | $status = $this->getRegistryProcessor()->getAttribute(RegistryKeys::STATUS); |
||
68 | |||
69 | // query whether or not we've import artefacts |
||
70 | if (isset($status[RegistryKeys::FILES])) { |
||
71 | // load the import artefacts from the import status |
||
72 | $files = $status[RegistryKeys::FILES]; |
||
73 | |||
74 | // append the files |
||
75 | foreach (array_keys($files) as $filename) { |
||
76 | $filenames[] = $filename; |
||
77 | } |
||
78 | } |
||
79 | |||
80 | // return the directories to be cleared |
||
81 | return $filenames; |
||
0 ignored issues
–
show
|
|||
82 | } |
||
83 | |||
84 | /** |
||
85 | * Return's the registry processor instance. |
||
86 | * |
||
87 | * @return \TechDivision\Import\Services\RegistryProcessorInterface The processor instance |
||
88 | */ |
||
89 | protected function getRegistryProcessor() |
||
90 | { |
||
91 | return $this->registryProcessor; |
||
92 | } |
||
93 | } |
||
94 |
If you return a value from a function or method, it should be a sub-type of the type that is given by the parent type f.e. an interface, or abstract method. This is more formally defined by the Lizkov substitution principle, and guarantees that classes that depend on the parent type can use any instance of a child type interchangably. This principle also belongs to the SOLID principles for object oriented design.
Let’s take a look at an example:
Our function
my_function
expects aPost
object, and outputs the author of the post. The base classPost
returns a simple string and outputting a simple string will work just fine. However, the child classBlogPost
which is a sub-type ofPost
instead decided to return anobject
, and is therefore violating the SOLID principles. If aBlogPost
were passed tomy_function
, PHP would not complain, but ultimately fail when executing thestrtoupper
call in its body.