og_mandatory does not seem to work with og-enabled modules...

leoburd - July 13, 2006 - 11:41
Project:Organic Groups Mandatory Group
Version:HEAD
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

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

#1

moshe weitzman - July 13, 2006 - 13:29
Project:Organic groups» Organic Groups Mandatory Group
Version:HEAD» HEAD
Component:og.module» Code

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

#2

pwolanin - July 15, 2006 - 00:51

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.

#3

pwolanin - July 15, 2006 - 02:56
Status:active» needs review

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

AttachmentSize
all_og_types_patch.txt 1.08 KB

#4

pwolanin - July 15, 2006 - 19:21

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

#5

jo1ene - July 21, 2006 - 23:31

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

#6

pwolanin - July 22, 2006 - 01:58
Status:needs review» reviewed & tested by the community

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

#7

pwolanin - July 22, 2006 - 12:42
Status:reviewed & tested by the community» fixed

Changes commited to HEAD and 4-7 branches.

#8

Anonymous - August 5, 2006 - 12:46
Status:fixed» closed
 
 

Drupal is a registered trademark of Dries Buytaert.