#1539972: og_get_group() is missing the @param for $states adds a mention of these constants, and I *think* I understand them.

But more a complete explanation of these is in order.

CommentFileSizeAuthor
#1 1539974-og-state-docs-1.patch1.01 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
1539974-og-state-docs-1.patch1.01 KB

How about this?

amitaibu’s picture

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

Probably can't heart committing -- Committed to 1.x and 2.x

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
CreditAttribution: tim.plunkett commented
Status: Fixed » Active

In 7.x-1.x, when calling og_get_group() from within a submit function where the group itself is being saved, you have to specify array(OG_STATE_ACTIVE, OG_STATE_PENDING).

It's really not clear why an existing group switches to OG_STATE_PENDING during submission. Is that a bug, or is that something that just needs to be clarified?
The changes you made seem to be about members, not the groups themselves.

amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
CreditAttribution: amitaibu commented

> why an existing group switches to OG_STATE_PENDING during submission

This sounds like a bug.

  • 6f5bd1e committed on 8.x-1.x 
    Issue #1539974 by Amitaibu | tim.plunkett: Add more complete docs for OG...