Module Bannières 6.3 : impose qu'on mette une image ou du HTML


2 posts in this topic

Posted · Report post

Bonjour,

Après migration du d'une boutique de Peel 6.2 en Peel 6.3 (avec premium), la mise à jour d'une bannière ne fonctionne plus. Le problème vient du fichier modules/banner/administrer/fonctions.php, fonction maj_banniere(...).

Concrètement PHP indique que la variable $sql est inexistante, ce qui est vrai lorsque l'on ne passe pas dans le premier test puisqu'il défini la requête a exécuter. Visuellement on obtient donc des erreurs PHP, des avertissements PHP, ainsi qu'une erreur lors de la mise à jour de la bannière, qui n'est effectivement pas mise à jour en base.

Bien Cordialement,

D.Renard

Share this post


: post
Share on other sites

Posted · Report post

Bonjour,

Lors de la modification d'une bannière publicitaire, le tag HTML ou un image doit obligatoirement être présent pour que la modification soit effective.

Vous pouvez modifier ce fonctionnement en supprimant le test sur les bannières


function maj_banniere($id, $img, &$frm)
{
/* Met à jour la table banniere */
$sql = 'UPDATE peel_banniere SET
description = "' . nohtml_real_escape_string($frm['description']) . '"
, date_debut = "' . nohtml_real_escape_string($frm['date_debut']) . '"
, date_fin = "' . nohtml_real_escape_string($frm['date_fin']) . '"
, image = "' . nohtml_real_escape_string($img) . '"
, etat = "' . nohtml_real_escape_string($frm['etat']) . '"';
if (is_annonce_module_active()) {
$sql .= ', annonce_number = "' . intval(vn($frm['annonce_number'])) . '"';
}
$sql .= ', position = "' . nohtml_real_escape_string($frm['position']) . '"
, lien = "' . nohtml_real_escape_string($frm['lien']) . '"
, lang = "' . nohtml_real_escape_string($frm['lang']) . '"
, target = "' . nohtml_real_escape_string($frm['target']) . '"
, tag_html = "' . real_escape_string($frm['tag_html']) . '"
, extra_javascript = "' . real_escape_string($frm['extra_javascript']) . '"
, id_categorie = "' . intval(vn($frm['id_categorie'])) . '"
, appearance = "' . nohtml_real_escape_string(vb($frm['appearance'])) . '"
, width = "' . intval(vn($frm['width'])) . '"
, height = "' . intval(vn($frm['height'])) . '"
, rang = "' . intval(vn($frm['rang'])) . '"
WHERE id = "' . intval($id) . '"';
query($sql);
}
[/CODE]

Share this post


: post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!


Register a new account

Sign in

Already have an account? Sign in here.


Sign In Now

Twitter Advisto ecommerce

Facebook PEEL Shopping