PHP Build a Basic PHP Website Building a Media Library in PHP Adding Active States to the Navigation

Martina Carrington
Martina Carrington
15,718 Points

I revived error message on header.php

I received a error message on header.php about line 16 . i carefully check the code but i don't see anything wrong <html> <head> <title><?php echo $pageTitle;?></title> <link rel="stylesheet" href="css/style.css" type="text/css"> </head> <body>

<div class="header">

    <div class="wrapper">

        <h1 class="branding-title"><a href="./">Personal Media Library</a></h1>

        <ul class="nav">
            <li class="books <?php if ($section == "book") {echo " on"; } ?>"><a href="catalog.php?cat=books">Books</a></li>
            <li class="movies<?php if ($section == "movies") {echo " on";} ?"><a href="catalog.php?cat=movies">Movies</a></li>
            <li class="music<?php if ($section == "music") {echo " on";} ?"><a href="catalog.php?cat=music">Music</a></li>
            <li class="suggest<?php if ($section == "suggest") {echo " on";} ?"><a href="suggest.php">Suggest</a></li>
        </ul>

    </div>

</div>

<div id="content">

2 Answers

Kai Nightmode
Kai Nightmode
36,240 Points

Ah, I think all that is missing are some closing ?> tags.

Here is updated code with closing PHP tags in the right places. It has some extra spaces throw in to make comparing each line a bit easier.

<title><?php echo $pageTitle;?></title>
<link rel="stylesheet" href="css/style.css" type="text/css">



<div class="header">

    <div class="wrapper">

        <h1 class="branding-title"><a href="./">Personal Media Library</a></h1>

        <ul class="nav">
            <li class="books<?php   if ($section == "book")    {echo " on";} ?>"><a href="catalog.php?cat=books">Books</a></li>
            <li class="movies<?php  if ($section == "movies")  {echo " on";} ?>"><a href="catalog.php?cat=movies">Movies</a></li>
            <li class="music<?php   if ($section == "music")   {echo " on";} ?>"><a href="catalog.php?cat=music">Music</a></li>
            <li class="suggest<?php if ($section == "suggest") {echo " on";} ?>"><a href="suggest.php">Suggest</a></li>
        </ul>

    </div>

</div>

<div id="content">

I have not actually run the above code on a PHP server so you may need to test / tweak a bit more.

Cheers!

Nick Dim
Nick Dim
12,885 Points

In the menu, the list items are wrong. The first is correct but the rest have mistakes.