[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#796823: jessie-pu: package gosa/2.7.4+reloaded2-1+deb8u2



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 <greg@brazcubas.br>
> +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


Reply to: