BugFix.

[mecon/yatta.git] / lib / YATTA / Controlador.php

diff --git a/lib/YATTA/Controlador.php b/lib/YATTA/Controlador.php
index 4d9a8cdab24d0809a1f6656761197842c1a215b9..610ba02d34e9e38f4649968958167f2e46f9b1d2 100644 (file)
--- a/lib/YATTA/Controlador.php
+++ b/lib/YATTA/Controlador.php
@@ -159,35 +159,37 @@ class YATTA_Controlador {
     function terminarProceso($db, $id_proceso, $id_server, $status, $nota =
             null, $archivo = null, $notificar = 0) {
 
     function terminarProceso($db, $id_proceso, $id_server, $status, $nota =
             null, $archivo = null, $notificar = 0) {
 

-        //Actualizo la base procesos
-        $sql = 'UPDATE yatta.procesos SET status = '. $status;
+        //NOTIFICO
+        if (@$notificar) {
+            $this->_emailUsuario ($db, $id_proceso, $status, $nota, $archivo);
+        }
+
+        //COPIO EL ARCHIVO
+        if (@$archivo) {
+            $arch = basename ($archivo);
+            rename ($archivo, PATH_TACHO.$arch);
+        }
+
+        //ACTUALIZO LA BASE PROCESOS
+        $sql = 'UPDATE yatta.procesos SET status = '. 
+            $db->quoteSmart($status);

         if (@$nota) {
             $sql.= ', nota = '. $db->quoteSmart($nota);
         }
         if (@$archivo) {
         if (@$nota) {
             $sql.= ', nota = '. $db->quoteSmart($nota);
         }
         if (@$archivo) {

-            $sql.= ', archivo = '. $db->quoteSmart($archivo);
+            $sql.= ', archivo = '. $db->quoteSmart(PATH_TACHO.$arch);

         }
         if (@$notificar) {
             $sql.= ', notificar = '. $db->quoteSmart($notificar);
         }
         }
         if (@$notificar) {
             $sql.= ', notificar = '. $db->quoteSmart($notificar);
         }

-        $sql.= ' WHERE id = '. $id_proceso;
+        $sql.= ' WHERE id = '. $db->quoteSmart($id_proceso);

         
         $res = $db->query($sql);
         if (DB::isError($res)) {
             return $res;
         }
 
         
         $res = $db->query($sql);
         if (DB::isError($res)) {
             return $res;
         }
 

-        //NOTIFICO
-        if (@$notificar) {
-            $this->_emailUsuario ($db, $id_proceso, $status, $nota, $archivo);
-        }

 
 

-        //COPIO EL ARCHIVO
-        if (@$archivo) {
-            $arch = basename ($archivo);
-            rename ($archivo, PATH_TACHO.$arch);
-        }
-        

         //Actualizo los servidores
         $SERVER =& new YATTA_Servidor;
         return $SERVER->quitarProceso($db, $id_server);
         //Actualizo los servidores
         $SERVER =& new YATTA_Servidor;
         return $SERVER->quitarProceso($db, $id_server);


@@ -253,18 +255,20 @@ class YATTA_Controlador {
         $res = $db->query(
                 "SELECT p.descripcion AS descripcion, p.owner AS owner, ".
                 "p.destinos AS destinos, p.resultado AS resultado, ".
         $res = $db->query(
                 "SELECT p.descripcion AS descripcion, p.owner AS owner, ".
                 "p.destinos AS destinos, p.resultado AS resultado, ".

-                "p.estado AS estado, p.nota AS nota ".
-                "FROM yatta.procesos AS p".
+                "p.status AS estado, p.nota AS nota ".
+                "FROM yatta.procesos AS p ".

                 "WHERE p.id = ". $id_proceso);
                 "WHERE p.id = ". $id_proceso);

+

         if (DB::isError($res)) {
             return $res;
         }
         $res = $res->fetchRow(DB_FETCHMODE_ASSOC);
 
         if (DB::isError($res)) {
             return $res;
         }
         $res = $res->fetchRow(DB_FETCHMODE_ASSOC);
 

+        

 //@FIXME El email se lo envio al owner y a los destinos, o al owner solamente?
         
         $dest = split (',', @$res['destinos']);
 //@FIXME El email se lo envio al owner y a los destinos, o al owner solamente?
         
         $dest = split (',', @$res['destinos']);

-        $to = $owner.'.gov.ar';
+        $to = $res['owner'].'.gov.ar';

         foreach ($dest as $d) {
             $to = $to . ", $d.gov.ar";
         }
         foreach ($dest as $d) {
             $to = $to . ", $d.gov.ar";
         }


@@ -296,10 +300,10 @@ class YATTA_Controlador {
                 "en la seccion Archivos del sistema YATTA.\n".
                 "Recuerde que este archivo puede ser borrado sin previo aviso\n".
                 "por tres motivos:\n".
                 "en la seccion Archivos del sistema YATTA.\n".
                 "Recuerde que este archivo puede ser borrado sin previo aviso\n".
                 "por tres motivos:\n".

-                "\t1- El archivo supera la antigüedad máxima establecida.\n".
-                "\t2- Se supero la capacidad máxima de información permitida\n".
-                "\t   por usuario.\n".
-                "\t3- Decisión administrativa por mantenimiento de los servidores.\n";
+                "1- El archivo supera la antigüedad máxima establecida.\n".
+                "2- Se supero la capacidad máxima de información permitida\n".
+                "   por usuario.\n".
+                "3- Decisión administrativa por mantenimiento de los servidores.\n";

         }
         
             
         }