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. |