php - Magento Block Class Rewrite In Config.xml - Second Guessing My Methodology -


so have written module works quite well. however, trying make sure write of modules conflict-free possible. short: module displays reviews associated products of grouped , configurable products on parent product page , listings.

in catalog/product/list.phtml there 3 lines of code if parent product has no reviews, though children products have reviews, not satisfy if condition in order display summary html. 

<?php if($_product->getratingsummary()): ?> <?php echo $this->getreviewssummaryhtml($_product) ?> <?php endif; ?>

so, goal add data each element in collection before reached template file.

$_productcollection=$this->getloadedproductcollection();

so, did rewrite of mage_catalog_block_product_list , rewrote method below.

protected function _getproductcollection()

actually, copied exact method on , added simple code bottom of , indeed want do.

protected function _getproductcollection() {   if (is_null($this->_productcollection)) {       $layer = $this->getlayer();       /* @var $layer mage_catalog_model_layer */       if ($this->getshowrootcategory()) {           $this->setcategoryid(mage::app()->getstore()->getrootcategoryid());       }        // if product view page       if (mage::registry('product')) {           // collection of categories product associated           $categories = mage::registry('product')->getcategorycollection()               ->setpage(1, 1)               ->load();           // if product associated category           if ($categories->count()) {               // show products category               $this->setcategoryid(current($categories->getiterator()));           }       }        $origcategory = null;       if ($this->getcategoryid()) {           $category = mage::getmodel('catalog/category')->load($this->getcategoryid());           if ($category->getid()) {               $origcategory = $layer->getcurrentcategory();               $layer->setcurrentcategory($category);               $this->addmodeltags($category);           }       }       $this->_productcollection = $layer->getproductcollection();        $this->preparesortablefieldsbycategory($layer->getcurrentcategory());        if ($origcategory) {           $layer->setcurrentcategory($origcategory);       }   }    foreach($this->_productcollection $product){     if(($product->gettypeid() == 'configurable' || $product->gettypeid() == 'grouped')      && !$product->getratingsummary()) {         $product->setratingsummary(1);     }   }    return $this->_productcollection; }

what cant second guess need rewrite function add data collection returns or there way add data collection before reaches template file without doing actual rewrite have done above. category product list 1 of main data collections on magento site , not crazy messing add tiny snippet of data. also, worry conflicts future extensions.

please let me know how alter collection object without rewrite, if possible.

thank you.

if going rewrite, i'd recommend methodology this. 

protected function _getproductcollection() {     $collection = parent::_getproductcollection();     //do stuff collection here     return $collection; }

your block class extends existing block class. means can call parent method , have same code run. when copy , paste code did in example, risk extension not working in past/future versions of magento code has changed.

my general rule of thumb on using rewrites is: if stay in control of code , can debug potential conflicts, rewrite fine. if don't stay in control of code, shouldn't rewrite method.

if want avoid rewrite, other option magento's event system. every collection in magento fires event before , after it's loaded. trick finding name of event you're after, requires either digging through source of mage::dispatch, or using tool like commerce bug (warning: self link) track down events your'e after.

i prefer second method, using commerce bug's observer's tab (which shows existing magento observers setup on page), can see magento has listener setup product collection. specifically, on catalog_product_collection_load_after event.

enter image description here

if @ source of magento's cataloginventory/observer, can see how magento modifies collection after been loaded

#file: app/code/core/mage/cataloginventory/model/observer.php public function addstockstatustocollection($observer) {     $productcollection = $observer->getevent()->getcollection();     if ($productcollection->hasflag('require_stock_items')) {         mage::getmodel('cataloginventory/stock')->additemstoproducts($productcollection);     } else {         mage::getmodel('cataloginventory/stock_status')->addstockstatustoproducts($productcollection);     }     return $this; }

if case, instead of checking require_stock_items flag on products collection, check state of system says it's loading page want information added.

that should give enough started own listener. luck!


-

Comments

Post a Comment

Popular posts from this blog

html - Sizing a high-res image (~8MB) to display entirely in a small div (circular, diameter 100px) -

first of all, thank being patient know simple question more couple of weeks' experience figure out. i've seen folks persevere , helpful on here , hope you're willing me. here fiddle: http://jsfiddle.net/65uzf/ here 1 of places searched answers on here: ( vertically center image thumbnails fix-sized div ) tried solutions on there , didn't work out me. i've searched ~4 or 5 others, , been unsuccessful those, too. what want take large image (what linked background-image #ysypic , #tabpic divs) , display thumbnail version of image in confines of divs have set it. is, using background properties have laid out in css, portion of image displays inside of circular divs. without these properties, display defaults smaller section of image [presumably pi*(50^2) area in top left] terrible resolution. i not care if entire image doesn't display inside of circles (it won't, given difference in shape), , overflow not being displayed fine. want able see figu
Read more

java - IntelliJ - No such instance method -

using intellij's 12 ultimate, i'm running following code in debugger: java import play.api.libs.json.jsvalue; public class foo { ... public jsvalue tojson() { ... } public class fooexample { ... foo foo = new foo(); system.out.println("...); //<-- breakpoint at breakpoint, right-clicked source code , picked "evaluate expression," typing in: foo.tojson() . but following error showed up: no such instance method: play.api.libs.json.jsvalue$class.com.foo.foo.tojson () am doing wrong? foo#tojson calls scala code, if matters. edit had breakpoint after instantiation of foo. downvoted it, deserved it. edit my answer no longer valid after op modified question (i.e. moved breakpoint actual is).. a breakpoint hit before line executed. in case foo has not yet been declared or instantiated (i.e. foo constructor has not yet been called). you'll need put breakpoint @ next line (or step on current line) if want evaluate foo .
Read more

identifier - Is it possible for an html5 document to have two ids? -

i wondering if html5 element can have 2 ids, this: <button id="thing1" id="thing2" name="button1" onclick="change()">click here</button> or this: <button id="thing1, thing2" name="button1" onclick="change()">click here</button> also, if 1 of ids said 1 thing, , other id said thing, 1 used? tl;dr : no. in versions of html, each element can have 1 id. see the spec : the id attribute specifies element's unique identifier (id). [dom] the value must unique amongst ids in element's home subtree , must contain @ least 1 character. the value must not contain space characters. note: there no other restrictions on form id can take; in particular, ids can consist of digits, start digit, start underscore, consist of punctuation , etc. note: element's unique identifier can used variety of purposes, notably way link specific parts of document usin
Read more