From 6fc5459214fc32876dad2acdde6a9fa7c7d6caa3 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?R=C3=A9mi=20Gaillard?= <remi.gaillard@prestashop.com>
Date: Mon, 25 Feb 2013 19:50:53 +0100
Subject: [PATCH] [-] MO: Fix max lenght allowed for 3 field on homeslider
 configuration #PSCFV-7897

---
 modules/homeslider/homeslider.php | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/modules/homeslider/homeslider.php b/modules/homeslider/homeslider.php
index 98c0c6e48..1609289be 100644
--- a/modules/homeslider/homeslider.php
+++ b/modules/homeslider/homeslider.php
@@ -463,15 +463,15 @@ class HomeSlider extends Module
 			$languages = Language::getLanguages(false);
 			foreach ($languages as $language)
 			{
-				if (strlen(Tools::getValue('title_'.$language['id_lang'])) > 40)
+				if (Tools::strlen(Tools::getValue('title_'.$language['id_lang'])) > 255)
 					$errors[] = $this->l('Title is too long');
-				if (strlen(Tools::getValue('legend_'.$language['id_lang'])) > 40)
+				if (Tools::strlen(Tools::getValue('legend_'.$language['id_lang'])) > 255)
 					$errors[] = $this->l('Legend is too long');
-				if (strlen(Tools::getValue('url_'.$language['id_lang'])) > 200)
+				if (Tools::strlen(Tools::getValue('url_'.$language['id_lang'])) > 255)
 					$errors[] = $this->l('URL is too long');
-				if (strlen(Tools::getValue('description_'.$language['id_lang'])) > 400)
+				if (Tools::strlen(Tools::getValue('description_'.$language['id_lang'])) > 4000)
 					$errors[] = $this->l('Description is too long');
-				if (strlen(Tools::getValue('url_'.$language['id_lang'])) > 0 && !Validate::isUrl(Tools::getValue('url_'.$language['id_lang'])))
+				if (Tools::strlen(Tools::getValue('url_'.$language['id_lang'])) > 0 && !Validate::isUrl(Tools::getValue('url_'.$language['id_lang'])))
 					$errors[] = $this->l('URL format is not correct');
 				if (Tools::getValue('image_'.$language['id_lang']) != null && !Validate::isFileName(Tools::getValue('image_'.$language['id_lang'])))
 					$errors[] = $this->l('Invalid filename');
@@ -481,11 +481,11 @@ class HomeSlider extends Module
 
 			/* Checks title/url/legend/description for default lang */
 			$id_lang_default = (int)Configuration::get('PS_LANG_DEFAULT');
-			if (strlen(Tools::getValue('title_'.$id_lang_default)) == 0)
+			if (Tools::strlen(Tools::getValue('title_'.$id_lang_default)) == 0)
 				$errors[] = $this->l('Title is not set');
-			if (strlen(Tools::getValue('legend_'.$id_lang_default)) == 0)
+			if (Tools::strlen(Tools::getValue('legend_'.$id_lang_default)) == 0)
 				$errors[] = $this->l('Legend is not set');
-			if (strlen(Tools::getValue('url_'.$id_lang_default)) == 0)
+			if (Tools::strlen(Tools::getValue('url_'.$id_lang_default)) == 0)
 				$errors[] = $this->l('URL is not set');
 			if (!Tools::isSubmit('has_picture') && (!isset($_FILES['image_'.$id_lang_default]) || empty($_FILES['image_'.$id_lang_default]['tmp_name'])))
 				$errors[] = $this->l('Image is not set');