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.
Attached patch extends the mergevars token support to allow more tokens, i.e. it adds in nested tokens of the next level.
Unrelated to that, it might make things simpler if the own "mergevars" api would be dropped in favor of just using tokens. That should suffice.
Even more unrelated, there are lots of whitespace problems in the code. Be sure to do not use tabs for indentation (see coding standards).
Comment | File | Size | Author |
---|---|---|---|
mailchimp_tokens.patch | 1.23 KB | fago | |
Comments
Comment #1
levelos CreditAttribution: levelos commentedThanks @fago, fixed in 04ae3843c8ce29d27259d9a9c824e1dc8c5d847c.
White space hiccups are taken care of as well.
Re: replacing the mergevars API with just tokens, happy to include that. Lets create as a standalone feature request and we'll put it in the queue.
Comment #2
fagoI noted you left a dpm() statement in the committed code:
Comment #3
fagosry, triple post
Comment #4
fagosry, triple post
Comment #5
levelos CreditAttribution: levelos commentedAlready removed in subsequent commits.