I installed the module and noticed there was an update available. When I ran `drush updatedb`, I got the following error:
> [notice] Update started: simple_sitemap_update_8203
> [error] Argument 1 passed to Drupal\Core\Config\Config::setData() must be of the type array, null given, called in /srv/www/site/docroot/modules/contrib/simple_sitemap/simple_sitemap.inst
all on line 298
> [error] Update failed: simple_sitemap_update_8203
The error is caused by saving `$old_config->get('settings');` when there are currently no settings yet.
Testing this locally, setting an if statement around the block checking first if `$old_config->get('settings');` exists, it resolves the issue.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | simple_sitemap-fix-update-8203-3177939-3--real.patch | 679 bytes | redbrickone |
Comments
Comment #2
redbrickone commentedI resolved the issue adding an if statement around the old settings. Attaching my patch here.
Comment #3
redbrickone commentedAdjusted code to be a little quicker.
Comment #4
redbrickone commentedComment #5
chrissnyderComment #6
gbyteWow community has been very quick with reviewing this one. ;)
Please always open up bug reports against the dev version.
Comment #7
gbyteThanks for the patch, however I just installed 2.1 and without doing anything else upgraded to 3.x without a hitch. If you were missing that data it probably means your installation went wrong in the first place, as the settings array is part of the configuration that gets installed on module install. Please reopen this issue if you manage to reproduce it.