Login | Register
My pages Projects Community openCollabNet

Discussions > dev > Re: [propel-dev] roadmap > Reply to message

propel
Reply to message

* = Required fields
* Subject
* Body
Attachments
Send reply to
Topic
Author (directly in email)
Please type the letters in the image above.

Original message

Author hlellelid
Full name Hans Lellelid
Date 2006-03-28 09:24:14 PST
Message David Z├╝lke wrote:
>>> Adding the pages sounds good to me. The only concern I have (which I
>>> can mention in comments on wiki too) is that certain methods (temporal
>>> methods) take additional parameters -- which might present a problem
>>> __call/__get/__set, unless we just standardize on an API that does not
>>> support these. Just something to discuss.
>>
>> No problem IMHO, call_user_func_array() is our friend.
>
> Wait a minute, you were talking about the other way round, right? No
> problem either, __call gets an array of params anyway...!?

Yeah, the problem doesn't really apply to __call(), you're right. More
for __get()/__set():

Current API:

print "Date is " . $obj->getDateValue("Y-m-d");

Desired API (as I understand it):

print "Date is " . date("Y-m-d", $obj->dateValue);

... so basically we wouldn't be able to support the date formatting
parameter. Probably not a big deal, but a difference from current system.

Hans