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.

[Progi1984]

@ivanlanin I agree with you but in the future, I hope we support DOC file format which no needs XML & ZIP, like HTML, WPS or PDF. I think that the requirement for a writer/reader must be precised in the documentation

[ghost]

But, why would anyone use PHPWord (with the Word) if they want to create a RTF file? Because they can't find PHPRtfLite?

Why not, @ivanlanin?

[ivanlanin]

Haha. Alright. I think we will have to make our RTF Writer better, then.

[Progi1984]

@ivanlanin A good objective. Don't hesitate to create issues with missing features.

[Progi1984]

@ivanlanin @RomanSyroeshko @bskrtich
Finally, what do you think about that : requirements ? recommandations + documentation ?

[ivanlanin]

I admit that there's a possibility that people will use PHPWord only for creating RTFs, but I still think that most people will use PHPWord to create Word documents. RTF creation is an extra feature for us. So, I support @bskrtich pull request #121

[ghost]

On the one hand, I agree that "ext-zip" and "ext-xml" should be optional in common case. Our documentation should be clear enough about usage of the extensions. Users should understand what the extensions are used for, and when they should be enabled.

On the other hand, at the current moment PHPWord works witn OOXML and ODT. These formats require ZIP and XML extensions.

So, I suggest consider that XML and ZIP extensions are required for now, because it's true. I suggest to return to the discussion when we support some format which doesn't require XML or ZIP extension.

By the way, we forgot two things.

1. "ext-xmlwriter" should be optional.
2. "ext-xsl" should be optional.

P.S.
As regards supporting DOC, I'm not sure it's a good idea. If somebody really needs it, lets discuss in a separate thread.

[Progi1984]

@RomanSyroeshko @ivanlanin @bskrtich
After seen your answers and search in Composer doc, I give an another idea : use the suggest key in composer.json (https://github.com/composer/composer/blob/master/composer.json#L35).

So :

• ext-xml & ext-zip in require
• ext-xmlwriter & ext-xsl in suggest

What do you think about that ?

[ghost]

Good idea. By the way, where did we get "ext-xml"? It seems PHP doesn't have such extension http:https://php.net/manual/en/extensions.membership.php.

[Progi1984]

@RomanSyroeshko The package is php-xml. The doc is here : http:https://php.net/manual/en/xml.installation.php

[ivanlanin]

Agree. Wrap it up and on to 0.8! :)

[ghost]

@Progi1984, OK.

@ivanlanin, 0.7.2 first. :)

[Progi1984]

@bskrtich Could you update your pull request, please ?

[ivanlanin]

@bskrtich Wake up and update your pull request, dude. We're going to release 0.8 :)

[hskrtich]

@ivanlanin Yes sir! lol I will get the push updated in the next few min.

[hskrtich]

Ok a new push is up. I think i did every thing right

[ivanlanin]

Thanks! 👍

[Progi1984]

Fixed in develop branch.