reginaldojunior / winners

app/Controller/ApiController.php

Braces +3 added lines, -2 removed lines

@@ -32,8 +32,9 @@
 				'id_usuario' => $this->getIdUser(),
 			);
 
-			if (isset($id_cliente))
-			 	$conditions['id'] = $id_cliente;
+			if (isset($id_cliente)) {
+						 	$conditions['id'] = $id_cliente;
+			}
 
 
 		    $cliente = $this->Cliente->find('all', 

app/Controller/AviseMeController.php

Braces +1 added lines, -2 removed lines

@@ -70,8 +70,7 @@
 			}
 
 			return true;
-		} 
-		catch (Exception $e) 
+		} catch (Exception $e) 
 		{
 			throw new Exception("Error Processing Request", 1);
 		}

app/Vendor/PHPExcel/PHPExcel/CachedObjectStorage/SQLite.php

Braces +18 added lines, -12 removed lines

@@ -60,8 +60,9 @@ 
 		if ($this->_currentCellIsDirty) {
 			$this->_currentObject->detach();
 
-			if (!$this->_DBHandle->queryExec("INSERT OR REPLACE INTO kvp_".$this->_TableName." VALUES('".$this->_currentObjectID."','".sqlite_escape_string(serialize($this->_currentObject))."')"))
-				throw new Exception(sqlite_error_string($this->_DBHandle->lastError()));
+			if (!$this->_DBHandle->queryExec("INSERT OR REPLACE INTO kvp_".$this->_TableName." VALUES('".$this->_currentObjectID."','".sqlite_escape_string(serialize($this->_currentObject))."')")) {
+							throw new Exception(sqlite_error_string($this->_DBHandle->lastError()));
+			}
 			$this->_currentCellIsDirty = false;
 		}
 		$this->_currentObjectID = $this->_currentObject = null;
@@ -162,8 +163,9 @@ 
 
 		//	Check if the requested entry exists in the cache
 		$query = "DELETE FROM kvp_".$this->_TableName." WHERE id='".$pCoord."'";
-		if (!$this->_DBHandle->queryExec($query))
-			throw new Exception(sqlite_error_string($this->_DBHandle->lastError()));
+		if (!$this->_DBHandle->queryExec($query)) {
+					throw new Exception(sqlite_error_string($this->_DBHandle->lastError()));
+		}
 
 		$this->_currentCellIsDirty = false;
 	}	//	function deleteCacheData()
@@ -177,8 +179,9 @@ 
 	public function getCellList() {
 		$query = "SELECT id FROM kvp_".$this->_TableName;
 		$cellIdsResult = $this->_DBHandle->unbufferedQuery($query,SQLITE_ASSOC);
-		if ($cellIdsResult === false)
-			throw new Exception(sqlite_error_string($this->_DBHandle->lastError()));
+		if ($cellIdsResult === false) {
+					throw new Exception(sqlite_error_string($this->_DBHandle->lastError()));
+		}
 
 		$cellKeys = array();
 		foreach($cellIdsResult as $row) {
@@ -199,8 +202,9 @@ 
 		//	Get a new id for the new table name
 		$tableName = str_replace('.','_',$this->_getUniqueID());
 		if (!$this->_DBHandle->queryExec('CREATE TABLE kvp_'.$tableName.' (id VARCHAR(12) PRIMARY KEY, value BLOB)
-													AS SELECT * FROM kvp_'.$this->_TableName))
-			throw new Exception(sqlite_error_string($this->_DBHandle->lastError()));
+													AS SELECT * FROM kvp_'.$this->_TableName)) {
+					throw new Exception(sqlite_error_string($this->_DBHandle->lastError()));
+		}
 
 		//	Copy the existing cell cache file
 		$this->_TableName = $tableName;
@@ -237,10 +241,12 @@ 
 			$_DBName = ':memory:';
 
 			$this->_DBHandle = new SQLiteDatabase($_DBName);
-			if ($this->_DBHandle === false)
-				throw new Exception(sqlite_error_string($this->_DBHandle->lastError()));
-			if (!$this->_DBHandle->queryExec('CREATE TABLE kvp_'.$this->_TableName.' (id VARCHAR(12) PRIMARY KEY, value BLOB)'))
-				throw new Exception(sqlite_error_string($this->_DBHandle->lastError()));
+			if ($this->_DBHandle === false) {
+							throw new Exception(sqlite_error_string($this->_DBHandle->lastError()));
+			}
+			if (!$this->_DBHandle->queryExec('CREATE TABLE kvp_'.$this->_TableName.' (id VARCHAR(12) PRIMARY KEY, value BLOB)')) {
+							throw new Exception(sqlite_error_string($this->_DBHandle->lastError()));
+			}
 		}
 	}	//	function __construct()
 

app/Vendor/PHPExcel/PHPExcel/CachedObjectStorage/SQLite3.php

Braces +18 added lines, -12 removed lines

@@ -64,8 +64,9 @@ 
 			$query->bindValue('id',$this->_currentObjectID,SQLITE3_TEXT);
 			$query->bindValue('data',serialize($this->_currentObject),SQLITE3_BLOB);
 			$result = $query->execute();
-			if ($result === false)
-				throw new Exception($this->_DBHandle->lastErrorMsg());
+			if ($result === false) {
+							throw new Exception($this->_DBHandle->lastErrorMsg());
+			}
 			$this->_currentCellIsDirty = false;
 		}
 		$this->_currentObjectID = $this->_currentObject = null;
@@ -166,8 +167,9 @@ 
 		//	Check if the requested entry exists in the cache
 		$query = "DELETE FROM kvp_".$this->_TableName." WHERE id='".$pCoord."'";
 		$result = $this->_DBHandle->exec($query);
-		if ($result === false)
-			throw new Exception($this->_DBHandle->lastErrorMsg());
+		if ($result === false) {
+					throw new Exception($this->_DBHandle->lastErrorMsg());
+		}
 
 		$this->_currentCellIsDirty = false;
 	}	//	function deleteCacheData()
@@ -181,8 +183,9 @@ 
 	public function getCellList() {
 		$query = "SELECT id FROM kvp_".$this->_TableName;
 		$cellIdsResult = $this->_DBHandle->query($query);
-		if ($cellIdsResult === false)
-			throw new Exception($this->_DBHandle->lastErrorMsg());
+		if ($cellIdsResult === false) {
+					throw new Exception($this->_DBHandle->lastErrorMsg());
+		}
 
 		$cellKeys = array();
 		while ($row = $cellIdsResult->fetchArray(SQLITE3_ASSOC)) {
@@ -203,8 +206,9 @@ 
 		//	Get a new id for the new table name
 		$tableName = str_replace('.','_',$this->_getUniqueID());
 		if (!$this->_DBHandle->exec('CREATE TABLE kvp_'.$tableName.' (id VARCHAR(12) PRIMARY KEY, value BLOB)
-		                                       AS SELECT * FROM kvp_'.$this->_TableName))
-			throw new Exception($this->_DBHandle->lastErrorMsg());
+		                                       AS SELECT * FROM kvp_'.$this->_TableName)) {
+					throw new Exception($this->_DBHandle->lastErrorMsg());
+		}
 
 		//	Copy the existing cell cache file
 		$this->_TableName = $tableName;
@@ -241,10 +245,12 @@ 
 			$_DBName = ':memory:';
 
 			$this->_DBHandle = new SQLite3($_DBName);
-			if ($this->_DBHandle === false)
-				throw new Exception($this->_DBHandle->lastErrorMsg());
-			if (!$this->_DBHandle->exec('CREATE TABLE kvp_'.$this->_TableName.' (id VARCHAR(12) PRIMARY KEY, value BLOB)'))
-				throw new Exception($this->_DBHandle->lastErrorMsg());
+			if ($this->_DBHandle === false) {
+							throw new Exception($this->_DBHandle->lastErrorMsg());
+			}
+			if (!$this->_DBHandle->exec('CREATE TABLE kvp_'.$this->_TableName.' (id VARCHAR(12) PRIMARY KEY, value BLOB)')) {
+							throw new Exception($this->_DBHandle->lastErrorMsg());
+			}
 		}
 	}	//	function __construct()
 

app/Vendor/PHPExcel/PHPExcel/Calculation.php

Braces +60 added lines, -22 removed lines

@@ -2208,9 +2208,13 @@ 
 		//	Basic validation that this is indeed a formula
 		//	We return an empty array if not
 		$formula = trim($formula);
-		if ((!isset($formula{0})) || ($formula{0} != '=')) return array();
+		if ((!isset($formula{0})) || ($formula{0} != '=')) {
+			return array();
+		}
 		$formula = ltrim(substr($formula,1));
-		if (!isset($formula{0})) return array();
+		if (!isset($formula{0})) {
+			return array();
+		}
 
 		//	Parse the formula and return the token stack
 		return $this->_parseFormula($formula);
@@ -2263,9 +2267,13 @@ 
 		//	Basic validation that this is indeed a formula
 		//	We simply return the "cell value" (formula) if not
 		$formula = trim($formula);
-		if ($formula{0} != '=') return self::_wrapResult($formula);
+		if ($formula{0} != '=') {
+			return self::_wrapResult($formula);
+		}
 		$formula = ltrim(substr($formula,1));
-		if (!isset($formula{0})) return self::_wrapResult($formula);
+		if (!isset($formula{0})) {
+			return self::_wrapResult($formula);
+		}
 
 		$wsTitle = "\x00Wrk";
 		if ($pCell !== NULL) {
@@ -2715,8 +2723,11 @@ 
 //				echo 'Element is a Closing bracket<br />';
 				$expectingOperand = false;
 				while (($o2 = $stack->pop()) && $o2['value'] != '(') {		//	Pop off the stack back to the last (
-					if ($o2 === NULL) return $this->_raiseFormulaError('Formula Error: Unexpected closing brace ")"');
-					else $output[] = $o2;
+					if ($o2 === NULL) {
+						return $this->_raiseFormulaError('Formula Error: Unexpected closing brace ")"');
+					} else {
+						$output[] = $o2;
+					}
 				}
 				$d = $stack->last(2);
 				if (preg_match('/^'.self::CALCULATION_REGEXP_FUNCTION.'$/i', $d['value'], $matches)) {	//	Did this parenthesis just close a function?
@@ -2794,8 +2805,12 @@ 
 			} elseif ($opCharacter == ',') {			//	Is this the separator for function arguments?
 //				echo 'Element is a Function argument separator<br />';
 				while (($o2 = $stack->pop()) && $o2['value'] != '(') {		//	Pop off the stack back to the last (
-					if ($o2 === NULL) return $this->_raiseFormulaError("Formula Error: Unexpected ,");
-					else $output[] = $o2;	// pop the argument expression stuff and push onto the output
+					if ($o2 === NULL) {
+						return $this->_raiseFormulaError("Formula Error: Unexpected ,");
+					} else {
+						$output[] = $o2;
+					}
+					// pop the argument expression stuff and push onto the output
 				}
 				//	If we've a comma when we're expecting an operand, then what we actually have is a null operand;
 				//		so push a null onto the stack
@@ -2804,8 +2819,9 @@ 
 				}
 				// make sure there was a function
 				$d = $stack->last(2);
-				if (!preg_match('/^'.self::CALCULATION_REGEXP_FUNCTION.'$/i', $d['value'], $matches))
-					return $this->_raiseFormulaError("Formula Error: Unexpected ,");
+				if (!preg_match('/^'.self::CALCULATION_REGEXP_FUNCTION.'$/i', $d['value'], $matches)) {
+									return $this->_raiseFormulaError("Formula Error: Unexpected ,");
+				}
 				$d = $stack->pop();
 				$stack->push($d['type'],++$d['value'],$d['reference']);	// increment the argument count
 				$stack->push('Brace', '(');	// put the ( back on, we'll need to pop back to it again
@@ -2876,12 +2892,16 @@ 
 						if (strpos('!',$startRowColRef) !== false) {
 							list($rangeWS1,$startRowColRef) = explode('!',$startRowColRef);
 						}
-						if ($rangeWS1 != '') $rangeWS1 .= '!';
+						if ($rangeWS1 != '') {
+							$rangeWS1 .= '!';
+						}
 						$rangeWS2 = $rangeWS1;
 						if (strpos('!',$val) !== false) {
 							list($rangeWS2,$val) = explode('!',$val);
 						}
-						if ($rangeWS2 != '') $rangeWS2 .= '!';
+						if ($rangeWS2 != '') {
+							$rangeWS2 .= '!';
+						}
 						if ((is_integer($startRowColRef)) && (ctype_digit($val)) &&
 							($startRowColRef <= 1048576) && ($val <= 1048576)) {
 							//	Row range
@@ -2977,8 +2997,10 @@ 
 		}
 
 		while (($op = $stack->pop()) !== NULL) {	// pop everything off the stack and push onto output
-			if ((is_array($opCharacter) && $opCharacter['value'] == '(') || ($opCharacter === '('))
-				return $this->_raiseFormulaError("Formula Error: Expecting ')'");	// if there are any opening braces on the stack, then braces were unbalanced
+			if ((is_array($opCharacter) && $opCharacter['value'] == '(') || ($opCharacter === '(')) {
+							return $this->_raiseFormulaError("Formula Error: Expecting ')'");
+			}
+			// if there are any opening braces on the stack, then braces were unbalanced
 			$output[] = $op;
 		}
 		return $output;
@@ -3002,7 +3024,9 @@ 
 
 	// evaluate postfix notation
 	private function _processTokenStack($tokens, $cellID = null, PHPExcel_Cell $pCell = null) {
-		if ($tokens == false) return false;
+		if ($tokens == false) {
+			return false;
+		}
 
 		//	If we're using cell caching, then $pCell may well be flushed back to the cache (which detaches the parent worksheet),
 		//		so we store the parent worksheet so that we can re-attach it when necessary
@@ -3019,8 +3043,12 @@ 
 			if (isset(self::$_binaryOperators[$token])) {
 //				echo 'Token is a binary operator<br />';
 				//	We must have two operands, error if we don't
-				if (($operand2Data = $stack->pop()) === NULL) return $this->_raiseFormulaError('Internal error - Operand value missing from stack');
-				if (($operand1Data = $stack->pop()) === NULL) return $this->_raiseFormulaError('Internal error - Operand value missing from stack');
+				if (($operand2Data = $stack->pop()) === NULL) {
+					return $this->_raiseFormulaError('Internal error - Operand value missing from stack');
+				}
+				if (($operand1Data = $stack->pop()) === NULL) {
+					return $this->_raiseFormulaError('Internal error - Operand value missing from stack');
+				}
 
 				$operand1 = self::_dataTestReference($operand1Data);
 				$operand2 = self::_dataTestReference($operand2Data);
@@ -3158,7 +3186,9 @@ 
 			// if the token is a unary operator, pop one value off the stack, do the operation, and push it back on
 			} elseif (($token === '~') || ($token === '%')) {
 //				echo 'Token is a unary operator<br />';
-				if (($arg = $stack->pop()) === NULL) return $this->_raiseFormulaError('Internal error - Operand value missing from stack');
+				if (($arg = $stack->pop()) === NULL) {
+					return $this->_raiseFormulaError('Internal error - Operand value missing from stack');
+				}
 				$arg = $arg['value'];
 				if ($token === '~') {
 //					echo 'Token is a negation operator<br />';
@@ -3388,7 +3418,9 @@ 
 			}
 		}
 		// when we're out of tokens, the stack should have a single element, the final result
-		if ($stack->count() != 1) return $this->_raiseFormulaError("internal error");
+		if ($stack->count() != 1) {
+			return $this->_raiseFormulaError("internal error");
+		}
 		$output = $stack->pop();
 		$output = $output['value'];
 
@@ -3502,8 +3534,12 @@ 
 
 	private function _executeNumericBinaryOperation($cellID,$operand1,$operand2,$operation,$matrixFunction,&$stack) {
 		//	Validate the two operands
-		if (!$this->_validateBinaryOperand($cellID,$operand1,$stack)) return false;
-		if (!$this->_validateBinaryOperand($cellID,$operand2,$stack)) return false;
+		if (!$this->_validateBinaryOperand($cellID,$operand1,$stack)) {
+			return false;
+		}
+		if (!$this->_validateBinaryOperand($cellID,$operand2,$stack)) {
+			return false;
+		}
 
 		$executeMatrixOperation = false;
 		//	If either of the operands is a matrix, we need to treat them both as matrices
@@ -3595,7 +3631,9 @@ 
 	protected function _raiseFormulaError($errorMessage) {
 		$this->formulaError = $errorMessage;
 		$this->debugLogStack = array();
-		if (!$this->suppressFormulaErrors) throw new Exception($errorMessage);
+		if (!$this->suppressFormulaErrors) {
+			throw new Exception($errorMessage);
+		}
 		trigger_error($errorMessage, E_USER_ERROR);
 	}	//	function _raiseFormulaError()
 

app/Vendor/PHPExcel/PHPExcel/Calculation/DateTime.php

Braces +6 added lines, -3 removed lines

@@ -515,8 +515,9 @@ 
 		if (($PHPDateArray !== False) && ($PHPDateArray['error_count'] == 0)) {
 			// Execute function
 			if ($PHPDateArray['year'] == '')	{ $PHPDateArray['year'] = strftime('%Y'); }
-			if ($PHPDateArray['year'] < 1900)
-				return PHPExcel_Calculation_Functions::VALUE();
+			if ($PHPDateArray['year'] < 1900) {
+							return PHPExcel_Calculation_Functions::VALUE();
+			}
 			if ($PHPDateArray['month'] == '')	{ $PHPDateArray['month'] = strftime('%m'); }
 			if ($PHPDateArray['day'] == '')		{ $PHPDateArray['day'] = strftime('%d'); }
 			$excelDateValue = floor(PHPExcel_Shared_Date::FormattedPHPToExcel($PHPDateArray['year'],$PHPDateArray['month'],$PHPDateArray['day'],$PHPDateArray['hour'],$PHPDateArray['minute'],$PHPDateArray['second']));
@@ -656,7 +657,9 @@ 
 				break;
 			case 'YM':
 				$retVal = intval($endMonths - $startMonths);
-				if ($retVal < 0) $retVal = 12 + $retVal;
+				if ($retVal < 0) {
+					$retVal = 12 + $retVal;
+				}
 				//	We're only interested in full months
 				if ($endDays < $startDays) {
 					--$retVal;

app/Vendor/PHPExcel/PHPExcel/Calculation/Engineering.php

Braces +15 added lines, -5 removed lines

@@ -749,10 +749,18 @@ 
 	 * @return	string		The "cleaned" complex number
 	 */
 	private static function _cleanComplex($complexNumber) {
-		if ($complexNumber{0} == '+') $complexNumber = substr($complexNumber,1);
-		if ($complexNumber{0} == '0') $complexNumber = substr($complexNumber,1);
-		if ($complexNumber{0} == '.') $complexNumber = '0'.$complexNumber;
-		if ($complexNumber{0} == '+') $complexNumber = substr($complexNumber,1);
+		if ($complexNumber{0} == '+') {
+			$complexNumber = substr($complexNumber,1);
+		}
+		if ($complexNumber{0} == '0') {
+			$complexNumber = substr($complexNumber,1);
+		}
+		if ($complexNumber{0} == '.') {
+			$complexNumber = '0'.$complexNumber;
+		}
+		if ($complexNumber{0} == '+') {
+			$complexNumber = substr($complexNumber,1);
+		}
 		return $complexNumber;
 	}
 
@@ -1658,7 +1666,9 @@ 
 			$realNumber	= (float) $realNumber;
 			$imaginary	= (float) $imaginary;
 
-			if ($suffix == '') $suffix = 'i';
+			if ($suffix == '') {
+				$suffix = 'i';
+			}
 			if ($realNumber == 0.0) {
 				if ($imaginary == 0.0) {
 					return (string) '0';

app/Vendor/PHPExcel/PHPExcel/Calculation/Financial.php

Braces +54 added lines, -19 removed lines

@@ -1304,7 +1304,9 @@ 
 
 
 	public static function IRR($values, $guess = 0.1) {
-		if (!is_array($values)) return PHPExcel_Calculation_Functions::VALUE();
+		if (!is_array($values)) {
+			return PHPExcel_Calculation_Functions::VALUE();
+		}
 		$values = PHPExcel_Calculation_Functions::flattenArray($values);
 		$guess = PHPExcel_Calculation_Functions::flattenSingleValue($guess);
 
@@ -1314,14 +1316,18 @@ 
 		$f1 = self::NPV($x1, $values);
 		$f2 = self::NPV($x2, $values);
 		for ($i = 0; $i < FINANCIAL_MAX_ITERATIONS; ++$i) {
-			if (($f1 * $f2) < 0.0) break;
+			if (($f1 * $f2) < 0.0) {
+				break;
+			}
 			if (abs($f1) < abs($f2)) {
 				$f1 = self::NPV($x1 += 1.6 * ($x1 - $x2), $values);
 			} else {
 				$f2 = self::NPV($x2 += 1.6 * ($x2 - $x1), $values);
 			}
 		}
-		if (($f1 * $f2) > 0.0) return PHPExcel_Calculation_Functions::VALUE();
+		if (($f1 * $f2) > 0.0) {
+			return PHPExcel_Calculation_Functions::VALUE();
+		}
 
 		$f = self::NPV($x1, $values);
 		if ($f < 0.0) {
@@ -1336,8 +1342,12 @@ 
 			$dx *= 0.5;
 			$x_mid = $rtb + $dx;
 			$f_mid = self::NPV($x_mid, $values);
-			if ($f_mid <= 0.0) $rtb = $x_mid;
-			if ((abs($f_mid) < FINANCIAL_PRECISION) || (abs($dx) < FINANCIAL_PRECISION)) return $x_mid;
+			if ($f_mid <= 0.0) {
+				$rtb = $x_mid;
+			}
+			if ((abs($f_mid) < FINANCIAL_PRECISION) || (abs($dx) < FINANCIAL_PRECISION)) {
+				return $x_mid;
+			}
 		}
 		return PHPExcel_Calculation_Functions::VALUE();
 	}	//	function IRR()
@@ -1385,7 +1395,9 @@ 
 
 
 	public static function MIRR($values, $finance_rate, $reinvestment_rate) {
-		if (!is_array($values)) return PHPExcel_Calculation_Functions::VALUE();
+		if (!is_array($values)) {
+			return PHPExcel_Calculation_Functions::VALUE();
+		}
 		$values				= PHPExcel_Calculation_Functions::flattenArray($values);
 		$finance_rate		= PHPExcel_Calculation_Functions::flattenSingleValue($finance_rate);
 		$reinvestment_rate	= PHPExcel_Calculation_Functions::flattenSingleValue($reinvestment_rate);
@@ -1810,8 +1822,9 @@ 
 			$rate = ($y1 * $x0 - $y0 * $x1) / ($y1 - $y0);
 			$x0 = $x1;
 			$x1 = $rate;
-			if (($nper * abs($pmt)) > ($pv - $fv))
-				$x1 = abs($x1);
+			if (($nper * abs($pmt)) > ($pv - $fv)) {
+							$x1 = abs($x1);
+			}
 
 			if (abs($rate) < FINANCIAL_PRECISION) {
 				$y = $pv * (1 + $nper * $rate) + $pmt * (1 + $rate * $type) * $nper + $fv;
@@ -2060,11 +2073,15 @@ 
 
 
 	public static function XIRR($values, $dates, $guess = 0.1) {
-		if ((!is_array($values)) && (!is_array($dates))) return PHPExcel_Calculation_Functions::VALUE();
+		if ((!is_array($values)) && (!is_array($dates))) {
+			return PHPExcel_Calculation_Functions::VALUE();
+		}
 		$values	= PHPExcel_Calculation_Functions::flattenArray($values);
 		$dates	= PHPExcel_Calculation_Functions::flattenArray($dates);
 		$guess = PHPExcel_Calculation_Functions::flattenSingleValue($guess);
-		if (count($values) != count($dates)) return PHPExcel_Calculation_Functions::NaN();
+		if (count($values) != count($dates)) {
+			return PHPExcel_Calculation_Functions::NaN();
+		}
 
 		// create an initial range, with a root somewhere between 0 and guess
 		$x1 = 0.0;
@@ -2072,14 +2089,18 @@ 
 		$f1 = self::XNPV($x1, $values, $dates);
 		$f2 = self::XNPV($x2, $values, $dates);
 		for ($i = 0; $i < FINANCIAL_MAX_ITERATIONS; ++$i) {
-			if (($f1 * $f2) < 0.0) break;
+			if (($f1 * $f2) < 0.0) {
+				break;
+			}
 			if (abs($f1) < abs($f2)) {
 				$f1 = self::XNPV($x1 += 1.6 * ($x1 - $x2), $values, $dates);
 			} else {
 				$f2 = self::XNPV($x2 += 1.6 * ($x2 - $x1), $values, $dates);
 			}
 		}
-		if (($f1 * $f2) > 0.0) return PHPExcel_Calculation_Functions::VALUE();
+		if (($f1 * $f2) > 0.0) {
+			return PHPExcel_Calculation_Functions::VALUE();
+		}
 
 		$f = self::XNPV($x1, $values, $dates);
 		if ($f < 0.0) {
@@ -2094,8 +2115,12 @@ 
 			$dx *= 0.5;
 			$x_mid = $rtb + $dx;
 			$f_mid = self::XNPV($x_mid, $values, $dates);
-			if ($f_mid <= 0.0) $rtb = $x_mid;
-			if ((abs($f_mid) < FINANCIAL_PRECISION) || (abs($dx) < FINANCIAL_PRECISION)) return $x_mid;
+			if ($f_mid <= 0.0) {
+				$rtb = $x_mid;
+			}
+			if ((abs($f_mid) < FINANCIAL_PRECISION) || (abs($dx) < FINANCIAL_PRECISION)) {
+				return $x_mid;
+			}
 		}
 		return PHPExcel_Calculation_Functions::VALUE();
 	}
@@ -2117,17 +2142,27 @@ 
 	 */
 	public static function XNPV($rate, $values, $dates) {
 		$rate = PHPExcel_Calculation_Functions::flattenSingleValue($rate);
-		if (!is_numeric($rate)) return PHPExcel_Calculation_Functions::VALUE();
-		if ((!is_array($values)) || (!is_array($dates))) return PHPExcel_Calculation_Functions::VALUE();
+		if (!is_numeric($rate)) {
+			return PHPExcel_Calculation_Functions::VALUE();
+		}
+		if ((!is_array($values)) || (!is_array($dates))) {
+			return PHPExcel_Calculation_Functions::VALUE();
+		}
 		$values	= PHPExcel_Calculation_Functions::flattenArray($values);
 		$dates	= PHPExcel_Calculation_Functions::flattenArray($dates);
 		$valCount = count($values);
-		if ($valCount != count($dates)) return PHPExcel_Calculation_Functions::NaN();
-		if ((min($values) > 0) || (max($values) < 0)) return PHPExcel_Calculation_Functions::VALUE();
+		if ($valCount != count($dates)) {
+			return PHPExcel_Calculation_Functions::NaN();
+		}
+		if ((min($values) > 0) || (max($values) < 0)) {
+			return PHPExcel_Calculation_Functions::VALUE();
+		}
 
 		$xnpv = 0.0;
 		for ($i = 0; $i < $valCount; ++$i) {
-			if (!is_numeric($values[$i])) return PHPExcel_Calculation_Functions::VALUE();
+			if (!is_numeric($values[$i])) {
+				return PHPExcel_Calculation_Functions::VALUE();
+			}
 			$xnpv += $values[$i] / pow(1 + $rate, PHPExcel_Calculation_DateTime::DATEDIF($dates[0],$dates[$i],'d') / 365);
 		}
 		return (is_finite($xnpv)) ? $xnpv : PHPExcel_Calculation_Functions::VALUE();

app/Vendor/PHPExcel/PHPExcel/Calculation/FormulaParser.php

Braces +3 added lines, -1 removed lines

@@ -159,7 +159,9 @@
 
 		// Check if the formula has a valid starting =
 		$formulaLength = strlen($this->_formula);
-		if ($formulaLength < 2 || $this->_formula{0} != '=') return;
+		if ($formulaLength < 2 || $this->_formula{0} != '=') {
+			return;
+		}
 
 		// Helper variables
 		$tokens1	= $tokens2 	= $stack = array();