Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.On user/register I get two of these:
* warning: array_keys() [function.array-keys]: The first argument should be an array in C:\xampplite\htdocs\drupal-4-7\modules\og_mandatory_group\og_mandatory_group.module on line 31.
I have created a closed OG called "Public" and under admin/settings/og have specified that to be the mandatory group. Other than enabling OG access control, I've made no other changes to a fresh Drupal 4.7 install + OG + og_basic + OG Mandatory group.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | og_mandatory_no_reg_form_6.txt | 1.89 KB | pwolanin |
| #5 | og_mandatory_no_reg_form_5.txt | 895 bytes | pwolanin |
| #4 | og_mandatory_no_reg_form_3.diff_0.txt | 1.11 KB | pwolanin |
| #3 | og_mandatory_no_reg_form_3.diff.txt | 1.11 KB | pwolanin |
Comments
Comment #1
webchickI see. When I check the "registration form" checkbox on the group, then it appears as a greyed-out checkbox and the error goes away.
Seems like it should be optional to show the group on the registration form or not, however.
Comment #2
pwolanin commentedYes, it should be optional. I added the "grayed-out" feature recently, so maybe I missed something when testing the code.
Comment #3
pwolanin commented@webchick - a quick look at the code suggests that the problem occurs when there are zero groups in the registration form.
A patch is attached, which checks for this condition before checking whether the mandatory group is in the form.
Comment #4
pwolanin commented@webchick - a quick look at the code suggests that the problem occurs when there are zero groups in the registration form.
A patch is attached, which checks for this condition before checking whether the mandatory group is in the form.
Comment #5
pwolanin commentedI think the attached patch is better
Comment #6
pwolanin commentedbetter still - improves selection of the message for the registration form as well.
Comment #7
pwolanin commentedcommited changes to 4.7 and HEAD
Comment #8
(not verified) commented