Mejora cierre conexión DB desde Jasper (#159)

* Se cierra conexion JDBC de manera explicita para que no quede dangling ante una excepcion.
SIU-Toba · Jun 29, 2023 · c5ef351 · c5ef351
1 parent fbc2e02
commit c5ef351
Showing 1 changed file with 20 additions and 13 deletions.
diff --git a/php/nucleo/lib/salidas/toba_vista_jasperreports.php b/php/nucleo/lib/salidas/toba_vista_jasperreports.php
@@ -323,17 +323,25 @@ function completar_con_datos()
 			$this->parametros->put($jrxpath->PARAMETER_XML_DATA_DOCUMENT, $document);		
 			$print = $this->jasper->fillReport($this->path_reporte, $this->parametros);			
 		}  else {													//El conjunto de datos viene de una db o datasource
-			if (! isset($this->conexion)) {
-				$this->conexion = $this->instanciar_conexion_default();
-			}			 
-			if ($this->conexion instanceof toba_db) {						//Si es una base toba, le configuro el schema
-				$con1 = $this->configurar_bd($this->conexion);
-			} else {
-				$con1 = $this->conexion;
-			}
-			//Creo el reporte finalmente con la conexion JDBC
-			$print = $this->jasper->fillReport($this->path_reporte, $this->parametros, $con1);
-			$con1->close();
+            try {
+                if (! isset($this->conexion)) {
+                    $this->conexion = $this->instanciar_conexion_default();
+                }			 
+                if ($this->conexion instanceof toba_db) {						//Si es una base toba, le configuro el schema
+                    $con1 = $this->configurar_bd($this->conexion);
+                } else {
+                    $con1 = $this->conexion;
+                }
+                //Creo el reporte finalmente con la conexion JDBC
+                $print = $this->jasper->fillReport($this->path_reporte, $this->parametros, $con1);
+            } catch (\Exception $e) {
+                toba::logger()->error('Error en la ejecución del reporte');
+                toba::logger()->error($e->getMessage());
+                throw $e;
+            } finally {
+                toba::logger()->debug("Jasper JDBC close Connection");
+                $con1->close();
+            }
 		}		
 		$this->lista_jrprint[] = $print;
 	}
@@ -423,5 +431,4 @@ protected function cabecera_http( $longitud )
 	}
 
 
-}
-?>
+}