Add background color support for text box element for Word writer

[hazington]

Description

It is now possible to fill the background color of text box elements (only Word2007 writer!). There is no documentation for the Textbox element, so I can't add the changes.

Fixes # (issue)

No bugfix.

Checklist:

• I have run composer run-script check --timeout=0 and no errors were reported
• The new code is covered by unit tests (check build/coverage for coverage report)
• I have updated the documentation to describe the changes

[Progi1984]

Suggested change

	* @var string
	* @var string|null

[Progi1984]

Suggested change

	public function setBgColor($value = null): void
	public function setBgColor(string $value = null): self

[hazington]

Return $this in a setter method is not used in any other styling class' setter method. Doing this for only one class or even worser for only one class method is very inconsistent. I think this should be changed for all setters in version 1.1.x.

[Progi1984]

Suggested change

	* @param string $value
	* @param string $value
	* @return self

[Progi1984]

Suggested change

	public function getBgColor()
	public function getBgColor(): ?string

[Progi1984]

@hazington Some feedbacks to apply before merge

[Progi1984]

@hazington Do you think you can add unit tests ?

[hazington]

@Progi1984

Yes, I can cover this code with an unit test. By the way, this code is based on version 0.18.x and copied from another style element of PHPWord that already supported setting the background color. So this is not my coding style, it simply followed the old project's style.

I appreciate your feedback as I personally prefer coding in PHP 8.1, with strict_types=1 and based on PSR-12. By any chance, is refactoring based on PSR-12 and at least strong typed coding on the road map? That would make supporting PHPWord so much easier! :)

[hazington]

So, the requested unit tests have been added and the code style fixed according to the php-cs-fixer configuration by running the fix script from composer.json.

[hazington]

I've some trouble with the PHP 7.1 and PHP 7.2 coverage. Using "assertRegExp" will fail the CI / coverage and using the "assertMatchesRegularExpression" will let the coverage for PHP 7.1 and 7.2 fail. Any solution here? Since PHP 7.4 is officially EOL, dropping coverage for at least PHP 7.1, 7.2 and 7.3 should be considered.

679a738

[hazington]

This workaround seems to solve the CI coverage issue:

53d34fd

        // Try to address CI coverage issue for PHP 7.1 and 7.2 when using regex match assertions
        if (method_exists(static::class, 'assertRegExp')) {
            self::assertRegExp('/z\-index:\-[0-9]*/', $style);
        } else {
            self::assertMatchesRegularExpression('/z\-index:\-[0-9]*/', $style);
        }

[Progi1984]

Thanks @hazington for your contribution