stakx-io / stakx

tests/StakxTestCase.php

Braces +15 added lines, -7 removed lines

@@ -72,7 +72,8 @@ 
 
     public function tearDown(): void
     {
-        if ($this->assetFolder !== null) {
+        if ($this->assetFolder !== null)
+        {
             fs::remove($this->assetFolder);
         }
     }
@@ -167,7 +168,8 @@ 
     {
         $body = $this->buildFrontMatterTemplate($frontMatter, $content);
 
-        if (!$filename) {
+        if (!$filename)
+        {
             $filename = hash('sha256', (string)random_int(0, mt_getrandmax()));
         }
 
@@ -194,7 +196,8 @@ 
     {
         $results = [];
 
-        foreach ($elements as $element) {
+        foreach ($elements as $element)
+        {
             $filename = $element['filename'] ?? null;
             $frontMatter = (!isset($element['frontmatter']) || empty($element['frontmatter'])) ? [] : $element['frontmatter'];
             $body = $element['body'] ?? 'Body Text';
@@ -405,7 +408,8 @@ 
             ->value()
         ;
 
-        if ($fxnToCall === null) {
+        if ($fxnToCall === null)
+        {
             throw new BadFunctionCallException(sprintf('No function by the name of "%s" in this class', $method));
         }
 
@@ -413,10 +417,14 @@ 
         $callUserFuncArray = [];
 
         /** @var ReflectionParameter $argument */
-        foreach ($arguments as $argument) {
-            if (isset($namedParams[$argument->getName()])) {
+        foreach ($arguments as $argument)
+        {
+            if (isset($namedParams[$argument->getName()]))
+            {
                 $callUserFuncArray[] = $namedParams[$argument->getName()];
-            } else {
+            }
+            else
+            {
                 $callUserFuncArray[] = $argument->getDefaultValue();
             }
         }

src/Compiler.php

Braces +26 added lines, -12 removed lines

@@ -176,7 +176,8 @@ 
      */
     public function compileAll(): void
     {
-        foreach ($this->pageViewsFlattened as &$pageView) {
+        foreach ($this->pageViewsFlattened as &$pageView)
+        {
             $this->compilePageView($pageView);
         }
     }
@@ -199,8 +200,10 @@ 
             'type' => $pageView->getType(),
         ]);
 
-        try {
-            switch ($pageView->getType()) {
+        try
+        {
+            switch ($pageView->getType())
+            {
                 case BasePageView::STATIC_TYPE:
                     $this->compileStaticPageView($pageView);
                     $this->compileStandardRedirects($pageView);
@@ -219,7 +222,9 @@ 
 
                     break;
             }
-        } catch (TemplateErrorInterface $e) {
+        }
+        catch (TemplateErrorInterface $e)
+        {
             throw new FileAwareException(
                 $e->getMessage(),
                 $e->getCode(),
@@ -258,8 +263,10 @@ 
         $contentItems = $pageView->getCollectableItems();
         $template = $this->createTwigTemplate($pageView);
 
-        foreach ($contentItems as &$contentItem) {
-            if ($contentItem->isDraft() && !Service::hasRunTimeFlag(RuntimeStatus::USING_DRAFTS)) {
+        foreach ($contentItems as &$contentItem)
+        {
+            if ($contentItem->isDraft() && !Service::hasRunTimeFlag(RuntimeStatus::USING_DRAFTS))
+            {
                 $this->logger->debug('{file}: marked as a draft', [
                     'file' => $contentItem->getRelativeFilePath(),
                 ]);
@@ -289,7 +296,8 @@ 
         $template = $this->createTwigTemplate($pageView);
         $permalinks = $pageView->getRepeaterPermalinks();
 
-        foreach ($permalinks as $permalink) {
+        foreach ($permalinks as $permalink)
+        {
             $this->writeToFilesystem(
                 $pageView->getTargetFile($permalink->getEvaluated()),
                 $this->buildRepeaterPageViewHTML($template, $pageView, $permalink),
@@ -329,7 +337,8 @@ 
     {
         $redirects = $pageView->getRedirects();
 
-        foreach ($redirects as $redirect) {
+        foreach ($redirects as $redirect)
+        {
             $redirectPageView = BasePageView::createRedirect(
                 $redirect,
                 $pageView->getPermalink(),
@@ -361,9 +370,11 @@ 
         $permalinks = $pageView->getRepeaterPermalinks();
 
         /** @var ExpandedValue[] $repeaterRedirect */
-        foreach ($pageView->getRepeaterRedirects() as $repeaterRedirect) {
+        foreach ($pageView->getRepeaterRedirects() as $repeaterRedirect)
+        {
             /** @var ExpandedValue $redirect */
-            foreach ($repeaterRedirect as $index => $redirect) {
+            foreach ($repeaterRedirect as $index => $redirect)
+            {
                 $redirectPageView = BasePageView::createRedirect(
                     $redirect->getEvaluated(),
                     $permalinks[$index]->getEvaluated(),
@@ -482,7 +493,8 @@ 
      */
     private function createTwigTemplate(BasePageView &$pageView): TemplateInterface
     {
-        try {
+        try
+        {
             $template = $this->templateBridge->createTemplate($pageView->getContent());
 
             $this->templateMapping[$template->getTemplateName()] = $pageView->getRelativeFilePath();
@@ -491,7 +503,9 @@ 
             $this->eventDispatcher->dispatch($event, CompilerTemplateCreation::NAME);
 
             return $template;
-        } catch (TemplateErrorInterface $e) {
+        }
+        catch (TemplateErrorInterface $e)
+        {
             $e
                 ->setTemplateLine($e->getTemplateLine() + $pageView->getLineOffset())
                 ->setContent($pageView->getContent())

src/AssetEngine/AssetEngineManager.php

Braces +6 added lines, -3 removed lines

@@ -19,7 +19,8 @@ 
 
     public function addAssetEngines(/* iterable */ $assetEngines): void
     {
-        foreach ($assetEngines as $assetEngine) {
+        foreach ($assetEngines as $assetEngine)
+        {
             $this->addAssetEngine($assetEngine);
         }
     }
@@ -30,7 +31,8 @@ 
 
         $e = $this->engines[] = $assetEngine;
 
-        foreach ($extensions as $extension) {
+        foreach ($extensions as $extension)
+        {
             $this->enginesByExtension[$extension] = $e;
         }
 
@@ -44,7 +46,8 @@ 
 
     public function getEngineByExtension($extension)
     {
-        if (isset($this->enginesByExtension[$extension])) {
+        if (isset($this->enginesByExtension[$extension]))
+        {
             return $this->enginesByExtension[$extension];
         }
 

src/AssetEngine/Sass/SassEngine.php

Braces +22 added lines, -10 removed lines

@@ -84,7 +84,8 @@ 
         $this->handleThemeImports($content);
 
         // We don't need to write the source map to a file
-        if (!$this->fileSourceMap) {
+        if (!$this->fileSourceMap)
+        {
             $this->compiler->setSourceMapOptions($sourceMapOptions);
 
             return $this->compiler->compile($content);
@@ -138,13 +139,16 @@ 
 
     public static function stringToFormatter($format)
     {
-        if ($format === 'nested') {
+        if ($format === 'nested')
+        {
             return Nested::class;
         }
-        if ($format === 'expanded') {
+        if ($format === 'expanded')
+        {
             return Expanded::class;
         }
-        if ($format === 'compact') {
+        if ($format === 'compact')
+        {
             return Compact::class;
         }
 
@@ -167,12 +171,17 @@ 
     {
         $sourceMap = __::get($this->options, 'sourcemap');
 
-        if ($sourceMap === 'inline') {
+        if ($sourceMap === 'inline')
+        {
             $this->compiler->setSourceMap(Compiler::SOURCE_MAP_INLINE);
-        } elseif ($sourceMap === true) {
+        }
+        elseif ($sourceMap === true)
+        {
             $this->compiler->setSourceMap(Compiler::SOURCE_MAP_FILE);
             $this->fileSourceMap = true;
-        } else {
+        }
+        else
+        {
             $this->compiler->setSourceMap(Compiler::SOURCE_MAP_NONE);
         }
     }
@@ -187,14 +196,16 @@ 
 
     private function initializeCompiler(): void
     {
-        if ($this->compiler) {
+        if ($this->compiler)
+        {
             return;
         }
 
         $cacheOptions = [];
 
         // If we have a cache directory set, use it.
-        if ($this->cacheDirectory) {
+        if ($this->cacheDirectory)
+        {
             $cacheOptions = [
                 'cacheDir' => (string)$this->cacheDirectory->getFilesystemPath(),
                 'forceRefresh' => false,
@@ -210,7 +221,8 @@ 
 
     private function handleThemeImports(&$content): void
     {
-        if ($themeName = $this->configuration->getTheme()) {
+        if ($themeName = $this->configuration->getTheme())
+        {
             $themePath = "../_themes/{$themeName}/_sass";
             $content = preg_replace("/(@import ['\"])(@theme)(.+)/", "$1{$themePath}$3", $content);
         }

src/Server/RouteMapper.php

Braces +4 added lines, -2 removed lines

@@ -46,11 +46,13 @@
      */
     public function registerPageView(BasePageView $pageView): void
     {
-        switch ($pageView->getType()) {
+        switch ($pageView->getType())
+        {
             case BasePageView::STATIC_TYPE:
                 $this->mapping[$pageView->getPermalink()] = $pageView;
 
-                foreach ($pageView->getRedirects() as $redirect) {
+                foreach ($pageView->getRedirects() as $redirect)
+                {
                     $this->redirects[$redirect] = $pageView->getPermalink();
                 }
 

src/Server/ExceptionRenderer.php

Braces +2 added lines, -1 removed lines

@@ -20,7 +20,8 @@
         $source = fs::getInternalResource('error.html.twig');
         $template = $compiler->getTemplateBridge()->createTemplate($source);
 
-        if ($exception instanceof TemplateErrorInterface) {
+        if ($exception instanceof TemplateErrorInterface)
+        {
             $message = [
                 sprintf('File: %s:%d', $exception->getRelativeFilePath(), $exception->getTemplateLine()),
             ];

src/Server/WebServer.php

Braces +25 added lines, -11 removed lines

@@ -51,29 +51,36 @@ 
     {
         $routes = Controller::create($routeMapper, $compiler);
 
-        return new Server(function (ServerRequestInterface $request) use ($routes, $compiler, $assetManager) {
+        return new Server(function (ServerRequestInterface $request) use ($routes, $compiler, $assetManager)
+        {
             $httpMethod = $request->getMethod();
             $urlPath = Controller::normalizeUrl($request->getUri()->getPath());
 
             // We're a static website, we should never support anything other than GET requests
-            if ($httpMethod !== 'GET') {
+            if ($httpMethod !== 'GET')
+            {
                 return new Response(406, ['Content-Type' => 'text/plain'], 'Method not allowed');
             }
 
             $context = new RequestContext($urlPath);
             $matcher = new UrlMatcher($routes, $context);
 
-            try {
+            try
+            {
                 $parameters = $matcher->match($urlPath);
 
-                if (isset($parameters['_controller'])) {
+                if (isset($parameters['_controller']))
+                {
                     $controller = $parameters['_controller'];
 
                     return $controller($request, ...array_values($parameters));
                 }
-            } catch (ResourceNotFoundException $e) {
+            }
+            catch (ResourceNotFoundException $e)
+            {
                 // If we have a "manual" asset, let's serve from it
-                if (($file = $assetManager->getExplicitAsset(self::normalizePath($urlPath))) !== null) {
+                if (($file = $assetManager->getExplicitAsset(self::normalizePath($urlPath))) !== null)
+                {
                     return self::makeResponse($file);
                 }
 
@@ -82,10 +89,13 @@ 
                 // reason, we manually look through the filesystem and load from there.
                 //
                 // @TODO this should be fixed to make the AssetManager the authority on assets.
-                if (($asset = self::searchAsset($urlPath)) !== null) {
+                if (($asset = self::searchAsset($urlPath)) !== null)
+                {
                     return $asset;
                 }
-            } catch (Exception $e) {
+            }
+            catch (Exception $e)
+            {
                 $response = ExceptionRenderer::render($e, $compiler);
 
                 return new Response(500, ['Content-Type' => 'text/html'], $response);
@@ -103,12 +113,16 @@ 
      */
     private static function searchAsset($url, $continueNesting = true): ?Response
     {
-        try {
+        try
+        {
             $file = new File(self::normalizePath($url));
 
             return self::makeResponse($file);
-        } catch (FileNotFoundException) {
-            if (!$continueNesting) {
+        }
+        catch (FileNotFoundException)
+        {
+            if (!$continueNesting)
+            {
                 // This will evaluate to true if we've searched inside the themes folder and it still doesn't exist
                 return null;
             }

src/Server/Controller.php

Braces +36 added lines, -18 removed lines

@@ -51,7 +51,8 @@ 
          * @var string $from
          * @var string $to
          */
-        foreach ($routeMapper->getRedirectMapping() as $from => $to) {
+        foreach ($routeMapper->getRedirectMapping() as $from => $to)
+        {
             $routeName = preg_replace('/[\/\.]/', '_', $from);
             $routeName = 'redirect_' . $routeName;
 
@@ -64,7 +65,8 @@ 
          * @var string       $routeUrl
          * @var BasePageView $pageView
          */
-        foreach ($routeMapper->getRouteMapping() as $routeUrl => $pageView) {
+        foreach ($routeMapper->getRouteMapping() as $routeUrl => $pageView)
+        {
             $routeName = $pageView->getRelativeFilePath();
             $routeName = preg_replace('/[\/\.]/', '_', $routeName);
 
@@ -75,7 +77,8 @@ 
             // Allow the last route parameter to have `/` in the permalink that's not part of the route itself
             //   see https://github.com/stakx-io/stakx/issues/98
             $requirements = [];
-            if (count($results) >= 2) {
+            if (count($results) >= 2)
+            {
                 $requirements[$results[1]] = '.*';
             }
 
@@ -109,12 +112,14 @@ 
      */
     private function staticPageViewAction(StaticPageView $pageView, Compiler $compiler): Closure
     {
-        return function () use ($pageView, $compiler) {
+        return function () use ($pageView, $compiler)
+        {
             Service::setOption('currentTemplate', $pageView->getAbsoluteFilePath());
 
             $compiler->getTemplateBridge()->clearTemplateCache();
 
-            if ($this->hasBeenTouched($pageView)) {
+            if ($this->hasBeenTouched($pageView))
+            {
                 $pageView->readContent();
             }
 
@@ -133,22 +138,26 @@ 
      */
     private function dynamicPageViewAction(DynamicPageView $pageView, Compiler $compiler): Closure
     {
-        return function (ServerRequestInterface $request) use ($pageView, $compiler) {
+        return function (ServerRequestInterface $request) use ($pageView, $compiler)
+        {
             Service::setOption('currentTemplate', $pageView->getAbsoluteFilePath());
 
             $compiler->getTemplateBridge()->clearTemplateCache();
 
             $contentItem = self::getContentItem($pageView, $request->getUri()->getPath());
 
-            if ($contentItem === null) {
+            if ($contentItem === null)
+            {
                 return WebServer::return404();
             }
 
-            if ($this->hasBeenTouched($pageView)) {
+            if ($this->hasBeenTouched($pageView))
+            {
                 $pageView->readContent();
             }
 
-            if ($this->hasBeenTouched($contentItem)) {
+            if ($this->hasBeenTouched($contentItem))
+            {
                 $contentItem->readContent();
             }
 
@@ -161,18 +170,21 @@ 
      */
     private function repeaterPageViewAction(RepeaterPageView $pageView, Compiler $compiler): Closure
     {
-        return function (ServerRequestInterface $request) use ($pageView, $compiler) {
+        return function (ServerRequestInterface $request) use ($pageView, $compiler)
+        {
             Service::setOption('currentTemplate', $pageView->getAbsoluteFilePath());
 
             $compiler->getTemplateBridge()->clearTemplateCache();
 
             $expandedValue = self::getExpandedValue($pageView, $request->getUri()->getPath());
 
-            if ($expandedValue === null) {
+            if ($expandedValue === null)
+            {
                 return WebServer::return404();
             }
 
-            if ($this->hasBeenTouched($pageView)) {
+            if ($this->hasBeenTouched($pageView))
+            {
                 $pageView->readContent();
             }
 
@@ -189,7 +201,8 @@ 
             BasePageView::STATIC_TYPE => $this->staticPageViewAction($pageView, $compiler),
             BasePageView::DYNAMIC_TYPE => $this->dynamicPageViewAction($pageView, $compiler),
             BasePageView::REPEATER_TYPE => $this->repeaterPageViewAction($pageView, $compiler),
-            default => function () {
+            default => function ()
+            {
                 $errMsg = 'This URL type has not yet been implemented.';
 
                 return new Response(501, ['Content-Type' => 'text/plain'], $errMsg);
@@ -218,7 +231,8 @@ 
     {
         $rPath = $document->getRelativeFilePath();
 
-        if (!isset($this->lastModified[$rPath])) {
+        if (!isset($this->lastModified[$rPath]))
+        {
             $this->lastModified[$rPath] = $document->getLastModified();
 
             return true;
@@ -236,8 +250,10 @@ 
     {
         $permalink = self::normalizeUrl($permalink);
 
-        foreach ($pageView->getCollectableItems() as $collectableItem) {
-            if ($collectableItem['permalink'] === $permalink) {
+        foreach ($pageView->getCollectableItems() as $collectableItem)
+        {
+            if ($collectableItem['permalink'] === $permalink)
+            {
                 return $collectableItem;
             }
         }
@@ -253,8 +269,10 @@ 
         $url = self::normalizeUrl($permalink);
         $repeaterPermalinks = $pageView->getRepeaterPermalinks();
 
-        foreach ($repeaterPermalinks as $expandedValue) {
-            if ($expandedValue->getEvaluated() === $url) {
+        foreach ($repeaterPermalinks as $expandedValue)
+        {
+            if ($expandedValue->getEvaluated() === $url)
+            {
                 return $expandedValue;
             }
         }

src/Website.php

Braces +23 added lines, -11 removed lines

@@ -43,7 +43,8 @@ 
      */
     public function build(): bool
     {
-        if (empty($this->getConfiguration()->getPageViewFolders())) {
+        if (empty($this->getConfiguration()->getPageViewFolders()))
+        {
             $this->logger->error('No PageViews were configured for this site. Check the `pageviews` key in your _config.yml.');
 
             return false;
@@ -63,10 +64,14 @@ 
         $this->compiler->setThemeName($theme);
         $this->compiler->compileAll();
 
-        if (Service::hasRunTimeFlag(RuntimeStatus::IN_PROFILE_MODE)) {
-            if (!$this->templateBridge->hasProfiler()) {
+        if (Service::hasRunTimeFlag(RuntimeStatus::IN_PROFILE_MODE))
+        {
+            if (!$this->templateBridge->hasProfiler())
+            {
                 $this->logger->writeln('This template engine currently does not support a profiler.');
-            } else {
+            }
+            else
+            {
                 $profilerText = $this->templateBridge->getProfilerOutput($this->compiler);
                 $this->logger->writeln($profilerText);
             }
@@ -82,7 +87,8 @@ 
         //
         // Theme Management
         //
-        if ($theme !== null) {
+        if ($theme !== null)
+        {
             $this->logger->notice("Looking for '{$theme}' theme...");
 
             $tm = new ThemeManager($theme, $this->eventDispatcher, $this->logger);
@@ -113,11 +119,13 @@ 
     {
         $targetDir = fs::absolutePath($this->getConfiguration()->getTargetFolder());
 
-        if (!Service::hasRunTimeFlag(RuntimeStatus::BOOT_WITHOUT_CLEAN)) {
+        if (!Service::hasRunTimeFlag(RuntimeStatus::BOOT_WITHOUT_CLEAN))
+        {
             fs::remove($targetDir);
         }
 
-        if (!Service::hasRunTimeFlag(RuntimeStatus::USING_CACHE)) {
+        if (!Service::hasRunTimeFlag(RuntimeStatus::USING_CACHE))
+        {
             fs::remove(fs::absolutePath(Configuration::CACHE_FOLDER, 'twig'));
             fs::mkdir(fs::absolutePath(fs::appendPath(Configuration::CACHE_FOLDER, 'twig')));
         }
@@ -132,20 +140,24 @@ 
     {
         $enabled = $this->getConfiguration()->isHighlighterEnabled();
 
-        if (!$enabled) {
+        if (!$enabled)
+        {
             return;
         }
 
         Service::setRuntimeFlag(RuntimeStatus::USING_HIGHLIGHTER);
 
-        if ($this->getConfiguration()->isHighlighterUsingLineNumbers()) {
+        if ($this->getConfiguration()->isHighlighterUsingLineNumbers())
+        {
             Service::setRuntimeFlag(RuntimeStatus::USING_LINE_NUMBERS);
         }
 
-        foreach ($this->getConfiguration()->getHighlighterCustomLanguages() as $lang => $path) {
+        foreach ($this->getConfiguration()->getHighlighterCustomLanguages() as $lang => $path)
+        {
             $fullPath = fs::absolutePath($path);
 
-            if (!fs::exists($fullPath)) {
+            if (!fs::exists($fullPath))
+            {
                 $this->logger->warning('The following language definition could not be found: {lang}', [
                     'lang' => $path,
                 ]);