Fix products duplicate across pagination when they have the same position value (#4750)

gdemarsico · kiatng · web-flow · commit da64ad64343f · 2025-04-16T23:42:28.000+08:00
Co-authored-by: Ng Kiat Siong &lt;kiatsiong.ng@gmail.com&gt;
diff --git a/app/code/core/Mage/Catalog/Model/Resource/Product/Collection.php b/app/code/core/Mage/Catalog/Model/Resource/Product/Collection.php
@@ -1520,7 +1520,7 @@ public function addAttributeToSort($attribute, $dir = self::SORT_ORDER_ASC)
             // optimize if using cat index
             $filters = $this->_productLimitationFilters;
             if (isset($filters['category_id']) || isset($filters['visibility'])) {
-                $this->getSelect()->order('cat_index.position ' . $dir);
+                $this->getSelect()->order(['cat_index.position ' . $dir, 'e.entity_id ' . $dir]);
             } else {
                 $this->getSelect()->order('e.entity_id ' . $dir);
             }

Original file line number	Diff line number	Diff line change
`@@ -1520,7 +1520,7 @@ public function addAttributeToSort($attribute, $dir = self::SORT_ORDER_ASC)`
`1520`	`1520`	`// optimize if using cat index`
`1521`	`1521`	`$filters = $this->_productLimitationFilters;`
`1522`	`1522`	`if (isset($filters['category_id']) \|\| isset($filters['visibility'])) {`
`1523`		`- $this->getSelect()->order('cat_index.position ' . $dir);`
	`1523`	`+ $this->getSelect()->order(['cat_index.position ' . $dir, 'e.entity_id ' . $dir]);`
`1524`	`1524`	`} else {`
`1525`	`1525`	`$this->getSelect()->order('e.entity_id ' . $dir);`
`1526`	`1526`	`}`