- 🇹🇭Thailand AlfTheCat
Patch from #43 applies but it seems only to work for node flags. I'm trying to use a token in a flag link on a custom ECK entity and it's not working.
- First commit to issue fork.
- 🇬🇷Greece vensires
Created MR!72 based on #44 🐛 Re-implement hook_tokens() Needs work without any other change. Useful for the automated testing overview.
- 🇬🇷Greece vensires
I am almost totally sure I didn't do the rebase correctly. Will retry later.
- 🇬🇷Greece vensires
Seems hook_tokens() was completely removed by commit 40257ad528474fa355cc91005f6fafd31a5435d7 in order to fix 📌 All coding standard fixes Active .
I will add it back again but we will have to test whether in latest 8.x.4.x versions the tokens have been address in another more dynamic way. - 🇬🇷Greece vensires
I have fixed the MR and reimplemented the changes manually to be as sure as I can that I got it all correctly. Please validate and try thoroughly!
I also take the opportunity and remove the "Needs tests" tag since there are various comments that tests are fixed etc previously in this issue. The current tests have some @todo comment which someone might like to consider and maybe add the "Needs tests" tag back again.
- First commit to issue fork.