- Issue created by @aurelianzaha
- Open on Drupal.org →Core: 10.2.x + Environment: PHP 8.2 & MySQL 8last update
9 months ago Not currently mergeable. - 🇦🇹Austria aurelianzaha
for some reason, I was not able to push the changes to the fork, attached the patch
- Status changed to Needs review
9 months ago 4:03pm 12 February 2024 - last update
9 months ago 31 pass - Status changed to Needs work
9 months ago 9:50am 20 February 2024 - 🇩🇪Germany mkalkbrenner 🇩🇪
The comment might be wrong. We must also react if a language gets deleted.
- Status changed to Needs review
7 months ago 11:16am 10 April 2024 - last update
7 months ago 31 pass - Status changed to Needs work
7 months ago 12:55pm 10 April 2024 - 🇩🇪Germany mkalkbrenner 🇩🇪
+++ b/src/EventSubscriber/ConfigSubscriber.php @@ -65,7 +65,9 @@ class ConfigSubscriber implements EventSubscriberInterface { + $saved_config->isNew())The check doesn't work.
isNew() never returns true in the event subscriber even if the config is "new". -
mkalkbrenner →
committed 080e896c on 4.x
Issue #3420882 by aurelianzaha, mkalkbrenner: Do not reindex language...
-
mkalkbrenner →
committed 080e896c on 4.x
-
mkalkbrenner →
committed 1b8e478d on 4.x
Issue #3420882 by aurelianzaha, mkalkbrenner: Do not reindex language...
-
mkalkbrenner →
committed 1b8e478d on 4.x
- 🇬🇧United Kingdom rossb89 Bristol
Looks like this was included in 4.3.3, can this issue be marked as Fixed now?
It looks like it was committed without the `$saved_config->isNew()` line, so is this all feature complete now?
- Status changed to Fixed
6 months ago 6:06pm 20 May 2024 Automatically closed - issue fixed for 2 weeks with no activity.