How do i make banner ads appear on the front page only?

[mjw2305]

Thanks

Here is the info you need

Zen Cart 1.3.7
Database Patch Level: 1.3.7
PHP Version 4.4.7
Template - Gloss Black

Site link
www.fantasy############srus.co.uk

Hope this is enough info

The Welcome greeting is currently in the Header 3 position

Many Thanks

[kuroi]

That's helpful but puzzling. The version of Zen Cart that you are using supports the $this_is_home_page variable, and Gloss Black plays nicely with it and doesn't over-ride the tpl_main_page file, so I'm puzzled as to where the code that you posted earlier came from, unless you have amended it yourself.

Also, I can see no reason why the code posted earlier shouldn't work, or evidence on your site of the error that you reported. Could you post the change that you made and a few lines before and after it and detail the circumstances in which the error occurs.

[mjw2305]

Here is the change that i made

if (SHOW_BANNERS_GROUP_SET2 != '' && $banner = zen_banner_exists('dynamic', SHOW_BANNERS_GROUP_SET2)) {

Changed to

if (SHOW_BANNERS_GROUP_SET2 != '' && $banner = zen_banner_exists('dynamic', SHOW_BANNERS_GROUP_SET2) && $this_is_home_page) {

which gave me this error

Fatal error: Call to a member function on a non-object in /home/jane776/public_html/shop/includes/templates/template_default/common/tpl_main_page.php on line 102

Which appeared as the only text in the centre column (it stopped everything else in the centre column from appearing)

I have removed the modification now as the site is in its busy time of day, but i really do appriciate your perseverance with this - thanks alot

[kuroi]

This doesn't make sense. The code that you say you changed is in the Gloss Black or template_default tpl_header.php (you didn't say which you changed), but the error has been generated by your template_default tpl_main_page.php file.

[mjw2305]

In default Zen Cart you would find the line in your tpl_main_page.php that is identical to the line to which I referred in my original post, except for referring to SET3 instead of SET2 and make the change that I recommended.

I changed the code for Header SET3 in template_default tpl_main_page.php file

SET3 doesn't appear in the tpl_header.php

Sorry for not saying which file earlier, but it would appear that the error does make sense given the changes i made

Kind Regards

[mjw2305]

Sorry just realised you asked for the changes with a few lines either side of it so here it is

?>
<td valign="top">
<!-- bof breadcrumb -->
<?php if (DEFINE_BREADCRUMB_STATUS == '1' || (DEFINE_BREADCRUMB_STATUS == '2' && !$this_is_home_page) ) { ?>
<div id="navBreadCrumb"><?php echo $breadcrumb->trail(BREAD_CRUMBS_SEPARATOR); ?></div>
<?php } ?>
<!-- eof breadcrumb -->

<?php
if (SHOW_BANNERS_GROUP_SET3 != '' && $banner = zen_banner_exists('dynamic', SHOW_BANNERS_GROUP_SET3)) {
if ($banner->RecordCount() > 0) {
?>
<div id="bannerThree" class="banners"><?php echo zen_display_banner('static', $banner); ?></div>
<?php
}
}
?>

<!-- bof upload alerts -->
<?php if ($messageStack->size('upload') > 0) echo $messageStack->output('upload'); ?>
<!-- eof upload alerts -->

<?php
/**
* prepares and displays center column
*
*/
require($body_code); ?>


I changed the section you instructed to this


if (SHOW_BANNERS_GROUP_SET3 != '' && $banner = zen_banner_exists('dynamic', SHOW_BANNERS_GROUP_SET3) && $this_is_home_page) {

Kind Regards

[mjw2305]

Any more ideas on what the problem might be Kuroi?

Or is there anyone else out there who has had this problem before?

Thanks in advance

[mjw2305]

Ok i have a little more information now i have been trying out a few different things.

IF I CHANGE

if (SHOW_BANNERS_GROUP_SET3 != '' && $banner = zen_banner_exists('dynamic', SHOW_BANNERS_GROUP_SET3)) {

TO

if (SHOW_BANNERS_GROUP_SET3 != '' && $banner = zen_banner_exists('dynamic', SHOW_BANNERS_GROUP_SET3) && $this_is_home_page) {

In the template_default tpl_main_page.php file, I get this error relating to the file i have changed

Fatal error: Call to a member function on a non-object

But the error is only present on the main page, all the other pages function perfectly without displaying the Header in SET3

As a test i did the same modification to SET2 in the tpl_header.php and had the same result - Main page wouldn't work but all the others were perfectly normal with the Header in SET2 missing just like it should be.

It appears that this added piece of code '&& $this_is_home_page' is performing as it should, in other words it is preventing the headers appearing outside the main page, but i can't seem to understand why the main page is displaying an error.

It seems that i am so close yet maybe not lol

I am trying to learn all i can about this error and its meaning, but i'm not getting anywhere fast

Does anyone out there know exactly what this error means

'Fatal error: Call to a member function on a non-object'

Many Thanks

Mat

[kuroi]

Sorry, to make clear to PHP exactly what we're doing, we need to modify that code very slightly to either

if (SHOW_BANNERS_GROUP_SET3 != '' && ($banner = zen_banner_exists('dynamic', SHOW_BANNERS_GROUP_SET3)) && $this_is_home_page) {

or my preferred version

if (SHOW_BANNERS_GROUP_SET3 != '' && $this_is_home_page && $banner = zen_banner_exists('dynamic', SHOW_BANNERS_GROUP_SET3)) {

But it should still be done in an over-ride file and not the template_default version.

[aggiesoap]

I've been following this thread as I want a banner only on the main page. I tried both changes that have been suggested, however the banner still shows on all pages. Is there something small I have overlooked? Another setting somewhere? I have the banner turned on in banner manager and all banners OFF in layout box controller.

Thanks!