- Merge request !1557Issue #3254575: Entities tokens might cause problems when field prefix is empty β (Open) created by MacSim
- πΊπΈUnited States smustgrave
Can the MR be updated for 10.1 please
Also left a few comments on the existing MR.
Will need a change record to announce the new parameter.
Thanks
- First commit to issue fork.
- π¦πΊAustralia acbramley
Added some minor comments on the MR, but this needs a full reroll on 11.x. Neither rebasing or merging locally worked well.
Also, I'm not sure that just checking for an empty field prefix is enough here. I know this is pedantic but what if someone had a field prefix of "sum" and then made an "mary" field, that'd result in the same bug.
Finally, the error message "The machine-readable name is already in use" is not accurate as the conflict is actually in the token system, not a field's machine name.
Also swapping the component here as this is not node specific.
- π«π·France MacSim
Updated MR to reflect what #14 said.
But since token service is not part of drupal core, tests can't succeed.
This could potentially move to Token issues. Anyway nothing would happen without the module being enabled.