Install fails because latest versions of og does not use variable og_node_types

Unfortunately subscriptions_og still not supports og 7.x #259029: How do I set up automatic subscription to Group?, and I'll be so busy for a while.

Blessings!

Haven't had time to test this yet, but you should be able to change all occurrences of:

  variable_get('og_node_types', array());

to:

  og_get_types('group');

because og_get_types('group') will return an empty array if there are no node types configured as groups.

I'll attach a patch if and when I get this working unless someone beats me to it.

Attached is the patch. It makes all the changes above and one more change -- subscriptions_og_get_groups_by_types() now returns an empty array if there are no types configured as groups, and presumably if there are no nodes of any types that are configured as groups.

The installation now works cleanly on a site that has no nodes configured as group types, and I suspect installation will work as before when there are nodes configured as group types, but that has yet to be tested.

Changing version to 5.x-1.1-1 since it's still an issue with this version.

Note also that I haven't done a compatibility check with the rest of og5.x-7.2, but this patch ought to help the next person continue on down the path.

Thank you very much @marcp and @dalin for your effort on it. I went to Cusco city(Perú) 4 days ago so will take another week for me to check patches and perform test, but thank you again.

Blessings!

I can't see any patch, but followed your comments, current dev version should install and migrate og notifications to subscriptions without issues. Please review http://drupal.org/node/272808#comment-896677 for further instructions.

Blessings!

You mean I have to attach the patch and not just pretend that it's attached? I'm glad you got the code in there without it because I can't even find the patch now!

Might be a while before I get a chance to test this, though.

How does it goes?