#2563: [PATCH][TEST] Form helper should be able to set class parameter to required for fields from multiple models ----------------------------------------------------+----------------------- Reporter: biesbjerg | Owner: mark_story Type: Enhancement | Status: new Priority: Low | Milestone: 1.3.x.x Component: Helpers | Version: Severity: Trivial | Resolution: Keywords: formHelper class required biesbjerg | Php_version: PHP 5 Cake_version: | ----------------------------------------------------+----------------------- Comment (by alanic):
At first glance this looks like the "right" approach given the current approach. I solved the same thing without knowing this bug here: #6430 in a less intrusive but less general way I guess. I kind of question doing this beforehand for every field even though they will not be used, even in the case of using only one model. I would advocate for doing it right when they are used like I did in line 666 in the patch attached to #6430 for efficiency's sake. But does it matter at all if one is using caching? I don't know, I just started using CakePHP last week:) -- Ticket URL: <https://trac.cakephp.org/ticket/2563#comment:8> CakePHP : The Rapid Development Framework for PHP <https://trac.cakephp.org/> Cake is a rapid development framework for PHP which uses commonly known design patterns like ActiveRecord, Association Data Mapping, Front Controller and MVC. Our primary goal is to provide a structured framework that enables PHP users at all levels to rapidly develop robust web applications, without any loss to flexibility. --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "tickets cakephp" group. To post to this group, send email to tickets-cakephp@googlegroups.com To unsubscribe from this group, send email to tickets-cakephp+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/tickets-cakephp?hl=en -~----------~----~----~----~------~----~------~--~---