Restructured the config file & other minor tweaks (#99)

* Restructured the config file & other minor tweaks

* Fixed the tests

Author: BelleNottelling

.github/workflows/ci.yml

@@ -38,8 +38,6 @@ jobs:
       - name: Install Dependencies
         run: composer install --no-interaction --prefer-dist --optimize-autoloader
 
-      - run: cp ./tests/Includes/Config.yaml ./src/Config/Config.yaml
-
       - run: php src/Vendor/bin/phpunit tests 
 
   PHPStan:

readme.md

@@ -1,4 +1,5 @@
 ![PHPStan Level](https://img.shields.io/badge/PHPStan-level%205-brightgreen)
+[![CI](https://github.com/AntCMS-org/AntCMS/actions/workflows/ci.yml/badge.svg)](https://github.com/AntCMS-org/AntCMS/actions/workflows/ci.yml)
 ![Supported PHP Versions](https://img.shields.io/badge/PHP%20Versions-8.0%7C8.1%7C8.2%7C8.3-brightgreen)
 
 # AntCMS

src/AntCMS/Auth.php

@@ -76,7 +76,7 @@ private function requireAuth(): void
         setcookie("auth", "valid");
 
         $siteInfo = Config::get('siteInfo');
-        header('WWW-Authenticate: Basic realm="' . $siteInfo['siteTitle'] . '"');
+        header('WWW-Authenticate: Basic realm="' . $siteInfo['title'] . '"');
         http_response_code(401);
         echo 'You must enter a valid username and password to access this page';
         exit;

src/AntCMS/Cache.php

@@ -16,7 +16,7 @@ class Cache
      */
     public function __construct(null|string $mode = null)
     {
-        $mode ??= Config::get('cacheMode') ?? 'auto';
+        $mode ??= Config::get('performance.cacheMode') ?? 'auto';
         if ($mode == 'auto') {
             $mode = function_exists('apcu_enabled') && apcu_enabled() ? 'apcu' : 'filesystem';
         }

src/AntCMS/Config.php

@@ -11,8 +11,7 @@ class Config
         'performance',
         'forceHttps',
         'activeTheme',
-        'cacheMode',
-        'debug',
+        'debugLevel',
         'baseUrl',
         'embed',
     ];
@@ -24,17 +23,17 @@ public static function generateConfig(): void
     {
         $defaultOptions = [
             'siteInfo' => [
-                'siteTitle' => 'AntCMS',
+                'title' => 'AntCMS',
             ],
             'performance' => [
                 'doOutputCompression' => true,
                 'compressTextAssets' => true,
                 'compressImageAssets' => true,
+                'cacheMode' => 'auto',
             ],
             'forceHttps' => !Enviroment::isPHPDevServer(),
             'activeTheme' => 'Default',
-            'cacheMode' => 'auto',
-            'debug' => true,
+            'debugLevel' => 1, // 0-2 at the moment
             'baseUrl' => $_SERVER['HTTP_HOST'] . dirname($_SERVER['PHP_SELF']),
             'embed' => [
                 'allowed_domains' => ['youtube.com', 'twitter.com', 'github.com', 'vimeo.com', 'flickr.com', 'instagram.com', 'facebook.com'],

src/AntCMS/Pages.php

@@ -20,7 +20,7 @@ private static function generatePageInfo(string $path): array
             'title' => $pageHeader['title'],
             'realPath' => $path,
             'functionalPath' => $functionalPath,
-            'url' => "//" . Tools::repairURL(baseUrl . $functionalPath),
+            'url' => "//" . Tools::repairURL(BASE_URL . $functionalPath),
             'active' => $functionalPath === self::$currentPage,
             'navItem' => $pageHeader['NavItem'] !== 'false',
         ];

src/AntCMS/Tools.php

@@ -190,7 +190,7 @@ private static function compressImage(string $path, int $quality = 85): string
     public static function getAssetCacheKey(string $path): string
     {
         $encoding = CompressionBuffer::getFirstMethodChoice();
-        if (compressTextAssets && self::isCompressableTextAsset($path)) {
+        if (COMPRESS_TEXT_ASSETS && self::isCompressableTextAsset($path)) {
             return Cache::createCacheKeyFile($path, "assetCompression-$encoding");
         }
         return Cache::createCacheKeyFile($path, 'asset');
@@ -206,7 +206,7 @@ public static function doAssetCompression(string $path): array
         $cacheKey = self::getAssetCacheKey($path);
         $encoding = 'identity';
 
-        if (compressTextAssets && self::isCompressableTextAsset($path)) {
+        if (COMPRESS_TEXT_ASSETS && self::isCompressableTextAsset($path)) {
             CompressionBuffer::enable(); // We will use CompressionBuffer to handle text content
             $encoding = CompressionBuffer::getFirstMethodChoice();
 
@@ -217,7 +217,7 @@ public static function doAssetCompression(string $path): array
             });
         }
 
-        if (compressImageAssets && self::isCompressableImage($path)) {
+        if (COMPRESS_IMAGES && self::isCompressableImage($path)) {
             $contents = $cache->get($cacheKey, function (ItemInterface $item) use ($path): string {
                 $item->expiresAfter(604800);
                 return self::compressImage($path);
@@ -245,6 +245,10 @@ private static function createDebugLogLine(string $wording, bool|string $value):
 
     public static function buildDebugInfo(): string
     {
+        if (DEBUG_LEVEL === 0) {
+            return '';
+        }
+
         $elapsed_time = round((hrtime(true) - START) / 1e+6, 2);
         $mem_usage = round(memory_get_peak_usage() / 1e+6, 2);
 
@@ -253,21 +257,23 @@ public static function buildDebugInfo(): string
         $result .= self::createDebugLogLine('Time to process request', "$elapsed_time ms");
         $result .= self::createDebugLogLine('Memory usage', "$mem_usage MB");
 
-        // System info
-        $result .= "<dt>System Info</dt>";
-        $result .= self::createDebugLogLine('Output compression', doOutputCompression);
-
-        if (CompressionBuffer::isEnabled() && doOutputCompression) {
-            $method = CompressionBuffer::getFirstMethodChoice();
-            if ($method === 'br') {
-                $method = 'brotli';
+        if (DEBUG_LEVEL >= 2) {
+            // System info
+            $result .= "<dt>System Info</dt>";
+            $result .= self::createDebugLogLine('Output compression', COMPRESS_OUTPUT);
+
+            if (CompressionBuffer::isEnabled() && COMPRESS_OUTPUT) {
+                $method = CompressionBuffer::getFirstMethodChoice();
+                if ($method === 'br') {
+                    $method = 'brotli';
+                }
+                $result .= self::createDebugLogLine('This page was compressed with', $method);
             }
-            $result .= self::createDebugLogLine('This page was compressed with', $method);
-        }
 
-        $result .= self::createDebugLogLine('Asset compression', compressTextAssets);
-        $result .= self::createDebugLogLine('Image compression', compressImageAssets);
-        $result .= self::createDebugLogLine('PHP version', PHP_VERSION);
+            $result .= self::createDebugLogLine('Asset compression', COMPRESS_TEXT_ASSETS);
+            $result .= self::createDebugLogLine('Image compression', COMPRESS_IMAGES);
+            $result .= self::createDebugLogLine('PHP version', PHP_VERSION);
+        }
 
         return $result . "</dl>";
     }

src/AntCMS/Twig.php

@@ -28,13 +28,13 @@ public static function registerTwig(?Environment $twigEnvironment = null): void
                     new ArrayLoader(),
                 ]),
                 [
-                    'cache' => (Config::get('enableCache') !== 'none') ? PATH_CACHE : false,
-                    'debug' => Config::get('debug'),
+                    'cache' => (Config::get('performance.cacheMode') !== 'none') ? PATH_CACHE : false,
+                    'debug' => DEBUG_LEVEL >= 2,
                     'use_yield' => false,
                 ]
             );
             $twigEnvironment->addExtension(new TwigFilters());
-            $twigEnvironment->addGlobal('AntCMSSiteTitle', Config::get('siteInfo')['siteTitle']);
+            $twigEnvironment->addGlobal('AntCMSSiteTitle', Config::get('siteInfo')['title']);
             self::$twigEnvironment = $twigEnvironment;
         }
     }

src/AntCMS/TwigFilters.php

@@ -17,7 +17,7 @@ public function getFilters(): array
 
     public function absUrl(string $relative): string
     {
-        return '//' . Tools::repairURL(baseUrl . '/' . trim($relative));
+        return '//' . Tools::repairURL(BASE_URL . '/' . trim($relative));
     }
 
     public function markdown(string $content): string

src/Bootstrap.php

@@ -43,7 +43,8 @@
 
 // Define config-related constants
 $config = Config::get();
-define('compressTextAssets', $config['performance']['compressTextAssets']);
-define('doOutputCompression', $config['performance']['doOutputCompression']);
-define('compressImageAssets', $config['performance']['compressImageAssets']);
-define('baseUrl', $config['baseUrl']);
+define('COMPRESS_TEXT_ASSETS', $config['performance']['compressTextAssets']);
+define('COMPRESS_OUTPUT', $config['performance']['doOutputCompression']);
+define('COMPRESS_IMAGES', $config['performance']['compressImageAssets']);
+define('BASE_URL', $config['baseUrl']);
+define('DEBUG_LEVEL', $config['debugLevel']);