- Issue created by @mxh
- 🇩🇪Germany jurgenhaas Gottmadingen
That's an interesting one. I wonder if the token API has any documentation about the keys being lowercase, of if we just assumed that would be what's happening? Wasn't able to find any such documentation, and assume that we've been too cautious here. Mixed case token keys should be perfectly OK. So, I tend to lean towards option 2, combined with a warning in the release notes that this may break existing models.
- 🇩🇪Germany mxh Offenburg
So, I tend to lean towards option 2, combined with a warning in the release notes that this may break existing models.
Agreed
- 🇩🇪Germany jurgenhaas Gottmadingen
MR is ready for review. I've also changed the issue title so that the breaking change will be visible in the release notes.
- Merge request !509Issue #3519647 by mxh, jurgenhaas: Breaking Change: token names and properties case-sensitive → (Merged) created by jurgenhaas
- 🇩🇪Germany mxh Offenburg
Thanks. While the code change looks exactly as described, there is a failing test which may require some attention: https://git.drupalcode.org/project/eca/-/jobs/5327402
Setting to NW due to the failed test.
- 🇩🇪Germany jurgenhaas Gottmadingen
Yeah, just saw that and fixed the existing test as well as appended a new test, so that we can test both cases: if the case of the token is equal and if it's not.
-
jurgenhaas →
committed 56790933 on 3.0.x
Issue #3519647 by jurgenhaas, mxh: Breaking Change: token names and...
-
jurgenhaas →
committed 56790933 on 3.0.x
- 🇩🇪Germany jurgenhaas Gottmadingen
Merged. I guess this shouldn't be back ported, though.