Hello there,

As far as I can tell, og_mandatory does not work with og-enabled modules. It only seems to work with groups of type "og"... is that true? If so, would it be possible to fix that?

Thanks in advance,

Leo

CommentFileSizeAuthor
#3 all_og_types_patch.txt1.08 KBpwolanin

Comments

moshe weitzman’s picture

moshe weitzman commented
Project: Organic Groups » Organic Groups Mandatory Group
Version: master »
Component: og.module » Code

moving to proper Project. fyi, i'm not working on this.

pwolanin’s picture

pwolanin commented

I'll take a look at the code when I have the chance. Killes wrote most of the code- I just finished the 4.7 conversion, so I don't have an immediate answer. It's quite possible that it only looks at groups of type "og" since it was written before the additional node type functionality was finalized.

pwolanin’s picture

pwolanin commented
Status: Active » Needs review
StatusFileSize
new all_og_types_patch.txt1.08 KB

Ok, try the attached patch- I think the problem is just this one JOIN statement. (note I haven't tested this myself yet)

pwolanin’s picture

pwolanin commented

minimally test the patch now- seems to work as expected. Please confirm that it functions in your environment.

jo1ene’s picture

jo1ene commented

+1 Patch works as expected. Module doesn't work for me without it.

pwolanin’s picture

pwolanin commented
Status: Needs review » Reviewed & tested by the community

Ok, I'll commit this change barring any howls of protest.

pwolanin’s picture

pwolanin commented
Status: Reviewed & tested by the community » Fixed

Changes commited to HEAD and 4-7 branches.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)