Skip to content

Comments

Master upstream#3

Open
graham73may wants to merge 2 commits intodjboris88:masterfrom
graham73may:master-upstream
Open

Master upstream#3
graham73may wants to merge 2 commits intodjboris88:masterfrom
graham73may:master-upstream

Conversation

@graham73may
Copy link

Relates to issue: #2

Deprecation warning:

PHP Deprecated: Since twig/twig 3.9: Twig node "Djboris88\Twig\Node\CommentedIncludeNode" is not marked as ready for using "yield" instead of "echo"; please make it ready and then flag it with the #[YieldReady] attribute. in /var/www/html/vendor/symfony/deprecation-contracts/function.php on line 25
PHP Deprecated: Since twig/twig 3.9: Twig node "Djboris88\Twig\Node\CommentedIncludeNode" is not marked as ready for using "yield" instead of "echo"; please make it ready and then flag it with the #[YieldReady] attribute. in /var/www/html/vendor/symfony/deprecation-contracts/function.php on line 25

Changes:

  1. echo replaced with yield to resolve the deprecation warning.
  2. YieldReady attribute added to the class as recommended here: https://github.com/twigphp/Twig/blob/3.x/CHANGELOG#L109

PHP Attributes are a PHP 8 feature, I'm not sure how that factors in to this PR. I know you use WordPress Timber, the requirements for that are PHP 7.4 >=.

Timber 2.0 requires you to use a PHP version >= 7.4.

PHP Deprecated: Since twig/twig 3.9: Twig node "Djboris88\Twig\Node\CommentedIncludeNode" is not marked as ready for using "yield" instead of "echo";
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant