[PEPr] +1 for File Formats::File_Sitemap

Till Klampaeckel (http://pear.php.net/user/till) has voted +1 on the proposal for File Formats::File_Sitemap. Proposal information: http://pear.php.net/pepr/pepr-proposal-show.php?id=555 Vote information: http://pear.php.net/pepr/pepr-vote-show.php?id=555&handle=till This vote is conditional. The condition is: I think this is a very useful package, I can see myself using it pretty soon. :) Thanks for proposing it and putting so much work into it. Here are some suggestions: * Some of the conditionals (if/else) could be improved/simplyfied. * DOM creation should be checked (imho). * DOM should be added to the required extensions (in package.xml). * Loading HTTP_Request should be double checked too, with just an include_once it will fail silently and make the code fatal later on. (* Is HTTP_Request really just optional?) * I'd suggest that you use single quotes instead of double quotes where possible. * There is an empty catch{} block in File_Sitemap_Index::add(), is that on purpose? * You could make use of the @package_version@ which is defined in package.xml. * zlib should be added to the required extensions (in package.xml). * Please move all global constants (defined in Sitemap/Exception.php) into class constants. * I'd replace is_null() with !== null, it's less expensive, since it saves a function call. * cfArray in Sitemap::parseChangefreq() could be a static -- Sent by PEPr, the automatic proposal system at http://pear.php.net