This project does not seem to handle request data directly as such no vulnerable execution paths were found.
include
, or for example
via PHP's auto-loading mechanism.
These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more
1 | <?php |
||
2 | /** |
||
3 | * Finance module for HiPanel |
||
4 | * |
||
5 | * @link https://github.com/hiqdev/hipanel-module-finance |
||
6 | * @package hipanel-module-finance |
||
7 | * @license BSD-3-Clause |
||
8 | * @copyright Copyright (c) 2015-2019, HiQDev (http://hiqdev.com/) |
||
9 | */ |
||
10 | |||
11 | namespace hipanel\modules\finance\forms; |
||
12 | |||
13 | use hipanel\modules\finance\logic\IntegrityException; |
||
14 | use hipanel\modules\finance\models\CertificateResource; |
||
15 | use yii\web\UnprocessableEntityHttpException; |
||
16 | |||
17 | /** |
||
18 | * Class CertificateTariffForm. |
||
19 | * |
||
20 | * @author Dmytro Naumenko <[email protected]> |
||
21 | */ |
||
22 | class CertificateTariffForm extends AbstractTariffForm |
||
23 | { |
||
24 | protected $certificateTypes = []; |
||
25 | |||
26 | public function __construct(array $config = []) |
||
27 | { |
||
28 | parent::__construct($config); |
||
29 | } |
||
30 | |||
31 | View Code Duplication | public function load($data, $formName = null) |
|
0 ignored issues
–
show
|
|||
32 | { |
||
33 | $this->setAttributes($data[$this->formName()]); |
||
34 | $this->setResources($data[(new CertificateResource())->formName()]); |
||
0 ignored issues
–
show
The class
hipanel\modules\finance\models\CertificateResource has been deprecated with message: Is implemented in plan module
This class, trait or interface has been deprecated. The supplier of the file has supplied an explanatory message. The explanatory message should give you some clue as to whether and when the type will be removed from the class and what other constant to use instead.
Loading history...
|
|||
35 | |||
36 | $this->initTariff(); |
||
37 | |||
38 | return true; |
||
39 | } |
||
40 | |||
41 | /** |
||
42 | * @return array |
||
43 | */ |
||
44 | public static function getPeriods() |
||
45 | { |
||
46 | return CertificateResource::getPeriods(); |
||
47 | } |
||
48 | |||
49 | public function setResources($resources) |
||
50 | { |
||
51 | $result = []; |
||
52 | foreach ($resources as $resource) { |
||
53 | $result[] = $this->createResource($resource); |
||
54 | } |
||
55 | |||
56 | $this->_resources = $result; |
||
57 | |||
58 | return $this; |
||
59 | } |
||
60 | |||
61 | protected function createResource($resource) |
||
62 | { |
||
63 | if ($resource instanceof CertificateResource) { |
||
64 | return $resource; |
||
65 | } |
||
66 | |||
67 | $model = new CertificateResource(['scenario' => $this->scenario ?: 'default']); |
||
68 | if ($model->load($resource, '') && $model->validate()) { |
||
69 | return $model; |
||
70 | } else { |
||
71 | throw new UnprocessableEntityHttpException('Failed to load resource model: ' . reset($model->getFirstErrors())); |
||
0 ignored issues
–
show
|
|||
72 | } |
||
73 | } |
||
74 | |||
75 | public function getCertificateTypes() |
||
76 | { |
||
77 | $result = []; |
||
78 | |||
79 | foreach ($this->tariff->resources as $resource) { |
||
80 | if (isset($this->certificateTypes[$resource->object_id])) { |
||
81 | $result[$resource->object_id] = $this->certificateTypes[$resource->object_id]; |
||
82 | } |
||
83 | } |
||
84 | |||
85 | return $result; |
||
86 | } |
||
87 | |||
88 | protected function getCertificateTypeId($type) |
||
89 | { |
||
90 | return array_search($type, $this->certificateTypes, true); |
||
91 | } |
||
92 | |||
93 | /** |
||
94 | * @param $type |
||
95 | * @throws IntegrityException |
||
96 | * @return CertificateResource[] |
||
97 | */ |
||
98 | public function getTypeResources($type) |
||
99 | { |
||
100 | return $this->extractResources($type, $this->tariff->resources); |
||
101 | } |
||
102 | |||
103 | /** |
||
104 | * @param array $certificateTypes |
||
105 | */ |
||
106 | public function setCertificateTypes($certificateTypes) |
||
107 | { |
||
108 | $this->certificateTypes = $certificateTypes; |
||
109 | } |
||
110 | |||
111 | /** |
||
112 | * @param $type |
||
113 | * @throws IntegrityException |
||
114 | * @return CertificateResource[] |
||
115 | */ |
||
116 | public function getTypeParentResources($type) |
||
117 | { |
||
118 | return $this->extractResources($type, $this->parentTariff->resources); |
||
119 | } |
||
120 | |||
121 | protected function extractResources($certificateType, $resources) |
||
122 | { |
||
123 | $id = $this->getCertificateTypeId($certificateType); |
||
124 | |||
125 | $tmpres = []; |
||
126 | |||
127 | foreach ($resources as $resource) { |
||
128 | if (strcmp($resource->object_id, $id) === 0 && $resource->isTypeCorrect()) { |
||
129 | $tmpres[$resource->type] = $resource; |
||
130 | } |
||
131 | } |
||
132 | |||
133 | $types = $resource->getTypes(); |
||
0 ignored issues
–
show
The variable
$resource seems to be defined by a foreach iteration on line 127 . Are you sure the iterator is never empty, otherwise this variable is not defined?
It seems like you are relying on a variable being defined by an iteration: foreach ($a as $b) {
}
// $b is defined here only if $a has elements, for example if $a is array()
// then $b would not be defined here. To avoid that, we recommend to set a
// default value for $b.
// Better
$b = 0; // or whatever default makes sense in your context
foreach ($a as $b) {
}
// $b is now guaranteed to be defined here.
Loading history...
|
|||
134 | /* XXX why die? let's try with empty resource |
||
135 | * if (count($tmpres) !== count($types)) { |
||
136 | throw new IntegrityException('Found ' . count($tmpres) . ' resources for certificate "' . $type . '". Must be exactly ' . count($types)); |
||
137 | } |
||
138 | |||
139 | // sorts $tmpres by order of $resource->getTypes() |
||
140 | $tmpres = array_merge($types, $tmpres); |
||
141 | */ |
||
142 | |||
143 | foreach (array_keys($types) as $type) { |
||
144 | $res[$type] = isset($tmpres[$type]) ? $tmpres[$type] : $this->createResource([ |
||
145 | 'object_id' => $id, |
||
146 | 'type' => $type, |
||
147 | ]); |
||
148 | } |
||
149 | |||
150 | return $res; |
||
151 | } |
||
152 | } |
||
153 |
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.