No longer aborting on errors, but reporting them

Author: weaverryan

src/CI/MissingFilesChecker.php

@@ -28,6 +28,8 @@ public function getMissingFiles(): array
             ->files()
             ->name('*.rst');
 
+        $orphanedFiles = [];
+
         foreach ($this->finder as $file) {
             $htmlFile = str_replace(
                 [$this->buildContext->getSourceDir(), '.rst'],
@@ -37,8 +39,10 @@ public function getMissingFiles(): array
 
             $firstLine = fgets(fopen($file->getRealPath(), 'r'));
             if (!$this->filesystem->exists($htmlFile) && ':orphan:' !== trim($firstLine)) {
-                $io->warning(sprintf('Missing file "%s"', $htmlFile));
+                $orphanedFiles[] = $htmlFile;
             }
         }
+
+        return $orphanedFiles;
     }
 }

src/Command/BuildDocsCommand.php

@@ -57,7 +57,14 @@ protected function configure()
                 null,
                 InputOption::VALUE_NONE,
                 'If provided, caching meta will be disabled'
-            );
+            )
+            ->addOption(
+                'save-errors',
+                null,
+                InputOption::VALUE_REQUIRED,
+                'Path where any errors should be saved'
+            )
+        ;
     }
 
     protected function initialize(InputInterface $input, OutputInterface $output)
@@ -76,11 +83,24 @@ protected function initialize(InputInterface $input, OutputInterface $output)
     protected function execute(InputInterface $input, OutputInterface $output)
     {
         $this->startBuild();
+        $buildErrors = $this->builder->getErrorManager()->getErrors();
 
-        $this->io->newLine(2);
         $this->io->success('HTML rendering complete!');
 
-        $this->missingFilesChecker->checkMissingFiles($this->io);
+        $missingFiles = $this->missingFilesChecker->getMissingFiles();
+        foreach ($missingFiles as $missingFile) {
+            $message = sprintf('Missing file "%s"', $missingFile);
+            $buildErrors[] = $message;
+            $this->io->warning($message);
+        }
+
+        if ($logPath = $input->getOption('save-errors')) {
+            if (count($buildErrors) > 0) {
+                array_unshift($buildErrors, sprintf('Build errors from "%s"', date('Y-m-d h:i:s')));
+            }
+
+            file_put_contents($logPath, implode("\n", $buildErrors));
+        }
 
         $metas = $this->getMetas();
         if (!$this->buildContext->getParseSubPath()) {

src/KernelFactory.php

@@ -18,6 +18,7 @@ public static function createKernel(BuildContext $buildContext, ?UrlChecker $url
         $configuration = new RSTParserConfiguration();
         $configuration->setCustomTemplateDirs([sprintf('%s/src/Templates', $buildContext->getBasePath())]);
         $configuration->setCacheDir(sprintf('%s/var/cache', $buildContext->getBasePath()));
+        $configuration->abortOnError(false);
 
         if ($buildContext->getDisableCache()) {
             $configuration->setUseCachedMetas(false);