If someone enables creation of contacts on user creation or uses the sync manager by hand, relate() first checks for a contact. When no contact is found one can be created, this contact receives as an owner the user account itself.

This should not be the case, rather the current user should be the owner and only the relation should interrelate contact and user.

This is also an easy pitfall since developers can assume that the uid they are seeing is the actual relationship when it isn't.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

grahl created an issue. See original summary.

grahl’s picture

grahl’s picture

Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 2: 3061993-2-fix-owner.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

grahl’s picture

FileSize
1.18 KB

Fixed the inteface

grahl’s picture

Status: Needs work » Needs review
FileSize
1.06 KB

Status: Needs review » Needs work

The last submitted patch, 6: 3061993-6-fix-owner.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

grahl’s picture

Status: Needs work » Needs review

  • grahl committed 7145046 on 8.x-3.x
    Issue #3061993 by grahl: User Sync Relation sets unideal owner
    
grahl’s picture

Status: Needs review » Fixed
grahl’s picture

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.