Skip to content
This repository has been archived by the owner on Aug 18, 2024. It is now read-only.

Other Groups field should respect bundles config #643

Merged
merged 1 commit into from
Jul 3, 2020

Conversation

alfaguru
Copy link

See issue #642

@pfrenssen
Copy link
Contributor

Thanks for this. I have not tested it but it seems to make sense to me.

We are in the process though of removing the OgComplex field (ref #570) so this is fixing code that is going to be removed. However since the removal of OgComplex seems to have stalled because it needs to be redone from scratch, maybe it is a good idea to get this in to help people in the short term.

@MPParsley what is your opinion?

@MPParsley
Copy link
Collaborator

Indeed, this issue should be fixed in the meanwhile.

OgComplex.php is removed in https://github.com/Gizra/og/pull/570/files#diff-9704db18679e46e953b1ea9a98c59078

I haven't tested it yet but I believe that removing the 'complex' field and using a 'simple' entity reference should prevent this issue from occurring.

@pfrenssen
Copy link
Contributor

Alright let's get this in.

Thanks @alfaguru for the contribution!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants