search

centurion-project / Centurion

Centurion is an open-source CMS delivered as a flexible PHP5 Content Management Framework
http://www.centurion-project.org
197 stars 51 forks source link

move line #94

Closed aladjadj closed 6 years ago

aladjadj commented 12 years ago

Grid/_form must be at the end of the table because it must be used if no view has been added previously in table

lchenay commented 12 years ago

If doing this, it's stop allow modifing this value in the preRenderForm() function. What the main problem of your pull request ?

Floby commented 12 years ago

I don't see what bug this patch is supposed to fix. Could you provide an example where the expected behaviour breaks ? Then we can figure out something to fix it without breaking any other use case.

aladjadj commented 12 years ago

Je vais écrire en français.

Effectivement il y a une erreur dans mon pull request, mon master n'est pas à jour par rapport au projet ou j'ai tracké le bug Sur le Master vous avez :

$this->view->formViewScript[] = 'grid/_form.phtml'; $this->_preRenderForm();

et dans mon pull request on devrait avoir :

$this->_preRenderForm(); $this->view->formViewScript[] = 'grid/_form.phtml';

quand on analyse le cheminement du selecteur de vue , on arrive à la méthode _selectScript dans Centurion_View

qui vas renvoyer la premier vue ( fichier .phtml ) qu'il trouve et dans notre cas ce serait grid/_form.phtml alors que l'on souhaiterai avoir la vue assigné dans preRenderForm();

donc dans la logique _selectScript dois tester l'existance de notre vue assigné dans _preRenderForm() puis si elle n'existe pas utiliser grid/_form.phtml.