| # |
Aug 8th 2008, 14:28 |
teknoid |
CrazyTux[m]: well, that's not a correct approach (most likely) in this case |
| # |
Aug 8th 2008, 14:27 |
TommyO |
it's gonna be 20 minutes of fixes before we even get to the ugly sql code on line 20. ughh |
| # |
Aug 8th 2008, 14:27 |
teknoid |
CrazyTux[m]: what exactly does your main/home page accompish? |
| # |
Aug 8th 2008, 14:27 |
CrazyTux[m] |
teknoid, well thats why I decided on a seperate controller? |
| # |
Aug 8th 2008, 14:27 |
teknoid |
CrazyTux[m]: that's for you to decide... |
| # |
Aug 8th 2008, 14:27 |
CrazyTux[m] |
or anything else that relates. |
| # |
Aug 8th 2008, 14:27 |
CrazyTux[m] |
TommyO, so why throw it in Conditions Controller, over Therapies |
| # |
Aug 8th 2008, 14:27 |
CrazyTux[m] |
TommyO, well, so does conditions. |
| # |
Aug 8th 2008, 14:26 |
TommyO |
well, could |
| # |
Aug 8th 2008, 14:26 |
TommyO |
it does directly relate |
| # |
Aug 8th 2008, 14:26 |
TommyO |
why have so many models in $uses when you only need one? |
| # |
Aug 8th 2008, 14:26 |
CrazyTux[m] |
even though the main page, does not directly relate, to controller X, or controller Y, yet you guys say throw a method in there, and route it to that, inside of a pre-existing controller? |
| # |
Aug 8th 2008, 14:26 |
TommyO |
I don't think you should. but set that aside for now |
| # |
Aug 8th 2008, 14:26 |
CrazyTux[m] |
So I should not create a seperate controller for the main page |
| # |
Aug 8th 2008, 14:26 |
TommyO |
and the key is in the phrase "best known practices" |
| # |
Aug 8th 2008, 14:26 |
TommyO |
*very |
| # |
Aug 8th 2008, 14:25 |
TommyO |
CrazyTux[m]: the "why?" means "why deviate from best known practices?" "why not?" therefore would mean "why not deviate?" which has a ver obvious answer to it. |
| # |
Aug 8th 2008, 14:24 |
TommyO |
and here you wouldn't need more than 1 model in it |
| # |
Aug 8th 2008, 14:24 |
teknoid |
CrazyTux[m]: because you are bending rules, loading extra objects and setting yourself for problems down the road (if you don't do things correctly to begin with) |
| # |
Aug 8th 2008, 14:24 |
TommyO |
CrazyTux[m]: in any case $uses is an override. you wouldn't need it if you used one of the real controllers |
| # |
Aug 8th 2008, 14:24 |
CrazyTux[m] |
TommyO, ok now your confusing me :) |
| # |
Aug 8th 2008, 14:23 |
TommyO |
why not is not as good a question as why? |
| # |
Aug 8th 2008, 14:23 |
CrazyTux[m] |
teknoid, why does this matter? |
| # |
Aug 8th 2008, 14:23 |
CrazyTux[m] |
teknoid, why not make it any other? |
| # |
Aug 8th 2008, 14:23 |
teknoid |
CrazyTux[m]: and? |
| # |
Aug 8th 2008, 14:22 |
CrazyTux[m] |
teknoid, this is the homepage. |
| # |
Aug 8th 2008, 14:22 |
teknoid |
CrazyTux[m]: you probably need only one model there if the other two are related to it... still why not make a main page in TherapiesController? |
| # |
Aug 8th 2008, 14:21 |
CrazyTux[m] |
TommyO, those two controllers exists, however this is the "main page" default, and I'm trying to show some info on it |
| # |
Aug 8th 2008, 14:21 |
TommyO |
or a ConitionsController |
| # |
Aug 8th 2008, 14:21 |
TommyO |
CrazyTux[m]: why not have this in a TherapiesController? |
| # |
Aug 8th 2008, 14:20 |
CrazyTux[m] |
Why is it when I remove the $uses = array('Therapy') bit it is no longer "included" yet you guys say I do not need to reference it? |
| # |
Aug 8th 2008, 14:20 |
TommyO |
what are you indexing here? |
| # |
Aug 8th 2008, 14:20 |
TommyO |
CrazyTux[m]: this controller is riddled with problems |
| # |
Aug 8th 2008, 14:20 |
CrazyTux[m] |
? |
| # |
Aug 8th 2008, 14:19 |
teknoid |
^^^ indeed |
| # |
Aug 8th 2008, 14:19 |
TommyO |
I'm afraid to ask what a MainController does exactly... |
| # |
Aug 8th 2008, 14:19 |
teknoid |
CrazyTux[m]: take a look at the manual |
| # |
Aug 8th 2008, 14:19 |
teknoid |
CrazyTux[m]: and also you don't need $uses (or at least not all of it), if you are using containable and have related models ther |
| # |
Aug 8th 2008, 14:19 |
CrazyTux[m] |
teknoid, I changed referencing it from Model.property, to just property, and same issue, any more references/documents on Containable, I'm looking at the API. |
| # |
Aug 8th 2008, 14:18 |
teknoid |
CrazyTux[m]: i told you earlier, that you are not using Containable correctly |
| # |
Aug 8th 2008, 14:18 |
CrazyTux[m] |
Model B: http://pastebin.com/d73d5bd5d |