Bump to travis php version

[michaelachrisco]

Small code coverage on section footers.

[michaelachrisco]

Looks like String is a protected word in php 7.0. Ill add in a patch (String -> CommonString).

[michaelachrisco]

Two bugs:

• Methods with the same name as their class will not be constructors in a future version of PHP; PclZip has a deprecated constructor
• Methods with the same name as their class will not be constructors in a future version of PHP; mPDF has a deprecated constructor

Looks like either these packages need to be updated or fixed of PHP 7 errors. Down the rabbit hole I go.

[Haringat]

You should revert 72927e3.
The package tecnick.com/tcpdf is abandoned and replaced by tecnickcom/tcpdf so the former dependency list was correct.

[michaelachrisco]

@Haringat Ill take a stab at this later today. I have a feeling that it wont be as simple as requiring the correct library and off we go. I have no idea if the two libraries can work with PHP 7.0. tecnickcom/tcpdf does not have a travis build by the look of it

[michaelachrisco]

Small update: I believe I am going to let 7.0 fail at the moment (unless someone else has a better option?), and make the test pass. I'll go down the rabbit hole on some of the dependencies to fix the 7.0 issue on their side and then push to this branch.

[ghost]

Partially duplicate of #658.

As regards coverage. I made couple experiments and found that @codeCoverageIgnore annotation sometimes has no effect. I have no idea why.

I think we should consider upgrading PHPUnit in the future. We use very old one, and it may have some bugs. Thanks for pointing this out.

[michaelachrisco]

Thanks @RomanSyroeshko I was stuck for a while on some of the tests and why they were not passing. Going into the dependent repos was a rabbit hole. Glad to see someone working on 7.0 coverage.