wmde / FundraisingFrontend

cli/FundraisingCli.php

Spacing +5 added lines, -5 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types = 1);
 
 namespace WMDE\Fundraising\Frontend\Cli;
 
@@ -25,12 +25,12 @@ 
 		return $this->app;
 	}
 	private function setApplicationInfo() {
-		$this->app->setName( 'Fundraising Console' );
-		$this->app->setVersion( '2.0' );
+		$this->app->setName('Fundraising Console');
+		$this->app->setVersion('2.0');
 	}
 	private function registerCommands() {
-		$this->app->add( new ValidateConfigCommand() );
-		$this->app->add( new RenderMailTemplatesCommand() );
+		$this->app->add(new ValidateConfigCommand());
+		$this->app->add(new RenderMailTemplatesCommand());
 	}
 	public function run() {
 		$this->newApplication()->run();

cli/RenderMailTemplatesCommand.php

Spacing +43 added lines, -43 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types = 1);
 
 namespace WMDE\Fundraising\Frontend\Cli;
 
@@ -26,43 +26,43 @@ 
 	const NAME = 'dump-mail-tpl';
 
 	protected function configure() {
-		$this->setName( self::NAME )
-			->setDescription( 'Dump rendered Mail_* Twig templates' )
+		$this->setName(self::NAME)
+			->setDescription('Dump rendered Mail_* Twig templates')
 			->setDefinition(
-				new InputDefinition( [
+				new InputDefinition([
 					new InputOption(
 						'output-path',
 						'o',
 						InputOption::VALUE_REQUIRED,
 						'Output path for rendered text'
 					),
-				] )
+				])
 			);
 	}
 
-	protected function execute( InputInterface $input, OutputInterface $output ) {
+	protected function execute(InputInterface $input, OutputInterface $output) {
 
 		$config = $this->getDefaultConfig();
 		$config['twig']['strict-variables'] = true;
 
-		$ffFactory = new FunFunFactory( $config );
+		$ffFactory = new FunFunFactory($config);
 
 		$app = require __DIR__ . '/../app/bootstrap.php';
 
 		$app->flush();
 
-		$ffFactory->setTwigEnvironment( $app['twig'] );
+		$ffFactory->setTwigEnvironment($app['twig']);
 
 		$testData = require __DIR__ . '/../tests/Data/mail_templates.php';
 
-		$this->validateTemplateFixtures( $testData, $config['twig']['loaders']['filesystem']['template-dir'], $output );
+		$this->validateTemplateFixtures($testData, $config['twig']['loaders']['filesystem']['template-dir'], $output);
 
-		$outputPath = $input->getOption( 'output-path' ) ?? '';
-		if ( $outputPath && substr( $outputPath, -1 ) !== '/' ) {
+		$outputPath = $input->getOption('output-path') ?? '';
+		if ($outputPath && substr($outputPath, -1) !== '/') {
 			$outputPath .= '/';
 		}
 
-		$this->renderTemplates( $testData, $ffFactory->getTwig(), $outputPath, $output );
+		$this->renderTemplates($testData, $ffFactory->getTwig(), $outputPath, $output);
 	}
 
 	private function getDefaultConfig(): array {
@@ -71,7 +71,7 @@ 
 		$configReader = new ConfigReader(
 			new SimpleFileFetcher(),
 			__DIR__ . '/../app/config/config.dist.json',
-			is_readable( $prodConfigPath ) ? $prodConfigPath : null
+			is_readable($prodConfigPath) ? $prodConfigPath : null
 		);
 
 		return $configReader->getConfig();
@@ -84,29 +84,29 @@ 
 	 * @param array $mailTemplatePaths Directories containing templates
 	 * @param OutputInterface $output Command output
 	 */
-	private function validateTemplateFixtures( array $testData, array $mailTemplatePaths, OutputInterface $output ): void {
+	private function validateTemplateFixtures(array $testData, array $mailTemplatePaths, OutputInterface $output): void {
 		$mailTemplatesOnDisk = [];
-		foreach ( $mailTemplatePaths as $path ) {
-			$mailFilesInFolder = glob( $path . '/Mail_*' );
-			array_walk( $mailFilesInFolder, function( & $filename ) {
-				$filename = basename( $filename ); // this would cause problems w/ mail templates in sub-folders
+		foreach ($mailTemplatePaths as $path) {
+			$mailFilesInFolder = glob($path . '/Mail_*');
+			array_walk($mailFilesInFolder, function(& $filename) {
+				$filename = basename($filename); // this would cause problems w/ mail templates in sub-folders
 			} );
-			$mailTemplatesOnDisk = array_merge( $mailTemplatesOnDisk, $mailFilesInFolder );
+			$mailTemplatesOnDisk = array_merge($mailTemplatesOnDisk, $mailFilesInFolder);
 		}
 
-		$testTemplateNames = array_keys( $testData );
+		$testTemplateNames = array_keys($testData);
 
-		$untestedTemplates = array_diff( $mailTemplatesOnDisk, $testTemplateNames );
-		if ( count( $untestedTemplates ) ) {
+		$untestedTemplates = array_diff($mailTemplatesOnDisk, $testTemplateNames);
+		if (count($untestedTemplates)) {
 			$output->writeln(
-				'<error>There are untested templates: ' . implode( ', ', $untestedTemplates ) . '</error>'
+				'<error>There are untested templates: ' . implode(', ', $untestedTemplates) . '</error>'
 			);
 		}
 
-		$strayTemplates = array_diff( $testTemplateNames, $mailTemplatesOnDisk );
-		if ( count( $strayTemplates ) ) {
+		$strayTemplates = array_diff($testTemplateNames, $mailTemplatesOnDisk);
+		if (count($strayTemplates)) {
 			$output->writeln(
-				'<error>There are tests for non-existing templates: ' . implode( ', ', $strayTemplates ) . '</error>'
+				'<error>There are tests for non-existing templates: ' . implode(', ', $strayTemplates) . '</error>'
 			);
 		}
 	}
@@ -119,37 +119,37 @@ 
 	 * @param string $outputPath Path where rendered templates will be written to
 	 * @param OutputInterface $output Command output
 	 */
-	private function renderTemplates( array $testData, Twig_Environment $twig, string $outputPath, OutputInterface $output ): void {
-		foreach( $testData as $template => $templateSettings ) {
+	private function renderTemplates(array $testData, Twig_Environment $twig, string $outputPath, OutputInterface $output): void {
+		foreach ($testData as $template => $templateSettings) {
 
-			if ( empty( $templateSettings['variants'] ) ) {
-				$templateSettings['variants'] = [ '' => [] ];
+			if (empty($templateSettings['variants'])) {
+				$templateSettings['variants'] = ['' => []];
 			}
 
-			foreach( $templateSettings['variants'] as $variantName => $additionalContext ) {
+			foreach ($templateSettings['variants'] as $variantName => $additionalContext) {
 				$outputName =
 					$outputPath .
-					basename( $template, '.txt.twig' ) .
-					( $variantName ? ".$variantName" : '' ) .
+					basename($template, '.txt.twig') .
+					($variantName ? ".$variantName" : '') .
 					'.txt';
 
-				$output->write( "$outputName" );
-				if ( file_exists( $outputName ) ) {
-					$output->writeln( "$outputName already exists, skipping ..." );
+				$output->write("$outputName");
+				if (file_exists($outputName)) {
+					$output->writeln("$outputName already exists, skipping ...");
 					continue;
 				}
 
 				try {
-					file_put_contents( $outputName, $twig->render( $template, array_merge_recursive(
+					file_put_contents($outputName, $twig->render($template, array_merge_recursive(
 						$templateSettings['context'],
 						$templateSettings['variants'][$variantName]
-					) ) );
-				} catch( Twig_Error $e ) {
-					$output->writeln( '' );
-					$output->writeln( '<error>' . $e->getMessage() . '</error>' );
-					$output->writeln( var_export( $e->getSourceContext(), true ) );
+					)));
+				} catch (Twig_Error $e) {
+					$output->writeln('');
+					$output->writeln('<error>' . $e->getMessage() . '</error>');
+					$output->writeln(var_export($e->getSourceContext(), true));
 				}
-				$output->writeln( '' );
+				$output->writeln('');
 			}
 		}
 	}

tests/Data/mail_templates.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare( strict_types = 1 );
+declare(strict_types = 1);
 
 /**
  * This file contains a list of all Mail templates and the variables rendered in them.