We would like to use [privatemsg_message:author:mail] in the "From e-mail address for notifications" field and doesn't work. If I write the token in the from address it appears literally in the from address of mails.

I'm aware this will reveal the sender's email and won't be a"private" message anymore but that's exactly the point. We would like privatemessage to be a reminder of the first contacts and we would prefer further communications to happen through e-mail.

Was this availabe in previous versions? (we are working with 7.x-2.x-dev).

I'm not sure if the category would be feature request or support request (beceause maybe I'm doing something wrong).


#4 private_message_1366118.patch1.22 KBchrisschaub
PASSED: [[SimpleTest]]: [MySQL] 3,125 pass(es).
[ View ]


javdich’s picture

@trevonio I think the option for this could be useful. Did you ever find out how to make this work?

Berdir’s picture

You just need to call token_replace() the same way as it's called for subject and boy, should be a one line patch.

chrisschaub’s picture

Category:feature» bug

Did this ever get checked in or fixed? Seems like a bug since tokens are allowed everywhere else? It should at least say not allowed if not.

chrisschaub’s picture

Status:Active» Needs review
new1.22 KB
PASSED: [[SimpleTest]]: [MySQL] 3,125 pass(es).
[ View ]

And here's a patch to make it happen. Seems harmless because you can just use the default behaviour if you want. Or you can use the tokens.

Status:Needs review» Needs work

The last submitted patch, private_message_1366118.patch, failed testing.

chrisschaub’s picture

Version:7.x-2.x-dev» 7.x-1.x-dev

Switching the version to 7.x-1 since that is the current rev. The patch is against that revision. It should work on 2.x though.

chrisschaub’s picture

Status:Needs work» Needs review

#4: private_message_1366118.patch queued for re-testing.

Berdir’s picture

Status:Needs review» Fixed

Thanks, commited to 7.x-2.x and 7.x-1.x

Status:Fixed» Closed (fixed)

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

trevonio’s picture

Thanks @cschaub

trevonio’s picture

Version:7.x-1.x-dev» 7.x-1.3
Status:Closed (fixed)» Active

Tried with 7.x-1.3 with no luck.

I used [current-user:mail] since seems that [privatemsg_message:author:mail] is not available anymore.

Was the code commited?

Berdir’s picture

Version:7.x-1.3» 7.x-1.x-dev
Status:Active» Closed (fixed)

Yes it was, but a7.x-1.3 is older than the commit date, so exists in the dev snapshot and will exist in the next stable version.