@@ -212,6 +212,6 @@ |
||
| 212 | 212 | */ |
| 213 | 213 | private function getCodeFromNameAndZoneCode($taxRateName, $zoneCode) |
| 214 | 214 | { |
| 215 | - return $this->getCodeFromName($taxRateName).'_'.strtolower($zoneCode); |
|
| 215 | + return $this->getCodeFromName($taxRateName) . '_' . strtolower($zoneCode); |
|
| 216 | 216 | } |
| 217 | 217 | } |
@@ -67,7 +67,7 @@ |
||
| 67 | 67 | * |
| 68 | 68 | * @throws \RuntimeException |
| 69 | 69 | * |
| 70 | - * @return string |
|
| 70 | + * @return integer |
|
| 71 | 71 | */ |
| 72 | 72 | private function tryToMapPaymentMethodName($paymentMethodName) |
| 73 | 73 | { |
@@ -15,9 +15,9 @@ |
||
| 15 | 15 | use Sylius\Behat\Page\Admin\Channel\IndexPageInterface; |
| 16 | 16 | use Sylius\Behat\Page\Admin\Channel\UpdatePageInterface; |
| 17 | 17 | use Sylius\Behat\Page\Shop\HomePageInterface; |
| 18 | +use Sylius\Behat\Service\SharedStorageInterface; |
|
| 18 | 19 | use Sylius\Bundle\ThemeBundle\Model\ThemeInterface; |
| 19 | 20 | use Sylius\Component\Core\Model\ChannelInterface; |
| 20 | -use Sylius\Behat\Service\SharedStorageInterface; |
|
| 21 | 21 | use Webmozart\Assert\Assert; |
| 22 | 22 | |
| 23 | 23 | /** |
@@ -530,7 +530,7 @@ discard block |
||
| 530 | 530 | */ |
| 531 | 531 | public function iShouldSeeThisShippingAddressAsShippingAddress($fullName) |
| 532 | 532 | { |
| 533 | - $address = $this->sharedStorage->get('shipping_address_'.StringInflector::nameToLowercaseCode($fullName)); |
|
| 533 | + $address = $this->sharedStorage->get('shipping_address_' . StringInflector::nameToLowercaseCode($fullName)); |
|
| 534 | 534 | Assert::true( |
| 535 | 535 | $this->summaryPage->hasShippingAddress($address), |
| 536 | 536 | 'Shipping address is improper.' |
@@ -542,7 +542,7 @@ discard block |
||
| 542 | 542 | */ |
| 543 | 543 | public function iShouldSeeThisBillingAddressAsBillingAddress($fullName) |
| 544 | 544 | { |
| 545 | - $address = $this->sharedStorage->get('billing_address_'.StringInflector::nameToLowercaseCode($fullName)); |
|
| 545 | + $address = $this->sharedStorage->get('billing_address_' . StringInflector::nameToLowercaseCode($fullName)); |
|
| 546 | 546 | Assert::true( |
| 547 | 547 | $this->summaryPage->hasBillingAddress($address), |
| 548 | 548 | 'Billing address is improper.' |
@@ -622,7 +622,7 @@ discard block |
||
| 622 | 622 | { |
| 623 | 623 | Assert::true( |
| 624 | 624 | $this->summaryPage->hasShippingTotal($price), |
| 625 | - sprintf('The shipping total should be %s, but it is not.',$price) |
|
| 625 | + sprintf('The shipping total should be %s, but it is not.', $price) |
|
| 626 | 626 | ); |
| 627 | 627 | } |
| 628 | 628 | |
@@ -1261,7 +1261,7 @@ |
||
| 1261 | 1261 | { |
| 1262 | 1262 | Assert::true( |
| 1263 | 1263 | $this->selectPaymentPage->isNextStepButtonUnavailable(), |
| 1264 | - 'The "next step" button should be disabled.' |
|
| 1264 | + 'The "next step" button should be disabled.' |
|
| 1265 | 1265 | ); |
| 1266 | 1266 | } |
| 1267 | 1267 | |
@@ -159,7 +159,7 @@ |
||
| 159 | 159 | (stripos($elementText, $customerName) !== false) && |
| 160 | 160 | (stripos($elementText, $street) !== false) && |
| 161 | 161 | (stripos($elementText, $city) !== false) && |
| 162 | - (stripos($elementText, $countryName.' '.$postcode) !== false) |
|
| 162 | + (stripos($elementText, $countryName . ' ' . $postcode) !== false) |
|
| 163 | 163 | ; |
| 164 | 164 | } |
| 165 | 165 | } |
@@ -40,7 +40,7 @@ discard block |
||
| 40 | 40 | { |
| 41 | 41 | $provinces = $this->getElement('provinces'); |
| 42 | 42 | |
| 43 | - return $provinces->has('css', '[value = "'.$provinceName.'"]'); |
|
| 43 | + return $provinces->has('css', '[value = "' . $provinceName . '"]'); |
|
| 44 | 44 | } |
| 45 | 45 | |
| 46 | 46 | /** |
@@ -50,7 +50,7 @@ discard block |
||
| 50 | 50 | { |
| 51 | 51 | $provinces = $this->getElement('provinces'); |
| 52 | 52 | |
| 53 | - return $provinces->has('css', '[value = "'.$provinceCode.'"]'); |
|
| 53 | + return $provinces->has('css', '[value = "' . $provinceCode . '"]'); |
|
| 54 | 54 | } |
| 55 | 55 | |
| 56 | 56 | /** |
@@ -137,7 +137,7 @@ discard block |
||
| 137 | 137 | if ($this->isThereProvince($provinceName)) { |
| 138 | 138 | $provinces = $this->getElement('provinces'); |
| 139 | 139 | |
| 140 | - $item = $provinces->find('css', 'div[data-form-collection="item"] input[value="'.$provinceName.'"]')->getParent(); |
|
| 140 | + $item = $provinces->find('css', 'div[data-form-collection="item"] input[value="' . $provinceName . '"]')->getParent(); |
|
| 141 | 141 | $item->fillField('Name', ''); |
| 142 | 142 | } |
| 143 | 143 | } |
@@ -385,7 +385,7 @@ discard block |
||
| 385 | 385 | (stripos($elementText, $customerName) !== false) && |
| 386 | 386 | (stripos($elementText, $street) !== false) && |
| 387 | 387 | (stripos($elementText, $city) !== false) && |
| 388 | - (stripos($elementText, $countryName.' '.$postcode) !== false) |
|
| 388 | + (stripos($elementText, $countryName . ' ' . $postcode) !== false) |
|
| 389 | 389 | ; |
| 390 | 390 | } |
| 391 | 391 | |
@@ -402,7 +402,7 @@ discard block |
||
| 402 | 402 | ['item' => $itemName] |
| 403 | 403 | ); |
| 404 | 404 | |
| 405 | - return $rows[0]->find('css', '.'.$property)->getText(); |
|
| 405 | + return $rows[0]->find('css', '.' . $property)->getText(); |
|
| 406 | 406 | } |
| 407 | 407 | |
| 408 | 408 | /** |
@@ -35,7 +35,7 @@ |
||
| 35 | 35 | { |
| 36 | 36 | $percentage = $number * 100; |
| 37 | 37 | |
| 38 | - return $percentage.' %'; |
|
| 38 | + return $percentage . ' %'; |
|
| 39 | 39 | } |
| 40 | 40 | |
| 41 | 41 | /** |
@@ -76,7 +76,7 @@ discard block |
||
| 76 | 76 | { |
| 77 | 77 | $this->queryBuilder->setParameter($field, $value); |
| 78 | 78 | |
| 79 | - return $this->queryBuilder->expr()->eq($field, ':'.$field); |
|
| 79 | + return $this->queryBuilder->expr()->eq($field, ':' . $field); |
|
| 80 | 80 | } |
| 81 | 81 | |
| 82 | 82 | /** |
@@ -86,7 +86,7 @@ discard block |
||
| 86 | 86 | { |
| 87 | 87 | $this->queryBuilder->setParameter($field, $value); |
| 88 | 88 | |
| 89 | - return $this->queryBuilder->expr()->neq($field, ':'.$field); |
|
| 89 | + return $this->queryBuilder->expr()->neq($field, ':' . $field); |
|
| 90 | 90 | } |
| 91 | 91 | |
| 92 | 92 | /** |
@@ -94,7 +94,7 @@ discard block |
||
| 94 | 94 | */ |
| 95 | 95 | public function lessThan($field, $value) |
| 96 | 96 | { |
| 97 | - $this->queryBuilder->andWhere($field.' < :'.$field)->setParameter($field, $value); |
|
| 97 | + $this->queryBuilder->andWhere($field . ' < :' . $field)->setParameter($field, $value); |
|
| 98 | 98 | } |
| 99 | 99 | |
| 100 | 100 | /** |
@@ -102,7 +102,7 @@ discard block |
||
| 102 | 102 | */ |
| 103 | 103 | public function lessThanOrEqual($field, $value) |
| 104 | 104 | { |
| 105 | - $this->queryBuilder->andWhere($field.' =< :'.$field)->setParameter($field, $value); |
|
| 105 | + $this->queryBuilder->andWhere($field . ' =< :' . $field)->setParameter($field, $value); |
|
| 106 | 106 | } |
| 107 | 107 | |
| 108 | 108 | /** |
@@ -110,7 +110,7 @@ discard block |
||
| 110 | 110 | */ |
| 111 | 111 | public function greaterThan($field, $value) |
| 112 | 112 | { |
| 113 | - $this->queryBuilder->andWhere($field.' > :'.$field)->setParameter($field, $value); |
|
| 113 | + $this->queryBuilder->andWhere($field . ' > :' . $field)->setParameter($field, $value); |
|
| 114 | 114 | } |
| 115 | 115 | |
| 116 | 116 | /** |
@@ -118,7 +118,7 @@ discard block |
||
| 118 | 118 | */ |
| 119 | 119 | public function greaterThanOrEqual($field, $value) |
| 120 | 120 | { |
| 121 | - $this->queryBuilder->andWhere($field.' => :%s'.$field)->setParameter($field, $value); |
|
| 121 | + $this->queryBuilder->andWhere($field . ' => :%s' . $field)->setParameter($field, $value); |
|
| 122 | 122 | } |
| 123 | 123 | |
| 124 | 124 | /** |
@@ -105,7 +105,7 @@ discard block |
||
| 105 | 105 | */ |
| 106 | 106 | public function walkCompositeExpression(CompositeExpression $expr, AbstractNode $parentNode) |
| 107 | 107 | { |
| 108 | - switch($expr->getType()) { |
|
| 108 | + switch ($expr->getType()) { |
|
| 109 | 109 | case CompositeExpression::TYPE_AND: |
| 110 | 110 | $node = $parentNode->andX(); |
| 111 | 111 | break; |
@@ -128,7 +128,7 @@ discard block |
||
| 128 | 128 | break; |
| 129 | 129 | } |
| 130 | 130 | |
| 131 | - switch($expr->getType()) { |
|
| 131 | + switch ($expr->getType()) { |
|
| 132 | 132 | case CompositeExpression::TYPE_AND: |
| 133 | 133 | $parentNode = $parentNode->andX(); |
| 134 | 134 | break; |
@@ -108,17 +108,17 @@ discard block |
||
| 108 | 108 | return [ |
| 109 | 109 | [ |
| 110 | 110 | Comparison::IN, |
| 111 | - [ 'one' ], |
|
| 111 | + ['one'], |
|
| 112 | 112 | 'ConstraintOrx ( jcr.operator.equal.to ( OperandDynamicField(o.hello) OperandStaticLiteral("one") ) )' |
| 113 | 113 | ], |
| 114 | 114 | [ |
| 115 | 115 | Comparison::IN, |
| 116 | - [ 'one', 'two', 'three' ], |
|
| 116 | + ['one', 'two', 'three'], |
|
| 117 | 117 | 'ConstraintOrx ( jcr.operator.equal.to ( OperandDynamicField(o.hello) OperandStaticLiteral("one") ) jcr.operator.equal.to ( OperandDynamicField(o.hello) OperandStaticLiteral("two") ) jcr.operator.equal.to ( OperandDynamicField(o.hello) OperandStaticLiteral("three") ) )', |
| 118 | 118 | ], |
| 119 | 119 | [ |
| 120 | 120 | Comparison::NIN, |
| 121 | - [ 'one' ], |
|
| 121 | + ['one'], |
|
| 122 | 122 | 'ConstraintNot ( ConstraintOrx ( jcr.operator.equal.to ( OperandDynamicField(o.hello) OperandStaticLiteral("one") ) ) )' |
| 123 | 123 | ], |
| 124 | 124 | ]; |
@@ -133,7 +133,7 @@ discard block |
||
| 133 | 133 | $expr2 = new Comparison('number', Comparison::GT, 8); |
| 134 | 134 | $expr = new CompositeExpression( |
| 135 | 135 | $type, |
| 136 | - [ $expr1, $expr2 ] |
|
| 136 | + [$expr1, $expr2] |
|
| 137 | 137 | ); |
| 138 | 138 | |
| 139 | 139 | $this->visitor->dispatch($expr); |
@@ -153,8 +153,8 @@ discard block |
||
| 153 | 153 | public function provideComposite() |
| 154 | 154 | { |
| 155 | 155 | return [ |
| 156 | - [ CompositeExpression::TYPE_AND, 'ConstraintAndx' ], |
|
| 157 | - [ CompositeExpression::TYPE_OR, 'ConstraintOrx' ] |
|
| 156 | + [CompositeExpression::TYPE_AND, 'ConstraintAndx'], |
|
| 157 | + [CompositeExpression::TYPE_OR, 'ConstraintOrx'] |
|
| 158 | 158 | ]; |
| 159 | 159 | } |
| 160 | 160 | |
@@ -166,7 +166,7 @@ discard block |
||
| 166 | 166 | $expr3 = new Comparison('date', Comparison::GT, '2015-12-10'); |
| 167 | 167 | $expr = new CompositeExpression( |
| 168 | 168 | $type, |
| 169 | - [ $expr1, $expr2, $expr3 ] |
|
| 169 | + [$expr1, $expr2, $expr3] |
|
| 170 | 170 | ); |
| 171 | 171 | |
| 172 | 172 | $this->visitor->dispatch($expr); |
@@ -197,12 +197,12 @@ discard block |
||
| 197 | 197 | $expr4 = new Comparison('date', Comparison::LT, '2016-12-10'); |
| 198 | 198 | $expr5 = new Comparison('date', Comparison::NEQ, '2016-12-10'); |
| 199 | 199 | |
| 200 | - $comp2 = new CompositeExpression(CompositeExpression::TYPE_AND, [ $expr2, $expr5 ]); |
|
| 201 | - $comp1 = new CompositeExpression(CompositeExpression::TYPE_OR, [ $expr1, $comp2 ]); |
|
| 200 | + $comp2 = new CompositeExpression(CompositeExpression::TYPE_AND, [$expr2, $expr5]); |
|
| 201 | + $comp1 = new CompositeExpression(CompositeExpression::TYPE_OR, [$expr1, $comp2]); |
|
| 202 | 202 | |
| 203 | 203 | $expr = new CompositeExpression( |
| 204 | 204 | $type, |
| 205 | - [ $comp1, $expr3, $expr4 ] |
|
| 205 | + [$comp1, $expr3, $expr4] |
|
| 206 | 206 | ); |
| 207 | 207 | |
| 208 | 208 | $this->visitor->dispatch($expr); |
@@ -146,7 +146,7 @@ discard block |
||
| 146 | 146 | OperandDynamicField(o.number) OperandStaticLiteral("8") |
| 147 | 147 | ) |
| 148 | 148 | ) |
| 149 | -EOT |
|
| 149 | +eot |
|
| 150 | 150 | ); |
| 151 | 151 | } |
| 152 | 152 | |
@@ -184,7 +184,7 @@ discard block |
||
| 184 | 184 | ) |
| 185 | 185 | ) |
| 186 | 186 | ) |
| 187 | -EOT |
|
| 187 | +eot |
|
| 188 | 188 | ); |
| 189 | 189 | } |
| 190 | 190 | |
@@ -230,7 +230,7 @@ discard block |
||
| 230 | 230 | ) |
| 231 | 231 | ) |
| 232 | 232 | ) |
| 233 | -EOT |
|
| 233 | +eot |
|
| 234 | 234 | ); |
| 235 | 235 | } |
| 236 | 236 | |