Highlight Current Page Using PHP

[andy.richin]

I found some code that highlights a nav link for each page that you are on this developer site http://www.elanman.co.uk/2009/01/hig...-php-includes/

I am trying to manipulate a few things so that my "Work" nav stays highlighted for multiple pages not just work.php. Here is the code I have so far from this tutorial

Original:

<?php
if (strpos($_SERVER['PHP_SELF'], 'about.php')) echo 'class="current"';
?>

<ul id="navigation">
    <li><a <?php if (strpos($_SERVER['PHP_SELF'], 'index.php')) echo 'class="current"';?> href="index.php">Home Page</a></li>
    <li><a <?php if (strpos($_SERVER['PHP_SELF'], 'about.php')) echo 'class="current"';?> href="about.php">About Me</a></li>
    <li><a <?php if (strpos($_SERVER['PHP_SELF'], 'contact.php')) echo 'class="current"';?> href="contact.php">Contact Me</a></li>
    <li><a <?php if (strpos($_SERVER['PHP_SELF'], 'work.php')) echo 'class="current"';?> href="work.php">Work</a></li>
</ul>

This is the code found in the same tutorial that explains how to supposedly do what I am trying to do.

$aboutPages = array('page1.php', 'page2.php', 'page3.php', 'page4.php');

<?php
foreach ($aboutPages as $page) {
 if (strpos($_SERVER['PHP_SELF'], $page)) echo 'class="current"';
}
?>

In the HTML code above, I am not sure what to insert in the php code in the <a> tag to identify the page. I tried $page but it didn't work. Any thoughts or help?

[Natdrip]

I am not proof positive about what you are trying to do. so this is what I think you are trying to do.

<ul id="navigation">
<?php 

$aboutPages = array('page1.php', 'page2.php', 'page3.php', 'page4.php'); 

foreach ($aboutPages as $page) { 
  echo <li>;
  echo '<a href=\''.$page.'\' ';
  (strpos($_SERVER['PHP_SELF'], $page))? echo 'class="current"':''; 
  echo ' > '.$page.'</a>';
  echo </li>;
} 
?>
</ul>

I didn't test but you can try the code above.

[andy.richin]

So you are saying put it all in the unordered list tag? When I tried that I get errors on "echo <li>;" "(strpos($_SERVER..." and the second "echo </li>;"

All I'm really trying to do is have my Work link in my main nav stay highlighted for multiple pages, not just work.php (such as work_print.php, work_web.php, etc). Hopefully that explains it better.

[Natdrip]

is the nav an include?
or is it hard coded on very page?

if it is hard coded then just add the class='current'

and build out the css to reflect that.

if it is included then you'll need to strip the url

    function ShowFileName($filepath) 
    { 
        preg_match('/[^?]*/', $filepath, $matches); 
        $string = $matches[0]; 
        #split the string by the literal dot in the filename 
        $pattern = preg_split('/\./', $string, -1, PREG_SPLIT_OFFSET_CAPTURE); 
        #get the last dot position 
        $lastdot = $pattern[count($pattern)-1][1]; 
        #now extract the filename using the basename function 
        $filename = basename(substr($string, 0, $lastdot-1)); 
        #return the filename part 
        return $filename; 
    } 

then call it

        $str = $_SERVER['SCRIPT_NAME'];
        ShowFileName($str); 

This should give you the file name

Then run it through the build array with a conditional that determines which page you should high light.

kind of what was in the first reply you may have to tweek it.

[criterion9]

(strpos($_SERVER['PHP_SELF'], $page))? echo 'class="current"':'';

probably meant

(strpos($_SERVER['PHP_SELF'], $page))? echo ' class="current"':; 

or

(strpos($_SERVER['PHP_SELF'], $page))? echo ' class="current"':echo ''; 

[andy.richin]

Yes that nav is in an include header.php file. Unfortunately I am not good at writing PHP and I am still not sure what you mean. Any example / tutorial you can point me to?

[andy.richin]

Because unfortunately I need it spelled out for me otherwise I will not figure it out!

[criterion9]

At first glance the code NatDrip initially posted should work, just replace that line that had bad syntax like I suggested.

[andy.richin]

So this? Inside of the <ul> tag? And I added your correction as well.

<ul id="navigation">
<?php 

$aboutPages = array('page1.php', 'page2.php', 'page3.php', 'page4.php'); 

foreach ($aboutPages as $page) { 
  echo <li>;
  echo '<a href=\''.$page.'\' ';
(strpos($_SERVER['PHP_SELF'], $page))? echo ' class="current"':;   echo ' > '.$page.'</a>';
  echo </li>;
} 
?>
</ul>

Do I need anything else?

[criterion9]

Did it work? I don't have an environment to test your code in...nor do I have all your code.

[andy.richin]

I get a syntax error on lines 7, 9 and 10

[andy.richin]

This is my original code that I was using. It works for each individual page, but not multiple pages with the same nav element.

<body onload="displayCD()">

<?php

if (strpos($_SERVER['PHP_SELF'], 'about.php'));
if (strpos($_SERVER['PHP_SELF'], 'index.php'));
if (strpos($_SERVER['PHP_SELF'], 'work_all.php'));

        <div id="main_nav">
        	<ul>
            	<li><a <?php if (strpos($_SERVER['PHP_SELF'], 'index.php')) echo 'class="active"';?> href="index.php">HOME</a></li>
                <li><a <?php if (strpos($_SERVER['PHP_SELF'], 'work_all.php')) echo 'class="active"';?> href="work_all.php">WORK</a></li>
                <li><a <?php if (strpos($_SERVER['PHP_SELF'], 'about.php')) echo 'class="active"';?> href="about.php">ABOUT</a></li>
            </ul>
        </div>

Now this is what was recommended but again, there are syntax errors on "echo <li>;", (strpos($_SERVER.....", and "echo </li>;".

<ul id="navigation"> 
<?php 

$aboutPages = array('page1.php', 'page2.php', 'page3.php', 'page4.php'); 

foreach ($aboutPages as $page) { 
**echo <li>; 
**echo '<a href=\''.$page.'\' '; 
(strpos($_SERVER['PHP_SELF'], $page))? echo ' class="current"':;***echo ' > '.$page.'</a>'; 
**echo </li>; 
} 
?> 
</ul>

Thoughts?

[criterion9]

Now this is what was recommended but again, there are syntax errors on "echo <li>;", (strpos($_SERVER.....", and "echo </li>;".

<ul id="navigation"> 
<?php 

$aboutPages = array('page1.php', 'page2.php', 'page3.php', 'page4.php'); 

foreach ($aboutPages as $page) { 
**echo <li>; 
**echo '<a href=\''.$page.'\' '; 
(strpos($_SERVER['PHP_SELF'], $page))? echo ' class="current"':;***echo ' > '.$page.'</a>'; 
**echo </li>; 
} 
?> 
</ul>

Thoughts?

Looks like you are missing quotes on your echo. Do you see how the syntax highlighting looks strange? Also I'm assuming your *** are marking the lines with errors and are not actually included.

echo "<li>"; 

[andy.richin]

That didn't do anything. I attached a screen shot of my code and where the errors are.

[criterion9]

<ul>
<?php
$aboutPages = array('page1.php', 'page2.php', 'page3.php', 'page4.php'); 

foreach ($aboutPages as $page) { 
   echo "<li>"; 
   echo "<a href=\''$page\""; 
   echo (strpos($_SERVER['PHP_SELF'], $page) > 0) ? " class=\"current\"": '';
   echo ">$page</a>"; 
   echo "</li>"; 
} 
?>
</ul>

Try that one.