UpdateSimpleProductPage::getNumberOfAttributes() - Code Metrics - Inspection of "Improve upgrade file" - pamil/Sylius - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — upgrade ( c4e463 )

by Kamil

created 2017-03-30 10:02 UTC

UpdateSimpleProductPage::getNumberOfAttributes() A

↳ Parent: UpdateSimpleProductPage

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	4
rs	10
c	0
b	0
f	0
cc	1
eloc	2
nc	1
nop	0

<?php

/*
 * This file is part of the Sylius package.
 *
 * (c) Paweł Jędrzejewski
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Sylius\Behat\Page\Admin\Product;

use Behat\Mink\Driver\Selenium2Driver;
use Behat\Mink\Element\NodeElement;
use Sylius\Behat\Behaviour\ChecksCodeImmutability;
use Sylius\Behat\Page\Admin\Crud\UpdatePage as BaseUpdatePage;
use Sylius\Behat\Service\AutocompleteHelper;
use Sylius\Behat\Service\SlugGenerationHelper;
use Sylius\Component\Core\Model\ChannelInterface;
use Sylius\Component\Core\Model\TaxonInterface;
use Sylius\Component\Currency\Model\CurrencyInterface;
use Sylius\Component\Product\Model\ProductAssociationTypeInterface;
use Webmozart\Assert\Assert;

/**
 * @author Łukasz Chruściel <[email protected]>
 * @author Gorka Laucirica <[email protected]>
 */
class UpdateSimpleProductPage extends BaseUpdatePage implements UpdateSimpleProductPageInterface
{
    use ChecksCodeImmutability;

    /**
     * {@inheritdoc}
     */
    public function nameItIn($name, $localeCode)
    {
        $this->activateLanguageTab($localeCode);
        $this->getElement('name', ['%locale%' => $localeCode])->setValue($name);

        if ($this->getDriver() instanceof Selenium2Driver) {
            SlugGenerationHelper::waitForSlugGeneration(
                $this->getSession(),
                $this->getElement('slug', ['%locale%' => $localeCode])
            );
        }
    }

    /**
     * {@inheritdoc}
     */
    public function specifyPrice($channelName, $price)
    {
        $this->getElement('price', ['%channelName%' => $channelName])->setValue($price);
    }

    /**
     * {@inheritdoc}
     */
    public function specifyOriginalPrice($channelName, $originalPrice)
    {
        $this->getElement('original_price', ['%channelName%' => $channelName])->setValue($originalPrice);
    }

    public function addSelectedAttributes()
    {
        $this->clickTabIfItsNotActive('attributes');
        $this->getDocument()->pressButton('Add attributes');

        $form = $this->getDocument()->find('css', 'form');

        $this->getDocument()->waitFor(1, function () use ($form) {
            return $form->hasClass('loading');
        });
    }

    /**
     * {@inheritdoc}
     */
    public function removeAttribute($attributeName, $localeCode)
    {
        $this->clickTabIfItsNotActive('attributes');

        $this->getElement('attribute_delete_button', ['%attributeName%' => $attributeName, '$localeCode%' => $localeCode])->press();
    }

    /**
     * {@inheritdoc}
     */
    public function getAttributeValue($attribute, $localeCode)
    {
        $this->clickTabIfItsNotActive('attributes');
        $this->clickLocaleTabIfItsNotActive($localeCode);

        return $this->getElement('attribute', ['%attributeName%' => $attribute, '%localeCode%' => $localeCode])->getValue();
    }

    /**
     * {@inheritdoc}
     */
    public function getNumberOfAttributes()
    {
        return count($this->getDocument()->findAll('css', '.attribute'));
    }

    /**
     * {@inheritdoc}
     */
    public function hasAttribute($attributeName)
    {
        return null !== $this->getDocument()->find('css', sprintf('.attribute .label:contains("%s")', $attributeName));
    }

    /**
     * {@inheritdoc}
     */
    public function selectMainTaxon(TaxonInterface $taxon)
    {
        $this->openTaxonBookmarks();

        $mainTaxonElement = $this->getElement('main_taxon')->getParent();

        AutocompleteHelper::chooseValue($this->getSession(), $mainTaxonElement, $taxon->getName());
/** @return stdClass|null */
function mayReturnNull() { }

function doesNotAcceptNull(stdClass $x) { }

// With potential error.
function withoutCheck() {
    $x = mayReturnNull();
    doesNotAcceptNull($x); // Potential error here.
}

// Safe - Alternative 1
function withCheck1() {
    $x = mayReturnNull();
    if ( ! $x instanceof stdClass) {
        throw new \LogicException('$x must be defined.');
    }
    doesNotAcceptNull($x);
}

// Safe - Alternative 2
function withCheck2() {
    $x = mayReturnNull();
    if ($x instanceof stdClass) {
        doesNotAcceptNull($x);
    }
}
    }

    /**
     * {@inheritdoc}
     */
    public function isMainTaxonChosen($taxonName)
    {
        $this->openTaxonBookmarks();

        return $taxonName === $this->getDocument()->find('css', '.search > .text')->getText();
    }

    public function disableTracking()
    {
        $this->getElement('tracked')->uncheck();
    }

    public function enableTracking()
    {
        $this->getElement('tracked')->check();
    }

    /**
     * {@inheritdoc}
     */
    public function isTracked()
    {
        return $this->getElement('tracked')->isChecked();
    }

    /**
     * {@inheritdoc}
     */
    public function enableSlugModification($locale)
    {
        SlugGenerationHelper::enableSlugModification(
            $this->getSession(),
            $this->getElement('toggle_slug_modification_button', ['%locale%' => $locale])
        );
    }

    /**
     * {@inheritdoc}
     */
    public function isImageWithTypeDisplayed($type)
    {
        $imageElement = $this->getImageElementByType($type);

        if (null === $imageElement) {
            return false;
        }

        $imageUrl = $imageElement->find('css', 'img')->getAttribute('src');
        $this->getDriver()->visit($imageUrl);
        $pageText = $this->getDocument()->getText();
        $this->getDriver()->back();

        return false === stripos($pageText, '404 Not Found');
    }

    /**
     * {@inheritdoc}
     */
    public function attachImage($path, $type = null)
    {
        $this->clickTabIfItsNotActive('media');

        $filesPath = $this->getParameter('files_path');

        $this->getDocument()->clickLink('Add');

        $imageForm = $this->getLastImageElement();
        if (null !== $type) {
            $imageForm->fillField('Type', $type);
        }

        $imageForm->find('css', 'input[type="file"]')->attachFile($filesPath.$path);
    }

    /**
     * {@inheritdoc}
     */
    public function changeImageWithType($type, $path)
    {
        $filesPath = $this->getParameter('files_path');

        $imageForm = $this->getImageElementByType($type);
        $imageForm->find('css', 'input[type="file"]')->attachFile($filesPath.$path);
    }

    /**
     * {@inheritdoc}
     */
    public function removeImageWithType($type)
    {
        $this->clickTabIfItsNotActive('media');

        $imageElement = $this->getImageElementByType($type);
        $imageElement->clickLink('Delete');
    }

    public function removeFirstImage()
    {
        $this->clickTabIfItsNotActive('media');

        $imageElement = $this->getFirstImageElement();
        $imageElement->clickLink('Delete');
    }

    /**
     * {@inheritdoc}
     */
    public function modifyFirstImageType($type)
    {
        $this->clickTabIfItsNotActive('media');

        $firstImage = $this->getFirstImageElement();
        $this->setImageType($firstImage, $type);
    }

    /**
     * {@inheritdoc}
     */
    public function countImages()
    {
        $imageElements = $this->getImageElements();

        return count($imageElements);
    }

    /**
     * {@inheritdoc}
     */
    public function isSlugReadonlyIn($locale)
    {
        return SlugGenerationHelper::isSlugReadonly(
            $this->getSession(),
            $this->getElement('slug', ['%locale%' => $locale])
        );
    }

    /**
     * {@inheritdoc}
     */
    public function associateProducts(ProductAssociationTypeInterface $productAssociationType, array $productsNames)

    {
        $this->clickTab('associations');

        Assert::isInstanceOf($this->getDriver(), Selenium2Driver::class);

        $dropdown = $this->getElement('association_dropdown', [
            '%association%' => $productAssociationType->getName()
        ]);
        $dropdown->click();

        foreach ($productsNames as $productName) {
            $dropdown->waitFor(5, function () use ($productName, $productAssociationType) {
                return $this->hasElement('association_dropdown_item', [
                    '%association%' => $productAssociationType->getName(),
                    '%item%' => $productName,
                ]);
            });

            $item = $this->getElement('association_dropdown_item', [
                '%association%' => $productAssociationType->getName(),
                '%item%' => $productName,
            ]);
            $item->click();
        }
    }

    /**
     * {@inheritdoc}
     */
    public function hasAssociatedProduct($productName, ProductAssociationTypeInterface $productAssociationType)

    {
        $this->clickTabIfItsNotActive('associations');

        return $this->hasElement('association_dropdown_item', [
            '%association%' => $productAssociationType->getName(),
            '%item%' => $productName,
        ]);
    }

    /**
     * {@inheritdoc}
     */
    public function removeAssociatedProduct($productName, ProductAssociationTypeInterface $productAssociationType)

    {
        $this->clickTabIfItsNotActive('associations');

        $item = $this->getElement('association_dropdown_item_selected', [
            '%association%' => $productAssociationType->getName(),
            '%item%' => $productName,
        ]);

        $deleteIcon = $item->find('css', 'i.delete');
        Assert::notNull($deleteIcon);
        $deleteIcon->click();
    }

    /**
     * {@inheritdoc}
     */
    public function getPricingConfigurationForChannelAndCurrencyCalculator(ChannelInterface $channel, CurrencyInterface $currency)
    {
        $priceConfigurationElement = $this->getElement('pricing_configuration');

        $priceElement = $priceConfigurationElement
            ->find('css', sprintf('label:contains("%s %s")', $channel->getCode(), $currency->getCode()))->getParent();

        return $priceElement->find('css', 'input')->getValue();
    }

    /**
     * {@inheritdoc}
     */
    public function getSlug($locale)
    {
        $this->activateLanguageTab($locale);

        return $this->getElement('slug', ['%locale%' => $locale])->getValue();
    }

    /**
     * {@inheritdoc}
     */
    public function specifySlugIn($slug, $locale)
    {
        $this->activateLanguageTab($locale);

        $this->getElement('slug', ['%locale%' => $locale])->setValue($slug);
    }

    /**
     * {@inheritdoc}
     */
    public function activateLanguageTab($locale)
    {
        if (!$this->getDriver() instanceof Selenium2Driver) {
            return;
        }

        $languageTabTitle = $this->getElement('language_tab', ['%locale%' => $locale]);
        if (!$languageTabTitle->hasClass('active')) {
            $languageTabTitle->click();
        }
    }

    public function getPriceForChannel($channelName)
    {
        return $this->getElement('price', ['%channelName%' => $channelName])->getValue();
    }

    /**
     * {@inheritdoc}
     */
    public function getOriginalPriceForChannel($channelName)
    {
        return $this->getElement('original_price', ['%channelName%' => $channelName])->getValue();
    }

    /**
     * {@inheritdoc}
     */
    public function isShippingRequired()
    {
        return $this->getElement('shipping_required')->isChecked();
    }

    /**
     * {@inheritdoc}
     */
    protected function getCodeElement()
    {
        return $this->getElement('code');
    }

    /**
     * {@inheritdoc}
     */
    protected function getElement($name, array $parameters = [])
    {
        if (!isset($parameters['%locale%'])) {
            $parameters['%locale%'] = 'en_US';
        }

        return parent::getElement($name, $parameters);
    }

    /**
     * {@inheritdoc}
     */
    protected function getDefinedElements()
    {
        return array_merge(parent::getDefinedElements(), [
            'association_dropdown' => '.field > label:contains("%association%") ~ .product-select',
            'association_dropdown_item' => '.field > label:contains("%association%") ~ .product-select > div.menu > div.item:contains("%item%")',
            'association_dropdown_item_selected' => '.field > label:contains("%association%") ~ .product-select > a.label:contains("%item%")',
            'attribute' => '.tab[data-tab="%localeCode%"] .attribute .label:contains("%attributeName%") ~ input',
            'attribute_delete_button' => '.tab[data-tab="%localeCode%"] .attribute .label:contains("%attributeName%") ~ button',
            'code' => '#sylius_product_code',
            'images' => '#sylius_product_images',
            'language_tab' => '[data-locale="%locale%"] .title',
            'locale_tab' => '#attributesContainer .menu [data-tab="%localeCode%"]',
            'name' => '#sylius_product_translations_%locale%_name',
            'original_price' => '#sylius_product_variant_channelPricings > .field:contains("%channelName%") input[name$="[originalPrice]"]',
            'price' => '#sylius_product_variant_channelPricings > .field:contains("%channelName%") input[name$="[price]"]',
            'pricing_configuration' => '#sylius_calculator_container',
            'main_taxon' => '#sylius_product_mainTaxon',
            'shipping_required' => '#sylius_product_variant_shippingRequired',
            'slug' => '#sylius_product_translations_%locale%_slug',
            'tab' => '.menu [data-tab="%name%"]',
            'taxonomy' => 'a[data-tab="taxonomy"]',
            'tracked' => '#sylius_product_variant_tracked',
            'toggle_slug_modification_button' => '[data-locale="%locale%"] .toggle-product-slug-modification',
        ]);
    }

    private function openTaxonBookmarks()
    {
        $this->getElement('taxonomy')->click();
    }

    /**
     * @param string $tabName
     */
    private function clickTabIfItsNotActive($tabName)
    {
        $attributesTab = $this->getElement('tab', ['%name%' => $tabName]);
        if (!$attributesTab->hasClass('active')) {
            $attributesTab->click();
        }
    }

    /**
     * @param string $tabName
     */
    private function clickTab($tabName)
    {
        $attributesTab = $this->getElement('tab', ['%name%' => $tabName]);
        $attributesTab->click();
    }

    /**
     * @param string $localeCode
     */
    private function clickLocaleTabIfItsNotActive($localeCode)
    {
        $localeTab = $this->getElement('locale_tab', ['%localeCode%' => $localeCode]);
        if (!$localeTab->hasClass('active')) {
            $localeTab->click();
        }
    }

    /**
     * @param string $type
     *
     * @return NodeElement
     */
    private function getImageElementByType($type)
    {
        $images = $this->getElement('images');
        $typeInput = $images->find('css', 'input[value="'.$type.'"]');

        if (null === $typeInput) {
            return null;
        }

        return $typeInput->getParent()->getParent()->getParent();
    }

    /**
     * @return NodeElement[]
     */
    private function getImageElements()
    {
        $images = $this->getElement('images');

        return $images->findAll('css', 'div[data-form-collection="item"]');
    }

    /**
     * @return NodeElement
     */
    private function getLastImageElement()
    {
        $imageElements = $this->getImageElements();

        Assert::notEmpty($imageElements);

        return end($imageElements);
    }

    /**
     * @return NodeElement
     */
    private function getFirstImageElement()
    {
        $imageElements = $this->getImageElements();

        Assert::notEmpty($imageElements);

        return reset($imageElements);
    }

    /**
     * @param NodeElement $imageElement
     * @param string $type
     */
    private function setImageType(NodeElement $imageElement, $type)
    {
        $typeField = $imageElement->findField('Type');
        $typeField->setValue($type);
    }
}


1			<?php
2
3			/*
4			* This file is part of the Sylius package.
5			*
6			* (c) Paweł Jędrzejewski
7			*
8			* For the full copyright and license information, please view the LICENSE
9			* file that was distributed with this source code.
10			*/
11
12			namespace Sylius\Behat\Page\Admin\Product;
13
14			use Behat\Mink\Driver\Selenium2Driver;
15			use Behat\Mink\Element\NodeElement;
16			use Sylius\Behat\Behaviour\ChecksCodeImmutability;
17			use Sylius\Behat\Page\Admin\Crud\UpdatePage as BaseUpdatePage;
18			use Sylius\Behat\Service\AutocompleteHelper;
19			use Sylius\Behat\Service\SlugGenerationHelper;
20			use Sylius\Component\Core\Model\ChannelInterface;
21			use Sylius\Component\Core\Model\TaxonInterface;
22			use Sylius\Component\Currency\Model\CurrencyInterface;
23			use Sylius\Component\Product\Model\ProductAssociationTypeInterface;
24			use Webmozart\Assert\Assert;
25
26			/**
27			* @author Łukasz Chruściel <[email protected]>
28			* @author Gorka Laucirica <[email protected]>
29			*/
30			class UpdateSimpleProductPage extends BaseUpdatePage implements UpdateSimpleProductPageInterface
31			{
32			use ChecksCodeImmutability;
33
34			/**
35			* {@inheritdoc}
36			*/
37			public function nameItIn($name, $localeCode)
38			{
39			$this->activateLanguageTab($localeCode);
40			$this->getElement('name', ['%locale%' => $localeCode])->setValue($name);
41
42			if ($this->getDriver() instanceof Selenium2Driver) {
43			SlugGenerationHelper::waitForSlugGeneration(
44			$this->getSession(),
45			$this->getElement('slug', ['%locale%' => $localeCode])
46			);
47			}
48			}
49
50			/**
51			* {@inheritdoc}
52			*/
53			public function specifyPrice($channelName, $price)
54			{
55			$this->getElement('price', ['%channelName%' => $channelName])->setValue($price);
56			}
57
58			/**
59			* {@inheritdoc}
60			*/
61			public function specifyOriginalPrice($channelName, $originalPrice)
62			{
63			$this->getElement('original_price', ['%channelName%' => $channelName])->setValue($originalPrice);
64			}
65
66			public function addSelectedAttributes()
67			{
68			$this->clickTabIfItsNotActive('attributes');
69			$this->getDocument()->pressButton('Add attributes');
70
71			$form = $this->getDocument()->find('css', 'form');
72
73			$this->getDocument()->waitFor(1, function () use ($form) {
74			return $form->hasClass('loading');
75			});
76			}
77
78			/**
79			* {@inheritdoc}
80			*/
81			public function removeAttribute($attributeName, $localeCode)
82			{
83			$this->clickTabIfItsNotActive('attributes');
84
85			$this->getElement('attribute_delete_button', ['%attributeName%' => $attributeName, '$localeCode%' => $localeCode])->press();
86			}
87
88			/**
89			* {@inheritdoc}
90			*/
91			public function getAttributeValue($attribute, $localeCode)
92			{
93			$this->clickTabIfItsNotActive('attributes');
94			$this->clickLocaleTabIfItsNotActive($localeCode);
95
96			return $this->getElement('attribute', ['%attributeName%' => $attribute, '%localeCode%' => $localeCode])->getValue();
97			}
98
99			/**
100			* {@inheritdoc}
101			*/
102			public function getNumberOfAttributes()
103			{
104			return count($this->getDocument()->findAll('css', '.attribute'));
105			}
106
107			/**
108			* {@inheritdoc}
109			*/
110			public function hasAttribute($attributeName)
111			{
112			return null !== $this->getDocument()->find('css', sprintf('.attribute .label:contains("%s")', $attributeName));
113			}
114
115			/**
116			* {@inheritdoc}
117			*/
118			public function selectMainTaxon(TaxonInterface $taxon)
119			{
120			$this->openTaxonBookmarks();
121
122			$mainTaxonElement = $this->getElement('main_taxon')->getParent();
123
124			AutocompleteHelper::chooseValue($this->getSession(), $mainTaxonElement, $taxon->getName());
			0 ignored issues – show Bug introduced 2017-03-14 18:21 UTC by Report Bug Copy Issue Report It seems like `$mainTaxonElement` defined by `$this->getElement('main_taxon')->getParent()` on line `122` can be `null`; however, `Sylius\Behat\Service\Aut...teHelper::chooseValue()` does not accept `null`, maybe add an additional type check? Unless you are absolutely sure that the expression can never be null because of other conditions, we strongly recommend to add an additional type check to your code: /** @return stdClass\|null */ function mayReturnNull() { } function doesNotAcceptNull(stdClass $x) { } // With potential error. function withoutCheck() { $x = mayReturnNull(); doesNotAcceptNull($x); // Potential error here. } // Safe - Alternative 1 function withCheck1() { $x = mayReturnNull(); if ( ! $x instanceof stdClass) { throw new \LogicException('$x must be defined.'); } doesNotAcceptNull($x); } // Safe - Alternative 2 function withCheck2() { $x = mayReturnNull(); if ($x instanceof stdClass) { doesNotAcceptNull($x); } } Loading history...
125			}
126
127			/**
128			* {@inheritdoc}
129			*/
130			public function isMainTaxonChosen($taxonName)
131			{
132			$this->openTaxonBookmarks();
133
134			return $taxonName === $this->getDocument()->find('css', '.search > .text')->getText();
135			}
136
137			public function disableTracking()
138			{
139			$this->getElement('tracked')->uncheck();
140			}
141
142			public function enableTracking()
143			{
144			$this->getElement('tracked')->check();
145			}
146
147			/**
148			* {@inheritdoc}
149			*/
150			public function isTracked()
151			{
152			return $this->getElement('tracked')->isChecked();
153			}
154
155			/**
156			* {@inheritdoc}
157			*/
158			public function enableSlugModification($locale)
159			{
160			SlugGenerationHelper::enableSlugModification(
161			$this->getSession(),
162			$this->getElement('toggle_slug_modification_button', ['%locale%' => $locale])
163			);
164			}
165
166			/**
167			* {@inheritdoc}
168			*/
169			public function isImageWithTypeDisplayed($type)
170			{
171			$imageElement = $this->getImageElementByType($type);
172
173			if (null === $imageElement) {
174			return false;
175			}
176
177			$imageUrl = $imageElement->find('css', 'img')->getAttribute('src');
178			$this->getDriver()->visit($imageUrl);
179			$pageText = $this->getDocument()->getText();
180			$this->getDriver()->back();
181
182			return false === stripos($pageText, '404 Not Found');
183			}
184
185			/**
186			* {@inheritdoc}
187			*/
188			public function attachImage($path, $type = null)
189			{
190			$this->clickTabIfItsNotActive('media');
191
192			$filesPath = $this->getParameter('files_path');
193
194			$this->getDocument()->clickLink('Add');
195
196			$imageForm = $this->getLastImageElement();
197			if (null !== $type) {
198			$imageForm->fillField('Type', $type);
199			}
200
201			$imageForm->find('css', 'input[type="file"]')->attachFile($filesPath.$path);
202			}
203
204			/**
205			* {@inheritdoc}
206			*/
207			public function changeImageWithType($type, $path)
208			{
209			$filesPath = $this->getParameter('files_path');
210
211			$imageForm = $this->getImageElementByType($type);
212			$imageForm->find('css', 'input[type="file"]')->attachFile($filesPath.$path);
213			}
214
215			/**
216			* {@inheritdoc}
217			*/
218			public function removeImageWithType($type)
219			{
220			$this->clickTabIfItsNotActive('media');
221
222			$imageElement = $this->getImageElementByType($type);
223			$imageElement->clickLink('Delete');
224			}
225
226			public function removeFirstImage()
227			{
228			$this->clickTabIfItsNotActive('media');
229
230			$imageElement = $this->getFirstImageElement();
231			$imageElement->clickLink('Delete');
232			}
233
234			/**
235			* {@inheritdoc}
236			*/
237			public function modifyFirstImageType($type)
238			{
239			$this->clickTabIfItsNotActive('media');
240
241			$firstImage = $this->getFirstImageElement();
242			$this->setImageType($firstImage, $type);
243			}
244
245			/**
246			* {@inheritdoc}
247			*/
248			public function countImages()
249			{
250			$imageElements = $this->getImageElements();
251
252			return count($imageElements);
253			}
254
255			/**
256			* {@inheritdoc}
257			*/
258			public function isSlugReadonlyIn($locale)
259			{
260			return SlugGenerationHelper::isSlugReadonly(
261			$this->getSession(),
262			$this->getElement('slug', ['%locale%' => $locale])
263			);
264			}
265
266			/**
267			* {@inheritdoc}
268			*/
269			public function associateProducts(ProductAssociationTypeInterface $productAssociationType, array $productsNames)
			0 ignored issues – show Comprehensibility Naming introduced 2016-11-14 10:43 UTC by Report Bug Copy Issue Report The variable name `$productAssociationType` exceeds the maximum configured length of `20`. Very long variable names usually make code harder to read. It is therefore recommended not to make variable names too verbose. Loading history...
270			{
271			$this->clickTab('associations');
272
273			Assert::isInstanceOf($this->getDriver(), Selenium2Driver::class);
274
275			$dropdown = $this->getElement('association_dropdown', [
276			'%association%' => $productAssociationType->getName()
277			]);
278			$dropdown->click();
279
280			foreach ($productsNames as $productName) {
281			$dropdown->waitFor(5, function () use ($productName, $productAssociationType) {
282			return $this->hasElement('association_dropdown_item', [
283			'%association%' => $productAssociationType->getName(),
284			'%item%' => $productName,
285			]);
286			});
287
288			$item = $this->getElement('association_dropdown_item', [
289			'%association%' => $productAssociationType->getName(),
290			'%item%' => $productName,
291			]);
292			$item->click();
293			}
294			}
295
296			/**
297			* {@inheritdoc}
298			*/
299			public function hasAssociatedProduct($productName, ProductAssociationTypeInterface $productAssociationType)
			0 ignored issues – show Comprehensibility Naming introduced 2016-11-14 10:43 UTC by Report Bug Copy Issue Report The variable name `$productAssociationType` exceeds the maximum configured length of `20`. Very long variable names usually make code harder to read. It is therefore recommended not to make variable names too verbose. Loading history...
300			{
301			$this->clickTabIfItsNotActive('associations');
302
303			return $this->hasElement('association_dropdown_item', [
304			'%association%' => $productAssociationType->getName(),
305			'%item%' => $productName,
306			]);
307			}
308
309			/**
310			* {@inheritdoc}
311			*/
312			public function removeAssociatedProduct($productName, ProductAssociationTypeInterface $productAssociationType)
			0 ignored issues – show Comprehensibility Naming introduced 2016-11-14 10:43 UTC by Report Bug Copy Issue Report The variable name `$productAssociationType` exceeds the maximum configured length of `20`. Very long variable names usually make code harder to read. It is therefore recommended not to make variable names too verbose. Loading history...
313			{
314			$this->clickTabIfItsNotActive('associations');
315
316			$item = $this->getElement('association_dropdown_item_selected', [
317			'%association%' => $productAssociationType->getName(),
318			'%item%' => $productName,
319			]);
320
321			$deleteIcon = $item->find('css', 'i.delete');
322			Assert::notNull($deleteIcon);
323			$deleteIcon->click();
324			}
325
326			/**
327			* {@inheritdoc}
328			*/
329			public function getPricingConfigurationForChannelAndCurrencyCalculator(ChannelInterface $channel, CurrencyInterface $currency)
330			{
331			$priceConfigurationElement = $this->getElement('pricing_configuration');
			0 ignored issues – show Comprehensibility Naming introduced 2016-11-16 11:53 UTC by Report Bug Copy Issue Report The variable name `$priceConfigurationElement` exceeds the maximum configured length of `20`. Very long variable names usually make code harder to read. It is therefore recommended not to make variable names too verbose. Loading history...
332			$priceElement = $priceConfigurationElement
333			->find('css', sprintf('label:contains("%s %s")', $channel->getCode(), $currency->getCode()))->getParent();
334
335			return $priceElement->find('css', 'input')->getValue();
336			}
337
338			/**
339			* {@inheritdoc}
340			*/
341			public function getSlug($locale)
342			{
343			$this->activateLanguageTab($locale);
344
345			return $this->getElement('slug', ['%locale%' => $locale])->getValue();
346			}
347
348			/**
349			* {@inheritdoc}
350			*/
351			public function specifySlugIn($slug, $locale)
352			{
353			$this->activateLanguageTab($locale);
354
355			$this->getElement('slug', ['%locale%' => $locale])->setValue($slug);
356			}
357
358			/**
359			* {@inheritdoc}
360			*/
361			public function activateLanguageTab($locale)
362			{
363			if (!$this->getDriver() instanceof Selenium2Driver) {
364			return;
365			}
366
367			$languageTabTitle = $this->getElement('language_tab', ['%locale%' => $locale]);
368			if (!$languageTabTitle->hasClass('active')) {
369			$languageTabTitle->click();
370			}
371			}
372
373			public function getPriceForChannel($channelName)
374			{
375			return $this->getElement('price', ['%channelName%' => $channelName])->getValue();
376			}
377
378			/**
379			* {@inheritdoc}
380			*/
381			public function getOriginalPriceForChannel($channelName)
382			{
383			return $this->getElement('original_price', ['%channelName%' => $channelName])->getValue();
384			}
385
386			/**
387			* {@inheritdoc}
388			*/
389			public function isShippingRequired()
390			{
391			return $this->getElement('shipping_required')->isChecked();
392			}
393
394			/**
395			* {@inheritdoc}
396			*/
397			protected function getCodeElement()
398			{
399			return $this->getElement('code');
400			}
401
402			/**
403			* {@inheritdoc}
404			*/
405			protected function getElement($name, array $parameters = [])
406			{
407			if (!isset($parameters['%locale%'])) {
408			$parameters['%locale%'] = 'en_US';
409			}
410
411			return parent::getElement($name, $parameters);
412			}
413
414			/**
415			* {@inheritdoc}
416			*/
417			protected function getDefinedElements()
418			{
419			return array_merge(parent::getDefinedElements(), [
420			'association_dropdown' => '.field > label:contains("%association%") ~ .product-select',
421			'association_dropdown_item' => '.field > label:contains("%association%") ~ .product-select > div.menu > div.item:contains("%item%")',
422			'association_dropdown_item_selected' => '.field > label:contains("%association%") ~ .product-select > a.label:contains("%item%")',
423			'attribute' => '.tab[data-tab="%localeCode%"] .attribute .label:contains("%attributeName%") ~ input',
424			'attribute_delete_button' => '.tab[data-tab="%localeCode%"] .attribute .label:contains("%attributeName%") ~ button',
425			'code' => '#sylius_product_code',
426			'images' => '#sylius_product_images',
427			'language_tab' => '[data-locale="%locale%"] .title',
428			'locale_tab' => '#attributesContainer .menu [data-tab="%localeCode%"]',
429			'name' => '#sylius_product_translations_%locale%_name',
430			'original_price' => '#sylius_product_variant_channelPricings > .field:contains("%channelName%") input[name$="[originalPrice]"]',
431			'price' => '#sylius_product_variant_channelPricings > .field:contains("%channelName%") input[name$="[price]"]',
432			'pricing_configuration' => '#sylius_calculator_container',
433			'main_taxon' => '#sylius_product_mainTaxon',
434			'shipping_required' => '#sylius_product_variant_shippingRequired',
435			'slug' => '#sylius_product_translations_%locale%_slug',
436			'tab' => '.menu [data-tab="%name%"]',
437			'taxonomy' => 'a[data-tab="taxonomy"]',
438			'tracked' => '#sylius_product_variant_tracked',
439			'toggle_slug_modification_button' => '[data-locale="%locale%"] .toggle-product-slug-modification',
440			]);
441			}
442
443			private function openTaxonBookmarks()
444			{
445			$this->getElement('taxonomy')->click();
446			}
447
448			/**
449			* @param string $tabName
450			*/
451			private function clickTabIfItsNotActive($tabName)
452			{
453			$attributesTab = $this->getElement('tab', ['%name%' => $tabName]);
454			if (!$attributesTab->hasClass('active')) {
455			$attributesTab->click();
456			}
457			}
458
459			/**
460			* @param string $tabName
461			*/
462			private function clickTab($tabName)
463			{
464			$attributesTab = $this->getElement('tab', ['%name%' => $tabName]);
465			$attributesTab->click();
466			}
467
468			/**
469			* @param string $localeCode
470			*/
471			private function clickLocaleTabIfItsNotActive($localeCode)
472			{
473			$localeTab = $this->getElement('locale_tab', ['%localeCode%' => $localeCode]);
474			if (!$localeTab->hasClass('active')) {
475			$localeTab->click();
476			}
477			}
478
479			/**
480			* @param string $type
481			*
482			* @return NodeElement
483			*/
484			private function getImageElementByType($type)
485			{
486			$images = $this->getElement('images');
487			$typeInput = $images->find('css', 'input[value="'.$type.'"]');
488
489			if (null === $typeInput) {
490			return null;
491			}
492
493			return $typeInput->getParent()->getParent()->getParent();
494			}
495
496			/**
497			* @return NodeElement[]
498			*/
499			private function getImageElements()
500			{
501			$images = $this->getElement('images');
502
503			return $images->findAll('css', 'div[data-form-collection="item"]');
504			}
505
506			/**
507			* @return NodeElement
508			*/
509			private function getLastImageElement()
510			{
511			$imageElements = $this->getImageElements();
512
513			Assert::notEmpty($imageElements);
514
515			return end($imageElements);
516			}
517
518			/**
519			* @return NodeElement
520			*/
521			private function getFirstImageElement()
522			{
523			$imageElements = $this->getImageElements();
524
525			Assert::notEmpty($imageElements);
526
527			return reset($imageElements);
528			}
529
530			/**
531			* @param NodeElement $imageElement
532			* @param string $type
533			*/
534			private function setImageType(NodeElement $imageElement, $type)
535			{
536			$typeField = $imageElement->findField('Type');
537			$typeField->setValue($type);
538			}
539			}
540

pamil / Sylius

Push — upgrade ( c4e463 )

UpdateSimpleProductPage::getNumberOfAttributes() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like