From b8c4d3a5132c7062bc6c12e2c9fa0ffe7dc9b1f3 Mon Sep 17 00:00:00 2001
From: Dany De Bontridder <danydb@noalyss.eu>
Date: Sun, 17 Dec 2017 18:23:18 +0100
Subject: [PATCH] Acc_Account : add function , improve function verify

---
 include/class/acc_account.class.php | 51 +++++++++++++++++++++++++++--
 1 file changed, 48 insertions(+), 3 deletions(-)

diff --git a/include/class/acc_account.class.php b/include/class/acc_account.class.php
index b3b8057f1..1d1e15e25 100644
--- a/include/class/acc_account.class.php
+++ b/include/class/acc_account.class.php
@@ -53,11 +53,14 @@ class Acc_Account
         $this->db=$p_cn;
         $id=-1;
         if ( trim($pcm_val)  != "" ) {
+            $pcm_val=$this->db->get_value("select format_account($1)",
+                                        array($pcm_val));
             $id=$p_cn->get_value("select id from tmp_pcmn where pcm_val=$1",[$pcm_val]);
         }
         if ( $id == "") { $id=-1;}
         $this->data_sql=new Tmp_Pcmn_SQL($p_cn, $id);
         $this->data_sql->pcm_val=$pcm_val;
+
     }
     public function get_parameter($p_string)
     {
@@ -103,6 +106,22 @@ class Acc_Account
         $sql="select count(*) from tmp_pcmn where pcm_val=$1";
         return $this->db->get_value($sql,array($p_value));
     }
+    /**
+     * Find the parent of an account
+     * @return string (pcm_val)
+     */
+    function find_parent() {
+        $name=$this->data_sql->pcm_val;
+        $length_name=strlen($name);
+        $parent="";
+        for ($i = 1;$i <$length_name;$i++) {
+            $parent=mb_substr($name, 0, $length_name-$i);
+            $exist=$this->db->get_value("select count(*) from tmp_pcmn where pcm_val=$1",[$parent]);
+            if ( $exist == 1) return $parent;
+        }
+        
+        return $parent;
+    }
     /**
      * Check before inserting or updating
      */
@@ -112,15 +131,36 @@ class Acc_Account
                            [$this->data_sql->pcm_val,$this->data_sql->id]);
         if ( $count > 0)
             throw new Exception (_("Poste en double"),EXC_DUPLICATE);
+        
         if (trim($this->data_sql->pcm_lib)=="")
             throw new Exception (_("Libellé vide"),EXC_PARAM_VALUE);
+        
+        // can not depend of itself
+        if ( $this->data_sql->pcm_val_parent == $this->data_sql->pcm_val)
+            throw new Exception (_("Poste parent incorrect"),EXC_PARAM_VALUE);
+        
+        if ( $this->data_sql->pcm_val_parent == "") {
+            $account=$this->find_parent();
+            $this->data_sql->pcm_val_parent=$account;
+            if ($account == "") 
+                throw new Exception (_("Poste Parent n'existe pas"),EXC_PARAM_VALUE);
+        }
+        
+        // purpose not clear
         if ( $this->data_sql->count(" where pcm_val = $1 and pcm_val <> $2",
                 [$this->data_sql->pcm_val_parent,$this->data_sql->pcm_val])  == 0)
-            throw new Exception (_("Parent n'existe pas"),EXC_PARAM_VALUE);
+            throw new Exception (_("Poste Parent n'existe pas"),EXC_PARAM_VALUE);
+        
+     
         if ( $this->data_sql->pcm_direct_use != 'N' && $this->data_sql->pcm_direct_use != 'Y') 
             throw new Exception (_("Paramètre incorrect"),EXC_PARAM_VALUE);
+        
         if ( trim($this->data_sql->pcm_val)==""||trim($this->data_sql->pcm_val_parent)=="")
             throw new Exception (_("Paramètre incorrect"),EXC_PARAM_VALUE);
+        
+        if ( strlen($this->data_sql->pcm_val)>40) {
+            throw new Exception (_("Poste comptable doit être de 40 caractères maximum"),EXC_PARAM_VALUE);
+        }
                 
     }
     function update() {
@@ -143,8 +183,13 @@ class Acc_Account
         $this->data_sql->delete();
 
     }
+    
     function save() {
-        $this->verify();
-        $this->data_sql->save();
+        try {
+            $this->verify();
+            $this->data_sql->save();
+        } catch (Exception $e) {
+            throw $e;
+        }
     }
  }