banner_box_all add problem

hello!

im having some problems w/ adding another banner box.

heres my site. https://www.kidnemo.com

i made duplicates of my original banner_box_all in modules and templates and renamed all of them (even the content) to banner_box_all2.

under banner box control, the banner_box_all2 shows but under layout settings, the banner_box_all2 doesnt.

When I set the copy to active on the site, all it does it duplicates the original...

am i missing a step somewhere?

thx for your help everyone

Becky Lee
Kid Nemo Company

i tried refering to the regular banners and see how it was setup and applied it to the 2 banner_box_all modules... but still no luck...

can anyone help me out please?

thank you in advance

What setting do you have for:

  $new_banner_search = zen_build_banners_group(SHOW_BANNERS_GROUP_SET_ALL);

In the tpl_ file?

Did that change to:
SHOW_BANNERS_GROUP_SET_ALL2

And does that have its own settings somewhere?

thank you in advance for helping ajeh!

heres what i am looking at.

/www/includes/modules/sideboxes/banner_box_all2.php

<?php
/**
 * banner_box2 sidebox - second box used to display "square" banners in sideboxes
 *
 * @package templateSystem
 * @copyright Copyright 2003-2005 Zen Cart Development Team
 * @copyright Portions Copyright 2003 osCommerce
 * @license http://www.zen-cart.com/license/2_0.txt GNU Public License V2.0
 * @version $Id: banner_box2.php 3133 2006-03-07 23:39:02Z ajeh $
 */

// test if box should display
  $show_banner_box2 = true;
  if (SHOW_BANNERS_GROUP_SET8 == '') {
    $show_banner_box2 = false;
  }

  if ($show_banner_box2 == true) {
    $banner_box[] = TEXT_BANNER_BOX2;
    $banner_box_group= SHOW_BANNERS_GROUP_SET8;

    require($template->get_template_dir('tpl_banner_box2.php',DIR_WS_TEMPLATE, $current_page_base,'sideboxes'). '/tpl_banner_box2.php');

// if no active banner in the specified banner group then the box will not show
// uses banners in the defined group $banner_box_group
    if ($banner->RecordCount() > 0) {

      $title =  BOX_HEADING_BANNER_BOX2;
      $title_link = false;
      require($template->get_template_dir($column_box_default, DIR_WS_TEMPLATE, $current_page_base,'common') . '/' . $column_box_default);
    }
  }
?>

/www/includes/templates/template_default/sideboxes/tpl_banner_box_all2.php

<?php
/**
 * Side Box Template
 *
 * @package templateSystem
 * @copyright Copyright 2003-2005 Zen Cart Development Team
 * @copyright Portions Copyright 2003 osCommerce
 * @license http://www.zen-cart.com/license/2_0.txt GNU Public License V2.0
 * @version $Id: tpl_banner_box_all2.php 2982 2006-02-07 07:56:41Z birdbrain $
 */
    $content = '';
    $content .= '<div id="' . str_replace('_', '-', $box_id . 'Content') . '" class="sideBoxContent centeredContent">';
// select banners_group to be used
  $new_banner_search = zen_build_banners_group(SHOW_BANNERS_GROUP_SET_ALL2);

  // secure pages
  switch ($request_type) {
    case ('SSL'):
      $my_banner_filter=" and banners_on_ssl= " . "1 ";
      break;
    case ('NONSSL'):
      $my_banner_filter='';
      break;
  }

  $sql = "select banners_id from " . TABLE_BANNERS . " where status = 1 " . $new_banner_search . $my_banner_filter . " order by banners_sort_order";
  $banners_all = $db->Execute($sql);



// if no active banner in the specified banner group then the box will not show
  $banner_cnt = 0;
  while (!$banners_all->EOF) {
    $banner_cnt++;
    $banner = zen_banner_exists('dynamic', SHOW_BANNERS_GROUP_SET_ALL2);
    $content .= zen_display_banner('static', $banners_all->fields['banners_id']);
// add spacing between banners
    if ($banner_cnt < $banners_all->RecordCount()) {
      $content .= '<br /><br />';
    }
    $banners_all->MoveNext();
  }
  $content .= '</div>';
?>

Thanks again for your kind help^^

Becky L

banner_box2.php or did you mean to post a copy of the clone you made for banner_box_all.php ...:unsure:

hi ajeh~

i wanted a second banner_box_all for my site, so I cloned the original banner_box_all and renamed it and its content in both the template and modules area.

i renamed it banner_box_all2.php

in my admin, the banner_box_all2 shows, but under layout setting in config, the option is not there to rename the banner destination.

You need to configure something to indicate the Banner Group(s) used by the new banner_box_all2 ...

yeah...

i already designated groups for all of the banner boxes already in configuration>layout settings...

but the banner_box_all2 isnt coming up in layout settings so I cant designate a group for it.

The setting in the configuration table for the configuration_key
SHOW_BANNERS_GROUP_SET_ALL

Manages the Banners All sidebox ...

This is an actual setting in the database ...

If you make a new one for:
SHOW_BANNERS_GROUP_SET_ALL2

Somewhere that either has to be defined in the database configuration table or you need a define that loads in the right sequence for the new banner box to hear it ...

hi ajeh... thank you for your help and patience.

do you happen to know where i should be looking then?

my html knowledge is almost zero so pointing me in the right direction is much appriciated.

Ajeh:

banner_box2.php or did you mean to post a copy of the clone you made for banner_box_all.php ...:unsure:

opps.. yeah the first one was the regular banner_box_all

The Banners Box All uses this sql statement to add itself to the Layout Settings:

INSERT INTO configuration (configuration_title, configuration_key, configuration_value, configuration_description, configuration_group_id, sort_order, use_function, set_function, date_added) VALUES ('Banner Display Group - Side Box banner_box_all', 'SHOW_BANNERS_GROUP_SET_ALL', 'BannersAll', 'The Banner Display Group may only be from one (1) Banner Group for the Banner All sidebox<br /><br />Default Group is BannersAll<br /><br />What Banner Group do you want to use in the Side Box - banner_box_all?<br />Leave blank for none', '19', '72', '', '', now());

You can adapt this INSERT to work with your new configuration_key for the:
SHOW_BANNERS_GROUP_SET_ALL2

This will allow you to configure it in the Layout Settings as well as ensure that it is loaded at the right time to work with your new sidebox ... providing all else is coded correctly ...

im really sorry ajeh,

where do i insert the adapted sql statement into?

If you use the Tools ... Insert SQL Patches ...

It will add it to your database configuration table for you ...

NOTE: read that carefully as that is the one for the original configuration_key for:
SHOW_BANNERS_GROUP_SET_ALL

You will want to edit it a bit for your configuration_key for your new sidebox:
SHOW_BANNERS_GROUP_SET_ALL2

And perhaps update the text etc. to reflect which box this is when viewing it from the Layout Settings ...

omg ajeh!

it worked!

you really made my week!

much praise to you for your great help and patience w/ my questions!

<3 ajeh and zencart !!!

-Becky Lee
KNC

/me thud ... :lamo:

Remember us when you are rich and famous ... :smartass:

i wish! lol

oh btw... i forgot to ask... (here i go again)

would you know how i can change the spacing between the banners in banner_all? i want the images to be a bit closer together.

lol thx!

View source ... see if this is breaks or class managed ...

i looked around in templates and modules and havent seen anything that mentions breaks or class in the banner_all groups

psst ...

// add spacing between banners
    if ($banner_cnt < $banners_all->RecordCount()) {
      $content .= '<br /><br />';
    }