Zend FR

sekaijin · 06-04-2009 09:51:14

salut à tous je passe de la 1.0.4 à la 1.7
oui je sais c'est le grand écart
je viens de tomber sur un petit pb

dans la 1.0.4 mes actions portaient des noms en Camel Case ex : getListAction showFormAction
et en passant à la 1.7 je tombe sur une erreur plutôt surprenante

Code:

Fatal error: Uncaught exception 'Zend_Controller_Action_Exception' with message 'Action "getlist" does not exist and was not trapped in __call()'

j'ai mis un certain temps à comprendre que cela venait de la majuscule
gasp une énorme régression getList doit s'écrire getlist showForm showform etc.

je ne sais pas d'où viens la décision d'un tel choix mais c'est contraire à toutes les règles codage

A+JYT

etaty · 06-04-2009 10:02:46

Sinon je crois qu'il faut appeler dans l'url avec un /controlleur/get-list

alien7 · 06-04-2009 10:06:54

Et renommer le dossier view en show-form.phtml

mikaelkael · 06-04-2009 10:40:41

Hello,

Sinon pour aller vite (si c'est nécessaire), tu as un flag pour revenir dans l'ancien mode. Regardes le chapitre migration dans Zend_Controller (passage de 1.0 à 1.5, de mémoire).

A+

sekaijin · 06-04-2009 14:51:51

je ne cherche pas à revenir en arrière je m'étonne d'une telle décision qui à mon avis ne va pas dans la bonne direction;

ça dénormalise le code et ça c'est jamais bon.

A+JYT

mikaelkael · 06-04-2009 14:58:35

Hello,

C'est bien dit dans la doc : "Option la moins souhaitable : vous pouvez forcer le distributeur à distribuer les noms d'action écrits en notationCamel avec un nouveau drapeau du contrôleur frontal, useCaseSensitiveActions".

Il est parfois dommage de retarder une évolution parce qu'on a une centaine d'actions et de vues à renommer. Ce flag permet de rapidement tester une nouvelle version sans tout casser le code. Ensuite tu peux au fur et à mesure engager la migration et une fois terminée, désactiver le flag.

A+

MoineauDeParis · 06-04-2009 15:02:25

vouais... je me suis plié aux nouvelles règles mais pas de bonne grâce

DDdeK · 06-04-2009 18:23:27

Ce choix a été fait, car il facilite l'indexation des moteurs de recherche, mais c'est vrai que vu du développeur, c'est pas terrible :-(

DDdeK · 06-04-2009 18:26:09

Du coup, on a des showlistAction, des showformAction... Heureusement que nos méthodes d'action ne sont pas longues. Pour les méthodes privées, on peut reprendre le camelCase.

nORKy · 06-04-2009 20:00:25

Et pourquoi mettre 'show' devant toutes tes fonctions ??

listAction suffit. Si t'es dans un controller "groupe", c'est compréhensible, sans compter le sous classement module.

Vous êtes vraiment difficiles..

MoineauDeParis · 07-04-2009 09:22:39

nORKy a écrit:
Et pourquoi mettre 'show' devant toutes tes fonctions ?? listAction suffit.

Bah tiens c'est pas bête ça, on prend des habitudes, on s'y tient, et un beau jour on se demande pourquoi ?

Mr Sekaijin, je sais que tu vas rebondir sur ce qui suit...
Dans nos contrôleur de type CRUD, nous avions la liste des actions suivantes :
showList(RETRIEVE),
add (CREATE), edit(UPDATE), showForm, checkForm, save,
delete, confirm, remove(DELETE)

Seules trois actions sont concernées par ce "souci" et rien n'empêche d'abréger tout en restant parfaitement compréhensible :
list,
add, edit, form, check, save,
delete, confirm, remove

Les notions de module et de contrôleur sont effectivement suffisant (a priori) pour que ces mots soient significatifs dans leur contexte.
Pour désigner une action on ne devrait avoir que des verbes et je sèche un peu pour remplacer 'form'... un show tout seul n'est pas très parlant et c'est d'ailleurs la raison du showForm originel ! Un 'get' peut-être ou un 'ask' ?...

Existe-t-il des "best practices" sur ces nommages ?

sekaijin · 07-04-2009 09:34:06

oui effectivement on peut toujours chercher un contournement qui soit plus ou moins acceptable
ça ne change pas le fond du problème qu'introduire une règle qui dénormalise n'apporte rien de bon.

que faire si dans un Contrôleur sur les contact j'ai deux action list ou get pour les groupes et les contacts par exemple ?

on pourra toujours trouver une solution
mais ne pas garder le camel case qu'est-ce que ça apporte à ZF ? je ne vois rien

A+JYT

mikaelkael · 07-04-2009 09:54:52

Hello,

sekaijin a écrit:
mais ne pas garder le camel case qu'est-ce que ça apporte à ZF ? je ne vois rien

Ben, tu peux le garder ! Tes nommages de vues et appels de liens doivent cependant changer.

Action : showListAction => Vue : show-list.phtml => Lien : /index/show-list
ou
Action : showlistAction => Vue : showlist.phtml => Lien : /index/showlist

A+

Zend FR

#1 06-04-2009 09:51:14

ZF 1.7

Code:

#2 06-04-2009 10:02:46

Re: ZF 1.7

#3 06-04-2009 10:06:54

Re: ZF 1.7

#4 06-04-2009 10:40:41

Re: ZF 1.7

#5 06-04-2009 14:51:51

Re: ZF 1.7

#6 06-04-2009 14:58:35

Re: ZF 1.7

#7 06-04-2009 15:02:25

Re: ZF 1.7

#8 06-04-2009 18:23:27

Re: ZF 1.7

#9 06-04-2009 18:26:09

Re: ZF 1.7

#10 06-04-2009 20:00:25

Re: ZF 1.7

#11 07-04-2009 09:22:39

Re: ZF 1.7

nORKy a écrit:

#12 07-04-2009 09:34:06

Re: ZF 1.7

#13 07-04-2009 09:54:52

Re: ZF 1.7

sekaijin a écrit:

Pied de page des forums