Make sure unserialized is array

[Cruiser13]

Changes in this pull request

Unserialized may be false sometimes which will cause the following error when running the migration:

[error] Migration Pimcore\Bundle\CoreBundle\Migrations\Version20230424084415 failed during Execution. Error: "array_key_exists(): Argument #2 ($array) must be of type array, bool given"
17:29:18 CRITICAL [console] Error thrown while running command "doctrine:migrations:migrate". Message: "array_key_exists(): Argument #2 ($array) must be of type array, bool given" ["exception" => TypeError { …},"command" => "doctrine:migrations:migrate","message" => "array_key_exists(): Argument #2 ($array) must be of type array, bool given"]
In Version20230424084415.php line 40:
array_key_exists(): Argument #2 ($array) must be of type array, bool given

Make sure that unserialized is an array and skip otherwise or Pimcore 10 > 11 updates might fail.

Additional info

WHAT

🤖 Generated by Copilot at 9f802ac

Fix migration of document editables from Pimcore 5 to Pimcore 10. Add array check to unserialize result in Version20230424084415.php.

🤖 Generated by Copilot at 9f802ac

unserialize /
check if array or not /
fall migration fix

HOW

🤖 Generated by Copilot at 9f802ac

• Add a check for array type before accessing keys of $unserialized variable in Version20230424084415.php (link)

[github-actions]

Review Checklist

• Target branch (11.0 for bug fixes, others 11.x)
• Tests (if it's testable code, there should be a test for it - get help)
• Docs (every functionality needs to be documented, see here)
• Migration incl. install.sql (e.g. if the database schema changes, ...)
• Upgrade notes (deprecations, important information, migration hints, ...)
• Label
• Milestone

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
1.1% Duplication

[dvesh3]

@Cruiser13 thanks!