@@ -127,7 +127,7 @@ |
||
127 | 127 | } |
128 | 128 | |
129 | 129 | /** |
130 | - * @param \Spryker\Zed\Discount\Dependency\Plugin\DecisionRulePluginInterface|\Spryker\Zed\Discount\Dependency\Plugin\DiscountRuleWithAttributesPluginInterface $specificationPlugin |
|
130 | + * @param DiscountRuleWithAttributesPluginInterface $specificationPlugin |
|
131 | 131 | * |
132 | 132 | * @return array |
133 | 133 | */ |
@@ -228,7 +228,7 @@ discard block |
||
228 | 228 | } |
229 | 229 | |
230 | 230 | /** |
231 | - * @return \Spryker\Zed\Discount\Dependency\Plugin\DecisionRulePluginInterface[] |
|
231 | + * @return SkuDecisionRulePlugin[] |
|
232 | 232 | */ |
233 | 233 | protected function createDecisionRulePlugins() |
234 | 234 | { |
@@ -248,7 +248,7 @@ discard block |
||
248 | 248 | } |
249 | 249 | |
250 | 250 | /** |
251 | - * @return \Spryker\Zed\Discount\Dependency\Plugin\CollectorPluginInterface[] |
|
251 | + * @return ItemBySkuCollectorPlugin[] |
|
252 | 252 | */ |
253 | 253 | protected function createCollectorPlugins() |
254 | 254 | { |
@@ -582,7 +582,7 @@ discard block |
||
582 | 582 | } |
583 | 583 | |
584 | 584 | /** |
585 | - * @return \PHPUnit_Framework_MockObject_MockObject|\Spryker\Zed\Discount\Dependency\Plugin\DiscountCalculatorPluginInterface |
|
585 | + * @return DiscountCalculatorPluginInterface|null |
|
586 | 586 | */ |
587 | 587 | protected function createCalculatorPluginMock() |
588 | 588 | { |
@@ -252,7 +252,7 @@ |
||
252 | 252 | */ |
253 | 253 | protected function createCollectorPlugins() |
254 | 254 | { |
255 | - $collectorProviderPlugins[] = new ItemBySkuCollectorPlugin(); |
|
255 | + $collectorProviderPlugins[] = new ItemBySkuCollectorPlugin(); |
|
256 | 256 | |
257 | 257 | return $collectorProviderPlugins; |
258 | 258 | } |
@@ -486,7 +486,7 @@ discard block |
||
486 | 486 | |
487 | 487 | $calculatorMock |
488 | 488 | ->method('calculateDiscount') |
489 | - ->willReturnCallback(function ($discountableItems, DiscountTransfer $discountTransfer) { |
|
489 | + ->willReturnCallback(function($discountableItems, DiscountTransfer $discountTransfer) { |
|
490 | 490 | return $discountTransfer->getAmount(); |
491 | 491 | }); |
492 | 492 | |
@@ -550,7 +550,7 @@ discard block |
||
550 | 550 | $calculatorPluginMock = $this->createCalculatorPluginMock(); |
551 | 551 | $calculatorPluginMock |
552 | 552 | ->method('calculateDiscount') |
553 | - ->willReturnCallback(function ($discountableItems, DiscountTransfer $discountTransfer) { |
|
553 | + ->willReturnCallback(function($discountableItems, DiscountTransfer $discountTransfer) { |
|
554 | 554 | return $discountTransfer->getAmount(); |
555 | 555 | }); |
556 | 556 | } |
@@ -54,7 +54,7 @@ |
||
54 | 54 | /** |
55 | 55 | * @param \Spryker\Zed\Discount\Business\QueryString\ComparatorOperatorsInterface|null $comparatorMock |
56 | 56 | * |
57 | - * @return \Spryker\Zed\Discount\Business\Collector\ItemPriceCollector |
|
57 | + * @return ItemQuantityCollector |
|
58 | 58 | */ |
59 | 59 | protected function createItemQuantityCollector(ComparatorOperatorsInterface $comparatorMock = null) |
60 | 60 | { |
@@ -47,7 +47,7 @@ |
||
47 | 47 | /** |
48 | 48 | * @param \Spryker\Zed\Discount\Business\QueryString\ComparatorOperatorsInterface|null $comparatorMock |
49 | 49 | * |
50 | - * @return \Spryker\Zed\Discount\Business\DecisionRule\ItemSkuDecisionRule |
|
50 | + * @return ItemQuantityDecisionRule |
|
51 | 51 | */ |
52 | 52 | protected function createItemQuantityDecisionRule(ComparatorOperatorsInterface $comparatorMock = null) |
53 | 53 | { |
@@ -30,7 +30,7 @@ |
||
30 | 30 | public function testDecisionRuleWhenCurrentItemQuantityMatchesShouldReturnTrue() |
31 | 31 | { |
32 | 32 | $comparatorMock = $this->createComparatorMock(); |
33 | - $comparatorMock->method('compare')->willReturnCallback(function (ClauseTransfer $clauseTransfer, $itemQuantity) { |
|
33 | + $comparatorMock->method('compare')->willReturnCallback(function(ClauseTransfer $clauseTransfer, $itemQuantity) { |
|
34 | 34 | return $clauseTransfer->getValue() === $itemQuantity; |
35 | 35 | }); |
36 | 36 |
@@ -53,7 +53,7 @@ |
||
53 | 53 | /** |
54 | 54 | * @param \Spryker\Zed\Discount\Business\QueryString\ComparatorOperatorsInterface|null $comparatorMock |
55 | 55 | * |
56 | - * @return \Spryker\Zed\Discount\Business\DecisionRule\GrandTotalDecisionRule |
|
56 | + * @return TotalQuantityDecisionRule |
|
57 | 57 | */ |
58 | 58 | protected function createTotalQuantityDecisionRule(ComparatorOperatorsInterface $comparatorMock = null) |
59 | 59 | { |
@@ -30,7 +30,7 @@ |
||
30 | 30 | public function testWhenTotalQuantityMatchesClauseShouldReturnTrue() |
31 | 31 | { |
32 | 32 | $comparatorMock = $this->createComparatorMock(); |
33 | - $comparatorMock->method('compare')->willReturnCallback(function (ClauseTransfer $clauseTransfer, $grandTotal) { |
|
33 | + $comparatorMock->method('compare')->willReturnCallback(function(ClauseTransfer $clauseTransfer, $grandTotal) { |
|
34 | 34 | return $clauseTransfer->getValue() === $grandTotal; |
35 | 35 | }); |
36 | 36 |
@@ -362,7 +362,7 @@ |
||
362 | 362 | } |
363 | 363 | |
364 | 364 | /** |
365 | - * @return \PHPUnit_Framework_MockObject_MockObject|\Spryker\Zed\Discount\Dependency\Plugin\DiscountableItemFilterPluginInterface |
|
365 | + * @return DiscountableItemFilterPluginInterface |
|
366 | 366 | */ |
367 | 367 | protected function createDiscountableItemFilterPluginMock() |
368 | 368 | { |
@@ -181,7 +181,7 @@ discard block |
||
181 | 181 | $filterPluginMock |
182 | 182 | ->expects($this->once()) |
183 | 183 | ->method('filter') |
184 | - ->willReturnCallback(function (CollectedDiscountTransfer $collectedDiscountTransfer) { |
|
184 | + ->willReturnCallback(function(CollectedDiscountTransfer $collectedDiscountTransfer) { |
|
185 | 185 | |
186 | 186 | $discountableItems = new ArrayObject(); |
187 | 187 | foreach ($collectedDiscountTransfer->getDiscountableItems() as $discountableItemTransfer) { |
@@ -319,37 +319,37 @@ discard block |
||
319 | 319 | |
320 | 320 | $container = new Container(); |
321 | 321 | |
322 | - $container[DiscountDependencyProvider::PLUGIN_COLLECTOR_STRATEGY_PLUGINS] = function () use ($collectorStrategyPlugins) { |
|
322 | + $container[DiscountDependencyProvider::PLUGIN_COLLECTOR_STRATEGY_PLUGINS] = function() use ($collectorStrategyPlugins) { |
|
323 | 323 | return $collectorStrategyPlugins; |
324 | 324 | }; |
325 | 325 | |
326 | - $container[DiscountDependencyProvider::DECISION_RULE_PLUGINS] = function () use ($decisionRulePlugins) { |
|
326 | + $container[DiscountDependencyProvider::DECISION_RULE_PLUGINS] = function() use ($decisionRulePlugins) { |
|
327 | 327 | return $decisionRulePlugins; |
328 | 328 | }; |
329 | 329 | |
330 | - $container[DiscountDependencyProvider::FACADE_MESSENGER] = function () use ($messengerFacade) { |
|
330 | + $container[DiscountDependencyProvider::FACADE_MESSENGER] = function() use ($messengerFacade) { |
|
331 | 331 | return $messengerFacade; |
332 | 332 | }; |
333 | 333 | |
334 | - $container[DiscountDependencyProvider::COLLECTOR_PLUGINS] = function () use ($collectorPlugins) { |
|
334 | + $container[DiscountDependencyProvider::COLLECTOR_PLUGINS] = function() use ($collectorPlugins) { |
|
335 | 335 | return $collectorPlugins; |
336 | 336 | }; |
337 | 337 | |
338 | - $container[DiscountDependencyProvider::CALCULATOR_PLUGINS] = function () use ($calculatorPlugins) { |
|
338 | + $container[DiscountDependencyProvider::CALCULATOR_PLUGINS] = function() use ($calculatorPlugins) { |
|
339 | 339 | return $calculatorPlugins; |
340 | 340 | }; |
341 | 341 | |
342 | - $container[DiscountDependencyProvider::PLUGIN_DISCOUNTABLE_ITEM_FILTER] = function () use ($filterPluginMock) { |
|
342 | + $container[DiscountDependencyProvider::PLUGIN_DISCOUNTABLE_ITEM_FILTER] = function() use ($filterPluginMock) { |
|
343 | 343 | return [ |
344 | 344 | $filterPluginMock |
345 | 345 | ]; |
346 | 346 | }; |
347 | 347 | |
348 | - $container[DiscountDependencyProvider::PLUGIN_DISCOUNT_APPLICABLE_FILTER_PLUGINS] = function () use ($applicableFilterPlugins) { |
|
348 | + $container[DiscountDependencyProvider::PLUGIN_DISCOUNT_APPLICABLE_FILTER_PLUGINS] = function() use ($applicableFilterPlugins) { |
|
349 | 349 | return $applicableFilterPlugins; |
350 | 350 | }; |
351 | 351 | |
352 | - $container[DiscountDependencyProvider::FACADE_CURRENCY] = function () use ($currencyFacade) { |
|
352 | + $container[DiscountDependencyProvider::FACADE_CURRENCY] = function() use ($currencyFacade) { |
|
353 | 353 | return $currencyFacade; |
354 | 354 | }; |
355 | 355 |
@@ -739,7 +739,7 @@ |
||
739 | 739 | } |
740 | 740 | |
741 | 741 | /** |
742 | - * @return \Generated\Shared\Transfer\DiscountConfiguratorTransfer |
|
742 | + * @return string|null |
|
743 | 743 | */ |
744 | 744 | protected function createDiscountConfiguratorTransfer() |
745 | 745 | { |
@@ -780,7 +780,7 @@ |
||
780 | 780 | $discountFacade = $this->createDiscountFacade(); |
781 | 781 | $factory = new DiscountBusinessFactory(); |
782 | 782 | $container = new Container(); |
783 | - $container[$dependencyType] = function () use ($discountRulePluginMock) { |
|
783 | + $container[$dependencyType] = function() use ($discountRulePluginMock) { |
|
784 | 784 | return [ |
785 | 785 | $discountRulePluginMock, |
786 | 786 | ]; |
@@ -198,7 +198,7 @@ |
||
198 | 198 | } |
199 | 199 | |
200 | 200 | /** |
201 | - * @param array $discountSaverMethods |
|
201 | + * @param string[] $discountSaverMethods |
|
202 | 202 | * @param array $queryContainerMethods |
203 | 203 | * |
204 | 204 | * @return \PHPUnit_Framework_MockObject_MockObject|\Spryker\Zed\Discount\Business\Persistence\DiscountOrderSaver |
@@ -113,7 +113,7 @@ discard block |
||
113 | 113 | } |
114 | 114 | |
115 | 115 | /** |
116 | - * @return \PHPUnit_Framework_MockObject_MockObject|\Spryker\Zed\Discount\Dependency\Plugin\CollectorPluginInterface |
|
116 | + * @return null|CollectorPluginInterface |
|
117 | 117 | */ |
118 | 118 | protected function createCollectorPluginMock() |
119 | 119 | { |
@@ -121,7 +121,7 @@ discard block |
||
121 | 121 | } |
122 | 122 | |
123 | 123 | /** |
124 | - * @return \PHPUnit_Framework_MockObject_MockObject|\Spryker\Zed\Discount\Business\QueryString\Specification\CollectorSpecification\CollectorSpecificationInterface |
|
124 | + * @return CollectorSpecificationInterface |
|
125 | 125 | */ |
126 | 126 | protected function createCollectorSpecificationMock() |
127 | 127 | { |