1 files changed, 8 insertions, 18 deletions

diff --git a/editproducts.cgi b/editproducts.cgi
index 618aec547..a989e4bc1 100755
--- a/editproducts.cgi
+++ b/editproducts.cgi
@@ -158,19 +158,15 @@ if ($action eq 'new') {
     check_token_data($token, 'add_product');
 
     my %create_params = (
-        classification      => $classification_name,
-        name                => $product_name,
-        description         => scalar $cgi->param('description'),
-        version             => scalar $cgi->param('version'),
-        defaultmilestone    => scalar $cgi->param('defaultmilestone'),
-        isactive            => scalar $cgi->param('is_active'),
-        create_series       => scalar $cgi->param('createseries'),
-        allows_unconfirmed  => scalar $cgi->param('allows_unconfirmed'),
-        default_platform_id => scalar $cgi->param('default_platform_id'),
-        default_op_sys_id   => scalar $cgi->param('default_op_sys_id'),
-        security_group_id   => scalar $cgi->param('security_group_id'),
+        classification   => $classification_name,
+        name             => $product_name,
+        description      => scalar $cgi->param('description'),
+        version          => scalar $cgi->param('version'),
+        defaultmilestone => scalar $cgi->param('defaultmilestone'),
+        isactive         => scalar $cgi->param('is_active'),
+        create_series    => scalar $cgi->param('createseries'),
+        allows_unconfirmed => scalar $cgi->param('allows_unconfirmed'),
     );
-
     my $product = Bugzilla::Product->create(\%create_params);
 
     delete_token($token);
@@ -283,12 +279,6 @@ if ($action eq 'update') {
         default_milestone  => scalar $cgi->param('defaultmilestone'),
     });
 
-    foreach my $field (qw( default_platform_id default_op_sys_id security_group_id )) {
-        my $value = $cgi->param($field);
-        detaint_natural($value);
-        $product->set($field, $value);
-    }
-
     my $changes = $product->update();
 
     delete_token($token);