Project

General

Profile

Feature #3193

Feature #3177: Upgrade bbPress to 2.x

bbPress 2.x dynamic roles and RBE

Added by Boone Gorges almost 5 years ago. Updated over 4 years ago.

Status:
Assigned
Priority name:
Normal
Assignee:
Category name:
Group Forums
Target version:
Start date:
2014-05-02
Due date:
% Done:

0%

Estimated time:

Description

From #3177 http://redmine.gc.cuny.edu/issues/3177#note-3

One other thing to note is bbPress has the concept of dynamic user roles. At the moment, this conflicts with Reply By Email because I wanted to copy bbPress' validation schema. And since it is hard to determine these dynamic roles when the user isn't logged in, replies will fail to post. (Or, I'm just not looking in the right place ;)

A way around this is to set the user role automatically in bbPress 2 so a role is saved in the database.

Ray - setting user role automatically is fine with me, as long as it doesn't cause other problems. I mention this because of https://bbpress.trac.wordpress.org/ticket/2210, which is the one area of bbPress that I am familiar with :)

So, it could be that this ticket can be resolved with a simple settings toggle. In any case, it's a placeholder to remind us to check.

History

#1 Updated by Boone Gorges over 4 years ago

  • Assignee changed from Raymond Hoh to Boone Gorges
  • Target version changed from 1.7 to Future release

I've thought about this a bit more, and I think that for now I'm going to do nothing.

The problem is worst when bbPress is activated networkwide, which it will not be in our case.

If sites activate bbPress, users will be added as Participants to the sites once they visit them. If this turns out to be a problem in the future, let's look into writing a bbPress plugin that will unhook this https://bbpress.trac.wordpress.org/browser/tags/2.5.4/includes/core/actions.php#L99 and then do the necessary default role setup at some other point: for example, when they attempt to post. It's likely that this will be somewhat complicated, so let's leave it for when we know it's an actual problem.

Also available in: Atom PDF