Vote Details for "System_Folders" by sfrausch

» Details
  • Voter: Stefano F. Rausch 
  • Vote: +1 (not conditional)
  • Reviews: Cursory source review, Run examples
» Comment
Please be so kind to correct the following issues before publishing the first official release:

- package.xml: please add the dependency to PEAR::Config to force the installation of the same and for avoiding the corresponding PHP fatal errors require_once-wise in Cached.php - I didn't have Config installed ...

- examples/cached.php: you're setting the documents folder to a non-existing one one my HD. Furthermore, if I would use the default configuration file in the future [$this->getAppData() . '/.net.php.pear.system.folders'], your '/home/cweiske/MyDocuments/' would most probably lead to some hick-ups!

I trust that you'll sort out the points I've raised.

Thanks for your package.