Rules integration broke in 6.x-1.17 [#1329196]

Hello, after upgrading to Token 6.x-1.17 all tokens in rules actions of type "Send a mail to an arbitrary mail address" stopped to work.

e.g. I got:

[...]
Titolo: [node:title]
Autore: [author:realname] ( [author:mail] )
Link di verifica: [:global:site-url]node/[node:nid]

Pagina degli avvisi pubblicati: [:global:site-url]notices
Pagina di gestione degli avvisi da pubblicare: [:global:site-url]manage_notices

After reverting to 6.x-1.16 I got it back working as expected:

[...]
Titolo: Dr. Antonello Lobianco
Autore: ANTONELLO LOBIANCO ( a.lobianco@univpm.it )
Link di verifica: http://www.econ.univpm.it/node/53911

Pagina degli avvisi pubblicati: http://www.econ.univpm.it/notices
Pagina di gestione degli avvisi da pubblicare: http://www.econ.univpm.it/manage_notices

And yes.. I did upgradedb and also cc.. :-)

Comment	File	Size	Author
#3	1329196-token-fix-replace-rules.patch	843 bytes	Dave Reid
#3

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

perpignan CreditAttribution: perpignan commented 2 November 2011 at 13:01

I have the same problem !
I did /upgrade.php and flush all cache but .. tokens aren't replace in mail.

Comment #2

fonant CreditAttribution: fonant commented 2 November 2011 at 14:22

I have this problem too.

Same bug as http://drupal.org/node/1328752

File	Size
1329196-token-fix-replace-rules.patch	843 bytes

Comment #4

castelar CreditAttribution: castelar commented 2 November 2011 at 19:37

Patch didn't work for me.

Comment #5

Dave Reid

he/him

English

Nebraska USA

CreditAttribution: Dave Reid commented 2 November 2011 at 20:26

I have added some debugging code to 6.x-1.x-dev release that will help trace what function is responsible for calling this incorrectly. Please download the 6.x-1.x-dev release in about 12 hours to help me debug this further.

Comment #6

Dave Reid

he/him

English

Nebraska USA

CreditAttribution: Dave Reid commented 2 November 2011 at 20:50

Title:

All tokens broken in rules->send a mail (reverting to 6.x-1.16 works)

» Rules integration broke in 6.x-1.17

Comment #7

svinod999 CreditAttribution: svinod999 commented 3 November 2011 at 07:24

same problem faced... subscribing...

Comment #8

tobiberlin

German

Berlin, Germany

CreditAttribution: tobiberlin commented 3 November 2011 at 12:30

Same with action "page redirection"... wanted to redirect a user to "/user/[account:uid]/edit" after logging in and now the token is not replaced by the users id.

Comment #9

f1mishutka CreditAttribution: f1mishutka commented 3 November 2011 at 14:12

Same problem. Downgrading to token 6.x-1.16

Comment #10

Dave Reid

he/him

English

Nebraska USA

CreditAttribution: Dave Reid commented 3 November 2011 at 15:25

Status:

Needs review

» Postponed (maintainer needs more info)

Everyone knows that there is a bug green 'Subscribe' button at the top right of this issue right? Please use it. Thanks.

I have committed a follow-up with http://drupalcode.org/project/token.git/commit/f02c914 that will allow token replacement to still work with arrays but it will also still throw a notice because modules which call with an array still need to be fixed. I will reiterate that it was never supposed to be supported. It only worked by accident.

Please test out the 6.x-1.x-dev release in about 12 hours once it has had the chance to regenerate.