- Issue created by @smulvih2
- 🇨🇦Canada SKAUGHT
patch attached.
-empties wxt_core_update_8501 and adds wxt_core_update_8502 to handle finding sites with legacy name and pulls any changes to that site if Users have alerted their own setups. - Status changed to Needs review
about 1 year ago 2:39pm 8 November 2023 - 🇨🇦Canada danrod Ottawa
This is working as expected, I ran the database updates (couldn't take screens), but I was able to see the config form and do a few changes in the countries' list, saw the changes in the webform:
- Status changed to RTBC
about 1 year ago 8:37pm 8 November 2023 - Status changed to Fixed
about 1 year ago 6:43pm 13 November 2023 - 🇨🇦Canada ionut_ottawa
I guess last change introduced another error when updatedb.
[error] Drupal\Core\Config\Config::setData(): Argument #1 ($data) must be of type array, Drupal\Core\Config\Config given, called in wxt_core/wxt_core.install on line 695
[error] Update failed: wxt_core_update_8502 - 🇨🇦Canada joseph.olstad
Yes I also noticed #10
wxt_core.install on line 695
- 🇨🇦Canada joseph.olstad
So in our situation, the wxt_core_countries.settings was manually deleted
wxt_core.settings.countries definately does existreviewing this, my patch isn't doing what it should but does prevent crashing in this case.
- 🇨🇦Canada joseph.olstad
So what I did was fix hook 8501
if for some reason it was missed, it gets run again in 8502 except with the expected path.
- 🇨🇦Canada joseph.olstad
see new issue
#3401668: Regression fix for Change wxt_core countries config name →same solution as patch #15
Automatically closed - issue fixed for 2 weeks with no activity.