- Issue created by @the_g_bomb
- πΊπ¦Ukraine i-trokhanenko Lutsk πΊπ¦
+1
I have the same issue with PHP 8 - @the_g_bomb opened merge request.
- π¬π§United Kingdom the_g_bomb
The above MR includes fixes that will solve π Error: Required parameter $context follows optional parameter $format RTBC
- Assigned to i-trokhanenko
- Status changed to Needs review
almost 2 years ago 3:57pm 9 February 2023 - Issue was unassigned.
- Status changed to RTBC
almost 2 years ago 4:16pm 9 February 2023 - πΊπ¦Ukraine i-trokhanenko Lutsk πΊπ¦
The MR-4 works well for me, tested with PHP 8.0.26 and Drupal 7.94. Making RTBC, please commit.
Thanks Gareth for the quick fix! - π¨π¦Canada joelpittet Vancouver
The MR is against the 1.0.x branch it should be on a 7.x-2.x-dev branch that doesn't exist. Not sure what to do but bump the version here.
Sorry, I think I sound grumpy... I'll go drink some coffee :D
The merge request fixes the fatal due to the require method called twice by drush when trying to enable the module so bumping the priority.
PHP Fatal error: Cannot redeclare site_migration_report_drush_command() (previously declared in /var/www/html/public/sites/all/modules/dev/upgrade_audit/site_migration_report.drush.inc:25) in /var/www/html/public/sites/all/modules/dev/upgrade_audit/site_migration_report.drush.inc on line 25