See http://drupal.org/node/459402; it's comment is not in the appropriate place, nor the comment itself is appropiate.

CommentFileSizeAuthor
#10 459418_shadow_copy.patch827 bytesavpaderno
#5 shadowcopy.diff662 bytessilverwing

Comments

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented

Well, to be fair, shadow topics link to the forum, not the post, which is really a PITA on a fast moving forum. Let's leave it a bit so he can see the message about using his tracker and then it can go.

The attitude may not be great but the frustration is understandable. If you don't know about the tracker, finding a post again, especially if it's been moved, is really hard.

Michelle

silverwing’s picture

silverwing commented

I moved that post out of hosting-support to general support (and I've moved a few lately).

But I agree that linking to the post in it's new location makes a lot more sense than linking to a forum.

I'm *sure* an issue has been filed on this - right? - at least for the redesign?

~silverwing

silverwing’s picture

silverwing commented

Michelle, found this issue #13793: link to moved forum topic, not the new forum - still able to do this?

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented

I'm probably not going to be doing any work on core, but the code is there in AF if someone wants to give it a go.

Michelle

silverwing’s picture

silverwing commented
Title: Removal of comment made from maurice_drews » Change shadow copy to link to node
StatusFileSize
new shadowcopy.diff662 bytes
avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Project: Drupal.org site moderators » Drupal core
Version: » 7.x-dev
Component: Content moderation » forum.module
Category: task » feature

I am moving the report to the proper queue to be considered.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Version: 7.x-dev » 8.x-dev

Maybe it's too late to be considered for Drupal 7; let's see if it can go for Drupal 8.

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented

Marked #13793: link to moved forum topic, not the new forum as a duplicate of this since this has a patch.

Michelle

naheemsays’s picture

naheemsays commented

This seems more like a bugfix than a feaure request... Is there a way to get that patch tested without having to re upload?

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Version: 8.x-dev » 7.x-dev
Status: Active » Needs review
StatusFileSize
new 459418_shadow_copy.patch827 bytes

I re-rolled the patch for Drupal 7. IMO, as the change doesn't create problems to the existing modules, the change can be done also in Drupal 7.
I am not sure if to consider it a bug, as the reported link is correct; it just takes the user to the forum where the post has been moved, not to the post.

naheemsays’s picture

naheemsays commented
Issue tags: +Needs usability review

Seems like a usability bug to me.

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented

@nbz: Yeah, I'd agree with that. On a small forum it might not be a big deal but it leads to a real PITA on drupal.org when they see their post has been moved but then they have to hunt through potentially several pages on that forum to find it.

Michelle

andypost’s picture

andypost
he/him
Primary language Russian
commented

Also related #561766: Regression: Shadow topics no longer appear in their previous forums

andypost’s picture

andypost
he/him
Primary language Russian
commented

Is there any steps to reproduce/test?

naheemsays’s picture

naheemsays commented

create a topic in one forum. click edit and move it to another forum while making sure "leave shadow copy" is ticked.

naheemsays’s picture

naheemsays commented
Issue tags: -Needs usability review

#10: 459418_shadow_copy.patch queued for re-testing.

naheemsays’s picture

naheemsays commented

#10: 459418_shadow_copy.patch queued for re-testing.

Status: Needs review » Needs work
Issue tags: +Needs usability review

The last submitted patch, 459418_shadow_copy.patch, failed testing.

Bojhan’s picture

Bojhan commented
Issue tags: -Needs usability review +Usability

No screen or clear proposal to review.

Status: Needs work » Closed (outdated)

Automatically closed because Drupal 7 security and bugfix support has ended as of 5 January 2025. If the issue verifiably applies to later versions, please reopen with details and update the version.