Next try to fix it (simple quotes)

diff --git a/inc/libs/rewrite_functions.php b/inc/libs/rewrite_functions.php
index 121f0c4579a3e063d6840658856b46b0f1bb0a25..fb52234bc95b3370a85d26136b0e0f529a3ef56a 100644 (file)
--- a/inc/libs/rewrite_functions.php
+++ b/inc/libs/rewrite_functions.php
@@ -42,40 +42,40 @@ if (!defined('__SECURITY')) {
        require($INC);
 }
 
-//
+// Rewrite links in HTML for better SEO (ugly part of mx!)
 function REWRITE_LINKS ($HTML) {
-       // Skip rewriting for the admin area (which search engine may enter here???)
-       if (ereg($GLOBALS['module'], getConfig('rewrite_skip'))) return $HTML;
+       // Skip rewriting for configured modules
+       if (eregi($GLOBALS['module'], getConfig('rewrite_skip')) !== false) return $HTML;
 
        // Strip slashes with double-backslashes for the preg_replace() function
-       $URL = str_replace("/", "\\/", constant('URL'));
+       $URL = str_replace('/', '\/', constant('URL'));
 
        // Generate target URL
        $target = constant('URL')."/cms/";
 
        // Convert modules.php?module=...
-       $output = preg_replace("/".$URL."\\/modules.php\\?module=/i", $target, $HTML);
+       $output = preg_replace('/' . $URL . '\/modules.php\?module=/i', $target, $HTML);
 
        if (eregi(constant('SERVER_URL'), $output)) {
                // Strip slashes with double-backslashes for the preg_replace() function
-               $URL = str_replace("/", "\\/", constant('SERVER_URL'));
+               $URL = str_replace('/', '\/', constant('SERVER_URL'));
 
                // Convert URLs from my server
-               $output = preg_replace("/".$URL."\\/modules.php\\?module=/i", constant('SERVER_URL')."/cms/", $output);
+               $output = preg_replace('/' . $URL . '\/modules.php\?module=/i', constant('SERVER_URL')."/cms/", $output);
        } // END - if
 
        // Strip slashes as above for the main URL
-       $target2 = str_replace("/", "\\/", $target);
+       $target2 = str_replace('/', '\/', $target);
 
        // Action variable
        $act = $target."\$1/act/";
 
        // Convert &|&action=...
-       $output = preg_replace("/".$target2."(.*)&action=/i", $act, $output);
+       $output = preg_replace('/'.$target2."(.*)&action=/i", $act, $output);
 
        // "The same procedure as last variable"... now for &what=
        $wht = $target."\$1/wht/";
-       $output = preg_replace("/".$target2."(.*)&what=/i", $wht, $output);
+       $output = preg_replace('/'.$target2."(.*)&what=/i", $wht, $output);
 
        // Repair missed &what=??? entries
        while (preg_match("/&what=(.*)\/(.*)\/(.*)/i", $output)) {