search

backdrop-contrib / coder_upgrade

Helps automate some/most of the work required to upgrade a module from Drupal to Backdrop 1.x
GNU General Public License v2.0
4 stars 7 forks source link

Broken Page | admin/config/development/coder-upgrade/settings #80

Closed rayjamesdev closed 1 year ago

rayjamesdev commented 1 year ago

After installing the module, on the /admin/modules page, when I click on CONFIGURE for that module, I get the following error the page located at admin/config/development/coder-upgrade/settings: Error set_exception_handler(): Argument #1 ($callback) must be a valid callback or null, class PGPParser does not have a method "exception_handler"

I have included screenshots as well. Thanks error_when_clicked settings_error

bugfolder commented 1 year ago

Hmm, this looks a lot like what was reported here:

https://github.com/backdrop-contrib/coder_upgrade/issues/78#issuecomment-1355166329

Can you check the watchdog log at path admin/reports/dblog, which should also give the fine and line number where that error is happening?

rayjamesdev commented 1 year ago

More info:

Fresh install of : Backdrop 1.24.0 Windows 11 Xampp version 8.1.12 PHP Version: 8.1.12 MySQL Version: 8.0.32 - MySQL Community Server - GPL

WATCHDOG RESULTS: TypeError: set_exception_handler(): Argument #1 ($callback) must be a valid callback or null, class PGPParser does not have a method "exception_handler" in set_exception_handler() (line 107 of D:\xampp8.1.12\htdocs\mysite\site\modules\Contrib\coder_upgrade\grammar_parser\parser.inc).

Thanks for your time. I'm also not able to convert a module, but I will post that issue separately. thx

bugfolder commented 1 year ago

Some testing confirms that this error happens reproducibly on PHP 8.1 and not on PHP 7.4 or less. Also happens on PHP 8.0.

docwilmot commented 1 year ago

@rayjamesdev could you please test that the latest code fixes this issue?

rayjamesdev commented 1 year ago

Hi, my apologies. I trashed my entire 8.1 setup and am using 7.2 for development. I no longer have the means of testing/using 8.1

bugfolder commented 1 year ago

OK. Closing, since it appears the fix for https://github.com/backdrop-contrib/coder_upgrade/issues/78 addressed the problem.