Skip to content

Commit

Permalink
Merge pull request #7780 from pamil/form-types-promotion
Browse files Browse the repository at this point in the history
Decouple promotion components from bundles and cleanup promotion forms
  • Loading branch information
pjedrzejewski committed Mar 20, 2017
2 parents 9dfdceb + 91235db commit c537128
Show file tree
Hide file tree
Showing 65 changed files with 638 additions and 1,337 deletions.
5 changes: 5 additions & 0 deletions UPGRADE.md
Expand Up @@ -200,6 +200,11 @@ You can bring back previous configuration by overriding current routing with you

* Removed class `Sylius\Bundle\CoreBundle\Form\Type\ProductTaxonChoiceType`, use `Sylius\Bundle\CoreBundle\Form\Type\Taxon\ProductTaxonAutocompleteChoiceType` instead.

* Removed `Sylius\Component\Core\Promotion\Action\ChannelBasedPromotionActionCommandInterface` and
`Sylius\Component\Core\Promotion\Checker\Rule\ChannelBasedRuleCheckerInterface` interfaces as they are not needed anymore.

* Removed `Sylius\Bundle\CoreBundle\Form\Type\Promotion\PromotionConfigurationType` class as it has no behaviour and is not used.

### Currency / CurrencyBundle

* The following classes were removed due to being no longer used in current implementation:
Expand Down
4 changes: 2 additions & 2 deletions src/Sylius/Behat/Page/Admin/Promotion/CreatePage.php
Expand Up @@ -229,7 +229,7 @@ private function getChannelConfigurationOfLastAction($channelName)
{
return $this
->getLastCollectionItem('actions')
->find('css', sprintf('[id*="sylius_promotion_actions"] .configuration .field:contains("%s")', $channelName))
->find('css', sprintf('[id$="configuration"] .field:contains("%s")', $channelName))
;
}

Expand All @@ -242,7 +242,7 @@ private function getChannelConfigurationOfLastRule($channelName)
{
return $this
->getLastCollectionItem('rules')
->find('css', sprintf('[id*="sylius_promotion_rules"] .configuration .field:contains("%s")', $channelName))
->find('css', sprintf('[id$="configuration"] .field:contains("%s")', $channelName))
;
}

Expand Down

This file was deleted.

This file was deleted.

This file was deleted.

0 comments on commit c537128

Please sign in to comment.