Excluding groups instead

mariano.barcia - August 21, 2008 - 15:12
Project:OG promote
Version:5.x-1.0
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:won't fix
Description

Attaching a patch to change the policy from include groups to exclude groups.
See http://drupal.org/node/268007.

This patch also includes the following patches to critical issues:
http://drupal.org/node/259026 (which fixes http://drupal.org/node/171320)
http://drupal.org/node/298156

HTH

AttachmentSize
og_promote.patch6.17 KB

#1

AjK - November 21, 2008 - 14:55
Status:active» won't fix

Not committed. The problem here is that if this went in it would present existing users with a problem. You haven't allowed for any way to tell existing users "hey, you need to reconfigure because we just broke your old settings".

Always keep in mind the upgrade path for existing users.

#2

mariano.barcia - November 27, 2008 - 16:07

I think this is an important feature, I have not provided an upgrade path because I wasn't myself needing it at the moment... that's all I can do for the moment...

 
 

Drupal is a registered trademark of Dries Buytaert.