Project

General

Profile

Bug #8991

RBE duplicate email message issue

Added by Matt Gold almost 2 years ago. Updated over 1 year ago.

Status:
Hold
Priority name:
Normal
Assignee:
Category name:
Reply By Email
Target version:
Start date:
2017-12-11
Due date:
% Done:

0%

Estimated time:

Description

Hi Ray -- I received a message saying that I was trying to post a message via RBE that had already been posted. It is true that I had posted the same message via RBE, but it was to a different group. I received the following message when I did:

Hi there,

Unfortunately, your new forum topic could not be posted because it appears you already created this topic before.

Here is a copy of your attempted topic:

"Forwarded message:


Related issues

Related to CUNY Academic Commons - Feature #10659: Post to multiple groups via emailAssigned2018-11-04

History

#1 Updated by Matt Gold almost 2 years ago

Note that I did wind up posting it a second time, this time through the web interface

#2 Updated by Raymond Hoh almost 2 years ago

RBE uses bbPress's logic to check for duplicate forum topics.

When you post on the web, since you are an administrator, you are able to bypass this check.

When posting via RBE, bbPress does a logged-in user check for administrator privileges. Since RBE functions in a logged-out state, your topic will be checked for duplicates regardless of your administrator role.

For a workaround when posting duplicate topics via RBE, alter the body of the message slightly.

Another way to address this is to get bbPress to modify their bbp_check_for_duplicate() function to check against the post author instead of the logged-in user.

If you feel that we do not need to keep bbPress's logic of duplicate forum topics for RBE, then let me know and I'll see what I can do.

#3 Updated by Matt Gold almost 2 years ago

thanks, Ray. Just to confirm: it checks across groups, rather than for duplicate posts in a single group?

Changing the message a bit is a good workaround and I don't mind doing it.

#4 Updated by Boone Gorges almost 2 years ago

  • Assignee changed from Raymond Hoh to Matt Gold
  • Target version set to Not tracked

My understanding is that this doesn't require any changes that are tracked in the codebase, so I've moved this out of regular milestones. Please correct if this is an error.

#5 Updated by Raymond Hoh almost 2 years ago

Just to confirm: it checks across groups, rather than for duplicate posts in a single group?

Yes, the duplicate check is done across all group forums.

Changing the message a bit is a good workaround and I don't mind doing it.

This is only necessary when using RBE for now.


Just quoting myself here:

Another way to address this is to get bbPress to modify their bbp_check_for_duplicate() function to check against the post author instead of the logged-in user.

I created a bbPress ticket for making the bbp_check_for_duplicate() function less reliant on the current user context:
https://bbpress.trac.wordpress.org/ticket/3186

We'll see what they say about this.

#6 Updated by Raymond Hoh over 1 year ago

  • Status changed from Assigned to Hold
  • Assignee changed from Matt Gold to Raymond Hoh

An update here, bbPress merged my fix. Not sure when they intend to ship their next release though.

In the meantime, the workaround to slightly modify the email content when posting duplicate topics/replies stands.

#7 Updated by Matt Gold over 1 year ago

Thank you, Ray

#8 Updated by Boone Gorges over 1 year ago

Thanks for following up on this, Ray.

#9 Updated by Raymond Hoh 11 months ago

#10 Updated by Raymond Hoh 6 months ago

  • Related to Bug #7762: RBE post didn't go through added

#11 Updated by Raymond Hoh 6 months ago

  • Related to deleted (Bug #7762: RBE post didn't go through)

Also available in: Atom PDF