Forums: Replies from archived topics are still displayed in user profiles

[SergeyBiryukov]

1. Archive a topic (but not individual replies).
2. Replies from that topic are still displayed in participants' profiles.

[SergeyBiryukov]

Related: #2408 (for WP.org profiles).

[SergeyBiryukov]

It's not just archived; replies from pending and spam topics are also displayed in profiles, unless replies themselves have a non-public status too.

[fierevere]

repiles from archived topics are displayed in RSS-feeds as well, unless they are archived too

[Clorith]

@coffee2code might have some information on the connection between profiles and the archived states here.

[SergeyBiryukov]

#4724 was marked as a duplicate.

[Clorith]

Thinking on this, getting the parent topic of posts and checking the parent state sounds relatively expensive, I wonder if the right approach here would be to hook into the status change and change any reply with a default state to be archived as well, we would have some floaters pre-implementation, but I think that's an acceptable tradeoff, and they can be dealt with manually if desired?

Not sure how we would solve the data that's been sent to the profiles section though, since that's technically a different system.

[johnjamesjacoby]

See: bbp_trash_topic_replies() and bbp_untrash_topic_replies()

When a Topic has some action taken on it, some other action needs to run on the children Replies.

Unfortunately, you cannot simply blindly take action on all of them, because some of them may already be trashed, and you don't want untrashing something to accidentally bring things back from the dead.

See the _bbp_pre_trashed_replies Topic meta key, as an example of this juggle. It makes sure that when a Topic is untrashed, that previously trashed Replies do not suddenly also become untrashed.

(The same happens when marking a Topic as Spam.)

My recommendation is to mirror this behavior for Topic Archives, and mark the replies as archived as well. This may require a bit of finesse, since Topics generally expect their replies to either be "public" or "non-public" by default, and Archived kinda walks the line.

[dd32]

Is this a bbPress core thing, or a weirdness due to an optimization on the WordPress.org side of things?

[dd32]

Milestone renamed

[dd32]

Milestone renamed

[SergeyBiryukov]

Related: #5503

[Clorith]

Replying to dd32:

Is this a bbPress core thing, or a weirdness due to an optimization on the WordPress.org side of things?

This is a dotorg thing, archived is a topic/reply status that was implemented here specifically, so I think comment:11 has the keys we need, our custom status needs to do the same kind of juggling with reply-statuses.

[dd32]

#6750 was marked as a duplicate.