-
-
Notifications
You must be signed in to change notification settings - Fork 61
Description
There is an issue with the synchronize option in admin->users as if you click on synchronize button at the top or any synchronize button associated with a particular user it will reset the NewBB count to zero even if a user has tons of posts in the forums.
I found out that the issue is related to SQL calls to table “bb_posts” while the correct table name is “newbb_posts” (it seem to me that this table has been renamed to “newbb_posts” at some point).
I have made the following changes in /modules/system/admin/users/jquery.php at line 53
$tables[] = array('table_name' => 'bb_posts', 'uid_column' => 'uid');
$tables[] = array('table_name' => 'newbb_posts', 'uid_column' => 'uid');for the per user synchronize option
And the following changes in /modules/system/admin/users/users.php at line 232
$tables[] = array('table_name' => 'bb_posts', 'uid_column' => 'uid');
$tables[] = array('table_name' => 'newbb_posts', 'uid_column' => 'uid');for the general (all users) synchronize option
Both corrections seem to work.
The file modules/newbb/include/config.php seem to have quite a few more references to the old naming convention ( from line 81) can’t really figure out what this is doing.
I am happy to submit corrections on github if that’s all needed to be done.