|
Previous
·
Next
|
JT
|
Date: 2/20/2008 12:35 am · Subject: C2: oops · Rating: 0
I must have been smoking some serious peyote when I wrote the ship and
pay docs for the new commerce system. They were really hosed. Sorry
for that. The new versions are checked into svn now.
JT Smith
ph: 703-286-2525 x810
fx: 312-264-5382
Create like a god. Command like a king. Work like a slave.
|
| Back to Top |
Rate [ | ]
|
| |
JT
|
Date: 2/21/2008 11:18 am · Subject: C2: Ship & Pay Driver Change · Rating: -3
I'm considering that maybe we should take the class methods from WebGUI::Shop::Pay and WebGUI::Shop::Ship and move them into WebGUI::Shop::PayDriver and WebGUI::Shop::ShipDriver. We can then of course rename those classes to the shorter versions of the name. My reason for this is to make it more consistent with the asset API. With asset instead of having this external class with it's own new() method, we create other methods like newByDynamicClass(). I'm not sure which way is better, but one thing that is definitely better is consistency. Colin, Joeri, and Arjan, since you guys are the ones working on this stuff, what is your opinion?
|
| Back to Top |
Rate [ | ]
|
| |
colink
|
Date: 2/21/2008 12:42 pm · Subject: Re: C2: Ship & Pay Driver Change · Rating: 4
Speaking for Ship only... I don't see any problems. create becomes createByClassname. We also move _loadDriver to ShipDriver. getShippers translates straight across, no issues. getDrivers translates straight across, no issues. That leaves Shop::Ship with the www methods, www_listShippers, www_editShipper, www_editShipperSave.
|
| Back to Top |
Rate [ | ]
|
| |
JT
|
Date: 2/21/2008 2:25 pm · Subject: Re: C2: Ship & Pay Driver Change · Rating: 2
> Speaking for Ship only...
>
> I don't see any problems.
>
> create becomes createByClassname. We also move _loadDriver to
> ShipDriver.
>
> getShippers translates straight across, no issues.
>
> getDrivers translates straight across, no issues.
>
> That leaves Shop::Ship with the www methods, www_listShippers,
> www_editShipper, www_editShipperSave.
>
No it doesn't. There is no Shop::Ship anymore. Or rather
Shop::ShipDriver becomes Shop::Ship and Shop::ShipDriver no longer
exists.
JT Smith
ph: 703-286-2525 x810
fx: 312-264-5382
Create like a god. Command like a king. Work like a slave.
|
| Back to Top |
Rate [ | ]
|
| |
JT
|
Date: 2/21/2008 2:30 pm · Subject: Re: C2: Ship & Pay Driver Change · Rating: 8
On second thought, just forget about it. It works fine the way it is.
JT Smith
ph: 703-286-2525 x810
fx: 312-264-5382
Create like a god. Command like a king. Work like a slave.
|
| Back to Top |
Rate [ | ]
|
| |
|
|
Recent Discussions Color Key |
| Design: |
|
| Development: |
|
| Et Cetera: |
|
| Install/Upgrade: |
|
| Smoketest: |
|
| Template Group: |
|
Re: Site paid for by advertizing by Klaus - Fri @ 02:27am Re: Site paid for by advertizing by pwrightson - Thu @ 10:59am Re: Site paid for by advertizing by JT - Thu @ 08:58am Re: Regelmäßiger Termin für Usertreffen in der Rhein-Neckar-Region by Klaus - Thu @ 06:11am Re: Site paid for by advertizing by knowmad - Wed @ 08:07pm
|
