zewadesign / framework

Zewa/App.php

Doc Comments +7 added lines, -1 removed lines

@@ -214,6 +214,9 @@ 
         }
     }
 
+    /**
+     * @param string $service
+     */
     public function setService($service, $class)
     {
         if($this->services === null) {
@@ -225,6 +228,7 @@ 
 
     /**
      * @param mixed string with reference to config
+     * @param string $config
      * @return mixed bool or config values
      */
     public function getConfiguration($config = null)
@@ -342,7 +346,6 @@ 
      * Attach (or remove) multiple callbacks to an event and trigger those callbacks when that event is called.
      *
      * @param string $event    name
-     * @param mixed  $value    the optional value to pass to each callback
      * @param mixed  $callback the method or function to call - FALSE to remove all callbacks for event
      */
 
@@ -356,6 +359,9 @@ 
         }
     }
 
+    /**
+     * @param string $event
+     */
     public function callEvent($event, $method = false, $arguments = [])
     {
         if (isset(self::$events[$event])) {

Spacing +2 added lines, -2 removed lines

@@ -231,9 +231,9 @@
     public function getConfiguration($config = null)
     {
         if ($config !== null) {
-            if (! empty($this->configuration->$config)) {
+            if (!empty($this->configuration->$config)) {
                 return $this->configuration->$config;
-            } elseif (! empty($this->files{$config})) {
+            } elseif (!empty($this->files{$config})) {
                 $vars = include $this->files{$config};
 
                 if ($vars === false) {

Zewa/View.php

Doc Comments +4 added lines, -2 removed lines

@@ -90,8 +90,6 @@ 
      *
      * @access public
      *
-     * @param string $requestedView relative path for the view
-     * @param string $renderName    array of data to expose to view
      *
      * @throws \Exception when a view can not be found
      */
@@ -181,6 +179,10 @@ 
      * @return string processed content
      */
     //@TODO: come back and clean up this and the way the view receives stuff
+
+    /**
+     * @param string|boolean $file
+     */
     private function process($file)
     {
         ob_start();

Zewa/Router.php

Doc Comments +22 added lines, -1 removed lines

@@ -129,11 +129,18 @@ 
     }
 
 
+    /**
+     * @param string $uri
+     */
     private function isURIClean($uri, $uriChunks)
     {
         if (!preg_match("/^[a-z0-9:_\/\.\[\]-]+$/i", $uri)
             || array_filter(
                 $uriChunks,
+
+                /**
+                 * @param string $uriChunk
+                 */
                 function ($uriChunk) {
                     if (strpos($uriChunk, '__') !== false) {
                         return true;
@@ -148,6 +155,10 @@ 
     }
 
     //@TODO add Security class.
+
+    /**
+     * @param string $data
+     */
     private function normalize($data)
     {
         if (is_numeric($data)) {
@@ -225,6 +236,9 @@ 
         return $normalizedURI;
     }
 
+    /**
+     * @param string $uri
+     */
     private function discoverRoute($uri)
     {
         $routes = $this->configuration->routes;
@@ -259,6 +273,7 @@ 
      * Normalize the $_SERVER vars for formatting the URI.
      *
      * @access public
+     * @param string $uri
      * @return string formatted/u/r/l
      */
     private function uri($uri)
@@ -312,6 +327,9 @@ 
         return array_merge($return, array_values($uriChunks));
     }
 
+    /**
+     * @param string $url
+     */
     private function addQueryString($url, $key, $value)
     {
         $url = preg_replace('/(.*)(\?|&)' . $key . '=[^&]+?(&)(.*)/i', '$1$2$4', $url . '&');
@@ -323,6 +341,9 @@ 
         }
     }
 
+    /**
+     * @param string $url
+     */
     private function removeQueryString($url, $key)
     {
         $url = preg_replace('/(.*)(\?|&)' . $key . '=[^&]+?(&)(.*)/i', '$1$2$4', $url . '&');
@@ -394,7 +415,7 @@ 
      * Set optional status header, and redirect to provided URL
      *
      * @access public
-     * @return bool
+     * @return false|null
      */
     public function redirect($url = '/', $status = null)
     {

Spacing +4 added lines, -4 removed lines

@@ -134,7 +134,7 @@ 
         if (!preg_match("/^[a-z0-9:_\/\.\[\]-]+$/i", $uri)
             || array_filter(
                 $uriChunks,
-                function ($uriChunk) {
+                function($uriChunk) {
                     if (strpos($uriChunk, '__') !== false) {
                         return true;
                     }
@@ -218,7 +218,7 @@ 
 
         $normalizedURI = ltrim(preg_replace('/\?.*/', '', $normalizedURI), '/');
 
-        if (! empty($this->configuration->routes)) {
+        if (!empty($this->configuration->routes)) {
             $normalizedURI = $this->discoverRoute($normalizedURI);
         }
         
@@ -237,11 +237,11 @@ 
 
                 $uri = $reroute;
 
-                if (! empty($params)) {
+                if (!empty($params)) {
                     $pat = '/(\$\d+)/';
                     $uri = preg_replace_callback(
                         $pat,
-                        function () use (&$params) {
+                        function() use (&$params) {
                             $first = $params[0];
                             array_shift($params);
                             return $first;