Fixes for extension problems while installing/removing (still double-registration...
[mailer.git] / inc / mysql-manager.php
index 84c5fd08fc1c74fe8b5656de27f63356d148cace..7137be73f1506e172de78d16de92b78503ed3168 100644 (file)
@@ -87,12 +87,12 @@ function getModuleTitle ($mod) {
 function getModuleDescription ($mode, $wht, $column = 'what') {
        // Fix empty 'what'
        if (empty($wht)) {
-               $wht = "welcome";
+               $wht = 'welcome';
                if (getConfig('index_home') != '') $wht = getConfig('index_home');
        } // END - if
 
        // Default is not found
-       $ret = "??? (".$wht.')';
+       $ret = '??? (' . $wht . ')';
 
        // Look for title
        $result = SQL_QUERY_ESC("SELECT title FROM `{!_MYSQL_PREFIX!}_%s_menu` WHERE %s='%s' LIMIT 1",
@@ -146,6 +146,7 @@ function checkModulePermissions ($mod) {
        // Check if cache is latest version
        if (GET_EXT_VERSION('cache') >= '0.1.2') {
                // Is the cache there?
+               //* DEBUG: */ DEBUG_LOG(__FUNCTION__, __LINE__, 'Using cache.');
                if (!isset($GLOBALS['cache_array']['modules'])) {
                        // This happens when a new extension is registered while ext-cache is installed
                        $ret = 'cache_miss';
@@ -154,6 +155,7 @@ function checkModulePermissions ($mod) {
                // Is the module cached?
                if (isset($GLOBALS['cache_array']['modules']['locked'][$mod_chk])) {
                        // Check cache
+                       //* DEBUG: */ DEBUG_LOG(__FUNCTION__, __LINE__, 'Cache found.');
                        $locked = $GLOBALS['cache_array']['modules']['locked'][$mod_chk];
                        $hidden = $GLOBALS['cache_array']['modules']['hidden'][$mod_chk];
                        $admin  = $GLOBALS['cache_array']['modules']['admin_only'][$mod_chk];
@@ -168,15 +170,23 @@ function checkModulePermissions ($mod) {
                }
        } elseif (!EXT_IS_ACTIVE('cache')) {
                // Check for module in database
-               $result = SQL_QUERY_ESC("SELECT locked, hidden, admin_only, mem_only FROM `{!_MYSQL_PREFIX!}_mod_reg` WHERE `module`='%s' LIMIT 1",
-               array($mod_chk), __FUNCTION__, __LINE__);
+               //* DEBUG: */ DEBUG_LOG(__FUNCTION__, __LINE__, 'Using database.');
+               $result = SQL_QUERY_ESC("SELECT `locked`, `hidden`, `admin_only`, `mem_only` FROM `{!_MYSQL_PREFIX!}_mod_reg` WHERE `module`='%s' LIMIT 1",
+                       array($mod_chk), __FUNCTION__, __LINE__);
                if (SQL_NUMROWS($result) == 1) {
                        // Read data
+                       //* DEBUG: */ DEBUG_LOG(__FUNCTION__, __LINE__, 'Entry found.');
                        list($locked, $hidden, $admin, $mem) = SQL_FETCHROW($result);
-                       SQL_FREERESULT($result);
                        $found = true;
+               } elseif (isDebugModeEnabled()) {
+                       // Debug message only in debug-mode...
+                       DEBUG_LOG(__FUNCTION__, __LINE__, 'Module ' . $mod_chk . ' not found!');
                }
+
+               // Free result
+               SQL_FREERESULT($result);
        }
+       //* DEBUG: */ DEBUG_LOG(__FUNCTION__, __LINE__, 'ret=' . $ret);
 
        // Is the module found?
        if ($found === true) {
@@ -207,7 +217,7 @@ function checkModulePermissions ($mod) {
        } // END - if
 
        // Still no luck or not found?
-       if (($ret == 'cache_miss') || (!$found)) {
+       if (($ret == 'cache_miss') || ($found === false)) {
                //              ----- Legacy module -----                                               ---- Module in base folder  ----                       --- Module with extension's name ---
                if ((isFileReadable(sprintf("%sinc/modules/%s.php", constant('PATH'), $mod))) || (isFileReadable(sprintf("%s%s.php", constant('PATH'), $mod))) || (isFileReadable(sprintf("%s%s/%s.php", constant('PATH'), $extension, $mod)))) {
                        // Data is missing so we add it
@@ -215,12 +225,12 @@ function checkModulePermissions ($mod) {
                                // Since 0.3.6 we have a has_menu column, this took me a half hour
                                // to find a loop here... *sigh*
                                SQL_QUERY_ESC("INSERT INTO `{!_MYSQL_PREFIX!}_mod_reg`
-(module, locked, hidden, mem_only, admin_only, has_menu) VALUES
+(`module`, `locked`, `hidden`, `mem_only`, `admin_only`, `has_menu`) VALUES
 ('%s','Y','N','N','N','N')", array($mod_chk), __FUNCTION__, __LINE__);
                        } else {
                                // Wrong/missing sql_patches!
                                SQL_QUERY_ESC("INSERT INTO `{!_MYSQL_PREFIX!}_mod_reg`
-(module, locked, hidden, mem_only, admin_only) VALUES
+(`module`, `locked`, `hidden`, `mem_only`, `admin_only`) VALUES
 ('%s','Y','N','N','N')", array($mod_chk), __FUNCTION__, __LINE__);
                        }
 
@@ -231,6 +241,7 @@ function checkModulePermissions ($mod) {
                        } // END - if
 
                        // Destroy cache here
+                       // @TODO Rewrite this to a filter
                        rebuildCacheFiles('modreg', 'modreg');
 
                        // And reload data
@@ -242,21 +253,22 @@ function checkModulePermissions ($mod) {
        } elseif (!$found) {
                // Problem with module detected
                DEBUG_LOG(__FUNCTION__, __LINE__, sprintf("Problem in module %s detected. ret=%s, locked=%s, hidden=%s, mem=%s, admin=%s",
-               $mod,
-               $ret,
-               $locked,
-               $hidden,
-               $mem,
-               $admin
+                       $mod,
+                       $ret,
+                       $locked,
+                       $hidden,
+                       $mem,
+                       $admin
                ));
        }
 
        // Return the value
+       //* DEBUG: */ DEBUG_LOG(__FUNCTION__, __LINE__, 'ret=' . $ret);
        return $ret;
 }
 
 // Add menu description pending on given file name (without path!)
-function ADD_DESCR ($ACC_LVL, $FQFN, $return = false, $output = true) {
+function ADD_DESCR ($accessLevel, $FQFN, $return = false, $output = true) {
        // Use only filename of the FQFN...
        $file = basename($FQFN);
 
@@ -270,7 +282,7 @@ function ADD_DESCR ($ACC_LVL, $FQFN, $return = false, $output = true) {
                // This is an action file!
                $type = 'action';
                $search = substr($file, 7);
-               switch ($ACC_LVL) {
+               switch ($accessLevel) {
                        case 'admin':
                                $modCheck = 'admin';
                                break;
@@ -278,7 +290,7 @@ function ADD_DESCR ($ACC_LVL, $FQFN, $return = false, $output = true) {
                        case 'sponsor':
                        case 'guest':
                        case 'member':
-                               $modCheck = $GLOBALS['module'];
+                               $modCheck = getModule();
                                break;
                }
                $AND = " AND (`what`='' OR `what` IS NULL)";
@@ -287,45 +299,46 @@ function ADD_DESCR ($ACC_LVL, $FQFN, $return = false, $output = true) {
                $type = 'what';
                $search = substr($file, 5);
                $AND = '';
-               switch ($ACC_LVL) {
+               switch ($accessLevel) {
                        case 'admin':
                                $modCheck = 'admin';
                                break;
 
                        case 'guest':
                        case 'member':
-                               $modCheck = $GLOBALS['module'];
+                               $modCheck = getModule();
                                if (!IS_ADMIN()) {
                                        $AND = " AND `visible`='Y' AND `locked`='N'";
                                }
                                break;
                }
+
                $dummy = substr($search, 0, -4);
-               $AND .= " AND `action`='".getModeAction($ACC_LVL, $dummy)."'";
-       } elseif (($ACC_LVL == 'sponsor') || ($ACC_LVL == "engine")) {
+               $AND .= " AND `action`='".getModeAction($accessLevel, $dummy)."'";
+       } elseif (($accessLevel == 'sponsor') || ($accessLevel == "engine")) {
                // Sponsor / engine menu
                $type     = 'what';
                $search   = $file;
-               $modCheck = $GLOBALS['module'];
+               $modCheck = getModule();
                $AND      = '';
        } else {
                // Other
-               $type     = "menu";
+               $type     = 'menu';
                $search   = $file;
-               $modCheck = $GLOBALS['module'];
+               $modCheck = getModule();
                $AND      = '';
        }
 
        // Begin the navigation line
        if ((!isset($GLOBALS['nav_depth'])) && (!$return)) {
                $GLOBALS['nav_depth'] = 0;
-               $prefix = "<div class=\"you_are_here\">{--YOU_ARE_HERE--}&nbsp;<strong><a class=\"you_are_here\" href=\"{!URL!}/modules.php?module=".$GLOBALS['module'].$LINK_ADD."\">Home</a></strong>";
+               $prefix = "<div class=\"you_are_here\">{--YOU_ARE_HERE--}&nbsp;<strong><a class=\"you_are_here\" href=\"{!URL!}/modules.php?module=".getModule().$LINK_ADD."\">Home</a></strong>";
        } else {
                if (!$return) $GLOBALS['nav_depth']++;
                $prefix = '';
        }
 
-       $prefix .= "&nbsp;-&gt;&nbsp;";
+       $prefix .= '&nbsp;-&gt;&nbsp;';
 
        // We need to remove .php and the end
        if (substr($search, -4, 4) == '.php') {
@@ -335,7 +348,7 @@ function ADD_DESCR ($ACC_LVL, $FQFN, $return = false, $output = true) {
 
        // Get the title from menu
        $result = SQL_QUERY_ESC("SELECT title FROM `{!_MYSQL_PREFIX!}_%s_menu` WHERE %s='%s' ".$AND." LIMIT 1",
-       array($ACC_LVL, $type, $search), __FUNCTION__, __LINE__);
+       array($accessLevel, $type, $search), __FUNCTION__, __LINE__);
 
        // Menu found?
        if (SQL_NUMROWS($result) == 1) {
@@ -348,18 +361,18 @@ function ADD_DESCR ($ACC_LVL, $FQFN, $return = false, $output = true) {
                        return $ret;
                } elseif (((GET_EXT_VERSION('sql_patches') >= '0.2.3') && (getConfig('youre_here') == 'Y')) || ((IS_ADMIN()) && ($modCheck == 'admin'))) {
                        // Output HTML code
-                       $OUT = $prefix."<strong><a class=\"you_are_here\" href=\"{!URL!}/modules.php?module=".$modCheck.'&amp;'.$type.'='.$search.$LINK_ADD."\">".$ret."</a></strong>\n";
+                       $OUT = $prefix . "<strong><a class=\"you_are_here\" href=\"{!URL!}/modules.php?module=" . $modCheck . '&amp;' . $type . '=' . $search.$LINK_ADD . "\">" . $ret . "</a></strong>\n";
 
                        // Can we close the you-are-here navigation?
-                       //* DEBUG: */ echo __LINE__."*".$type.'/'.$GLOBALS['what']."*<br />\n";
-                       if (($type == 'what') || (($type == 'action') && ((!isset($GLOBALS['what'])) || ($GLOBALS['what'] == "overview")))) {
+                       //* DEBUG: */ echo __LINE__."*".$type.'/'.getWhat()."*<br />\n";
+                       if (($type == 'what') || (($type == 'action') && ((!isWhatSet()) || (getWhat() == 'overview')))) {
                                //* DEBUG: */ echo __LINE__.'+'.$type."+<br />\n";
                                // Add closing div and br-tag
                                $OUT .= "</div><br />\n";
                                $GLOBALS['nav_depth'] = '0';
 
                                // Run the filter chain
-                               $ret = runFilterChain('post_youhere_line', array('access_level' => $ACC_LVL, 'type' => $type, 'content' => ""));
+                               $ret = runFilterChain('post_youhere_line', array('access_level' => $accessLevel, 'type' => $type, 'content' => ""));
                                $OUT .= $ret['content'];
                        } // END - if
                }
@@ -397,24 +410,24 @@ function ADD_MENU ($mode, $act, $wht) {
        } // END - if
 
        // Load SQL data and add the menu to the output stream...
-       $result_main = SQL_QUERY_ESC("SELECT title, action FROM `{!_MYSQL_PREFIX!}_%s_menu` WHERE (`what`='' OR `what` IS NULL)".$AND." ORDER BY `sort`",
-       array($mode), __FUNCTION__, __LINE__);
-       //* DEBUG: */ echo __LINE__.'/'.$main_cnt.'/'.$main_action.'/'.$sub_what.':'.$GLOBALS['what']."*<br />\n";
+       $result_main = SQL_QUERY_ESC("SELECT `title`, `action` FROM `{!_MYSQL_PREFIX!}_%s_menu` WHERE (`what`='' OR `what` IS NULL)".$AND." ORDER BY `sort`",
+               array($mode), __FUNCTION__, __LINE__);
+       //* DEBUG: */ echo __LINE__.'/'.$main_cnt.'/'.$main_action.'/'.$sub_what.':'.getWhat()."*<br />\n";
        if (SQL_NUMROWS($result_main) > 0) {
                OUTPUT_HTML("<table border=\"0\" cellspacing=\"0\" cellpadding=\"0\" class=\"".$mode."_menu\">");
                // There are menus available, so we simply display them... :)
                while ($content = SQL_FETCHARRAY($result_main)) {
-                       //* DEBUG: */ echo __LINE__.'/'.$main_cnt.'/'.$content['action'].'/'.$sub_what.':'.$GLOBALS['what']."*<br />\n";
+                       //* DEBUG: */ echo __LINE__.'/'.$main_cnt.'/'.$content['action'].'/'.$sub_what.':'.getWhat()."*<br />\n";
                        // Init variables
-                       $GLOBALS['block_mode'] = false;
+                       enableBlockMode(false);
                        $act = $content['action'];
 
                        // Load menu header template
-                       LOAD_TEMPLATE($mode."_menu_title", false, $content);
+                       LOAD_TEMPLATE($mode . '_menu_title', false, $content);
 
                        // Sub menu
-                       $result_sub = SQL_QUERY_ESC("SELECT title AS sub_title, what AS sub_what FROM `{!_MYSQL_PREFIX!}_%s_menu` WHERE `action`='%s' AND `what` != '' AND `what` IS NOT NULL ".$AND." ORDER BY `sort`",
-                       array($mode, $content['action']), __FUNCTION__, __LINE__);
+                       $result_sub = SQL_QUERY_ESC("SELECT `title` AS sub_title, `what` AS sub_what FROM `{!_MYSQL_PREFIX!}_%s_menu` WHERE `action`='%s' AND `what` != '' AND `what` IS NOT NULL ".$AND." ORDER BY `sort`",
+                               array($mode, $content['action']), __FUNCTION__, __LINE__);
 
                        // Get number of rows
                        $ctl = SQL_NUMROWS($result_sub);
@@ -441,7 +454,7 @@ function ADD_MENU ($mode, $act, $wht) {
                                                }
 
                                                // Navigation link
-                                               $OUT .= "<a name=\"menu\" class=\"menu_blur\" href=\"{!URL!}/modules.php?module=".$GLOBALS['module']."&amp;what=".$content['sub_what']."\" target=\"_self\">";
+                                               $OUT .= "<a name=\"menu\" class=\"menu_blur\" href=\"{!URL!}/modules.php?module=".getModule()."&amp;what=".$content['sub_what']."\" target=\"_self\">";
                                        } else {
                                                $OUT .= "<em>";
                                        }
@@ -477,23 +490,25 @@ function ADD_MENU ($mode, $act, $wht) {
                                }
                        } else {
                                // This is a menu block... ;-)
-                               $GLOBALS['block_mode'] = true;
+                               enableBlockMode();
+
+                               // Load menu block
                                $INC_BLOCK = sprintf("inc/modules/%s/action-%s.php", $mode, $content['action']);
                                if (isFileReadable($INC_BLOCK)) {
                                        // Load include file
                                        if ((!EXT_IS_ACTIVE($content['action'])) || ($content['action'] == 'online')) OUTPUT_HTML("<tr>
 
   <td class=\"".$mode."_menu_whats\">");
-                                       //* DEBUG: */ echo __LINE__.'/'.$main_cnt.'/'.$content['action'].'/'.$content['sub_what'].':'.$GLOBALS['what']."*<br />\n";
+                                       //* DEBUG: */ echo __LINE__.'/'.$main_cnt.'/'.$content['action'].'/'.$content['sub_what'].':'.getWhat()."*<br />\n";
                                        loadInclude($INC_BLOCK);
-                                       //* DEBUG: */ echo __LINE__.'/'.$main_cnt.'/'.$content['action'].'/'.$content['sub_what'].':'.$GLOBALS['what']."*<br />\n";
+                                       //* DEBUG: */ echo __LINE__.'/'.$main_cnt.'/'.$content['action'].'/'.$content['sub_what'].':'.getWhat()."*<br />\n";
                                        if ((!EXT_IS_ACTIVE($content['action'])) || ($content['action'] == 'online')) OUTPUT_HTML("  </td>
 </tr>");
                                }
-                               //* DEBUG: */ echo __LINE__.'/'.$main_cnt.'/'.$content['action'].'/'.$content['sub_what'].':'.$GLOBALS['what']."*<br />\n";
+                               //* DEBUG: */ echo __LINE__.'/'.$main_cnt.'/'.$content['action'].'/'.$content['sub_what'].':'.getWhat()."*<br />\n";
                        }
                        $main_cnt++;
-                       //* DEBUG: */ echo __LINE__.'/'.$main_cnt.'/'.$content['action'].'/'.$content['sub_what'].':'.$GLOBALS['what']."*<br />\n";
+                       //* DEBUG: */ echo __LINE__.'/'.$main_cnt.'/'.$content['action'].'/'.$content['sub_what'].':'.getWhat()."*<br />\n";
                        if (SQL_NUMROWS($result_main) > $main_cnt) {
                                OUTPUT_HTML("<tr><td class=\"".$mode."_menu_seperator\"></td></tr>");
                        } // END - if
@@ -503,7 +518,7 @@ function ADD_MENU ($mode, $act, $wht) {
                SQL_FREERESULT($result_main);
 
                // Close table
-               //* DEBUG: */ echo __LINE__.'/'.$main_cnt.'/'.$content['action'].'/'.$content['sub_what'].':'.$GLOBALS['what']."*<br />\n";
+               //* DEBUG: */ echo __LINE__.'/'.$main_cnt.'/'.$content['action'].'/'.$content['sub_what'].':'.getWhat()."*<br />\n";
                OUTPUT_HTML("</table>");
        }
 }
@@ -731,7 +746,7 @@ function isMenuActionValid ($mode, $act, $wht, $UPDATE=false) {
                // Update guest or member menu
                $sql = SQL_QUERY_ESC("UPDATE `{!_MYSQL_PREFIX!}_%s_menu` SET counter=counter+1 WHERE `action`='%s' AND `what`='%s'".$add." LIMIT 1",
                array($mode, $act, $wht), __FUNCTION__, __LINE__, false);
-       } elseif (($wht != "overview") && (!empty($wht))) {
+       } elseif (($wht != 'overview') && (!empty($wht))) {
                // Other actions
                $sql = SQL_QUERY_ESC("SELECT id, what FROM `{!_MYSQL_PREFIX!}_%s_menu` WHERE `action`='%s' AND `what`='%s'".$add." ORDER BY action DESC LIMIT 1",
                array($mode, $act, $wht), __FUNCTION__, __LINE__, false);
@@ -898,13 +913,13 @@ function countModuleHit($mod) {
 }
 
 // Get action value from mode (admin/guest/member) and what-value
-function getModeAction ($mode, &$wht) {
+function getModeAction ($mode, $wht) {
        // Init status
        $ret = '';
 
-       //* DEBUG: */ echo __LINE__.'='.$mode.'/'.$wht.'/'.$GLOBALS['action']."=<br />";
+       //* DEBUG: */ echo __LINE__.'='.$mode.'/'.$wht.'/'.getAction()."=<br />";
        if ((empty($wht)) && ($mode != 'admin')) {
-               $wht = "welcome";
+               $wht = 'welcome';
                if (getConfig('index_home') != '') $wht = getConfig('index_home');
        } // END - if
 
@@ -913,16 +928,16 @@ function getModeAction ($mode, &$wht) {
                if (REQUEST_ISSET_GET('action')) {
                        // Use from request!
                        return REQUEST_GET('action');
-               } elseif (!empty($GLOBALS['action'])) {
+               } elseif (isActionSet()) {
                        // Get it directly from URL
-                       return $GLOBALS['action'];
-               } elseif (($wht == "overview") || (empty($GLOBALS['what']))) {
+                       return getAction();
+               } elseif (($wht == 'overview') || (!isWhatSet())) {
                        // Default value for admin area
-                       $ret = "login";
+                       $ret = 'login';
                }
-       } elseif (!empty($GLOBALS['action'])) {
+       } elseif (isActionSet()) {
                // Get it directly from URL
-               return $GLOBALS['action'];
+               return getAction();
        }
        //* DEBUG: */ echo __FUNCTION__."(<font color=\"#0000aa\">".__LINE__."</font>): ret=".$ret."<br />\n";
 
@@ -931,11 +946,11 @@ function getModeAction ($mode, &$wht) {
                // Rewriting modules to menu
                switch ($mode) {
                        case 'index': $mode = 'guest';  break;
-                       case "login": $mode = 'member'; break;
+                       case 'login': $mode = 'member'; break;
                } // END - switch
 
-               // Guest and member menu is "main" as the default
-               if (empty($ret)) $ret = "main";
+               // Guest and member menu is 'main' as the default
+               if (empty($ret)) $ret = 'main';
 
                // Load from database
                $result = SQL_QUERY_ESC("SELECT action FROM `{!_MYSQL_PREFIX!}_%s_menu` WHERE `what`='%s' LIMIT 1",
@@ -1717,10 +1732,11 @@ function getWhatFromModule ($modCheck) {
                return REQUEST_GET('what');
        } // END - if
 
+       // Default is empty
        $wht = '';
+
        //* DEBUG: */ echo __LINE__.'!'.$modCheck."!<br />\n";
-       switch ($modCheck)
-       {
+       switch ($modCheck) {
                case 'admin':
                        $wht = 'overview';
                        break;
@@ -1734,7 +1750,7 @@ function getWhatFromModule ($modCheck) {
                default:
                        $wht = '';
                        break;
-       }
+       } // END - switch
 
        // Return what value
        return $wht;
@@ -1744,9 +1760,10 @@ function getWhatFromModule ($modCheck) {
 function SUB_POINTS ($subject, $uid, $points) {
        // Add points to used points
        SQL_QUERY_ESC("UPDATE `{!_MYSQL_PREFIX!}_user_data` SET `used_points`=`used_points`+%s WHERE userid=%s LIMIT 1",
-       array($points, bigintval($uid)), __FUNCTION__, __LINE__);
+               array($points, bigintval($uid)), __FUNCTION__, __LINE__);
 
        // Insert booking record
+       // @TODO Rewrite this to a filter
        if (EXT_IS_ACTIVE('booking')) {
                // Add record
                ADD_BOOKING_RECORD($subject, $uid, $points, 'sub');
@@ -1762,7 +1779,7 @@ function SUB_POINTS ($subject, $uid, $points) {
 // Update config entries
 function updateConfiguration ($entries, $values, $updateMode='') {
        // Do not update config in CSS mode
-       if (($GLOBALS['output_mode'] == '1') || ($GLOBALS['output_mode'] == -1)) {
+       if ((getOutputMode() == '1') || (getOutputMode() == -1)) {
                return;
        } // END - if