Hi,
I try to create the container, but the forum was created.
However, when the line 97 was changed, the container was neatly created.

// $Id: forum.module,v 1.256 2005/05/31 21:14:26 unconed Exp $

$status = taxonomy_save_term($edit);
to
$status = taxonomy_save_term(&$edit);

Is this bug?

CommentFileSizeAuthor
#1 forum_14.patch688 bytesUwe Hermann
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Uwe Hermann’s picture

Uwe Hermann CreditAttribution: Uwe Hermann commented
Status: Active » Needs review
FileSize
forum_14.patch688 bytes

Confirmed in HEAD. Patch attached.

chx’s picture

chx CreditAttribution: chx commented
Status: Needs review » Closed (won't fix)

I have a complete solution for this -- and it's long deprecated to pass by reference.

DriesK’s picture

DriesK CreditAttribution: DriesK commented

Until chx's patch (http://drupal.org/node/29102) gets committed, this would be consistent with what's currently in HEAD:

list($status, $object) = array_values(taxonomy_save_term($edit));
if (arg(3) == 'container') {
  $containers = variable_get('forum_containers', array());
  $containers[] = $object['tid'];
}
Uwe Hermann’s picture

Uwe Hermann CreditAttribution: Uwe Hermann commented
Status: Closed (won't fix) » Closed (duplicate)

Duplicate of http://drupal.org/node/29102. DriesK, I'm fine with waiting for chx's patch to land; if you want another fix earlier, please change the status appropriately.