If a user who does not currently have an Ironweed account, subscribes to GJG from an invitation to join a group, she will be able to register and create an account fine, but when she is logged in, she will not be auto-subscribed to the group. There is red confirmation text, however, that says that she has been subscribed to the group.
Fixing this defect is critical for our beta testing..
The 'long' reason is that even with this change there are still bugs; array_merge errors and prompting for an e-mail address during the registration form when one already exists. :\ Still working on a solution...
OK, this is now fixed. I have uploaded a new copy of og.module to the dev server (Josh will need to move it to beta because I don't have access -- will send him an e-mail right after this post).
For those following along at home. ;) There are two changes you need to make. The one outlined above, as well as:
I've also updated gjg_event, invite_api, and user_invite with this change.
I will be rolling a new patch for og.module in the coming days to incorporate these changes into the GJG extensions of OG, but I'm holding off for now to see what comes of this issue: http://drupal.org/node/49179
Comments
Comment #1
sdemps3 commentedIf a user who does not currently have an Ironweed account, subscribes to GJG from an invitation to join a group, she will be able to register and create an account fine, but when she is logged in, she will not be auto-subscribed to the group. There is red confirmation text, however, that says that she has been subscribed to the group.
Fixing this defect is critical for our beta testing..
Comment #2
webchickOk. So the 'short' reason this is doing this is the following line in the function og_invite_accept:
db_query("REPLACE INTO {node_access} (nid, realm, gid, grant_view) VALUES (%d, 'og_uid', %d, 1)", $gid, $uid);
needs to be changed to:
db_query("REPLACE INTO {og_uid} (nid, uid, is_active) VALUES (%d, %d, 1)", $gid, $uid);
The 'long' reason is that even with this change there are still bugs; array_merge errors and prompting for an e-mail address during the registration form when one already exists. :\ Still working on a solution...
Comment #3
webchickAh, finally. :) Gotta love cascading errors. :)
OK, this is now fixed. I have uploaded a new copy of og.module to the dev server (Josh will need to move it to beta because I don't have access -- will send him an e-mail right after this post).
For those following along at home. ;) There are two changes you need to make. The one outlined above, as well as:
$callbacks[] = array('accept callback' => 'user_invite_form', 'type' => 'user');
change to:
$callbacks[] = array('accept callback' => 'user_invite_form', 'type' => 'user', 'callback arguments' => array());
I've also updated gjg_event, invite_api, and user_invite with this change.
I will be rolling a new patch for og.module in the coming days to incorporate these changes into the GJG extensions of OG, but I'm holding off for now to see what comes of this issue: http://drupal.org/node/49179
Comment #4
sdemps3 commentedFix verifed.. Thanks so much!