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

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

Revision: 9410
          http://xoops.svn.sourceforge.net/xoops/?rev=9410&view=rev
Author:   mageg
Date:     2012-04-29 20:11:00 +0000 (Sun, 29 Apr 2012)
Log Message:
-----------
update banners

Modified Paths:
--------------
    XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/admin/menu.php
    XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/class/form/banner.php
    XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/language/english/admin.php
    XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/language/english/help/help.html
    XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/templates/admin/banners_admin_banners.html
    XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/templates/admin/banners_admin_clients.html

Modified: XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/admin/menu.php
===================================================================
--- XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/admin/menu.php	2012-04-29 20:09:15 UTC (rev 9409)
+++ XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/admin/menu.php	2012-04-29 20:11:00 UTC (rev 9410)
@@ -19,22 +19,21 @@
  * @author          Mage Gr\xE9gory (AKA Mage)
  * @version         $Id$
  */
-$pathIcon32 = '../../media/xoops/images/icons/32/';
 
 $adminmenu = array();
 $i = 1;
 $adminmenu[$i]['title'] = _MI_BANNERS_INDEX;
 $adminmenu[$i]['link'] = "admin/index.php";
-$adminmenu[$i]['icon'] = $pathIcon32 . 'home.png';
+$adminmenu[$i]['icon'] = 'home.png';
 $i++;
 $adminmenu[$i]['title'] = _MI_BANNERS_CLIENTS;
 $adminmenu[$i]['link'] = "admin/clients.php";
-$adminmenu[$i]['icon'] = $pathIcon32 . 'index.png';
+$adminmenu[$i]['icon'] = 'index.png';
 $i++;
 $adminmenu[$i]['title'] = _MI_BANNERS_BANNERS;
 $adminmenu[$i]['link'] = "admin/banners.php";
-$adminmenu[$i]['icon'] = $pathIcon32 . 'banners.png';
+$adminmenu[$i]['icon'] = 'banners.png';
 $i++;
 $adminmenu[$i]['title'] = _MI_BANNERS_ABOUT;
 $adminmenu[$i]['link'] = 'admin/about.php';
-$adminmenu[$i]['icon'] = $pathIcon32 . 'about.png';
\ No newline at end of file
+$adminmenu[$i]['icon'] = 'about.png';
\ No newline at end of file

Modified: XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/class/form/banner.php
===================================================================
--- XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/class/form/banner.php	2012-04-29 20:09:15 UTC (rev 9409)
+++ XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/class/form/banner.php	2012-04-29 20:11:00 UTC (rev 9410)
@@ -57,9 +57,11 @@
         $client_select = new XoopsFormSelect( _AM_BANNERS_CLIENTS_NAME, 'cid', $obj->getVar('cid') );
         $client_select->addOptionArray($client_Handler->getList());
         $this->addElement($client_select, true);
+        
+        $imptotal = new XoopsFormText( _AM_BANNERS_BANNERS_IMPRESSIONSP, 'imptotal', 20, 255, $obj->getVar('imptotal') );
+        $imptotal->setPattern('^[0-9]*[0-9]+$|^[0-9]+[0-9]*$', _AM_BANNERS_BANNERS_IMPRESSIONSP_PATTERN);
+        $this->addElement($imptotal, true );
 
-        $this->addElement(new XoopsFormText( _AM_BANNERS_BANNERS_IMPRESSIONSP, 'imptotal', 20, 255, $obj->getVar('imptotal') ), true );
-
         $imgtray_img = new XoopsFormElementTray( _AM_BANNERS_BANNERS_IMAGE, '<br /><br />' );
         $imgtray_img->addElement(new XoopsFormText( _AM_BANNERS_BANNERS_IMGURL, 'imageurl', 80, 255, $obj->getVar('imageurl')));
         $imgpath_img = sprintf( _AM_BANNERS_BANNERS_IMAGE_PATH, XOOPS_UPLOAD_PATH . '/banners/' );

Modified: XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/language/english/admin.php
===================================================================
--- XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/language/english/admin.php	2012-04-29 20:09:15 UTC (rev 9409)
+++ XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/language/english/admin.php	2012-04-29 20:11:00 UTC (rev 9410)
@@ -59,6 +59,7 @@
 define("_AM_BANNERS_BANNERS_IMPRESIONLEFT","Impressions left");
 define("_AM_BANNERS_BANNERS_IMPRESSIONS","Impressions");
 define("_AM_BANNERS_BANNERS_IMPRESSIONSP","Impressions purchased");
+define("_AM_BANNERS_BANNERS_IMPRESSIONSP_PATTERN","You must enter an integer");
 define("_AM_BANNERS_BANNERS_LIST","List of banners");
 define("_AM_BANNERS_BANNERS_NCLICKS","% Clicks");
 define("_AM_BANNERS_BANNERS_RELOAD","Reload banner");

Modified: XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/language/english/help/help.html
===================================================================
--- XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/language/english/help/help.html	2012-04-29 20:09:15 UTC (rev 9409)
+++ XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/language/english/help/help.html	2012-04-29 20:11:00 UTC (rev 9410)
@@ -1,17 +1,10 @@
-<div id="help-template" class="outer">
-    <h1 class="head">Help:
-        <a class="ui-corner-all tooltip" href="<{$xoops_url}>/modules/banners/admin/index.php" title="Back to the administration of Banners">
-        Banners <img src="<{xoAdminIcons home.png}>"alt="Back to the administration of Banners"/>
-        </a>
-    </h1>
-    <!-- ===== Help Content ======= -->
-    <h4 class="odd">Description</h4>
-    <p class="even">
-    Banners are advertisements displayed in the top section of the page. They may be images or flash movies, e.g. the wildly gyrating XOOPS banner that is part of the default installation. XOOPS has a built in system for managing banners and creating accounts for advertising clients, which is found in the System Module -> Banners. You can have multiple banners on a XOOPS site – the display area rotates through each of the active banners in turn each time a new page is loaded.<br /><br />
+<!-- ===== Help Content ======= -->
+<h4>Description</h4>
+<p>
+Banners are advertisements displayed in the top section of the page. They may be images or flash movies, e.g. the wildly gyrating XOOPS banner that is part of the default installation. XOOPS has a built in system for managing banners and creating accounts for advertising clients, which is found in the System Module -> Banners. You can have multiple banners on a XOOPS site – the display area rotates through each of the active banners in turn each time a new page is loaded.<br /><br />
 
-    Each time a banner is displayed to a visitor this counts as one ‘impression’, which is the common trading currency of banners. Clients typically pay for a certain number of impressions – i.e. to have their advertisement displayed a certain number of times.<br /><br />
+Each time a banner is displayed to a visitor this counts as one ‘impression’, which is the common trading currency of banners. Clients typically pay for a certain number of impressions – i.e. to have their advertisement displayed a certain number of times.<br /><br />
 
-    Please note: If you are in the business of selling banners based on the number of impressions there is one important thing you should do: Go to System Module -> Preferences -> General Settings and fill in the ‘Your IP Address’ field (if you have one). This will prevent your visits to the site from being included in the impression count. Failure to do this could be considered somewhat dishonest (noting, however, that many Administrators don’t have a permanent IP – e.g. those using dial up modems).
-    </p>
-    <!-- ===== Help Content ======= -->
-</div> 
+Please note: If you are in the business of selling banners based on the number of impressions there is one important thing you should do: Go to System Module -> Preferences -> General Settings and fill in the ‘Your IP Address’ field (if you have one). This will prevent your visits to the site from being included in the impression count. Failure to do this could be considered somewhat dishonest (noting, however, that many Administrators don’t have a permanent IP – e.g. those using dial up modems).
+</p>
+<!-- ===== Help Content ======= -->

Modified: XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/templates/admin/banners_admin_banners.html
===================================================================
--- XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/templates/admin/banners_admin_banners.html	2012-04-29 20:09:15 UTC (rev 9409)
+++ XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/templates/admin/banners_admin_banners.html	2012-04-29 20:11:00 UTC (rev 9410)
@@ -24,11 +24,11 @@
         <td class="txtcenter"><{$banner.percent}>%</td>
         <td class="txtcenter"><{$banner.name}></td>
         <td class="xo-actions txtcenter">
-            <img class="tooltip" onclick="display_dialog(<{$banner.bid}>, true, true, 'slide', 'slide', 200, 520);" src="<{xoAdminIcons display.png}>" alt="<{$smarty.const._AM_BANNERS_VIEW}>" title="<{$smarty.const._AM_BANNERS_VIEW}>" />
-            <a class="tooltip" href="banners.php?op=edit&amp;bid=<{$banner.bid}>" title="<{$smarty.const._AM_BANNERS_EDIT}>">
+            <img onclick="display_dialog(<{$banner.bid}>, true, true, 'slide', 'slide', 200, 520);" src="<{xoAdminIcons display.png}>" alt="<{$smarty.const._AM_BANNERS_VIEW}>" title="<{$smarty.const._AM_BANNERS_VIEW}>" />
+            <a href="banners.php?op=edit&amp;bid=<{$banner.bid}>" title="<{$smarty.const._AM_BANNERS_EDIT}>">
                 <img src="<{xoAdminIcons edit.png}>" alt="<{$smarty.const._AM_BANNERS_EDIT}>" />
             </a>
-            <a class="tooltip" href="banners.php?op=delete&amp;bid=<{$banner.bid}>" title="<{$smarty.const._AM_BANNERS_DELETE}>">
+            <a href="banners.php?op=delete&amp;bid=<{$banner.bid}>" title="<{$smarty.const._AM_BANNERS_DELETE}>">
                 <img src="<{xoAdminIcons delete.png}>" alt="<{$smarty.const._AM_BANNERS_DELETE}>" />
             </a>
         </td>
@@ -74,11 +74,11 @@
         <td class="txtcenter"><{$banner_finish.percent}>%</td>
         <td class="txtcenter"><{$banner_finish.name}></td>
         <td class="xo-actions txtcenter">
-            <img class="tooltip" onclick="display_dialog(<{$banner_finish.bid}>, true, true, 'slide', 'slide', 200, 520);" src="<{xoAdminIcons display.png}>" alt="<{$smarty.const._AM_BANNERS_VIEW}>" title="<{$smarty.const._AM_BANNERS_VIEW}>" />
-            <a class="tooltip" href="banners.php?op=reload&amp;bid=<{$banner_finish.bid}>" title="<{$smarty.const._AM_BANNERS_BANNERS_RELOAD}>">
+            <img onclick="display_dialog(<{$banner_finish.bid}>, true, true, 'slide', 'slide', 200, 520);" src="<{xoAdminIcons display.png}>" alt="<{$smarty.const._AM_BANNERS_VIEW}>" title="<{$smarty.const._AM_BANNERS_VIEW}>" />
+            <a href="banners.php?op=reload&amp;bid=<{$banner_finish.bid}>" title="<{$smarty.const._AM_BANNERS_BANNERS_RELOAD}>">
                 <img src="<{xoAdminIcons reload.png}>" alt="<{$smarty.const._AM_BANNERS_BANNERS_RELOAD}>"/>
             </a>
-            <a class="tooltip" href="banners.php?op=delete&amp;bid=<{$banner_finish.bid}>" title="<{$smarty.const._AM_BANNERS_DELETE}>">
+            <a href="banners.php?op=delete&amp;bid=<{$banner_finish.bid}>" title="<{$smarty.const._AM_BANNERS_DELETE}>">
                 <img src="<{xoAdminIcons delete.png}>" alt="<{$smarty.const._AM_BANNERS_DELETE}>" />
             </a>
         </td>

Modified: XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/templates/admin/banners_admin_clients.html
===================================================================
--- XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/templates/admin/banners_admin_clients.html	2012-04-29 20:09:15 UTC (rev 9409)
+++ XoopsCore/branches/2.6.x/2.6.0/htdocs/modules/banners/templates/admin/banners_admin_clients.html	2012-04-29 20:11:00 UTC (rev 9410)
@@ -25,11 +25,11 @@
         <td class="txtcenter"><{$client.banner_active}></td>        
         <td class="txtcenter"><{$client.email}></td>
         <td class="xo-actions txtcenter">
-            <img class="tooltip" onclick="display_dialog(<{$client.cid}>, true, true, 'slide', 'slide', 250, 400);" src="<{xoAdminIcons display.png}>" alt="<{$smarty.const._AM_BANNERS_VIEW}>" title="<{$smarty.const._AM_BANNERS_VIEW}>" />
-            <a class="tooltip" href="clients.php?op=edit&amp;cid=<{$client.cid}>" title="<{$smarty.const._AM_BANNERS_EDIT}>">
+            <img onclick="display_dialog(<{$client.cid}>, true, true, 'slide', 'slide', 250, 400);" src="<{xoAdminIcons display.png}>" alt="<{$smarty.const._AM_BANNERS_VIEW}>" title="<{$smarty.const._AM_BANNERS_VIEW}>" />
+            <a href="clients.php?op=edit&amp;cid=<{$client.cid}>" title="<{$smarty.const._AM_BANNERS_EDIT}>">
                 <img src="<{xoAdminIcons edit.png}>" alt="<{$smarty.const._AM_BANNERS_EDIT}>" />
             </a>
-            <a class="tooltip" href="clients.php?op=delete&amp;cid=<{$client.cid}>" title="<{$smarty.const._AM_BANNERS_DELETE}>">
+            <a href="clients.php?op=delete&amp;cid=<{$client.cid}>" title="<{$smarty.const._AM_BANNERS_DELETE}>">
                 <img src="<{xoAdminIcons delete.png}>" alt="<{$smarty.const._AM_BANNERS_DELETE}>" />
             </a>
         </td>

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