Revert "Revert "Merge branch 'cleanmaster' into modules""
This reverts commit b3ac365b45.
Conflicts:
Readme.md
core/lib/Thelia/Controller/Admin/AbstractCrudController.php
core/lib/Thelia/Core/Template/Assets/AssetManagerInterface.php
core/lib/Thelia/Core/Template/Assets/AsseticAssetManager.php
core/lib/Thelia/Core/Template/Smarty/SmartyParser.php
core/lib/Thelia/Core/Template/TemplateDefinition.php
core/lib/Thelia/Model/Base/Coupon.php
core/lib/Thelia/Model/Base/CouponQuery.php
core/lib/Thelia/Model/Base/CouponVersion.php
core/lib/Thelia/Model/Base/CouponVersionQuery.php
core/lib/Thelia/Model/Base/OrderCouponQuery.php
This commit is contained in:
@@ -152,7 +152,6 @@ Sed facilisis pellentesque nisl, eu tincidunt erat scelerisque a. Nullam malesua
|
||||
$serializedConditions = $conditionFactory->serializeConditionCollection($conditions);
|
||||
$coupon1->setSerializedConditions($serializedConditions);
|
||||
|
||||
|
||||
$coupon1->setMaxUsage(40);
|
||||
$coupon1->setIsCumulative(true);
|
||||
$coupon1->setIsRemovingPostage(false);
|
||||
@@ -186,7 +185,6 @@ Sed facilisis pellentesque nisl, eu tincidunt erat scelerisque a. Nullam malesua
|
||||
|
||||
$couponManager = new RemoveXAmount($stubFacade);
|
||||
|
||||
|
||||
$condition1 = new MatchForTotalAmount($stubFacade);
|
||||
$operators = array(
|
||||
MatchForTotalAmount::INPUT1 => Operators::SUPERIOR,
|
||||
@@ -219,7 +217,6 @@ Sed facilisis pellentesque nisl, eu tincidunt erat scelerisque a. Nullam malesua
|
||||
->method('unserializeConditionCollection')
|
||||
->will($this->returnValue($conditions));
|
||||
|
||||
|
||||
$stubContainer->expects($this->any())
|
||||
->method('get')
|
||||
->will($this->onConsecutiveCalls($stubFacade, $couponManager, $stubConditionFactory));
|
||||
@@ -250,8 +247,6 @@ Sed facilisis pellentesque nisl, eu tincidunt erat scelerisque a. Nullam malesua
|
||||
|
||||
$couponManager = new RemoveXAmount($stubFacade);
|
||||
|
||||
|
||||
|
||||
$stubContainer->expects($this->any())
|
||||
->method('get')
|
||||
->will($this->onConsecutiveCalls($stubFacade, $couponManager));
|
||||
@@ -288,7 +283,6 @@ Sed facilisis pellentesque nisl, eu tincidunt erat scelerisque a. Nullam malesua
|
||||
|
||||
$couponManager = new RemoveXAmount($stubFacade);
|
||||
|
||||
|
||||
$condition1 = new MatchForTotalAmount($stubFacade);
|
||||
$operators = array(
|
||||
MatchForTotalAmount::INPUT1 => Operators::SUPERIOR,
|
||||
@@ -321,7 +315,6 @@ Sed facilisis pellentesque nisl, eu tincidunt erat scelerisque a. Nullam malesua
|
||||
->method('unserializeConditionCollection')
|
||||
->will($this->returnValue($conditions));
|
||||
|
||||
|
||||
$stubContainer->expects($this->any())
|
||||
->method('get')
|
||||
->will($this->onConsecutiveCalls($stubFacade, $couponManager, $stubConditionFactory));
|
||||
@@ -353,7 +346,6 @@ Sed facilisis pellentesque nisl, eu tincidunt erat scelerisque a. Nullam malesua
|
||||
|
||||
$couponManager = new RemoveXAmount($stubFacade);
|
||||
|
||||
|
||||
$condition1 = new MatchForTotalAmount($stubFacade);
|
||||
$operators = array(
|
||||
MatchForTotalAmount::INPUT1 => Operators::SUPERIOR,
|
||||
@@ -384,7 +376,6 @@ Sed facilisis pellentesque nisl, eu tincidunt erat scelerisque a. Nullam malesua
|
||||
->method('unserializeConditionCollection')
|
||||
->will($this->returnValue($conditions));
|
||||
|
||||
|
||||
$stubContainer->expects($this->any())
|
||||
->method('get')
|
||||
->will($this->onConsecutiveCalls($stubFacade, $couponManager, $stubConditionFactory));
|
||||
|
||||
@@ -58,7 +58,6 @@ class CouponManagerTest extends \PHPUnit_Framework_TestCase
|
||||
{
|
||||
}
|
||||
|
||||
|
||||
/**
|
||||
* Generate a valid Coupon model
|
||||
*/
|
||||
@@ -114,7 +113,6 @@ Sed facilisis pellentesque nisl, eu tincidunt erat scelerisque a. Nullam malesua
|
||||
$serializedConditions = $conditionFactory->serializeConditionCollection($conditions);
|
||||
$coupon1->setSerializedConditions($serializedConditions);
|
||||
|
||||
|
||||
$coupon1->setMaxUsage(40);
|
||||
$coupon1->setIsCumulative(true);
|
||||
$coupon1->setIsRemovingPostage(true);
|
||||
@@ -154,7 +152,7 @@ Sed facilisis pellentesque nisl, eu tincidunt erat scelerisque a. Nullam malesua
|
||||
|
||||
$couponFactory = new CouponFactory($stubContainer);
|
||||
$model1 = $this->generateCouponModel($stubFacade, $conditionFactory);
|
||||
$model1->setAmount(21.00);
|
||||
$model1->setCode('XMAS')->setIsRemovingPostage(false)->setAmount(21.00);
|
||||
$coupon1 = $couponFactory->buildCouponFromModel($model1);
|
||||
|
||||
$model2 = $this->generateCouponModel($stubFacade, $conditionFactory);
|
||||
@@ -172,12 +170,16 @@ Sed facilisis pellentesque nisl, eu tincidunt erat scelerisque a. Nullam malesua
|
||||
->will($this->returnValue(122.53));
|
||||
|
||||
$couponManager = new CouponManager($stubContainer);
|
||||
|
||||
$couponManager->addAvailableCoupon($coupon1);
|
||||
|
||||
$couponManager->addAvailableCoupon($coupon2);
|
||||
|
||||
$actual = $couponManager->getDiscount();
|
||||
$expected = 50.80;
|
||||
$expected = 21 + 21.50;
|
||||
|
||||
$this->assertEquals($expected, $actual);
|
||||
$this->assertTrue($couponManager->isCouponRemovingPostage());
|
||||
}
|
||||
|
||||
/**
|
||||
@@ -303,8 +305,6 @@ Sed facilisis pellentesque nisl, eu tincidunt erat scelerisque a. Nullam malesua
|
||||
|
||||
$conditionFactory = new ConditionFactory($stubContainer);
|
||||
|
||||
|
||||
|
||||
$stubFacade->expects($this->any())
|
||||
->method('getCurrentCoupons')
|
||||
->will($this->returnValue(array()));
|
||||
@@ -330,7 +330,6 @@ Sed facilisis pellentesque nisl, eu tincidunt erat scelerisque a. Nullam malesua
|
||||
$coupon1 = $couponFactory->buildCouponFromModel($model1);
|
||||
$coupon2 = clone $coupon1;
|
||||
|
||||
|
||||
$couponManager = new CouponManager($stubContainer);
|
||||
$couponManager->addAvailableCoupon($coupon1);
|
||||
$couponManager->addAvailableCoupon($coupon2);
|
||||
@@ -383,7 +382,6 @@ Sed facilisis pellentesque nisl, eu tincidunt erat scelerisque a. Nullam malesua
|
||||
->method('get')
|
||||
->will($this->onConsecutiveCalls($stubFacade));
|
||||
|
||||
|
||||
$couponManager = new CouponManager($stubContainer);
|
||||
$couponManager->addAvailableCondition($condition1);
|
||||
$couponManager->addAvailableCondition($condition2);
|
||||
@@ -440,10 +438,8 @@ Sed facilisis pellentesque nisl, eu tincidunt erat scelerisque a. Nullam malesua
|
||||
->method('get')
|
||||
->will($this->onConsecutiveCalls($stubFacade));
|
||||
|
||||
|
||||
$couponManager = new CouponManager($stubContainer);
|
||||
|
||||
|
||||
$stubModel = $this->getMockBuilder('\Thelia\Model\Coupon')
|
||||
->disableOriginalConstructor()
|
||||
->getMock();
|
||||
@@ -507,10 +503,8 @@ Sed facilisis pellentesque nisl, eu tincidunt erat scelerisque a. Nullam malesua
|
||||
->method('get')
|
||||
->will($this->onConsecutiveCalls($stubFacade));
|
||||
|
||||
|
||||
$couponManager = new CouponManager($stubContainer);
|
||||
|
||||
|
||||
$stubModel = $this->getMockBuilder('\Thelia\Model\Coupon')
|
||||
->disableOriginalConstructor()
|
||||
->getMock();
|
||||
|
||||
Reference in New Issue
Block a user