Magento – Sorting custom attribute not working in magento

magento-1.9product-collectionsorting

I need to sort the product collection by custom attribute called popular which having value 1 or 0. Below is my code,

    $collection1 = Mage::getModel('catalog/product')->getCollection();
    $collection1->addAttributeToFilter(array(array('attribute' => 'status', 'eq' => 1)));
    $collection1->addAttributeToSelect(array(array('attribute'=> 'price')));
    $collection1->addAttributeToSelect(array(array('attribute'=> 'special_price')));
    $collection1->addAttributeToSort('popular', 'desc');
    foreach ($collection1 as $col) {
                        echo $col->getId();
                        echo '<br>';

    }
    exit;

Here I just printing the product id and every load it is showing different products. I re-indexed the products and I have set the product attribute options all to yes in admin. Like Used for Sorting in Product Listing to yes, Used in Product Listing to yes, but still Im not getting unique product list. Is there something I missed here ..? And if we apply two sorting to the collections which sort will it take ?. please help me.

Note: I have tried with price attribute type for this attribute but still no luck (I thought if the attribute type is price then sorting will work correctly)

Update 1: I already tried with setorder, but that is also not working.

Best Answer

Please add attribute to select then apply sorting by using order()

    $collection1 = Mage::getModel('catalog/product')->getCollection();
    $collection1->addAttributeToSelect('*');
    $collection1->addAttributeToFilter(array(array('attribute' => 'status', 'eq' => 1)));
    //$collection1->addAttributeToSort('popular', 'desc');
$collection1->getSelect()->order('popular DESC');
    foreach ($collection1 as $col) {
                        echo $col->getId();
                        echo '<br>';

    }
    exit;

Related Solutions

Magento 1.8 – Fix Sort by Yes/No Attribute Not Working

Short answer: The sorting doesn't work for Yes/No attributes.

Long Answer:
The problem:

When calling addAttributeToSort on a product collection, if the attribute has a source model then Magento calls addValueSortToCollection from that source model to sort the products.
The Yes/No attributes use the source model Mage_Eav_Model_Entity_Attribute_Source_Boolean that does not implement the method addValueSortToCollection For this model the method just returns $this without processing the collection.

A possible solution:
You can rewrite the Mage_Eav_Model_Entity_Attribute_Source_Boolean and add your addValueSortToCollection method. It should basically be the same code as it is in the class Mage_Catalog_Model_Product_Visibility or Mage_Catalog_Model_Product_Status (not entirely sure about this) which is:

public function addValueSortToCollection($collection, $dir = 'asc')
{
    $attributeCode  = $this->getAttribute()->getAttributeCode();
    $attributeId    = $this->getAttribute()->getId();
    $attributeTable = $this->getAttribute()->getBackend()->getTable();

    if ($this->getAttribute()->isScopeGlobal()) {
        $tableName = $attributeCode . '_t';
        $collection->getSelect()
            ->joinLeft(
                array($tableName => $attributeTable),
                "e.entity_id={$tableName}.entity_id"
                    . " AND {$tableName}.attribute_id='{$attributeId}'"
                    . " AND {$tableName}.store_id='0'",
                array());
        $valueExpr = $tableName . '.value';
    }
    else {
        $valueTable1 = $attributeCode . '_t1';
        $valueTable2 = $attributeCode . '_t2';
        $collection->getSelect()
            ->joinLeft(
                array($valueTable1 => $attributeTable),
                "e.entity_id={$valueTable1}.entity_id"
                    . " AND {$valueTable1}.attribute_id='{$attributeId}'"
                    . " AND {$valueTable1}.store_id='0'",
                array())
            ->joinLeft(
                array($valueTable2 => $attributeTable),
                "e.entity_id={$valueTable2}.entity_id"
                    . " AND {$valueTable2}.attribute_id='{$attributeId}'"
                    . " AND {$valueTable2}.store_id='{$collection->getStoreId()}'",
                array()
            );
            $valueExpr = $collection->getConnection()->getCheckSql(
                $valueTable2 . '.value_id > 0',
                $valueTable2 . '.value',
                $valueTable1 . '.value'
            );
    }

    $collection->getSelect()->order($valueExpr . ' ' . $dir);
    return $this;
}

Explanation provided here also for a similar issue

Magento-1 – How to Resolve Issues Sorting by Two Attributes

I think the problem here is that there are some special code that needs to be run to sort by price.

If you take a look into the function setOrder on the model Mage_Catalog_Model_Resource_Product_Collection you will see that it does something "special" when dealing with the attribute price.

public function setOrder($attribute, $dir = 'desc')
{
    if ($attribute == 'price') {
        $this->addAttributeToSort($attribute, $dir);
    } else {
        parent::setOrder($attribute, $dir);
    }
    return $this;
}

My suggestion for you is to follow this example and try adding the sort by price in the same fashion and by calling addAttributeToSort.

Best Answer

Related Solutions

Magento 1.8 – Fix Sort by Yes/No Attribute Not Working

Magento-1 – How to Resolve Issues Sorting by Two Attributes

Related Topic