XoopsModules25x / myiframe

class/myiframe.php

Spacing +12 added lines, -12 removed lines

@@ -10,7 +10,7 @@ 
     die('XOOPS root path not defined');
 }
 
-include_once XOOPS_ROOT_PATH . '/kernel/object.php';
+include_once XOOPS_ROOT_PATH.'/kernel/object.php';
 
 /**
  * Class Myiframe
@@ -43,7 +43,7 @@ 
             if (is_array($id)) {
                 $this->assignVars($id);
             } else {
-                $this->load((int)$id);
+                $this->load((int) $id);
             }
         } else {
             $this->setNew();
@@ -55,7 +55,7 @@ 
      */
     public function load($id)
     {
-        $sql   = 'SELECT * FROM ' . $this->db->prefix('myiframe') . ' WHERE frame_frameid=' . (int)$id;
+        $sql   = 'SELECT * FROM '.$this->db->prefix('myiframe').' WHERE frame_frameid='.(int) $id;
         $myrow = $this->db->fetchArray($this->db->query($sql));
         $this->assignVars($myrow);
         if (!$myrow) {
@@ -90,7 +90,7 @@ 
     public function &get($id)
     {
         $ret = null;
-        $sql = 'SELECT * FROM ' . $this->db->prefix('myiframe') . '  WHERE frame_frameid=' . (int)$id;
+        $sql = 'SELECT * FROM '.$this->db->prefix('myiframe').'  WHERE frame_frameid='.(int) $id;
         if (!$result = $this->db->query($sql)) {
             return $ret;
         }
@@ -185,11 +185,11 @@ 
     {
         $ret   = array();
         $limit = $start = 0;
-        $sql   = 'SELECT * FROM ' . $this->db->prefix('myiframe');
+        $sql   = 'SELECT * FROM '.$this->db->prefix('myiframe');
         if (isset($criteria) && is_subclass_of($criteria, 'criteriaelement')) {
-            $sql .= ' ' . $criteria->renderWhere();
+            $sql .= ' '.$criteria->renderWhere();
             if ($criteria->getSort() !== '') {
-                $sql .= ' ORDER BY ' . $criteria->getSort() . ' ' . $criteria->getOrder();
+                $sql .= ' ORDER BY '.$criteria->getSort().' '.$criteria->getOrder();
             }
             $limit = $criteria->getLimit();
             $start = $criteria->getStart();
@@ -215,9 +215,9 @@ 
      */
     public function getCount($criteria = null)
     {
-        $sql = 'SELECT COUNT(*) FROM ' . $this->db->prefix('myiframe');
+        $sql = 'SELECT COUNT(*) FROM '.$this->db->prefix('myiframe');
         if (isset($criteria) && is_subclass_of($criteria, 'criteriaelement')) {
-            $sql .= ' ' . $criteria->renderWhere();
+            $sql .= ' '.$criteria->renderWhere();
         }
         $result = $this->db->query($sql);
         if (!$result) {
@@ -234,9 +234,9 @@ 
      */
     public function deleteAll($criteria = null)
     {
-        $sql = 'DELETE FROM ' . $this->db->prefix('myiframe');
+        $sql = 'DELETE FROM '.$this->db->prefix('myiframe');
         if (isset($criteria) && is_subclass_of($criteria, 'criteriaelement')) {
-            $sql .= ' ' . $criteria->renderWhere();
+            $sql .= ' '.$criteria->renderWhere();
         }
         if (!$result = $this->db->query($sql)) {
             return false;
@@ -250,7 +250,7 @@ 
      */
     public function updatehits($frame_id)
     {
-        $sql = sprintf('UPDATE %s SET frame_hits = frame_hits+1 WHERE frame_frameid=%u', $this->db->prefix('myiframe'), (int)$frame_id);
+        $sql = sprintf('UPDATE %s SET frame_hits = frame_hits+1 WHERE frame_frameid=%u', $this->db->prefix('myiframe'), (int) $frame_id);
         $this->db->queryF($sql);
     }
 }

xoops_version.php

Spacing +3 added lines, -3 removed lines

@@ -69,15 +69,15 @@
 
 if (is_object($xoopsModule) && $xoopsModule->getVar('dirname') == $modversion['dirname'] && $xoopsModule->getVar('isactive')) {
     $i = 0;
-    include_once XOOPS_ROOT_PATH . '/modules/myiframe/include/functions.php';
+    include_once XOOPS_ROOT_PATH.'/modules/myiframe/include/functions.php';
     $myts = MyTextSanitizer::getInstance();
     if (myiframe_getmoduleoption('showinmenu')) {
-        $sql    = 'SELECT * FROM ' . $xoopsDB->prefix('myiframe') . ' ORDER BY frame_description';
+        $sql    = 'SELECT * FROM '.$xoopsDB->prefix('myiframe').' ORDER BY frame_description';
         $result = $xoopsDB->query($sql);
         while ($myrow = $xoopsDB->fetchArray($result)) {
             if (xoops_trim($myrow['frame_description']) !== '') {
                 $modversion['sub'][$i]['name'] = $myts->htmlSpecialChars($myrow['frame_description']);
-                $modversion['sub'][$i]['url']  = 'index.php?iframeid=' . (int)$myrow['frame_frameid'];
+                $modversion['sub'][$i]['url']  = 'index.php?iframeid='.(int) $myrow['frame_frameid'];
                 $i++;
             }
         }

blocks/myiframe_iframe.php

Spacing +7 added lines, -7 removed lines

@@ -6,7 +6,7 @@ 
  * ****************************************************************************
  */
 
-include_once XOOPS_ROOT_PATH . '/modules/myiframe/include/functions.php';
+include_once XOOPS_ROOT_PATH.'/modules/myiframe/include/functions.php';
 
 /**
  * @param $options
@@ -22,7 +22,7 @@ 
         'left',
         'rigth'
     );
-    $tblscrolling  = array(
+    $tblscrolling = array(
         'yes',
         'no',
         'auto'
@@ -35,11 +35,11 @@ 
         $block['longdesc']     = $frame->getVar('frame_description');
         $block['width']        = $frame->getVar('frame_width');
         $block['height']       = $frame->getVar('frame_height');
-        $block['align']        = $tblalign[$frame->getVar('frame_align') - 1];
+        $block['align']        = $tblalign[$frame->getVar('frame_align')-1];
         $block['frameborder']  = $frame->getVar('frame_frameborder');
         $block['marginwidth']  = $frame->getVar('frame_marginwidth');
         $block['marginheight'] = $frame->getVar('frame_marginheight');
-        $block['scrolling']    = $tblscrolling[$frame->getVar('frame_scrolling') - 1];
+        $block['scrolling']    = $tblscrolling[$frame->getVar('frame_scrolling')-1];
         $block['url']          = $frame->getVar('frame_url');
     }
     return $block;
@@ -57,13 +57,13 @@ 
     $critere->setSort('frame_description');
     $frarray = $iframeHandler->getObjects($critere);
 
-    $form = '' . _MB_MYIFRAME_IFRAME . "&nbsp;<select name='options[0]'>";
+    $form = ''._MB_MYIFRAME_IFRAME."&nbsp;<select name='options[0]'>";
     foreach ($frarray as $oneframe) {
-        $form .= "<option value='" . $oneframe->getVar('frame_frameid') . "'";
+        $form .= "<option value='".$oneframe->getVar('frame_frameid')."'";
         if ($options[0] == $oneframe->getVar('frame_frameid')) {
             $form .= " selected='selected'";
         }
-        $form .= '>' . $oneframe->getVar('frame_description') . '</option>';
+        $form .= '>'.$oneframe->getVar('frame_description').'</option>';
     }
     $form .= "</select>\n";
     return $form;

admin/manage.php

Switch Indentation +160 added lines, -160 removed lines

@@ -103,172 +103,172 @@
 }
 
 switch ($op) {
-    case 'verifybeforeedit':
-        if (isset($_POST['submit']) && $_POST['submit'] !== '') {
-            if ($_POST['longdesc'] === '') {
-                xoops_cp_header();
-                $adminObject = \Xmf\Module\Admin::getInstance();
-                $adminObject->displayNavigation(basename(__FILE__));
-                echo "<table width='100%' border='0' cellspacing='1' class='outer'>\n";
-                echo '<tr><td class="odd">';
-                echo "<a href='manage.php'><h4>" . _AM_MYIFRAME_CONFIG . '</h4></a>';
-                echo _AM_MYIFRAME_ERROR_ADD_INDEX;
-                echo '</td></tr></table>';
-                include_once __DIR__ . '/admin_footer.php';
-                xoops_cp_footer();
-                exit();
-            }
+    	case 'verifybeforeedit':
+        	if (isset($_POST['submit']) && $_POST['submit'] !== '') {
+            	if ($_POST['longdesc'] === '') {
+                	xoops_cp_header();
+                	$adminObject = \Xmf\Module\Admin::getInstance();
+                	$adminObject->displayNavigation(basename(__FILE__));
+                	echo "<table width='100%' border='0' cellspacing='1' class='outer'>\n";
+                	echo '<tr><td class="odd">';
+                	echo "<a href='manage.php'><h4>" . _AM_MYIFRAME_CONFIG . '</h4></a>';
+                	echo _AM_MYIFRAME_ERROR_ADD_INDEX;
+                	echo '</td></tr></table>';
+                	include_once __DIR__ . '/admin_footer.php';
+                	xoops_cp_footer();
+                	exit();
+            	}
 
-            $frameid = $_POST['frameid'];
-            $frame   = $iframeHandler->get($frameid);
-            $frame->unsetNew();
-            $frame->setVar('frame_description', $_POST['longdesc']);
-            $frame->setVar('frame_width', $_POST['width']);
-            $frame->setVar('frame_height', $_POST['height']);
-            $frame->setVar('frame_align', $_POST['align']);
-            $frame->setVar('frame_frameborder', $_POST['frameborder']);
-            $frame->setVar('frame_marginwidth', $_POST['marginwidth']);
-            $frame->setVar('frame_marginheight', $_POST['marginheight']);
-            $frame->setVar('frame_scrolling', $_POST['scrolling']);
-            $frame->setVar('frame_url', $_POST['url']);
-            $frame->setVar('frame_uid', $xoopsUser->getVar('uid'));
-            $res = $iframeHandler->insert($frame);
-            if (!$res) {
-                redirect_header('manage.php', 1, _AM_MYIFRAME_ERROR_MODIFY_DB);
-                exit();
-            }
-            redirect_header('manage.php', 1, _AM_MYIFRAME_DBUPDATED);
-            exit();
-        }
-        break;
+            	$frameid = $_POST['frameid'];
+            	$frame   = $iframeHandler->get($frameid);
+            	$frame->unsetNew();
+            	$frame->setVar('frame_description', $_POST['longdesc']);
+            	$frame->setVar('frame_width', $_POST['width']);
+            	$frame->setVar('frame_height', $_POST['height']);
+            	$frame->setVar('frame_align', $_POST['align']);
+            	$frame->setVar('frame_frameborder', $_POST['frameborder']);
+            	$frame->setVar('frame_marginwidth', $_POST['marginwidth']);
+            	$frame->setVar('frame_marginheight', $_POST['marginheight']);
+            	$frame->setVar('frame_scrolling', $_POST['scrolling']);
+            	$frame->setVar('frame_url', $_POST['url']);
+            	$frame->setVar('frame_uid', $xoopsUser->getVar('uid'));
+            	$res = $iframeHandler->insert($frame);
+            	if (!$res) {
+                	redirect_header('manage.php', 1, _AM_MYIFRAME_ERROR_MODIFY_DB);
+                	exit();
+            	}
+            	redirect_header('manage.php', 1, _AM_MYIFRAME_DBUPDATED);
+            	exit();
+        	}
+        	break;
 
-    case 'edit':
-        xoops_cp_header();
-        $adminObject = \Xmf\Module\Admin::getInstance();
-        $adminObject->displayNavigation(basename(__FILE__));
-        if (isset($_GET['frameid'])) {
-            $frameid = (int)$_GET['frameid'];
-            $frame   = $iframeHandler->get($frameid);
-            AddEditForm($frameid, 'verifybeforeedit', _AM_MYIFRAME_CONFIG, $frame->getVar('frame_description', 'e'), $frame->getVar('frame_width', 'e'), $frame->getVar('frame_height', 'e'), $frame->getVar('frame_align', 'e'),
-                        $frame->getVar('frame_frameborder', 'e'), $frame->getVar('frame_marginwidth', 'e'), $frame->getVar('frame_marginheight', 'e'), $frame->getVar('frame_scrolling', 'e'), $frame->getVar('frame_url', 'e'), _AM_MYIFRAME_UPDATE);
-        } else {
-            xoops_cp_header();
-            $adminObject = \Xmf\Module\Admin::getInstance();
-            $adminObject->displayNavigation(basename(__FILE__));
-            echo "<table width='100%' border='0' cellspacing='1' class='outer'>\n";
-            echo '<tr><td class="odd">';
-            echo "<a href='manage.php'><h4>" . _AM_MYIFRAME_CONFIG . '</h4></a>';
-            echo _AM_MYIFRAME_ERROR_ADD_INDEX;
-            echo "</td></tr></table>\n";
-            include_once __DIR__ . '/admin_footer.php';
-            xoops_cp_footer();
-            exit();
-        }
-        break;
+    	case 'edit':
+        	xoops_cp_header();
+        	$adminObject = \Xmf\Module\Admin::getInstance();
+        	$adminObject->displayNavigation(basename(__FILE__));
+        	if (isset($_GET['frameid'])) {
+            	$frameid = (int)$_GET['frameid'];
+            	$frame   = $iframeHandler->get($frameid);
+            	AddEditForm($frameid, 'verifybeforeedit', _AM_MYIFRAME_CONFIG, $frame->getVar('frame_description', 'e'), $frame->getVar('frame_width', 'e'), $frame->getVar('frame_height', 'e'), $frame->getVar('frame_align', 'e'),
+                        	$frame->getVar('frame_frameborder', 'e'), $frame->getVar('frame_marginwidth', 'e'), $frame->getVar('frame_marginheight', 'e'), $frame->getVar('frame_scrolling', 'e'), $frame->getVar('frame_url', 'e'), _AM_MYIFRAME_UPDATE);
+        	} else {
+            	xoops_cp_header();
+            	$adminObject = \Xmf\Module\Admin::getInstance();
+            	$adminObject->displayNavigation(basename(__FILE__));
+            	echo "<table width='100%' border='0' cellspacing='1' class='outer'>\n";
+            	echo '<tr><td class="odd">';
+            	echo "<a href='manage.php'><h4>" . _AM_MYIFRAME_CONFIG . '</h4></a>';
+            	echo _AM_MYIFRAME_ERROR_ADD_INDEX;
+            	echo "</td></tr></table>\n";
+            	include_once __DIR__ . '/admin_footer.php';
+            	xoops_cp_footer();
+            	exit();
+        	}
+        	break;
 
-    case 'delete':
-        if (!isset($_POST['ok'])) {
-            xoops_cp_header();
-            $adminObject = \Xmf\Module\Admin::getInstance();
-            $adminObject->displayNavigation(basename(__FILE__));
-            echo '<h4>' . _AM_MYIFRAME_CONFIG . '</h4>';
-            xoops_confirm(array(
-                              'op'      => 'delete',
-                              'frameid' => (int)$_GET['frameid'],
-                              'ok'      => 1
-                          ), 'manage.php', _AM_MYIFRAME_RUSUREDEL);
-            include_once __DIR__ . '/admin_footer.php';
-        } else {
-            if (empty($_POST['frameid'])) {
-                redirect_header('manage.php', 2, _AM_MYIFRAME_ERROR_ADD_INDEX);
-                exit();
-            }
-            $frameid = (int)$_POST['frameid'];
-            $critere = new Criteria('frame_frameid', $frameid, '=');
-            $iframeHandler->deleteAll($critere);
-            redirect_header('manage.php', 1, _AM_MYIFRAME_DBUPDATED);
-            exit();
-        }
-        break;
+    	case 'delete':
+        	if (!isset($_POST['ok'])) {
+            	xoops_cp_header();
+            	$adminObject = \Xmf\Module\Admin::getInstance();
+            	$adminObject->displayNavigation(basename(__FILE__));
+            	echo '<h4>' . _AM_MYIFRAME_CONFIG . '</h4>';
+            	xoops_confirm(array(
+                              	'op'      => 'delete',
+                              	'frameid' => (int)$_GET['frameid'],
+                              	'ok'      => 1
+                          	), 'manage.php', _AM_MYIFRAME_RUSUREDEL);
+            	include_once __DIR__ . '/admin_footer.php';
+        	} else {
+            	if (empty($_POST['frameid'])) {
+                	redirect_header('manage.php', 2, _AM_MYIFRAME_ERROR_ADD_INDEX);
+                	exit();
+            	}
+            	$frameid = (int)$_POST['frameid'];
+            	$critere = new Criteria('frame_frameid', $frameid, '=');
+            	$iframeHandler->deleteAll($critere);
+            	redirect_header('manage.php', 1, _AM_MYIFRAME_DBUPDATED);
+            	exit();
+        	}
+        	break;
 
-    case 'verifytoadd':
-        if (isset($_POST['submit']) && $_POST['submit'] !== '') {
-            if ($_POST['url'] === '') {
-                xoops_cp_header();
-                $adminObject = \Xmf\Module\Admin::getInstance();
-                $adminObject->displayNavigation(basename(__FILE__));
-                echo "<table width='100%' border='0' cellspacing='1' class='outer'>\n";
-                echo '<tr><td class="odd">';
-                echo "<a href='manage.php'><h4>" . _AM_MYIFRAME_CONFIG . '</h4></a>';
-                echo _AM_MYIFRAME_ERROR_ADD_INDEX;
-                echo "</td></tr></table>\n";
-                include_once __DIR__ . '/admin_footer.php';
-                xoops_cp_footer();
-                $adminObject = \Xmf\Module\Admin::getInstance();
-                $adminObject->displayNavigation(basename(__FILE__));
-                exit();
-            }
-            $frame = $iframeHandler->create(true);
-            $frame->setVar('frame_description', $_POST['longdesc']);
-            $frame->setVar('frame_width', $_POST['width']);
-            $frame->setVar('frame_height', $_POST['height']);
-            $frame->setVar('frame_align', $_POST['align']);
-            $frame->setVar('frame_frameborder', $_POST['frameborder']);
-            $frame->setVar('frame_marginwidth', $_POST['marginwidth']);
-            $frame->setVar('frame_marginheight', $_POST['marginheight']);
-            $frame->setVar('frame_scrolling', $_POST['scrolling']);
-            $frame->setVar('frame_url', $_POST['url']);
-            $frame->setVar('frame_created', time());
-            $frame->setVar('frame_uid', $xoopsUser->getVar('uid'));
-            $res = $iframeHandler->insert($frame);
-            if (!$res) {
-                redirect_header('manage.php', 1, _AM_MYIFRAME_ERROR_ADD_INDEX);
-                exit();
-            }
-            redirect_header('manage.php', 1, _AM_MYIFRAME_ADDED_OK);
-        }
-        break;
+    	case 'verifytoadd':
+        	if (isset($_POST['submit']) && $_POST['submit'] !== '') {
+            	if ($_POST['url'] === '') {
+                	xoops_cp_header();
+                	$adminObject = \Xmf\Module\Admin::getInstance();
+                	$adminObject->displayNavigation(basename(__FILE__));
+                	echo "<table width='100%' border='0' cellspacing='1' class='outer'>\n";
+                	echo '<tr><td class="odd">';
+                	echo "<a href='manage.php'><h4>" . _AM_MYIFRAME_CONFIG . '</h4></a>';
+                	echo _AM_MYIFRAME_ERROR_ADD_INDEX;
+                	echo "</td></tr></table>\n";
+                	include_once __DIR__ . '/admin_footer.php';
+                	xoops_cp_footer();
+                	$adminObject = \Xmf\Module\Admin::getInstance();
+                	$adminObject->displayNavigation(basename(__FILE__));
+                	exit();
+            	}
+            	$frame = $iframeHandler->create(true);
+            	$frame->setVar('frame_description', $_POST['longdesc']);
+            	$frame->setVar('frame_width', $_POST['width']);
+            	$frame->setVar('frame_height', $_POST['height']);
+            	$frame->setVar('frame_align', $_POST['align']);
+            	$frame->setVar('frame_frameborder', $_POST['frameborder']);
+            	$frame->setVar('frame_marginwidth', $_POST['marginwidth']);
+            	$frame->setVar('frame_marginheight', $_POST['marginheight']);
+            	$frame->setVar('frame_scrolling', $_POST['scrolling']);
+            	$frame->setVar('frame_url', $_POST['url']);
+            	$frame->setVar('frame_created', time());
+            	$frame->setVar('frame_uid', $xoopsUser->getVar('uid'));
+            	$res = $iframeHandler->insert($frame);
+            	if (!$res) {
+                	redirect_header('manage.php', 1, _AM_MYIFRAME_ERROR_ADD_INDEX);
+                	exit();
+            	}
+            	redirect_header('manage.php', 1, _AM_MYIFRAME_ADDED_OK);
+        	}
+        	break;
 
-    case 'addframe':
-        xoops_cp_header();
-        $adminObject = \Xmf\Module\Admin::getInstance();
-        $adminObject->displayNavigation(basename(__FILE__));
-        AddEditForm(0, 'verifytoadd', _AM_MYIFRAME_CONFIG, '', '100%', '', '', '0', '0', '0', 1, '', _AM_MYIFRAME_ADDBUTTON);
-        break;
+    	case 'addframe':
+        	xoops_cp_header();
+        	$adminObject = \Xmf\Module\Admin::getInstance();
+        	$adminObject->displayNavigation(basename(__FILE__));
+        	AddEditForm(0, 'verifytoadd', _AM_MYIFRAME_CONFIG, '', '100%', '', '', '0', '0', '0', 1, '', _AM_MYIFRAME_ADDBUTTON);
+        	break;
 
-    case 'default':
-    default:
-        xoops_cp_header();
-        $adminObject = \Xmf\Module\Admin::getInstance();
-        $adminObject->displayNavigation(basename(__FILE__));
-        echo '<h4>' . _AM_MYIFRAME_CONFIG . "</h4><br />\n";
-        echo "<table width='100%' border='0' cellspacing='1' class='outer'>\n";
-        echo "<tr><th align='center'>" . _AM_MYIFRAME_ID . "</th><th align='center'>" . _AM_MYIFRAME_DESC . "</th><th align='center'>" . _AM_MYIFRAME_CREATED . "</th><th align='center'>" . _AM_MYIFRAME_HITS . "</th><th align='center'>"
-             . _AM_MYIFRAME_ACTION . "</th></tr>\n";
-        $critere = new Criteria('1', '1', '=');
-        $critere->setSort('frame_description');
-        $frarray = $iframeHandler->getObjects($critere);
-        $class   = 'even';
-        $baseurl = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/admin/manage.php';
-        if (count($frarray) > 0) {
-            foreach ($frarray as $frame) {
-                $action_edit   = "<a href='" . $baseurl . '?op=edit&frameid=' . $frame->getVar('frame_frameid') . "'><img src='../assets/images/edit.png' alt='" . _AM_MYIFRAME_EDIT . "'></a>";
-                $action_delete = "<a href='" . $baseurl . '?op=delete&frameid=' . $frame->getVar('frame_frameid') . "'><img src='../assets/images/delete.png' alt='" . _AM_MYIFRAME_DELETE . "'></a>";
-                if (xoops_trim($frame->getVar('frame_description') === '')) {
-                    $liendesc = $frame->getVar('frame_url');
-                } else {
-                    $liendesc = "<a href='" . XOOPS_URL . '/modules/myiframe/index.php?iframeid=' . $frame->getVar('frame_frameid') . "'>" . $frame->getVar('frame_description') . '</a>';
-                }
-                echo "<tr class='" . $class . "'><td align='center'>" . $frame->getVar('frame_frameid') . "</td><td align='center'>" . $liendesc . "</td><td align='center'>" . formatTimestamp($frame->getVar('frame_created'))
-                     . "</td><td align='center'>" . $frame->getVar('frame_hits') . "</td><td align='center'>" . $action_edit . '&nbsp;-&nbsp;' . $action_delete . "</td></tr>\n";
-                $class = ($class === 'even') ? 'odd' : 'even';
-            }
-        }
-        echo "<tr class='" . $class . "'><td colspan='5' align='center'><form name='faddframe' method='post' action='manage.php'><input type='hidden' name='op' value='addframe'><input type='submit' name='submit' value='" . _AM_MYIFRAME_ADD
-             . "'></td></tr>";
-        echo '</table>';
-        include_once __DIR__ . '/admin_footer.php';
-        break;
+    	case 'default':
+    	default:
+        	xoops_cp_header();
+        	$adminObject = \Xmf\Module\Admin::getInstance();
+        	$adminObject->displayNavigation(basename(__FILE__));
+        	echo '<h4>' . _AM_MYIFRAME_CONFIG . "</h4><br />\n";
+        	echo "<table width='100%' border='0' cellspacing='1' class='outer'>\n";
+        	echo "<tr><th align='center'>" . _AM_MYIFRAME_ID . "</th><th align='center'>" . _AM_MYIFRAME_DESC . "</th><th align='center'>" . _AM_MYIFRAME_CREATED . "</th><th align='center'>" . _AM_MYIFRAME_HITS . "</th><th align='center'>"
+             	. _AM_MYIFRAME_ACTION . "</th></tr>\n";
+        	$critere = new Criteria('1', '1', '=');
+        	$critere->setSort('frame_description');
+        	$frarray = $iframeHandler->getObjects($critere);
+        	$class   = 'even';
+        	$baseurl = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/admin/manage.php';
+        	if (count($frarray) > 0) {
+            	foreach ($frarray as $frame) {
+                	$action_edit   = "<a href='" . $baseurl . '?op=edit&frameid=' . $frame->getVar('frame_frameid') . "'><img src='../assets/images/edit.png' alt='" . _AM_MYIFRAME_EDIT . "'></a>";
+                	$action_delete = "<a href='" . $baseurl . '?op=delete&frameid=' . $frame->getVar('frame_frameid') . "'><img src='../assets/images/delete.png' alt='" . _AM_MYIFRAME_DELETE . "'></a>";
+                	if (xoops_trim($frame->getVar('frame_description') === '')) {
+                    	$liendesc = $frame->getVar('frame_url');
+                	} else {
+                    	$liendesc = "<a href='" . XOOPS_URL . '/modules/myiframe/index.php?iframeid=' . $frame->getVar('frame_frameid') . "'>" . $frame->getVar('frame_description') . '</a>';
+                	}
+                	echo "<tr class='" . $class . "'><td align='center'>" . $frame->getVar('frame_frameid') . "</td><td align='center'>" . $liendesc . "</td><td align='center'>" . formatTimestamp($frame->getVar('frame_created'))
+                     	. "</td><td align='center'>" . $frame->getVar('frame_hits') . "</td><td align='center'>" . $action_edit . '&nbsp;-&nbsp;' . $action_delete . "</td></tr>\n";
+                	$class = ($class === 'even') ? 'odd' : 'even';
+            	}
+        	}
+        	echo "<tr class='" . $class . "'><td colspan='5' align='center'><form name='faddframe' method='post' action='manage.php'><input type='hidden' name='op' value='addframe'><input type='submit' name='submit' value='" . _AM_MYIFRAME_ADD
+             	. "'></td></tr>";
+        	echo '</table>';
+        	include_once __DIR__ . '/admin_footer.php';
+        	break;
 }
 
 xoops_cp_footer();

Spacing +41 added lines, -41 removed lines

@@ -6,25 +6,25 @@ 
  * ****************************************************************************
  */
 
-include __DIR__ . '/../../../include/cp_header.php';
-include_once XOOPS_ROOT_PATH . '/modules/myiframe/include/functions.php';
+include __DIR__.'/../../../include/cp_header.php';
+include_once XOOPS_ROOT_PATH.'/modules/myiframe/include/functions.php';
 
 // Verify if the table is up to date
 if (!myiframe_FieldExists('frame_frameid', $xoopsDB->prefix('myiframe'))) {
-    $result = $xoopsDB->queryF('ALTER TABLE ' . $xoopsDB->prefix('myiframe') . ' CHANGE `frameid` `frame_frameid` INT( 8 ) NOT NULL AUTO_INCREMENT');
-    $result = $xoopsDB->queryF('ALTER TABLE ' . $xoopsDB->prefix('myiframe') . " CHANGE `created` `frame_created` INT( 10 ) UNSIGNED NOT NULL DEFAULT '0'");
-    $result = $xoopsDB->queryF('ALTER TABLE ' . $xoopsDB->prefix('myiframe') . " CHANGE `uid` `frame_uid` MEDIUMINT( 8 ) UNSIGNED NOT NULL DEFAULT '0'");
-    $result = $xoopsDB->queryF('ALTER TABLE ' . $xoopsDB->prefix('myiframe') . ' CHANGE `longdesc` `frame_description` VARCHAR( 255 ) NOT NULL');
-    $result = $xoopsDB->queryF('ALTER TABLE ' . $xoopsDB->prefix('myiframe') . ' CHANGE `width` `frame_width` VARCHAR( 15 ) NOT NULL');
-    $result = $xoopsDB->queryF('ALTER TABLE ' . $xoopsDB->prefix('myiframe') . ' CHANGE `height` `frame_height` VARCHAR( 15 ) NOT NULL');
-    $result = $xoopsDB->queryF('ALTER TABLE ' . $xoopsDB->prefix('myiframe') . " CHANGE `align` `frame_align` SMALLINT( 2 ) NOT NULL DEFAULT '0'");
-    $result = $xoopsDB->queryF('ALTER TABLE ' . $xoopsDB->prefix('myiframe') . " CHANGE `frameborder` `frame_frameborder` SMALLINT( 3 ) NOT NULL DEFAULT '0'");
-    $result = $xoopsDB->queryF('ALTER TABLE ' . $xoopsDB->prefix('myiframe') . " CHANGE `marginwidth` `frame_marginwidth` SMALLINT( 3 ) NOT NULL DEFAULT '0'");
-    $result = $xoopsDB->queryF('ALTER TABLE ' . $xoopsDB->prefix('myiframe') . " CHANGE `marginheight` `frame_marginheight` SMALLINT( 3 ) NOT NULL DEFAULT '0'");
-    $result = $xoopsDB->queryF('ALTER TABLE ' . $xoopsDB->prefix('myiframe') . " CHANGE `scrolling` `frame_scrolling` SMALLINT( 1 ) NOT NULL DEFAULT '0'");
-    $result = $xoopsDB->queryF('ALTER TABLE ' . $xoopsDB->prefix('myiframe') . " CHANGE `hits` `frame_hits` INT( 8 ) UNSIGNED NOT NULL DEFAULT '0'");
-    $result = $xoopsDB->queryF('ALTER TABLE ' . $xoopsDB->prefix('myiframe') . ' CHANGE `url` `frame_url` VARCHAR( 255 ) NOT NULL');
-    header('Location : ' . XOOPS_URL . '/modules/system/admin.php?fct=modulesadmin&op=update&module=myiframe');
+    $result = $xoopsDB->queryF('ALTER TABLE '.$xoopsDB->prefix('myiframe').' CHANGE `frameid` `frame_frameid` INT( 8 ) NOT NULL AUTO_INCREMENT');
+    $result = $xoopsDB->queryF('ALTER TABLE '.$xoopsDB->prefix('myiframe')." CHANGE `created` `frame_created` INT( 10 ) UNSIGNED NOT NULL DEFAULT '0'");
+    $result = $xoopsDB->queryF('ALTER TABLE '.$xoopsDB->prefix('myiframe')." CHANGE `uid` `frame_uid` MEDIUMINT( 8 ) UNSIGNED NOT NULL DEFAULT '0'");
+    $result = $xoopsDB->queryF('ALTER TABLE '.$xoopsDB->prefix('myiframe').' CHANGE `longdesc` `frame_description` VARCHAR( 255 ) NOT NULL');
+    $result = $xoopsDB->queryF('ALTER TABLE '.$xoopsDB->prefix('myiframe').' CHANGE `width` `frame_width` VARCHAR( 15 ) NOT NULL');
+    $result = $xoopsDB->queryF('ALTER TABLE '.$xoopsDB->prefix('myiframe').' CHANGE `height` `frame_height` VARCHAR( 15 ) NOT NULL');
+    $result = $xoopsDB->queryF('ALTER TABLE '.$xoopsDB->prefix('myiframe')." CHANGE `align` `frame_align` SMALLINT( 2 ) NOT NULL DEFAULT '0'");
+    $result = $xoopsDB->queryF('ALTER TABLE '.$xoopsDB->prefix('myiframe')." CHANGE `frameborder` `frame_frameborder` SMALLINT( 3 ) NOT NULL DEFAULT '0'");
+    $result = $xoopsDB->queryF('ALTER TABLE '.$xoopsDB->prefix('myiframe')." CHANGE `marginwidth` `frame_marginwidth` SMALLINT( 3 ) NOT NULL DEFAULT '0'");
+    $result = $xoopsDB->queryF('ALTER TABLE '.$xoopsDB->prefix('myiframe')." CHANGE `marginheight` `frame_marginheight` SMALLINT( 3 ) NOT NULL DEFAULT '0'");
+    $result = $xoopsDB->queryF('ALTER TABLE '.$xoopsDB->prefix('myiframe')." CHANGE `scrolling` `frame_scrolling` SMALLINT( 1 ) NOT NULL DEFAULT '0'");
+    $result = $xoopsDB->queryF('ALTER TABLE '.$xoopsDB->prefix('myiframe')." CHANGE `hits` `frame_hits` INT( 8 ) UNSIGNED NOT NULL DEFAULT '0'");
+    $result = $xoopsDB->queryF('ALTER TABLE '.$xoopsDB->prefix('myiframe').' CHANGE `url` `frame_url` VARCHAR( 255 ) NOT NULL');
+    header('Location : '.XOOPS_URL.'/modules/system/admin.php?fct=modulesadmin&op=update&module=myiframe');
 }
 
 $module_id     = $xoopsModule->getVar('mid');
@@ -48,10 +48,10 @@ 
  */
 function addEditForm($frameid, $Action, $FormTitle, $longdesc, $width, $height, $align, $frameborder, $marginwidth, $marginheight, $scrolling, $url, $LabelSubmitButton)
 {
-    include_once XOOPS_ROOT_PATH . '/class/xoopsformloader.php';
+    include_once XOOPS_ROOT_PATH.'/class/xoopsformloader.php';
     global $xoopsModule;
 
-    $sform = new XoopsThemeForm($FormTitle, 'indexform', XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/admin/manage.php');
+    $sform = new XoopsThemeForm($FormTitle, 'indexform', XOOPS_URL.'/modules/'.$xoopsModule->getVar('dirname').'/admin/manage.php');
     $sform->addElement(new XoopsFormText(_AM_MYIFRAME_DESC, 'longdesc', 50, 255, $longdesc), false);
     $sform->addElement(new XoopsFormText(_AM_MYIFRAME_WIDTH, 'width', 10, 15, $width), false);
     $sform->addElement(new XoopsFormText(_AM_MYIFRAME_HEIGHT, 'height', 10, 15, $height), false);
@@ -89,7 +89,7 @@ 
     $button_tray->addElement($cancel_btn);
     $sform->addElement($button_tray);
     $sform->display();
-    include_once __DIR__ . '/admin_footer.php';
+    include_once __DIR__.'/admin_footer.php';
 }
 
 // ******************************************************************************************************************************************
@@ -111,10 +111,10 @@ 
                 $adminObject->displayNavigation(basename(__FILE__));
                 echo "<table width='100%' border='0' cellspacing='1' class='outer'>\n";
                 echo '<tr><td class="odd">';
-                echo "<a href='manage.php'><h4>" . _AM_MYIFRAME_CONFIG . '</h4></a>';
+                echo "<a href='manage.php'><h4>"._AM_MYIFRAME_CONFIG.'</h4></a>';
                 echo _AM_MYIFRAME_ERROR_ADD_INDEX;
                 echo '</td></tr></table>';
-                include_once __DIR__ . '/admin_footer.php';
+                include_once __DIR__.'/admin_footer.php';
                 xoops_cp_footer();
                 exit();
             }
@@ -147,7 +147,7 @@ 
         $adminObject = \Xmf\Module\Admin::getInstance();
         $adminObject->displayNavigation(basename(__FILE__));
         if (isset($_GET['frameid'])) {
-            $frameid = (int)$_GET['frameid'];
+            $frameid = (int) $_GET['frameid'];
             $frame   = $iframeHandler->get($frameid);
             AddEditForm($frameid, 'verifybeforeedit', _AM_MYIFRAME_CONFIG, $frame->getVar('frame_description', 'e'), $frame->getVar('frame_width', 'e'), $frame->getVar('frame_height', 'e'), $frame->getVar('frame_align', 'e'),
                         $frame->getVar('frame_frameborder', 'e'), $frame->getVar('frame_marginwidth', 'e'), $frame->getVar('frame_marginheight', 'e'), $frame->getVar('frame_scrolling', 'e'), $frame->getVar('frame_url', 'e'), _AM_MYIFRAME_UPDATE);
@@ -157,10 +157,10 @@ 
             $adminObject->displayNavigation(basename(__FILE__));
             echo "<table width='100%' border='0' cellspacing='1' class='outer'>\n";
             echo '<tr><td class="odd">';
-            echo "<a href='manage.php'><h4>" . _AM_MYIFRAME_CONFIG . '</h4></a>';
+            echo "<a href='manage.php'><h4>"._AM_MYIFRAME_CONFIG.'</h4></a>';
             echo _AM_MYIFRAME_ERROR_ADD_INDEX;
             echo "</td></tr></table>\n";
-            include_once __DIR__ . '/admin_footer.php';
+            include_once __DIR__.'/admin_footer.php';
             xoops_cp_footer();
             exit();
         }
@@ -171,19 +171,19 @@ 
             xoops_cp_header();
             $adminObject = \Xmf\Module\Admin::getInstance();
             $adminObject->displayNavigation(basename(__FILE__));
-            echo '<h4>' . _AM_MYIFRAME_CONFIG . '</h4>';
+            echo '<h4>'._AM_MYIFRAME_CONFIG.'</h4>';
             xoops_confirm(array(
                               'op'      => 'delete',
-                              'frameid' => (int)$_GET['frameid'],
+                              'frameid' => (int) $_GET['frameid'],
                               'ok'      => 1
                           ), 'manage.php', _AM_MYIFRAME_RUSUREDEL);
-            include_once __DIR__ . '/admin_footer.php';
+            include_once __DIR__.'/admin_footer.php';
         } else {
             if (empty($_POST['frameid'])) {
                 redirect_header('manage.php', 2, _AM_MYIFRAME_ERROR_ADD_INDEX);
                 exit();
             }
-            $frameid = (int)$_POST['frameid'];
+            $frameid = (int) $_POST['frameid'];
             $critere = new Criteria('frame_frameid', $frameid, '=');
             $iframeHandler->deleteAll($critere);
             redirect_header('manage.php', 1, _AM_MYIFRAME_DBUPDATED);
@@ -199,10 +199,10 @@ 
                 $adminObject->displayNavigation(basename(__FILE__));
                 echo "<table width='100%' border='0' cellspacing='1' class='outer'>\n";
                 echo '<tr><td class="odd">';
-                echo "<a href='manage.php'><h4>" . _AM_MYIFRAME_CONFIG . '</h4></a>';
+                echo "<a href='manage.php'><h4>"._AM_MYIFRAME_CONFIG.'</h4></a>';
                 echo _AM_MYIFRAME_ERROR_ADD_INDEX;
                 echo "</td></tr></table>\n";
-                include_once __DIR__ . '/admin_footer.php';
+                include_once __DIR__.'/admin_footer.php';
                 xoops_cp_footer();
                 $adminObject = \Xmf\Module\Admin::getInstance();
                 $adminObject->displayNavigation(basename(__FILE__));
@@ -241,33 +241,33 @@ 
         xoops_cp_header();
         $adminObject = \Xmf\Module\Admin::getInstance();
         $adminObject->displayNavigation(basename(__FILE__));
-        echo '<h4>' . _AM_MYIFRAME_CONFIG . "</h4><br />\n";
+        echo '<h4>'._AM_MYIFRAME_CONFIG."</h4><br />\n";
         echo "<table width='100%' border='0' cellspacing='1' class='outer'>\n";
-        echo "<tr><th align='center'>" . _AM_MYIFRAME_ID . "</th><th align='center'>" . _AM_MYIFRAME_DESC . "</th><th align='center'>" . _AM_MYIFRAME_CREATED . "</th><th align='center'>" . _AM_MYIFRAME_HITS . "</th><th align='center'>"
-             . _AM_MYIFRAME_ACTION . "</th></tr>\n";
+        echo "<tr><th align='center'>"._AM_MYIFRAME_ID."</th><th align='center'>"._AM_MYIFRAME_DESC."</th><th align='center'>"._AM_MYIFRAME_CREATED."</th><th align='center'>"._AM_MYIFRAME_HITS."</th><th align='center'>"
+             . _AM_MYIFRAME_ACTION."</th></tr>\n";
         $critere = new Criteria('1', '1', '=');
         $critere->setSort('frame_description');
         $frarray = $iframeHandler->getObjects($critere);
         $class   = 'even';
-        $baseurl = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/admin/manage.php';
+        $baseurl = XOOPS_URL.'/modules/'.$xoopsModule->getVar('dirname').'/admin/manage.php';
         if (count($frarray) > 0) {
             foreach ($frarray as $frame) {
-                $action_edit   = "<a href='" . $baseurl . '?op=edit&frameid=' . $frame->getVar('frame_frameid') . "'><img src='../assets/images/edit.png' alt='" . _AM_MYIFRAME_EDIT . "'></a>";
-                $action_delete = "<a href='" . $baseurl . '?op=delete&frameid=' . $frame->getVar('frame_frameid') . "'><img src='../assets/images/delete.png' alt='" . _AM_MYIFRAME_DELETE . "'></a>";
+                $action_edit   = "<a href='".$baseurl.'?op=edit&frameid='.$frame->getVar('frame_frameid')."'><img src='../assets/images/edit.png' alt='"._AM_MYIFRAME_EDIT."'></a>";
+                $action_delete = "<a href='".$baseurl.'?op=delete&frameid='.$frame->getVar('frame_frameid')."'><img src='../assets/images/delete.png' alt='"._AM_MYIFRAME_DELETE."'></a>";
                 if (xoops_trim($frame->getVar('frame_description') === '')) {
                     $liendesc = $frame->getVar('frame_url');
                 } else {
-                    $liendesc = "<a href='" . XOOPS_URL . '/modules/myiframe/index.php?iframeid=' . $frame->getVar('frame_frameid') . "'>" . $frame->getVar('frame_description') . '</a>';
+                    $liendesc = "<a href='".XOOPS_URL.'/modules/myiframe/index.php?iframeid='.$frame->getVar('frame_frameid')."'>".$frame->getVar('frame_description').'</a>';
                 }
-                echo "<tr class='" . $class . "'><td align='center'>" . $frame->getVar('frame_frameid') . "</td><td align='center'>" . $liendesc . "</td><td align='center'>" . formatTimestamp($frame->getVar('frame_created'))
-                     . "</td><td align='center'>" . $frame->getVar('frame_hits') . "</td><td align='center'>" . $action_edit . '&nbsp;-&nbsp;' . $action_delete . "</td></tr>\n";
+                echo "<tr class='".$class."'><td align='center'>".$frame->getVar('frame_frameid')."</td><td align='center'>".$liendesc."</td><td align='center'>".formatTimestamp($frame->getVar('frame_created'))
+                     . "</td><td align='center'>".$frame->getVar('frame_hits')."</td><td align='center'>".$action_edit.'&nbsp;-&nbsp;'.$action_delete."</td></tr>\n";
                 $class = ($class === 'even') ? 'odd' : 'even';
             }
         }
-        echo "<tr class='" . $class . "'><td colspan='5' align='center'><form name='faddframe' method='post' action='manage.php'><input type='hidden' name='op' value='addframe'><input type='submit' name='submit' value='" . _AM_MYIFRAME_ADD
+        echo "<tr class='".$class."'><td colspan='5' align='center'><form name='faddframe' method='post' action='manage.php'><input type='hidden' name='op' value='addframe'><input type='submit' name='submit' value='"._AM_MYIFRAME_ADD
              . "'></td></tr>";
         echo '</table>';
-        include_once __DIR__ . '/admin_footer.php';
+        include_once __DIR__.'/admin_footer.php';
         break;
 }
 

admin/admin_footer.php

Spacing +2 added lines, -2 removed lines

@@ -17,13 +17,13 @@
  * @author     XOOPS Development Team
  */
 
-$pathIcon32      = Xmf\Module\Admin::iconUrl('', 32);
+$pathIcon32 = Xmf\Module\Admin::iconUrl('', 32);
 
 echo "<div class='adminfooter'>\n"
      ."  <div style='text-align: center;'>\n"
      ."    <a href='http://www.xoops.org' rel='external'><img src='{$pathIcon32}/xoopsmicrobutton.gif' alt='XOOPS' title='XOOPS'></a>\n"
      ."  </div>\n"
-     .'  ' . _AM_MODULEADMIN_ADMIN_FOOTER . "\n"
+     .'  '._AM_MODULEADMIN_ADMIN_FOOTER."\n"
      .'</div>';
 
 xoops_cp_footer();

admin/menu.php

Spacing +3 added lines, -3 removed lines

@@ -23,19 +23,19 @@
     array(
         'title' => _MI_MYIFRAME_NAME,
         'link'  => 'admin/index.php',
-        'icon'  => $pathIcon32 . '/home.png'
+        'icon'  => $pathIcon32.'/home.png'
     ),
 
     array(
         'title' => _MI_MYIFRAME_ADMENU1,
         'link'  => 'admin/manage.php',
-        'icon'  => $pathIcon32 . '/manage.png'
+        'icon'  => $pathIcon32.'/manage.png'
     ),
 
     // Category
     array(
         'title' => _MI_MYIFRAME_ADMENU2,
         'link'  => 'admin/about.php',
-        'icon'  => $pathIcon32 . '/about.png'
+        'icon'  => $pathIcon32.'/about.png'
     ),
 );

admin/about.php

Spacing +2 added lines, -2 removed lines

@@ -9,11 +9,11 @@
 #  Licence : GPL 							#
 #######################################################
 
-require_once __DIR__ . '/admin_header.php';
+require_once __DIR__.'/admin_header.php';
 xoops_cp_header();
 $adminObject = \Xmf\Module\Admin::getInstance();
 $adminObject->displayNavigation(basename(__FILE__));
 \Xmf\Module\Admin::setPaypal('aerograf@shmel.org');
 $adminObject->displayAbout(false);
 
-require_once __DIR__ . '/admin_footer.php';
+require_once __DIR__.'/admin_footer.php';

admin/index.php

Spacing +2 added lines, -2 removed lines

@@ -10,10 +10,10 @@
  * @version             $Id: blocks_mytype.php 2017-06-06
  **/
 
-include_once __DIR__ . '/admin_header.php';
+include_once __DIR__.'/admin_header.php';
 xoops_cp_header();
 $adminObject = \Xmf\Module\Admin::getInstance();
 $adminObject->displayNavigation(basename(__FILE__));
 $adminObject->displayIndex();
 
-include_once __DIR__ . '/admin_footer.php';
+include_once __DIR__.'/admin_footer.php';

admin/admin_header.php

Spacing +6 added lines, -6 removed lines

@@ -1,16 +1,16 @@
 <?php
-include __DIR__ . '/../../../mainfile.php';
-include XOOPS_ROOT_PATH . '/include/cp_functions.php';
-include_once XOOPS_ROOT_PATH . '/kernel/module.php';
-include __DIR__ . '/../../../include/cp_header.php';
+include __DIR__.'/../../../mainfile.php';
+include XOOPS_ROOT_PATH.'/include/cp_functions.php';
+include_once XOOPS_ROOT_PATH.'/kernel/module.php';
+include __DIR__.'/../../../include/cp_header.php';
 if ($xoopsUser) {
     $xoopsModule = XoopsModule::getByDirname('myiframe');
     if (!$xoopsUser->isAdmin($xoopsModule->mid())) {
-        redirect_header(XOOPS_URL . '/', 3, _NOPERM);
+        redirect_header(XOOPS_URL.'/', 3, _NOPERM);
         exit();
     }
 } else {
-    redirect_header(XOOPS_URL . '/', 3, _NOPERM);
+    redirect_header(XOOPS_URL.'/', 3, _NOPERM);
     exit();
 }