We should improve the logic in og_field_create_instance() to create a field on User entity, if needed.

CommentFileSizeAuthor
#3 1779558-auto-attach-field-user-3.patch4.15 KBamitaibu
#1 1779558-auto-attach-field-user-1.patch3.82 KBamitaibu
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
CreditAttribution: amitaibu commented
Status: Active » Needs review
FileSize
1779558-auto-attach-field-user-1.patch3.82 KB

Patch, with test.

Status: Needs review » Needs work

The last submitted patch, 1779558-auto-attach-field-user-1.patch, failed testing.

amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
CreditAttribution: amitaibu commented
Status: Needs work » Needs review
FileSize
1779558-auto-attach-field-user-3.patch4.15 KB

Fixed failing test (which was due to change in field name)

amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
CreditAttribution: amitaibu commented
Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

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