Current File : /home/tojsat/public_html/ydx/book/vendor/zetacomponents/document/review-1.1.txt
Review Alexandru 05-11-2008
---------------------------
[x] Wrong headers in some files ("File containing the XXX class", "Base class...").
[x] Wrong package names (Table, PageBreak, etc) (see issue #13829).
[ ] rst2xhtml conversion does not work for all ezc tutorials (some
elements are not supported, like replace).
kn: Those are just not implemented yet. But the errors are ignorable now.
[ ] Linux-style path separator used in some files.
kn: PHP should handle this just fine.
[ ] Some functions which return bool are documented with '@return void'.
- Some typos in the code and comments:
[x] /src/exceptions/errnous_xml.php
Errnous -> Erroneous
[x] /src/document/xml/xhtml/filter/element/base.php
boolena -> bool
Review Toby 2008-11-21
----------------------
[x] Missing tutorial for new formats.
[x] Wierd @apichange tag in Document/src/options/document_rst.php.
[x] I'd suggest to have a wrapper class for each Wiki format, to allow
creation of Wiki documents in the same way as for other documents. For
example ezcDocumentWikiCreole to create a wiki document with the creole
tokenizer.
[x] It sounds like a good idea to register a default directive for all
directives mentioned in the RST specification. Otherwise people get an
error, if the try to convert documents containing such directives.
kn: As discussed: Changed to an error of level "Warning", which can
easily be ignored.
[x] The docanalysis tool throws an exception because a class used in a type
hint does not exist.
Mini Shell