Closed eduardomozart closed 1 year ago
Is this fine to merge for you? Did you also test with other images than svg?
Is this fine to merge for you? Did you also test with other images than svg?
Yes. SVG embedding is still broken, but it didn't seem to be related to these fixes. It seems that the function _addStringAsSVGImage
is broken. I will create a bug report and attempt to fix it when I have time. But it's working as expected with other image files.
The code was updated to fetch remote images using DokuWiki HTTPClient. The
simplexml_load_file
allows you to specify a SVG file path or load it's XML content inline, sofile_exists
verification was breaking the_addStringAsSVGImage()
public function which callsgetImageSize
to get inline SVG image size. This commit also returns width and height0
if size can't be calculated asgetimagesize
does.This commit also disable PHP timeouts issue with larger wikis when exporting multiple pages to ODT with Bookcreator (this piece of code was based on Dw2Pdf code).
This commit also includes small fixes for PHP 8 Warning for non-defined variables error and the rendering of Wrap plugin on ordered/unordered lists (#297).