review-1.2.txt 3.5 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596
  1. Parsing Issues with RST:
  2. 1. in docs/install.txt, it doesn't like
  3. spl_autoload_register_
  4. on line 285. Replacing it with:
  5. `spl_autoload_register_`
  6. makes it work.
  7. The error message is:
  8. PHP Fatal error: Uncaught exception 'ezcDocumentVisitException' with
  9. message 'Visitor error: Warning: 'Missing reference target 'register'.' in
  10. line 285 at position 22.' in
  11. /home/derick/dev/ezcomponents/trunk/Document/src/document/rst/visitor.php:167
  12. kn: Fixed in revision 10453.
  13. 2. in docs/dev_process.txt it throws:
  14. PHP Fatal error: Uncaught exception 'ezcDocumentVisitException' with
  15. message 'Visitor error: Warning: 'Missing reference target 'register'.' in
  16. line 285 at position 22.' in
  17. /home/derick/dev/ezcomponents/trunk/Document/src/document/rst/visitor.php:167
  18. It seems that it doesn't see the:
  19. 1. _`Present the Idea`
  20. on line 37 as a reference target.
  21. kn: Fixed.
  22. 3. docs/articles/2009-01-06-webdav-locking.txt dies with:
  23. PHP Fatal error: Uncaught exception 'ezcDocumentVisitException' with
  24. message 'Visitor error: Notice: 'Unhandeled field list type
  25. 'revision'.' in line 5 at position 1.' in
  26. /home/derick/dev/ezcomponents/trunk/Document/src/document/rst/visitor.php:167
  27. and:
  28. PHP Fatal error: Uncaught exception 'ezcDocumentVisitException' with
  29. message 'Visitor error: Notice: 'Unhandeled field list type 'status'.'
  30. in line 6 at position 1.' in
  31. /home/derick/dev/ezcomponents/trunk/Document/src/document/rst/visitor.php:167
  32. perhaps we can do something with it, as it prevents quite a few things from
  33. parsing. (Also 2007-09-25-workflow-thesis.txt)
  34. kn: You can just reduce the error reporting level for such cases, so that
  35. notices won't be converted into exception anymore::
  36. $rstDocument->options->errorReporting = E_PARSE | E_ERROR | E_WARNING;
  37. PDF generation issues
  38. 1. After fixing 1. under RST, the generator outputs:
  39. Unknown: http://docbook.org/ns/docbook:itemizedlist
  40. Unknown: http://docbook.org/ns/docbook:literallayout
  41. ...
  42. This is send to stdout, which makes it appear in the PDF upon redirection.
  43. I understand that itemizedlist is not understood, but what is wrong with
  44. literallayout?
  45. kn: Implements the common error reporting interface now, so that errors aer
  46. either converted into exceptions, or can be requested later using
  47. getErrors().
  48. 2. While rendering 2009-01-06-webdav-locking.txt there are issues with image
  49. inclusion. First of all, the file paths in RST are where the images can be
  50. found *after* the files have been converted. But with PDF generation the
  51. files have to be the current directory. Perhaps we should add an option to
  52. "rebase" or "map" paths for image inclusion?
  53. kn: The original document location has not been maintained when converting
  54. between multiple document formats, so that the images could not be located
  55. anymore. Fixed.
  56. 3. While rendering 2009-01-06-webdav-locking.txt and fixing 2, I see that the
  57. image is included on page 2, but no text is below the image. A similar
  58. issue happens when we render 2007-12-20-tree-yui.txt on page 5.
  59. kn: The alt text is not used as a caption for an image. Using a figure you
  60. can also specify a caption for an image., like::
  61. .. figure:: image.png
  62. My caption goes here.
  63. Which works, with the mentioned article.
  64. 4. Rendering 2006-01-16-image-manipulation.txt creates a double line of text
  65. under the image on page 5.
  66. kn: Fixed in revision 10448.
  67. 5. There are still some test failures: http://ezc.pastebin.com/m9c77063