There are a double if on node.tpl.php code

  <?php if (!$page): ?>
    <?php if (!$page): ?>
      <h2<?php print $title_attributes; ?>>
        <a href="<?php print $node_url; ?>"><?php print $title; ?></a></h2>
    <?php endif; ?>
  <?php endif; ?>

This not important issue, but is not correct syntax. It's a minor issue.

Comments

eloivaque’s picture

Alka Kumari’s picture

Attached is the patch with correct naming convention.

Alka Kumari’s picture

Status: Active » Needs review
urvigala’s picture

Assigned: Unassigned » urvigala
urvigala’s picture

Status: Needs review » Reviewed & tested by the community
Alka Kumari’s picture

Can this be committed?

hongpong’s picture

StatusFileSize
new573 bytes

Attached is a patch against 7.x-4.x . The line break now matches 7.x-5.x . I think the 7.x-5.x patch is good to go.

  • HongPong committed ef36ae0 on 7.x-5.x
    Issue #2500909 by Alka Kumari, eloiv, HongPong, urvigala: variable page...

  • HongPong committed d79dfb5 on 7.x-4.x
    Issue #2500909 by Alka Kumari, eloiv, HongPong, urvigala: variable page...
hongpong’s picture

Status: Reviewed & tested by the community » Fixed

Is now fixed on 7.x-5.x and 7.x-4.x . Thank you!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.