Hi. I know this is marked as fixed and contributions are preferred as a fork+MR. However, I'm having these two errors below and I used part of this issue's solution to fix it, so I thought I might as well share the patch in case is useful to anyone.
I'm using Ctools version 7.x-1.19 with PHP 8.1 and these are the deprecated msgs I'm getting.
Deprecated: Optional parameter $data declared before required parameter $plugin is implicitly treated as a required parameter in /var/www/html/docroot/sites/all/modules/contrib/ctools/plugins/contexts/entity.inc on line 58
Deprecated: Optional parameter $conf declared before required parameter $plugin is implicitly treated as a required parameter in /var/www/html/docroot/sites/all/modules/contrib/ctools/plugins/contexts/entity.inc on line 58I attach a simple patch based in https://git.drupalcode.org/project/ctools/-/merge_requests/9 to fix it.
Here's a patch for PHP 8.1 compatibility fix for Autoload module.
beagaliana → created an issue.
Added patch for PHP 7.0 compatibility fix for Views data export - JSON support module.
beagaliana → created an issue.
Created patch for this error for version 7.x-1.14 for HTTP Parallel Request & Threading Library module. I'll attach it here in case it might come in handy.
It basically solves the deprecation in the same way, also adding parenthesis to solve the ambiguity. However uses the parenthesis in the latter part as I believed it made more sense for the logic of the condition.
Added patch for PHP 7.0 deprecation fix for Hook Update Deploy Tools module.
beagaliana → created an issue. See original summary → .
Hi @jose-reyero, thank you so much for your feedback! I updated the MR. Now the routes' existence is check using form validation, I also added a simple functional test to it.
beagaliana → made their first commit to this issue’s fork.