[xoops-svn] SF.net SVN: xoops:[8676] XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/ banners

[<ma...@us...>]

Revision: 8676
          http://xoops.svn.sourceforge.net/xoops/?rev=8676&view=rev
Author:   mageg
Date:     2012-01-08 17:26:36 +0000 (Sun, 08 Jan 2012)
Log Message:
-----------
fix html error

Modified Paths:
--------------
    XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/admin/banners.php
    XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/index.php

Modified: XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/admin/banners.php
===================================================================
--- XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/admin/banners.php	2012-01-08 13:55:00 UTC (rev 8675)
+++ XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/admin/banners.php	2012-01-08 17:26:36 UTC (rev 8676)
@@ -106,13 +106,11 @@
                 if ($htmlbanner) {
                     $img .= html_entity_decode($htmlcode);
                 } else {
-                    $img = '<div id="xo-bannerfix">';
                     if (stristr($imageurl, '.swf')) {
                         $img .= '<object type="application/x-shockwave-flash" width="468" height="60" data="' . $imageurl . '" style="z-index:100;">' . '<param name="movie" value="' . $imageurl . '" />' . '<param name="wmode" value="opaque" />' . '</object>';
                     } else {
                         $img .= '<img src="' . $imageurl . '" alt="" />';
                     }
-                    $img .= '</div>';
                 }
 
                 $banner['bid'] = $banner_arr[$i]->getVar("bid");

Modified: XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/index.php
===================================================================
--- XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/index.php	2012-01-08 13:55:00 UTC (rev 8675)
+++ XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/index.php	2012-01-08 17:26:36 UTC (rev 8676)
@@ -146,7 +146,11 @@
                 $banner['imageurl'] = $img;
                 $banner['name'] = $client_name;
                 $banner['uid'] = $client_uid;
-                $banner['clickurl'] = $banner_arr[$i]->getVar("clickurl");
+                if ($banner_arr[$i]->getVar("clickurl") == '') {
+                    $banner['clickurl'] = '#';
+                } else {
+                    $banner['clickurl'] = $banner_arr[$i]->getVar("clickurl");
+                }
                 $xoops->tpl->append_by_ref('banner', $banner);
                 $xoops->tpl->append_by_ref('popup_banner', $banner);
                 unset($banner);

This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.