The closing of the php Tag is missing in template.php thus creating problems when adding custom functions above in the template.php

Comments

Romejoe’s picture

Romejoe commented
Title: The closing of php Tag is missing in template.php » Slideshow banner blocked by Kaspersky AV (Anti-Banner module)

I upgraded from Opera 9.x to 10.53 recently. Since that upgrade the banner place on the frontpage is blank and showing no picture.

As I found now it was caused by the Kaspersky AV Anti-Banner Module but I have no Idea why it was working until the Opera upgrade (Kaspersky has not been changed besides automatic upgrade).

As far as I know it filters Graphics with words like BANNER in their name or in the templates.
Therefore I recomend renaming the slideshow banner in templates and graphics to something like "front_display" or such.

danpros’s picture

danpros commented
Category: bug » feature

Hi,

Thanks for reporting it, problems that previously unthinkable :)

Yes, I'll try to change that in our next release. Can you test that by changing it to another ids or classes

<div id="slideshow-wrapper">
<div class="slideshow-wrapper-inner">
<div id="slideshow-wrapper-preface">
 <?php if ($preface): ?>
          <div id="preface">
            <?php print $preface; ?>
          </div><!-- /preface -->
 <?php endif; ?>
</div>
<?php if ($mission) : ?><div id="slideshow-bottom">
<div id="mission"><?php print $mission; ?></div></div><?php endif; ?>
<div class="slideshow">
<img src="<?php print $base_path . $directory; ?>/images/banners/sea.jpg" width="950" height="355" alt="Slideshow 1"/>
<img src="<?php print $base_path . $directory; ?>/images/banners/noon.jpg" width="950" height="355" alt="Slideshow 2"/>
<img src="<?php print $base_path . $directory; ?>/images/banners/snow.jpg" width="950" height="355" alt="Slideshow 3"/>
</div>
</div>
</div>

Change the CSS based on the changes that have been done on Ids or Classes above.

Romejoe’s picture

Romejoe commented

Did not work just with the changes above.

I changed the "banner" directory (on the server and in the page-front-template) to "slideshow" then it worked even with the Kasperski-Anti-Banner module on.

You should change this:

/* Banner + Preface top + Slideshow
-------------------------------------------------------------- */

as well to be sure it will work with future versions of Kasperski I think. But it worked even with this in the css.

danpros’s picture

danpros commented
Version: 6.x-1.9 » 6.x-1.10

Hi,

Thanks Gork, Yes it's a big changes.

Romejoe’s picture

Romejoe commented

Just to clarify:
Worked with changed directory AND your code. So both is needed.

danpros’s picture

danpros commented
Status: Active » Closed (fixed)

Hi,

I already fix this.