I've committed the initial D6 port to HEAD (#222217: Port to 6.x), but I haven't yet tested any of the Token module integration. Let's use this issue for any discussion, patches, etc, related to that. Thanks.

Comments

geodaniel’s picture

geodaniel commented

Token substitution seems to be working as expected after some basic testing (inserting some tokens into the text).

dww’s picture

dww
we/he/they
commented

Seems to work fine in my testing with both broadcast and confirmation emails, too. Haven't tried reminder emails, since I haven't looked at #330821: Test/fix CCK date 6.x-2.* integration yet. ;)

However, the one thing I'm seeing is a PHP warning like this:

notice: Undefined index: token in /.../modules/cck/content.module on line 2160

On first glance, that doesn't seem to be the fault of anything signup module is doing wrong, and as I said, all the tokens are being replaced just fine...

dww’s picture

dww
we/he/they
commented
Status: Active » Fixed

Tested with reminder email, too. Everything's working with token integration. In IRC, greggles said that D6 token throws a lot of E_ALL warnings, so I'm not going to worry about that.

domesticat’s picture

domesticat commented

I've used the profile module to add extra fields for users, and tokens for those fields (in our case, %profile[profile_workphone] etc.) aren't being substituted. Not sure if I should reopen this issue for this problem, though.

dww’s picture

dww
we/he/they
commented

@domesticat: That's because user tokens aren't replaced at all, only node tokens. See #295542: Add user tokens to those able to be substituted in Signup emails.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.