ByTIC / framework

src/form/element/Abstract.php

Doc Comments +18 added lines

@@ -67,6 +67,9 @@ 
         return $this;
     }
 
+    /**
+     * @return string
+     */
     public function getLabel() {
         return $this->getAttrib('label');
     }
@@ -98,6 +101,9 @@ 
         return $this->getDataFromRequest($data);
     }
 
+    /**
+     * @param boolean $isRequired
+     */
     public function setRequired($isRequired) {
         $this->_isRequired = (bool) $isRequired;
         return $this;
@@ -107,6 +113,9 @@ 
         return (bool) $this->_isRequired;
     }
 
+    /**
+     * @param boolean $isRendered
+     */
     public function setRendered($isRendered) {
         $this->_isRendered = (bool) $isRendered;
         return $this;
@@ -226,6 +235,9 @@ 
         return $this->_attribs[$key];
     }
 
+    /**
+     * @param string $key
+     */
     public function delAttrib($key) {
         $key = (string) $key;
         unset($this->_attribs[$key]);
@@ -264,6 +276,9 @@ 
         return $this;
     }
 
+    /**
+     * @param string $key
+     */
     public function getOption($key)
     {
         $key = (string) $key;
@@ -288,6 +303,9 @@ 
         return $this;
     }
 
+    /**
+     * @param boolean $position
+     */
     public function getDecoratorsByPosition($position) {
         return $this->_decorators[$position];
     }

Indentation +7 added lines, -7 removed lines

@@ -144,13 +144,13 @@
     public function validate() {
         if ($this->isRequired() && !$this->getValue()) {
             $message = $this->getForm()->getMessageTemplate('no-' . $this->getName());
-			if (!$message) {
-				$translateSlug = 'general.form.errors.required';
-				$message = nip__($translateSlug, array('label' => $this->getLabel()));
-				if ($message == $translateSlug) {
-					$message = $message ? $message : 'The field `' . $this->getLabel() . '` is mandatory.';
-				}
-			}
+            if (!$message) {
+                $translateSlug = 'general.form.errors.required';
+                $message = nip__($translateSlug, array('label' => $this->getLabel()));
+                if ($message == $translateSlug) {
+                    $message = $message ? $message : 'The field `' . $this->getLabel() . '` is mandatory.';
+                }
+            }
             $this->addError($message);
         }
     }

Spacing +2 added lines, -2 removed lines

@@ -50,7 +50,7 @@ 
     public function getJSID()
     {
         $name = $this->getUniqueId();        
-        return str_replace(array('][','[',']'), array('-','-',''), $this->getUniqueId());
+        return str_replace(array('][', '[', ']'), array('-', '-', ''), $this->getUniqueId());
     }
 
     public function setName($name) {
@@ -191,7 +191,7 @@ 
     /**
      * @return Nip_Form_Element_Abstract
      */
-    public function setAttrib($key, $value)  {
+    public function setAttrib($key, $value) {
         $key = (string) $key;
         $this->_attribs[$key] = $value;
         return $this;

src/form/element/Checkbox.php

Doc Comments +3 added lines

@@ -25,6 +25,9 @@
         return $this->getAttrib('checked') == 'checked';
     }
 
+    /**
+     * @param boolean $checked
+     */
     public function setChecked($checked) {
         if ($checked === true) {
             $this->setAttrib('checked', 'checked');

Indentation +7 added lines, -7 removed lines

@@ -8,17 +8,17 @@
         $this->setAttrib('type', 'checkbox');
     }
 
-	public function getDataFromRequest($request) {
-		$this->setChecked($request != null);			
+    public function getDataFromRequest($request) {
+        $this->setChecked($request != null);			
         return parent::getDataFromRequest($request);
     }
 	
     public function getDataFromModel($value) {
-		$inputValue = $this->getValue();
-		if ($inputValue == null && $value) {
-			$this->setChecked(true);
-		}
-		return parent::getDataFromModel($data);
+        $inputValue = $this->getValue();
+        if ($inputValue == null && $value) {
+            $this->setChecked(true);
+        }
+        return parent::getDataFromModel($data);
     }
 	
     public function isChecked() {

src/form/element/Dateinput.php

Doc Comments +3 added lines

@@ -46,6 +46,9 @@
         return $this->_hasTime;
     }
 
+    /**
+     * @return string
+     */
     public function getValue($requester = 'abstract')
     {
         $value = parent::getValue($requester);

src/form/renderer/DisplayGroup.php

Doc Comments +1 added lines, -2 removed lines

@@ -10,7 +10,6 @@ 
 
 
     /**
-     * @param  Nip_Form_DisplayGroup $form
      * @return Nip_Form_Renderer_DisplayGroup
      */
     public function setGroup(Nip_Form_DisplayGroup $group)
@@ -20,7 +19,7 @@ 
     }
 
     /**
-     * @return Nip_Form_Renderer_DisplayGroup|null
+     * @return Nip_Form_DisplayGroup
      */
     public function getGroup()
     {

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
     
     public function render()
     {
-        $return = '<fieldset'.$this->renderAttributes().'>';
+        $return = '<fieldset' . $this->renderAttributes() . '>';
         $return .= '<legend>' . $this->getGroup()->getLegend() . '</legend>';
 
         $renderer = clone $this->getGroup()->getForm()->getRenderer();

src/functions/common.php

Doc Comments +7 added lines, -1 removed lines

@@ -84,7 +84,6 @@ 
  * Transforms a date's string representation into $format
  *
  * @param string $format
- * @param string|int $datetime
  * @return string/bool
  */
 function _strtotime($date, $format = false)
@@ -111,6 +110,9 @@ 
 
 if (!function_exists("pluck")) {
 
+    /**
+     * @param Nip_RecordCollection $array
+     */
     function pluck($array, $property)
     {
         return Nip_Helper_Array::instance()->pluck($array, $property);
@@ -275,6 +277,10 @@ 
 }
 
 if (!function_exists("money_format")) {
+
+    /**
+     * @param string $format
+     */
     function money_format($format, $number)
     {
         $regex = '/%((?:[\^!\-]|\+|\(|\=.)*)([0-9]+)?' .

Indentation +1 added lines, -1 removed lines

@@ -366,7 +366,7 @@
 if (!function_exists("json_decode")) {
 
     function json_decode($json, $assoc = FALSE, /* emu_args */
-                         $n = 0, $state = 0, $waitfor = 0)
+                            $n = 0, $state = 0, $waitfor = 0)
     {
 
         #-- result var

Spacing +11 added lines, -12 removed lines

@@ -125,11 +125,11 @@ 
 
     $unit = strtoupper(substr($post_max_size, -1));
     $multiplier = ($unit == 'M' ? 1048576 : ($unit == 'K' ? 1024 : ($unit == 'G' ? 1073741824 : 1)));
-    $post_max_size = ((int)$post_max_size) * $multiplier;
+    $post_max_size = ((int) $post_max_size) * $multiplier;
 
     $unit = strtoupper(substr($upload_max_filesize, -1));
     $multiplier = ($unit == 'M' ? 1048576 : ($unit == 'K' ? 1024 : ($unit == 'G' ? 1073741824 : 1)));
-    $upload_max_filesize = ((int)$upload_max_filesize) * $multiplier;
+    $upload_max_filesize = ((int) $upload_max_filesize) * $multiplier;
 
     return round((min($post_max_size, $upload_max_filesize) / 1048576), 2) . 'MB';
 }
@@ -220,8 +220,7 @@ 
         }
         /* Split digits and add. */
         $checksum += $current_num % 10;
-        if
-        ($current_num > 9
+        if ($current_num > 9
         ) {
             $checksum += 1;
         }
@@ -295,9 +294,9 @@ 
                 'nosimbol' => preg_match('/\!/', $fmatch[1]) > 0,
                 'isleft' => preg_match('/\-/', $fmatch[1]) > 0
             );
-            $width = trim($fmatch[2]) ? (int)$fmatch[2] : 0;
-            $left = trim($fmatch[3]) ? (int)$fmatch[3] : 0;
-            $right = trim($fmatch[4]) ? (int)$fmatch[4] : $locale['int_frac_digits'];
+            $width = trim($fmatch[2]) ? (int) $fmatch[2] : 0;
+            $left = trim($fmatch[3]) ? (int) $fmatch[3] : 0;
+            $right = trim($fmatch[4]) ? (int) $fmatch[4] : $locale['int_frac_digits'];
             $conversion = $fmatch[5];
 
             $positive = true;
@@ -416,7 +415,7 @@ 
                 }
             } #-> end of sub-call (array/object)
             elseif ($waitfor && (strpos($waitfor, $c) !== false)) {
-                return array($val, $n);  // return current value and state
+                return array($val, $n); // return current value and state
             } #-= in-array
             elseif ($state === ']') {
                 list($v, $n) = json_decode($json, 0, $n, 0, ",]");
@@ -426,7 +425,7 @@ 
                 }
             } #-= in-object
             elseif ($state === '}') {
-                list($i, $n) = json_decode($json, 0, $n, 0, ":");   // this allowed non-string indicies
+                list($i, $n) = json_decode($json, 0, $n, 0, ":"); // this allowed non-string indicies
                 list($v, $n) = json_decode($json, 0, $n + 1, 0, ",}");
                 $val[$i] = $v;
                 if ($json[$n] == "}") {
@@ -464,15 +463,15 @@ 
                     $val = $uu[1];
                     $n += strlen($uu[0]) - 1;
                     if (strpos($val, ".")) {  // float
-                        $val = (float)$val;
+                        $val = (float) $val;
                     } elseif ($val[0] == "0") {  // oct
                         $val = octdec($val);
                     } else {
-                        $val = (int)$val;
+                        $val = (int) $val;
                     }
                     // exponent?
                     if (isset($uu[2])) {
-                        $val *= pow(10, (int)$uu[2]);
+                        $val *= pow(10, (int) $uu[2]);
                     }
                 } #-> boolean or null
                 elseif (preg_match("#^(true|false|null)\b#", substr($json, $n), $uu)) {

src/Helpers/Array.php

Doc Comments -1 removed lines

@@ -177,7 +177,6 @@
      *
      * @param array $array
      * @param array $params
-     * @param string $key
      * @return array
      */
     public function findAll($array, $params, $returnKey = false)

Braces +6 added lines, -4 removed lines

@@ -237,8 +237,9 @@ 
     public function toXML($data, $rootNodeName = 'ResultSet', &$xml = null)
     {
         // turn off compatibility mode as simple xml throws a wobbly if you don't.
-        if (ini_get('zend.ze1_compatibility_mode') == 1)
-            ini_set('zend.ze1_compatibility_mode', 0);
+        if (ini_get('zend.ze1_compatibility_mode') == 1) {
+                    ini_set('zend.ze1_compatibility_mode', 0);
+        }
 
         if (is_null($xml)) {
             $xml = simplexml_load_string("<?xml version='1.0' encoding='utf-8'?><$rootNodeName />");
@@ -260,8 +261,9 @@ 
                 $node = $this->isAssoc($value) || $numeric ? $xml->addChild($key) : $xml;
 
                 // recursive call
-                if ($numeric)
-                    $key = 'anon';
+                if ($numeric) {
+                                    $key = 'anon';
+                }
                 $this->toXML($value, $key, $node);
             } else {
                 // add single node.

src/Helpers/Color.php

Doc Comments +4 added lines, -1 removed lines

@@ -41,6 +41,9 @@ 
     }
 
 
+    /**
+     * @param string $color
+     */
     public function hsl($color)
     {
         list($r, $g, $b) = $this->rgb($color);
@@ -135,7 +138,7 @@ 
      *
      * @param string|array $color1
      * @param string|array $color2
-     * @return int
+     * @return double
      */
     public function lumDiff($color1, $color2)
     {

src/Helpers/Time.php

Doc Comments +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
 	/**
 	 * Singleton
 	 *
-	 * @return Nip_Helper_Date
+	 * @return Nip_Helper_Time
 	 */
 	static public function instance()
 	{

Indentation +28 added lines, -28 removed lines

@@ -12,42 +12,42 @@
         return $minutes;
     }
 	
-	public function secondsInStringTime($seconds)
-	{
-		if ($days = intval((floor($seconds / 86400)))) {
+    public function secondsInStringTime($seconds)
+    {
+        if ($days = intval((floor($seconds / 86400)))) {
             $seconds = $seconds - $days*86400;
-			$return .= ($return ? ' ' : '') . str_pad($days, 2, 0,STR_PAD_LEFT).'d';
-		}
-		if ($hours = intval((floor($seconds / 3600))) OR $return) {
+            $return .= ($return ? ' ' : '') . str_pad($days, 2, 0,STR_PAD_LEFT).'d';
+        }
+        if ($hours = intval((floor($seconds / 3600))) OR $return) {
             $seconds = $seconds - $hours*3600;
-			$return .= ($return ? ' ' : '') . str_pad($hours, 2, 0,STR_PAD_LEFT).'h';
-		}
-		if ($minutes = intval((floor($seconds / 60))) OR $return) {
-			$seconds = $seconds - $minutes*60;
-			$return .= ($return ? ' ' : '') . str_pad($minutes, 2, 0, STR_PAD_LEFT).'m';
-		}
+            $return .= ($return ? ' ' : '') . str_pad($hours, 2, 0,STR_PAD_LEFT).'h';
+        }
+        if ($minutes = intval((floor($seconds / 60))) OR $return) {
+            $seconds = $seconds - $minutes*60;
+            $return .= ($return ? ' ' : '') . str_pad($minutes, 2, 0, STR_PAD_LEFT).'m';
+        }
         
-		$seconds = round($seconds, 2);  
+        $seconds = round($seconds, 2);  
         if ($seconds) {
             $return .= ($return ? ' ' : '') . str_pad($seconds, 2, 0, STR_PAD_LEFT).'s';
         }
 			
-		return $return;
-	}
+        return $return;
+    }
 
 
-	/**
-	 * Singleton
-	 *
-	 * @return Nip_Helper_Date
-	 */
-	static public function instance()
-	{
-		static $instance;
-		if (!($instance instanceof self)) {
-			$instance = new self();
-		}
-		return $instance;
-	}
+    /**
+     * Singleton
+     *
+     * @return Nip_Helper_Date
+     */
+    static public function instance()
+    {
+        static $instance;
+        if (!($instance instanceof self)) {
+            $instance = new self();
+        }
+        return $instance;
+    }
     
 }

Spacing +8 added lines, -8 removed lines

@@ -7,7 +7,7 @@ 
         $minutes = false;
         if (strpos($time, ':')) {            
             list($hours, $minutes) = explode(':', $time);
-            $minutes += $hours*60;
+            $minutes += $hours * 60;
         }
         return $minutes;
     }
@@ -15,21 +15,21 @@ 
 	public function secondsInStringTime($seconds)
 	{
 		if ($days = intval((floor($seconds / 86400)))) {
-            $seconds = $seconds - $days*86400;
-			$return .= ($return ? ' ' : '') . str_pad($days, 2, 0,STR_PAD_LEFT).'d';
+            $seconds = $seconds - $days * 86400;
+			$return .= ($return ? ' ' : '') . str_pad($days, 2, 0, STR_PAD_LEFT) . 'd';
 		}
 		if ($hours = intval((floor($seconds / 3600))) OR $return) {
-            $seconds = $seconds - $hours*3600;
-			$return .= ($return ? ' ' : '') . str_pad($hours, 2, 0,STR_PAD_LEFT).'h';
+            $seconds = $seconds - $hours * 3600;
+			$return .= ($return ? ' ' : '') . str_pad($hours, 2, 0, STR_PAD_LEFT) . 'h';
 		}
 		if ($minutes = intval((floor($seconds / 60))) OR $return) {
-			$seconds = $seconds - $minutes*60;
-			$return .= ($return ? ' ' : '') . str_pad($minutes, 2, 0, STR_PAD_LEFT).'m';
+			$seconds = $seconds - $minutes * 60;
+			$return .= ($return ? ' ' : '') . str_pad($minutes, 2, 0, STR_PAD_LEFT) . 'm';
 		}
         
 		$seconds = round($seconds, 2);  
         if ($seconds) {
-            $return .= ($return ? ' ' : '') . str_pad($seconds, 2, 0, STR_PAD_LEFT).'s';
+            $return .= ($return ? ' ' : '') . str_pad($seconds, 2, 0, STR_PAD_LEFT) . 's';
         }
 			
 		return $return;

src/Helpers/View/GoogleAnalytics.php

Doc Comments +3 added lines

@@ -26,6 +26,9 @@
         return Nip_Config::instance()->ANALYTICS->domain;
     }
 	
+    /**
+     * @param string $method
+     */
     public function addOperation($method, $params, $position = 'below')
     {        
 		if ($position == 'prepend') {

Indentation +55 added lines, -55 removed lines

@@ -28,13 +28,13 @@ 
 	
     public function addOperation($method, $params, $position = 'below')
     {        
-		if ($position == 'prepend') {
-			array_unshift($this->_operations, array($method, $params));
-		} else {
-			$this->_operations[] = array($method, $params);			
-		}
+        if ($position == 'prepend') {
+            array_unshift($this->_operations, array($method, $params));
+        } else {
+            $this->_operations[] = array($method, $params);			
+        }
 		
-		return $this;
+        return $this;
     }	
 
     /**
@@ -81,36 +81,36 @@ 
     public function parseTransactions($prefix = '') {
         $transactions = $this->getTransactions();
 
-		$prefix = $prefix ? $prefix.'.' : '';
+        $prefix = $prefix ? $prefix.'.' : '';
 		
-		if ($this->transactions) {
-			foreach ($this->transactions as $transaction) {
-				$this->addOperation($prefix.'_addTrans', array(					
-					$transaction->orderId,
-					$transaction->affiliation,
-					$transaction->total,
-					$transaction->tax,
-					$transaction->shipping,
-					$transaction->city,
-					$transaction->state,
-					$transaction->country
-				));
-
-				if ($transaction->items) {
-					foreach ($transaction->items as $item) {
-						$this->addOperation($prefix.'_addItem', array(	
-							$item->orderId,
-							$item->sku,
-							$item->name,
-							$item->category,
-							$item->price,
-							$item->quantity
-						));
-					}
-				}
-			}
-			$this->addOperation($prefix.'_trackTrans'); //submits transaction to the Analytics servers
-		}
+        if ($this->transactions) {
+            foreach ($this->transactions as $transaction) {
+                $this->addOperation($prefix.'_addTrans', array(					
+                    $transaction->orderId,
+                    $transaction->affiliation,
+                    $transaction->total,
+                    $transaction->tax,
+                    $transaction->shipping,
+                    $transaction->city,
+                    $transaction->state,
+                    $transaction->country
+                ));
+
+                if ($transaction->items) {
+                    foreach ($transaction->items as $item) {
+                        $this->addOperation($prefix.'_addItem', array(	
+                            $item->orderId,
+                            $item->sku,
+                            $item->name,
+                            $item->category,
+                            $item->price,
+                            $item->quantity
+                        ));
+                    }
+                }
+            }
+            $this->addOperation($prefix.'_trackTrans'); //submits transaction to the Analytics servers
+        }
     }	
 
     public function setPage($page)
@@ -131,7 +131,7 @@ 
         $this->addOperation('_setDomainName', $this->getDomain(), 'prepend');
         $this->addOperation('_setAccount', $this->getUA(), 'prepend');
 		
-		$this->parseTransactions('');
+        $this->parseTransactions('');
 		
         $return = '<script type="text/javascript">';
         $return .= 'var _gaq = _gaq || [];';
@@ -139,19 +139,19 @@ 
         foreach ($this->_operations as $operation) {			
             $return .= "_gaq.push([";
             $return .= "'{$operation[0]}'";
-			if (isset($operation[1]) && $operation[1] !== null) {
-				$return .= ",";
-				$params = array();
-				if (is_array($operation[1])) {
-					foreach ($operation[1] as $param) {
-						$params[] = $this->renderOperationParam($param);						
-					}
-				} else {
-					$params[] = $this->renderOperationParam($operation[1]);						
-				}
+            if (isset($operation[1]) && $operation[1] !== null) {
+                $return .= ",";
+                $params = array();
+                if (is_array($operation[1])) {
+                    foreach ($operation[1] as $param) {
+                        $params[] = $this->renderOperationParam($param);						
+                    }
+                } else {
+                    $params[] = $this->renderOperationParam($operation[1]);						
+                }
 																
-				$return .= implode(',', $params);
-			}
+                $return .= implode(',', $params);
+            }
             $return .= "]);";
         }
 
@@ -165,11 +165,11 @@ 
         return $return;
     }
 	
-	public function renderOperationParam($param)
-	{
-		if (is_bool($param)) {
-			return $param=== true ? 'true' : 'false';										
-		}
-		return "'{$param}'";			
-	}
+    public function renderOperationParam($param)
+    {
+        if (is_bool($param)) {
+            return $param=== true ? 'true' : 'false';										
+        }
+        return "'{$param}'";			
+    }
 }
\ No newline at end of file

Spacing +5 added lines, -5 removed lines

@@ -81,11 +81,11 @@ 
     public function parseTransactions($prefix = '') {
         $transactions = $this->getTransactions();
 
-		$prefix = $prefix ? $prefix.'.' : '';
+		$prefix = $prefix ? $prefix . '.' : '';
 		
 		if ($this->transactions) {
 			foreach ($this->transactions as $transaction) {
-				$this->addOperation($prefix.'_addTrans', array(					
+				$this->addOperation($prefix . '_addTrans', array(					
 					$transaction->orderId,
 					$transaction->affiliation,
 					$transaction->total,
@@ -98,7 +98,7 @@ 
 
 				if ($transaction->items) {
 					foreach ($transaction->items as $item) {
-						$this->addOperation($prefix.'_addItem', array(	
+						$this->addOperation($prefix . '_addItem', array(	
 							$item->orderId,
 							$item->sku,
 							$item->name,
@@ -109,7 +109,7 @@ 
 					}
 				}
 			}
-			$this->addOperation($prefix.'_trackTrans'); //submits transaction to the Analytics servers
+			$this->addOperation($prefix . '_trackTrans'); //submits transaction to the Analytics servers
 		}
     }	
 
@@ -168,7 +168,7 @@ 
 	public function renderOperationParam($param)
 	{
 		if (is_bool($param)) {
-			return $param=== true ? 'true' : 'false';										
+			return $param === true ? 'true' : 'false';										
 		}
 		return "'{$param}'";			
 	}