Project

General

Profile

Actions

Bug #8991

open

RBE duplicate email message issue

Added by Matt Gold over 6 years ago. Updated about 6 years 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:
Deployment actions:

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 emailAssignedRaymond Hoh2018-11-04

Actions
Actions #1

Updated by Matt Gold over 6 years ago

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

Actions #2

Updated by Raymond Hoh over 6 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.

Actions #3

Updated by Matt Gold over 6 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.

Actions #4

Updated by Boone Gorges over 6 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.

Actions #5

Updated by Raymond Hoh over 6 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.

Actions #6

Updated by Raymond Hoh about 6 years 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.

Actions #7

Updated by Matt Gold about 6 years ago

Thank you, Ray

Actions #8

Updated by Boone Gorges about 6 years ago

Thanks for following up on this, Ray.

Actions #9

Updated by Raymond Hoh over 5 years ago

Actions #10

Updated by Raymond Hoh about 5 years ago

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

Updated by Raymond Hoh about 5 years ago

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

Also available in: Atom PDF