Update Requirments in the Readme

[hskrtich]

Please update the requirement in the ReadMe to include:

PHP Extension ZipArchive
PHP Extension libxml

[ivanlanin]

I've put the requirement for ZipArchive in the README. Where do we use libxml? CMIIW, I think we're using DOMDocument and SimpleXML.

[hskrtich]

Looking at the code, it uses XMLWriter and XMLWriter requires libxml. So instead of libxml we should list XMLWriter. Also it would be a good idea to update composer.json with these requirements.

[ivanlanin]

Yup. You're right. So we should move ext-zip from recommend to require in composer.json?

[hskrtich]

Yes until someone (maybe me) adds PCL ZIP support.

[ivanlanin]

PHPWord users will like it when you can't sleep ;)

[Progi1984]

@ivanlanin @bskrtich Could you send a pull request for the readme.MD and the composer.json ? Thanks :)

[Progi1984]

I think it is rather recommandations rather than requirements. Why ? Imagine a developer which only want to generate RTF Files and no ODT/DOCX files.

[hskrtich]

For both zip & xml, correct?

[Progi1984]

Yes... What do you think about that ?

[hskrtich]

Sounds reasonable to me. I will make the change now.

[ivanlanin]

Yes, we don't need xml and zip for RTF. But, why would anyone use PHPWord (with the Word) if they want to create a RTF file? Because they can't find PHPRtfLite? :) Btw, we can use some of their codes.