Editing (v7+)

eCommerce - H1 for product name on product page, not search results?

Permalink May 18, 2011 at 4:32 PM

Ok, so I can change the core_commerce display.php file to add an h1 tag to the product name which displays nicely on a product page. Unfortunately this also adds an h1 tag to all search results, really breaking it. How can I have the h1 just on the product page?

Code below

<div>
      <?php   if ($displayNameP) { ?>
         <h1 class="commercehead"><strong><?php  echo $link_before.$product->getProductName().$link_after?></strong> 
      <?php   } ?>
      <?php   if ($displayNameP && $displayPriceP) { ?>
      - 
      <?php   } ?>
      <?php   if ($displayPriceP) { ?>
         <?php  echo Loader::packageElement('product/price', 'core_commerce', array('product' => $product, 'displayDiscount' => $displayDiscountP)); ?>
      <?php   } ?></h1>
      </div>

Thanks!

bentvision replied on May 18, 2011 at 11:40 pm Permalink Reply

Might I suggest that you use the STYLE command instead of H1:

<div class="commercehead" style="font-size:16px; font-weight:bold; color:#A30000;"><strong><?php  echo $link_before.$product->getProductName().$link_after?></strong> 
      <?php   } ?>
      <?php   if ($displayNameP && $displayPriceP) { ?>
      - 
      <?php   } ?>
      <?php   if ($displayPriceP) { ?>
         <?php  echo Loader::packageElement('product/price', 'core_commerce', array('product' => $product, 'displayDiscount' => $displayDiscountP)); ?>
      <?php   } ?></div>

RandomBrad replied on May 20, 2011 at 11:38 am Permalink Reply

I'm looking for H1 so that each product page will automatically generate the H1 for search engines, rather than manually doing it. It made sense to use the product name. Thanks for the suggestion!

bentvision replied on May 18, 2011 at 11:46 pm Permalink Reply

Or acutally, to target ONLY your product lists, don't edit the display.php at all.

Edit the block/ product_list/ view.php You'll find it here:

/packages/core_commerce/blocks/product_list/view.php

Copy this view.php file to your root C5 directory under:

/blocks/product_list/view.php

Then edit this file to your liking.

Good luck.

RandomBrad replied on May 20, 2011 at 11:45 am Permalink Reply

I'm no good with PHP :) I can't find where in here I would add the <h1> tag to affect the product name?

<?php   
defined('C5_EXECUTE') or die(_("Access Denied."));  
global $c;
$uh = Loader::helper('urls', 'core_commerce');
$im = Loader::helper('image');
if ($options['show_search_form']) {
   $this->inc('view_search_form.php', array( 'c'=>$c, 'b'=>$b, 'controller'=>$controller,'block_args'=>$block_args ) );
}
?>
<?php   if ($options['show_products'] || $_REQUEST['search'] == '1') { ?>
   <?php  
   $nh = Loader::helper('navigation');
   $productList = $this->controller->getRequestedSearchResults();
   $products = $productList->getPage();
   $paginator = $productList->getPagination();

Viewing 15 lines of 111 lines. View entire code block.

 
<?php   
defined('C5_EXECUTE') or die(_("Access Denied."));  
 
global $c;
 
$uh = Loader::helper('urls', 'core_commerce');
$im = Loader::helper('image');
if ($options['show_search_form']) {
   $this->inc('view_search_form.php', array( 'c'=>$c, 'b'=>$b, 'controller'=>$controller,'block_args'=>$block_args ) );
}
?>
<?php   if ($options['show_products'] || $_REQUEST['search'] == '1') { ?>
   <?php  
   $nh = Loader::helper('navigation');
 
   $productList = $this->controller->getRequestedSearchResults();
   $products = $productList->getPage();
   $paginator = $productList->getPagination();
   ?>
   <?php   if(count($products)>0) { ?>
 
      <table border="0" cellspacing="0" cellpadding="0">
      <tr>
         <td>
         <?php   if($paging['show_top'] || $paging['show_bottom']) {
            echo '<div class="ccm-core-commerce-summary">';
            $productList->displaySummary();
            echo '</div>';
         }
         ?>            
         </td><?php  
            if (count($sort_columns)>0) { ?>
               <td style="padding-left: 20px">
               <div class="product-list-sort-header"><?php  echo  t('Sort by:'); ?> <select class="product-list-sort-select">
               <?php  
               $current_col = $_REQUEST['ccm_order_by'];
               foreach ($sort_columns as $col => $name) {
                  $selected = ($current_col == $col && $_REQUEST['ccm_order_dir'] == 'asc') ? "selected" : "";
                  echo '<option '.$selected.' value="';
                  $productList->getSortByURL($col, 'asc', $bu);
                  echo '">'.$name.' ' . t('Ascending') . '</option>';
                  $selected = ($current_col == $col && $_REQUEST['ccm_order_dir'] == 'desc') ? "selected" : "";
 
                  echo '<option '.$selected.' value="';
                  $productList->getSortByURL($col, 'desc', $bu);
                  echo '">'.$name.' ' . t('Descending') . '</option>';
               }
               ?></select>
 
               </div>
               </td>
               <?php  
            } ?>
      </tr>
      </table>
      <br/>
   <div style="clear: both"></div>
      <?php   if($paging['show_top'] && $paginator && strlen($paginator->getPages())>0){ ?>   
      <div class="pagination">   
          <span class="pageLeft"><?php   echo $paginator->getPrevious(t('Previous'))?></span>
          <?php   echo $paginator->getPages()?>
          <span class="pageRight"><?php   echo $paginator->getNext(t('Next'))?></span>
      </div>
      <br/>
      <?php   } ?>
 
      <table width="100%" class="ccm-core-commerce-product-list-results" border="0" cellspacing="<?php  echo  (int) $layout['spacing'] ?>" cellpadding="<?php  echo  (int) $layout['padding'] ?>">
      <?php  
      if ($layout['records_per_row'] > 0) {
         $modwidth = round(100 / $layout['records_per_row']);
      }
      for ($i = 0; $i < count($products); $i++) {   
 
         if ($i % $layout['records_per_row'] == 0) {
            if ($i > 0) {
               print '</tr>';
            }
            if ($i + 1 < count($products)) {
               print '<tr>';
            }
         }
 
         $pr = $products[$i];
         $args['product'] = $pr;
         $args['valign'] = $layout['cell_vertical_align'];
         $args['halign'] = $layout['cell_horizontal_align'];
         $args['id'] = $pr->getProductID() . '-' . $b->getBlockID();
         foreach($this->controller->getSets() as $key => $value) {
            $args[$key] = $value;
         }
         if ($args['imagePosition'] == 'T') {
            $valign = 'top';
         } else if ($args['imagePosition'] == 'B') {
            $valign = 'bottom';
         }
         print '<td valign="' . $valign . '" width="' . $modwidth . '%" style="border:' . (int) $layout['table_border_width'] . 'px ' . $layout['table_border_style'] . ' ' . $table_border_color . ';">';
         Loader::packageElement('product/display', 'core_commerce', $args);
         print '</td>';
      }
 
      if ($i % $layout['records_per_row'] != 0) {
         while ($i % $layout['records_per_row'] != 0) {
            print '<td>&nbsp;</td>';
            $i++;
         }
         print '</tr>';
      }
   ?>
   </table>
 
   <?php  
 
   } else { ?>
      <?php  echo  t('No products found'); ?>
   <?php   } ?>
<?php   }
 
if($paging['show_bottom'] && $paginator && strlen($paginator->getPages())>0){ ?>   
<div class="pagination">   
    <span class="pageLeft"><?php   echo $paginator->getPrevious(t('Previous'))?></span>
    <?php   echo $paginator->getPages()?>
    <span class="pageRight"><?php   echo $paginator->getNext(t('Next'))?></span>
</div>   
<?php   } ?>

Thank you

RandomBrad replied on May 26, 2011 at 2:06 pm Permalink Reply

Jeff, any ideas? I haven't been able to figure this one out yet. I appreciate the input.

MAKG replied on Jun 5, 2011 at 5:39 am Permalink Reply

In the ecommerce version prior 2.0.3 changing the product name was easy through inline css added to productname(in product_list / view.php). Looks like you've been discussing this version here.

Current update has many changes in the code and productname attribute seems to be replaced by a variable ($pr I think). I tried to change the css of ccm-core-commerce-product-list-results but it's not defined in any of the css files (?) I searched through the whole module for it.

Can anybody shed some light on how to control/change the product name css in product and product list blocks. Not a programmer/developer so sorry if i missed something obvious.

Mnkras replied on Jun 5, 2011 at 11:53 am Permalink Reply

open

core_commerce/elements/product/display/properties.php

and find
<h2><?=$link_before.$product->getProductName().$link_after?></h2>
thats the title
so
<h2 class="core-commerce-product-name"><?=$link_before.$product->getProductName().$link_after?></h2>

RandomBrad replied on Jun 8, 2011 at 3:31 pm Permalink Reply

Mnkras, I only have
core_commerce/elements/product/display.php
no properties.php file
I have checked and everything is updated.

MAKG replied on Jun 10, 2011 at 7:48 pm Permalink Reply

Thanks Mnkras but it didn't work. I replaced the code btw line 26 was this (with 'echo'):

<h2><?php echo $link_before.$product->getProductName().$link_after?></h2>

anyway, I tried different things, also removing htis line all together. No changes. Also changed other parts just to make sure I wasn't running it from cache. All other things were updating except the bloody heading...

jincmd replied on Jan 17, 2012 at 12:55 pm Permalink Reply

could you assist me in setting up the product tittle in H1?

chris123uk replied on Jul 18, 2013 at 9:13 am Permalink Reply

Mnkras. thanks that works

Forums

Editing (v7+)

eCommerce - H1 for product name on product page, not search results?

Code

Post Reply

Delete Post

Mark Post as Spam

Destroy Spammer

Sign In?