dinislage/PrestaShop
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[-] MO: Fix max lenght allowed for 3 field on homeslider configuration #PSCFV-7897

Browse Source
This commit is contained in:
Rémi Gaillard
2013-02-25 19:50:53 +01:00
parent 95f43d3ac6
commit 6fc5459214
1 changed files with 8 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files
modules/homeslider/homeslider.php
+8 -8
View File
@@ -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');