Skip to content

Commit 42edfaf

Browse files
committed
Fixing phpcs and phpcbf
1 parent 7ec7d65 commit 42edfaf

File tree

1 file changed

+37
-31
lines changed

1 file changed

+37
-31
lines changed

app/code/Magento/SalesRule/Model/RulesApplier.php

Lines changed: 37 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,10 @@
77
namespace Magento\SalesRule\Model;
88

99
use Magento\Quote\Model\Quote\Address;
10+
use Magento\Quote\Model\Quote\Item\AbstractItem;
1011
use Magento\SalesRule\Model\Quote\ChildrenValidationLocator;
1112
use Magento\Framework\App\ObjectManager;
13+
use Magento\SalesRule\Model\ResourceModel\Rule\Collection;
1214
use Magento\SalesRule\Model\Rule\Action\Discount\CalculatorFactory;
1315

1416
/**
@@ -42,9 +44,9 @@ class RulesApplier
4244

4345
/**
4446
* @param \Magento\SalesRule\Model\Rule\Action\Discount\CalculatorFactory $calculatorFactory
45-
* @param \Magento\Framework\Event\ManagerInterface $eventManager
46-
* @param \Magento\SalesRule\Model\Utility $utility
47-
* @param ChildrenValidationLocator|null $childrenValidationLocator
47+
* @param \Magento\Framework\Event\ManagerInterface $eventManager
48+
* @param \Magento\SalesRule\Model\Utility $utility
49+
* @param ChildrenValidationLocator|null $childrenValidationLocator
4850
*/
4951
public function __construct(
5052
\Magento\SalesRule\Model\Rule\Action\Discount\CalculatorFactory $calculatorFactory,
@@ -62,18 +64,18 @@ public function __construct(
6264
/**
6365
* Apply rules to current order item
6466
*
65-
* @param \Magento\Quote\Model\Quote\Item\AbstractItem $item
66-
* @param \Magento\SalesRule\Model\ResourceModel\Rule\Collection $rules
67-
* @param bool $skipValidation
68-
* @param mixed $couponCode
69-
* @return array
67+
* @param AbstractItem $item
68+
* @param Collection $rules
69+
* @param bool $skipValidation
70+
* @param mixed $couponCode
71+
* @return array
7072
* @SuppressWarnings(PHPMD.CyclomaticComplexity)
7173
*/
7274
public function applyRules($item, $rules, $skipValidation, $couponCode)
7375
{
7476
$address = $item->getAddress();
7577
$appliedRuleIds = [];
76-
/* @var $rule \Magento\SalesRule\Model\Rule */
78+
/* @var $rule Rule */
7779
foreach ($rules as $rule) {
7880
if (!$this->validatorUtility->canProcessRule($rule, $address)) {
7981
continue;
@@ -111,8 +113,8 @@ public function applyRules($item, $rules, $skipValidation, $couponCode)
111113
/**
112114
* Add rule discount description label to address object
113115
*
114-
* @param Address $address
115-
* @param \Magento\SalesRule\Model\Rule $rule
116+
* @param Address $address
117+
* @param Rule $rule
116118
* @return $this
117119
*/
118120
public function addDiscountDescription($address, $rule)
@@ -142,10 +144,12 @@ public function addDiscountDescription($address, $rule)
142144
}
143145

144146
/**
145-
* @param \Magento\Quote\Model\Quote\Item\AbstractItem $item
146-
* @param \Magento\SalesRule\Model\Rule $rule
147-
* @param \Magento\Quote\Model\Quote\Address $address
148-
* @param mixed $couponCode
147+
* Apply Rule
148+
*
149+
* @param AbstractItem $item
150+
* @param Rule $rule
151+
* @param \Magento\Quote\Model\Quote\Address $address
152+
* @param mixed $couponCode
149153
* @return $this
150154
*/
151155
protected function applyRule($item, $rule, $address, $couponCode)
@@ -160,8 +164,10 @@ protected function applyRule($item, $rule, $address, $couponCode)
160164
}
161165

162166
/**
163-
* @param \Magento\Quote\Model\Quote\Item\AbstractItem $item
164-
* @param \Magento\SalesRule\Model\Rule $rule
167+
* Get discount Data
168+
*
169+
* @param AbstractItem $item
170+
* @param Rule $rule
165171
* @return \Magento\SalesRule\Model\Rule\Action\Discount\Data
166172
*/
167173
protected function getDiscountData($item, $rule)
@@ -187,7 +193,7 @@ protected function getDiscountData($item, $rule)
187193

188194
/**
189195
* @param \Magento\SalesRule\Model\Rule\Action\Discount\Data $discountData
190-
* @param \Magento\Quote\Model\Quote\Item\AbstractItem $item
196+
* @param AbstractItem $item
191197
* @return $this
192198
*/
193199
protected function setDiscountData($discountData, $item)
@@ -203,9 +209,9 @@ protected function setDiscountData($discountData, $item)
203209
/**
204210
* Set coupon code to address if $rule contains validated coupon
205211
*
206-
* @param Address $address
207-
* @param \Magento\SalesRule\Model\Rule $rule
208-
* @param mixed $couponCode
212+
* @param Address $address
213+
* @param Rule $rule
214+
* @param mixed $couponCode
209215
* @return $this
210216
*/
211217
public function maintainAddressCouponCode($address, $rule, $couponCode)
@@ -214,7 +220,7 @@ public function maintainAddressCouponCode($address, $rule, $couponCode)
214220
Rule is a part of rules collection, which includes only rules with 'No Coupon' type or with validated coupon.
215221
As a result, if rule uses coupon code(s) ('Specific' or 'Auto' Coupon Type), it always contains validated coupon
216222
*/
217-
if ($rule->getCouponType() != \Magento\SalesRule\Model\Rule::COUPON_TYPE_NO_COUPON) {
223+
if ($rule->getCouponType() != Rule::COUPON_TYPE_NO_COUPON) {
218224
$address->setCouponCode($couponCode);
219225
}
220226

@@ -224,16 +230,16 @@ public function maintainAddressCouponCode($address, $rule, $couponCode)
224230
/**
225231
* Fire event to allow overwriting of discount amounts
226232
*
227-
* @param \Magento\SalesRule\Model\Rule\Action\Discount\Data $discountData
228-
* @param \Magento\Quote\Model\Quote\Item\AbstractItem $item
229-
* @param \Magento\SalesRule\Model\Rule $rule
230-
* @param float $qty
233+
* @param \Magento\SalesRule\Model\Rule\Action\Discount\Data $discountData
234+
* @param AbstractItem $item
235+
* @param Rule $rule
236+
* @param float $qty
231237
* @return $this
232238
*/
233239
protected function eventFix(
234240
\Magento\SalesRule\Model\Rule\Action\Discount\Data $discountData,
235-
\Magento\Quote\Model\Quote\Item\AbstractItem $item,
236-
\Magento\SalesRule\Model\Rule $rule,
241+
AbstractItem $item,
242+
Rule $rule,
237243
$qty
238244
) {
239245
$quote = $item->getQuote();
@@ -255,11 +261,11 @@ protected function eventFix(
255261
}
256262

257263
/**
258-
* @param \Magento\Quote\Model\Quote\Item\AbstractItem $item
259-
* @param int[] $appliedRuleIds
264+
* @param AbstractItem $item
265+
* @param int[] $appliedRuleIds
260266
* @return $this
261267
*/
262-
public function setAppliedRuleIds(\Magento\Quote\Model\Quote\Item\AbstractItem $item, array $appliedRuleIds)
268+
public function setAppliedRuleIds(AbstractItem $item, array $appliedRuleIds)
263269
{
264270
$address = $item->getAddress();
265271
$quote = $item->getQuote();

0 commit comments

Comments
 (0)