Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.
This is all on a new git clone of branch 6.x-1.x.
I'm seeing the following notice on viewing a node_reply enabled node:
user notice: The node_reply_message_form() function called token replacement with an array rather than a string for $text in /opt/www/powysm6/html/modules/token/token.module on line 263.
I had a look at the code in function node_reply_message_form() and looked at the code in token.module line 252 onwards and then tried the following fix on line 225 in node_reply.module.
replace
Comments
Comment #1
acrazyanimal commentedComment #2
hutch commentedThis is all on a new git clone of branch 6.x-1.x.
I'm seeing the following notice on viewing a node_reply enabled node:
I had a look at the code in function node_reply_message_form() and looked at the code in token.module line 252 onwards and then tried the following fix on line 225 in node_reply.module.
replace
with
This stops the notice from appearing but I don't know for sure if it affects anything else.
Programatically is looks like the right thing to do IMO.
Comment #3
acrazyanimal commentedInteresting, the array replacement was already being used. It works fine in my implementation and I don't see that warning. I'll investigate.
Comment #4
acrazyanimal commentedThis is a completely separate issue so I've created a new one and this is going back to being closed.
http://drupal.org/node/1393360