ExportOrderObserver::execute() - Code Metrics - commerceleague/magento2-module-activecampaign - Measure and Improve Code Quality continuously with Scrutinizer

ExportOrderObserver::execute() A
last analyzed 2019-07-18 12:58 UTC

↳ Parent: ExportOrderObserver

Complexity

Conditions	4
Paths	3

Size

Total Lines	16
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

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

<?php
declare(strict_types=1);
/**
 */

namespace CommerceLeague\ActiveCampaign\Observer\Sales;

use CommerceLeague\ActiveCampaign\Helper\Config as ConfigHelper;
use CommerceLeague\ActiveCampaign\MessageQueue\Topics;
use Magento\Framework\Event\Observer;
use Magento\Framework\Event\ObserverInterface;
use Magento\Framework\MessageQueue\PublisherInterface;
use Magento\Sales\Model\Order as MagentoOrder;

/**
 * Class ExportOrderObserver
 */
class ExportOrderObserver implements ObserverInterface
{
    /**
     * @var ConfigHelper
     */
    private $configHelper;

    /**
     * @var PublisherInterface
     */
    private $publisher;

    /**
     * @param ConfigHelper $configHelper
     * @param PublisherInterface $publisher
     */
    public function __construct(
        ConfigHelper $configHelper,
        PublisherInterface $publisher
    ) {
        $this->configHelper = $configHelper;
        $this->publisher = $publisher;
    }

    /**
     * @inheritDoc
     */
    public function execute(Observer $observer)
    {
        if (!$this->configHelper->isEnabled() || !$this->configHelper->isOrderExportEnabled()) {
            return;
        }

        /** @var MagentoOrder $magentoOrder */
        $magentoOrder = $observer->getEvent()->getData('order');

        if ($magentoOrder->getCustomerIsGuest()) {
0   == false // true
0   == null  // true
123 == false // false
123 == null  // false

// It is often better to use strict comparison
0 === false // false
0 === null  // false
            return;
        }

        $this->publisher->publish(
            Topics::SALES_ORDER_EXPORT,
            json_encode(['magento_order_id' => $magentoOrder->getId()])
            /** @scrutinizer ignore-type */ json_encode(['magento_order_id' => $magentoOrder->getId()])
        );
    }
}


commerceleague / magento2-module-activecampaign

GitHub Access Token became invalid

ExportOrderObserver::execute() A last analyzed 2019-07-18 12:58 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

ExportOrderObserver::execute() A
last analyzed 2019-07-18 12:58 UTC