Fix: Refonte de la gestion des erreurs du module webcalendar

htdocs/compta/clients.php

@@ -20,11 +20,21 @@
  * $Source$
  *
  */
+
+/*!
+	    \file       htdocs/compta/clients.php
+        \ingroup    compta
+		\brief      Page accueil des clients
+		\version    $Revision$
+*/
+ 
 require("./pre.inc.php");
 require("../contact.class.php");
-require("../lib/webcal.class.php");
 require("../cactioncomm.class.php");
 require("../actioncomm.class.php");
+if ($conf->webcal->enabled) {
+    require("../lib/webcal.class.php");
+}
 
 llxHeader();
 

htdocs/compta/fiche.php

@@ -29,7 +29,6 @@
 
 require("./pre.inc.php");
 require("../contact.class.php");
-require("../lib/webcal.class.php");
 require("../cactioncomm.class.php");
 require("../actioncomm.class.php");
 require("../facture.class.php");

htdocs/lib/webcal.class.php

@@ -1,5 +1,6 @@
 <?php
 /* Copyright (C) 2002-2003 Rodolphe Quiedeville <rodolphe@quiedeville.org>
+ * Copyright (C) 2004      Laurent Destailleur  <eldy@users.sourceforge.net>
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -20,53 +21,63 @@
  *
  */
 
-/*!	\file htdocs/lib/webcal.class.php
+/*!
-		\brief Classe permettant d'acceder a la database webcalendar.
+        \file       htdocs/lib/webcal.class.php
+        \ingroup    webcal
+		\brief      Ensemble des fonctions permettant d'acceder a la database webcalendar.
 		\author     Rodolphe Quiedeville.
 		\version    $Revision$
-
-		Ensemble des fonctions permettant d'acceder a la database webcalendar.
 */
 
-/*! \class Webcal
+require_once (DOL_DOCUMENT_ROOT ."/lib/".$conf->webcal->db->type.".lib.php");
-		\brief Classe permettant d'acceder a la database webcalendar
 
-		Ensemble des fonctions permettant d'acceder a la database webcalendar
+/*!
+        \class      Webcal
+		\brief      Classe permettant d'acceder a la database webcalendar
 */
 
 class Webcal {
   var $localdb;
   var $heure = -1;
   var $duree = 0;
+  var $date;
+  var $texte;
+  var $desc;
   
 /*!
-		\brief Permet de se connecter a la database webcalendar.
+		\brief      Constructeur de la classe d'interface à Webcalendar
 */
 
   function Webcal()
     {
       global $conf;
 
-      $this->localdb = new Db($conf->webcal->db->type,
+      // On initie la connexion à la base Webcalendar
+      $this->localdb = new DoliDb(
+                    $conf->webcal->db->type,
                     $conf->webcal->db->host,
                     $conf->webcal->db->user,
                     $conf->webcal->db->pass,
                     $conf->webcal->db->name);
     }
 
+
 /*!
-		\brief ajoute une entree dans le calendrier de l'utilsateur
+		\brief      Ajoute une entree dans le calendrier de l'utilisateur
 		\param[in]  user		le login de l'utilisateur
 		\param[in]  date		la date de l'evenement dans le calendrier
 		\param[in]  texte		le titre a indiquer dans l'evenement
 		\param[in]  desc		la description a indiquer dans l'evenement
+        \return     int         1 en cas de succès, -1,-2, -3 en cas d'erreur
 */
 
   function add($user, $date, $texte, $desc)
     {
 
+      // Recupère l'id max+1 dans la base webcalendar
       $id = $this->get_next_id();
 
+      if ($id > 0) {
           $cal_id = $id;
           $cal_create_by = $user->webcal_login;
           $cal_date = strftime('%Y%m%d', $date);
@@ -94,30 +105,35 @@ class Webcal {
             
             		if ( $this->localdb->query($sql) )
             		{
-
+            		    // OK
+                        return 1;        
             		}
             		else
             		{
             		    $error = $this->localdb->error() . '<br>' .$sql;
+                        return -1;
             		}
             	}
             else
             	{
                 	$error = $this->localdb->error() . '<br>' .$sql;
+                    return -2;
+            	}
+        }
+        else {
+        	$error = $this->localdb->error() . '<br>' .$sql;
+            return -3;
+        }
     }
 
-      	$this->localdb->close();
-    	}
 
 /*!
-		\brief obtient l'id suivant dans le webcalendar
+		\brief      Obtient l'id suivant dans le webcalendar
 		\return     id	retourne l'id suivant dans le webcalendar
 */
 
-
   function get_next_id()
     {
-
         $sql = "SELECT max(cal_id) FROM webcal_entry";
 
         if ($this->localdb->query($sql))
@@ -128,6 +144,7 @@ class Webcal {
         else
         {
             print $this->localdb->error();
+            return -1;
         }
     }
 }