chillerlan / php-bbcode

src/ParserExtension.php

Spacing +4 added lines, -4 removed lines

@@ -15,7 +15,7 @@ 
 /**
  * An empty parser extension as ground to start from
  */
-class ParserExtension implements ParserExtensionInterface{
+class ParserExtension implements ParserExtensionInterface {
 
 	/**
 	 * @var \chillerlan\bbcode\ParserOptions
@@ -27,7 +27,7 @@ 
 	 *
 	 * @param \chillerlan\bbcode\ParserOptions $options
 	 */
-	public function __construct(ParserOptions $options){
+	public function __construct(ParserOptions $options) {
 		$this->options = $options;
 	}
 
@@ -41,7 +41,7 @@ 
 	 *
 	 * @return string preparsed bbcode
 	 */
-	public function pre($bbcode){
+	public function pre($bbcode) {
 		return $bbcode;
 	}
 
@@ -62,7 +62,7 @@ 
 	 *
 	 * @return string postparsed bbcode
 	 */
-	public function post($bbcode){
+	public function post($bbcode) {
 		return $bbcode;
 	}
 

src/ParserExtensionInterface.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
 /**
  * Implements pre-/post-parser methods, for example: parsing Smileys
  */
-interface ParserExtensionInterface{
+interface ParserExtensionInterface {
 
 	/**
 	 * Pre-parser

src/ParserOptions.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
  *
  * @see \chillerlan\bbcode\Parser::__construct()
  */
-class ParserOptions{
+class ParserOptions {
 
 	/**
 	 * The language class to use (FQN)

src/Traits/ClassLoaderTrait.php

Spacing +4 added lines, -4 removed lines

@@ -17,7 +17,7 @@ 
 /**
  * Trait ClassLoaderTrait
  */
-trait ClassLoaderTrait{
+trait ClassLoaderTrait {
 
 	/**
 	 * A simple class loader
@@ -32,11 +32,11 @@ 
 	 * @return object of type $interface
 	 * @throws \chillerlan\bbcode\BBCodeException
 	 */
-	protected function __loadClass($class, $interface, ...$params){ // phpDocumentor stumbles across the ... syntax
-		if(class_exists($class)){
+	protected function __loadClass($class, $interface, ...$params) { // phpDocumentor stumbles across the ... syntax
+		if (class_exists($class)) {
 			$reflectionClass = new ReflectionClass($class);
 
-			if(!$reflectionClass->implementsInterface($interface)){
+			if (!$reflectionClass->implementsInterface($interface)) {
 				throw new BBCodeException($class.' does not implement '.$interface);
 			}
 

tests/critical/Modules/HTML5ModuleTestCritical.php

Spacing +11 added lines, -11 removed lines

@@ -28,20 +28,20 @@ 
  * @link https://github.com/travis-ci/travis-ci/issues/5323
  * @link https://github.com/travis-ci/travis-ci/issues/5332
  */
-class HTML5ModuleTestCritical extends \PHPUnit_Framework_TestCase{
+class HTML5ModuleTestCritical extends \PHPUnit_Framework_TestCase {
 
 	/**
 	 * @var \chillerlan\bbcode\Parser
 	 */
 	protected $parser;
 
-	protected function setUp(){
+	protected function setUp() {
 		$options = new ParserOptions;
 		$options->allow_all = true;
 		$this->parser = new Parser($options);
 	}
 
-	public function bbcodeDataProvider(){
+	public function bbcodeDataProvider() {
 		return [
 			// basics
 			['', ''], // empty string test (coverage)
@@ -65,15 +65,15 @@ 
 	/**
 	 * @dataProvider bbcodeDataProvider
 	 */
-	public function testParser($bbcode, $expected){
+	public function testParser($bbcode, $expected) {
 		$this->assertEquals($expected, $this->parser->parse($bbcode));
 	}
 
-	public function nestingDataProvider(){
+	public function nestingDataProvider() {
 		return [
-			[0,   'bbcode_nesting.txt'],
-			[1,   'results/html5_nesting_1.txt'],
-			[10,  'results/html5_nesting_10.txt'],
+			[0, 'bbcode_nesting.txt'],
+			[1, 'results/html5_nesting_1.txt'],
+			[10, 'results/html5_nesting_10.txt'],
 			[100, 'results/html5_nesting_100.txt'],
 		];
 	}
@@ -81,7 +81,7 @@ 
 	/**
 	 * @dataProvider nestingDataProvider
 	 */
-	public function testNesting($limit, $resultfile){
+	public function testNesting($limit, $resultfile) {
 		$options = new ParserOptions;
 		$options->allow_all = true;
 		$options->nesting_limit = $limit;
@@ -97,8 +97,8 @@ 
 		$this->assertEquals($expected, $parsed);
 	}
 
-	public function testCodeModule(){
-		foreach(array_keys($this->parser->getTagmap(), Code::class) as $lang){
+	public function testCodeModule() {
+		foreach (array_keys($this->parser->getTagmap(), Code::class) as $lang) {
 			$bbcode = file_get_contents(dirname(__FILE__).'/../../bbcode_samples/bbcode_code_'.$lang.'.txt');
 			$expected = file_get_contents(dirname(__FILE__).'/../../bbcode_samples/results/html5_code_'.$lang.'.txt');
 			$parsed = $this->parser->parse($bbcode);

tests/normal/LanguageBaseTest.php

Indentation +2 added lines, -2 removed lines

@@ -24,8 +24,8 @@
 		Chinese::class,
 		English::class,
 		French::class,
-	    German::class,
-	    Spanish::class,
+		German::class,
+		Spanish::class,
 	];
 
 	public function testIterateLanguage(){

Spacing +4 added lines, -4 removed lines

@@ -18,7 +18,7 @@ 
 use chillerlan\bbcode\Language\Spanish;
 use ReflectionClass;
 
-class LanguageBaseTest extends \PHPUnit_Framework_TestCase{
+class LanguageBaseTest extends \PHPUnit_Framework_TestCase {
 
 	const LANGUAGES = [
 		Chinese::class,
@@ -28,12 +28,12 @@ 
 	    Spanish::class,
 	];
 
-	public function testIterateLanguage(){
-		foreach(self::LANGUAGES as $languageInterface){
+	public function testIterateLanguage() {
+		foreach (self::LANGUAGES as $languageInterface) {
 			$reflectionClass = new ReflectionClass($languageInterface);
 			$lang = $reflectionClass->newInstance();
 
-			foreach($lang as $key => $string){
+			foreach ($lang as $key => $string) {
 				$this->assertEquals($lang->string($key, $languageInterface), $lang->{$key}($languageInterface));
 			}
 		}

tests/normal/Modules/BaseModuleTest.php

Spacing +6 added lines, -6 removed lines

@@ -21,7 +21,7 @@ 
 use chillerlan\bbcode\Modules\Text\TextBaseModule;
 use ReflectionClass;
 
-class BaseModuleTest extends \PHPUnit_Framework_TestCase{
+class BaseModuleTest extends \PHPUnit_Framework_TestCase {
 
 	const BASE_MODULES = [
 		Html5BaseModule::class,
@@ -52,12 +52,12 @@ 
 	 */
 	protected $module;
 
-	protected function setUp(){
+	protected function setUp() {
 		$this->BBTemp = new BBTemp;
 	}
 
-	public function testBaseModules(){
-		foreach(self::BASE_MODULES as $base_module){
+	public function testBaseModules() {
+		foreach (self::BASE_MODULES as $base_module) {
 			$baseModuleReflection = new ReflectionClass($base_module);
 			$this->assertEquals(BaseModuleInterface::class, $baseModuleReflection->getInterfaceNames()[0]);
 
@@ -68,14 +68,14 @@ 
 
 			// mimicking Parser::setOptions() here
 			$tagmap = [];
-			foreach($moduleInfoReflection->getProperty('modules')->getValue($this->moduleInfo) as $module){
+			foreach ($moduleInfoReflection->getProperty('modules')->getValue($this->moduleInfo) as $module) {
 				$moduleReflection = new ReflectionClass($module);
 				$this->assertEquals(ModuleInterface::class, $moduleReflection->getInterfaceNames()[1]);
 
 				$this->module = $moduleReflection->newInstanceArgs([$this->BBTemp]);
 				$tagmapArray = $moduleReflection->getMethod('getTags')->invoke($this->module);
 
-				foreach($tagmapArray->tags as $tag){
+				foreach ($tagmapArray->tags as $tag) {
 					$tagmap[$tag] = $module;
 				}
 			}

tests/normal/Modules/HTML5ModuleTest.php

Spacing +26 added lines, -26 removed lines

@@ -21,7 +21,7 @@ 
 /**
  * Class HTML5ModuleTestCritical
  */
-class HTML5ModuleTest extends \PHPUnit_Framework_TestCase{
+class HTML5ModuleTest extends \PHPUnit_Framework_TestCase {
 
 	/**
 	 * @todo TRAVIS REMINDER!
@@ -33,7 +33,7 @@ 
 	 */
 	protected $parser;
 
-	protected function setUp(){
+	protected function setUp() {
 		(new Dotenv(__DIR__.'/../../', self::DOTENV))->load(); // nasty
 
 		$options = new ParserOptions;
@@ -44,19 +44,19 @@ 
 		$this->parser = new Parser($options);
 	}
 
-	public function testEmptyTags(){
+	public function testEmptyTags() {
 		$singletags = $this->parser->getSingle();
-		$exceptions = ['td','th'];
+		$exceptions = ['td', 'th'];
 
-		foreach(array_keys($this->parser->getTagmap()) as $tag){
-			if(!in_array($tag, $singletags) && !in_array($tag, $exceptions)){
+		foreach (array_keys($this->parser->getTagmap()) as $tag) {
+			if (!in_array($tag, $singletags) && !in_array($tag, $exceptions)) {
 				$parsed = $this->parser->parse('['.$tag.'][/'.$tag.']');
 				$this->assertEquals('', $parsed);
 			}
 		}
 	}
 
-	public function containerDataProvider(){
+	public function containerDataProvider() {
 		return [
 			['[div]a div[/div]', '<div class="bb-container left">a div</div>'],
 			['[div align=right]an aligned div[/div]', '<div class="bb-container right">an aligned div</div>'],
@@ -72,12 +72,12 @@ 
 	/**
 	 * @dataProvider containerDataProvider
 	 */
-	public function testContainerModule($bbcode, $expected){
+	public function testContainerModule($bbcode, $expected) {
 		$parsed = $this->parser->parse($bbcode);
 		$this->assertEquals($expected, $parsed);
 	}
 
-	public function expanderDataProvider(){
+	public function expanderDataProvider() {
 		return [
 			['[expander]expander[/expander]', '<div data-id="abcdef12" title="Expander" class="expander-header expander">Expander</div><div id="abcdef12" class="expander-body" style="display:block">expander</div>'],
 			['[expander hide=1]expander[/expander]', '<div data-id="abcdef12" title="Expander" class="expander-header expander">Expander</div><div id="abcdef12" class="expander-body" style="display:none">expander</div>'],
@@ -93,13 +93,13 @@ 
 	/**
 	 * @dataProvider expanderDataProvider
 	 */
-	public function testExpanderModule($bbcode, $expected){
+	public function testExpanderModule($bbcode, $expected) {
 		$parsed = $this->parser->parse($bbcode);
 		$parsed = preg_replace('/\"([a-f\d]{8})\"/i', '"abcdef12"', $parsed);
 		$this->assertEquals($expected, $parsed);
 	}
 
-	public function linkDataProvider(){
+	public function linkDataProvider() {
 		return [
 			['[url]https://github.com/chillerlan/bbcode[/url]', '<a href="https://github.com/chillerlan/bbcode" target="_blank" class="blank">https://github.com/chillerlan/bbcode</a>'],
 			['[url=https://github.com/chillerlan/bbcode]chillerlan/bbcode[/url]', '<a href="https://github.com/chillerlan/bbcode" target="_blank" class="blank">chillerlan/bbcode</a>'],
@@ -110,17 +110,17 @@ 
 	/**
 	 * @dataProvider linkDataProvider
 	 */
-	public function testLinkModule($bbcode, $expected){
+	public function testLinkModule($bbcode, $expected) {
 		$parsed = $this->parser->parse($bbcode);
 		$this->assertEquals($expected, $parsed);
 	}
 
-	public function testListModule(){
+	public function testListModule() {
 		$list_inner_bbcode = '[*]blah[*]blubb[*]foo[*]bar';
 		$list_inner_html = '<li>blah</li><li>blubb</li><li>foo</li><li>bar</li>';
 
-		foreach(['0' => 'decimal-leading-zero', '1' => 'decimal', 'a' => 'lower-alpha',
-				'A' => 'upper-alpha', 'i' => 'lower-roman', 'I' => 'upper-roman'] as $type => $css){
+		foreach (['0' => 'decimal-leading-zero', '1' => 'decimal', 'a' => 'lower-alpha',
+				'A' => 'upper-alpha', 'i' => 'lower-roman', 'I' => 'upper-roman'] as $type => $css) {
 			// type only
 			$parsed = $this->parser->parse('[list type='.$type.']'.$list_inner_bbcode.'[/list]');
 			$this->assertEquals('<ol class="bb-list '.$css.'">'.$list_inner_html.'</ol>', $parsed);
@@ -135,7 +135,7 @@ 
 			$this->assertEquals('<ol start="42" reversed="true" title="WAT" class="foobar bb-list '.$css.'">'.$list_inner_html.'</ol>', $parsed);
 		}
 
-		foreach(['c' => 'circle', 'd' => 'disc', 's' => 'square'] as $type => $css){
+		foreach (['c' => 'circle', 'd' => 'disc', 's' => 'square'] as $type => $css) {
 			$expected = '<ul class="bb-list '.$css.'">'.$list_inner_html.'</ul>';
 			// type only
 			$parsed = $this->parser->parse('[list type='.$type.']'.$list_inner_bbcode.'[/list]');
@@ -147,22 +147,22 @@ 
 		}
 	}
 
-	public function testSimpletextModule(){
-		foreach(array_keys($this->parser->getTagmap(), Simpletext::class) as $tag){
+	public function testSimpletextModule() {
+		foreach (array_keys($this->parser->getTagmap(), Simpletext::class) as $tag) {
 			$parsed = $this->parser->parse('['.$tag.' class=foobar]WAT[/'.$tag.']');
 			$this->assertEquals('<'.$tag.' class="foobar">WAT</'.$tag.'>', $parsed);
 		}
 	}
 
-	public function testSingletagModule(){
-		foreach(array_keys($this->parser->getTagmap(), Singletags::class) as $tag){
+	public function testSingletagModule() {
+		foreach (array_keys($this->parser->getTagmap(), Singletags::class) as $tag) {
 			$parsed = $this->parser->parse('['.$tag.']');
 			$expected = $tag === 'clear' ? '<br class="bb-clear both" />' : '<'.$tag.' />';
 			$this->assertEquals($expected, $parsed);
 		}
 	}
 
-	public function styledTextDataProvider(){
+	public function styledTextDataProvider() {
 		return [
 			['[color=#424242]color[/color]', '<span class="bb-text color" style="color:#424242">color</span>'],
 			['[font=Helvetica]font[/font]', '<span class="bb-text font" style="font-family:Helvetica">font</span>'],
@@ -178,12 +178,12 @@ 
 	/**
 	 * @dataProvider styledTextDataProvider
 	 */
-	public function testStyledTextModule($bbcode, $expected){
+	public function testStyledTextModule($bbcode, $expected) {
 		$parsed = $this->parser->parse($bbcode);
 		$this->assertEquals($expected, $parsed);
 	}
 
-	public function videoURLDataProvider(){
+	public function videoURLDataProvider() {
 		return [
 			['https://vimeo.com/136964218', '<iframe src="https://player.vimeo.com/video/136964218" allowfullscreen></iframe>'],
 			['https://www.youtube.com/watch?v=6r1-HTiwGiY', '<iframe src="https://www.youtube.com/embed/6r1-HTiwGiY" allowfullscreen></iframe>'],
@@ -203,7 +203,7 @@ 
 		$this->assertEquals('<div class="bb-video">'.$expected.'</div>', $parsed);
 	}*/
 
-	public function videoBBCodeDataProvider(){
+	public function videoBBCodeDataProvider() {
 		return [
 			['[video]http://youtu.be/6r1-HTiwGiY[/video]', '<div class="bb-video"><iframe src="https://www.youtube.com/embed/6r1-HTiwGiY" allowfullscreen></iframe></div>'],
 			['[youtube]http://youtu.be/6r1-HTiwGiY[/youtube]', '<div class="bb-video"><iframe src="https://www.youtube.com/embed/6r1-HTiwGiY" allowfullscreen></iframe></div>'],
@@ -225,7 +225,7 @@ 
 
 	}*/
 
-	public function tableDataProvider(){
+	public function tableDataProvider() {
 		return [
 			['[table width=300px class=mybbtyble][tr][td]foobar[/td][/tr][/table]', '<table class="mybbtyble bb-table" style="width:300px"><tr><td>foobar</td></tr></table>'],
 #			['[table][/table]', ''],
@@ -235,7 +235,7 @@ 
 	/**
 	 * @dataProvider tableDataProvider
 	 */
-	public function testTableModule($bbcode, $expected){
+	public function testTableModule($bbcode, $expected) {
 		$parsed = $this->parser->parse($bbcode);
 		$this->assertEquals($expected, $parsed);
 	}

tests/normal/ParserTest.php

Spacing +10 added lines, -10 removed lines

@@ -15,7 +15,7 @@ 
 use chillerlan\bbcode\ParserOptions;
 use ReflectionClass;
 
-class ParserTest extends \PHPUnit_Framework_TestCase{
+class ParserTest extends \PHPUnit_Framework_TestCase {
 
 	/**
 	 * @var \chillerlan\bbcode\Parser
@@ -27,34 +27,34 @@ 
 	 */
 	protected $reflectionClass;
 
-	protected function setUp(){
+	protected function setUp() {
 		$this->reflectionClass = new ReflectionClass(Parser::class);
 	}
 
-	public function testInstance(){
+	public function testInstance() {
 		$this->parser = $this->reflectionClass->newInstance();
 		$this->assertInstanceOf(Parser::class, $this->parser);
 	}
 
-	public function testGetAllowed(){
+	public function testGetAllowed() {
 		$options = new ParserOptions;
-		$options->allowed_tags = ['noparse','code','img'];
+		$options->allowed_tags = ['noparse', 'code', 'img'];
 
 		$method = $this->reflectionClass->getMethod('getAllowed');
 		$this->parser = $this->reflectionClass->newInstanceArgs([$options]);
-		$this->assertEquals(['code','img','noparse'], $method->invoke($this->parser));
+		$this->assertEquals(['code', 'img', 'noparse'], $method->invoke($this->parser));
 	}
 
-	public function testGetNoparse(){
-		$noparse_tags = ['code','css','html','js','json','noparse','nsis','php','pre','sql','xml'];
+	public function testGetNoparse() {
+		$noparse_tags = ['code', 'css', 'html', 'js', 'json', 'noparse', 'nsis', 'php', 'pre', 'sql', 'xml'];
 
 		$method = $this->reflectionClass->getMethod('getNoparse');
 		$this->parser = $this->reflectionClass->newInstance();
 		$this->assertEquals($noparse_tags, $method->invoke($this->parser));
 	}
 
-	public function testGetSingle(){
-		$singletags = ['br','clear','col','hr'];
+	public function testGetSingle() {
+		$singletags = ['br', 'clear', 'col', 'hr'];
 
 		$method = $this->reflectionClass->getMethod('getSingle');
 		$this->parser = $this->reflectionClass->newInstance();