1. Log in to the administration account
2. Go to the forums and pick one of the forum topics
3. Click edit
4. Change the pulldown to a different Forum
5. Select "Leave shadow copy"
6. Navigate back to the old container forum

Expected behavior: node is moved and shadow topic shows up in the old forum
Actual behavior: node is moved, but no shadow topic appears in the old forum

CommentFileSizeAuthor
#4 shadow-topic-broken-bug45850.patch439 bytesWesley Tanaka

Comments

chx’s picture

chx commented

The quality of Wesley's bug reports are astonishing. I will bookmark this one and point those that post "it does not work" to it.

Wesley Tanaka’s picture

Wesley Tanaka commented

In response to that totally off-topic comment... ;)

If Drupal doesn't already have bug reporting guidelines, it might be a better idea to write some up instead. Mozilla's Bug Writing Guidelines are reasonably well written. They're mozilla-specific, but might make a good template for a Drupal version.

If low-quality bug reports are a problem, another simple idea that you might try is to pre-populate the description field with an empty template. Maybe something like:

Steps to reproduce:
1.
2.
3.

Actual Behavior:

Expected Behavior:

Version of PHP:

Version of Database:

In any case, I suggest that nobody else gets to follow up on this topic here, lest people get confused and forget about the bug. =)

Wesley Tanaka’s picture

Wesley Tanaka commented

bug introduced in forum.module revision 1.301

Wesley Tanaka’s picture

Wesley Tanaka commented
Status: Active » Needs review
StatusFileSize
new shadow-topic-broken-bug45850.patch439 bytes
Wesley Tanaka’s picture

Wesley Tanaka commented
Title: Not possible to create shadow topics? » Not possible to create shadow topics (regression introduced: forum.module v1.301)
Priority: Normal » Critical

Marking critical, since this is a regression.

dries’s picture

dries commented

taxonomy_nodeapi() is supposed to load the terms ...

function taxonomy_nodeapi($node, $op, $arg = 0) {
  switch ($op) {
    case 'load':
     $output['taxonomy'] = taxonomy_node_get_terms($node->nid);
     ...
moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Needs work
dries’s picture

dries commented
Status: Needs work » Fixed

Fixed this by rolling back the previous change.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)