Followup commit for r13825

- Take other CopyPasteVars in account, too, when preparing
  for copy & paste, because otherwise some values might
  be missing, leading to strange effects.
- Remove sambaGroupType from CopyPasteVars because it leads
  to an access violation when saving the object, although
  the sambaGroupType is saved properly in the resulting
  object (identical to the value in the previous object)

git-svn-id: https://oss.gonicus.de/repositories/gosa/branches/2.6-lhm@13914 594d385d-05f5-0310-b6e9-bd551577e9d8

diff --git a/trunk/gosa-core/plugins/admin/groups/class_groupGeneric.inc b/trunk/gosa-core/plugins/admin/groups/class_groupGeneric.inc
index 3d611352aa21960e3a7b655ba85b8d6cc5f0235b..50d0673bb16be3a2923cba39a5c1aff5cf501172 100644 (file)
--- a/trunk/gosa-core/plugins/admin/groups/class_groupGeneric.inc
+++ b/trunk/gosa-core/plugins/admin/groups/class_groupGeneric.inc
@@ -70,7 +70,7 @@ class group extends plugin
   var $attributes= array("cn", "description", "gidNumber","memberUid","sambaGroupType","sambaSID","accessTo","trustModel");
   var $objectclasses= array("top", "posixGroup");
 
-  var $CopyPasteVars  = array("force_gid","fon_group","smbgroup","groupType","sambaSID","sambaDomainName","SID","nagios_group","sambaGroupType");
+  var $CopyPasteVars  = array("force_gid","fon_group","smbgroup","groupType","sambaSID","sambaDomainName","SID","nagios_group");
 
   var $multiple_support = TRUE;
 
@@ -1408,7 +1408,9 @@ class group extends plugin
     plugin::PrepareForCopyPaste($source);
     
     $source_o = new group($this->config, $source['dn']);
-    $this->smbgroup = $source_o->smbgroup;
+    foreach($this->CopyPasteVars as $var) {
+        $this->$var = $source_o->$var;
+    }
 
     $this->memberUid = array();
     if(isset($source['memberUid'])){