I’m testing the upgrade from OJS 3.2.1.2 to 3.3.0.2. And the upgrade fails with the following error:
PHP Fatal error: Uncaught TypeError: Argument 1 passed to Illuminate\Database\Query\Builder::cleanBindings() must be of the type array, null given, called in \lib\pkp\lib\vendor\laravel\framework\src\Illuminate\Database\Query\Builder.php on line 919 and defined in \lib\pkp\lib\vendor\laravel\framework\src\Illuminate\Database\Query\Builder.php:3088
Stack trace:
#0 \lib\pkp\lib\vendor\laravel\framework\src\Illuminate\Database\Query\Builder.php(919): Illuminate\Database\Query\Builder->cleanBindings(NULL)
#1 \lib\pkp\lib\vendor\laravel\framework\src\Illuminate\Database\Query\Builder.php(946): Illuminate\Database\Query\Builder->whereIn('js.locale', NULL, 'and', true)
#2 \lib\pkp\classes\migration\upgrade\PKPv3_3_0UpgradeMigration.inc.php(191): Illuminate\Database\Query\Builder->whereNotIn('js.locale' in \lib\pkp\lib\vendor\laravel\framework\src\Illuminate\Database\Query\Builder.php on line 3088
Thanks for reporting! I think I’ve discovered the source of this. I filed an issue (#6710). Can you please look at the PR that is linked in the second comment there, apply the change, and let me know if that fixes the problem for you?
Hi @Serega_SWorld you may have confused the file:
OJSv3_3_0UpgradeMigration.inc at \classes\migration\upgrade
vs file:
PKPv3_3_0UpgradeMigration.inc at lib\pkp\classes\migration\upgrade
As far as I understand, the changes indicated in the post by respected @NateWr affect the 2nd file.
I’m new to this community and could be wrong.
I see you’ve already posted this elsewhere; please see the response on that thread. (It’s best not to post the same question in several places – it clutters the forum.)
Regards,
Alec Smecher
Public Knowledge Project Team