Trying to move an advert but just get syntax errors everytime I try!

[nickc25]

Hi

I am trying to move an advert to the top of the page, but everytime I try and move it I always get some sort of error.

It is usually like this:

Parse error: syntax error, unexpected $end in /nfs/c08/h02/mnt/116350/domains/dev.anfieldindex.com/html/wp-content/themes/anfieldindex/templates/authors.php on line 110

This is the original file:

<?php
/**
* Displays a grid of authors.
* 
* Called by accessing /authors, which is a custom rewrite rule in functions.php
* 
* @package Wordpress
* @subpackage Anfield Index
* @author Tim Carr
* @version 1
* @copyright n7 Studios
*/

global $paywallPro;
?>
<section id="authors">
    <?php
    // Get authors, paginated
    $authors = get_users_of_blog(); // Note: will need changing for 3.1 to work
    foreach ($authors as $key=>$author) {
        if (count_user_posts($author->user_id) > 0) $authorIDs[$key] = $author->user_id;
    }
    rsort($authorIDs); // Sort IDs highest to lowest i.e. newest to oldest

$totalAuthors = count($authorIDs);
$paged = get_query_var('paged');
$authorsPerPage = ((get_option('posts_per_page') % 2) > 0) ? get_option('posts_per_page') + 1 : get_option('posts_per_page'); // Ensure it is an even number for the grid
$startKey = (($paged == 0) ? 0 : ($paged-1)*$authorsPerPage);
$endKey = ($startKey + $authorsPerPage);
$count = 0;

for ($i = $startKey; $i < $endKey; $i++) {
    if ($authorIDs[$i] == '') continue;
    $user = get_userdata($authorIDs[$i]);
    $userposts = get_posts('author='.$authorIDs[$i]);
    $totalPosts = count($userposts);
    $count++;

    // Advert    
    if ($count == 3) {
        if (!$paywallPro->user->acl['removeAds']) { 
            ?>
            <div class="advert">
                <script type="text/javascript"><!--
google_ad_client = "pub-6230056227145824";
/* AnfieldIndex.com Main Site */
google_ad_slot = "3598117926";
google_ad_width = 468;
google_ad_height = 60;
//-->
</script>
<script type="text/javascript"
src="http://pagead2.googlesyndication.com/pagead/show_ads.js">
</script>
                </div>
                <div class="advert-notice"><a href="/register">Subscribe Now to remove these adverts.</a></div>
                <?php    

            }
        }
        ?>
        <article class="<?php echo (($i % 2) > 0 ? 'grid end' : 'grid'); ?>">
            <header>
                <h1><a href="<?php bloginfo('url'); ?>/author/<?php echo $user->user_login; ?>" title="View Posts by <?php echo $user->display_name; ?>"><?php echo $user->display_name; ?></a></h1>
                <ul>
                    <li><time datetime="<?php echo date('Y-m-d', strtotime($user->user_registered)).'T'.date('H:i', strtotime($user->user_registered)); ?>" pubdate>Joined <?php echo date('d/m/Y', strtotime($user->user_registered)); ?></time></li>
                    <li class="comments"><?php echo $totalPosts; ?> Posts</li>
                </ul>
            </header> 
            <div class="content">
                <p>
                    <?php echo get_avatar($user->user_email, '40'); ?>
                    <?php echo ($user->description != '' ? $user->description : 'This user hasn\'t added a description about themselves yet.'); ?>
                </p>
                <p><a href="<?php bloginfo('url'); ?>/author/<?php echo $user->user_login; ?>" title="View Posts by <?php echo $user->display_name; ?>" class="button">Read Blog &raquo;</a></p>
            </div>
        </article>
        <?php
    }
    wp_reset_query();
    ?>

<!-- Nav: Pagination --> 
<nav id="pagination">
    <ul>
        <?php if ($startKey > 0) { ?><li id="newer"><?php previous_posts_link('&laquo; Newer Entries') ?></li><?php } ?>
        <?php if ($totalAuthors > $endKey) { ?><li id="older"><?php next_posts_link('Older Entries &raquo;') ?></li><?php } ?>
    </ul>
</nav>
</section>

This is the advert I am trying to move.

// Advert

if ($count == 3) {
if (!$paywallPro->user->acl['removeAds']) {
?>
<div class="advert">
<script type="text/javascript"><!--
google_ad_client = "pub-6230056227145824";
/ AnfieldIndex.com Main Site /
google_ad_slot = "3598117926";
google_ad_width = 468;
google_ad_height = 60;
//-->
</script>
<script type="text/javascript"
src="http://pagead2.googlesyndication.com/pagead/show_ads.js">
</script>
</div>
<div class="advert-notice"><a href="/register">Subscribe Now to remove these adverts.</a></div>

Any ideas? Thanks

[johanafm]

The reason for indenting code is to easily spot what blocks of code go together, even though it is decided by actual code, not by indentation. That's just a visual aid when reading the code.

<?php
01        // Advert    

02       if ($count == 3) {
03            if (!$paywallPro->user->acl['removeAds']) { 
04                ?>
05                <div class="advert">
06                    <script type="text/javascript"><!-- google_ad_client = "pub-6230056227145824"; /* AnfieldIndex.com Main Site */ google_ad_slot = "3598117926"; google_ad_width = 468; google_ad_height = 60; //-->
07</script>
08<script type="text/javascript" src="http://pagead2.googlesyndication.com/pagead/show_ads.js"></script>
09                </div>
10                <div class="advert-notice"><a href="/register">Subscribe Now to remove these adverts.</a></div>
11                <?php    

12            }
13        }
 

And as you can see, the if-statement and its opening bracket on line 3 line up perfectly with the closing bracket on line 12. Everything between these brackets belong to the same code execution block (runs when the if statement holds true).

In the same way, the closing bracket on line 13 belongs together with the opening bracket on line 2. An opening bracket always requires a following closing bracket. A closing bracket always requires a preceeding opening bracket. You can't have one without the other, and you can never move one bracket someplace else in the code without affecting what happens (shuch as excluding some lines from being executed or including more lines than before).
Thus, if you want the code to execute at another point in your script, move the entire code block.

[nickc25]

johanafm;10977233 wrote:

The reason for indenting code is to easily spot what blocks of code go together, even though it is decided by actual code, not by indentation. That's just a visual aid when reading the code.

<?php
01        // Advert    

02       if ($count == 3) {
03            if (!$paywallPro->user->acl['removeAds']) { 
04                ?>
05                <div class="advert">
06                    <script type="text/javascript"><!-- google_ad_client = "pub-6230056227145824"; /* AnfieldIndex.com Main Site */ google_ad_slot = "3598117926"; google_ad_width = 468; google_ad_height = 60; //-->
07</script>
08<script type="text/javascript" src="http://pagead2.googlesyndication.com/pagead/show_ads.js"></script>
09                </div>
10                <div class="advert-notice"><a href="/register">Subscribe Now to remove these adverts.</a></div>
11                <?php    

12            }
13        }
 

And as you can see, the if-statement and its opening bracket on line 3 line up perfectly with the closing bracket on line 12. Everything between these brackets belong to the same code execution block (runs when the if statement holds true).

In the same way, the closing bracket on line 13 belongs together with the opening bracket on line 2. An opening bracket always requires a following closing bracket. A closing bracket always requires a preceeding opening bracket. You can't have one without the other, and you can never move one bracket someplace else in the code without affecting what happens (shuch as excluding some lines from being executed or including more lines than before).
Thus, if you want the code to execute at another point in your script, move the entire code block.

So the above code in your post if I move that to the top then it should work? In the end, I actually moved that code to the top and got no errors, but got no advert nor a White space where the advert should be. Maybe it is a wordpress issue? I'm used to using vbulletin than wp. Ideally I would like to put it at the bottom of the header.php so it would then appear below the menu on all pages, that way I would not have to edit several pages.

The code is on dev server but you can see it in action at anfieldindex.com. At the moment it is a smaller ad banner slightly down the page, wanted a bigger one across the menu but centred.

On wordpress, why are there so many <?php tags? I'm sure on vb there is just one at the top and one at bottom? Also on this above code, does there not need to be one final ?< as there is a <?php and two brackets at the bottom? Or just copy that entire code you pasted? Thanks for the advice!

[nickc25]

I moved the code to below global $paywallPro; because I assume that it needs to be below that to control if paying users see the adverts or not.

I can put the google code in fine no problems without having to put the paywall pro code in but then paying users see adverts which is what we don't want..

I have put the code there and get no errors, I did this yesterday and got the same result as today - no errors but no advert either, not even a blank space where the advert should be.

This is the new code:

<?php
/**
* Displays a grid of authors.
* 
* Called by accessing /authors, which is a custom rewrite rule in functions.php
* 
* @package Wordpress
* @subpackage Anfield Index
* @author Tim Carr
* @version 1
* @copyright n7 Studios
*/

global $paywallPro;

    // Advert    
    if ($count == 3) {
        if (!$paywallPro->user->acl['removeAds']) { 
            ?>
            <div class="advert">
                <script type="text/javascript"><!--
google_ad_client = "pub-6230056227145824";
/* AnfieldIndex.com Main Site */
google_ad_slot = "3598117926";
google_ad_width = 468;
google_ad_height = 60;
//-->
</script>
<script type="text/javascript"
src="http://pagead2.googlesyndication.com/pagead/show_ads.js">
</script>
                </div>
                <div class="advert-notice"><a href="/register">Subscribe Now to remove these adverts.</a></div>
                <?php    

            }
        }




?>
<section id="authors">
    <?php
    // Get authors, paginatedhead
    $authors = get_users_of_blog(); // Note: will need changing for 3.1 to work
    foreach ($authors as $key=>$author) {
        if (count_user_posts($author->user_id) > 0) $authorIDs[$key] = $author->user_id;
    }
    rsort($authorIDs); // Sort IDs highest to lowest i.e. newest to oldest

$totalAuthors = count($authorIDs);
$paged = get_query_var('paged');
$authorsPerPage = ((get_option('posts_per_page') % 2) > 0) ? get_option('posts_per_page') + 1 : get_option('posts_per_page'); // Ensure it is an even number for the grid
$startKey = (($paged == 0) ? 0 : ($paged-1)*$authorsPerPage);
$endKey = ($startKey + $authorsPerPage);
$count = 0;

for ($i = $startKey; $i < $endKey; $i++) {
    if ($authorIDs[$i] == '') continue;
    $user = get_userdata($authorIDs[$i]);
    $userposts = get_posts('author='.$authorIDs[$i]);
    $totalPosts = count($userposts);
    $count++;

?>
        <article class="<?php echo (($i % 2) > 0 ? 'grid end' : 'grid'); ?>">
            <header>
                <h1><a href="<?php bloginfo('url'); ?>/author/<?php echo $user->user_login; ?>" title="View Posts by <?php echo $user->display_name; ?>"><?php echo $user->display_name; ?></a></h1>
                <ul>
                    <li><time datetime="<?php echo date('Y-m-d', strtotime($user->user_registered)).'T'.date('H:i', strtotime($user->user_registered)); ?>" pubdate>Joined <?php echo date('d/m/Y', strtotime($user->user_registered)); ?></time></li>
                    <li class="comments"><?php echo $totalPosts; ?> Posts</li>
                </ul>
            </header> 
            <div class="content">
                <p>
                    <?php echo get_avatar($user->user_email, '40'); ?>
                    <?php echo ($user->description != '' ? $user->description : 'This user hasn\'t added a description about themselves yet.'); ?>
                </p>
                <p><a href="<?php bloginfo('url'); ?>/author/<?php echo $user->user_login; ?>" title="View Posts by <?php echo $user->display_name; ?>" class="button">Read Blog &raquo;</a></p>
            </div>
        </article>
        <?php
    }
    wp_reset_query();
    ?>

<!-- Nav: Pagination --> 
<nav id="pagination">
    <ul>
        <?php if ($startKey > 0) { ?><li id="newer"><?php previous_posts_link('&laquo; Newer Entries') ?></li><?php } ?>
        <?php if ($totalAuthors > $endKey) { ?><li id="older"><?php next_posts_link('Older Entries &raquo;') ?></li><?php } ?>
    </ul>
</nav>
</section>

continued....

[nickc25]

Continued as it would not allow me to post it all in one post...

As mentioned in the previous post, Ideally I would like the advert code to go at the bottom of the header, thus below the menu.

This is the header code:

<?php

/**

* Site wide header

* 

* @package Wordpress

* @subpackage Anfield Index

* @author Tim Carr

* @version 1

* @copyright n7 Studios

*/



ob_start();

?>

<!DOCTYPE html>

<html lang="en"> 

<head>    

<!-- Metadata -->

<meta charset="utf-8" />

<meta name="viewport" content="width=device-width; initial-scale=1.0; maximum-scale=1.0;">

<meta name="google-site-verification" content="" />



<!-- Title -->

<title><?php wp_title(''); ?> - <?php bloginfo('name'); ?></title>





<!--[if lt IE 9]>

<script src="http://html5shiv.googlecode.com/svn/trunk/html5.js"></script>

<meta http-equiv="X-UA-Compatible" content="IE=edge,chrome=1">

<![endif]-->

<script src="<?php bloginfo('template_url'); ?>/js/modernizr-1.6.min.js"></script> 



<!-- Stylesheets -->

<link rel="stylesheet" type="text/css" href="<?php bloginfo('template_url'); ?>/css/reset.css" />

 <link rel="stylesheet" type="text/css" href="<?php bloginfo('template_url'); ?>/css/screen.css" />

<!--[if lt IE 9]><link rel="stylesheet" type="text/css" href="<?php bloginfo('template_url'); ?>/css/ie.css" media="screen" /><![endif]-->





<!-- Icons -->

<link rel="shortcut icon" href="/favicon.ico">

<link rel="apple-touch-icon" href="<?php bloginfo('template_url'); ?>/images/apple/icon-iphone.png" />

<link rel="apple-touch-icon" sizes="72x72" href="<?php bloginfo('template_url'); ?>/images/apple/icon-ipad.png" />

<link rel="apple-touch-icon" sizes="114x114" href="<?php bloginfo('template_url'); ?>/images/apple/icon-iphone4.png" />



<!-- RSS -->

<link rel="alternate" type="application/rss+xml" title="<?php bloginfo('name'); ?> RSS Feed" href="<?php bloginfo('rss2_url'); ?>" />

<link rel="alternate" type="application/atom+xml" title="<?php bloginfo('name'); ?> Atom Feed" href="<?php bloginfo('atom_url'); ?>" />

<link rel="pingback" href="<?php bloginfo('pingback_url'); ?>" />



<?php 

wp_head();

global $paywallPro;

?>        

</head>



<body>

<div class="container">

    <div id="outline">

        <header id="header">

            <h1><a href="/" title="Home"><?php bloginfo('name'); ?></a></h1>       

        </header>



        <nav id="main-menu">

            <?php 

            echo preg_replace(array(  

                '/\t/', // remove tabs  

                '/'.str_replace('//','\/\/', get_bloginfo('url')).'/i', // remove full URL  

                '/current_page_item\s*/i',  

                '/current_page_ancestor\s*/i',  

                '/current_page_parent\s*/i',  

                '/page_item\s+/i',  

                '/page-item-\d+\s*/i',  

                '/children\s*/i',  

                '/\s*class=["\']["\']/i', // empty classes  

                '/\s*title="[^"]+"/i', // all titles  

                '/\s+>/i',  

                '/div>/i' // change div to nav  

              ),  

              array(  

                '',  

                '',  

                'active',  

                'open',  

                '',  

                '',  

                '',  

                '',  

                '',  

                '',  

                '>',  

                'nav>'  

              ),  

              wp_nav_menu(array('container' => '', 'theme_location' => header, 'menu_class' => '', 'echo' => false))  

            );

            ?>

        </nav> 



        <header id="strapline">

            <ul id="ticker">

                <?php

                $opts = get_option('ai');

                if (is_array($opts['ticker'])) {

                    foreach ($opts['ticker'] as $key=>$ticker) {

                        if ($ticker['label'] == '' OR $ticker['heading'] == '') continue;

                        ?>

                        <li>

                            <?php if ($ticker['url'] != '') { ?><a href="<?php echo $ticker['url']; ?>" title="Continue Reading <?php echo $ticker['heading']; ?>"><?php } ?>

                            <span style="background-color: #<?php echo $ticker['labelColour']; ?>"><?php echo $ticker['label']; ?></span>

                            <?php echo $ticker['heading']; ?>

                            <?php if ($ticker['url'] != '') { ?></a><?php } ?>

                        </li> 

                        <?php

                    }

                }

                ?>

            </ul>

            <ul id="account-actions">

                <?php

                if (is_user_logged_in()) {

                    ?>

                    <li class="user">Logged in as <?php echo $paywallPro->user->user_login; ?></li>

                    <li class="button"><a href="/account" title="My Account">My Account</a></li>

                    <li class="button"><a href="<?php echo wp_logout_url(get_bloginfo('url')); ?>" title="Logout">Logout</a></li>

                    <?php

                } else {

                    ?>

                    <li class="button"><a href="/register" title="Register">Register</a></li>

                    <li class="button"><a href="/login" title="Sign In">Sign In</a></li>

                    <?php

                }    

                ?>

            </ul>



        </header>



        <div id="main">

Would I just put the advert code below the </header> tag at the bottom?

So it would look like this:

</header>
 // Advert    

        if ($count == 3) {
            if (!$paywallPro->user->acl['removeAds']) { 
                ?>
                <div class="advert">
                    <script type="text/javascript"><!--
google_ad_client = "pub-6230056227145824";
/* AnfieldIndex.com Main Site */
google_ad_slot = "3598117926";
google_ad_width = 468;
google_ad_height = 60;
//-->
</script>
<script type="text/javascript"
src="http://pagead2.googlesyndication.com/pagead/show_ads.js">
</script>
                </div>
                <div class="advert-notice"><a href="/register">Subscribe Now to remove these adverts.</a></div>
                <?php    

            }
        }
?>
<div id="main">

Because everytime I do I get Parse error: syntax error, unexpected '}' in /nfs/c08/h02/mnt/116350/domains/dev.anfieldindex.com/html/wp-content/themes/anfieldindex/header.php on line 286

So I remove the final both the } and get the advert appearing below the header but with this above the advert:
// Advert if ($count == 3) { if (!$paywallPro->user->acl['removeAds']) { ?>

The advert then displays when logged in or not. So there is one thing I have missed out and it is probably pretty simple but do not know if it is to do with the global $paywallPro; tag thing not being directly above the code or not.

Thanks for the help on this. Been annoying me for a while!!

[nickc25]

Ok so I realise I did not put the <?php below the header end tag like this </header>
<?php

But when I do I get a further error

Parse error: syntax error, unexpected $end in /nfs/c08/h02/mnt/116350/domains/dev.anfieldindex.com/html/wp-content/themes/anfieldindex/header.php on line 291

Line 291 has nothing on it. Sorry for posting a few in the row, my posts are being moderated and it is taking a day to get them moderated for some reason, it is my off today (althogh i will probably be working a bit in the evening). I was asked to setup the forum on this site which I have done but then asked to do a further few things so trying to do that but more comfortable with vbulletin than wordpress. Thanks again.

[johanafm]

No idea, about your syntax error since I can't guess what your code looks like at the moment.

However, if ($count == 3) checks if 3 posts have been displayed, and if so, the though was to show the ad. You have now moved the code block someplace where $count most likely isn't defined (which should give an E_WARNING or E_NOTICE iirc - in your dev environment, turn on ALL errors, including notice and strict). And if I understand you correctly, you want to unconditionally show the ad... which means losing the if check.

[johanafm]

Acutally, the syntax error is most likely because you moved a php opening or closing tag when you moved the code block, which means you'd have to add it back in where you originally cut the code away.

[nickc25]

Hi
Basically I want the advert to show to guests / not logged in users. Logged in users do not see ads. This is the case at the moment, but I just want to move the advert to below the header / menu so that it is at the top of every page and not after the first 3 posts. Thanks

So this is why when I was correctly moving the code, I was getting no errors but no advert either, because there was not 3 posts above it. I removed the if statement and the ads are now showing on the authors page. But still having a struggle getting it at the bottom of the header!

[nickc25]

I have actually got this code working in this section. But now I think it would be better if it was in the header, that way I would not have to edit several files on the site.

This is the original header code:

[nickc25]

nickc25;10977323 wrote:

I have actually got this code working in this section. But now I think it would be better if it was in the header, that way I would not have to edit several files on the site.

This is the original header code:

Not sure why that posted!

Code

<?php

/**

* Site wide header

* 

* @package Wordpress

* @subpackage Anfield Index

* @author Tim Carr

* @version 1

* @copyright n7 Studios

*/



ob_start();

?>

<!DOCTYPE html>

<html lang="en"> 

<head>    

<!-- Metadata -->

<meta charset="utf-8" />

<meta name="viewport" content="width=device-width; initial-scale=1.0; maximum-scale=1.0;">

<meta name="google-site-verification" content="" />



<!-- Title -->

<title><?php wp_title(''); ?> - <?php bloginfo('name'); ?></title>





<!--[if lt IE 9]>

<script src="http://html5shiv.googlecode.com/svn/trunk/html5.js"></script>

<meta http-equiv="X-UA-Compatible" content="IE=edge,chrome=1">

<![endif]-->

<script src="<?php bloginfo('template_url'); ?>/js/modernizr-1.6.min.js"></script> 



<!-- Stylesheets -->

<link rel="stylesheet" type="text/css" href="<?php bloginfo('template_url'); ?>/css/reset.css" />

 <link rel="stylesheet" type="text/css" href="<?php bloginfo('template_url'); ?>/css/screen.css" />

<!--[if lt IE 9]><link rel="stylesheet" type="text/css" href="<?php bloginfo('template_url'); ?>/css/ie.css" media="screen" /><![endif]-->





<!-- Icons -->

<link rel="shortcut icon" href="/favicon.ico">

<link rel="apple-touch-icon" href="<?php bloginfo('template_url'); ?>/images/apple/icon-iphone.png" />

<link rel="apple-touch-icon" sizes="72x72" href="<?php bloginfo('template_url'); ?>/images/apple/icon-ipad.png" />

<link rel="apple-touch-icon" sizes="114x114" href="<?php bloginfo('template_url'); ?>/images/apple/icon-iphone4.png" />



<!-- RSS -->

<link rel="alternate" type="application/rss+xml" title="<?php bloginfo('name'); ?> RSS Feed" href="<?php bloginfo('rss2_url'); ?>" />

<link rel="alternate" type="application/atom+xml" title="<?php bloginfo('name'); ?> Atom Feed" href="<?php bloginfo('atom_url'); ?>" />

<link rel="pingback" href="<?php bloginfo('pingback_url'); ?>" />



<?php 

wp_head();

global $paywallPro;

?>        

</head>



<body>

<div class="container">

    <div id="outline">

        <header id="header">

            <h1><a href="/" title="Home"><?php bloginfo('name'); ?></a></h1>       

        </header>



        <nav id="main-menu">

            <?php 

            echo preg_replace(array(  

                '/\t/', // remove tabs  

                '/'.str_replace('//','\/\/', get_bloginfo('url')).'/i', // remove full URL  

                '/current_page_item\s*/i',  

                '/current_page_ancestor\s*/i',  

                '/current_page_parent\s*/i',  

                '/page_item\s+/i',  

                '/page-item-\d+\s*/i',  

                '/children\s*/i',  

                '/\s*class=["\']["\']/i', // empty classes  

                '/\s*title="[^"]+"/i', // all titles  

                '/\s+>/i',  

                '/div>/i' // change div to nav  

              ),  

              array(  

                '',  

                '',  

                'active',  

                'open',  

                '',  

                '',  

                '',  

                '',  

                '',  

                '',  

                '>',  

                'nav>'  

              ),  

              wp_nav_menu(array('container' => '', 'theme_location' => header, 'menu_class' => '', 'echo' => false))  

            );

            ?>

        </nav> 



        <header id="strapline">

            <ul id="ticker">

                <?php

                $opts = get_option('ai');

                if (is_array($opts['ticker'])) {

                    foreach ($opts['ticker'] as $key=>$ticker) {

                        if ($ticker['label'] == '' OR $ticker['heading'] == '') continue;

                        ?>

                        <li>

                            <?php if ($ticker['url'] != '') { ?><a href="<?php echo $ticker['url']; ?>" title="Continue Reading <?php echo $ticker['heading']; ?>"><?php } ?>

                            <span style="background-color: #<?php echo $ticker['labelColour']; ?>"><?php echo $ticker['label']; ?></span>

                            <?php echo $ticker['heading']; ?>

                            <?php if ($ticker['url'] != '') { ?></a><?php } ?>

                        </li> 

                        <?php

                    }

                }

                ?>

            </ul>

            <ul id="account-actions">

                <?php

                if (is_user_logged_in()) {

                    ?>

                    <li class="user">Logged in as <?php echo $paywallPro->user->user_login; ?></li>

                    <li class="button"><a href="/account" title="My Account">My Account</a></li>

                    <li class="button"><a href="<?php echo wp_logout_url(get_bloginfo('url')); ?>" title="Logout">Logout</a></li>

                    <?php

                } else {

                    ?>

                    <li class="button"><a href="/register" title="Register">Register</a></li>

                    <li class="button"><a href="/login" title="Sign In">Sign In</a></li>

                    <?php

                }    

                ?>

            </ul>



        </header>
        <div id="main">

Again this is the advert I want to put at the bottom of the header.

<?php
// Advert    

            if (!$paywallPro->user->acl['removeAds']) { 
                ?>
                <div class="advert">
                    <script type="text/javascript"><!--
google_ad_client = "pub-6230056227145824";
/* AnfieldIndex.com Main Site */
google_ad_slot = "3598117926";
google_ad_width = 468;
google_ad_height = 60;
//-->
</script>
<script type="text/javascript"
src="http://pagead2.googlesyndication.com/pagead/show_ads.js">
</script>
                </div>
                <div class="advert-notice"><a href="/register">Subscribe Now to remove these adverts.</a></div>
                <?php    


    ?>

Tried a few times to either put it below the <div id="main"> or above it but getting Parse error: syntax error, unexpected $end in /nfs/c08/h02/mnt/116350/domains/dev.anfieldindex.com/html/wp-content/themes/anfieldindex/header.php on line 288

Thanks for the help!

[nickc25]

I did it! Thanks for the help!!

[nickc25]

I think I might need to start a new topic on this one but this is another thing I need to do. Currently the blog has a rectangle block with 1 post and then a lot of squares with more posts below (see image). How would I change this code to make it so that all the articles were in rectangles like the one at the top? Thanks

This is the code (posts.php)

<?php
/**
* Displays one wide article + grid of additional posts, with pagination.
* 
* Called by archive.php, index.php
* 
* @package Wordpress
* @subpackage Anfield Index
* @author Tim Carr
* @version 1
* @copyright n7 Studios
*/

global $paywallPro;
$count = 0;
if (have_posts()) {
    while (have_posts()) {
        the_post();
        $count++;

    // Define Label
    $user = get_userdata($post->post_author);
    foreach ($user->wp_capabilities as $role=>$enabled) {
        $label = ($role == 'administrator' OR $role == 'editor') ? 'Staff' : '';
    }


    ?> 
    <article class="<?php echo ($count == 1 ? 'full featured' : ($count % 2 > 0 ? 'grid end' : 'grid')); ?>">
        <?php 
        if ($count == 1) { 
            if ($label != '') { ?><div class="label"><span><?php echo $label; ?></span><div class="label-corner"></div></div><?php } ?>
            <div class="content<?php echo ($label != '' ? ' has-label' : ''); ?>">
                <?php if (has_post_thumbnail()) { ?>
                    <div class="main-image"><a href="<?php the_permalink(); ?>" title="Continue Reading <?php the_title(); ?>"><?php the_post_thumbnail('square'); ?></a></div>
                <?php } ?>
                <header>
                    <h1<?php echo (($label != '' AND !has_post_thumbnail()) ? ' class="no-post-thumbnail"' : ''); ?>><a href="<?php the_permalink(); ?>" title="Continue Reading <?php the_title(); ?>"><?php the_title(); ?></a></h1>
                    <ul>
                        <li><time datetime="<?php echo get_the_time('Y-m-d').'T'.get_the_time('H:i'); ?>" pubdate><?php echo get_the_time('jS F Y'); ?></time></li>
                        <li class="comments"><?php comments_number('0', '1', '%'); ?></li>
                    </ul>
                </header> 
                <?php the_excerpt(); ?>
                <p><a href="<?php the_permalink(); ?>" title="Continue Reading <?php the_title(); ?>" class="button">Continue Reading &raquo;</a></p>
            </div>
            <ul class="meta">
                <li class="categories"><?php the_category(', '); ?></li>
                <li class="author"><?php the_author_posts_link(); ?></li>
            </ul>
            <?php 
        } else { 
            ?>
            <header>
                <h1><a href="<?php the_permalink(); ?>" title="Continue Reading <?php the_title(); ?>"><?php the_title(); ?></a></h1>
                <ul>
                    <li><time datetime="<?php echo get_the_time('Y-m-d').'T'.get_the_time('H:i'); ?>" pubdate><?php echo get_the_time('jS F Y'); ?></time></li>
                    <li class="comments"><?php comments_number('0', '1', '%'); ?></li>
                </ul>
            </header>
            <?php if ($label != '') { ?><div class="label"><span><?php echo $label; ?></span><div class="label-corner"></div></div><?php } ?>
            <div class="content<?php echo ($label != '' ? ' has-label' : ''); ?>"> 
                <?php the_excerpt(); ?>
                <p><a href="<?php the_permalink(); ?>" title="Continue Reading <?php the_title(); ?>" class="button">Continue Reading &raquo;</a></p>
            </div>
            <ul class="meta">
                <li class="categories"><?php the_category(', '); ?></li>
                <li class="author"><?php the_author_posts_link(); ?></li>
            </ul>
        <?php 
        } 
        ?>
    </article>
    <?php   
}
?>

<!-- Nav: Pagination --> 
<nav id="pagination">
    <ul>
        <li id="newer"><?php previous_posts_link('&laquo; Newer Entries') ?></li>
        <li id="older"><?php next_posts_link('Older Entries &raquo;') ?></li>
    </ul>
</nav>
<?php
} else {
    ?>
    <article class="full">
        <div class="content">
            <p>There are currently no posts.</p>
        </div>
    </article>
    <?php
}
?>