Error in system.log browsing product details page

Found this:

2014-08-06T11:09:43+00:00 ERR (3): Notice: Use of undefined constant active -- assumed ‘active’ in \app\design\frontend\default\jm_summertemplate\ca talog\product\view\media.phtml on line 98
2014-08-06T11:09:43+00:00 ERR (3): Notice: Undefined variable: identifier in \app\design\frontend\default\jm_summertemplate\ca talog\product\view.phtml on line 192
2014-08-06T11:09:43+00:00 ERR (3): Notice: Undefined variable: identifier in \app\design\frontend\default\jm_summertemplate\ca talog\product\view.phtml on line 193

3 answers

Profile photo of Saguaros 0.00 $tone August 7, 2014
Public

May I know which version of magento are you using with our theme? If possible, PM me URL, admin and ftp credentials of your site, I will have a look at this.

#1
Profile photo of Sven Lessmann -30.00 $tone August 8, 2014
Public

Hello Saguaros,
excuse me, but the side is not live. It’s on my development system to get rid of these errors before deploying. It is Magento 1.9.0.1.

1.About active line 98

Code:

<?php foreach ($galleryImages as $_image):?>
	    	<?php $thumb_image_name= strrchr($_image->getFile(), '/');?>
	        <li class="<?php if ($thumb_image_name==$main_image_name){ $start= $i;echo(active);}?>">
	        	<a href="<?php echo $this->helper('catalog/image')->init($this->getProduct(), 'image', $_image->getFile())->resize ( $productlimagewidth, $productlimageheight ); ?>"
				title="<?php echo $_product->getName();?>"
				onclick="return false;"> <!-- The below should remain the same as before -->
					<i class="fa fa-caret-up"></i>
					<img
					src="<?php echo $this->helper('catalog/image')->init($this->getProduct(), 'thumbnail', $_image->getFile())->keepAspectRatio(true)->resize(80, 80); ?>"
					alt="<?php echo $this->htmlEscape($_image->getLabel()) ?>"
					title="<?php echo $this->htmlEscape($_image->getLabel()) ?>" />
				</a>
			</li>

I think it’s a typo, ‘active’ should be echoed to mark the active thumbnail of the image gallary below the main image.
We have to use single quotes! The classname of the <li> should contain ‘active’ for CSS.

2.About $identifier line 192

Code:

<div id="ja-tab-products" class="product-collateral">
		<?php
			//add the customtab to the product tab
			$attribute = $_product->getResource()->getAttribute('size_chart');
			if(is_object($attribute)){
			  $identifier = $_product->getData("size_chart");
			  }
			$blockcustom =  Mage::app()->getLayout()->createBlock('cms/block')->setBlockId($identifier);
			$blockcustommodel = Mage::getModel('cms/block')->load($identifier);
			$customtitle = $blockcustommodel->getTitle();
			$customtab = $blockcustom->toHtml();
		?>

Here, a cms/block should be loaded, for an additional product tab in details view.I have seen this in another theme, maybe JM Megamall. In JM Summer documentation nothing told me to set up a size_chart product attribute and build a new cms/block. So the $identifier variable is not initalized by getdata() call and the loading of the cms block fails.
Is there something missing in documentation, or do we have to comment out this additional tab loading? (code copy bug)

And btw. when will updates be available on this website, because I found JM Megamenu V.1.2.7 for the august theme:
http://www.joomlart.com/forums/downl&#8230;?do=cat&id=530
Or is there another way to get updates for single licensed developers for used extensions?

#2

Please login or Register to Submit Answer

Written By

Comments