feat: collect all possible propertyGroups of each product for comparison

Author: tinect

src/Page/CompareProductPage.php

@@ -3,11 +3,13 @@
 namespace Frosh\FroshProductCompare\Page;
 
 use Shopware\Core\Content\Product\SalesChannel\Listing\ProductListingResult;
+use Shopware\Core\Content\Property\PropertyGroupCollection;
 use Shopware\Storefront\Page\Page;
 
 class CompareProductPage extends Page
 {
     protected ProductListingResult $products;
+    protected PropertyGroupCollection $properties;
 
     public function getProducts(): ProductListingResult
     {
@@ -18,4 +20,14 @@ public function setProducts(ProductListingResult $products): void
     {
         $this->products = $products;
     }
+
+    public function getProperties(): PropertyGroupCollection
+    {
+        return $this->properties;
+    }
+
+    public function setProperties(PropertyGroupCollection $properties): void
+    {
+        $this->properties = $properties;
+    }
 }

src/Page/CompareProductPageLoader.php

@@ -96,6 +96,7 @@ public function load(array $productIds, Request $request, SalesChannelContext $s
 
         $page = $this->genericLoader->load($request, $salesChannelContext);
 
+        /** @var CompareProductPage $page */
         $page = CompareProductPage::createFrom($page);
 
         if (empty($productIds)) {
@@ -111,6 +112,22 @@ public function load(array $productIds, Request $request, SalesChannelContext $s
 
         $page->setProducts($result);
 
+        $properties = new PropertyGroupCollection();
+
+        /** @var SalesChannelProductEntity $product */
+        foreach ($result as $product) {
+            foreach ($product->getSortedProperties() as $group) {
+                // we don't need more data of the PropertyGroup so we just set id and translated instead of cloning
+                $propertyGroup = new PropertyGroupEntity();
+                $propertyGroup->setId($group->getId());
+                $propertyGroup->setTranslated($group->getTranslated());
+
+                $properties->add($propertyGroup);
+            }
+        }
+
+        $page->setProperties($properties);
+
         return $page;
     }
 

src/Resources/views/storefront/component/compare/partial/properties.html.twig

@@ -2,7 +2,7 @@
     {% for product in products %}
         {% set matchProperty = false %}
         {% for group in product.sortedProperties %}
-            {% if group.id == property.id %}
+            {% if group.id == linePropertyId %}
                 {% set matchProperty = true %}
                 {% block frosh_product_compare_properties_cell %}
                     <td class="properties-value">

src/Resources/views/storefront/component/compare/section/specification.html.twig

@@ -4,13 +4,11 @@
             <th class="text-uppercase" colspan="{{ productsCount + 1 }}">{{ "froshProductCompare.section.specification.title"|trans|sw_sanitize }}</th>
         </tr>
 
-        {% set setProperties = products.first.sortedProperties %}
-
-        {% for property in setProperties %}
+        {% for property in page.properties %}
             <tr class="property">
                 <th class="capitalize">{{ property.translated.name|e }}</th>
                 {% sw_include '@Storefront/storefront/component/compare/partial/properties.html.twig' with {
-                    property: property,
+                    linePropertyId: property.id,
                 } %}
             </tr>
         {% endfor %}