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 | namespace Loevgaard\AltaPay\Payload; |
||
3 | |||
4 | use Assert\Assert; |
||
5 | use Loevgaard\AltaPay; |
||
6 | use Money\Money; |
||
7 | |||
8 | class OrderLine extends Payload implements OrderLineInterface |
||
9 | { |
||
10 | const GOODS_TYPE_SHIPMENT = 'shipment'; |
||
11 | const GOODS_TYPE_HANDLING = 'handling'; |
||
12 | const GOODS_TYPE_ITEM = 'item'; |
||
13 | const GOODS_TYPE_REFUND = 'refund'; |
||
14 | |||
15 | /** |
||
16 | * Used to create Money objects |
||
17 | * |
||
18 | * @var string |
||
19 | */ |
||
20 | protected $currency; |
||
21 | |||
22 | /** |
||
23 | * @var string |
||
24 | */ |
||
25 | protected $description; |
||
26 | |||
27 | /** |
||
28 | * @var string |
||
29 | */ |
||
30 | protected $itemId; |
||
31 | |||
32 | /** |
||
33 | * @var float |
||
34 | */ |
||
35 | protected $quantity; |
||
36 | |||
37 | /** |
||
38 | * @var int |
||
39 | */ |
||
40 | protected $unitPrice; |
||
41 | |||
42 | /** |
||
43 | * @var float |
||
44 | */ |
||
45 | protected $taxPercent; |
||
46 | |||
47 | /** |
||
48 | * @var int |
||
49 | */ |
||
50 | protected $taxAmount; |
||
51 | |||
52 | /** |
||
53 | * @var string |
||
54 | */ |
||
55 | protected $unitCode; |
||
56 | |||
57 | /** |
||
58 | * @var int |
||
59 | */ |
||
60 | protected $discount; |
||
61 | |||
62 | /** |
||
63 | * @var string |
||
64 | */ |
||
65 | protected $goodsType; |
||
66 | |||
67 | /** |
||
68 | * @var string |
||
69 | */ |
||
70 | protected $imageUrl; |
||
71 | |||
72 | 15 | public function __construct(string $description, string $itemId, float $quantity, Money $unitPrice) |
|
73 | { |
||
74 | 15 | $this->currency = $unitPrice->getCurrency()->getCode(); |
|
75 | |||
76 | 15 | $this->setDescription($description); |
|
77 | 15 | $this->setItemId($itemId); |
|
78 | 15 | $this->setQuantity($quantity); |
|
79 | 15 | $this->setUnitPrice($unitPrice); |
|
80 | 15 | } |
|
81 | |||
82 | 3 | public function getPayload() : array |
|
83 | { |
||
84 | 3 | $this->validate(); |
|
85 | |||
86 | $payload = [ |
||
87 | 3 | 'description' => $this->description, |
|
88 | 3 | 'itemId' => $this->itemId, |
|
89 | 3 | 'quantity' => $this->quantity, |
|
90 | 3 | 'unitPrice' => AltaPay\floatFromMoney($this->getUnitPrice()), |
|
91 | 3 | 'taxPercent' => $this->taxPercent, |
|
92 | 3 | 'taxAmount' => AltaPay\floatFromMoney($this->getTaxAmount()), |
|
93 | 3 | 'unitCode' => $this->unitCode, |
|
94 | 3 | 'discount' => AltaPay\floatFromMoney($this->getDiscount()), |
|
95 | 3 | 'goodsType' => $this->goodsType, |
|
96 | 3 | 'imageUrl' => $this->imageUrl, |
|
97 | ]; |
||
98 | |||
99 | 3 | return static::simplePayload($payload); |
|
100 | } |
||
101 | |||
102 | 3 | public function validate() |
|
103 | { |
||
104 | 3 | Assert::that($this->description)->string(); |
|
105 | 3 | Assert::that($this->itemId)->string(); |
|
106 | 3 | Assert::that($this->quantity)->float(); |
|
107 | 3 | Assert::that($this->getUnitPrice())->isInstanceOf(Money::class); |
|
108 | 3 | Assert::thatNullOr($this->taxPercent)->float(); |
|
109 | 3 | Assert::thatNullOr($this->getTaxAmount())->isInstanceOf(Money::class); |
|
110 | 3 | Assert::thatNullOr($this->unitCode)->string(); |
|
111 | 3 | Assert::thatNullOr($this->getDiscount())->isInstanceOf(Money::class); |
|
112 | 3 | Assert::thatNullOr($this->goodsType)->string()->inArray(static::getGoodsTypes()); |
|
113 | 3 | Assert::thatNullOr($this->imageUrl)->string(); |
|
114 | 3 | } |
|
115 | |||
116 | /** |
||
117 | * @return array |
||
118 | */ |
||
119 | 6 | public static function getGoodsTypes() : array |
|
120 | { |
||
121 | return [ |
||
122 | 6 | self::GOODS_TYPE_HANDLING, |
|
123 | 6 | self::GOODS_TYPE_ITEM, |
|
124 | 6 | self::GOODS_TYPE_REFUND, |
|
125 | 6 | self::GOODS_TYPE_SHIPMENT, |
|
126 | ]; |
||
127 | } |
||
128 | |||
129 | /** |
||
130 | * @return string |
||
131 | */ |
||
132 | 3 | public function getDescription() : string |
|
133 | { |
||
134 | 3 | return $this->description; |
|
135 | } |
||
136 | |||
137 | /** |
||
138 | * @param string $description |
||
139 | * @return OrderLine |
||
140 | */ |
||
141 | 15 | public function setDescription(string $description) : self |
|
142 | { |
||
143 | 15 | $this->description = $description; |
|
144 | 15 | return $this; |
|
145 | } |
||
146 | |||
147 | /** |
||
148 | * @return string |
||
149 | */ |
||
150 | 3 | public function getItemId() : string |
|
151 | { |
||
152 | 3 | return $this->itemId; |
|
153 | } |
||
154 | |||
155 | /** |
||
156 | * @param string $itemId |
||
157 | * @return OrderLine |
||
158 | */ |
||
159 | 15 | public function setItemId(string $itemId) : self |
|
160 | { |
||
161 | 15 | $this->itemId = $itemId; |
|
162 | 15 | return $this; |
|
163 | } |
||
164 | |||
165 | /** |
||
166 | * @return float |
||
167 | */ |
||
168 | 3 | public function getQuantity() : float |
|
169 | { |
||
170 | 3 | return $this->quantity; |
|
171 | } |
||
172 | |||
173 | /** |
||
174 | * @param float $quantity |
||
175 | * @return OrderLine |
||
176 | */ |
||
177 | 15 | public function setQuantity(float $quantity) : self |
|
178 | { |
||
179 | 15 | $this->quantity = $quantity; |
|
180 | 15 | return $this; |
|
181 | } |
||
182 | |||
183 | /** |
||
184 | * @return Money |
||
185 | */ |
||
186 | 6 | public function getUnitPrice() : ?Money |
|
187 | { |
||
188 | 6 | return AltaPay\createMoney((string)$this->currency, (int)$this->unitPrice); |
|
189 | } |
||
190 | |||
191 | /** |
||
192 | * @param Money $unitPrice |
||
193 | * @return OrderLine |
||
194 | */ |
||
195 | 15 | public function setUnitPrice(Money $unitPrice) : self |
|
196 | { |
||
197 | 15 | if ($unitPrice->getCurrency()->getCode() !== $this->currency) { |
|
198 | throw new \InvalidArgumentException('The $unitPrice does not have the same currency as this order line'); |
||
199 | } |
||
200 | |||
201 | 15 | $this->unitPrice = $unitPrice->getAmount(); |
|
0 ignored issues
–
show
|
|||
202 | 15 | return $this; |
|
203 | } |
||
204 | |||
205 | /** |
||
206 | * @return float |
||
207 | */ |
||
208 | 3 | public function getTaxPercent() : ?float |
|
209 | { |
||
210 | 3 | return $this->taxPercent; |
|
211 | } |
||
212 | |||
213 | /** |
||
214 | * @param float $taxPercent |
||
215 | * @return OrderLine |
||
216 | */ |
||
217 | 3 | public function setTaxPercent(float $taxPercent) : self |
|
218 | { |
||
219 | 3 | $this->taxPercent = $taxPercent; |
|
220 | 3 | return $this; |
|
221 | } |
||
222 | |||
223 | /** |
||
224 | * @return Money |
||
225 | */ |
||
226 | 6 | public function getTaxAmount() : ?Money |
|
227 | { |
||
228 | 6 | if (is_null($this->taxAmount)) { |
|
229 | 3 | return null; |
|
230 | } |
||
231 | |||
232 | 6 | return AltaPay\createMoney((string)$this->currency, (int)$this->taxAmount); |
|
233 | } |
||
234 | |||
235 | /** |
||
236 | * @param Money $taxAmount |
||
237 | * @return OrderLine |
||
238 | */ |
||
239 | 6 | public function setTaxAmount(Money $taxAmount) : self |
|
240 | { |
||
241 | 6 | if ($taxAmount->getCurrency()->getCode() !== $this->currency) { |
|
242 | throw new \InvalidArgumentException('The $taxAmount does not have the same currency as this order line'); |
||
243 | } |
||
244 | |||
245 | 6 | $this->taxAmount = $taxAmount->getAmount(); |
|
0 ignored issues
–
show
The property
$taxAmount was declared of type integer , but $taxAmount->getAmount() is of type string . Maybe add a type cast?
This check looks for assignments to scalar types that may be of the wrong type. To ensure the code behaves as expected, it may be a good idea to add an explicit type cast. $answer = 42;
$correct = false;
$correct = (bool) $answer;
![]() |
|||
246 | 6 | return $this; |
|
247 | } |
||
248 | |||
249 | /** |
||
250 | * @return string |
||
251 | */ |
||
252 | 3 | public function getUnitCode() : ?string |
|
253 | { |
||
254 | 3 | return $this->unitCode; |
|
255 | } |
||
256 | |||
257 | /** |
||
258 | * @param string $unitCode |
||
259 | * @return OrderLine |
||
260 | */ |
||
261 | 6 | public function setUnitCode(string $unitCode) : self |
|
262 | { |
||
263 | 6 | $this->unitCode = $unitCode; |
|
264 | 6 | return $this; |
|
265 | } |
||
266 | |||
267 | /** |
||
268 | * @return Money |
||
269 | */ |
||
270 | 6 | public function getDiscount() : ?Money |
|
271 | { |
||
272 | 6 | if (is_null($this->discount)) { |
|
273 | 3 | return null; |
|
274 | } |
||
275 | |||
276 | 3 | return AltaPay\createMoney((string)$this->currency, (int)$this->discount); |
|
277 | } |
||
278 | |||
279 | /** |
||
280 | * @param Money $discount |
||
281 | * @return OrderLine |
||
282 | */ |
||
283 | 3 | public function setDiscount(Money $discount) : self |
|
284 | { |
||
285 | 3 | if ($discount->getCurrency()->getCode() !== $this->currency) { |
|
286 | throw new \InvalidArgumentException('The $discount does not have the same currency as this order line'); |
||
287 | } |
||
288 | |||
289 | 3 | $this->discount = $discount->getAmount(); |
|
0 ignored issues
–
show
The property
$discount was declared of type integer , but $discount->getAmount() is of type string . Maybe add a type cast?
This check looks for assignments to scalar types that may be of the wrong type. To ensure the code behaves as expected, it may be a good idea to add an explicit type cast. $answer = 42;
$correct = false;
$correct = (bool) $answer;
![]() |
|||
290 | 3 | return $this; |
|
291 | } |
||
292 | |||
293 | /** |
||
294 | * @return string |
||
295 | */ |
||
296 | 3 | public function getGoodsType() : ?string |
|
297 | { |
||
298 | 3 | return $this->goodsType; |
|
299 | } |
||
300 | |||
301 | /** |
||
302 | * @param string $goodsType |
||
303 | * @return OrderLine |
||
304 | */ |
||
305 | 6 | public function setGoodsType(string $goodsType) : self |
|
306 | { |
||
307 | 6 | $this->goodsType = $goodsType; |
|
308 | 6 | return $this; |
|
309 | } |
||
310 | |||
311 | /** |
||
312 | * @return string |
||
313 | */ |
||
314 | 3 | public function getImageUrl() : ?string |
|
315 | { |
||
316 | 3 | return $this->imageUrl; |
|
317 | } |
||
318 | |||
319 | /** |
||
320 | * @param string $imageUrl |
||
321 | * @return OrderLine |
||
322 | */ |
||
323 | 3 | public function setImageUrl(string $imageUrl) : self |
|
324 | { |
||
325 | 3 | $this->imageUrl = $imageUrl; |
|
326 | 3 | return $this; |
|
327 | } |
||
328 | } |
||
329 |
This check looks for assignments to scalar types that may be of the wrong type.
To ensure the code behaves as expected, it may be a good idea to add an explicit type cast.