Add PHP 7.2 Support for count()

[tobiasgraeber]

Info: Generating css on PHP 7. 2 does not work due warning. Check compiled css on PHP7.2 (doesnt work).

[kaystrobach]

Can you please add the warnings here as logfile? Thanknyou.

[tobiasgraeber]

Problem is this change here in PHP 7.2 i think: https://www.php.net/manual/de/migration72.incompatible.php#migration72.incompatible.warn-on-non-countable-types

Sorry i cannot provide the log at the moment as i wont get back to the machine. Just came here to inform you. It would be the best you check your code / extension on PHP 7.2 regarding this. Should be visible instantly looking in the generated .css urls from source after cleaning caches and reloading frontend on PHP 7.2. with i think. There are multiple occurrences of this in several files.

[kaystrobach]

Thank you.

[Mairex]

Core: Error handler (FE): PHP Warning: count(): Parameter must be an array or an object that implements Countable in /home/www/website/typo3conf/ext/dyncss_less/Resources/Private/Php/less.php/Visitor/toCSS.php line 113

from $nodeRuleCnt = count($rulesetNode->rules);

to $nodeRuleCnt = ($rulesetNode->rules === null ? 0 : count($rulesetNode->rules));

I'm not a coder, but I have tried to fix the problem. At the moment it seems to work. ;)

[kaystrobach]

will check if updating the less parser works easily.

[kaystrobach]

still in the pipeline, we are working on automating the release process, incl. updating dependencies for ter

[tbal]

Solution on the problem we are using in production:

-               $nodeRuleCnt = count($rulesetNode->rules);
+               $nodeRuleCnt = is_array($rulesetNode->rules) ? count($rulesetNode->rules) : 0;`