Merge pull request #112 from magento-commerce/feature/add-to-cart-tracking

Feature/add to cart tracking

Author: bgiamarino

app/code/Meta/BusinessExtension/Helper/MagentoDataHelper.php

@@ -17,8 +17,10 @@
 
 namespace Meta\BusinessExtension\Helper;
 
+use Magento\Catalog\Api\Data\ProductInterface;
 use Magento\Catalog\Api\ProductRepositoryInterface;
 use Magento\Customer\Api\CustomerMetadataInterface;
+use Magento\Framework\Exception\LocalizedException;
 use Magento\Store\Model\StoreManagerInterface;
 use Magento\Checkout\Model\Session as CheckoutSession;
 use Magento\Quote\Model\Quote;
@@ -50,57 +52,57 @@ class MagentoDataHelper extends AbstractHelper
     /**
      * @var StoreManagerInterface
      */
-    private $storeManager;
+    private StoreManagerInterface $storeManager;
 
     /**
      * @var CustomerMetadataInterface
      */
-    private $customerMetadata;
+    private CustomerMetadataInterface $customerMetadata;
 
     /**
      * @var ProductRepositoryInterface
      */
-    private $productRepository;
+    private ProductRepositoryInterface $productRepository;
 
     /**
      * @var ProductIdentifier
      */
-    private $productIdentifier;
+    private ProductIdentifier $productIdentifier;
 
     /**
      * @var CheckoutSession
      */
-    private $checkoutSession;
+    private CheckoutSession $checkoutSession;
 
     /**
-     * @var Quote
+     * @var Quote|null
      */
     private $quote;
 
     /**
      * @var CustomerSession
      */
-    private $customerSession;
+    private CustomerSession $customerSession;
 
     /**
      * @var CategoryRepositoryInterface
      */
-    private $categoryRepository;
+    private CategoryRepositoryInterface $categoryRepository;
 
     /**
      * @var PricingHelper
      */
-    private $pricingHelper;
+    private PricingHelper $pricingHelper;
 
     /**
      * @var AddressFactory
      */
-    private $addressFactory;
+    private AddressFactory $addressFactory;
 
     /**
      * @var RegionFactory
      */
-    private $regionFactory;
+    private RegionFactory $regionFactory;
 
     /**
      * MagentoDataHelper constructor
@@ -174,25 +176,30 @@ public function getLastName(): string
     }
 
     /**
-     * Return currently logged in users' Date of Birth.
+     * Return the product by the given sku
      *
-     * @return string
+     * @param string $productSku
+     * @return ProductInterface | bool
      */
-    public function getDateOfBirth(): string
+    public function getProductBySku(string $productSku)
     {
-        return $this->customerSession->getCustomer()->getDob();
+        try {
+            return $this->productRepository->get($productSku);
+        } catch (NoSuchEntityException $e) {
+            return false;
+        }
     }
 
     /**
-     * Return the product by the given sku
+     * Get Product by ID
      *
-     * @param string $productSku
-     * @return \Magento\Catalog\Api\Data\ProductInterface | bool
+     * @param mixed $productId
+     * @return false|ProductInterface
      */
-    public function getProductBySku($productSku)
+    public function getProductById($productId)
     {
         try {
-            return $this->productRepository->get($productSku);
+            return $this->productRepository->getById($productId);
         } catch (NoSuchEntityException $e) {
             return false;
         }
@@ -201,7 +208,7 @@ public function getProductBySku($productSku)
     /**
      * Return the categories for the given product
      *
-     * @param \Magento\Catalog\Model\Product $product
+     * @param Product $product
      * @return string
      */
     public function getCategoriesForProduct($product): string
@@ -235,7 +242,7 @@ public function getContentType(Product $product): string
     }
 
     /**
-     * Get content id
+     * Get Content IDs (Product IDs)
      *
      * @param Product $product
      * @return bool|int|string
@@ -248,10 +255,10 @@ public function getContentId(Product $product)
     /**
      * Return the price for the given product
      *
-     * @param \Magento\Catalog\Model\Product $product
+     * @param Product $product
      * @return float
      */
-    public function getValueForProduct($product): float
+    public function getValueForProduct(Product $product): float
     {
         $price = $product->getFinalPrice();
         return $this->pricingHelper->currency($price, false, false);
@@ -261,7 +268,8 @@ public function getValueForProduct($product): float
      * Return the currency used in the store
      *
      * @return string
-     * @throws \Magento\Framework\Exception\NoSuchEntityException
+     * @throws NoSuchEntityException
+     * @throws LocalizedException
      */
     public function getCurrency(): string
     {
@@ -376,7 +384,7 @@ public function getOrderContentIds(): array
      *
      * @return float|null
      */
-    public function getOrderTotal()
+    public function getOrderTotal(): ?float
     {
         $order = $this->checkoutSession->getLastRealOrder();
         if (!$order) {
@@ -452,7 +460,7 @@ public function getCurrentCustomer(): ?Customer
     /**
      * Return the address of a given customer
      *
-     * @param object $customer
+     * @param mixed $customer
      * @return Address
      */
     public function getCustomerAddress($customer): Address
@@ -461,42 +469,12 @@ public function getCustomerAddress($customer): Address
         return $this->addressFactory->create()->load($customerAddressId);
     }
 
-    /**
-     * Return the region's code for the given address
-     *
-     * @param object $address
-     * @return string|null
-     */
-    public function getRegionCodeForAddress($address): ?string
-    {
-        $region = $this->regionFactory->create()->load($address->getRegionId());
-        if ($region) {
-            return $region->getCode();
-        } else {
-            return null;
-        }
-    }
-
-    /**
-     * Return the string representation of the customer gender
-     *
-     * @param object $customer
-     * @return string|null
-     */
-    public function getGenderAsString($customer): ?string
-    {
-        if ($customer->getGender()) {
-            return $customer->getResource()->getAttribute('gender')->getSource()->getOptionText($customer->getGender());
-        }
-        return null;
-    }
-
     /**
      * Return all of the match keys that can be extracted from order information
      *
      * @return array
-     * @throws \Magento\Framework\Exception\LocalizedException
-     * @throws \Magento\Framework\Exception\NoSuchEntityException
+     * @throws LocalizedException
+     * @throws NoSuchEntityException
      */
     public function getUserDataFromOrder(): array
     {
@@ -537,8 +515,8 @@ public function getUserDataFromOrder(): array
      * Return all of the match keys that can be extracted from user session
      *
      * @return array
-     * @throws \Magento\Framework\Exception\LocalizedException
-     * @throws \Magento\Framework\Exception\NoSuchEntityException
+     * @throws LocalizedException
+     * @throws NoSuchEntityException
      */
     public function getUserDataFromSession(): array
     {
@@ -576,7 +554,7 @@ public function getUserDataFromSession(): array
     }
 
     /**
-     * Hash value
+     * Get Hash value
      *
      * @param string $string
      * @return string
@@ -590,6 +568,8 @@ private function hashValue($string): string
      * Get active quote
      *
      * @return Quote
+     * @throws LocalizedException
+     * @throws NoSuchEntityException
      */
     public function getQuote(): Quote
     {