Merge pull request #150 from eperezf/release/v0.6.0

#143 Fix division por 0
eperezf · Jan 18, 2020 · 4c47999 · 4c47999
2 parents 5949049 + 44a3090
commit 4c47999
Showing 1 changed file with 12 additions and 3 deletions.
diff --git a/web/app/Http/Controllers/GraficasController.php b/web/app/Http/Controllers/GraficasController.php
@@ -222,8 +222,13 @@ public function getEstadisticasEmpresas()
 		$empresasValidadasCount = $empresasValidadas->count();
 		$empresasNoValidadasCount = $empresasNoValidadas->count();
 		$empresasTotal = $empresasValidadasCount + $empresasNoValidadasCount;
-		$empresasPorcentajeValidadas = round($empresasValidadasCount / $empresasTotal * 100, 2);
-		$empresasPorcentajeNoValidadas = round($empresasNoValidadasCount / $empresasTotal * 100, 2);
+		if ($empresasTotal == 0) {
+			$empresasPorcentajeValidadas = 0;
+			$empresasPorcentajeNoValidadas = 0;
+		} else {
+			$empresasPorcentajeValidadas = round($empresasValidadasCount / $empresasTotal * 100, 2);
+			$empresasPorcentajeNoValidadas = round($empresasNoValidadasCount / $empresasTotal * 100, 2);
+		}
 
 
 		$estadisticasEmpresas = array(
@@ -290,7 +295,11 @@ public function isPasantiaTerminada($pasantia)
 	{
 		$horasSemanales = $pasantia->horasSemanales;
 		$totalHoras = 810;
-		$totalSemanas = round($totalHoras / $horasSemanales);
+		if ($horasSemanales == 0) {
+			return false;
+		} else {
+			$totalSemanas = round($totalHoras / $horasSemanales);
+		}
 		$fechaTermino = Carbon::parse($pasantia->fechaInicio);
 		$fechaTermino->addWeeks($totalSemanas);
 		if (Carbon::now() > $fechaTermino) {