Storebox article title no longer visible

Commercial shopping Joomla template to easy create webshop with various extensions supported like ViruteMart, K2 and K2Store.
Rate this topic: Evaluations: 1, 1.00 on the average.Evaluations: 1, 1.00 on the average.Evaluations: 1, 1.00 on the average.Evaluations: 1, 1.00 on the average.Evaluations: 1, 1.00 on the average.Evaluations: 1, 1.00 on the average.1.00 out of 6 based on 1 vote(s)
GK User
Thu Jun 12, 2014 1:09 am
Reply with quote
Report this post
I applied the latest update to gk_storebopx which fixed a couple of issues in the category blog.
However, now all my single article pages have no title at all. The title shows up in in category view, but in article view it simply not there.

The code in the %template_name%/html/com_content/default.php file reads like this:

Code: Select all
<?php if ($params->get('show_title')) : ?>This is the title
      <h1 itemprop="name"
         <?php if ($params->get('link_titles') && !empty($this->item->readmore_link)) : ?>
            <a href="<?php echo $this->item->readmore_link; ?>" itemprop="url"
               <?php echo $this->escape($this->item->title); ?>
</a>
         <?php else : ?>
            <?php echo $this->escape($this->item->title); ?>
         <?php endif; ?>
      </h1>
      <?php endif; ?>


However, in the page the generated HTML looks like this:

Code: Select all
<header> <h1 h1="" <="" us="" about="" itemprop="name"> </h1></header>


Obviously, the actual title inside the H1 is missing, I don't even know what to make of this:
<h1 h1="" <="" us="" about="" itemprop="name">

In the older version I started out with (which works by the way) that snippet looks like this:

Code: Select all
<h1>
         <?php if ($params->get('link_titles') && !empty($this->item->readmore_link)) : ?>
            <a href="<?php echo $this->item->readmore_link; ?>">
               <?php echo $this->escape($this->item->title); ?>
            </a>
         <?php else : ?>
            <?php echo $this->escape($this->item->title); ?>
         <?php endif; ?>
      </h1>


It is obviously missing the various "itemprop" calls. But I was able to fix the whole thing by simply adding a > a the end of the H1 like this:
Code: Select all
<h1>  itemprop="name" 

instead of
Code: Select all
<h1  itemprop="name" 


Thank you
User avatar
Junior Boarder

GK User
Thu Jun 12, 2014 1:13 am
Reply with quote
Report this post
Sorry I messed up the examples above. The > gors like this and both the H1 and the <a> are wrong:

Original

Code: Select all
<header>
      <?php if ($params->get('show_title')) : ?>
      <h1 itemprop="name"
         <?php if ($params->get('link_titles') && !empty($this->item->readmore_link)) : ?>
            <a href="<?php echo $this->item->readmore_link; ?>" itemprop="url"
               <?php echo $this->escape($this->item->title); ?>
            </a>
         <?php else : ?>
            <?php echo $this->escape($this->item->title); ?>
         <?php endif; ?>
      </h1>


Fixed:
Code: Select all
<header>
      <?php if ($params->get('show_title')) : ?>
      <h1 itemprop="name">
         <?php if ($params->get('link_titles') && !empty($this->item->readmore_link)) : ?>
            <a href="<?php echo $this->item->readmore_link; ?>" itemprop="url">
               <?php echo $this->escape($this->item->title); ?>
            </a>
         <?php else : ?>
            <?php echo $this->escape($this->item->title); ?>
         <?php endif; ?>
      </h1>


basically at the end of
itemprop="name" and itemprop="url" the is a need to close the tag or all sort of mess happens
User avatar
Junior Boarder

GK User
Fri Jun 20, 2014 11:31 am
Reply with quote
Report this post
Same problem here: http://statief.nl/op-maat
An urgent solution would be appreciated.
User avatar
Fresh Boarder

GK User
Tue Jun 24, 2014 11:29 am
Reply with quote
Report this post
Is this post followed up? This should really only take minutes to solve...
User avatar
Fresh Boarder

GK User
Wed Jun 25, 2014 12:15 pm
Reply with quote
Report this post
Please try to replace this code :

Code: Select all
<?php if ($params->get('show_title')) : ?>This is the title
      <h1 itemprop="name"
         <?php if ($params->get('link_titles') && !empty($this->item->readmore_link)) : ?>
            <a href="<?php echo $this->item->readmore_link; ?>" itemprop="url"
               <?php echo $this->escape($this->item->title); ?>
</a>
         <?php else : ?>
            <?php echo $this->escape($this->item->title); ?>
         <?php endif; ?>
      </h1>
      <?php endif; ?>



with this one :

Code: Select all
<?php if ($params->get('show_title')) : ?>This is the title
      <h1 itemprop="name">
         <?php if ($params->get('link_titles') && !empty($this->item->readmore_link)) : ?>
            <a href="<?php echo $this->item->readmore_link; ?>" itemprop="url"
               <?php echo $this->escape($this->item->title); ?>
</a>
         <?php else : ?>
            <?php echo $this->escape($this->item->title); ?>
         <?php endif; ?>
      </h1>
      <?php endif; ?>


This should resolve your issue.
User avatar
Platinum Boarder

GK User
Thu Jun 26, 2014 7:46 am
Reply with quote
Report this post
OK, Will do. I guess you will also solve this in the template for future template updates?
User avatar
Fresh Boarder

GK User
Fri Jun 27, 2014 11:46 am
Reply with quote
Report this post
You code supplied contains an error. This line lacks a closing tag for the <a> element:

<a href="<?php echo $this->item->readmore_link; ?>" itemprop="url"

It should be:

<a href="<?php echo $this->item->readmore_link; ?>" itemprop="url" >

Please also confirm if this will be part of any furure updates for the template.
User avatar
Fresh Boarder

GK User
Sat Jun 28, 2014 11:12 am
Reply with quote
Report this post
Did this change finally resolve this issue? If yes, I'll add a fix to template.
User avatar
Platinum Boarder

GK User
Mon Jun 30, 2014 8:27 am
Reply with quote
Report this post
Including the small correction I mentioned to close the <a> tag, it dit, so please include it in the template.
User avatar
Fresh Boarder

GK User
Thu Jul 03, 2014 9:10 pm
Reply with quote
Report this post
Thanks surfsimon. I lost 2 hours on it and than I found your solution. Thanks :)

bkrztuk - it's good to update the source of that template.
User avatar
Fresh Boarder

GK User
Fri Jul 04, 2014 1:55 pm
Reply with quote
Report this post
I've already fixed this and updated package will not have this issue.
User avatar
Platinum Boarder


cron