From b950043618da0464c63658aeb4c121e595604a1a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20FRANCE?= <frederic.france@free.fr>
Date: Sun, 16 Sep 2018 15:16:08 +0200
Subject: [PATCH 1/2] trimParameters

---
 .../sociales/class/cchargesociales.class.php  | 37 ++++++-------------
 htdocs/core/class/commonobject.class.php      | 18 ++++++++-
 2 files changed, 29 insertions(+), 26 deletions(-)

diff --git a/htdocs/compta/sociales/class/cchargesociales.class.php b/htdocs/compta/sociales/class/cchargesociales.class.php
index 409ee99aaae26..4778a135f7b24 100644
--- a/htdocs/compta/sociales/class/cchargesociales.class.php
+++ b/htdocs/compta/sociales/class/cchargesociales.class.php
@@ -77,31 +77,18 @@ public function create(User $user, $notrigger = false)
 
 		$error = 0;
 
-		// Clean parameters
-
-		if (isset($this->libelle)) {
-			 $this->libelle = trim($this->libelle);
-		}
-		if (isset($this->deductible)) {
-			 $this->deductible = trim($this->deductible);
-		}
-		if (isset($this->active)) {
-			 $this->active = trim($this->active);
-		}
-		if (isset($this->code)) {
-			 $this->code = trim($this->code);
-		}
-		if (isset($this->fk_pays)) {
-			 $this->fk_pays = trim($this->fk_pays);
-		}
-		if (isset($this->module)) {
-			 $this->module = trim($this->module);
-		}
-		if (isset($this->accountancy_code)) {
-			 $this->accountancy_code = trim($this->accountancy_code);
-		}
-
-
+        // Clean parameters
+        $this->trimParameters(
+            array(
+                'libelle',
+                'deductible',
+                'active',
+                'code',
+                'fk_pays',
+                'module',
+                'accountancy_code',
+            )
+        );
 
 		// Check parameters
 		// Put here code to add control on parameters values
diff --git a/htdocs/core/class/commonobject.class.php b/htdocs/core/class/commonobject.class.php
index 5fd7e89684fe0..27c8ed1563c41 100644
--- a/htdocs/core/class/commonobject.class.php
+++ b/htdocs/core/class/commonobject.class.php
@@ -7324,7 +7324,7 @@ public function fetchComments()
 		$comment = new Comment($this->db);
 		$result=$comment->fetchAllFor($this->element, $this->id);
 		if ($result<0) {
-			$this->errors=array_merge($this->errors,$comment->errors);
+			$this->errors=array_merge($this->errors, $comment->errors);
 			return -1;
 		} else {
 			$this->comments = $comment->comments;
@@ -7341,4 +7341,20 @@ public function getNbComments()
 	{
 		return count($this->comments);
 	}
+
+    /**
+     * Trim object parameters
+     * @param string[] $parameters array of parameters to trim
+     *
+     * @return void
+     */
+    public function trimParameters($parameters)
+    {
+        if (!is_array($parameters)) return;
+        foreach ($parameters as $parameter) {
+            if (isset($this->$parameter)) {
+                $this->$parameter = trim($this->$parameter);
+            }
+        }
+    }
 }

From f935701d3010480bfd90039ade6ba48ae15a716e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20FRANCE?= <frederic.france@free.fr>
Date: Sun, 16 Sep 2018 15:35:19 +0200
Subject: [PATCH 2/2] trimParameters

---
 .../sociales/class/cchargesociales.class.php  | 33 +++++++------------
 1 file changed, 11 insertions(+), 22 deletions(-)

diff --git a/htdocs/compta/sociales/class/cchargesociales.class.php b/htdocs/compta/sociales/class/cchargesociales.class.php
index 4778a135f7b24..ea055ac86996c 100644
--- a/htdocs/compta/sociales/class/cchargesociales.class.php
+++ b/htdocs/compta/sociales/class/cchargesociales.class.php
@@ -230,28 +230,17 @@ public function update(User $user, $notrigger = false)
 
 		// Clean parameters
 
-		if (isset($this->libelle)) {
-			 $this->libelle = trim($this->libelle);
-		}
-		if (isset($this->deductible)) {
-			 $this->deductible = trim($this->deductible);
-		}
-		if (isset($this->active)) {
-			 $this->active = trim($this->active);
-		}
-		if (isset($this->code)) {
-			 $this->code = trim($this->code);
-		}
-		if (isset($this->fk_pays)) {
-			 $this->fk_pays = trim($this->fk_pays);
-		}
-		if (isset($this->module)) {
-			 $this->module = trim($this->module);
-		}
-		if (isset($this->accountancy_code)) {
-			 $this->accountancy_code = trim($this->accountancy_code);
-		}
-
+        $this->trimParameters(
+            array(
+                'libelle',
+                'deductible',
+                'active',
+                'code',
+                'fk_pays',
+                'module',
+                'accountancy_code',
+            )
+        );
 
 
 		// Check parameters