Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
The langcode in config/optional
changes during import, because the config it stored in the sites default langcode. I use German as default language. This leads to duplicate field types on schema export.
Proposed resolution
Add a dedicated language property to the config entity.
Remaining tasks
User interface changes
API changes
Data model changes
Comment | File | Size | Author |
---|---|---|---|
#8 | 2686575-8.patch | 11.41 KB | mkalkbrenner |
#2 | 2686575-1.patch | 6.75 KB | webflo |
Comments
Comment #2
webflo CreditAttribution: webflo at UEBERBIT GmbH commentedComment #3
mkalkbrennerI can't reproduce the issue the way you described it. But in my setup something else happened.
I added German as second language. Then I set German as default language. After that I added Dutch.
But instead of getting a duplicate like you described I got an Exception:
Drupal\Core\Entity\EntityStorageException: 'solr_field_type' entity with ID 'm_text_nl_nl_5_2_0' already exists. in Drupal\Core\Entity\EntityStorageBase->doPreSave() (line 430 of /Users/mkalkbrenner/git/drupal/core/lib/Drupal/Core/Entity/EntityStorageBase.php).
So there's definitely an issue. But the exception I faced won't be solved by your patch.
Did you upgrade a previous installation with the code from dev? Or did you do a fresh installation?
Comment #4
webflo CreditAttribution: webflo at UEBERBIT GmbH commentedSteps to reproduce
Comment #5
mkalkbrennerI'll try that.
BTW I did a fresh install and this time my steps from #3 didn't lead to an exception.
Comment #6
mkalkbrennerTogether with webflo I was able to reproduce the issue. It occurs if you select a language different than English in the first step of the installer. The issue does not occur if you change the default language later!
Comment #7
mkalkbrennerComment #8
mkalkbrennerI rerolled the patch against the latest code base and fixed the filed type list page.
Comment #10
mkalkbrenner