Fix trim values for meta data

Author: AlfredoRamos

includes/helper.php

@@ -226,34 +226,7 @@ public function set_metadata($data = [])
 		// Map values to correct properties
 		foreach ($data as $key => $value)
 		{
-			if (is_string($value))
-			{
-				$value = trim($value);
-			}
-			else if (is_array($value))
-			{
-				$value = array_map(function($val) {
-					if (is_string($val))
-					{
-						return trim($val);
-					}
-
-					if (is_array($val))
-					{
-						foreach ($val as $k => $v)
-						{
-							if (!is_string($v))
-							{
-								continue;
-							}
-
-							$val[$k] = trim($v);
-						}
-
-						return $val;
-					}
-				}, $value);
-			}
+			$value = $this->trim_items($value);
 
 			switch ($key)
 			{
@@ -1411,6 +1384,56 @@ public function is_wide_image($width = 0, $height = 0)
 		return $is_wide;
 	}
 
+	/**
+	 * Trim strings from given data, recursively.
+	 *
+	 * @param mixed		$data
+	 * @param integer	$depth
+	 *
+	 * @return mixed
+	 */
+	public function trim_items($data = [], $depth = 0)
+	{
+		if (empty($data))
+		{
+			return [];
+		}
+
+		$max_depth = 5;
+		$depth = abs($depth) + 1;
+
+		// Do not go deeper, return data as is
+		if ($depth > $max_depth)
+		{
+			return $data;
+		}
+
+		if (!is_string($data) && !is_array($data))
+		{
+			return $data;
+		}
+
+		if (is_string($data))
+		{
+			return trim($data);
+		}
+
+		// Trim strings
+		return array_map(function($item) use ($depth) {
+			if (is_string($item))
+			{
+				return trim($item);
+			}
+
+			if (is_array($item))
+			{
+				return $this->trim_items($item, $depth);
+			}
+
+			return $item;
+		}, $data);
+	}
+
 	/**
 	 * Remove empty items from an array, recursively.
 	 *

phpunit.xml.dist

@@ -3,9 +3,9 @@
 	backupStaticAttributes="true"
 	bootstrap="../../../../tests/bootstrap.php"
 	colors="true"
-	convertErrorsToExceptions="false"
-	convertNoticesToExceptions="false"
-	convertWarningsToExceptions="false"
+	convertErrorsToExceptions="true"
+	convertNoticesToExceptions="true"
+	convertWarningsToExceptions="true"
 	processIsolation="false"
 	stopOnError="false"
 	stopOnFailure="false"