Mistralys / application-utils

src/XMLHelper/SimpleXML.php

Spacing +3 added lines, -3 removed lines

@@ -83,7 +83,7 @@ 
         // error wrappers.
         $errors = libxml_get_errors();
         
-        foreach($errors as $error) 
+        foreach ($errors as $error) 
         {
             $this->errors[] = new XMLHelper_SimpleXML_Error($this, $error);
         }
@@ -97,7 +97,7 @@ 
         
         $xml = $function($subject);
         
-        if($xml instanceof \SimpleXMLElement)
+        if ($xml instanceof \SimpleXMLElement)
         {
             return $xml;
         }
@@ -107,7 +107,7 @@ 
     
     public function getConverter() : XMLHelper_Converter
     {
-        if($this->element instanceof \SimpleXMLElement)
+        if ($this->element instanceof \SimpleXMLElement)
         {
             return XMLHelper::convertElement($this->element);
         }

src/XMLHelper/DOMErrors/Error.php

Spacing +3 added lines, -3 removed lines

@@ -46,7 +46,7 @@ 
     {
         $data = @json_decode($serialized, true);
         
-        if(!is_array($data))
+        if (!is_array($data))
         {
             throw new XMLHelper_Exception(
                 'Could not unserialize error data',
@@ -72,9 +72,9 @@ 
     
     private static function checkErrorData(array $data) : void
     {
-        foreach(self::$requiredKeys as $key)
+        foreach (self::$requiredKeys as $key)
         {
-            if(!array_key_exists($key, $data))
+            if (!array_key_exists($key, $data))
             {
                 throw new XMLHelper_Exception(
                     'Required key missing in error data',

src/XMLHelper/HTMLLoader.php

Spacing +2 added lines, -2 removed lines

@@ -107,7 +107,7 @@ 
     */
     private static function checkFragment(string $fragment) : void
     {
-        if(!stristr($fragment, '<body') && !stristr($fragment, 'doctype'))
+        if (!stristr($fragment, '<body') && !stristr($fragment, 'doctype'))
         {
             return;
         }
@@ -207,7 +207,7 @@ 
         
         // capture all elements except the body tag itself
         $xml = '';
-        foreach($nodes as $child) 
+        foreach ($nodes as $child) 
         {
             $xml .= $this->dom->saveXML($child);
         }

src/XMLHelper/DOMErrors.php

Spacing +14 added lines, -14 removed lines

@@ -33,13 +33,13 @@ 
     */
     public function __construct(array $libxmlErrors)
     {
-        foreach($libxmlErrors as $error)
+        foreach ($libxmlErrors as $error)
         {
-            if($error instanceof XMLHelper_DOMErrors_Error)
+            if ($error instanceof XMLHelper_DOMErrors_Error)
             {
                 $this->errors[] = $error;
             }
-            else if($error instanceof \LibXMLError)
+            else if ($error instanceof \LibXMLError)
             {
                 $this->errors[] = new XMLHelper_DOMErrors_Error($error);
             }
@@ -117,9 +117,9 @@ 
     {
         $result = array();
         
-        foreach($this->errors as $error)
+        foreach ($this->errors as $error)
         {
-            if($error->isLevel($level))
+            if ($error->isLevel($level))
             {
                 $result[] = $error;
             }
@@ -138,9 +138,9 @@ 
     {
         $result = array();
         
-        foreach($this->errors as $error)
+        foreach ($this->errors as $error)
         {
-            if($error->isCode($code))
+            if ($error->isCode($code))
             {
                 $result[] = $error;
             }
@@ -157,9 +157,9 @@ 
     */
     public function hasErrorsByLevel(int $level) : bool
     {
-        foreach($this->errors as $error)
+        foreach ($this->errors as $error)
         {
-            if($error->isLevel($level))
+            if ($error->isLevel($level))
             {
                 return true;
             }
@@ -176,9 +176,9 @@ 
     */
     public function hasErrorsByCode(int $code) : bool
     {
-        foreach($this->errors as $error)
+        foreach ($this->errors as $error)
         {
-            if($error->isCode($code))
+            if ($error->isCode($code))
             {
                 return true;
             }
@@ -191,7 +191,7 @@ 
     {
         $result = array();
         
-        foreach($this->errors as $error)
+        foreach ($this->errors as $error)
         {
             $result[] = $error->toArray();
         }
@@ -210,7 +210,7 @@ 
     {
         $data = array();
         
-        foreach($this->errors as $error)
+        foreach ($this->errors as $error)
         {
             $data[] = $error->serialize();
         }
@@ -231,7 +231,7 @@ 
         $parts = explode(self::SERIALIZE_SEPARATOR, $serialized);
         $list = array();
         
-        foreach($parts as $part)
+        foreach ($parts as $part)
         {
             $list[] = XMLHelper_DOMErrors_Error::fromSerialized($part);
         }

docs/libxml/generate-errorcodes.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
     $lines = explode("\n", $list);
     $reverseArray = array();
     
-    foreach($lines as $line)
+    foreach ($lines as $line)
     {
         $parts = explode('=', $line);
         $name = trim(str_replace('XML_ERR_', '', $parts[0]));

src/OperationResult.php

Spacing +5 added lines, -5 removed lines

@@ -142,7 +142,7 @@ 
     * @param string $message Should not contain a date, just the system specific info.
     * @return OperationResult
     */
-    public function makeSuccess(string $message, int $code=0) : OperationResult
+    public function makeSuccess(string $message, int $code = 0) : OperationResult
     {
         return $this->setMessage(self::TYPE_SUCCESS, $message, $code, true);
     }
@@ -153,7 +153,7 @@ 
     * @param string $message Should be as detailed as possible.
     * @return OperationResult
     */
-    public function makeError(string $message, int $code=0) : OperationResult
+    public function makeError(string $message, int $code = 0) : OperationResult
     {
         return $this->setMessage(self::TYPE_ERROR, $message, $code, false);
     }
@@ -233,11 +233,11 @@ 
         return $this->code;
     }
     
-    public function getMessage(string $type='') : string
+    public function getMessage(string $type = '') : string
     {
-        if(!empty($type))
+        if (!empty($type))
         {
-            if($this->type === $type)
+            if ($this->type === $type)
             {
                 return $this->message;
             }

src/OperationResult/Collection.php

Spacing +25 added lines, -25 removed lines

@@ -31,27 +31,27 @@ 
     */
     protected $results = array();
     
-    public function makeError(string $message, int $code=0) : OperationResult
+    public function makeError(string $message, int $code = 0) : OperationResult
     {
         return $this->add('makeError', $message, $code);
     }
     
-    public function makeSuccess(string $message, int $code=0) : OperationResult
+    public function makeSuccess(string $message, int $code = 0) : OperationResult
     {
         return $this->add('makeSuccess', $message, $code);
     }
 
-    public function makeWarning(string $message, int $code=0) : OperationResult
+    public function makeWarning(string $message, int $code = 0) : OperationResult
     {
         return $this->add('makeWarning', $message, $code);
     }
     
-    public function makeNotice(string $message, int $code=0) : OperationResult
+    public function makeNotice(string $message, int $code = 0) : OperationResult
     {
         return $this->add('makeNotice', $message, $code);
     }
     
-    protected function add(string $method, string $message, int $code=0) : OperationResult
+    protected function add(string $method, string $message, int $code = 0) : OperationResult
     {
         $result = new OperationResult($this->subject);
         $result->$method($message, $code);
@@ -63,7 +63,7 @@ 
     
     public function addResult(OperationResult $result) : OperationResult_Collection
     {
-        if($result instanceof OperationResult_Collection)
+        if ($result instanceof OperationResult_Collection)
         {
             return $this->importCollection($result);
         }
@@ -75,7 +75,7 @@ 
     {
         $results = $collection->getResults();
         
-        foreach($results as $result)
+        foreach ($results as $result)
         {
             $this->addResult($result);
         }
@@ -111,9 +111,9 @@ 
     
     public function isValid() : bool
     {
-        foreach($this->results as $result)
+        foreach ($this->results as $result)
         {
-            if(!$result->isValid())
+            if (!$result->isValid())
             {
                 return false;
             }
@@ -124,9 +124,9 @@ 
     
     public function hasCode() : bool
     {
-        foreach($this->results as $result)
+        foreach ($this->results as $result)
         {
-            if($result->hasCode())
+            if ($result->hasCode())
             {
                 return true;
             }
@@ -137,9 +137,9 @@ 
     
     public function getCode() : int
     {
-        foreach($this->results as $result)
+        foreach ($this->results as $result)
         {
-            if($result->hasCode())
+            if ($result->hasCode())
             {
                 return $result->getCode();
             }
@@ -148,13 +148,13 @@ 
         return 0;
     }
     
-    public function getMessage(string $type='') : string
+    public function getMessage(string $type = '') : string
     {
-        foreach($this->results as $result)
+        foreach ($this->results as $result)
         {
             $msg = $result->getMessage($type);
             
-            if(!empty($msg))
+            if (!empty($msg))
             {
                 return $msg;
             }
@@ -165,9 +165,9 @@ 
     
     public function containsCode(int $code) : bool
     {
-        foreach($this->results as $result)
+        foreach ($this->results as $result)
         {
-            if($result->getCode() === $code)
+            if ($result->getCode() === $code)
             {
                 return true;
             }
@@ -200,9 +200,9 @@ 
     {
         $amount = 0;
         
-        foreach($this->results as $result)
+        foreach ($this->results as $result)
         {
-            if($result->isType($type))
+            if ($result->isType($type))
             {
                 $amount++;
             }
@@ -240,9 +240,9 @@ 
     {
         $results = array();
         
-        foreach($this->results as $result)
+        foreach ($this->results as $result)
         {
-            if($result->isType($type))
+            if ($result->isType($type))
             {
                 $results[] = $result;
             }
@@ -253,9 +253,9 @@ 
     
     public function isType(string $type) : bool
     {
-        foreach($this->results as $result)
+        foreach ($this->results as $result)
         {
-            if($result->isType($type))
+            if ($result->isType($type))
             {
                 return true;
             }
@@ -271,7 +271,7 @@ 
         $lines[] = 'Collection #'.$this->getID();
         $lines[] = 'Subject: '.get_class($this->subject);
         
-        foreach($this->results as $result)
+        foreach ($this->results as $result)
         {
             $lines[] = ' - '.$result->getType().' #'.$result->getCode().' "'.$result->getMessage($result->getType()).'"';
         }

src/Traits/Classable.php

Spacing +4 added lines, -4 removed lines

@@ -32,7 +32,7 @@ 
     
     public function addClass(string $name)
     {
-        if(!in_array($name, $this->classes)) {
+        if (!in_array($name, $this->classes)) {
             $this->classes[] = $name;
         }
         
@@ -41,7 +41,7 @@ 
     
     public function addClasses(array $names)
     {
-        foreach($names as $name) {
+        foreach ($names as $name) {
             $this->addClass($name);
         }
         
@@ -57,7 +57,7 @@ 
     {
         $idx = array_search($name, $this->classes);
         
-        if($idx !== false) {
+        if ($idx !== false) {
             unset($this->classes[$idx]);
             sort($this->classes);
         }
@@ -91,7 +91,7 @@ 
     */
     public function classesToAttribute() : string
     {
-        if(!empty($this->classes))
+        if (!empty($this->classes))
         {
             return sprintf(
                 ' class="%s" ',

src/Request.php

Spacing +31 added lines, -31 removed lines

@@ -90,11 +90,11 @@ 
     public function getParam($name, $default = null)
     {
         $value = $default;
-        if(isset($_REQUEST[$name])) {
+        if (isset($_REQUEST[$name])) {
             $value = $_REQUEST[$name];
         }
         
-        if(isset($this->knownParams[$name])) {
+        if (isset($this->knownParams[$name])) {
             $value = $this->knownParams[$name]->validate($value);
         }
         
@@ -185,13 +185,13 @@ 
      * @param string $dispatcher Relative path to script to use for the URL. Append trailing slash if needed.
      * @return string
      */
-    public function buildURL($params = array(), string $dispatcher='')
+    public function buildURL($params = array(), string $dispatcher = '')
     {
-        $url = rtrim($this->getBaseURL(), '/') . '/' . $dispatcher;
+        $url = rtrim($this->getBaseURL(), '/').'/'.$dispatcher;
         
         // append any leftover parameters to the end of the URL
         if (!empty($params)) {
-            $url .= '?' . http_build_query($params, '', '&');
+            $url .= '?'.http_build_query($params, '', '&');
         }
         
         return $url;
@@ -222,7 +222,7 @@ 
      */
     public function registerParam($name)
     {
-        if(!isset($this->knownParams[$name])) {
+        if (!isset($this->knownParams[$name])) {
             $param = new Request_Param($this, $name);
             $this->knownParams[$name] = $param;
         }
@@ -239,7 +239,7 @@ 
     */
     public function getRegisteredParam(string $name) : Request_Param
     {
-        if(isset($this->knownParams[$name])) {
+        if (isset($this->knownParams[$name])) {
             return $this->knownParams[$name];
         }
         
@@ -299,7 +299,7 @@ 
     {
         static $accept;
         
-        if(!isset($accept)) {
+        if (!isset($accept)) {
             $accept = new Request_AcceptHeaders();
         }
         
@@ -318,7 +318,7 @@ 
     {
         $_REQUEST[$name] = $value;
         
-        if(isset($this->knownParams[$name])) {
+        if (isset($this->knownParams[$name])) {
             unset($this->knownParams[$name]);
         }
         
@@ -352,11 +352,11 @@ 
     */
     public function removeParam(string $name) : Request
     {
-        if(isset($_REQUEST[$name])) {
+        if (isset($_REQUEST[$name])) {
             unset($_REQUEST[$name]);
         }
         
-        if(isset($this->knownParams[$name])) {
+        if (isset($this->knownParams[$name])) {
             unset($this->knownParams[$name]);
         }
         
@@ -371,7 +371,7 @@ 
     */
     public function removeParams(array $names) : Request
     {
-        foreach($names as $name) {
+        foreach ($names as $name) {
             $this->removeParam($name);
         }
         
@@ -387,10 +387,10 @@ 
      * @param string $name
      * @return bool
      */
-    public function getBool($name, $default=false)
+    public function getBool($name, $default = false)
     {
         $value = $this->getParam($name, $default);
-        if(ConvertHelper::isBoolean($value)) {
+        if (ConvertHelper::isBoolean($value)) {
             return ConvertHelper::string2bool($value);
         }
         
@@ -399,11 +399,11 @@ 
     
     public function validate()
     {
-        foreach($this->knownParams as $param) 
+        foreach ($this->knownParams as $param) 
         {
             $name = $param->getName();
             
-            if($param->isRequired() && !$this->hasParam($name)) 
+            if ($param->isRequired() && !$this->hasParam($name)) 
             {
                 throw new Request_Exception(
                     'Missing request parameter '.$name,
@@ -425,10 +425,10 @@ 
      * @param mixed $default
      * @return string
      */
-    public function getFilteredParam($name, $default=null)
+    public function getFilteredParam($name, $default = null)
     {
         $val = $this->getParam($name, $default);
-        if(is_string($val)) {
+        if (is_string($val)) {
             $val = htmlspecialchars(trim(strip_tags($val)), ENT_QUOTES, 'UTF-8');
         }
         
@@ -447,24 +447,24 @@ 
     * @see Request::getJSONAssoc()
     * @see Request::getJSONObject()
     */
-    public function getJSON(string $name, bool $assoc=true)
+    public function getJSON(string $name, bool $assoc = true)
     {
         $value = $this->getParam($name);
         
-        if(!empty($value) && is_string($value)) 
+        if (!empty($value) && is_string($value)) 
         {
             $data = json_decode($value, $assoc);
             
-            if($assoc && is_array($data)) {
+            if ($assoc && is_array($data)) {
                 return $data;
             }
             
-            if(is_object($data)) {
+            if (is_object($data)) {
                 return $data;
             }
         }
         
-        if($assoc) {
+        if ($assoc) {
             return array();
         }
         
@@ -481,7 +481,7 @@ 
     public function getJSONAssoc(string $name) : array
     {
         $result = $this->getJSON($name);
-        if(is_array($result)) {
+        if (is_array($result)) {
             return $result;
         }
         
@@ -498,7 +498,7 @@ 
     public function getJSONObject(string $name) : object
     {
         $result = $this->getJSON($name, false);
-        if(is_object($result)) {
+        if (is_object($result)) {
             return $result;
         }
         
@@ -511,10 +511,10 @@ 
     * @param array|string $data
     * @param bool $exit Whether to exit the script afterwards.
     */
-    public static function sendJSON($data, bool $exit=true)
+    public static function sendJSON($data, bool $exit = true)
     {
         $payload = $data;
-        if(!is_string($payload)) {
+        if (!is_string($payload)) {
             $payload = json_encode($payload);
         }
         
@@ -524,7 +524,7 @@ 
         
         echo $payload;
         
-        if($exit) 
+        if ($exit) 
         {
             exit;
         }
@@ -536,7 +536,7 @@ 
     * @param string $html
     * @param bool $exit Whether to exit the script afterwards.
     */
-    public static function sendHTML(string $html, bool $exit=true)
+    public static function sendHTML(string $html, bool $exit = true)
     {
         header('Cache-Control: no-cache, must-revalidate');
         header('Expires: Mon, 26 Jul 1997 05:00:00 GMT');
@@ -544,7 +544,7 @@ 
         
         echo $html;
         
-        if($exit)
+        if ($exit)
         {
             exit;
         }
@@ -560,7 +560,7 @@ 
     * @param array $limitParams Whether to limit the comparison to these specific parameter names (if present)
     * @return Request_URLComparer
     */
-    public function createURLComparer(string $sourceURL, string $targetURL, array $limitParams=array()) : Request_URLComparer
+    public function createURLComparer(string $sourceURL, string $targetURL, array $limitParams = array()) : Request_URLComparer
     {
         $comparer = new Request_URLComparer($this, $sourceURL, $targetURL);
         $comparer->addLimitParams($limitParams);