las93 / venus3

bundles/lib/Date.php

Doc Comments -2 removed lines

@@ -192,8 +192,6 @@
 	 * set name of image
 	 *
 	 * @access public
-	 * @param  int $iWeek number of week
-	 * @param  int $iYear year
 	 * @return array
 	 */
 	public static function getActualMiddleWeek() : array

Spacing +10 added lines, -10 removed lines

@@ -41,12 +41,12 @@ 
 	 */
 	public static function getWeek(int $iWeek, int $iYear, string $sFormat = "Y-m-d") : Date
 	{
-		$iFirstDayInYear = date("N",mktime(0, 0, 0, 1, 1, $iYear));
+		$iFirstDayInYear = date("N", mktime(0, 0, 0, 1, 1, $iYear));
 
 		if ($iFirstDayInYear < 5) { $iShift = -($iFirstDayInYear - 1) * 86400; }
 		else { $iShift = (8 - $iFirstDayInYear) * 86400; }
 
-		if ($iWeek > 1) { $iWeekInSeconds = ($iWeek-1) * 604800; }
+		if ($iWeek > 1) { $iWeekInSeconds = ($iWeek - 1) * 604800; }
 		else { $iWeekInSeconds = 0; }
 
 		$iTimestamp = mktime(0, 0, 0, 1, 1, $iYear) + $iWeekInSeconds + $iShift;
@@ -158,12 +158,12 @@ 
 	 */
 	public static function getMiddleWeek(int $iWeek, int $iYear, string $sFormat = "Y-m-d") : array
 	{
-		$iFirstDayInYear = date("N",mktime(0, 0, 0, 1, 1, $iYear));
+		$iFirstDayInYear = date("N", mktime(0, 0, 0, 1, 1, $iYear));
 
 		if ($iFirstDayInYear < 5) { $iShift = -($iFirstDayInYear - 1) * 86400; }
 		else { $iShift = (8 - $iFirstDayInYear) * 86400; }
 
-		if ($iWeek > 1) { $iWeekInSeconds = ($iWeek-1) * 604800; }
+		if ($iWeek > 1) { $iWeekInSeconds = ($iWeek - 1) * 604800; }
 		else { $iWeekInSeconds = 0; }
 
 		if (date('N') > 2) {
@@ -182,7 +182,7 @@ 
 		if (preg_replace('/^([0-9]+)-[0-9]+-[0-9]+$/', '$1', $aDates[0]) != date('Y')) {
 
 			$aDates[0] = preg_replace('/^[0-9]+(-[0-9]+-[0-9]+)$/', date('Y').'$1', $aDates[0]);
-			$aDates[1] = preg_replace('/^[0-9]+(-[0-9]+-[0-9]+)$/', (date('Y')+1).'$1', $aDates[1]);
+			$aDates[1] = preg_replace('/^[0-9]+(-[0-9]+-[0-9]+)$/', (date('Y') + 1).'$1', $aDates[1]);
 		}
 
 		return $aDates;
@@ -225,10 +225,10 @@ 
 		$oDateTime = DateTime::createFromFormat('Y-m-d H:i:s', $sDateTime);
 		$iTimeStamp = time() - $oDateTime->getTimestamp();
 
-		if ($iTimeStamp < 3600) { return $sStartReturn.' '.(int)($iTimeStamp/60).' '.$sMinutes.$sEndReturn; }
-		if ($iTimeStamp < 86400) { return $sStartReturn.' '.(int)($iTimeStamp/3600).' '.$sHours.$sEndReturn; }
-		if ($iTimeStamp < 2592000) { return $sStartReturn.' '.(int)($iTimeStamp/86400).' '.$sDays.$sEndReturn; }
-		if ($iTimeStamp < 31536000) { return $sStartReturn.' '.(int)($iTimeStamp/2592000).' '.$sMonths.$sEndReturn; }
-		else { return $sStartReturn.' '.(int)($iTimeStamp/31536000).' '.$sYears.$sEndReturn; }
+		if ($iTimeStamp < 3600) { return $sStartReturn.' '.(int)($iTimeStamp / 60).' '.$sMinutes.$sEndReturn; }
+		if ($iTimeStamp < 86400) { return $sStartReturn.' '.(int)($iTimeStamp / 3600).' '.$sHours.$sEndReturn; }
+		if ($iTimeStamp < 2592000) { return $sStartReturn.' '.(int)($iTimeStamp / 86400).' '.$sDays.$sEndReturn; }
+		if ($iTimeStamp < 31536000) { return $sStartReturn.' '.(int)($iTimeStamp / 2592000).' '.$sMonths.$sEndReturn; }
+		else { return $sStartReturn.' '.(int)($iTimeStamp / 31536000).' '.$sYears.$sEndReturn; }
 	}
 }

Braces +9 added lines, -33 removed lines

@@ -43,11 +43,9 @@ 
 	{
 		$iFirstDayInYear = date("N",mktime(0, 0, 0, 1, 1, $iYear));
 
-		if ($iFirstDayInYear < 5) { $iShift = -($iFirstDayInYear - 1) * 86400; }
-		else { $iShift = (8 - $iFirstDayInYear) * 86400; }
+		if ($iFirstDayInYear < 5) { $iShift = -($iFirstDayInYear - 1) * 86400; } else { $iShift = (8 - $iFirstDayInYear) * 86400; }
 
-		if ($iWeek > 1) { $iWeekInSeconds = ($iWeek-1) * 604800; }
-		else { $iWeekInSeconds = 0; }
+		if ($iWeek > 1) { $iWeekInSeconds = ($iWeek-1) * 604800; } else { $iWeekInSeconds = 0; }
 
 		$iTimestamp = mktime(0, 0, 0, 1, 1, $iYear) + $iWeekInSeconds + $iShift;
 		$iTimestampLastDay = mktime(0, 0, 0, 1, 6, $iYear) + $iWeekInSeconds + $iShift + 604800;
@@ -78,18 +76,7 @@ 
 	{
 		if ($sLanguage == 'fr') {
 
-			if ($sMonth == '01' || $sMonth == 1) { return 'Janvier'; }
-			else if ($sMonth == '02' || $sMonth == 2) { return 'Février'; }
-			else if ($sMonth == '03' || $sMonth == 3) { return 'Mars'; }
-			else if ($sMonth == '04' || $sMonth == 4) { return 'Avril'; }
-			else if ($sMonth == '05' || $sMonth == 5) { return 'Mai'; }
-			else if ($sMonth == '06' || $sMonth == 6) { return 'Juin'; }
-			else if ($sMonth == '07' || $sMonth == 7) { return 'Juillet'; }
-			else if ($sMonth == '08' || $sMonth == 8) { return 'Août'; }
-			else if ($sMonth == '09' || $sMonth == 9) { return 'Septembre'; }
-			else if ($sMonth == 10) { return 'Octobre'; }
-			else if ($sMonth == 11) { return 'Novembre'; }
-			else if ($sMonth == 12) { return 'Décembre'; }
+			if ($sMonth == '01' || $sMonth == 1) { return 'Janvier'; } else if ($sMonth == '02' || $sMonth == 2) { return 'Février'; } else if ($sMonth == '03' || $sMonth == 3) { return 'Mars'; } else if ($sMonth == '04' || $sMonth == 4) { return 'Avril'; } else if ($sMonth == '05' || $sMonth == 5) { return 'Mai'; } else if ($sMonth == '06' || $sMonth == 6) { return 'Juin'; } else if ($sMonth == '07' || $sMonth == 7) { return 'Juillet'; } else if ($sMonth == '08' || $sMonth == 8) { return 'Août'; } else if ($sMonth == '09' || $sMonth == 9) { return 'Septembre'; } else if ($sMonth == 10) { return 'Octobre'; } else if ($sMonth == 11) { return 'Novembre'; } else if ($sMonth == 12) { return 'Décembre'; }
 		}
 	}
 
@@ -105,13 +92,7 @@ 
 	{
 		if ($sLanguage == 'fr') {
 
-			if ($sDay == 0) { return 'dimanche'; }
-			else if ($sDay == 1) { return 'lundi'; }
-			else if ($sDay == 2) { return 'mardi'; }
-			else if ($sDay == 3) { return 'mercredi'; }
-			else if ($sDay == 4) { return 'jeudi'; }
-			else if ($sDay == 5) { return 'vendredi'; }
-			else if ($sDay == 6) { return 'samedi'; }
+			if ($sDay == 0) { return 'dimanche'; } else if ($sDay == 1) { return 'lundi'; } else if ($sDay == 2) { return 'mardi'; } else if ($sDay == 3) { return 'mercredi'; } else if ($sDay == 4) { return 'jeudi'; } else if ($sDay == 5) { return 'vendredi'; } else if ($sDay == 6) { return 'samedi'; }
 		}
 	}
 
@@ -138,8 +119,7 @@ 
 			if ($iMonth == $aToday['mois']) {
 
 				if ($iDay > $aToday['jour']) { $iYears--; }
-			}
-			else {
+			} else {
 
 				$iYears--;
 			}
@@ -160,18 +140,15 @@ 
 	{
 		$iFirstDayInYear = date("N",mktime(0, 0, 0, 1, 1, $iYear));
 
-		if ($iFirstDayInYear < 5) { $iShift = -($iFirstDayInYear - 1) * 86400; }
-		else { $iShift = (8 - $iFirstDayInYear) * 86400; }
+		if ($iFirstDayInYear < 5) { $iShift = -($iFirstDayInYear - 1) * 86400; } else { $iShift = (8 - $iFirstDayInYear) * 86400; }
 
-		if ($iWeek > 1) { $iWeekInSeconds = ($iWeek-1) * 604800; }
-		else { $iWeekInSeconds = 0; }
+		if ($iWeek > 1) { $iWeekInSeconds = ($iWeek-1) * 604800; } else { $iWeekInSeconds = 0; }
 
 		if (date('N') > 2) {
 
 			$iTimestamp = mktime(0, 0, 0, 1, 1, $iYear) + $iWeekInSeconds + $iShift + 172800;
 			$iTimestampLastDay = $iTimestamp + 604800;
-		}
-		else {
+		} else {
 
 			$iTimestamp = mktime(0, 0, 0, 1, 1, $iYear) + $iWeekInSeconds + $iShift - 432000;
 			$iTimestampLastDay = $iTimestamp + 604800;
@@ -228,7 +205,6 @@ 
 		if ($iTimeStamp < 3600) { return $sStartReturn.' '.(int)($iTimeStamp/60).' '.$sMinutes.$sEndReturn; }
 		if ($iTimeStamp < 86400) { return $sStartReturn.' '.(int)($iTimeStamp/3600).' '.$sHours.$sEndReturn; }
 		if ($iTimeStamp < 2592000) { return $sStartReturn.' '.(int)($iTimeStamp/86400).' '.$sDays.$sEndReturn; }
-		if ($iTimeStamp < 31536000) { return $sStartReturn.' '.(int)($iTimeStamp/2592000).' '.$sMonths.$sEndReturn; }
-		else { return $sStartReturn.' '.(int)($iTimeStamp/31536000).' '.$sYears.$sEndReturn; }
+		if ($iTimeStamp < 31536000) { return $sStartReturn.' '.(int)($iTimeStamp/2592000).' '.$sMonths.$sEndReturn; } else { return $sStartReturn.' '.(int)($iTimeStamp/31536000).' '.$sYears.$sEndReturn; }
 	}
 }

bundles/lib/Facebook.php

Doc Comments +5 added lines, -5 removed lines

@@ -150,7 +150,7 @@ 
 	 * add one scope
 	 *
 	 * @access public
-	 * @return object
+	 * @return Object
 	 */
 	public function getUserInfo() : object
 	{
@@ -163,7 +163,7 @@ 
 	 *
 	 * @access public
 	 * @param  string $sName name to call
-	 * @return object
+	 * @return Object
 	 * @throws Exception
 	 */
 	public function getInfo(string $sName) : object
@@ -185,7 +185,7 @@ 
 	 *
 	 * @access public
 	 * @param  string $sFql fql query
-	 * @return object
+	 * @return Object
 	 * @throws Exception
 	 */
 	public function getInfoByFql(string $sFql) : object
@@ -217,7 +217,7 @@ 
 	 * @param string $sImg
 	 * @param string $sName
 	 * @param string $sDescription
-	 * @return object
+	 * @return Object
 	 * @throws Exception
 	 */
 	public function publishWallPost(string $sMessage, string $sLink, string $sImg, string $sName, string $sDescription) : object
@@ -250,7 +250,7 @@ 
 	 * @param string $sLink
 	 * @param string $sImg
 	 * @param string $sDescription
-	 * @return object
+	 * @return Object
 	 * @throws Exception
 	 */
 	public function publishMyWallPost(string $sPageId, string $sMessage, string $sLink, string $sImg, string $sDescription) : object

Indentation +6 added lines, -6 removed lines

@@ -227,12 +227,12 @@
 		try {
 
 			$this->_oFacebook->api('/'.$sUser.'/feed', 'post', array(
-                    'message' => $sMessage,
-                    'link'    => $sLink,
-                    'picture' => $sImg,
-                    'name'    => $sName,
-                    'description'=> $sDescription
-           		)
+					'message' => $sMessage,
+					'link'    => $sLink,
+					'picture' => $sImg,
+					'name'    => $sName,
+					'description'=> $sDescription
+		   		)
 			);
 		}
 		catch(Exception $e) {

Spacing +9 added lines, -9 removed lines

@@ -55,8 +55,8 @@ 
 	 * app secret of Facebook
 	 * @var array
 	 */
-	private $_aScope = array('email','user_birthday','user_location','user_work_history',
-					'user_about_me','user_hometown');
+	private $_aScope = array('email', 'user_birthday', 'user_location', 'user_work_history',
+					'user_about_me', 'user_hometown');
 
 	/**
 	 * object Facebook of the SDK
@@ -174,7 +174,7 @@ 
 
 			return $this->_oFacebook->api('/'.$sUser.'/'.$sName);
 		}
-		catch(Exception $e) {
+		catch (Exception $e) {
 
 			throw new Exception(__FILE__.':'.__METHOD__.' : You can\'t ask: '.'/'.$sUser.'/'.$sName.' !');
 		}
@@ -194,7 +194,7 @@ 
 
 		try {
 
-			$aParam  =   array(
+			$aParam = array(
 				'method' => 'fql.query',
 				'query' => $sFql,
 				'callback' => ''
@@ -202,7 +202,7 @@ 
 
 			return $this->_oFacebook->api($aParam);
 		}
-		catch(Exception $e) {
+		catch (Exception $e) {
 
 			throw new Exception(__FILE__.':'.__METHOD__.' : You can\'t ask: '.$sFql.' !');
 		}
@@ -235,7 +235,7 @@ 
            		)
 			);
 		}
-		catch(Exception $e) {
+		catch (Exception $e) {
 
 			throw new Exception(__FILE__.':'.__METHOD__.' : You can\'t ask: '.'/'.$sUser.'/feed'.' !');
 		}
@@ -257,9 +257,9 @@ 
 	{
 		$user = $this->_oFacebook->getUser();
 
-		if ($user){
+		if ($user) {
 
-			try{
+			try {
 
 				$accounts = $this->_oFacebook->api('/me/accounts');
 
@@ -290,7 +290,7 @@ 
 				)
 			);
 		}
-		catch(Exception $e) {
+		catch (Exception $e) {
 
 			var_dump($e);
 			throw new Exception(__FILE__.':'.__METHOD__.' : You can\'t ask: '.'/'.$sPageId.'/feed'.' !');

Braces +6 added lines, -12 removed lines

@@ -102,8 +102,7 @@ 
 		if ($this->_oFacebook->getUser()) {
 
 			return '<a href="'.$this->_oFacebook->getLogoutUrl().'">'.$sLabelName.'</a>';
-		}
-		else {
+		} else {
 
 			return '<a href="'.$this->_oFacebook->getLoginUrl(
 				array(
@@ -173,8 +172,7 @@ 
 		try {
 
 			return $this->_oFacebook->api('/'.$sUser.'/'.$sName);
-		}
-		catch(Exception $e) {
+		} catch(Exception $e) {
 
 			throw new Exception(__FILE__.':'.__METHOD__.' : You can\'t ask: '.'/'.$sUser.'/'.$sName.' !');
 		}
@@ -201,8 +199,7 @@ 
 			);
 
 			return $this->_oFacebook->api($aParam);
-		}
-		catch(Exception $e) {
+		} catch(Exception $e) {
 
 			throw new Exception(__FILE__.':'.__METHOD__.' : You can\'t ask: '.$sFql.' !');
 		}
@@ -234,8 +231,7 @@ 
                     'description'=> $sDescription
            		)
 			);
-		}
-		catch(Exception $e) {
+		} catch(Exception $e) {
 
 			throw new Exception(__FILE__.':'.__METHOD__.' : You can\'t ask: '.'/'.$sUser.'/feed'.' !');
 		}
@@ -265,8 +261,7 @@ 
 
 				echo '<pre>';
 				print_r($accounts); /* on affiche les informations retournées */
-			}
-			catch (\Exception $e) {
+			} catch (\Exception $e) {
 				
 				print_r($e);
 				$user = null;
@@ -289,8 +284,7 @@ 
 					'description'=> $sDescription
 				)
 			);
-		}
-		catch(Exception $e) {
+		} catch(Exception $e) {
 
 			var_dump($e);
 			throw new Exception(__FILE__.':'.__METHOD__.' : You can\'t ask: '.'/'.$sPageId.'/feed'.' !');

bundles/lib/Form.php

Doc Comments +1 added lines, -2 removed lines

@@ -304,7 +304,7 @@ 
 	 * get global object form
 	 *
 	 * @access public
-	 * @return object
+	 * @return \stdClass
 	 */
 	public function getFormInObject()
 	{
@@ -411,7 +411,6 @@ 
 	 * set the entity to synchronize with the formular
 	 *
 	 * @access public
-	 * @param  string $sSeparator separator between the fields
 	 * @param  int $iId id of the primary key
 	 * @return \Venus\lib\Form
 	 */

Indentation +52 added lines, -52 removed lines

@@ -141,14 +141,14 @@ 
 	 */
 	public function add($sName, $mType, $sLabel = null, $mValue = null, $mOptions = null)
 	{
-	    if ($mType instanceof Container) {
+		if ($mType instanceof Container) {
 	        
-	        $this->_aElement[$sName] = $mType;
-	    }
+			$this->_aElement[$sName] = $mType;
+		}
 		else if ($mType === 'text' || $mType === 'submit' || $mType === 'password' || $mType === 'file' || $mType === 'tel'
-	        || $mType === 'url' || $mType === 'email' || $mType === 'search' || $mType === 'date' || $mType === 'time'
-	        || $mType === 'datetime' || $mType === 'month' || $mType === 'week' || $mType === 'number' || $mType === 'range'
-	        || $mType === 'color') {
+			|| $mType === 'url' || $mType === 'email' || $mType === 'search' || $mType === 'date' || $mType === 'time'
+			|| $mType === 'datetime' || $mType === 'month' || $mType === 'week' || $mType === 'number' || $mType === 'range'
+			|| $mType === 'color') {
 
 			$this->_aElement[$sName] = new Input($sName, $mType, $sLabel, $mValue);
 		}
@@ -294,7 +294,7 @@ 
 
 		$oContainer = new Container;
 		$oContainer->setView($sFormContent)
-		           ->setForm($this);
+				   ->setForm($this);
 		
 		return $oContainer;
 	}
@@ -308,67 +308,67 @@ 
 	 */
 	public function getFormInObject()
 	{
-	    if ($this->_iIdEntity > 0 && $this->_sSynchronizeEntity !== null && count($_POST) < 1) {
+		if ($this->_iIdEntity > 0 && $this->_sSynchronizeEntity !== null && count($_POST) < 1) {
 	
-	        $sModelName = str_replace('Entity', 'Model', $this->_sSynchronizeEntity);
-	        $oModel = new $sModelName;
+			$sModelName = str_replace('Entity', 'Model', $this->_sSynchronizeEntity);
+			$oModel = new $sModelName;
 	        	
-	        $oEntity = new $this->_sSynchronizeEntity;
-	        $sPrimaryKey = LibEntity::getPrimaryKeyNameWithoutMapping($oEntity);
-	        $sMethodName = 'findOneBy'.$sPrimaryKey;
-	        $oCompleteEntity = call_user_func_array(array(&$oModel, $sMethodName), array($this->_iIdEntity));
+			$oEntity = new $this->_sSynchronizeEntity;
+			$sPrimaryKey = LibEntity::getPrimaryKeyNameWithoutMapping($oEntity);
+			$sMethodName = 'findOneBy'.$sPrimaryKey;
+			$oCompleteEntity = call_user_func_array(array(&$oModel, $sMethodName), array($this->_iIdEntity));
 	
-	        if (is_object($oCompleteEntity)) {
+			if (is_object($oCompleteEntity)) {
 	
-	            foreach ($this->_aElement as $sKey => $sValue) {
+				foreach ($this->_aElement as $sKey => $sValue) {
 	
-	                if ($sValue instanceof \Venus\lib\Form\Radio) {
+					if ($sValue instanceof \Venus\lib\Form\Radio) {
 	
-	                    $sExKey = $sKey;
-	                    $sKey = substr($sKey, 0, -6);
-	                }
+						$sExKey = $sKey;
+						$sKey = substr($sKey, 0, -6);
+					}
 	                	
-	                if ($sValue instanceof Form) {
+					if ($sValue instanceof Form) {
 	                    
-	                    ;
-	                }
-	                else {
+						;
+					}
+					else {
 	                 
-    	                $sMethodNameInEntity = 'get_'.$sKey;
-    	                $mValue = $oCompleteEntity->$sMethodNameInEntity();
+						$sMethodNameInEntity = 'get_'.$sKey;
+						$mValue = $oCompleteEntity->$sMethodNameInEntity();
     	
-    	                if ($sValue instanceof \Venus\lib\Form\Radio && method_exists($this->_aElement[$sExKey], 'setValueChecked')) {
+						if ($sValue instanceof \Venus\lib\Form\Radio && method_exists($this->_aElement[$sExKey], 'setValueChecked')) {
     	
-    	                    $this->_aElement[$sExKey]->setValueChecked($mValue);
-    	                }
-    	                else if (isset($mValue) && method_exists($this->_aElement[$sKey], 'setValue')) {
+							$this->_aElement[$sExKey]->setValueChecked($mValue);
+						}
+						else if (isset($mValue) && method_exists($this->_aElement[$sKey], 'setValue')) {
     	
-    	                    $this->_aElement[$sKey]->setValue($mValue);
-    	                }
-	                }
-	            }
-	        }
-	    }
+							$this->_aElement[$sKey]->setValue($mValue);
+						}
+					}
+				}
+			}
+		}
 	
-	    $oForm = new \StdClass();
-	    $oForm->start = '<form name="form'.$this->_iFormNumber.'" method="post"><input type="hidden" value="1" name="validform'.$this->_iFormNumber.'">';
-	    $oForm->form = array();
+		$oForm = new \StdClass();
+		$oForm->start = '<form name="form'.$this->_iFormNumber.'" method="post"><input type="hidden" value="1" name="validform'.$this->_iFormNumber.'">';
+		$oForm->form = array();
 	
-	    foreach ($this->_aElement as $sKey => $sValue) {
+		foreach ($this->_aElement as $sKey => $sValue) {
 
-	        if ($sValue instanceof Container) {
+			if ($sValue instanceof Container) {
 	        
-	            $oForm->form[$sKey] = $sValue;
-	        }
-	        else {
+				$oForm->form[$sKey] = $sValue;
+			}
+			else {
 	            
-	            $oForm->form[$sKey] = $sValue->fetch();
-	        }
-	    }
+				$oForm->form[$sKey] = $sValue->fetch();
+			}
+		}
 	
-	    $oForm->end = '</form>';
+		$oForm->end = '</form>';
 	
-	    return $oForm;
+		return $oForm;
 	}
 
 	/**
@@ -449,7 +449,7 @@ 
 	 */
 	public function getElement() {
 	    
-	    return $this->_aElement;
+		return $this->_aElement;
 	}
 	
 	/**
@@ -460,7 +460,7 @@ 
 	 */
 	public function getIdEntity() {
 	    
-	    return $this->_iIdEntity;
+		return $this->_iIdEntity;
 	}
 	
 	/**
@@ -471,6 +471,6 @@ 
 	 */
 	public function getSynchronizeEntity() {
 	    
-	    return $this->_sSynchronizeEntity;
+		return $this->_sSynchronizeEntity;
 	}
 }

Spacing +3 added lines, -3 removed lines

@@ -181,13 +181,13 @@ 
 		}
 		else  if ($mType === 'radio') {
 
-			$this->_aElement[$sName.rand(100000,999999)] = new Radio($sName, $sLabel, $mValue, $mOptions);
+			$this->_aElement[$sName.rand(100000, 999999)] = new Radio($sName, $sLabel, $mValue, $mOptions);
 		}
 		else  if ($mType === 'date') {
 
 			$aDay = array();
 
-			for ($i = 1 ; $i <= 31 ; $i++) {
+			for ($i = 1; $i <= 31; $i++) {
 
 				if ($i < 10) { $aDay['0'.$i] = '0'.$i; }
 				else { $aDay[$i] = $i; }
@@ -214,7 +214,7 @@ 
 
 			$aYear = array();
 
-			for ($i = 1900 ; $i <= 2013 ; $i++) {
+			for ($i = 1900; $i <= 2013; $i++) {
 
 				$aYear[$i] = $i;
 			}

Braces +12 added lines, -24 removed lines

@@ -144,27 +144,22 @@ 
 	    if ($mType instanceof Container) {
 	        
 	        $this->_aElement[$sName] = $mType;
-	    }
-		else if ($mType === 'text' || $mType === 'submit' || $mType === 'password' || $mType === 'file' || $mType === 'tel'
+	    } else if ($mType === 'text' || $mType === 'submit' || $mType === 'password' || $mType === 'file' || $mType === 'tel'
 	        || $mType === 'url' || $mType === 'email' || $mType === 'search' || $mType === 'date' || $mType === 'time'
 	        || $mType === 'datetime' || $mType === 'month' || $mType === 'week' || $mType === 'number' || $mType === 'range'
 	        || $mType === 'color') {
 
 			$this->_aElement[$sName] = new Input($sName, $mType, $sLabel, $mValue);
-		}
-		elseif ($mType === 'textarea') {
+		} elseif ($mType === 'textarea') {
 
 			$this->_aElement[$sName] = new Textarea($sName, $sLabel, $mValue);
-		}
-		else  if ($mType === 'select') {
+		} else  if ($mType === 'select') {
 
 			$this->_aElement[$sName] = new Select($sName, $mOptions, $sLabel, $mValue);
-		}
-		else  if ($mType === 'label') {
+		} else  if ($mType === 'label') {
 
 			$this->_aElement[$sName] = new Label($sName);
-		}
-		else  if ($mType === 'list_checkbox') {
+		} else  if ($mType === 'list_checkbox') {
 
 			$i = 0;
 			
@@ -174,23 +169,19 @@ 
 			
 				$this->_aElement[$sName.'_'.$i++] = new Checkbox($sName, $sValue, $mKey, $mOptions);
 			}
-		}
-		else  if ($mType === 'checkbox') {
+		} else  if ($mType === 'checkbox') {
 
 			$this->_aElement[$sName] = new Checkbox($sName, $sLabel, $mValue, $mOptions);
-		}
-		else  if ($mType === 'radio') {
+		} else  if ($mType === 'radio') {
 
 			$this->_aElement[$sName.rand(100000,999999)] = new Radio($sName, $sLabel, $mValue, $mOptions);
-		}
-		else  if ($mType === 'date') {
+		} else  if ($mType === 'date') {
 
 			$aDay = array();
 
 			for ($i = 1 ; $i <= 31 ; $i++) {
 
-				if ($i < 10) { $aDay['0'.$i] = '0'.$i; }
-				else { $aDay[$i] = $i; }
+				if ($i < 10) { $aDay['0'.$i] = '0'.$i; } else { $aDay[$i] = $i; }
 			}
 
 			$this->_aElement[$sName.'_day'] = new Select($sName, $aDay);
@@ -331,8 +322,7 @@ 
 	                if ($sValue instanceof Form) {
 	                    
 	                    ;
-	                }
-	                else {
+	                } else {
 	                 
     	                $sMethodNameInEntity = 'get_'.$sKey;
     	                $mValue = $oCompleteEntity->$sMethodNameInEntity();
@@ -340,8 +330,7 @@ 
     	                if ($sValue instanceof \Venus\lib\Form\Radio && method_exists($this->_aElement[$sExKey], 'setValueChecked')) {
     	
     	                    $this->_aElement[$sExKey]->setValueChecked($mValue);
-    	                }
-    	                else if (isset($mValue) && method_exists($this->_aElement[$sKey], 'setValue')) {
+    	                } else if (isset($mValue) && method_exists($this->_aElement[$sKey], 'setValue')) {
     	
     	                    $this->_aElement[$sKey]->setValue($mValue);
     	                }
@@ -359,8 +348,7 @@ 
 	        if ($sValue instanceof Container) {
 	        
 	            $oForm->form[$sKey] = $sValue;
-	        }
-	        else {
+	        } else {
 	            
 	            $oForm->form[$sKey] = $sValue->fetch();
 	        }

bundles/lib/Form/Container.php

Doc Comments +1 added lines, -1 removed lines

@@ -69,7 +69,7 @@
 	 * get the Value
 	 *
 	 * @access public
-	 * @return object
+	 * @return \stdClass
 	 */
 	public function createView()
 	{

Indentation +50 added lines, -50 removed lines

@@ -73,24 +73,24 @@ 
 	 */
 	public function createView()
 	{
-	    $oView = new \stdClass;
-	    $oView->form = $this->_sView;
-	    $oView->form_start = $this->_oForm->getFormInObject()->start;
-	    $oView->form_end = $this->_oForm->getFormInObject()->end;
-	    $oView->form_row = array();
+		$oView = new \stdClass;
+		$oView->form = $this->_sView;
+		$oView->form_start = $this->_oForm->getFormInObject()->start;
+		$oView->form_end = $this->_oForm->getFormInObject()->end;
+		$oView->form_row = array();
 	    
-	    foreach ($this->_oForm->getFormInObject()->form as $sKey => $mValue) {
+		foreach ($this->_oForm->getFormInObject()->form as $sKey => $mValue) {
 	        
-	        if ($mValue instanceof Container) {
+			if ($mValue instanceof Container) {
 	            
-	            $oNewForm = $mValue->createView();
-	            $oView->form_row[$sKey] = $oNewForm->form_row;
-	        }
-	        else {
+				$oNewForm = $mValue->createView();
+				$oView->form_row[$sKey] = $oNewForm->form_row;
+			}
+			else {
 	            
-	           $oView->form_row[$sKey] = $mValue;
-	        }
-	    }
+			   $oView->form_row[$sKey] = $mValue;
+			}
+		}
 	       
 		return $oView;
 	}
@@ -117,48 +117,48 @@ 
 	 */
 	public function handleRequest(array $aRequest) : bool
 	{
-	    if (!count($_POST)) { return true; }
+		if (!count($_POST)) { return true; }
 	    
-	    // Validation
-	    foreach ($this->_oForm->getElement() as $sKey => $sValue) {
+		// Validation
+		foreach ($this->_oForm->getElement() as $sKey => $sValue) {
 	    
-	        if (!$sValue instanceof self && !$this->_validate($sValue)) {
+			if (!$sValue instanceof self && !$this->_validate($sValue)) {
 
-	            return false;
-	        }
-	    }
+				return false;
+			}
+		}
 	    
-	    // Save
+		// Save
 		if ($this->_oForm->getIdEntity() > 0 && $this->_oForm->getSynchronizeEntity() !== null && count($aRequest) > 0) {
 		
-		    $sModelName = str_replace('Entity', 'Model', $this->_oForm->getSynchronizeEntity());
-		    $oModel = new $sModelName;
+			$sModelName = str_replace('Entity', 'Model', $this->_oForm->getSynchronizeEntity());
+			$oModel = new $sModelName;
 		
-		    $oEntity = new $this->_oForm->getSynchronizeEntity();
-		    $sPrimaryKey = LibEntity::getPrimaryKeyNameWithoutMapping($oEntity);
-		    $sMethodName = 'set_'.$sPrimaryKey;
+			$oEntity = new $this->_oForm->getSynchronizeEntity();
+			$sPrimaryKey = LibEntity::getPrimaryKeyNameWithoutMapping($oEntity);
+			$sMethodName = 'set_'.$sPrimaryKey;
 		
-		    call_user_func_array(array(&$oEntity, $sMethodName), array($this->_oForm->getIdEntity()));
+			call_user_func_array(array(&$oEntity, $sMethodName), array($this->_oForm->getIdEntity()));
 		
-		    foreach ($this->_oForm->getElement() as $sKey => $sValue) {
+			foreach ($this->_oForm->getElement() as $sKey => $sValue) {
 		
-		        $sMethodName = 'set_'.$sValue->getName().'';
-		        call_user_func_array(array(&$oEntity, $sMethodName), array($aRequest[$sValue->getName()]));
-		    }
+				$sMethodName = 'set_'.$sValue->getName().'';
+				call_user_func_array(array(&$oEntity, $sMethodName), array($aRequest[$sValue->getName()]));
+			}
 		
-		    $oEntity->save();
+			$oEntity->save();
 		}
 		else if ($this->_oForm->getSynchronizeEntity() !== null && isset($aRequest) && count($aRequest) > 0) {
 		
-		    $oEntity = new $this->_oForm->_sSynchronizeEntity;
+			$oEntity = new $this->_oForm->_sSynchronizeEntity;
 		
-		    foreach ($this->_oForm->getElement() as $sKey => $sValue) {
+			foreach ($this->_oForm->getElement() as $sKey => $sValue) {
 		
-		        $sMethodName = 'set_'.$sValue->getName().'';
-		        call_user_func_array(array(&$oEntity, $sMethodName), array($aRequest[$sValue->getName()]));
-		    }
+				$sMethodName = 'set_'.$sValue->getName().'';
+				call_user_func_array(array(&$oEntity, $sMethodName), array($aRequest[$sValue->getName()]));
+			}
 		
-		    $this->_oForm->setIdEntityCreated($oEntity->save());
+			$this->_oForm->setIdEntityCreated($oEntity->save());
 		}
 		
 		$this->_bHandleRequestActivate = true;
@@ -199,13 +199,13 @@ 
 	 */
 	public function isSubmitted() : bool
 	{
-	    if (isset($_POST['validform'.$this->_oForm->getFormNumber()]) && $_POST['validform'.$this->_oForm->getFormNumber()] == 1) {
+		if (isset($_POST['validform'.$this->_oForm->getFormNumber()]) && $_POST['validform'.$this->_oForm->getFormNumber()] == 1) {
 	        
-	        return true;
-	    }
+			return true;
+		}
 		else { 
 		    
-		    return false;
+			return false;
 		}
 	}
 
@@ -218,7 +218,7 @@ 
 	 */
 	public function isClicked(string $sElementName) : bool
 	{
-	    if (isset($_POST[$sElementName]) && $_POST[$sElementName]) { return true; }
+		if (isset($_POST[$sElementName]) && $_POST[$sElementName]) { return true; }
 		else { return false; }
 	}
 
@@ -231,14 +231,14 @@ 
 	 */
 	private function _validate($oElement) : bool
 	{
-	    foreach ($oElement->getConstraint() as $oConstraint) {
+		foreach ($oElement->getConstraint() as $oConstraint) {
 	        
-	        if (!$oConstraint->validate($_POST[$oElement->getName()])) {
+			if (!$oConstraint->validate($_POST[$oElement->getName()])) {
 	            
-	            return false;
-	        }
-	    }
+				return false;
+			}
+		}
 	    
-	    return true;
+		return true;
 	}
 }

Braces +5 added lines, -10 removed lines

@@ -85,8 +85,7 @@ 
 	            
 	            $oNewForm = $mValue->createView();
 	            $oView->form_row[$sKey] = $oNewForm->form_row;
-	        }
-	        else {
+	        } else {
 	            
 	           $oView->form_row[$sKey] = $mValue;
 	        }
@@ -147,8 +146,7 @@ 
 		    }
 		
 		    $oEntity->save();
-		}
-		else if ($this->_oForm->getSynchronizeEntity() !== null && isset($aRequest) && count($aRequest) > 0) {
+		} else if ($this->_oForm->getSynchronizeEntity() !== null && isset($aRequest) && count($aRequest) > 0) {
 		
 		    $oEntity = new $this->_oForm->_sSynchronizeEntity;
 		
@@ -187,8 +185,7 @@ 
 	 */
 	public function isValid() : bool
 	{
-		if ($this->_bHandleRequestActivate === true) { return true; }
-		else { return false; }
+		if ($this->_bHandleRequestActivate === true) { return true; } else { return false; }
 	}
 
 	/**
@@ -202,8 +199,7 @@ 
 	    if (isset($_POST['validform'.$this->_oForm->getFormNumber()]) && $_POST['validform'.$this->_oForm->getFormNumber()] == 1) {
 	        
 	        return true;
-	    }
-		else { 
+	    } else { 
 		    
 		    return false;
 		}
@@ -218,8 +214,7 @@ 
 	 */
 	public function isClicked(string $sElementName) : bool
 	{
-	    if (isset($_POST[$sElementName]) && $_POST[$sElementName]) { return true; }
-		else { return false; }
+	    if (isset($_POST[$sElementName]) && $_POST[$sElementName]) { return true; } else { return false; }
 	}
 
 	/**

bundles/lib/Form/Input.php

Doc Comments +1 added lines, -1 removed lines

@@ -151,7 +151,7 @@
 	 *
 	 * @access public
 	 * @param  string $sType type of input;
-	 * @return bool
+	 * @return string
 	 */
 	public function isClicked(string $sType) : string
 	{

Indentation +4 added lines, -4 removed lines

@@ -174,10 +174,10 @@
 		$sContent = '';
 
 		if ($this->getType() === 'text' || $this->getType() === 'password' || $this->getType() === 'file'
-            || $this->getType() === 'tel' || $this->getType() === 'url' || $this->getType() === 'email'
-            || $this->getType() === 'search' || $this->getType() === 'date' || $this->getType() === 'time'
-            || $this->getType() === 'datetime' || $this->getType() === 'month' || $this->getType() === 'week'
-            || $this->getType() === 'number' || $this->getType() === 'range' || $this->getType() === 'color') {
+			|| $this->getType() === 'tel' || $this->getType() === 'url' || $this->getType() === 'email'
+			|| $this->getType() === 'search' || $this->getType() === 'date' || $this->getType() === 'time'
+			|| $this->getType() === 'datetime' || $this->getType() === 'month' || $this->getType() === 'week'
+			|| $this->getType() === 'number' || $this->getType() === 'range' || $this->getType() === 'color') {
 
 			$sContent .= '<label>'.$this->getLabel().'</label> ';
 		}

Braces +1 added lines, -2 removed lines

@@ -60,8 +60,7 @@
 		$this->setName($sName);
 		$this->setValue($sValue);
 
-		if ($sLabel !== null) { $this->setLabel($sLabel); }
-		else { $this->setLabel($sName); }
+		if ($sLabel !== null) { $this->setLabel($sLabel); } else { $this->setLabel($sName); }
 	}
 
 	/**

bundles/lib/Form/Radio.php

Doc Comments +3 added lines, -3 removed lines

@@ -96,7 +96,7 @@ 
 	 *
 	 * @access public
 	 * @param  string $sValue Value of input;
-	 * @return \Venus\lib\Form\Input
+	 * @return Radio
 	 */
 	public function setValue(string $sValue) : Input
 	{
@@ -120,7 +120,7 @@ 
 	 *
 	 * @access public
 	 * @param  string $sValueChecked Value of input;
-	 * @return \Venus\lib\Form\Input
+	 * @return Radio
 	 */
 	public function setValueChecked(string $sValueChecked) : Input
 	{
@@ -144,7 +144,7 @@ 
 	 *
 	 * @access public
 	 * @param  string $sLabel Label of input;
-	 * @return \Venus\lib\Form\Input
+	 * @return Radio
 	 */
 	public function setLabel(string $sLabel) : Input
 	{

bundles/lib/Request.php

Doc Comments +1 added lines, -2 removed lines

@@ -59,8 +59,7 @@
 	 * if the request is ajax
 	 *
 	 * @access public
-	 * @param  string $sName name of the template
-	 * @return bool
+	 * @return boolean|null
 	 */
 	public static function isXmlHttpRequest()
 	{

Indentation +40 added lines, -40 removed lines

@@ -47,11 +47,11 @@ 
 	 */
 	public function __construct()
 	{
-        $this->cookies = function() { return new Cookies(); };
-        $this->files = function() { return new Files(); };
-        $this->headers = function() { return new Headers(); };
-        $this->query = function() { return new Query(); };
-        $this->request = function() { return new RequestPost(); };
+		$this->cookies = function() { return new Cookies(); };
+		$this->files = function() { return new Files(); };
+		$this->headers = function() { return new Headers(); };
+		$this->query = function() { return new Query(); };
+		$this->request = function() { return new RequestPost(); };
 		$this->server = function() { return new Server(); };
 	}
 
@@ -120,13 +120,13 @@ 
 	 *
 	 * @access public
 	 * @return bool
-     * @deprecated  don't use this method because they return a false result
-     *              delete in the version 5
-     * @throws \Exception
+	 * @deprecated  don't use this method because they return a false result
+	 *              delete in the version 5
+	 * @throws \Exception
 	 */
 	public static function getPreferredLanguage()
 	{
-        throw new \Exception("Use getLanguages() method now!");
+		throw new \Exception("Use getLanguages() method now!");
 	}
 
 	/**
@@ -171,8 +171,8 @@ 
 	 * @access public
 	 * @param  string $name
 	 * @return mixed
-     * @deprecated  please use $this->request->get()
-     *              delete in the version 5
+	 * @deprecated  please use $this->request->get()
+	 *              delete in the version 5
 	 */
 	public function getPost(string $name)
 	{
@@ -187,22 +187,22 @@ 
 	 */
 	public static function getPut() 
 	{	    
-	    $aPut = array();
+		$aPut = array();
 	    
-	    $rPutResource = fopen("php://input", "r");
+		$rPutResource = fopen("php://input", "r");
 	    
-	    while ($sData = fread($rPutResource, 1024)) {
+		while ($sData = fread($rPutResource, 1024)) {
 
-	        $aSeparatePut = explode('&', $sData);
+			$aSeparatePut = explode('&', $sData);
 	        
-	        foreach($aSeparatePut as $sOne) {
+			foreach($aSeparatePut as $sOne) {
 	            
-	            $aOnePut = explode('=', $sOne);
-	            $aPut[$aOnePut[0]] = $aOnePut[1];
-	        }
-	    }
+				$aOnePut = explode('=', $sOne);
+				$aPut[$aOnePut[0]] = $aOnePut[1];
+			}
+		}
 	    
-	    return $aPut;
+		return $aPut;
 	}
 
 	/**
@@ -221,29 +221,29 @@ 
 		else if ($iCode === 404) { header('HTTP/1.1 404 Not Found'); }
 	}
 
-    /**
-     * get http method
-     * @return string
-     */
-    public function getMethod() : string
-    {
-        return $this->server->get('REQUEST_METHOD');
-    }
-
-    /**
-     * return languages accepted by the customer
-     * @return array
-     */
-    public function getLanguages() : array
-    {
-        if (!self::isCliRequest()) { return explode(',', preg_replace('/^([^;]);?.*$/', '$1', $_SERVER['HTTP_ACCEPT_LANGUAGE'])); }
-        else { return array(); }
-    }
+	/**
+	 * get http method
+	 * @return string
+	 */
+	public function getMethod() : string
+	{
+		return $this->server->get('REQUEST_METHOD');
+	}
+
+	/**
+	 * return languages accepted by the customer
+	 * @return array
+	 */
+	public function getLanguages() : array
+	{
+		if (!self::isCliRequest()) { return explode(',', preg_replace('/^([^;]);?.*$/', '$1', $_SERVER['HTTP_ACCEPT_LANGUAGE'])); }
+		else { return array(); }
+	}
 
 	/**
 	 * get path info
 	 * @return string
-     */
+	 */
 	public function getPathInfo() : string
 	{
 		return $this->server->get('REQUEST_URI');

Spacing +4 added lines, -4 removed lines

@@ -195,7 +195,7 @@ 
 
 	        $aSeparatePut = explode('&', $sData);
 	        
-	        foreach($aSeparatePut as $sOne) {
+	        foreach ($aSeparatePut as $sOne) {
 	            
 	            $aOnePut = explode('=', $sOne);
 	            $aPut[$aOnePut[0]] = $aOnePut[1];
@@ -214,9 +214,9 @@ 
 	 */
 	public static function setStatus($iCode)
 	{
-		if ($iCode === 200) { header('HTTP/1.1 200 Ok');  }
-		else if ($iCode === 201) { header('HTTP/1.1 201 Created');  }
-		else if ($iCode === 204) { header("HTTP/1.0 204 No Content");  }
+		if ($iCode === 200) { header('HTTP/1.1 200 Ok'); }
+		else if ($iCode === 201) { header('HTTP/1.1 201 Created'); }
+		else if ($iCode === 204) { header("HTTP/1.0 204 No Content"); }
 		else if ($iCode === 403) { header('HTTP/1.1 403 Forbidden'); }
 		else if ($iCode === 404) { header('HTTP/1.1 404 Not Found'); }
 	}

Braces +9 added lines, -21 removed lines

@@ -69,8 +69,7 @@ 
 			if (array_key_exists('HTTP_X_REQUESTED_WITH', $_SERVER) && strtolower($_SERVER['HTTP_X_REQUESTED_WITH']) == 'xmlhttprequest') {
 
 				return true;
-			}
-			else {
+			} else {
 
 				return false;
 			}
@@ -85,8 +84,7 @@ 
 	 */
 	public static function isHttpRequest()
 	{
-		if (isset($_SERVER) && isset($_SERVER['HTTP_HOST'])) { return true; }
-		else { return false; }
+		if (isset($_SERVER) && isset($_SERVER['HTTP_HOST'])) { return true; } else { return false; }
 	}
 
 	/**
@@ -97,8 +95,7 @@ 
 	 */
 	public static function isHttpsRequest()
 	{
-		if (isset($_SERVER['HTTPS']) && strtolower($_SERVER['HTTPS']) == 'on') { return true; }
-		else { return false; }
+		if (isset($_SERVER['HTTPS']) && strtolower($_SERVER['HTTPS']) == 'on') { return true; } else { return false; }
 	}
 
 	/**
@@ -111,8 +108,7 @@ 
 	{
 		$sSapiType = php_sapi_name();
 
-		if (substr($sSapiType, 0, 3) == 'cgi' || defined('STDIN')) { return true; }
-		else { return false; }
+		if (substr($sSapiType, 0, 3) == 'cgi' || defined('STDIN')) { return true; } else { return false; }
 	}
 
 	/**
@@ -137,8 +133,7 @@ 
 	 */
 	public static function getParameters()
 	{
-		if (isset($_GET)) { return $_GET; }
-		else { return array(); }
+		if (isset($_GET)) { return $_GET; } else { return array(); }
 	}
 
 	/**
@@ -149,8 +144,7 @@ 
 	 */
 	public static function getPostParameters()
 	{
-		if (isset($_POST)) { return $_POST; }
-		else { return array(); }
+		if (isset($_POST)) { return $_POST; } else { return array(); }
 	}
 
 	/**
@@ -161,8 +155,7 @@ 
 	 */
 	public function isPost()
 	{
-		if (isset($_POST) && count($_POST) > 0) { return true; }
-		else { return false; }
+		if (isset($_POST) && count($_POST) > 0) { return true; } else { return false; }
 	}
 
 	/**
@@ -214,11 +207,7 @@ 
 	 */
 	public static function setStatus($iCode)
 	{
-		if ($iCode === 200) { header('HTTP/1.1 200 Ok');  }
-		else if ($iCode === 201) { header('HTTP/1.1 201 Created');  }
-		else if ($iCode === 204) { header("HTTP/1.0 204 No Content");  }
-		else if ($iCode === 403) { header('HTTP/1.1 403 Forbidden'); }
-		else if ($iCode === 404) { header('HTTP/1.1 404 Not Found'); }
+		if ($iCode === 200) { header('HTTP/1.1 200 Ok');  } else if ($iCode === 201) { header('HTTP/1.1 201 Created');  } else if ($iCode === 204) { header("HTTP/1.0 204 No Content");  } else if ($iCode === 403) { header('HTTP/1.1 403 Forbidden'); } else if ($iCode === 404) { header('HTTP/1.1 404 Not Found'); }
 	}
 
     /**
@@ -236,8 +225,7 @@ 
      */
     public function getLanguages() : array
     {
-        if (!self::isCliRequest()) { return explode(',', preg_replace('/^([^;]);?.*$/', '$1', $_SERVER['HTTP_ACCEPT_LANGUAGE'])); }
-        else { return array(); }
+        if (!self::isCliRequest()) { return explode(',', preg_replace('/^([^;]);?.*$/', '$1', $_SERVER['HTTP_ACCEPT_LANGUAGE'])); } else { return array(); }
     }
 
 	/**

bundles/lib/Upload.php

Doc Comments +1 added lines, -1 removed lines

@@ -81,7 +81,7 @@
 	 *
 	 * @access public
 	 * @param  string $sFile
-	 * @return bool|object
+	 * @return null|boolean
 	 */
 	public function upload(string $sFile)
 	{

Indentation +4 added lines, -4 removed lines

@@ -118,10 +118,10 @@
 			$aImageSizes = getimagesize($_FILES[$sFile]['tmp_name']);
 
 			$fRatio = min($aImageSizes[0] / $this->_iWidth, $aImageSizes[1] / $this->_iHeight);
-    		$iHeight =  $aImageSizes[1] / $fRatio;
-    		$iWidth =  $aImageSizes[0] / $fRatio;
-    		$fY = ($iHeight - $this->_iHeight) / 2 * $fRatio;
-    		$fX = ($iWidth - $this->_iWidth) / 2 * $fRatio;
+			$iHeight =  $aImageSizes[1] / $fRatio;
+			$iWidth =  $aImageSizes[0] / $fRatio;
+			$fY = ($iHeight - $this->_iHeight) / 2 * $fRatio;
+			$fX = ($iWidth - $this->_iWidth) / 2 * $fRatio;
 
 			$rNewImage = imagecreatefromjpeg($_FILES[$sFile]['tmp_name']);
 

Spacing +7 added lines, -7 removed lines

@@ -99,7 +99,7 @@ 
 
 		$sExtension = strtolower(substr(strrchr($_FILES[$sFile]['name'], '.'), 1));
 
-		if (count($this->_aAllowExtension) > 0 && !in_array($sExtension ,$this->_aAllowExtension)) {
+		if (count($this->_aAllowExtension) > 0 && !in_array($sExtension, $this->_aAllowExtension)) {
 
 			$this->_sError = "The extension is not good";
 			return false;
@@ -111,24 +111,24 @@ 
 		if ($this->_sExtension === null) { $this->setExtension($sExtension); }
 
 		if ($this->_sName) { $sName = $sPath.$this->_sName.'.'.$this->_sExtension; }
-		else { $sName = $sPath.md5(uniqid(rand(), true)).'.'.$this->_sExtension;}
+		else { $sName = $sPath.md5(uniqid(rand(), true)).'.'.$this->_sExtension; }
 
 		if ($this->_bProportion == true && ($this->_iWidth || $this->_iHeight)) {
 
 			$aImageSizes = getimagesize($_FILES[$sFile]['tmp_name']);
 
 			$fRatio = min($aImageSizes[0] / $this->_iWidth, $aImageSizes[1] / $this->_iHeight);
-    		$iHeight =  $aImageSizes[1] / $fRatio;
-    		$iWidth =  $aImageSizes[0] / $fRatio;
+    		$iHeight = $aImageSizes[1] / $fRatio;
+    		$iWidth = $aImageSizes[0] / $fRatio;
     		$fY = ($iHeight - $this->_iHeight) / 2 * $fRatio;
     		$fX = ($iWidth - $this->_iWidth) / 2 * $fRatio;
 
 			$rNewImage = imagecreatefromjpeg($_FILES[$sFile]['tmp_name']);
 
-			$rNewImgTrueColor = imagecreatetruecolor($this->_iWidth , $this->_iHeight);
-			imagecopyresampled($rNewImgTrueColor , $rNewImage, 0, 0, $fX, $fY, $this->_iWidth, $this->_iHeight, $iWidth * $fRatio - $fX * 2, $iHeight * $fRatio - $fY * 2);
+			$rNewImgTrueColor = imagecreatetruecolor($this->_iWidth, $this->_iHeight);
+			imagecopyresampled($rNewImgTrueColor, $rNewImage, 0, 0, $fX, $fY, $this->_iWidth, $this->_iHeight, $iWidth * $fRatio - $fX * 2, $iHeight * $fRatio - $fY * 2);
 
-			imagejpeg($rNewImgTrueColor , $sName, 100);
+			imagejpeg($rNewImgTrueColor, $sName, 100);
 		}
 		else {
 

Braces +2 added lines, -4 removed lines

@@ -110,8 +110,7 @@ 
 
 		if ($this->_sExtension === null) { $this->setExtension($sExtension); }
 
-		if ($this->_sName) { $sName = $sPath.$this->_sName.'.'.$this->_sExtension; }
-		else { $sName = $sPath.md5(uniqid(rand(), true)).'.'.$this->_sExtension;}
+		if ($this->_sName) { $sName = $sPath.$this->_sName.'.'.$this->_sExtension; } else { $sName = $sPath.md5(uniqid(rand(), true)).'.'.$this->_sExtension;}
 
 		if ($this->_bProportion == true && ($this->_iWidth || $this->_iHeight)) {
 
@@ -129,8 +128,7 @@ 
 			imagecopyresampled($rNewImgTrueColor , $rNewImage, 0, 0, $fX, $fY, $this->_iWidth, $this->_iHeight, $iWidth * $fRatio - $fX * 2, $iHeight * $fRatio - $fY * 2);
 
 			imagejpeg($rNewImgTrueColor , $sName, 100);
-		}
-		else {
+		} else {
 
 			$bResultat = move_uploaded_file($_FILES[$sFile]['tmp_name'], $sName);
 

bundles/src/Batch/app/Controller/Phpunit.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use \Venus\lib\Bash as Bash;
 use \Venus\src\Batch\common\Controller as Controller;
-use PHPUnit_Util_Fileloader;
 
 class Phpunit extends Controller
 {

Indentation +38 added lines, -38 removed lines

@@ -8,42 +8,42 @@
 
 class Phpunit extends Controller
 {
-    /**
-     * Constructs a test case with the given name.
-     *
-     */
-    public function __construct()
-    {
-        parent::__construct();
-    }
-
-    /**
-     * new method to launch a web server
-     * @param array $options
-     * @tutorial php bin/console server:run
-     *           php bin/console server:run -a 192.168.0.1:8000
-     */
-    public function phpunit(array $options = array())
-    {
-        ob_get_clean();
-
-        $files = scandir(__DIR__.'/../../../../tests');
-
-        foreach ($files as $one) {
-
-            if (is_dir(__DIR__.'/../../../../tests/'.$one) && $one != '..' && $one != '.') {
-
-                $controllerFiles = scandir(__DIR__.'/../../../../tests'.'/'.$one.'/app/Controller');
-
-                foreach ($controllerFiles as $oneController) {
-
-                    if (is_file(__DIR__.'/../../../../tests/'.$one.'/app/Controller/' . $oneController) && $oneController != '..' && $oneController != '.') {
-
-                        $unitTest = new \PHPUnit_TextUI_Command;
-                        $unitTest->run([__DIR__.'/../../../../tests/'.$one.'/app/Controller/' . $oneController]);
-                    }
-                }
-            }
-        }
-    }
+	/**
+	 * Constructs a test case with the given name.
+	 *
+	 */
+	public function __construct()
+	{
+		parent::__construct();
+	}
+
+	/**
+	 * new method to launch a web server
+	 * @param array $options
+	 * @tutorial php bin/console server:run
+	 *           php bin/console server:run -a 192.168.0.1:8000
+	 */
+	public function phpunit(array $options = array())
+	{
+		ob_get_clean();
+
+		$files = scandir(__DIR__.'/../../../../tests');
+
+		foreach ($files as $one) {
+
+			if (is_dir(__DIR__.'/../../../../tests/'.$one) && $one != '..' && $one != '.') {
+
+				$controllerFiles = scandir(__DIR__.'/../../../../tests'.'/'.$one.'/app/Controller');
+
+				foreach ($controllerFiles as $oneController) {
+
+					if (is_file(__DIR__.'/../../../../tests/'.$one.'/app/Controller/' . $oneController) && $oneController != '..' && $oneController != '.') {
+
+						$unitTest = new \PHPUnit_TextUI_Command;
+						$unitTest->run([__DIR__.'/../../../../tests/'.$one.'/app/Controller/' . $oneController]);
+					}
+				}
+			}
+		}
+	}
 }
\ No newline at end of file

Spacing +2 added lines, -2 removed lines

@@ -37,10 +37,10 @@
 
                 foreach ($controllerFiles as $oneController) {
 
-                    if (is_file(__DIR__.'/../../../../tests/'.$one.'/app/Controller/' . $oneController) && $oneController != '..' && $oneController != '.') {
+                    if (is_file(__DIR__.'/../../../../tests/'.$one.'/app/Controller/'.$oneController) && $oneController != '..' && $oneController != '.') {
 
                         $unitTest = new \PHPUnit_TextUI_Command;
-                        $unitTest->run([__DIR__.'/../../../../tests/'.$one.'/app/Controller/' . $oneController]);
+                        $unitTest->run([__DIR__.'/../../../../tests/'.$one.'/app/Controller/'.$oneController]);
                     }
                 }
             }