Next thing I would like to add in. Also, make recently added collapsible fieldset for the node reply form configurable.

Comments

acrazyanimal’s picture

Status: Active » Closed (fixed)
hutch’s picture

Status: Closed (fixed) » Active

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

  $text = token_replace_multiple($text, $tokens);

with

  foreach ($text as $key => $value) {
    $text[$key] = token_replace_multiple($value, $tokens);
  }

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.

acrazyanimal’s picture

Interesting, the array replacement was already being used. It works fine in my implementation and I don't see that warning. I'll investigate.

acrazyanimal’s picture

Status: Active » Closed (fixed)

This 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