XOOPS / XoopsCore

htdocs/modules/menus/class/menu.php

Spacing +8 added lines, -8 removed lines

@@ -62,26 +62,26 @@
      */
     public function updateWeights(MenusMenu $obj)
     {
-        $sql = "UPDATE " . $this->table
+        $sql = "UPDATE ".$this->table
         . " SET weight = weight+1"
-        . " WHERE weight >= " . $obj->getVar('weight')
-        . " AND id <> " . $obj->getVar('id')
+        . " WHERE weight >= ".$obj->getVar('weight')
+        . " AND id <> ".$obj->getVar('id')
         /*. " AND pid = " . $obj->getVar('pid')*/
-        . " AND mid = " . $obj->getVar('mid')
+        . " AND mid = ".$obj->getVar('mid')
         ;
         $originalForce = $this->db2->getForce();
         $this->db2->setForce(true);
         $this->db2->query($sql);
 
-        $sql = "SELECT id FROM " . $this->table
-        . " WHERE mid = " . $obj->getVar('mid')
+        $sql = "SELECT id FROM ".$this->table
+        . " WHERE mid = ".$obj->getVar('mid')
         /*. " AND pid = " . $obj->getVar('pid')*/
         . " ORDER BY weight ASC"
         ;
         $result = $this->db2->query($sql);
-        $i = 1;  //lets start at 1 please!
+        $i = 1; //lets start at 1 please!
         while (false !== (list($id) = $result->fetch(FetchMode::NUMERIC))) {
-            $sql = "UPDATE " . $this->table
+            $sql = "UPDATE ".$this->table
             . " SET weight = {$i}"
             . " WHERE id = {$id}"
             ;

htdocs/modules/menus/admin/admin_menu.php

Spacing +8 added lines, -8 removed lines

@@ -20,7 +20,7 @@ 
  * @version         $Id$
  */
 
-include_once __DIR__ . '/header.php';
+include_once __DIR__.'/header.php';
 
 $xoops = Xoops::getInstance();
 $helper = Menus::getInstance();
@@ -94,7 +94,7 @@ 
         $msg[] = _AM_MENUS_SAVE;
 
         $id = Request::getInt('id', 0);
-        if (isset($id) && $id !=0) {
+        if (isset($id) && $id != 0) {
             $obj = $helper->getHandlerMenu()->get($id);
         } else {
             $obj = $helper->getHandlerMenu()->create();
@@ -119,7 +119,7 @@ 
 
         if ($this_handler->insert($obj)) {
             $this_handler->updateWeights($obj);
-            $xoops->redirect('admin_menu.php?op=list&amp;menu_id=' . $obj->getVar('mid'), 2, implode('<br />', $msg));
+            $xoops->redirect('admin_menu.php?op=list&amp;menu_id='.$obj->getVar('mid'), 2, implode('<br />', $msg));
         }
         echo $xoops->alert('error', $obj->getHtmlErrors());
         $form = $helper->getForm($obj, 'menus_menu');
@@ -135,7 +135,7 @@ 
                 $xoops->redirect('admin_menu.php', 3, implode(',', $xoops->security()->getErrors()));
             }
             if ($helper->getHandlerMenu()->delete($obj)) {
-                $xoops->redirect('admin_menu.php?menu_id=' . $menu_id, 2, _AM_MENUS_MSG_SUCCESS);
+                $xoops->redirect('admin_menu.php?menu_id='.$menu_id, 2, _AM_MENUS_MSG_SUCCESS);
             } else {
                 echo $xoops->alert('error', $obj->getHtmlErrors());
             }
@@ -143,7 +143,7 @@ 
             echo $xoops->confirm(
                 array('ok' => 1, 'id' => $id, 'op' => 'del', 'menu_id' => $menu_id),
                 $helper->url('admin/admin_menu.php'),
-                _AM_MENUS_MSG_SUREDEL . '<br /><strong>' . $obj->getVar('title') . '</strong>'
+                _AM_MENUS_MSG_SUREDEL.'<br /><strong>'.$obj->getVar('title').'</strong>'
             );
         }
         break;
@@ -154,7 +154,7 @@ 
         $obj->setVar('weight', $weight);
         $this_handler->insert($obj);
         $this_handler->updateWeights($obj);
-        $xoops->redirect('admin_menu.php?op=list&amp;menu_id=' . $obj->getVar('mid'), 2, _AM_MENUS_SAVE);
+        $xoops->redirect('admin_menu.php?op=list&amp;menu_id='.$obj->getVar('mid'), 2, _AM_MENUS_SAVE);
         break;
 
     case 'toggle':
@@ -163,12 +163,12 @@ 
         $obj = $this_handler->get($id);
         $obj->setVar('visible', $visible);
         $this_handler->insert($obj);
-        $xoops->redirect('admin_menu.php?op=list&amp;menu_id=' . $obj->getVar('mid'), 2, _AM_MENUS_SAVE);
+        $xoops->redirect('admin_menu.php?op=list&amp;menu_id='.$obj->getVar('mid'), 2, _AM_MENUS_SAVE);
         break;
 
     case 'list':
     default:
-        $admin_page->addItemButton(_AM_MENUS_ADD_MENUS, 'admin_menu.php?op=add&amp;menu_id=' . $menu_id, 'add');
+        $admin_page->addItemButton(_AM_MENUS_ADD_MENUS, 'admin_menu.php?op=add&amp;menu_id='.$menu_id, 'add');
         $admin_page->renderButton();
 
         $this_handler = $helper->getHandlerMenu();