Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

move line #94

Open
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants

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

@AndyLCorp AndyLCorp Update library/Centurion/Controller/CRUD.php
Grid/_form must be at the end of the table
83c07e2
Contributor

lchenay commented Aug 31, 2012

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

Owner

Floby commented Aug 31, 2012

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.

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment