On 7/21/05, David Zülke <[EMAIL PROTECTED]> wrote: > I think the arguments should be passed to the constructor. Initialize() > already gets the context as the first parameter. This breaks with the > current conventions in Agavi, tho, where __construct() is basically never > used.
agreed. > We'd drop that SingletonModel class, by the way. All models, singleton or > not, will extend Model. So the "be a Singleton, baby" decision is moved from > the declaration to the code. That means that you cannot quickly change a > Singleton model to a usual model or vice versa without changing all code > lines that retrieve the model instance. I don't like this, and it was an > essential point of my previous draft that singleton and non-singleton models > should be retrieved in the code the exactly same way, but apparently there's > no way Mike and the others will agree to the approach I already implemented. > At least that's the impression I got ;) Aww, that's not true! I just didnt like the implementation. When's your host going to get around to upgrading? :) -Mike _______________________________________________ agavi-dev mailing list [email protected] http://labworkz.com/cgi-bin/mailman/listinfo/agavi-dev
