Dotclear

Ticket #1527 (closed enhancement: fixed)

Opened 11 years ago

Last modified 11 years ago

Erreur quand l'on valide une action sans séléctionner de post

Reported by: lipki Owned by: bruno
Priority: normal Milestone: 2.5.3
Component: module:core Version: 2.5
Severity: normal Keywords:
Cc:

Description

Si l'on valide par erreur une action dans posts.php, sans avoir cocher de post, on a cette erreur :

Notice: Undefined variable: action in C:\programmes\wamp\www\dotclear-default\admin\posts_actions.php on line 183

Une notice, et une demi-admin.

Cela est dû, je pense à cette première condition qui fait que la variable $action n'est jamais initialisé.

if (!empty($_POSTaction?) && !empty($_POSTentries?))

Change History

comment:1 Changed 11 years ago by franck

Sur quelle version ? Parce qu'avec une 2.5.1 il ne se passe rien de spécial.

comment:2 Changed 11 years ago by kozlika

Idem avec une 2.6 dev, je n'ai pas le souci.

comment:3 Changed 11 years ago by lipki

en 2.6 dev avec le javascript désactivez.

J'ai des modifs que je veux pas commit tout de suite, donc je peux pas tester sur 2.5, mais je pense que c'est aussi le cas avec le javascript désactivez.

Last edited 11 years ago by lipki (previous) (diff)

comment:4 Changed 11 years ago by franck <carnet.franck.paul@…>

  • Status changed from new to closed
  • Resolution set to fixed

(In [2250db7a1b33]) $action not initialized, fixes #1527

comment:5 Changed 11 years ago by lipki

  • Status changed from closed to reopened
  • Resolution fixed deleted

eu .. pas si simple.

deux nouvelles erreurs : Notice: Undefined variable: posts in C:\programmes\wamp\www\dotclear-default\admin\posts_actions.php on line 215

et

Fatal error: Call to a member function fetch() on a non-object in C:\programmes\wamp\www\dotclear-default\admin\posts_actions.php on line 215

Puis si on les corriges

Notice: Undefined variable: redir in C:\programmes\wamp\www\dotclear-default\admin\posts_actions.php on line 330

Et même ensuite on se retrouve avec seulement un bouton retour.

On peux y mettre un message d'erreur: "Vous devez choisir un ou plusieurs billet pour cette action"

Ou oublier cette page de résultat vide en redirigant vers posts.php. (je suis en train de regarder)

Ou les deux, une redirection vers posts.php plus message d'erreur.

comment:6 Changed 11 years ago by lipki

Sauf que ce message d'erreur n'est pas forcément justifié si une action ajouter via le behavior n'a pas besoin d'avoir de case coché.

J'ai fait une proposition, qui a besoin d'être bien relus et testé.  https://bitbucket.org/dotclear/dotclear/pull-request/51/ticket-1527/diff

comment:7 Changed 11 years ago by franck

  • Milestone changed from A definir to 2.5.3

comment:8 Changed 11 years ago by bruno

  • Owner changed from team to bruno
  • Status changed from reopened to new

comment:9 Changed 11 years ago by Dsls

  • Status changed from new to closed
  • Resolution set to fixed

(In [136cc800ade6]) Added entries list in posts_actions.php, ability to uncheck some entries in this page. see #1540 Added proper message when no entry selected, or no action set. closes #1527

comment:10 Changed 11 years ago by Dsls

(In [5a3fad0f97e2]) Tuned posts_actions.php, closes #1527 (again), see #1540.

Note: See TracTickets for help on using tickets.

Sites map