Package: release.debian.org Severity: normal User: release.debian....@packages.debian.org Usertags: unblock User: debian-...@lists.debian.org Usertags: debian-edu
Here is a request to approve an upload of gosa to wheezy, to fix a bug Debian Edu need to have fixed in our Debian Edu Wheezy release. The same fix was uploaded as versoin 2.7.4-4.3 to unstable with this changelog entry: * debian/patches/fix-mass-ldapimport.patch: New patch, fixes LDAP mass import, by Giorgio Pioda and Petter Reinholdtsen. (Closes: #698840) We need it to make sure schools can create users for all teachers and users in a school starting to use Debian Edu (or migrating to the new version) without having to create them one by one. Attached is the complete diff between version 2.7.4-4.3 in unstable and version 2.7.4-4.2 in wheezy. Please let us know if this is OK before we upload, and tell us which version number to use for this upload. -- Happy hacking Petter Reinholdtsen
diff -Nru gosa-2.7.4/debian/changelog gosa-2.7.4/debian/changelog --- gosa-2.7.4/debian/changelog 2013-02-07 21:28:31.000000000 +0100 +++ gosa-2.7.4/debian/changelog 2013-06-14 11:00:02.000000000 +0200 @@ -1,3 +1,11 @@ +gosa (2.7.4-4.3) unstable; urgency=low + + * Non-maintainer upload. + * debian/patches/fix-mass-ldapimport.patch: New patch, fixes LDAP mass + import, by Giorgio Pioda and Petter Reinholdtsen. (Closes: #698840) + + -- Michael Banck <mba...@debian.org> Fri, 14 Jun 2013 10:59:37 +0200 + gosa (2.7.4-4.2) unstable; urgency=low [ Jonathan Wiltshire ] diff -Nru gosa-2.7.4/debian/patches/fix-mass-ldapimport.patch gosa-2.7.4/debian/patches/fix-mass-ldapimport.patch --- gosa-2.7.4/debian/patches/fix-mass-ldapimport.patch 1970-01-01 01:00:00.000000000 +0100 +++ gosa-2.7.4/debian/patches/fix-mass-ldapimport.patch 2013-06-14 10:58:23.000000000 +0200 @@ -0,0 +1,48 @@ +Index: addons/ldapmanager/class_csvimport.inc +=================================================================== +--- gosa/ldapmanager/addons/ldapmanager/class_csvimport.inc (revisjon 21231) ++++ gosa/ldapmanager/addons/ldapmanager/class_csvimport.inc (arbeidskopi) +@@ -175,17 +175,25 @@ + + + /* Set all submitted vars */ +- if(is_array($keys)) ++ if(is_array($keys)){ + foreach($keys as $key) { + if($key != "status"){ + $this->usertab->$key = $dat[$key]; ++ foreach($this->usertab->by_object as $pname => $plugin){ ++ if(isset($this->usertab->by_object[$pname]->$key)){ ++ $this->usertab->by_object[$pname]->$key = $dat[$key]; ++ } ++ } + } + } ++ } + + /* Initialize template */ + if($this->csvinfo['tplid']!=0){ +- $this->usertab->adapt_from_template($this->csvinfo['arrtempls'][$this->csvinfo['tplid']]['dn']); +- $this->usertab->save_object(); ++ $tpl_dn = $this->csvinfo['arrtempls'][$this->csvinfo['tplid']]['dn']; ++ $this->usertab->adapt_from_template($tpl_dn,array("uid","cn","givenName","sn")); ++ $tpl_base = preg_replace("/^[^,]+,".preg_quote(get_people_ou(), '/i')."/", '', $tpl_dn); ++ $this->usertab->by_object['user']->base= $tpl_base; + } else { + + /* Reset vars */ +Index: addons/ldapmanager/contentcsv.tpl +=================================================================== +--- gosa/ldapmanager/addons/ldapmanager/contentcsv.tpl (revisjon 21231) ++++ gosa/ldapmanager/addons/ldapmanager/contentcsv.tpl (arbeidskopi) +@@ -1,5 +1,3 @@ +-<font color='red'>Temporary disabled</font> +-<!-- + <p> + {t}The CSV import plug-in provides methods to generate user accounts from a file containing Comma Separated Values. The administrator can decide which columns should be transfered to which attribute. Note that you must have at least the UID, GIVENNAME and SURNAME set.{/t} + </p> +@@ -122,4 +120,3 @@ + {/if} + </div> + <input type="hidden" name="ignore"> +---> diff -Nru gosa-2.7.4/debian/patches/series gosa-2.7.4/debian/patches/series --- gosa-2.7.4/debian/patches/series 2012-06-19 09:57:50.000000000 +0200 +++ gosa-2.7.4/debian/patches/series 2013-06-08 18:58:37.000000000 +0200 @@ -5,3 +5,4 @@ 03_fix_class_mapping.patch 04_fix_locale_location.patch no-image-warning.patch +fix-mass-ldapimport.patch