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

Ticket #1448 (closed new feature: fixed)

Opened 3 years ago

Last modified 2 years ago

Permettre l'utilisation de __call dans les controlleurs.

Reported by: flav Owned by: flav
Priority: normal Milestone: Jelix 1.4
Component: jelix:controllers Version: 1.3.1
Severity: normal Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

allow to use

__call

method in controllers.

Ceci peut servir à pas mal de choses. Evidemment on peut toujours jongler avec la réécriture d'url personnalisée, mais pour un moindre coup (moins d'une ligne à modifier), cette solution offre des avantages de souplesses et de clareté en ce qui concerne la conception de la couche contrôlleurs (par exemple, on peut faire un système de squelette à la SPIP...)

Dans /lib/jelix/core/jCoordinator.class.php ligne 142, remplacer :

method_exists($ctrl, $selector->method)

par :

is_callable(array($ctrl, $selector->method))

Attachments

patch.txt (720 bytes) - added by flav 3 years ago.

Change History

Changed 3 years ago by flav

comment:1 Changed 3 years ago by flav

  • Component changed from jelix to jelix:controllers

comment:2 Changed 3 years ago by laurentj

  • Owner set to flav
  • Status changed from new to assigned
  • Milestone set to Jelix 1.4

Un petit pull request sur le master ? ;-)

comment:3 Changed 3 years ago by flav

Ouais, en fait désolé, mais le pull request, je le sens pas encore... :/

comment:4 Changed 3 years ago by foxmask

Pull Request pour le compte de Flav : https://github.com/jelix/jelix/pull/47

comment:5 Changed 2 years ago by laurentj

  • Status changed from assigned to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.