Se arregla un bug que hacía que sea posible asignar de padre al mismo

[mecon/ai.git] / lib / AI / GrupoSecciones.php

diff --git a/lib/AI/GrupoSecciones.php b/lib/AI/GrupoSecciones.php
index c7d54d93d712844906c5b00e8356edce29fa2471..589b1270d54d9c1e1503f9a5c55fdb60323a751e 100644 (file)
--- a/lib/AI/GrupoSecciones.php
+++ b/lib/AI/GrupoSecciones.php
@@ -1,5 +1,5 @@
 <?php
 <?php

-// vim: set expandtab tabstop=4 shiftwidth=4 binary:
+// vim: set binary expandtab tabstop=4 shiftwidth=4:

 // +--------------------------------------------------------------------+
 // |                      Ministerio de Economía                        |
 // |                  AI (Administrador de Intranet)                    |
 // +--------------------------------------------------------------------+
 // |                      Ministerio de Economía                        |
 // |                  AI (Administrador de Intranet)                    |


@@ -27,8 +27,17 @@
 // $Id$
 //
 
 // $Id$
 //
 

+// +X2C includes
+require_once 'AI/DBTreeObject.php';
+// ~X2C
+

 require_once 'AI/Error.php';
 
 require_once 'AI/Error.php';
 

+/**
+ * Archivo de configuración.
+ */
+define('AI_GRUPOSECCIONES_CONFFILE', dirname(__FILE__).'/GrupoSecciones.ini');
+

 // +X2C Class 486 :AI_GrupoSecciones
 /**
  * Grupo de secciones.
 // +X2C Class 486 :AI_GrupoSecciones
 /**
  * Grupo de secciones.


@@ -36,7 +45,7 @@ require_once 'AI/Error.php';
  * @package AI
  * @access public
  */
  * @package AI
  * @access public
  */

-class AI_GrupoSecciones {
+class AI_GrupoSecciones extends AI_DBTreeObject {

     /**
      * ID del grupo de secciones.
      *
     /**
      * ID del grupo de secciones.
      *


@@ -59,14 +68,6 @@ class AI_GrupoSecciones {
      */
     var $grupo_padre = 0;
 
      */
     var $grupo_padre = 0;
 

-    /**
-     * Grupos de secciones que tiene este grupo.
-     *
-     * @var    array $hijos
-     * @access protected
-     */
-    var $_hijos = array();
-

     /**
      * Secciones que pertenecen a este grupo.
      *
     /**
      * Secciones que pertenecen a este grupo.
      *


@@ -99,17 +100,6 @@ class AI_GrupoSecciones {
      */
     var $habilitado = true;
 
      */
     var $habilitado = true;
 

-    /**
-     * Gets Hijos.
-     *
-     * @return array
-     * @access public
-     */
-    function getHijos()
-    {
-        return $this->_hijos;
-    }
-

     // ~X2C
 
     // +X2C Operation 495
     // ~X2C
 
     // +X2C Operation 495


@@ -121,7 +111,7 @@ class AI_GrupoSecciones {
      */
     function AI_GrupoSecciones($grupo = 0) // ~X2C
     {
      */
     function AI_GrupoSecciones($grupo = 0) // ~X2C
     {

-        $this->grupo = $grupo;
+        parent::AI_DBTreeObject($grupo, AI_GRUPOSECCIONES_CONFFILE);

     }
     // -X2C
 
     }
     // -X2C
 


@@ -134,40 +124,18 @@ class AI_GrupoSecciones {
      */
     function cargar($db) // ~X2C
     {
      */
     function cargar($db) // ~X2C
     {

-        $grupo = intval($this->grupo);
-        if (is_a($db, 'db_result')) {
-            $result = $db;
-            $db     = $result->dbh;
-        // Si no es un resultado, hago el query.
-        } else {
-            $result = $db->query(
-                "SELECT *
-                    FROM grupo_secciones
-                    WHERE grupo = $grupo"
-            );
-            if (DB::isError($result)) {
-                return $result;
-            }
-        }
-        // Obtengo la fila.
-        $row = $result->fetchRow(DB_FETCHMODE_ASSOC);
-        if (!$row) {
-            return new AI_Error(AI_ERROR_NO_RESULTADOS,
-                "No hay más resultados en la DB [grupo=$grupo]");
+        // Obtengo campos comunes.
+        $err = parent::cargar($db);
+        if (PEAR::isError($err)) {
+            return $err;

         }
         }

-        // Asigno valores al objeto.
-        extract($row);
-        $this->grupo        = $grupo;
-        $this->nombre       = $nombre;
-        $this->grupo_padre  = $grupo_padre;
-        $this->habilitado   = $habilitado;
-        $this->antiguedad   = $antiguedad;
-        $this->mostrar_hijos= $mostrar_hijos;

         // Obtengo secciones.
         // Obtengo secciones.

+        $id = intval($this->grupo);
+        extract($this->conf['secciones'], EXTR_PREFIX_ALL, 'cnf');

         $secciones = $db->getCol(
         $secciones = $db->getCol(

-            "SELECT seccion
-                FROM grupo_secciones_seccion
-                WHERE grupo = $grupo"
+            "SELECT $cnf_campo
+                FROM $cnf_base.$cnf_tabla
+                WHERE $cnf_id = $id"

         );
         if (DB::isError($secciones)) {
             return $secciones;
         );
         if (DB::isError($secciones)) {
             return $secciones;


@@ -191,8 +159,7 @@ class AI_GrupoSecciones {
      */
     function guardar($db, $nuevo = false) // ~X2C
     {
      */
     function guardar($db, $nuevo = false) // ~X2C
     {

-        $grupo = intval($this->grupo);
-        $where = '';
+        // Guardo datos generales.

         $datos = array(
             'grupo_padre'   => intval($this->grupo_padre),
             'nombre'        => $this->nombre,
         $datos = array(
             'grupo_padre'   => intval($this->grupo_padre),
             'nombre'        => $this->nombre,


@@ -200,31 +167,20 @@ class AI_GrupoSecciones {
             'antiguedad'    => intval($this->antiguedad),
             'mostrar_hijos' => $this->mostrar_hijos ? 1 : 0,
         );
             'antiguedad'    => intval($this->antiguedad),
             'mostrar_hijos' => $this->mostrar_hijos ? 1 : 0,
         );

-        if ($grupo and !$nuevo) {
-            $accion = DB_AUTOQUERY_UPDATE;
-            $where  = "grupo = $grupo";
-        } else {
-            $accion = DB_AUTOQUERY_INSERT;
-            // Si no tiene ID, le asigno uno nuevo.
-            if (!$grupo) {
-                $grupo = $db->nextId('grupo_secciones');
-                if (DB::isError($grupo)) {
-                    return $grupo;
-                }
-                $this->grupo = $grupo;
-            }
-            $datos['grupo'] = $grupo;
+        $err = parent::guardar($db, $datos, $nuevo);
+        if (PEAR::isError($err)) {
+            return $err;

         }
         }

-        $res = $db->autoExecute('grupo_secciones', $datos, $accion, $where);
+        // Guardo datos de secciones.
+               $id_field = $this->conf['id'];
+        $id = intval($this->$id_field);
+        extract($this->conf['secciones'], EXTR_PREFIX_ALL, 'cnf');
+        $res = $db->query("DELETE FROM $cnf_base.$cnf_tabla WHERE $cnf_id = $id");

         if (DB::isError($res)) {
             return $res;
         }
         if (DB::isError($res)) {
             return $res;
         }

-        $res = $db->query("DELETE FROM grupo_secciones_seccion WHERE grupo = $grupo");
-        if (DB::isError($res)) {
-            return $res;
-        }
-        $prepare = $db->prepare("INSERT INTO grupo_secciones_seccion
-            (grupo, seccion) VALUES ($grupo, ?)");
+        $prepare = $db->prepare("INSERT INTO $cnf_base.$cnf_tabla
+            ($cnf_id, $cnf_campo) VALUES ($id, ?)");

         if (DB::isError($prepare)) {
             return $prepare;
         }
         if (DB::isError($prepare)) {
             return $prepare;
         }


@@ -245,52 +201,20 @@ class AI_GrupoSecciones {
      */
     function borrar($db) // ~X2C
     {
      */
     function borrar($db) // ~X2C
     {

-        $grupo = intval($this->grupo);
-        if ($grupo) {
-            $res = $db->query(
-                "DELETE FROM grupo_secciones WHERE grupo = $grupo");
-            if (DB::isError($res)) {
-                return $res;
-            }
-            $res = $db->query(
-                "DELETE FROM grupo_secciones_seccion WHERE grupo = $grupo");
-            if (DB::isError($res)) {
-                return $res;
-            }
-            return true;
+        // Obtengo campos comunes.
+        $err = parent::borrar($db);
+        if (PEAR::isError($err)) {
+            return $err;

         }
         }

-        return PEAR::raiseError("No hay un grupo válido para borrar");
-    }
-    // -X2C
-
-    // +X2C Operation 499
-    /**
-     * @param  DB $db DB de donde cargar los hijos.
-     *
-     * @return PEAR_Error
-     * @access public
-     */
-    function cargarHijos($db) // ~X2C
-    {
-        $grupo  = intval($this->grupo);
-        $result = $db->query("SELECT * FROM grupo_secciones WHERE grupo_padre = $grupo");
-        if (DB::isError($result)) {
-            return $result;
-        }
-        $this->_hijos = array();
-        $hijo = new GrupoSecciones;
-        $err = $hijo->cargar($result);
-        while (!PEAR::isError($err)) {
-            $this->_hijos[] = $hijo->__clone();
-            $err = $hijo->cargar($result);
-        }
-        // Si no hay mas resultados, entonces terminó bien.
-        if (AI_Error::isError($err)
-                and $err->getCode() == AI_ERROR_NO_RESULTADOS) {
-            return true;
+        // Obtengo secciones.
+        $id = intval($this->grupo);
+        extract($this->conf['secciones'], EXTR_PREFIX_ALL, 'cnf');
+        $res = $db->query(
+            "DELETE FROM $cnf_base.$cnf_tabla WHERE $cnf_id = $id");
+        if (DB::isError($res)) {
+            return $res;

         }
         }

-        // Si no, se devuelve el error.
-        return $err;
+        return true;

     }
     // -X2C
 
     }
     // -X2C
 


@@ -306,28 +230,16 @@ class AI_GrupoSecciones {
      */
     function getSeccionesArray($db = null) // ~X2C
     {
      */
     function getSeccionesArray($db = null) // ~X2C
     {

-               if ($db === null) {
-                       $db =& DB::connect('mysql://mark:mark@intranet-db.mecon.ar/MEconDAV',
-                               array('persistent' => true));
-               }
-               if (DB::isError($db)) {
-                       return $db;
-               }
-               return $db->getAssoc('
-            SELECT   S.section_id, S.description
-            FROM     MEconDAV.Sections as S
-            ORDER BY description');
-    }
-    // -X2C
-
-    // +X2C Operation 500
-    /**
-     * @return GrupoSecciones
-     * @access public
-     */
-    function __clone() // ~X2C
-    {
-        return $this;
+        static $conf;
+        if (!$conf) {
+            $conf = parse_ini_file(AI_GRUPOSECCIONES_CONFFILE, true);
+            $conf = $conf['mecondav'];
+        }
+        extract($conf);
+               return $db->getAssoc("
+            SELECT   $id, $campo
+            FROM     $base.$tabla
+            ORDER BY $campo ASC");

     }
     // -X2C
 
     }
     // -X2C