chillerlan / php-bbcode

src/Parser.php

Doc Comments +1 added lines, -1 removed lines

@@ -259,7 +259,7 @@
 	/**
 	 * strng regexp bbcode killer
 	 *
-	 * @param string|array $bbcode BBCode as string or matches as array - callback from preg_replace_callback()
+	 * @param string $bbcode BBCode as string or matches as array - callback from preg_replace_callback()
 	 *
 	 * @return string
 	 * @throws \chillerlan\bbcode\BBCodeException

src/Modules/BaseModuleInterface.php

Indentation -1 removed lines

@@ -101,7 +101,6 @@
 
 	/**
 	 * Checks if an attribute exists and if it exists as key in a whitelist
-
 	 * @param string $name      the desired attributes name
 	 * @param array  $whitelist an array with whitelisted key -> value pairs
 	 * @param mixed  $default   [optional] a default value in case the attribute isn't set, defaults to false

src/Modules/BaseModule.php

Indentation -1 removed lines

@@ -101,7 +101,6 @@
 
 	/**
 	 * Checks if an attribute exists and if it exists as key in a whitelist
-
 	 * @param string $name      the desired attributes name
 	 * @param array  $whitelist an array with whitelisted key -> value pairs
 	 * @param mixed  $default   [optional] a default value in case the attribute isn't set, defaults to false

Doc Comments +4 added lines, -4 removed lines

@@ -248,7 +248,7 @@ 
 
 	 * @param string $name      the desired attributes name
 	 * @param array  $whitelist an array with whitelisted key -> value pairs
-	 * @param mixed  $default   [optional] a default value in case the attribute isn't set, defaults to false
+	 * @param string  $default   [optional] a default value in case the attribute isn't set, defaults to false
 	 *
 	 * @return mixed boolean if no $default is set, otherwise the whitelist value to the given key in case it exists or $default
 	 */
@@ -262,7 +262,7 @@ 
 	 * Checks if the current tag is whitelisted
 	 *
 	 * @param array $whitelist an array with whitelisted tag names
-	 * @param mixed $default   [optional] a default value in case the tag isn't whitelisted
+	 * @param string $default   [optional] a default value in case the tag isn't whitelisted
 	 *
 	 * @return mixed boolean if no $default is set, otherwise the whitelisted tag or $default
 	 */
@@ -311,7 +311,7 @@ 
 	 * shorthand for self::attributeIn('__BBTAG__', $array)
 	 *
 	 * @param array $array
-	 * @param mixed $default
+	 * @param string $default
 	 *
 	 * @return mixed
 	 */
@@ -324,7 +324,7 @@ 
 	 *
 	 * @param string $url the URL to check
 	 *
-	 * @return bool|string the url if valid, otherwise false
+	 * @return false|string the url if valid, otherwise false
 	 */
 	public function checkUrl(string $url){
 		if(filter_var($url, FILTER_VALIDATE_URL) === false){

src/Modules/Html5/Video.php

Unused Use Statements +1 added lines, -1 removed lines

@@ -12,8 +12,8 @@
 
 namespace chillerlan\bbcode\Modules\Html5;
 
-use chillerlan\bbcode\Modules\ModuleInterface;
 use chillerlan\TinyCurl\Traits\RequestTrait;
+use chillerlan\bbcode\Modules\ModuleInterface;
 
 /**
  * Transforms several video tags into HTML5

Indentation +7 added lines, -7 removed lines

@@ -99,11 +99,11 @@ 
 	protected function flashPlayer(string $video_url):string{
 
 		return '<div'.$this->getCssClass($this->cssclass).'>'
-		       .'<object type="application/x-shockwave-flash" data="'.$video_url.'">'
-		       .'<param name="allowfullscreen" value="true">'
-		       .'<param name="wmode" value="opaque" />'
-		       .'<param name="movie" value="'.$video_url.'" />'
-		       .'</object></div>';
+			   .'<object type="application/x-shockwave-flash" data="'.$video_url.'">'
+			   .'<param name="allowfullscreen" value="true">'
+			   .'<param name="wmode" value="opaque" />'
+			   .'<param name="movie" value="'.$video_url.'" />'
+			   .'</object></div>';
 	}
 
 	/**
@@ -114,7 +114,7 @@ 
 	protected function embedPlayer(string $video_url):string{
 
 		return '<div'.$this->getCssClass($this->cssclass).'>'
-		       .'<iframe src="'.$video_url.'" allowfullscreen></iframe></div>';
+			   .'<iframe src="'.$video_url.'" allowfullscreen></iframe></div>';
 	}
 
 	/**
@@ -123,7 +123,7 @@ 
 	protected function html5Player():string{
 
 		return '<video src="'.$this->checkUrl($this->content).'"'
-		       .$this->getCssClass($this->cssclass).' preload="auto" controls="true"></video>';
+			   .$this->getCssClass($this->cssclass).' preload="auto" controls="true"></video>';
 	}
 
 	/**

src/Traits/ClassLoaderTrait.php

Doc Comments +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
 	 * @param string $class     FQCN
 	 * @param string $interface FQCN
 	 *
-	 * @param mixed  $params    [optional] the following arguments are optional and will be passed to the class constructor if present.
+	 * @param \chillerlan\bbcode\ParserOptions  $params    [optional] the following arguments are optional and will be passed to the class constructor if present.
 	 *
 	 * @link https://github.com/chillerlan/framework/blob/master/src/Core/Traits/ClassLoaderTrait.php
 	 *

Unused Use Statements +1 added lines, -1 removed lines

@@ -11,8 +11,8 @@
 
 namespace chillerlan\bbcode\Traits;
 
-use chillerlan\bbcode\BBCodeException;
 use ReflectionClass;
+use chillerlan\bbcode\BBCodeException;
 
 /**
  * Trait ClassLoaderTrait

Braces +2 added lines, -1 removed lines

@@ -32,7 +32,8 @@
 	 * @return object of type $interface
 	 * @throws \chillerlan\bbcode\BBCodeException
 	 */
-	protected function __loadClass(string $class, string $interface, ...$params){ // phpDocumentor stumbles across the ... syntax
+	protected function __loadClass(string $class, string $interface, ...$params){
+// phpDocumentor stumbles across the ... syntax
 		if(class_exists($class)){
 			$reflectionClass = new ReflectionClass($class);
 

src/Modules/Html5/Links.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
 			return '';
 		}
 
-		$url = $this->checkUrl($this->bbtag() ? : $this->content);
+		$url = $this->checkUrl($this->bbtag() ?: $this->content);
 		$host = parse_url($url, PHP_URL_HOST);
 		$target = (!empty($host) && (isset($_SERVER['SERVER_NAME']) && $host === $_SERVER['SERVER_NAME']))
 			|| empty($host) ? 'self' : 'blank';

src/Modules/Markdown/Links.php

Spacing +2 added lines, -2 removed lines

@@ -40,10 +40,10 @@
 		if(!empty($this->content)){
 			return $url && $this->checkUrl($url)
 				? '['.$this->content.']('.$url.')'
-				: $this->checkUrl($this->content) ? : $this->content;
+				: $this->checkUrl($this->content) ?: $this->content;
 		}
 
-		return $this->checkUrl($url) ? : '';
+		return $this->checkUrl($url) ?: '';
 	}
 
 }

src/Modules/Markdown/StyledText.php

Spacing +2 added lines, -2 removed lines

@@ -41,9 +41,9 @@
 
 		$str = [
 			'b'      => '**', // bold
-			'c'      => '`',  // inline code
+			'c'      => '`', // inline code
 			'del'    => '~~', // strikethrough
-			'i'      => '_',  // italic
+			'i'      => '_', // italic
 			's'      => '~~', // strikethrough
 			'strong' => '**', // bold
 		][$this->tag];

src/Modules/Markup/MarkupBaseModule.php

Spacing +1 added lines, -1 removed lines

@@ -127,7 +127,7 @@
 	 */
 	protected function getCssClass(array $additional_classes = []){
 		$classes = $this->getAttribute('class', '').' '.implode(' ', $additional_classes);
-		$classes =preg_replace('/[^a-z\d\- ]/i', '', $classes);
+		$classes = preg_replace('/[^a-z\d\- ]/i', '', $classes);
 		$classes = trim($classes);
 
 		return !empty($classes) ? ' class="'.$classes.'"' : '';