Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.7.2

Bug #527 shorten the file tag attributes
Submitted: 2004-01-06 14:18 UTC
From: smith at backendmedia dot com Assigned: cellog
Status: Closed Package: PEAR_PackageFileManager
PHP Version: Irrelevant OS: Irrelevant
Roadmaps: (Not assigned)    
Subscription  


 [2004-01-06 14:18 UTC] smith at backendmedia dot com
Description: ------------ As far as I know you dont have to define the basedir repeatedly. Furthermore it would be a good idea to make use of the <dir> tag to shorten the file location this would improve readablity

Comments

 [2004-01-07 01:26 UTC] cellog
if you use a dir tag you don't need the baseinstalldir, but the file tag outside a dir tag does, <file> inherits from <dir>
 [2004-01-07 02:00 UTC] cellog
the Problem is that PEAR_Common::xmlFromInfo(), which is used to generate the actual XML, can only output a single flat array. This stems from the fact that PEAR_Common::infoFrom*() methods flatten any <dir> tags into top-level <file> tags. Fortunately, I can step around this by extending PEAR_Common, as my dirtag generator has to flatten the file tags, they are parsed recursively from the directory structure to begin with. Good thing I added an option to specify the PEAR_Common class to use :). Gotta love forward compatibility
 [2004-01-07 03:10 UTC] cellog
This bug has been fixed in CVS. In case this was a documentation problem, the fix will show up at the end of next Sunday (CET) on pear.php.net. In case this was a pear.php.net website problem, the change will show up on the website in short time. Thank you for the report, and for helping us make PEAR better. Hi there, I just implemented this. It also adds a nice touch that I found unbearably annoying in phpDocumentor's package.xml: a closing </dir> tag has a comment following it with the full path of the directory it is closing. This way, one can easily navigate even the ending of the <dir> tags for complex dir structures. ok guys, knock yourself out. I won't be unit testing this feature for a bit due to other obligations, but when it is tested, I will do a new release. Feel free to use CVS, it is stable by my testing. Greg