RE: [Catalyst] Refactoring question

2010-05-27 Thread Morad IGMIR
Hello Steve, I think your business logic should be moved to the model, not the controller. The model does the heavy lifting. Your controllers would be easier to maintain if they only did simple tasks like dispatching, calling models and forwarding to views, as said on the catalyst wiki :

Re: [Catalyst] Refactoring question

2010-05-27 Thread Steve
On 5/27/2010 3:51 AM, Morad IGMIR wrote: Hello Steve, I think your business logic should be moved to the model, not the controller. The model does the heavy lifting. My model is currently comprised only of my DBIC Result and ResultSet classes. Are you suggesting that some of the logic

RE: [Catalyst] Refactoring question

2010-05-27 Thread Morad IGMIR
If you’re using DBIC, De : Steve [mailto:st...@matsch.com] Envoyé : jeudi 27 mai 2010 14:38 À : The elegant MVC web framework Objet : Re: [Catalyst] Refactoring question On 5/27/2010 3:51 AM, Morad IGMIR wrote: Hello Steve, I think your business logic should be moved to the

RE: [Catalyst] Refactoring question

2010-05-27 Thread Morad IGMIR
Oops, sorry ; I accidentally hit « send » too fast ! Since you’re using DBIC, I suggest you take a look at the cat tutorial section on CPAN about “exploring the power of DBIC” : http://search.cpan.org/~hkclark/Catalyst-Manual-5.8004/lib/Catalyst/Manual/T

Re: [Catalyst] Refactoring question

2010-05-27 Thread Bill Moseley
On Thu, May 27, 2010 at 5:38 AM, Steve st...@matsch.com wrote: I think your business logic should be moved to the model, not the controller. The model does the heavy lifting. My model is currently comprised only of my DBIC Result and ResultSet classes. Are you suggesting that some of the

[Catalyst] Re: A point of confusion/frustration on chained actions

2010-05-27 Thread Aristotle Pagaltzis
* Ido Perlmuter i...@ido50.net [2010-05-20 00:35]: In this case all you have to do is create the comment action as a mid point, and then create another action with a different name, but the same PathPart (e.g. comment) only this time as an end-point, and all this action needs to do is forward

Re: [Catalyst] Refactoring question

2010-05-27 Thread Steve
Steve Schafer Matsch Systems Phone: 616-477-9629 Mobile: 616-304-9440 Email: st...@matsch.com Web: http://www.matsch.com/ On 5/27/2010 9:41 AM, Bill Moseley wrote: On Thu, May 27, 2010 at 5:38 AM, Steve st...@matsch.com wrote: I think your business logic

Re: [Catalyst] Refactoring question

2010-05-27 Thread Steve
Done. On 5/27/2010 11:29 AM, Charlie Garrison wrote: Good morning, On 27/05/10 at 11:09 AM -0400, Steve st...@matsch.com wrote: !DOCTYPE HTML PUBLIC -//W3C//DTD HTML 4.01 Transitional//EN html head meta content=text/html; charset=ISO-8859-1 http-equiv=Content-Type /head Don't we have rules

Re: [Catalyst] Refactoring question

2010-05-27 Thread Bill Moseley
On Thu, May 27, 2010 at 8:09 AM, Steve st...@matsch.com wrote: At present, the only DBIC-looking code is contained in two lines (chaining resultsets). I'm not sure it's worth the effort for this application, but it certainly will be for others. Can you elaborate on the custom base classes