stakx-io / stakx

tests/StakxTestCase.php

Spacing +9 added lines, -9 removed lines

@@ -225,7 +225,7 @@ 
     // Mock Objects
     //
 
-    protected function getMockAssetEngineManager(): MockBuilder|AssetEngineManager
+    protected function getMockAssetEngineManager(): MockBuilder | AssetEngineManager
     {
         return new AssetEngineManager();
     }
@@ -235,7 +235,7 @@ 
         return new AssetManager($this->getMockEventDispatcher(), $this->getMockLogger());
     }
 
-    protected function getMockConfiguration(): MockObject|Configuration
+    protected function getMockConfiguration(): MockObject | Configuration
     {
         $stub = $this->getMockBuilder(Configuration::class)
             ->disableOriginalConstructor()
@@ -248,7 +248,7 @@ 
         return $stub;
     }
 
-    protected function getMockPageManager(): PageManager|MockObject
+    protected function getMockPageManager(): PageManager | MockObject
     {
         $stub = $this->getMockBuilder(PageManager::class)
             ->disableOriginalConstructor()
@@ -260,7 +260,7 @@ 
         return $stub;
     }
 
-    protected function getMockMenuManager(): MockObject|MenuManager
+    protected function getMockMenuManager(): MockObject | MenuManager
     {
         $stub = $this->getMockBuilder(MenuManager::class)
             ->disableOriginalConstructor()
@@ -272,7 +272,7 @@ 
         return $stub;
     }
 
-    protected function getMockCollectionManager(): MockObject|CollectionManager
+    protected function getMockCollectionManager(): MockObject | CollectionManager
     {
         $stub = $this->getMockBuilder(CollectionManager::class)
             ->disableOriginalConstructor()
@@ -284,7 +284,7 @@ 
         return $stub;
     }
 
-    protected function getMockDataManager(): MockObject|DataManager
+    protected function getMockDataManager(): MockObject | DataManager
     {
         $stub = $this->getMockBuilder(DataManager::class)
             ->disableOriginalConstructor()
@@ -327,7 +327,7 @@ 
         return $markupEngine;
     }
 
-    protected function getMockRedirectMapper(): RedirectMapper|MockObject
+    protected function getMockRedirectMapper(): RedirectMapper | MockObject
     {
         $stub = $this->getMockBuilder(RedirectMapper::class)
             ->getMock()
@@ -341,7 +341,7 @@ 
     /**
      * Get a mock EventDispatcher.
      */
-    protected function getMockEventDispatcher(): MockObject|EventDispatcherInterface
+    protected function getMockEventDispatcher(): MockObject | EventDispatcherInterface
     {
         return $this->getMockBuilder(EventDispatcherInterface::class)->getMock();
     }
@@ -349,7 +349,7 @@ 
     /**
      * Get a mock logger.
      */
-    protected function getMockLogger(): MockObject|LoggerInterface
+    protected function getMockLogger(): MockObject | LoggerInterface
     {
         return $this->getMockBuilder(LoggerInterface::class)->getMock();
     }

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();
             }
         }

tests/StreamInterceptor.php

Braces +2 added lines, -1 removed lines

@@ -18,7 +18,8 @@
 
     public function filter($in, $out, &$consumed, $closing)
     {
-        while ($bucket = stream_bucket_make_writeable($in)) {
+        while ($bucket = stream_bucket_make_writeable($in))
+        {
             self::$output .= $bucket->data;
             $consumed += $bucket->datalen;
         }

src/Compiler.php

Spacing +10 added lines, -10 removed lines

@@ -191,7 +191,7 @@ 
      *
      * @since 0.1.1
      */
-    public function compilePageView(DynamicPageView|RepeaterPageView|StaticPageView &$pageView): void
+    public function compilePageView(DynamicPageView | RepeaterPageView | StaticPageView & $pageView): void
     {
         Service::setOption('currentTemplate', $pageView->getAbsoluteFilePath());
         $this->logger->debug('Compiling {type} PageView: {pageview}', [
@@ -237,7 +237,7 @@ 
      *
      * @throws TemplateErrorInterface
      */
-    private function compileStaticPageView(StaticPageView &$pageView): void
+    private function compileStaticPageView(StaticPageView & $pageView): void
     {
         $this->writeToFilesystem(
             $pageView->getTargetFile(),
@@ -253,7 +253,7 @@ 
      *
      * @throws TemplateErrorInterface
      */
-    private function compileDynamicPageView(DynamicPageView &$pageView): void
+    private function compileDynamicPageView(DynamicPageView & $pageView): void
     {
         $contentItems = $pageView->getCollectableItems();
         $template = $this->createTwigTemplate($pageView);
@@ -284,7 +284,7 @@ 
      *
      * @throws TemplateErrorInterface
      */
-    private function compileRepeaterPageView(RepeaterPageView &$pageView): void
+    private function compileRepeaterPageView(RepeaterPageView & $pageView): void
     {
         $template = $this->createTwigTemplate($pageView);
         $permalinks = $pageView->getRepeaterPermalinks();
@@ -325,7 +325,7 @@ 
      *
      * @since 0.1.1
      */
-    private function compileStandardRedirects(PermalinkDocument &$pageView): void
+    private function compileStandardRedirects(PermalinkDocument & $pageView): void
     {
         $redirects = $pageView->getRedirects();
 
@@ -356,7 +356,7 @@ 
      *
      * @since 0.1.1
      */
-    private function compileExpandedRedirects(RepeaterPageView &$pageView): void
+    private function compileExpandedRedirects(RepeaterPageView & $pageView): void
     {
         $permalinks = $pageView->getRepeaterPermalinks();
 
@@ -395,7 +395,7 @@ 
      *
      * @since  0.1.1
      */
-    private function buildRepeaterPageViewHTML(TemplateInterface &$template, RepeaterPageView &$pageView, ExpandedValue &$expandedValue): string
+    private function buildRepeaterPageViewHTML(TemplateInterface & $template, RepeaterPageView & $pageView, ExpandedValue & $expandedValue): string
     {
         $defaultContext = [
             'this' => $pageView->createJail(),
@@ -425,7 +425,7 @@ 
      *
      * @since  0.1.1
      */
-    private function buildDynamicPageViewHTML(TemplateInterface &$template, CollectableItem|TemplateReadyDocument &$twigItem): string
+    private function buildDynamicPageViewHTML(TemplateInterface & $template, CollectableItem | TemplateReadyDocument & $twigItem): string
     {
         $defaultContext = [
             'this' => $twigItem->createJail(),
@@ -452,7 +452,7 @@ 
      *
      * @throws TemplateErrorInterface
      */
-    private function buildStaticPageViewHTML(StaticPageView &$pageView): string
+    private function buildStaticPageViewHTML(StaticPageView & $pageView): string
     {
         $defaultContext = [
             'this' => $pageView->createJail(),
@@ -480,7 +480,7 @@ 
      *
      * @throws TemplateErrorInterface
      */
-    private function createTwigTemplate(BasePageView &$pageView): TemplateInterface
+    private function createTwigTemplate(BasePageView & $pageView): TemplateInterface
     {
         try {
             $template = $this->templateBridge->createTemplate($pageView->getContent());

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

Spacing +1 added lines, -1 removed lines

@@ -183,7 +183,7 @@
     /**
      * Return the appropriate action based on a PageView's type.
      */
-    private function createAction(BasePageView|DynamicPageView|RepeaterPageView|StaticPageView $pageView, Compiler $compiler): Closure
+    private function createAction(BasePageView | DynamicPageView | RepeaterPageView | StaticPageView $pageView, Compiler $compiler): Closure
     {
         return match ($pageView->getType()) {
             BasePageView::STATIC_TYPE => $this->staticPageViewAction($pageView, $compiler),

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;
             }
         }