Arlisaha / openMindParser

src/Converters/Model/AbstractConverter.php

Doc Comments -1 removed lines

@@ -38,7 +38,6 @@
 	/**
 	 * Abstract class to actually perform the conversion from the Document instance passed in parameter.
 	 * 
-	 * @param mixed $data : The data to convert. Here : a string as the file path or the Document instance.
 	 * @param array $options : An array of options for the conversion.
 	 * 
 	 * @return mixed : The result of the conversion.

Spacing +3 added lines, -3 removed lines

@@ -19,13 +19,13 @@
 	 * @return mixed : The result of the conversion.
 	 */
 	public function convert($data, $options = []) {
-		if(!is_string($data) && !($data instanceof Document)) {
+		if (!is_string($data) && !($data instanceof Document)) {
 			throw new InvalidArgumentException('The $data variable must be of type "string" (the file path), or an instance of "Document".');
-		} elseif(!is_array($options)) {
+		} elseif (!is_array($options)) {
 			throw new InvalidArgumentException('The $options variable must be and array.');
 		}
 		
-		if(is_string($data)) {
+		if (is_string($data)) {
 			$parser = new Parser();
 			$data = $parser->buildDocumentTreeFromFilePath($data);	
 		}

src/Models/Node.php

Spacing +5 added lines, -5 removed lines

@@ -357,23 +357,23 @@
 	public function toArray() {
 		$array = [];
 		$sorter = function($value, $key) use(&$array) {
-			if($value instanceof NodeList) {
+			if ($value instanceof NodeList) {
 				$newValue = [];
-				foreach($value as $node) {
+				foreach ($value as $node) {
 					$newValue[] = $node->toArray();
 				}
 				$value = $newValue;
 			}
-			elseif($value instanceof Icon) {
+			elseif ($value instanceof Icon) {
 				$value = $value->getFilePath();
 			}
-			elseif($value instanceof DOMElement) {
+			elseif ($value instanceof DOMElement) {
 				return;
 			}
 			$array[$key] = $value;
 		};
 		
-		foreach(get_object_vars($this) as $key => $value) {
+		foreach (get_object_vars($this) as $key => $value) {
 			$sorter($value, $key);
 		}
 		

Braces +2 added lines, -4 removed lines

@@ -363,11 +363,9 @@
 					$newValue[] = $node->toArray();
 				}
 				$value = $newValue;
-			}
-			elseif($value instanceof Icon) {
+			} elseif($value instanceof Icon) {
 				$value = $value->getFilePath();
-			}
-			elseif($value instanceof DOMElement) {
+			} elseif($value instanceof DOMElement) {
 				return;
 			}
 			$array[$key] = $value;

src/Models/NodeList.php

Spacing +3 added lines, -3 removed lines

@@ -14,12 +14,12 @@
 	private $list;
 	
 	public function __construct(array $list = null) {
-		if(empty($list)) {
+		if (empty($list)) {
 			$list = [];
 		}
 		
-		foreach($list as $node) {
-			if(!($node instanceof Node)) {
+		foreach ($list as $node) {
+			if (!($node instanceof Node)) {
 				throw new InvalidArgumentException('The array must contain only Node objects. "'.get_class($node).'" given.');
 			}
 		}

src/Models/Icon.php

Spacing +2 added lines, -2 removed lines

@@ -42,7 +42,7 @@ 
 	 * @param String $extension : The extension of the icon file without the leading dot. If it is a builtin icon, then it is 'png'.
 	 */ 
 	public function __construct($name = null, $extension = null, $path = null, $shortUri = null, $size = null) {
-		if(!empty($name)) {
+		if (!empty($name)) {
 			$this->setIcon($name, $extension, $path, $shortUri, $size);
 		}
 	}
@@ -64,7 +64,7 @@ 
 		$this->shortUri = $shortUri ?: '/img/'.$this->fullName;
 		$this->path = $path ?: realpath(__DIR__.'/../../img/'.$this->fullName);
 		
-		if(!file_exists($this->path)) {
+		if (!file_exists($this->path)) {
 			throw new UnexistentFileException('The file '.$this->path.' does not exist !');
 		}
 		

usage/SampleGenericHTMLConverter.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use OpenMindParser\Converters\Model\AbstractConverter;
 use OpenMindParser\Models\Document;
 use OpenMindParser\Models\Node;
-use OpenMindParser\Parser;
 use \DOMDocument;
 
 /*A singleton to convert a document tree (object Document) in a HTML tree following few options.*/

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
 	GenericHTMLConverter::MAIN_ICON_KEY => [
 		GenericHTMLConverter::DISPLAY_ICON_KEY => true,
 		GenericHTMLConverter::PATH_ICON_KEY    => [
-			GenericHTMLConverter::CALLBACK_PATH_ICON_KEY => function($fullName, $options = null){return '/openMindParser/img/'.$fullName;},
+			GenericHTMLConverter::CALLBACK_PATH_ICON_KEY => function($fullName, $options = null) {return '/openMindParser/img/'.$fullName; },
 		],
 	],
 ]);

src/Models/Document.php

Spacing +3 added lines, -3 removed lines

@@ -148,15 +148,15 @@
 	public function toArray() {
 		$array = [];
 		$sorter = function($value, $key) use(&$array) {
-			if($value instanceof Node) {
+			if ($value instanceof Node) {
 				$value = $value->toArray();
-			} elseif($value instanceof DOMDocument) {
+			} elseif ($value instanceof DOMDocument) {
 				return;
 			}
 			$array[$key] = $value;
 		};
 		
-		foreach(get_object_vars($this) as $key => $value) {
+		foreach (get_object_vars($this) as $key => $value) {
 			$sorter($value, $key);
 		}
 		

src/Parser.php

Spacing +11 added lines, -11 removed lines

@@ -63,7 +63,7 @@ 
 	 * @return Document : The document instance with all its nodes instances.
 	 */
 	public function buildDocumentTreeFromFilePath($filePath) {
-		if(!file_exists($filePath)) {
+		if (!file_exists($filePath)) {
 			throw new InvalidArgumentException('The given path : "'.$filePath.'" is invalid.');
 		}
 		
@@ -85,7 +85,7 @@ 
 	 */
 	private function fillNode(DOMElement $domNode) {
 		//The given node name must be self::NODE_NODENAME
-		if($domNode->nodeName !== self::NODE_NODENAME) {
+		if ($domNode->nodeName !== self::NODE_NODENAME) {
 			throw new InvalidNodeNameException('The node name must be "node". "'.$domNode->nodeName.'" given.');
 		}
 		
@@ -95,14 +95,14 @@ 
 		
 		//Build the list of children nodes and fill font information.
 		$children = new NodeList();
-		foreach($domNode->childNodes as $childNode) {
-			if($childNode->nodeName === self::NODE_NODENAME) {
+		foreach ($domNode->childNodes as $childNode) {
+			if ($childNode->nodeName === self::NODE_NODENAME) {
 				$children->add($this->fillNode($childNode));
-			} elseif($childNode->nodeName === self::FONT_NODENAME) {
+			} elseif ($childNode->nodeName === self::FONT_NODENAME) {
 				$this->fillNodeAttributes($childNode->attributes, self::$fontAvailableAttributes, $node);
-			} elseif($childNode->nodeName === self::ICON_NODENAME) {
-				foreach($childNode->attributes as $attribute) {
-					if(array_key_exists($attribute->nodeName, self::$iconAvailableAttributes)) {
+			} elseif ($childNode->nodeName === self::ICON_NODENAME) {
+				foreach ($childNode->attributes as $attribute) {
+					if (array_key_exists($attribute->nodeName, self::$iconAvailableAttributes)) {
 						$node->setIcon(new Icon($attribute->nodeValue));
 					}
 				}
@@ -121,9 +121,9 @@ 
 	 * @param array $availableAttributes : One of the static array of this class to describe the list of known attributes.
 	 * @param Node $node : The Node object to fill in.
 	 */
-	private function fillNodeAttributes (DOMNamedNodeMap $nodeAtributes, array $availableAttributes, Node $node) {
-		foreach($nodeAtributes as $attribute) {
-			if(array_key_exists($attribute->nodeName, $availableAttributes)) {
+	private function fillNodeAttributes(DOMNamedNodeMap $nodeAtributes, array $availableAttributes, Node $node) {
+		foreach ($nodeAtributes as $attribute) {
+			if (array_key_exists($attribute->nodeName, $availableAttributes)) {
 				call_user_func([
 						$node, 
 						sprintf('openMindParser\Models\Node::set%s', ucfirst($availableAttributes[$attribute->nodeName]))

src/Converters/HTML/GenericHTMLConverter.php

Spacing +13 added lines, -14 removed lines

@@ -138,7 +138,7 @@ 
 		$domElementA = $this->buildElement($document, $options[self::MAIN_TAG_KEY][0]);
 		
 		//Merge font size and font family to add inside the second wrapping DOMElement.
-		if($options[self::MAIN_TAG_MERGE_STYLE]) {
+		if ($options[self::MAIN_TAG_MERGE_STYLE]) {
 			$options[self::MAIN_TAG_KEY][1][self::ATTRIBUTES_KEY] = array_merge(
 				$options[self::MAIN_TAG_KEY][1][self::ATTRIBUTES_KEY], 
 				[
@@ -156,19 +156,18 @@ 
 		$domElementB = $this->buildElement($document, $options[self::MAIN_TAG_KEY][1]);
 		
 		//Set the icon in an img tag whom will be wrapped in the second DOMElement.
-		if(!empty($node->getIcon()) && $options[self::MAIN_ICON_KEY][self::DISPLAY_ICON_KEY]) {
+		if (!empty($node->getIcon()) && $options[self::MAIN_ICON_KEY][self::DISPLAY_ICON_KEY]) {
 			$icon = $node->getIcon();
 			$domElementImg = $document->createElement('img');
 			
 			$iconUri = $icon->getShortUri();
-			if(array_key_exists(self::PATH_ICON_KEY, $options[self::MAIN_ICON_KEY])) {
-				if(!is_callable($callback = $options[self::MAIN_ICON_KEY][self::PATH_ICON_KEY][self::CALLBACK_PATH_ICON_KEY])) {
+			if (array_key_exists(self::PATH_ICON_KEY, $options[self::MAIN_ICON_KEY])) {
+				if (!is_callable($callback = $options[self::MAIN_ICON_KEY][self::PATH_ICON_KEY][self::CALLBACK_PATH_ICON_KEY])) {
 					throw new InvalidArgumentException('The argument with the key "'.self::CALLBACK_PATH_ICON_KEY.'" (self::CALLBACK_PATH_ICON_KEY) must be a valid callback.');
 				}
 				
 				$callBackoptions = array_key_exists(self::OPTIONS_PATH_ICON_KEY, $options[self::MAIN_ICON_KEY][self::PATH_ICON_KEY]) ?
-					$options[self::MAIN_ICON_KEY][self::PATH_ICON_KEY][self::OPTIONS_PATH_ICON_KEY] :
-					null;
+					$options[self::MAIN_ICON_KEY][self::PATH_ICON_KEY][self::OPTIONS_PATH_ICON_KEY] : null;
 				
 				$iconUri = $callback($icon->getFullName(), $callBackoptions);
 			}
@@ -182,13 +181,13 @@ 
 		$text = $document->createTextNode($node->getText());
 		
 		//Bold an italic in old HTML way in order not to spread it to all children.
-		if($options[self::MAIN_TAG_MERGE_DECORATION]) {
-			if($node->isBold()) {
+		if ($options[self::MAIN_TAG_MERGE_DECORATION]) {
+			if ($node->isBold()) {
 				$bTag = $document->createElement('b');
 				$bTag->appendChild($text);
 				$text = $bTag;
 			}
-			if($node->isItalic()) {
+			if ($node->isItalic()) {
 				$iTag = $document->createElement('i');
 				$iTag->appendChild($text);
 				$text = $iTag;
@@ -196,7 +195,7 @@ 
 		}
 		
 		//Append text node (real one or the one already wrapped inside <b> and/or <i> tags)
-		if(isset($options[self::MAIN_TAG_KEY][2])) {
+		if (isset($options[self::MAIN_TAG_KEY][2])) {
 			$domElementC = $this->buildElement($document, $options[self::MAIN_TAG_KEY][2]);
 			$domElementC->appendChild($text);
 			$domElementB->appendChild($domElementC);
@@ -206,7 +205,7 @@ 
 		
 		$domElementA->appendChild($domElementB);
 		
-		foreach($node->getChildren() as $child) {
+		foreach ($node->getChildren() as $child) {
 			$domElementB->appendChild($this->buildHTMLTreeFromNode($document, $child, $options));
 		}
 		
@@ -219,8 +218,8 @@ 
 	 * @param array &$options : The options array passed as reference.
 	 */
 	private function fillOptionalOptions(array &$options) {
-		foreach([self::MAIN_TAG_MERGE_DECORATION, self::MAIN_TAG_MERGE_STYLE] as $key) {
-			if(!array_key_exists($key, $options) || !is_bool($options[$key])) {
+		foreach ([self::MAIN_TAG_MERGE_DECORATION, self::MAIN_TAG_MERGE_STYLE] as $key) {
+			if (!array_key_exists($key, $options) || !is_bool($options[$key])) {
 				$options[$key] = true;
 			}
 		}
@@ -236,7 +235,7 @@ 
 	 */
 	private function buildElement(DOMDocument $document, array $description) {
 		$domElement = $document->createElement($description[self::TAG_KEY]);
-		foreach($description[self::ATTRIBUTES_KEY] as $name => $attribute) {
+		foreach ($description[self::ATTRIBUTES_KEY] as $name => $attribute) {
 			$domElement->setAttribute($name, $attribute);
 		}