Vote Details for "Livejournal" by cweiske

» Details
» Comment
Beside the issues Justin found, there are more things:
- Why are you polluting the global namespace for error constants that can be defined as class constants?
- Use the pear directory structure in your examples. You do things like >require_once('../../Services_Livejournal/xxx')< that don't work in pear-installed environment.
- ... probably some more

The package idea itself is good, but the code not yet.