tmgmt_local_install() should check \Drupal::isConfigSyncing() before changing the config

Issue created by @leksat
Comment 10 months ago →
🇮🇹Italy apaderno Brescia, 🇮🇹
hook_install() implementations get a $is_syncing parameter. It is sufficient to check its value, like forum_install() does.
Merge request !61Issue #3436093: tmgmt_local_install() should check the syncing state → (Open) created by apaderno
Open in Jenkins → Open on Drupal.org →
Core: 9.5.x + Environment: PHP 7.3 & MySQL 8
last update 10 months ago
118 pass
Status changed to Needs review 10 months ago8:59pm 26 March 2024
Comment 10 months ago →
🇮🇹Italy apaderno Brescia, 🇮🇹
Pipeline finished with Failed
10 months ago
Total: 391s
#129968
Status changed to RTBC 10 months ago8:42am 27 March 2024
Comment 10 months ago →
leksat
@apaderno Even better 🎉 Thanks!

The MR looks good and solves the issue ✅
Open in Jenkins → Open on Drupal.org →
Core: 9.5.x + Environment: PHP 7.3 & MySQL 8
last update 9 months ago
116 pass, 2 fail
Comment 9 months ago →
plov
Moved changes to the patch
Comment 9 months ago →
System Message
The last submitted patch, 6: 3436093-6-tmgmtlocalinstall-should-check.patch, failed testing. View results →
Open in Jenkins → Open on Drupal.org →
Core: 9.5.x + Environment: PHP 7.3 & MySQL 8
last update 9 months ago
116 pass, 2 fail

Merge Requests