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 12 years ago

Closed 12 years ago

Last modified 12 years ago

#690 closed enhancement (fixed)

Pour avoir un $this->param() dans les zone comme dans les contrôllurs

Reported by: analogue Owned by:
Priority: low Milestone: Jelix 1.1 beta 1
Component: jelix:utils Version: trunk
Severity: normal Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

Dans les contrôleurs, on doit utiliser $this->param() pour récupérer un paramètre, alors que la récupération d'un paramètre dans une zone se fait par $this->getParam().

Ce patch applicable sur la branche 1.0.x et sur le trunk permet de récupérer un paramètre dans une zone en utilisant $this->param() à la place de $this->getParam() pour éviter la confusion.

Attachments (2)

param.diff (1.5 KB) - added by analogue 12 years ago.
patch_param_amended (1.9 KB) - added by analogue 12 years ago.
Patch original prenant en compte les commentaires

Download all attachments as: .zip

Change History (9)

Changed 12 years ago by analogue

comment:1 Changed 12 years ago by laurentj

  • Milestone set to Jelix 1.1 beta 1
  • review changed from review? to review+

Rajouter @deprecated 1.1 dans le commentaire de getParam.

Et puis vu la super complexité de param(), je pense que plutôt d'appeler param dans getParam, mettre directement le code de param() serait mieux (perf, toussa...)

comment:2 Changed 12 years ago by laurentj

N'oublie pas de mettre à jour CREDITS aussi...

comment:3 Changed 12 years ago by laurentj

  • Version set to trunk

Et patch uniquement sur le trunk.

Changed 12 years ago by analogue

Patch original prenant en compte les commentaires

comment:4 Changed 12 years ago by analogue

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

Fixed in commit 1084

comment:5 Changed 12 years ago by analogue

  • Documentation needed unset

Doc modifiée dans le manuel français 1.1, pas de manuel anglais 1.1

comment:6 Changed 12 years ago by laurentj

  • Documentation needed set

comment:7 Changed 12 years ago by laurentj

  • Documentation needed unset
Note: See TracTickets for help on using tickets.