XaBerr / JATE

dist/jate/modules/Module/Module.php

Doc Comments +13 added lines

@@ -39,6 +39,10 @@ 
 					array_push($temp,$i);
 			return $temp;
 		}
+
+		/**
+		 * @param string $_mods
+		 */
 		public function addModules( $_mods ) {
 			if(!is_array($_mods))
 				throw new InvalidArgumentException("Parameter must be an array.");
@@ -64,6 +68,10 @@ 
 				throw new InvalidArgumentException("Parameter must be a string.");
 			array_push($this->files, $_file);
 		}
+
+		/**
+		 * @param string $_files
+		 */
 		public function addFilesRequired( $_files ) {
 			if(!is_array($_files))
 				throw new InvalidArgumentException("Parameter must be an array.");
@@ -80,6 +88,11 @@ 
 			$this->tags["js"] = $this->getJs();
 			$this->tags["jsVariables"] = $this->getJsVariables();
 		}
+
+		/**
+		 * @param string $_function
+		 * @param string $_extenction
+		 */
 		protected function getRequire( $_function, $_extenction) {
 			$temp = [];
 			foreach ($this->required as $i)

Spacing +24 added lines, -24 removed lines

@@ -10,9 +10,9 @@ 
 		public $parameters;
 		public function __construct() {
 			$this->name				= get_class($this);
-			$this->modules		= [];
+			$this->modules = [];
 			$this->files			= [];
-			$this->required		= [];
+			$this->required = [];
 			$this->data				= [];
 			$this->tags				= [];
 			$this->connection	= null;
@@ -22,56 +22,56 @@ 
 		// abstract public function init();
 		// abstract public function draw();
 		public function getCss() {
-			return $this->getRequire("getCss",".css");
+			return $this->getRequire("getCss", ".css");
 		}
 		public function getJs() {
-			return $this->getRequire("getJs",".js");
+			return $this->getRequire("getJs", ".js");
 		}
 		public function getJsVariables() {
 			$temp = [];
 			foreach ($this->required as $i)
 				if (is_array($i))
-					array_push($temp,$i);
+					array_push($temp, $i);
 			foreach ($this->modules as $i)
-				$temp = array_merge( $temp, $i->getJsVariables() );
+				$temp = array_merge($temp, $i->getJsVariables());
 			foreach ($this->files as $i)
 				if (is_array($i))
-					array_push($temp,$i);
+					array_push($temp, $i);
 			return $temp;
 		}
-		public function addModules( $_mods ) {
-			if(!is_array($_mods))
+		public function addModules($_mods) {
+			if (!is_array($_mods))
 				throw new InvalidArgumentException("Parameter must be an array.");
 			foreach ($_mods as $value)
 				$this->addModule($value);
 		}
-		public function addModule( $_mod ) {
-			if(!is_object($_mod))
+		public function addModule($_mod) {
+			if (!is_object($_mod))
 				throw new InvalidArgumentException("Parameter must be a class.");
-			if(! is_subclass_of ($_mod, "Module"))
+			if (!is_subclass_of($_mod, "Module"))
 				throw new InvalidArgumentException("Parameter must be a class.");
 			$this->modules[$_mod->name] = $_mod;
 			$this->modules[$_mod->name]->parameters = &$this->parameters;
 		}
-		public function addFiles( $_files ) {
-			if(!is_array($_files))
+		public function addFiles($_files) {
+			if (!is_array($_files))
 				throw new InvalidArgumentException("Parameter must be an array.");
 			foreach ($_files as $value)
 				$this->addFile($value);
 		}
-		public function addFile( $_file ) {
-			if(!is_string($_file))
+		public function addFile($_file) {
+			if (!is_string($_file))
 				throw new InvalidArgumentException("Parameter must be a string.");
 			array_push($this->files, $_file);
 		}
-		public function addFilesRequired( $_files ) {
-			if(!is_array($_files))
+		public function addFilesRequired($_files) {
+			if (!is_array($_files))
 				throw new InvalidArgumentException("Parameter must be an array.");
 			foreach ($_files as $value)
 				$this->addFileRequired($value);
 		}
-		public function addFileRequired( $_file ) {
-			if(!is_string($_file))
+		public function addFileRequired($_file) {
+			if (!is_string($_file))
 				throw new InvalidArgumentException("Parameter must be a string.");
 			array_push($this->required, $_file);
 		}
@@ -80,16 +80,16 @@ 
 			$this->tags["js"] = $this->getJs();
 			$this->tags["jsVariables"] = $this->getJsVariables();
 		}
-		protected function getRequire( $_function, $_extenction) {
+		protected function getRequire($_function, $_extenction) {
 			$temp = [];
 			foreach ($this->required as $i)
 				if (!is_array($i) && strpos($i, $_extenction) !== FALSE)
-					array_push($temp,$i);
+					array_push($temp, $i);
 			foreach ($this->modules as $i)
-				$temp = array_merge( $temp, $i->$_function() );
+				$temp = array_merge($temp, $i->$_function());
 			foreach ($this->files as $i)
 				if (!is_array($i) && strpos($i, $_extenction) !== FALSE)
-					array_push($temp,$i);
+					array_push($temp, $i);
 			return $temp;
 		}
 	}

tests/jate/modules/Module/ModuleTest.php

Spacing +6 added lines, -6 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-	declare( strict_types = 1 );
+	declare(strict_types = 1);
 
 	use PHPUnit\Framework\TestCase;
 
@@ -32,7 +32,7 @@ 
 		public function testAddModuleNoModule() {
 			$this->expectException(InvalidArgumentException::class);
 			$module = new Module();
-			$module->addModule((object)[1,2,3]);
+			$module->addModule((object) [1, 2, 3]);
 		}
 		public function testAddModules() {
 			$this->expectException(InvalidArgumentException::class);
@@ -41,13 +41,13 @@ 
 		}
 		public function testGetCss() {
 			$module = new Module();
-			$module->addFiles(["1.css","2.js","3.css","4.js"]);
-			$this->assertEquals(["1.css","3.css"], $module->getCss());
+			$module->addFiles(["1.css", "2.js", "3.css", "4.js"]);
+			$this->assertEquals(["1.css", "3.css"], $module->getCss());
 		}
 		public function testGetJs() {
 			$module = new Module();
-			$module->addFiles(["1.css","2.js","3.css","4.js"]);
-			$this->assertEquals(["2.js","4.js"], $module->getJs());
+			$module->addFiles(["1.css", "2.js", "3.css", "4.js"]);
+			$this->assertEquals(["2.js", "4.js"], $module->getJs());
 		}
 	}
 ?>