developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 13 years ago

Closed 13 years ago

#595 closed enhancement (invalid)

jelix plugin history : reload and back should accept additional params

Reported by: bibo Owned by:
Priority: normal Milestone:
Component: jelix Version: trunk
Severity: minor Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

Just a little enhancement to history plugin.

reload is a handy helpful method to execute a cibling action and return to the calling action. I see a need to pass additional parameters when returning to the calling action. Just to describe eventual modification induced by cibling action.

this ticket is about adding an optional argument to back and reload describing additional params.

Attachments (1)

jelix-trunk-#595.patch (4.4 KB) - added by bibo 13 years ago.

Download all attachments as: .zip

Change History (4)

Changed 13 years ago by bibo

comment:1 Changed 13 years ago by bibo

  • review set to review?
  • Severity changed from normal to minor
  • Type changed from bug to enhancement
  • Version changed from 1.0.3 to trunk

comment:2 Changed 13 years ago by Lipki

Je ne suis pas vraiment d'accord, si on lui passe des paramètres le reload n'est plus un reload.

Le principe de reload n'est pas de recharger la page, mais de fournir le même requête que précédemment. Ensuite tu en fait ce que tu veut.

exemple :

$history = $GLOBALS['gJCoord']->getPlugin('history');
$retour = $history->reload( $this->getResponse('redirect') );

// modifications des parametres
$retour->param['truc'] = 'hop';

return $retour;

Le reload fait sont boulot, le développeur fait le sien.

comment:3 Changed 13 years ago by bibo

  • Resolution set to invalid
  • review review? deleted
  • Status changed from new to closed

Ok, désolé de ne pas avoir vu cette possibilité de suite. Cependant, il serait bon d'ajouter cela dans la doc.

Cependant, je trouve le nom de 'reload' un peu ambigu cote developpeur. car en fait, le cas typique d'utilisation (donné dans la doc) est le suivant:

  • une action affiche un lien/un bouton de submit
  • le click sur ce lien execute une action cible
  • le code de cette action cible fait appel a 'history->reload' qui en fait renvoie l'action précédente (qui est la derniere enregistré par l'historique)

Du point de vue développeur, c'est plutot un back. Meme si du point de vue utilisateur, celui-ci reste sur la meme url.

Peut-etre un meilleur nom : history->loadLast() et pour back history->loadBack()

Note: See TracTickets for help on using tickets.