diff --git a/modules/homeslider/HomeSlide.php b/modules/homeslider/HomeSlide.php index 68bee5a2e..1975ebbaf 100644 --- a/modules/homeslider/HomeSlide.php +++ b/modules/homeslider/HomeSlide.php @@ -44,7 +44,7 @@ class HomeSlide extends ObjectModel 'active', 'position' ); - protected $fieldsRequiredLang = array('title', 'description', 'url', 'legend'); + protected $fieldsRequiredLang = array('title', 'url', 'legend'); protected $fieldsSizeLang = array( 'description' => 4000, 'title' => 255, @@ -53,7 +53,7 @@ class HomeSlide extends ObjectModel 'image' => 255 ); protected $fieldsValidateLang = array( - 'title' => 'isName', + 'title' => 'isCleanHtml', 'description' => 'isCleanHtml', 'url' => 'isUrl', 'legend' => 'isCleanHtml', @@ -100,7 +100,6 @@ class HomeSlide extends ObjectModel INSERT INTO `'._DB_PREFIX_.'homeslider` (`id_shop`, `id_slide`) VALUES('.(int)$id_shop.', '.(int)$this->id.')' ); - return $res; } diff --git a/modules/homeslider/homeslider.php b/modules/homeslider/homeslider.php index fafe68d3c..192a885ad 100644 --- a/modules/homeslider/homeslider.php +++ b/modules/homeslider/homeslider.php @@ -262,7 +262,7 @@ class HomeSlider extends Module
1 - '.$this->l('Upload your slide').''; /* Image */ - $this->_html .= '
'; + $this->_html .= '
'; foreach ($languages as $language) { $this->_html .= '
'; @@ -293,7 +293,7 @@ class HomeSlider extends Module /* Form content */ /* Title */ - $this->_html .= '
'; + $this->_html .= '
'; foreach ($languages as $language) { $this->_html .= ' @@ -305,7 +305,7 @@ class HomeSlider extends Module $this->_html .= '


'; /* URL */ - $this->_html .= '
'; + $this->_html .= '
'; foreach ($languages as $language) { $this->_html .= ' @@ -317,7 +317,7 @@ class HomeSlider extends Module $this->_html .= '


'; /* Legend */ - $this->_html .= '
'; + $this->_html .= '
'; foreach ($languages as $language) { $this->_html .= ' @@ -330,7 +330,7 @@ class HomeSlider extends Module /* Description */ $this->_html .= ' - +
'; foreach ($languages as $language) { @@ -350,6 +350,7 @@ class HomeSlider extends Module No active) AND (int)$slide->active == 0)) ? 'checked="checked" ' : '').' value="0" /> +
'; /* Save */ @@ -361,6 +362,7 @@ class HomeSlider extends Module /* End of fieldset & form */ $this->_html .= ' +

*'.$this->l('Required fields').'

'; } @@ -405,9 +407,9 @@ class HomeSlider extends Module $errors[] = $this->l('Title is too long'); if (strlen(Tools::getValue('legend_'.$language['id_lang'])) > 40) $errors[] = $this->l('Legend is too long'); - if (strlen(Tools::getValue('url_'.$language['id_lang'])) > 40) + if (strlen(Tools::getValue('url_'.$language['id_lang'])) > 200) $errors[] = $this->l('URL is too long'); - if (strlen(Tools::getValue('description_'.$language['id_lang'])) > 200) + if (strlen(Tools::getValue('description_'.$language['id_lang'])) > 400) $errors[] = $this->l('Description is too long'); if (strlen(Tools::getValue('url_'.$language['id_lang'])) > 0 && !Validate::isUrl(Tools::getValue('url_'.$language['id_lang']))) $errors[] = $this->l('URL format is not correct'); @@ -425,8 +427,6 @@ class HomeSlider extends Module $errors[] = $this->l('Legend is not set'); if (strlen(Tools::getValue('url_'.$defaultLanguage)) == 0) $errors[] = $this->l('URL is not set'); - if (strlen(Tools::getValue('description_'.$defaultLanguage)) == 0) - $errors[] = $this->l('Description is not set'); if (Tools::getValue('image_'.$defaultLanguage) == "" AND !Validate::isFileName(Tools::getValue('image_'.$defaultLanguage)) AND !Tools::getValue('image_old_'.defaultLanguage)) $errors[] = $this->l("Image is not set"); if (Tools::getValue('image_old_'.$defaultLanguage) AND !Validate::isFileName(Tools::getValue('image_old_'.$defaultLanguage)))