Merge pull request #4183 from magento-performance/MC-16312

[performance] BugFixes

Author: naydav

app/code/Magento/Sitemap/Model/ResourceModel/Catalog/Product.php

@@ -194,7 +194,6 @@ protected function _addFilter($storeId, $attributeCode, $value, $type = '=')
                 break;
             default:
                 return false;
-                break;
         }
 
         $attribute = $this->_getAttribute($attributeCode);
@@ -310,10 +309,6 @@ public function getCollection($storeId)
         }
 
         $connection = $this->getConnection();
-        $urlsConfigCondition = '';
-        if ($this->isCategoryProductURLsConfig($storeId)) {
-            $urlsConfigCondition = 'NOT ';
-        }
 
         $this->_select = $connection->select()->from(
             ['e' => $this->getMainTable()],
@@ -324,9 +319,7 @@ public function getCollection($storeId)
             []
         )->joinLeft(
             ['url_rewrite' => $this->getTable('url_rewrite')],
-            'e.entity_id = url_rewrite.entity_id AND url_rewrite.is_autogenerated = 1 '
-            . 'AND NULLIF(url_rewrite.metadata,"") IS '
-            . $urlsConfigCondition . 'NULL'
+            'e.entity_id = url_rewrite.entity_id AND url_rewrite.is_autogenerated = 1 AND url_rewrite.metadata IS NULL'
             . $connection->quoteInto(' AND url_rewrite.store_id = ?', $store->getId())
             . $connection->quoteInto(' AND url_rewrite.entity_type = ?', ProductUrlRewriteGenerator::ENTITY_TYPE),
             ['url' => 'request_path']
@@ -490,20 +483,4 @@ private function getProductImageUrl($image)
     {
         return $this->imageUrlBuilder->getUrl($image, 'product_page_image_large');
     }
-
-    /**
-     * Return Use Categories Path for Product URLs config value
-     *
-     * @param null|string $storeId
-     *
-     * @return bool
-     */
-    private function isCategoryProductURLsConfig($storeId)
-    {
-        return $this->scopeConfig->isSetFlag(
-            HelperProduct::XML_PATH_PRODUCT_URL_USE_CATEGORY,
-            ScopeInterface::SCOPE_STORE,
-            $storeId
-        );
-    }
 }