Hello,
First of all, thanks for this great module.

On a Durpal-6.22 and phpBB3.0.8 fresh install, I can't properly sync roles / groups.

Setup :

Master: Drupal
role1 matches group1
role2 matches group 2

procedure : adding and editing users in Drupal, checking results in phpBB

test sequence :

Drupal: create user1 without any assigned role
Drupal: create user2 with role2 assigned to it
phpBB: both users have been created but user2 was not added to group2
Drupal: edit user2 (change nothing, just save)
phpBB: now, user2 has been added to group2
Drupal: edit user1, assign role1, save
phpBB: user1 has been added to group1
Drupal: edit user2, unassign role2, save
phpBB: user2 still belongs to group2
Drupal: edit user2, assign role1, save
phpBB: user2 has been added to group1 (belongs to both groups but endorses only role 1)
Drupal: delete user2
phpBB: user2 is deleted and removed from it's groups

Summary :

When a user is created in Drupal, it's roles are not synced in phpBB
When a role is added in Drupal to an already synced user, the new role is properly synced
When a role taken away from an existing user in Drupal, the user still belongs to the corresponding group in phpBB

or acconding to the Create/Update/Delete scheme :

Create : user only, no groups
Update : add to group only, no retract from group
Delete : OK

I hope it can help those of you who have programming skills...

Comments

fizk’s picture

This also effects 7.x-1.0-alpha3. I'm running into this issue as well.

fizk’s picture

StatusFileSize
new785 bytes

Here's a patch that fixes the following issue in 7.x-1.0-alpha3:

"When a user is created in Drupal, it's roles are not synced in phpBB"

fizk’s picture

StatusFileSize
new1.09 KB

This patch fixes the following issues in 7.x-1.0-alpha3:

"When a user is created in Drupal, it's roles are not synced in phpBB"
"When a role taken away from an existing user in Drupal, the user still belongs to the corresponding group in phpBB"

fizk’s picture

StatusFileSize
new1.09 KB

Sorry, the previous patch was reversed. Here's the proper patch.

fizk’s picture

Status: Active » Needs review
madhatterwales’s picture

I can confirm that patch has worked - tested on live and testing environment for both this module and the drupalbridge (phpbb) variant.

madhatterwales’s picture

Status: Needs review » Reviewed & tested by the community
fizk’s picture

bump.

fizk’s picture

I'll try to get commit access and commit this patch.

vb’s picture

done in last commit

fizk’s picture

Status: Reviewed & tested by the community » Closed (fixed)

Thanks vb!