Vote Details for "Services_Ebay" by alan_k

» Details
  • Voter: Alan Knowles 
  • Vote: +1 (conditional)
  • Reviews: Cursory source review
» Comment
- consider calling the load* methods callFactory() and modelFactory()

- can you remove the define XXXX for the includes, and follow PEAR CS.

- consider removing includes of call/session/transport base classes, at top? - load them only as needed, in the factory method?

- consider using PEAR_Exception?