The controller code overall looks a bit excessive, but I can't really put it into context without seeing the associated view files. Also, it looks like you might be duplicating functionality that's built into Cake with UserController::buildMonthList, but I can't tell, since it's not posted.
Also, you're passing the view mutliple copies of the same data. What I'd recommend instead is using the Model's afterFind( ) callback to create virtual fields that represent the data to be displayed, i.e.: ['Plan']['flatRateText'] = 'Yes'. There's really a lot you could do here to reduce the overall volume of code, but it's hard to say just by looking at this one part of it. --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Cake PHP" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/cake-php -~----------~----~----~----~------~----~------~--~---
