Control: tag -1 moreinfo On Mon, Aug 24, 2015 at 21:46:04 +0200, Mike Gabriel wrote:
> diff -Nru > gosa-2.7.4+reloaded2/debian/patches/1010_fix-entry-removal-in-mail-plugin.patch > > gosa-2.7.4+reloaded2/debian/patches/1010_fix-entry-removal-in-mail-plugin.patch > --- > gosa-2.7.4+reloaded2/debian/patches/1010_fix-entry-removal-in-mail-plugin.patch > 1970-01-01 01:00:00.000000000 +0100 > +++ > gosa-2.7.4+reloaded2/debian/patches/1010_fix-entry-removal-in-mail-plugin.patch > 2015-08-24 21:24:28.000000000 +0200 > @@ -0,0 +1,30 @@ > +Description: Fix entry removal in mail plugin > +Author: eannj <[email protected]> > +Abstract: > + Without this patch, it is not possible to remove entries from > + "Alternative addresses" or from "Forward messages to non group members" > + list in "mail" tab of group objects. One selects an address and press > + "delete" but nothing happens, the page is refreshed but address entry > + remains. The same operation succeeds for users. It fails only for group > + objects. The attached patch fixes the issue. > + > +--- a/mail/admin/groups/mail/class_groupMail.inc > ++++ b/mail/admin/groups/mail/class_groupMail.inc > +@@ -394,7 +394,7 @@ > + } > + } > + if (isset($_POST['delete_forwarder'])){ > +- $this->delForwarder (get_post('forwarder_list')); > ++ $this->delForwarder ($_POST('forwarder_list')); > + } > + > + > +@@ -432,7 +432,7 @@ > + } > + } > + if (isset($_POST['delete_alternate']) && > isset($_POST['alternates_list'])){ > +- $this->delAlternate (get_post('alternates_list')); > ++ $this->delAlternate ($_POST('alternates_list')); > + } > + > + Shouldn't this be $_POST['alternates_list'] (and similar above)? Thankfully I'm not a php person, so please forgive the possibly stupid question. Cheers, Julien
