chillerlan / php-bbcode

src/BBCode.php

Doc Comments +1 added lines, -1 removed lines

@@ -208,7 +208,7 @@
 	}
 
 	/**
-	 * @param $bbcode
+	 * @param string $bbcode
 	 *
 	 * @return string
 	 */

Spacing +3 added lines, -3 removed lines

@@ -141,7 +141,7 @@ 
 		}
 
 		if($this->options->sanitizeInput || $this->options->sanitizeOutput){
-			$this->sanitizerInterface  = new $this->options->sanitizerInterface($this->options);
+			$this->sanitizerInterface = new $this->options->sanitizerInterface($this->options);
 
 			if(!$this->sanitizerInterface instanceof SanitizerInterface){
 				throw new BBcodeException('invalid SanitizerInterface');
@@ -253,7 +253,7 @@ 
 			return '';
 		}
 
-		if($callback_count < $this->limit && !in_array($tag, $this->noparse , true)){
+		if($callback_count < $this->limit && !in_array($tag, $this->noparse, true)){
 			$content = preg_replace_callback('#\[(\w+)((?:\s|=)[^]]*)?]((?:[^[]|\[(?!/?\1((?:\s|=)[^]]*)?])|(?R))*)\[/\1]#', __METHOD__, $content);
 			$e = preg_last_error();
 
@@ -268,7 +268,7 @@ 
 			if($e !== PREG_NO_ERROR){
 				$this->logger->debug('preg_error', ['errno' => $e, '$content' => $content]);
 
-				$content = $match ?? '';//$content ?? $bbcode ??
+				$content = $match ?? ''; //$content ?? $bbcode ??
 			}
 		}
 

src/Output/BBCodeModuleAbstract.php

Indentation -1 removed lines

@@ -246,7 +246,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

Spacing +1 added lines, -1 removed lines

@@ -84,7 +84,7 @@
 			return $this->transform();
 		}
 
-		return  '';//$this->match;
+		return  ''; //$this->match;
 	}
 
 	/**

Doc Comments +5 added lines, -5 removed lines

@@ -210,8 +210,8 @@ 
 	/**
 	 * shorthand for self::attributeIn('__BBTAG__', $array)
 	 *
-	 * @param array $array
-	 * @param mixed $default
+	 * @param string[] $array
+	 * @param string $default
 	 *
 	 * @return mixed
 	 */
@@ -254,7 +254,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
 	 */
@@ -267,8 +267,8 @@ 
 	/**
 	 * 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[] $whitelist an array with whitelisted tag names
+	 * @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
 	 */

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

@@ -12,9 +12,9 @@
 
 namespace chillerlan\BBCode\Output;
 
-use chillerlan\Settings\SettingsContainerInterface;
 use Psr\Log\LoggerInterface;
 use Psr\SimpleCache\CacheInterface;
+use chillerlan\Settings\SettingsContainerInterface;
 
 abstract class BBCodeModuleAbstract implements BBCodeModuleInterface{
 

src/Output/BBCodeModuleInterface.php

Doc Comments +1 added lines

@@ -24,6 +24,7 @@
 	 * @param \chillerlan\Settings\SettingsContainerInterface $options
 	 * @param \Psr\SimpleCache\CacheInterface       $cache
 	 * @param \Psr\Log\LoggerInterface              $logger
+	 * @return void
 	 */
 	public function __construct(SettingsContainerInterface $options, CacheInterface $cache, LoggerInterface $logger);
 

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

@@ -12,9 +12,9 @@
 
 namespace chillerlan\BBCode\Output;
 
-use chillerlan\Settings\SettingsContainerInterface;
 use Psr\Log\LoggerInterface;
 use Psr\SimpleCache\CacheInterface;
+use chillerlan\Settings\SettingsContainerInterface;
 
 interface BBCodeModuleInterface{
 

src/Output/BBCodeOutputAbstract.php

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

@@ -12,9 +12,9 @@
 
 namespace chillerlan\BBCode\Output;
 
-use chillerlan\Settings\SettingsContainerInterface;
 use Psr\Log\LoggerInterface;
 use Psr\SimpleCache\CacheInterface;
+use chillerlan\Settings\SettingsContainerInterface;
 
 abstract class BBCodeOutputAbstract implements BBCodeOutputInterface{
 

src/Output/BBCodeOutputInterface.php

Doc Comments +1 added lines

@@ -24,6 +24,7 @@
 	 * @param \chillerlan\Settings\SettingsContainerInterface $options
 	 * @param \Psr\SimpleCache\CacheInterface  $cache
 	 * @param \Psr\Log\LoggerInterface         $logger
+	 * @return void
 	 */
 	public function __construct(SettingsContainerInterface $options, CacheInterface $cache, LoggerInterface $logger);
 

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

@@ -12,9 +12,9 @@
 
 namespace chillerlan\BBCode\Output;
 
-use chillerlan\Settings\SettingsContainerInterface;
 use Psr\Log\LoggerInterface;
 use Psr\SimpleCache\CacheInterface;
+use chillerlan\Settings\SettingsContainerInterface;
 
 interface BBCodeOutputInterface{
 

src/Output/Markdown/StyledText.php

Spacing +3 added lines, -3 removed lines

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

src/Output/Markdown/Basic.php

Spacing +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
 	 */
 	protected function url():string{ // @todo linktext
 
-		$url = filter_var($this->bbtag() ?? $this->content, FILTER_VALIDATE_URL);// @todo
+		$url = filter_var($this->bbtag() ?? $this->content, FILTER_VALIDATE_URL); // @todo
 
 		if($url){
 			return  '[url]('.$url.')';

src/Output/HTML/Expanders.php

Indentation +6 added lines, -6 removed lines

@@ -31,9 +31,9 @@ 
 		$id = $this->randomID();
 
 		return '<div class="'.$this->tag.'-container">'.
-		       '<div data-id="'.$id.'" class="'.$this->tag.'-header expander"><span>'.$this->tag.': '.$this->getAttribute('desc').'</span></div>'. // @todo: desc in tag attribute
-		       '<div id="'.$id.'" class="'.$this->tag.'-body" style="display:none;">'.$this->content.'</div>'.
-		       '</div>';
+			   '<div data-id="'.$id.'" class="'.$this->tag.'-header expander"><span>'.$this->tag.': '.$this->getAttribute('desc').'</span></div>'. // @todo: desc in tag attribute
+			   '<div id="'.$id.'" class="'.$this->tag.'-body" style="display:none;">'.$this->content.'</div>'.
+			   '</div>';
 	}
 
 	/**
@@ -50,9 +50,9 @@ 
 
 		// @todo
 		return '<div class="quote-container">'.
-		       '<div data-id="'.$id.'" class="quote-header expander">quote '.($this->getAttribute('source', null) ?? '').(!empty($url) ? ' <small>[<a href="'.$url.'">link</a>]<small>' : '').'</div>'.
-		       '<blockquote id="'.$id.'" class="quote-body" style="display:block;">'.$this->content.'</blockquote>'. // @todo: collapse (js: collapse child elements etc.)
-		       '</div>';
+			   '<div data-id="'.$id.'" class="quote-header expander">quote '.($this->getAttribute('source', null) ?? '').(!empty($url) ? ' <small>[<a href="'.$url.'">link</a>]<small>' : '').'</div>'.
+			   '<blockquote id="'.$id.'" class="quote-body" style="display:block;">'.$this->content.'</blockquote>'. // @todo: collapse (js: collapse child elements etc.)
+			   '</div>';
 	}
 
 }

Spacing +2 added lines, -2 removed lines

@@ -31,7 +31,7 @@ 
 		$id = $this->randomID();
 
 		return '<div class="'.$this->tag.'-container">'.
-		       '<div data-id="'.$id.'" class="'.$this->tag.'-header expander"><span>'.$this->tag.': '.$this->getAttribute('desc').'</span></div>'. // @todo: desc in tag attribute
+		       '<div data-id="'.$id.'" class="'.$this->tag.'-header expander"><span>'.$this->tag.': '.$this->getAttribute('desc').'</span></div>'.// @todo: desc in tag attribute
 		       '<div id="'.$id.'" class="'.$this->tag.'-body" style="display:none;">'.$this->content.'</div>'.
 		       '</div>';
 	}
@@ -51,7 +51,7 @@ 
 		// @todo
 		return '<div class="quote-container">'.
 		       '<div data-id="'.$id.'" class="quote-header expander">quote '.($this->getAttribute('source', null) ?? '').(!empty($url) ? ' <small>[<a href="'.$url.'">link</a>]<small>' : '').'</div>'.
-		       '<blockquote id="'.$id.'" class="quote-body" style="display:block;">'.$this->content.'</blockquote>'. // @todo: collapse (js: collapse child elements etc.)
+		       '<blockquote id="'.$id.'" class="quote-body" style="display:block;">'.$this->content.'</blockquote>'.// @todo: collapse (js: collapse child elements etc.)
 		       '</div>';
 	}
 

src/Output/HTML/Basic.php

Indentation +5 added lines, -5 removed lines

@@ -169,11 +169,11 @@
 		$list_tag = (count($this->attributes) === 0 || $this->attributeIn('type', $ul) ? 'ul' : 'ol');
 
 		return '<'.$list_tag.' class="bb-list '.$this->attributeKeyIn('type', $types, 'disc').'" '
-		       .(is_numeric($start) && $this->attributeIn('type', $ol) ? ' start="'.ceil($start).'"' : '')
-		       .($this->getAttribute('reversed') && $this->attributeIn('type', $ol) ? ' reversed="true"' : '')
-		       .'>'
-		       .'<li>'.implode(array_slice(explode('[*]', $this->content), true), '</li><li>').'</li>' // nasty
-		       .'</'.$list_tag.'>';
+			   .(is_numeric($start) && $this->attributeIn('type', $ol) ? ' start="'.ceil($start).'"' : '')
+			   .($this->getAttribute('reversed') && $this->attributeIn('type', $ol) ? ' reversed="true"' : '')
+			   .'>'
+			   .'<li>'.implode(array_slice(explode('[*]', $this->content), true), '</li><li>').'</li>' // nasty
+			   .'</'.$list_tag.'>';
 	}
 
 }

Spacing +3 added lines, -3 removed lines

@@ -18,7 +18,7 @@ 
 	 * @var array
 	 */
 	protected $tags = [
-		'noparse', 'nobb', 'color', 'font', 'size','br', 'hr', 'clear','img', 'url', 'c', 'list',
+		'noparse', 'nobb', 'color', 'font', 'size', 'br', 'hr', 'clear', 'img', 'url', 'c', 'list',
 		'h1', 'h2', 'h3', 'h4', 'h5', 'h6',
 		'strong', 'sub', 'sup', 'del', 'small', 'em', 's', 'b', 'u', 'i', 'tt',
 	];
@@ -116,7 +116,7 @@ 
 	 * @return string
 	 */
 	protected function img():string{
-		$url = filter_var($this->content, FILTER_VALIDATE_URL);// @todo
+		$url = filter_var($this->content, FILTER_VALIDATE_URL); // @todo
 
 		if(!$url){
 			return '';
@@ -134,7 +134,7 @@ 
 			return '';
 		}
 
-		$url    = filter_var($this->bbtag() ?? $this->content, FILTER_VALIDATE_URL);// @todo
+		$url    = filter_var($this->bbtag() ?? $this->content, FILTER_VALIDATE_URL); // @todo
 		$host   = parse_url($url, PHP_URL_HOST);
 		$target = (!empty($host) && (isset($_SERVER['SERVER_NAME']) && $host === $_SERVER['SERVER_NAME'])) || empty($host) ? 'self' : 'blank';
 

Braces +6 added lines, -3 removed lines

@@ -64,7 +64,8 @@ 
 	/**
 	 * @return string
 	 */
-	protected function c(){ // @todo
+	protected function c(){
+// @todo
 		$this->clearPseudoClosingTags();
 		return '<code class="bb-inline-code" style="display: inline">'.$this->content.'</code>';
 	}
@@ -80,14 +81,16 @@ 
 	/**
 	 * @return string
 	 */
-	protected function font(){ // @todo: restrict fonts via classname
+	protected function font(){
+// @todo: restrict fonts via classname
 		return '<span class="bb-text font comic-sans">'.$this->content.'</span>';
 	}
 
 	/**
 	 * @return string
 	 */
-	protected function size(){ // @todo: restrict sizes via css
+	protected function size(){
+// @todo: restrict sizes via css
 		return '<span class="bb-text size extra-tiny">'.$this->content.'</span>';
 	}