spryker-eco / ratepay

src/SprykerEco/Zed/Ratepay/Business/Expander/ProductExpander.php

<?php

/**
 * MIT License
 * Use of this software requires acceptance of the Evaluation License Agreement. See LICENSE file.
 */

namespace SprykerEco\Zed\Ratepay\Business\Expander;

use Generated\Shared\Transfer\CartChangeTransfer;
use SprykerEco\Zed\Ratepay\Dependency\Facade\RatepayToProductInterface;

class ProductExpander implements ProductExpanderInterface
{
    /**
     * @var \SprykerEco\Zed\Ratepay\Dependency\Facade\RatepayToProductBridge
     */
    protected $ratepayToProductBridge;

    /**
     * @param \SprykerEco\Zed\Ratepay\Dependency\Facade\RatepayToProductInterface $ratepayToProductBridge
     */
    public function __construct(RatepayToProductInterface $ratepayToProductBridge)
    {
        $this->ratepayToProductBridge = $ratepayToProductBridge;

$ratepayToProductBridge is of type SprykerEco\Zed\Ratepay\Dependency\Facade\RatepayToProductInterface, but the property $ratepayToProductBridge was declared to be of type SprykerEco\Zed\Ratepay\Dependency\Facade\RatepayToProductBridge. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

    }

    /**
     * @param \Generated\Shared\Transfer\CartChangeTransfer $change
     *
     * @return \Generated\Shared\Transfer\CartChangeTransfer
     */
    public function expandItems(CartChangeTransfer $change)
    {
        foreach ($change->getItems() as $cartItem) {
            $productConcreteTransfer = $this->ratepayToProductBridge->getProductConcrete($cartItem->getSku());

            foreach ($productConcreteTransfer->getLocalizedAttributes() as $localizedAttribute) {
                $attr = $localizedAttribute->getAttributes();
                if (isset($attr['short_description'])) {
                    $cartItem->setDescriptionAddition($attr['short_description']);
                }
                if (isset($attr['long_description'])) {
                    $cartItem->setDescription($attr['long_description']);
                }
            }
        }

        return $change;
    }
}