Remove trailing comma for PHP 7.4 support + PHPDoc (#480)

Author: zlik

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

@@ -72,7 +72,7 @@ public function updateCatalogConfiguration(
         string $catalogId,
         string $commercePartnerIntegrationId,
         string $pixelId,
-        bool $triggerFullSync = true,
+        bool $triggerFullSync = true
     ): void {
         $oldCatalogId = $this->systemConfig->getCatalogId($storeId);
         try {

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

@@ -438,9 +438,8 @@ public function fetchAndSaveAAMSettings($pixelId, $storeId)
     /**
      * Check admin permissions
      *
-     * @param string $pixelId
-     * @param int $storeId
      * @return void
+     * @throws LocalizedException
      */
     public function checkAdminEndpointPermission()
     {

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

@@ -38,7 +38,6 @@
  */
 class GraphAPIAdapter
 {
-    private const ORDER_STATE_CREATED = 'CREATED';
     public const ORDER_FILTER_REFUNDS = 'refunds';
     public const ORDER_FILTER_CANCELLATIONS = 'cancellations';
     private const GET_ORDERS_LIMIT = 25;

app/code/Meta/BusinessExtension/Model/System/Config.php

@@ -328,7 +328,7 @@ public function isCommerceExtensionEnabled($scopeId = null, $scope = ScopeInterf
      *
      * @param int|null $scopeId
      * @param string $scope
-     * @return bool
+     * @return string
      */
     public function getCommerceExtensionBaseURL($scopeId = null, $scope = ScopeInterface::SCOPE_STORE): string
     {
@@ -989,8 +989,8 @@ public function getWeightUnit(int $scopeId = null, string $scope = null)
     /**
      * Check if unsupported products are disabled
      *
-     * @param int $scopeId
-     * @param string $scope
+     * @param int|null $scopeId
+     * @param string|null $scope
      * @return bool
      */
     public function isUnsupportedProductsDisabled(int $scopeId = null, string $scope = null): bool

app/code/Meta/Catalog/Block/Adminhtml/Diagnostics.php

@@ -165,7 +165,7 @@ public function getSampleAffectedItems(array $diagnosticItem, int $catalogId, in
      * Get admin url
      *
      * @param ProductInterface $product
-     * @param int $store
+     * @param int|null $store
      * @return string
      */
     public function getAdminUrl(ProductInterface $product, int $store = null)

app/code/Meta/Catalog/Model/Product/Feed/Builder.php

@@ -972,7 +972,7 @@ public function getHeaderFields()
     /**
      * Get base url media
      *
-     * @return mixed
+     * @return string
      */
     public function getBaseUrlMedia()
     {

app/code/Meta/Catalog/Model/Product/Feed/Mapper.php

@@ -143,7 +143,7 @@ public function mapLink(BundleLinkInterface $link): array
      * Maps, and optionally filters, an array of product links into the Meta feed format
      *
      * @param ProductLinkInterface[] $product_links
-     * @param string $link_type
+     * @param string|null $link_type
      * @return array
      */
     public function mapProductLinks(array $product_links, string $link_type = null): array

app/code/Meta/Catalog/Model/Product/Feed/Method/BatchApi.php

@@ -112,6 +112,7 @@ private function buildProductRequest(Product $product, $method = self::ATTR_UPDA
      * Build request for individual product
      *
      * @param string $productIdentifier
+     * @return array
      */
     public function buildDeleteProductRequest(string $productIdentifier): array
     {

app/code/Meta/Catalog/Model/Product/Feed/Method/FeedApi.php

@@ -149,6 +149,7 @@ private function getFbFeedId()
      *
      * @param string $catalogId
      * @param string $feedName
+     * @return mixed
      * @throws GuzzleException
      */
     private function createNewFeedAndSave(string $catalogId, string $feedName)

app/code/Meta/Catalog/Model/Product/Feed/ProductRetriever/Configurable.php

@@ -92,9 +92,9 @@ public function __construct(
      * Set store id
      *
      * @param int $storeId
-     * @return ProductRetrieverInterface|void
+     * @return ProductRetrieverInterface
      */
-    public function setStoreId($storeId)
+    public function setStoreId($storeId): ProductRetrieverInterface
     {
         $this->storeId = $storeId;
         return $this;