0

Below is a function that i'm unsure how to implement. I need to conditionally filter this collection. Is there a way without continuously applying one filter after one another? Thanks

public function filterProducts($collection, $params)
{
    if ($model = $params['product_model']) {
        //filter collection by model
    }
    if ($productType = $params['product_type']) {
        //filter collection by productType
    }
    if ($brand = $params['brand_name']) {
        //filter collection by brand
    }
    if ($year = $params['product_application_year']) {
        //filter collection by year
    }
    elseif($year = Mage::getModel('core/cookie')->get('cookie')){
        //filter collection by year
    }
    return $collection;
}

Below is what i've come up with thus far.

public function filterProducts($collection, $params)
{
    if ($params['product_model']) {
        $collection->addAttributeToFilter('product_model',$params['product_model']);
    }
    if ($params['product_type']) {
        $collection->addAttributeToFilter('product_type',$params['product_type']);
    }
    if ($params['brand_name']) {
        $collection->addAttributeToFilter('brand_name',$params['brand_name']);
    }
    if ($params['product_application_year']) {
        $collection->addAttributeToFilter('product_application_year',$params['product_application_year']);
    } elseif (Mage::getModel('core/cookie')->get('myMustang')) {
        $collection->addAttributeToFilter('product_application_year',$params['product_application_year']);

    }
    return $collection;
}

This is how i'm calling this function:

$_products = $this->filterProducts($this->getProductCollection(),$this->getRequest()->getParams());
  • Why do you think it's bad to apply filters in such way? – Lord Skeletor Jan 28 '15 at 19:37
  • @Reindex'EmAll Wouldn't it be much more time intensive to apply a filter one after another in such a manner? Although, this may be a necessity case. – easymoden00b Jan 28 '15 at 19:39
  • 2
    No it would not. You're not doing query on DB until you load the collection. – Lord Skeletor Jan 28 '15 at 19:47
  • @Reindex'EmAll Good to know, take a look at what i've come up with. – easymoden00b Jan 28 '15 at 20:07
1

Updated Answer

Based on what we discussed this should be your solution:

public function filterProducts($collection, $params)
{
   if ($params['product_model']) {
       $collection->addAttributeToFilter('product_model',$params['product_model']);
    }
    if ($params['product_type']) {
       $collection->addAttributeToFilter('product_type',$params['product_type']);
    }
    if ($params['brand_name']) {
        $collection->addAttributeToFilter('brand_name',$params['brand_name']);
    }
    if ($params['product_application_year']) {
        $collection->addAttributeToFilter('product_application_year',$params['product_application_year']);
    } elseif (Mage::getModel('core/cookie')->get('myMustang')) {
        $collection->addAttributeToFilter('product_application_year',$params['product_application_year']);

    }
    return $collection;
}

$_products = $this->filterProducts($this->getProductCollection(),$this->getRequest()->getParams());  
$_products->load();  

Another thing to note:

I don't know if you have made a block with this function getProductCollection but make sure it exists. If not you will have to make it or do this something like this:

$collection = Mage::getModel('catalog/product')->getCollection();
$_products = $this->filterProducts($collection,$this->getRequest()->getParams());

Old answer

I think this depends on what you are doing and what exactly you have in your $params array.

Here is one way. If the things you are trying to filter is the only thing in the params array then you can do something like:

foreach ($params as $param => $value) {
    $collection->addFieldToFilter($param, $value);
}

You should also check out addAttributeToFilter().

Hopefully this helps a little.

You can check out this link too http://www.magentocommerce.com/knowledge-base/entry/magento-for-dev-part-8-varien-data-collections

  • There are more than the above parameters contained within the params array, so the conditional must take place. I've edited my code a bit to reflect your foreach loop mechanism. Tell me what you think and whether it can be further condensed. – easymoden00b Jan 28 '15 at 19:56
  • 1
    If you are only doing those conditions then I think your initial code is the better way to go. You would be adding more steps to do the same thing. You can think of it in terms of cost. An if else statement costs 1. A array assignment costs 1, so 1 X number of assignments. Then foreach loop, 1 X number of elements in the array. Too much @_@. I would only suggest the foreach loop if you are doing more than just the conditions or if your params array just contains the filters. If this is all you are doing then i agree with @Reindex 'Em All. – Aedonis Jan 28 '15 at 20:01
  • 1
    Also to note: You must from my knowledge call addFieldToFilter() before you do $collection->load(). Magento does something called LazyLoading which means it doesn't do any SQL until you call $collection->load(). So I would suggest you not load the collection until you have filtered the collection. – Aedonis Jan 28 '15 at 20:08
  • 1
    I think your solution now after the edit is correct. (without the space between $this-> and filterProducts()) – Aedonis Jan 28 '15 at 20:11
  • I'm unsure what i've done is correct then. Nowhere in this .phtml file is $collection->load() called. – easymoden00b Jan 28 '15 at 20:54

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service, privacy policy and cookie policy

Not the answer you're looking for? Browse other questions tagged or ask your own question.