Update composer.json

[wandrewchan]

remove abandoned zend-escaper, replace with laminas

[stephanvierkant]

You also changed some other files. Can you explain those changes?

[internalsystemerror]

the latest version of laminas/laminas-escaper requires PHP >= 5.6

[stephanvierkant]

the latest version of laminas/laminas-escaper requires PHP >= 5.6

What's the point?

[internalsystemerror]

U asked what the other changes were for. As far as I can see, those changes are removing the support for PHP < 5.6.

[stephanvierkant]

I mean files like src/PhpWord/Escaper/AbstractEscaper.php

[liborm85]

Why is upgraded minimal php version? This is BC break. You can use "laminas/laminas-escaper": "^2.2",.

And PR contains other irrelevant changes in AbstractEscaper.php.
PR is wrong.

[stephanvierkant]

Yes, in theory this might be a BC break. But it has been upgraded to a PHP version that hasn't got support for years now. I don't consider removing support for EOL-versions a BC-break.

But yeah, it has other irrelevant changes and that is a problem. @wandrewchan can you please change it?

[wandrewchan]

Actually I was trying to replace with laminas as what @tacman, but it was failed. And I found that zend-escaper has deprecated, but it still works. So this merge is discard and closed.

[liborm85]

I'm adding compatibility with PHP 7.4 and PHP 8.0 in PR #1946, was necessary fix travis and scrutinizer. So also migrate to Laminas with compatibility PHP 5.3+, commit: f2516b0

[stephanvierkant]

That would be (at least) the third attempt to replace zend by laminas. And again, that PR contains other changes as well.

[liborm85]

Without next changes it is not possible to test whether it works in travis.

[tacman]

My PR was a one-line fix, and it gets rid of the warning that shows up every time. Was there a problem with that PR? Yes, at the moment the deprecated version still works, but there's no reason to keep it.

PHP 5 is dead, dead, dead. Anyone using it should upgrade. That's a different PR, of course.