Titles are displayed twice in front page or teaser view. [#671858]

Hello,

The node's title gets to display twice, one is with

tag and hyper-link while 2nd one is the
without the hyper-link title.
After some digging into the codes of node.tpl.php in the template folder, I found out that it could be a missing if else statement.

Here is what I do:
Change these lines:

<h2 class="title"><a href="<?php print $node_url ?>" title="<?php print $title ?>"><?php print $title ?></a></h2>
<?php endif; ?>

<h1 class="title"><?php print $title; ?></h1>

To:

<?php if ($page == 0): ?>
<h2 class="title"><a href="<?php print $node_url ?>" title="<?php print $title ?>"><?php print $title ?></a></h2>
<?php else: ?>
<h1 class="title"><?php print $title; ?></h1>
<?php endif; ?>

Then the
title field will disappear from the front page view and the teaser view while the link to the node will stay.
Don't know if I am doing it correctly though.

Leon

Comment	File	Size	Author
#2	missingelse.patch	542 bytes	visum

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

leon85321 CreditAttribution: leon85321 commented 31 December 2009 at 22:16

errr I forgot to remove the < and > from the h1 and h2 in the above post... should have click preview first lol

The node's title gets to display twice, one is with h2 tag and hyper-link while 2nd one is the h1 tag without the hyper-link title.
After some digging into the codes of node.tpl.php in the template folder, I found out that it could be a missing if else statement.

Here is what I do:
Change these lines:

<h2 class="title"><a href="<?php print $node_url ?>" title="<?php print $title ?>"><?php print $title ?></a></h2>
<?php endif; ?>

<h1 class="title"><?php print $title; ?></h1>

To:

<?php if ($page == 0): ?>
<h2 class="title"><a href="<?php print $node_url ?>" title="<?php print $title ?>"><?php print $title ?></a></h2>
<?php else: ?>
<h1 class="title"><?php print $title; ?></h1>
<?php endif; ?>

Then the h1 tagged title field will disappear from the front page view and the teaser view while the links with h2 tag to the node will stay.
Don't know if I am doing it correctly though.

Leon

Comment #2

visum CreditAttribution: visum commented 26 February 2010 at 23:11

File	Size
missingelse.patch	542 bytes

I came to the same conclusion. Here's a patch.

Comment #3

koldtoft CreditAttribution: koldtoft commented 3 May 2010 at 15:24

I removed the 2nd title using the code from leon85321 example, but now I have a high empty space between the clickable title and the body content.

Any hints on how to remove this space and place the title, right above the content?

Comment #4

Skelly1983 CreditAttribution: Skelly1983 commented 17 May 2010 at 18:36

i removed the h1 tag in the teaser and its txt by using:

<?php $str = preg_replace('#(<h1.*?>).*?(</h1>)#', '$1$2', $rows); echo strip_tags($str, '<a> <div> <p> <h2> <h3> <?php> <span> <img> <strong> <em> <table> <td> <tr> <th> <tbody> <center> <ul> <li> <ol> <font> <blockquote> <br> <fieldset> <input> <textarea> <tfoot> <thead> <button> <form> <h4> <h5> <h6> <label> <pre> <select> <sub> <sup> <tt>'); ?>

the $rows string is for views teasers, the 2nd part of the strip_tags is all the tags i want to keep in my teasers just incase i use them all other tags will be taken out so if u dont want some of them just remove them from the code and they will be striped out aswell.

i know there is prob an easier or better way to do this but works perfectly fine with no side efects. i placed this in my custom theme's views-view--NODENAME.tlp.php file which i had created and just replaced:

print $rows;

i use artisteer aswell, and there is no reuirment to add this extra code to template.php