From: lpsolit%gmail.com <>
Date: Wed, 12 Oct 2005 09:27:06 +0000 (+0000)
Subject: Bug 311419: Update the group_controls method from Product.pm to use Group.pm - Patch... 
X-Git-Tag: bugzilla-2.22rc1~234
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=5eee7ab7fb527d926bfdf5927268862048cd4479;p=thirdparty%2Fbugzilla.git

Bug 311419: Update the group_controls method from Product.pm to use Group.pm - Patch by André Batosti <batosti@async.com.br> r=LpSolit a=justdave
---

diff --git a/Bugzilla/Product.pm b/Bugzilla/Product.pm
index c6dd698b6f..94547476fe 100644
--- a/Bugzilla/Product.pm
+++ b/Bugzilla/Product.pm
@@ -129,12 +129,6 @@ sub group_controls {
     if (!defined $self->{group_controls}) {
         my $query = qq{SELECT
                        groups.id,
-                       groups.name,
-                       groups.description,
-                       groups.isbuggroup,
-                       groups.last_changed,
-                       groups.userregexp,
-                       groups.isactive,
                        group_control_map.entry,
                        group_control_map.membercontrol,
                        group_control_map.othercontrol,
@@ -145,8 +139,12 @@ sub group_controls {
                   WHERE group_control_map.product_id = ?
                   AND   groups.isbuggroup != 0
                   ORDER BY groups.name};
-        my $self->{group_controls} = 
+        $self->{group_controls} = 
             $dbh->selectall_hashref($query, 'id', undef, $self->id);
+        foreach my $group (keys(%{$self->{group_controls}})) {
+            $self->{group_controls}->{$group}->{'group'} = 
+                new Bugzilla::Group($group);
+        }
     }
     return $self->{group_controls};
 }
@@ -354,8 +352,9 @@ Product.pm represents a product object.
 
  Params:      none.
 
- Returns:     A hash with group id as key and hash containing the
-              group data as value.
+ Returns:     A hash with group id as key and hash containing 
+              a Bugzilla::Group object and the properties of group
+              relative to the product.
 
 =item C<versions()>