diff --git a/classes/controller/class.modrewrite_content_controller.php b/classes/controller/class.modrewrite_content_controller.php index ed75e19..18e23b3 100644 --- a/classes/controller/class.modrewrite_content_controller.php +++ b/classes/controller/class.modrewrite_content_controller.php @@ -27,12 +27,6 @@ if (!defined('CON_FRAMEWORK')) { */ class ModRewrite_ContentController extends ModRewrite_ControllerAbstract { - public $_oView; - public $_client; - /** - * @var array - */ - public $_cfg; /** * Index action */ @@ -47,12 +41,13 @@ class ModRewrite_ContentController extends ModRewrite_ControllerAbstract { public function saveAction() { $bDebug = $this->getProperty('bDebug'); $aSeparator = $this->getProperty('aSeparator'); + $aWordSeparator = $this->getProperty('aWordSeparator'); $routingSeparator = $this->getProperty('routingSeparator'); $bError = false; - $aMR = []; + $aMR = array(); - $request = ($_POST !== []) ? $_POST : $_GET; + $request = (count($_POST) > 0) ? $_POST : $_GET; mr_requestCleanup($request); // use cl-mod-rewrite @@ -326,10 +321,10 @@ class ModRewrite_ContentController extends ModRewrite_ControllerAbstract { // routing if (isset($request['rewrite_routing'])) { - $aRouting = []; + $aRouting = array(); $items = explode("\n", $request['rewrite_routing']); - foreach ($items as $item) { - $routingDef = explode($routingSeparator, $item); + foreach ($items as $p => $v) { + $routingDef = explode($routingSeparator, $v); if (count($routingDef) !== 2) { continue; } @@ -344,7 +339,7 @@ class ModRewrite_ContentController extends ModRewrite_ControllerAbstract { $aMR['cl-mod-rewrite']['routing'] = $aRouting; } else { $this->_oView->rewrite_routing = ''; - $aMR['cl-mod-rewrite']['routing'] = []; + $aMR['cl-mod-rewrite']['routing'] = array(); } // redirect invalid article to errorsite @@ -390,8 +385,9 @@ class ModRewrite_ContentController extends ModRewrite_ControllerAbstract { /** * Checks, if any sseparators setting is modified or not * @param array $aNewCfg New configuration send by requests. + * @return bool */ - protected function _separatorModified($aNewCfg): bool { + protected function _separatorModified($aNewCfg) { $aCfg = ModRewrite::getConfig(); if ($aCfg['category_seperator'] != $aNewCfg['category_seperator']) { @@ -409,19 +405,18 @@ class ModRewrite_ContentController extends ModRewrite_ControllerAbstract { /** * Does some checks like 'is_start_compatible' check. * Adds notifications, if something will went wrong... - * - * @todo review text and translations */ protected function _doChecks() { + // Check for not supported '$cfg["is_start_compatible"] = true;' mode if (!empty($this->_cfg['is_start_compatible']) && true === $this->_cfg['is_start_compatible']) { $sMsg = i18n("Your Contenido installation runs with the setting 'is_start_compatible'. This plugin will not work properly in this mode.
Please check following topic in Contenido forum to change this:

is_start_compatible auf neue Version umstellen", "cl-mod-rewrite"); $this->_oView->content_before .= $this->_notifyBox('warning', $sMsg); } // Check for empty urlpath entries in cat_lang table - $dbConLite = new DB_ConLite(); + $db = new DB_Contenido(); $sql = "SELECT idcatlang FROM " . $this->_cfg['tab']['cat_lang'] . " WHERE urlpath = ''"; - if ($dbConLite->query($sql) && $dbConLite->next_record()) { + if ($db->query($sql) && $db->next_record()) { $sMsg = i18n("It seems as if some categories don't have a set 'urlpath' entry in the database. Please reset empty aliases in %sFunctions%s area.", "cl-mod-rewrite"); $sMsg = sprintf($sMsg, '', ''); $this->_oView->content_before .= $this->_notifyBox('warning', $sMsg); diff --git a/classes/controller/class.modrewrite_contentexpert_controller.php b/classes/controller/class.modrewrite_contentexpert_controller.php index e49daa1..a0ae01f 100644 --- a/classes/controller/class.modrewrite_contentexpert_controller.php +++ b/classes/controller/class.modrewrite_contentexpert_controller.php @@ -77,9 +77,17 @@ class ModRewrite_ContentExpertController extends ModRewrite_ControllerAbstract { return; } - $source = $type == 'restrictive' ? $this->_htaccessRestrictive : $this->_htaccessSimple; + if ($type == 'restrictive') { + $source = $this->_htaccessRestrictive; + } else { + $source = $this->_htaccessSimple; + } - $dest = $copy == 'contenido' ? $aInfo['contenido_full_path'] . '.htaccess' : $aInfo['client_full_path'] . '.htaccess'; + if ($copy == 'contenido') { + $dest = $aInfo['contenido_full_path'] . '.htaccess'; + } else { + $dest = $aInfo['client_full_path'] . '.htaccess'; + } if (!$result = @copy($source, $dest)) { $this->_oView->content_before = $this->_notifyBox('info', 'Die .htaccess konnte nicht von ' . $source . ' nach ' . $dest . ' kopiert werden!'); @@ -100,12 +108,16 @@ class ModRewrite_ContentExpertController extends ModRewrite_ControllerAbstract { return; } - $source = $type == 'restrictive' ? $this->_htaccessRestrictive : $this->_htaccessSimple; + if ($type == 'restrictive') { + $source = $this->_htaccessRestrictive; + } else { + $source = $this->_htaccessSimple; + } $this->_oView->content = file_get_contents($source); header('Content-Type: text/plain'); - header('Etag: ' . md5(random_int(0, mt_getrandmax()))); + header('Etag: ' . md5(mt_rand())); header('Content-Disposition: attachment; filename="' . $type . '.htaccess"'); $this->render('{CONTENT}'); } diff --git a/classes/controller/class.modrewrite_contenttest_controller.php b/classes/controller/class.modrewrite_contenttest_controller.php index 0b0c5fd..1f095ee 100644 --- a/classes/controller/class.modrewrite_contenttest_controller.php +++ b/classes/controller/class.modrewrite_contenttest_controller.php @@ -60,25 +60,33 @@ class ModRewrite_ContentTestController extends ModRewrite_ControllerAbstract { $this->_oView->content = ''; // Array for testcases - $aTests = []; + $aTests = array(); // Instance of mr test - $modRewriteTest = new ModRewriteTest($this->_iMaxItems); + $oMRTest = new ModRewriteTest($this->_iMaxItems); $startTime = getmicrotime(); // Fetch complete CONTENIDO page structure - $aStruct = $modRewriteTest->fetchFullStructure(); + $aStruct = $oMRTest->fetchFullStructure(); ModRewriteDebugger::add($aStruct, 'ModRewrite_ContentTestController::testAction() $aStruct'); // Loop through the structure and compose testcases - foreach ($aStruct as $aCat) { + foreach ($aStruct as $idcat => $aCat) { // category - $aTests[] = ['url' => $modRewriteTest->composeURL($aCat, 'c'), 'level' => $aCat['level'], 'name' => $aCat['name']]; + $aTests[] = array( + 'url' => $oMRTest->composeURL($aCat, 'c'), + 'level' => $aCat['level'], + 'name' => $aCat['name'] + ); - foreach ($aCat['articles'] as $aArt) { + foreach ($aCat['articles'] as $idart => $aArt) { // articles - $aTests[] = ['url' => $modRewriteTest->composeURL($aArt, 'a'), 'level' => $aCat['level'], 'name' => $aCat['name'] . ' :: ' . $aArt['title']]; + $aTests[] = array( + 'url' => $oMRTest->composeURL($aArt, 'a'), + 'level' => $aCat['level'], + 'name' => $aCat['name'] . ' :: ' . $aArt['title'] + ); } } @@ -88,7 +96,7 @@ class ModRewrite_ContentTestController extends ModRewrite_ControllerAbstract { $oMRUrlStack = ModRewriteUrlStack::getInstance(); // first loop to add urls to mr url stack - foreach ($aTests as $v) { + foreach ($aTests as $p => $v) { $oMRUrlStack->add($v['url']); } @@ -96,15 +104,15 @@ class ModRewrite_ContentTestController extends ModRewrite_ControllerAbstract { $failCounter = 0; // second loop to do the rest - foreach ($aTests as $aTest) { - $url = mr_buildNewUrl($aTest['url']); - $arr = $modRewriteTest->resolveUrl($url); + foreach ($aTests as $p => $v) { + $url = mr_buildNewUrl($v['url']); + $arr = $oMRTest->resolveUrl($url); $error = ''; - $resUrl = $modRewriteTest->getResolvedUrl(); + $resUrl = $oMRTest->getResolvedUrl(); $color = 'green'; if ($url !== $resUrl) { - if ($modRewriteTest->getRoutingFoundState()) { + if ($oMRTest->getRoutingFoundState()) { $successCounter++; $resUrl = 'route to -> ' . $resUrl; } else { @@ -136,18 +144,18 @@ class ModRewrite_ContentTestController extends ModRewrite_ControllerAbstract { } } - $pref = str_repeat(' ', $aTest['level']); + $pref = str_repeat(' ', $v['level']); // render resolve information for current item $itemTpl = $this->_oView->lng_result_item_tpl; $itemTpl = str_replace('{pref}', $pref, $itemTpl); - $itemTpl = str_replace('{name}', $aTest['name'], $itemTpl); - $itemTpl = str_replace('{url_in}', $aTest['url'], $itemTpl); + $itemTpl = str_replace('{name}', $v['name'], $itemTpl); + $itemTpl = str_replace('{url_in}', $v['url'], $itemTpl); $itemTpl = str_replace('{url_out}', $url, $itemTpl); $itemTpl = str_replace('{color}', $color, $itemTpl); $itemTpl = str_replace('{url_res}', $resUrl, $itemTpl); $itemTpl = str_replace('{err}', $error, $itemTpl); - $itemTpl = str_replace('{data}', $modRewriteTest->getReadableResolvedData($arr), $itemTpl); + $itemTpl = str_replace('{data}', $oMRTest->getReadableResolvedData($arr), $itemTpl); $this->_oView->content .= "\n" . $itemTpl . "\n"; } diff --git a/classes/controller/class.modrewrite_controller_abstract.php b/classes/controller/class.modrewrite_controller_abstract.php index de104f9..cf1bb01 100644 --- a/classes/controller/class.modrewrite_controller_abstract.php +++ b/classes/controller/class.modrewrite_controller_abstract.php @@ -72,13 +72,13 @@ abstract class ModRewrite_ControllerAbstract { * Template file or template string to render * @var string */ - protected $_template; + protected $_template = null; /** * Additional properties list * @var array */ - protected $_properties = []; + protected $_properties = array(); /** * Debug flag @@ -138,18 +138,20 @@ abstract class ModRewrite_ControllerAbstract { /** * Property setter. * @param string $key + * @param mixed $value */ - public function setProperty($key, mixed $value) { + public function setProperty($key, $value) { $this->_properties[$key] = $value; } /** * Property getter. * @param string $key + * @param mixed $default * @return mixed */ - public function getProperty($key, mixed $default = null) { - return $this->_properties[$key] ?? $default; + public function getProperty($key, $default = null) { + return (isset($this->_properties[$key])) ? $this->_properties[$key] : $default; } /** @@ -200,7 +202,7 @@ abstract class ModRewrite_ControllerAbstract { * @param mixed $default The default value * @return mixed */ - protected function _getParam($key, mixed $default = null) { + protected function _getParam($key, $default = null) { if (isset($_GET[$key])) { return $_GET[$key]; } elseif (isset($_POST[$key])) { diff --git a/includes/functions.mod_rewrite.php b/includes/functions.mod_rewrite.php index 59772c1..a787210 100644 --- a/includes/functions.mod_rewrite.php +++ b/includes/functions.mod_rewrite.php @@ -23,10 +23,6 @@ * @link http://www.4fb.de * @link http://www.contenido.org */ - -use ConLite\Log\LogWriter; -use ConLite\Log\Log; - if (!defined('CON_FRAMEWORK')) { die('Illegal call'); } @@ -683,9 +679,9 @@ function mr_setConfiguration($clientId, array $config) { try { mkdir($sConfigClientPath, 0777, true); } catch (Exception $ex) { - $writer = LogWriter::factory("File", array('destination' => 'contenido.log')); - $log = new Log($writer); - $log->log($ex->getFile() . " (" . $ex->getLine() . "): " . $ex->getMessage(), Log::WARN); + $oWriter = cLogWriter::factory("File", array('destination' => 'contenido.log')); + $oLog = new cLog($oWriter); + $oLog->log($ex->getFile() . " (" . $ex->getLine() . "): " . $ex->getMessage(), cLog::WARN); } }