Magento – Get all products of a category only – without products of its child categories

category-productscollection-filteringproduct-collection

I'm trying to get all products of a certain category. Unfortunately I always get not only all products of the category but also all products from its child categories. What am I doing wrong?

For example I'm working with following code snippet:

$catId = 160;
$category = Mage::getModel('catalog/category')->load($catId);
$collection = $category->getProductCollection()->addAttributeToSort('position');
Mage::getModel('catalog/layer')->prepareProductCollection($collection);

foreach ($collection as $product) {
var_dump( $product->getId() );
}

Above category (ID 160) contains 17 products but I get 51 products. If I look these up I see that all these products are assigned to one of the child categories of category ID 160. Sometimes even a sub-sub-category of #160.

Ran a full reindex – didn't help. Caches are all disabled.
Magento ver. 1.9.2.0

Thank you so much in advance!

Best Answer

Figured it out myself.

$category_id = Mage::getModel('catalog/layer')->getCurrentCategory()->getId();
$category = Mage::getModel('catalog/category')->load($category_id);
$products = Mage::getModel('catalog/product')
    ->getCollection()
    ->addAttributeToSelect('*')
    ->addCategoryFilter($category)
    ->setOrder('price', 'ASC')
    ->load();

you can also use for more than one category as bellow:

$categoryIds = array(2,4);//category id

$collection = Mage::getModel('catalog/product')
                             ->getCollection()
                             ->joinField('category_id', 'catalog/category_product', 'category_id', 'product_id = entity_id', null, 'left')
                             ->addAttributeToSelect('*')
                             ->addAttributeToFilter('category_id', array('in' => $categoryIds))

I hope this will help you.

Related Solutions

Magento – Get Product Collection Without Sub-Products of Configurable Products

Figured it out by myself. The table catalog_product_relation contains information about parent-child relationships for configurable products. I just had to join the table to my collection and look for product records that do not have a corresponding row in catalog_product_relation with their entity_id as child_id. Quite simple if you know what you have to do.

    $products = Mage::getResourceModel('catalog/product_collection')
        ->joinTable('catalog/product_relation', 'child_id=entity_id', array(
            'parent_id' => 'parent_id'
        ), null, 'left')
        ->addAttributeToFilter(array(
            array(
                'attribute' => 'parent_id',
                'null' => null
            )
        ));

Magento – Bad performance looping through all products

Product Collection from Category

// Step 1
$category = Mage::getModel('catalog/category')->load($category_id);

// Step 2
$productCollection = Mage::getModel('catalog/product')->getCollection();

// Step 3
$productCollection->addCategoryFilter($category);

// Step 4
$productCollection
    ->addAttributeToSelect('sku')
    ->addAttributeToSelect('another_attribute')
    ->addAttributeToFilter('type_id', 'simple')
    ->addAttributeToFilter('status', 1);

// Now you can loop through your collection
foreach($productCollection as $product) {
    $product->getSku();
    $product->getAnotherAttribute();
    // and so on...
}

So what's going on?

We loaded a category model. We will use that in the product collection.
We instantiated the product collection model.
We added the category for filtering.
We added the appropriate attributes for selection and filtering.

NOTE: Do not use the asterisk unless you plan on using every attribute being returned. Just like you shouldn't use the asterisk in a MySQL query unless you plan on getting everything from that table.

Once I have all my attributes and filters in place, I can loop through my collection. I do not need to call the load() method because lazy loading does that for me.

Once I'm in the loop, I have access to the product object Mage_Catalog_Model_Product. I do not need to instantiate another model inside the loop. The collection does this for me.