[Fusionforge-commits] FusionForge branch master updated. f8f83044e4b77b3bc8a4e30f0ea6dc708382206d

Franck Villaume nerville at fusionforge.org
Sun Apr 20 16:08:09 CEST 2014


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "FusionForge".

The branch, master has been updated
       via  f8f83044e4b77b3bc8a4e30f0ea6dc708382206d (commit)
      from  25823d95f6f1739198119fd358a727d4264a083f (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit f8f83044e4b77b3bc8a4e30f0ea6dc708382206d
Author: Franck Villaume <franck.villaume at trivialdev.com>
Date:   Sun Apr 20 16:07:10 2014 +0200

    use cookie to store warning_msg, error_msg & feedback values

diff --git a/src/common/account/actions/addsshkey.php b/src/common/account/actions/addsshkey.php
index 8fee9d7..b60a929 100644
--- a/src/common/account/actions/addsshkey.php
+++ b/src/common/account/actions/addsshkey.php
@@ -30,25 +30,27 @@ $uploaded_filekey = getUploadedFile('uploaded_filekey');
 if (strlen($authorized_key)) {
 	checkKeys($authorized_key);
 	if (!$u->addAuthorizedKey($authorized_key)) {
-		session_redirect('/account/?&error_msg='.urlencode($u->getErrorMessage()));
+		$error_msg = $u->getErrorMessage();
+		session_redirect('/account/');
 	}
 	$feedback = _('SSH Key added successfully.');
-	session_redirect('/account/?&feedback='.urlencode($feedback));
+	session_redirect('/account/');
 }
 
 if (!is_uploaded_file($uploaded_filekey['tmp_name'])) {
-	$return_msg = _('Invalid file name.');
-	session_redirect('/account/?&error_msg='.urlencode($return_msg));
+	$error_msg = _('Invalid file name.');
+	session_redirect('/account/');
 }
 
 $payload = fread(fopen($uploaded_filekey['tmp_name'], 'r'), $uploaded_filekey['size']);
 if (strlen($payload)) {
 	checkKeys($payload);
 	if (!$u->addAuthorizedKey($payload)) {
-		session_redirect('/account/?&error_msg='.urlencode($u->getErrorMessage()));
+		$error_msg = $u->getErrorMessage();
+		session_redirect('/account/');
 	}
 	$feedback = _('SSH Key added successfully.');
-	session_redirect('/account/?&feedback='.urlencode($feedback));
+	session_redirect('/account/');
 }
 
 session_redirect('/account/');
diff --git a/src/common/account/actions/deletesshkey.php b/src/common/account/actions/deletesshkey.php
index 077a006..8020346 100644
--- a/src/common/account/actions/deletesshkey.php
+++ b/src/common/account/actions/deletesshkey.php
@@ -25,10 +25,11 @@ global $u;
 $keyid = getStringFromRequest('keyid');
 if (is_numeric($keyid)) {
 	if (!$u->deleteAuthorizedKey($keyid)) {
-		session_redirect('/account/?&error_msg='.urlencode($u->getErrorMessage()));
+		$error_msg = $u->getErrorMessage();
+		session_redirect('/account/');
 	}
 	$feedback = _('SSH Key deleted successfully.');
-	session_redirect('/account/?&feedback='.urlencode($feedback));
+	session_redirect('/account/');
 }
 
 session_redirect('/account/');
diff --git a/src/common/docman/actions/monitorfile.php b/src/common/docman/actions/monitorfile.php
index 209dc27..e08c5f6 100644
--- a/src/common/docman/actions/monitorfile.php
+++ b/src/common/docman/actions/monitorfile.php
@@ -54,11 +54,10 @@ switch ($option) {
 		foreach ($arr_fileid as $fileid) {
 			if (!empty($fileid)) {
 				$d = new Document($g, $fileid);
-				if ($d->isError())
-					session_redirect($redirecturl.'&error_msg='.urlencode($d->getErrorMessage()));
-
-				if (!$d->addMonitoredBy($LUSER->getID()))
-					session_redirect($redirecturl.'&error_msg='.urlencode($d->getErrorMessage()));
+				if ($d->isError() || !$d->addMonitoredBy($LUSER->getID())) {
+					$error_msg = $d->getErrorMessage();
+					session_redirect($redirecturl);
+				}
 			} else {
 				$warning_msg = _('No action to perform');
 				session_redirect($redirecturl);
@@ -72,12 +71,10 @@ switch ($option) {
 		foreach ($arr_fileid as $fileid) {
 			if (!empty($fileid)) {
 				$d = new Document($g, $fileid);
-				if ($d->isError())
-					session_redirect($redirecturl.'&error_msg='.urlencode($d->getErrorMessage()));
-
-				if (!$d->removeMonitoredBy($LUSER->getID()))
-					session_redirect($redirecturl.'&error_msg='.urlencode($d->getErrorMessage()));
-
+				if ($d->isError() || !$d->removeMonitoredBy($LUSER->getID())) {
+					$error_msg = $d->getErrorMessage();
+					session_redirect($redirecturl);
+				}
 			} else {
 				$warning_msg = _('No action to perform');
 				session_redirect($redirecturl);
@@ -88,7 +85,7 @@ switch ($option) {
 		break;
 	}
 	default: {
-		$error_msg = _('Docman: monitoring action unknown.');
+		$error_msg = _('Docman')._(': ')._('monitoring action unknown.');
 		session_redirect($redirecturl);
 	}
 }
diff --git a/src/common/docman/views/addfile.php b/src/common/docman/views/addfile.php
index 7a5a406..8b813ee 100644
--- a/src/common/docman/views/addfile.php
+++ b/src/common/docman/views/addfile.php
@@ -51,8 +51,8 @@ if ($dgf->isError())
 	exit_error($dgf->getErrorMessage(), 'docman');
 
 if (!forge_check_perm('docman', $group_id, 'submit')) {
-	$return_msg = _('Document Manager Action Denied.');
-	session_redirect($redirecturl.'&warning_msg='.urlencode($return_msg));
+	$warning_msg = _('Document Manager Action Denied.');
+	session_redirect($redirecturl);
 }
 echo html_ao('script', array('type' => 'text/javascript'));
 ?>
@@ -80,8 +80,10 @@ echo html_ao('div', array('class' => 'docmanDivIncluded'));
 if ($dgf->getNested() == NULL) {
 	$dg = new DocumentGroup($g);
 
-	if ($dg->isError())
-		session_redirect('/docman/?group_id='.$group_id.'&error_msg='.urlencode($dg->getErrorMessage()));
+	if ($dg->isError()) {
+		$error_msg = $dg->getErrorMessage();
+		session_redirect('/docman/?group_id='.$group_id);
+	}
 
 	if ($dg->create('Uncategorized Submissions')) {
 		session_redirect('/docman/?group_id='.$group_id.'&view=additem');
diff --git a/src/common/docman/views/additem.php b/src/common/docman/views/additem.php
index 8505a46..c1ae539 100644
--- a/src/common/docman/views/additem.php
+++ b/src/common/docman/views/additem.php
@@ -28,8 +28,8 @@ global $group_id; // id of the group
 global $dirid; // id of the doc_group
 
 if (!forge_check_perm('docman', $group_id, 'submit')) {
-	$return_msg= _('Document Manager Access Denied');
-	session_redirect('/docman/?group_id='.$group_id.'&warning_msg='.urlencode($return_msg));
+	$warning_msg = _('Document Manager Access Denied');
+	session_redirect('/docman/?group_id='.$group_id);
 }
 
 echo html_ao('script', array('type' => 'text/javascript'));
diff --git a/src/common/docman/views/addsubdocgroup.php b/src/common/docman/views/addsubdocgroup.php
index 7c2edd2..c9a46f6 100644
--- a/src/common/docman/views/addsubdocgroup.php
+++ b/src/common/docman/views/addsubdocgroup.php
@@ -31,8 +31,8 @@ global $group_id; // id of the group
 global $dirid; // id of the doc_group
 
 if (!forge_check_perm('docman', $group_id, 'approve')) {
-	$return_msg= _('Document Manager Access Denied');
-	session_redirect('/docman/?group_id='.$group_id.'&warning_msg='.urlencode($return_msg));
+	$warning_msg = _('Document Manager Access Denied');
+	session_redirect('/docman/?group_id='.$group_id);
 }
 
 // plugin projects-hierarchy
diff --git a/src/common/docman/views/admin.php b/src/common/docman/views/admin.php
index a58535d..d713f6a 100644
--- a/src/common/docman/views/admin.php
+++ b/src/common/docman/views/admin.php
@@ -31,8 +31,8 @@ global $g; //group object
 global $group_id; // id of the group
 
 if ( !forge_check_perm('docman', $group_id, 'admin')) {
-	$return_msg= _('Document Manager Access Denied');
-	session_redirect('/docman/?group_id='.$group_id.'&warning_msg='.urlencode($return_msg));
+	$warning_msg = _('Document Manager Access Denied');
+	session_redirect('/docman/?group_id='.$group_id);
 }
 
 echo html_ao('div', array('id' => 'principalAdminDiv', 'class' => 'docmanDivIncluded'));
diff --git a/src/common/docman/views/editdocgroup.php b/src/common/docman/views/editdocgroup.php
index 4ee338f..0d5efd0 100644
--- a/src/common/docman/views/editdocgroup.php
+++ b/src/common/docman/views/editdocgroup.php
@@ -36,8 +36,8 @@ global $dm; // the Document Manager object
 global $HTML;
 
 if (!forge_check_perm('docman', $group_id, 'approve')) {
-	$return_msg= _('Document Manager Access Denied');
-	session_redirect('/docman/?group_id='.$group_id.'&warning_msg='.urlencode($return_msg));
+	$warning_msg = _('Document Manager Access Denied');
+	session_redirect('/docman/?group_id='.$group_id);
 }
 
 // plugin projects-hierarchy
@@ -48,8 +48,10 @@ if ($childgroup_id) {
 }
 
 $dg = new DocumentGroup($g, $dirid);
-if ($dg->isError())
-	session_redirect('/docman/?group_id='.$group_id.'&error_msg='.urlencode($dg->getErrorMessage()));
+if ($dg->isError()) {
+	$error_msg = $dg->getErrorMessage();
+	session_redirect('/docman/?group_id='.$group_id);
+}
 
 echo html_ao('div', array('class' => 'docmanDivIncluded'));
 echo $HTML->openForm(array('name' => 'editgroup', 'action' => util_make_uri($actionurl), 'method' => 'post'));
diff --git a/src/common/docman/views/editfile.php b/src/common/docman/views/editfile.php
index e3c8627..0bef110 100644
--- a/src/common/docman/views/editfile.php
+++ b/src/common/docman/views/editfile.php
@@ -28,8 +28,8 @@ global $group_id; // id of the group
 global $HTML;
 
 if (!forge_check_perm('docman', $group_id, 'approve')) {
-	$return_msg= _('Document Manager Access Denied');
-	session_redirect('/docman/?group_id='.$group_id.'&warning_msg='.urlencode($return_msg));
+	$warning_msg = _('Document Manager Access Denied');
+	session_redirect('/docman/?group_id='.$group_id);
 }
 
 echo html_ao('div', array('id' => 'editFile'));
diff --git a/src/common/docman/views/help.php b/src/common/docman/views/help.php
index e0e1d51..2bbb390 100644
--- a/src/common/docman/views/help.php
+++ b/src/common/docman/views/help.php
@@ -28,8 +28,8 @@ global $group_id; // id of the group
 global $g; // the project object
 
 if (!forge_check_perm('docman', $group_id, 'read')) {
-	$return_msg= _('Document Manager Access Denied');
-	session_redirect('/docman/?group_id='.$group_id.'&warning_msg='.urlencode($return_msg));
+	$warning_msg = _('Document Manager Access Denied');
+	session_redirect('/docman/?group_id='.$group_id);
 }
 
 echo html_ao('div', array('class' => 'docmanDivIncluded'));
diff --git a/src/common/docman/views/listfile.php b/src/common/docman/views/listfile.php
index 887b6b1..3ffd920 100644
--- a/src/common/docman/views/listfile.php
+++ b/src/common/docman/views/listfile.php
@@ -39,8 +39,8 @@ $linkmenu = 'listfile';
 $baseredirecturl = '/docman/?group_id='.$group_id;
 $redirecturl = $baseredirecturl.'&view='.$linkmenu.'&dirid='.$dirid;
 if (!forge_check_perm('docman', $group_id, 'read')) {
-	$return_msg= _('Document Manager Access Denied');
-	session_redirect($baseredirecturl.'&warning_msg='.urlencode($return_msg));
+	$warning_msg = _('Document Manager Access Denied');
+	session_redirect($baseredirecturl);
 }
 
 echo html_ao('div', array('id' => 'leftdiv'));
@@ -51,8 +51,8 @@ echo html_ac(html_ap() - 1);
 $childgroup_id = getIntFromRequest('childgroup_id');
 if ($childgroup_id) {
 	if (!forge_check_perm('docman', $childgroup_id, 'read')) {
-		$return_msg= _('Document Manager Access Denied');
-		session_redirect($baseredirecturl.'&warning_msg='.urlencode($return_msg));
+		$warning_msg = _('Document Manager Access Denied');
+		session_redirect($baseredirecturl);
 	}
 	$redirecturl .= '&childgroup_id='.$childgroup_id;
 	$g = group_get_object($childgroup_id);
@@ -99,11 +99,12 @@ if ($dirid) {
 	$DocGroupName = $ndg->getName();
 	$dgpath = $ndg->getPath(true, false);
 	if (!$DocGroupName) {
-		session_redirect($baseredirecturl.'&error_msg='.urlencode($g->getErrorMessage()));
+		$error_msg = $g->getErrorMessage();
+		session_redirect($baseredirecturl);
 	}
 	if ($ndg->getState() != 1) {
 		$error_msg = _('Invalid folder');
-		session_redirect($baseredirecturl.'&view=listfile&error_msg='.urlencode($error_msg));
+		session_redirect($baseredirecturl.'&view=listfile');
 	}
 }
 
diff --git a/src/common/docman/views/listtrashfile.php b/src/common/docman/views/listtrashfile.php
index c7091a7..4b06275 100644
--- a/src/common/docman/views/listtrashfile.php
+++ b/src/common/docman/views/listtrashfile.php
@@ -36,8 +36,8 @@ $childgroup_id = getIntFromRequest('childgroup_id');
 $baseredirecturl = '/docman/?group_id='.$group_id;
 $redirecturl = $baseredirecturl.'&view='.$linkmenu.'&dirid='.$dirid;
 if (!forge_check_perm('docman', $group_id, 'approve')) {
-	$return_msg= _('Document Manager Access Denied');
-	session_redirect($baseredirecturl.'&warning_msg='.urlencode($return_msg));
+	$warning_msg = _('Document Manager Access Denied');
+	session_redirect($baseredirecturl);
 }
 
 echo html_ao('div', array('id' => 'leftdiv'));
@@ -48,8 +48,8 @@ echo html_ac(html_ap() - 1);
 $childgroup_id = getIntFromRequest('childgroup_id');
 if ($childgroup_id) {
 	if (!forge_check_perm('docman', $childgroup_id, 'read')) {
-		$return_msg= _('Document Manager Access Denied');
-		session_redirect($baseredirecturl.'&warning_msg='.urlencode($return_msg));
+		$warning_msg = _('Document Manager Access Denied');
+		session_redirect($baseredirecturl);
 	}
 	$redirecturl .= '&childgroup_id='.$childgroup_id;
 	$g = group_get_object($childgroup_id);
@@ -74,11 +74,12 @@ if ($dirid) {
 	$ndg = new DocumentGroup($g, $dirid);
 	$DocGroupName = $ndg->getName();
 	if (!$DocGroupName) {
-		session_redirect($baseredirecturl.'&error_msg='.urlencode($g->getErrorMessage()));
+		$error_msg = $g->getErrorMessage();
+		session_redirect($baseredirecturl);
 	}
 	if ($ndg->getState() != 2) {
 		$error_msg = _('Invalid folder');
-		session_redirect($baseredirecturl.'&view='.$linkmenu.'&error_msg='.urlencode($error_msg));
+		session_redirect($baseredirecturl.'&view='.$linkmenu);
 	}
 }
 
diff --git a/src/common/docman/views/menu.php b/src/common/docman/views/menu.php
index 7318c84..f319ede 100644
--- a/src/common/docman/views/menu.php
+++ b/src/common/docman/views/menu.php
@@ -32,8 +32,8 @@ global $d_arr; // document array
 global $group_id; // id of group
 
 if (!forge_check_perm('docman', $group_id, 'read')) {
-	$return_msg= _('Document Manager Access Denied');
-	session_redirect('/docman/?group_id='.$group_id.'&warning_msg='.urlencode($return_msg));
+	$warning_msg = _('Document Manager Access Denied');
+	session_redirect('/docman/?group_id='.$group_id);
 }
 
 /* create the submenu following role, rules and content */
diff --git a/src/common/docman/views/movefile.php b/src/common/docman/views/movefile.php
index 6135025..9f77ff9 100644
--- a/src/common/docman/views/movefile.php
+++ b/src/common/docman/views/movefile.php
@@ -31,8 +31,8 @@ global $dm; // the Document Manager object
 global $HTML;
 
 if (!forge_check_perm('docman', $group_id, 'approve')) {
-	$return_msg= _('Document Manager Access Denied');
-	session_redirect('/docman/?group_id='.$group_id.'&warning_msg='.urlencode($return_msg));
+	$warning_msg = _('Document Manager Access Denied');
+	session_redirect('/docman/?group_id='.$group_id);
 }
 
 // plugin projects-hierarchy
diff --git a/src/common/docman/views/pendingfiles.php b/src/common/docman/views/pendingfiles.php
index 1a78b30..7ca2ca1 100644
--- a/src/common/docman/views/pendingfiles.php
+++ b/src/common/docman/views/pendingfiles.php
@@ -33,8 +33,8 @@ global $nested_groups;
 global $redirecturl; // built url from listfile.php (handle the hierarchy)
 
 if (!forge_check_perm('docman', $g->getID(), 'approve')) {
-	$return_msg= _('Document Manager Access Denied');
-	session_redirect($redirecturl.'&warning_msg='.urlencode($return_msg));
+	$warning_msg = _('Document Manager Access Denied');
+	session_redirect($redirecturl);
 }
 
 if (!isset($nested_pending_docs)) {
diff --git a/src/common/docman/views/reporting.php b/src/common/docman/views/reporting.php
index 2cb2d29..7058eac 100644
--- a/src/common/docman/views/reporting.php
+++ b/src/common/docman/views/reporting.php
@@ -50,8 +50,8 @@ global $group_id; // id of group
 global $g; // the group object
 
 if ( !forge_check_perm('docman', $group_id, 'admin')) {
-	$return_msg= _('Document Manager Access Denied');
-	session_redirect('/docman/?group_id='.$group_id.'&warning_msg='.urlencode($return_msg));
+	$warning_msg = _('Document Manager Access Denied');
+	session_redirect('/docman/?group_id='.$group_id);
 }
 
 $start = getIntFromRequest('start');
diff --git a/src/common/docman/views/search.php b/src/common/docman/views/search.php
index 6327424..4175ec4 100644
--- a/src/common/docman/views/search.php
+++ b/src/common/docman/views/search.php
@@ -32,8 +32,8 @@ global $gfcommon;
 global $HTML;
 
 if (!forge_check_perm('docman', $group_id, 'read')) {
-	$return_msg= _('Document Manager Access Denied');
-	session_redirect('/docman/?group_id='.$group_id.'&warning_msg='.urlencode($return_msg));
+	$warning_msg = _('Document Manager Access Denied');
+	session_redirect('/docman/?group_id='.$group_id);
 }
 
 $is_editor = forge_check_perm('docman', $g->getID(), 'approve');
diff --git a/src/common/docman/views/tree.php b/src/common/docman/views/tree.php
index b3c4f1d..1a27b1d 100644
--- a/src/common/docman/views/tree.php
+++ b/src/common/docman/views/tree.php
@@ -33,8 +33,8 @@ global $g; // the group object
 global $dirid; // the selected directory
 
 if (!forge_check_perm('docman', $group_id, 'read')) {
-	$return_msg= _('Document Manager Access Denied');
-	session_redirect('/docman/?group_id='.$group_id.'&warning_msg='.urlencode($return_msg));
+	$warning_msg= _('Document Manager Access Denied');
+	session_redirect('/docman/?group_id='.$group_id);
 }
 
 echo html_ao('div', array('id' => 'documenttree'));
diff --git a/src/common/docman/views/views.php b/src/common/docman/views/views.php
index 82749d9..19ac2ab 100644
--- a/src/common/docman/views/views.php
+++ b/src/common/docman/views/views.php
@@ -22,8 +22,8 @@
  * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
  */
 if (!forge_check_perm('docman', $group_id, 'read')) {
-	$return_msg= _('Document Manager Access Denied');
-	session_redirect('/docman/?group_id='.$group_id.'&warning_msg='.urlencode($return_msg));
+	$warning_msg = _('Document Manager Access Denied');
+	session_redirect('/docman/?group_id='.$group_id);
 }
 
 $view = getStringFromRequest('view', 'listfile');
diff --git a/src/common/include/account.php b/src/common/include/account.php
index 501c86c..950bc1c 100644
--- a/src/common/include/account.php
+++ b/src/common/include/account.php
@@ -320,9 +320,9 @@ function checkKeys($keys) {
 			 ends with zero or up to three '=' and the length must be >= 512 bits (157 base64 characters).
 			 The whole key ends with an optional comment. */
 			if ( preg_match("@^(((no-port-forwarding|no-X11-forwarding|no-agent-forwarding|no-pty|command=\"[^\"]+\"|from=\"?[A-Za-z0-9\.-]+\"?),?)*\s+)?ssh-(rsa|dss)\s+[A-Za-z0-9+/]{157,}={0,2}(\s+.*)?$@", $key) === 0 ) { // Warning: we must use === for the test
-				$msg = sprintf(_('The following key has a wrong format: |%s|.  Please, correct it by going back to the previous page.'),
+				$error_msg = sprintf(_('The following key has a wrong format: |%s|.  Please, correct it by going back to the previous page.'),
 						htmlspecialchars($key));
-				session_redirect('/account/?&error_msg='.urlencode($msg));
+				session_redirect('/account/');
 			}
 		}
 		$key = strtok("\n");
diff --git a/src/common/include/exit.php b/src/common/include/exit.php
index ab45ae2..d011470 100644
--- a/src/common/include/exit.php
+++ b/src/common/include/exit.php
@@ -71,25 +71,23 @@ function exit_no_group() {
 /**
  * exit_missing_param() - Exit with missing required parameters error
  *
- * @param	string	$url	URL, usually $_SERVER['HTTP_REFERER']
+ * @param	string	$url		URL, usually $_SERVER['HTTP_REFERER']
  * @param	array	$missing_params	array of missing parameters
- * @param	string	$toptab needed for navigation
+ * @param	string	$toptab		needed for navigation
  */
 function exit_missing_param($url = '', $missing_params = array(), $toptab = '') {
+	global $error_msg;
 	if (!empty($missing_params)) {
-		$error = _('Missing Required Parameters')._(': ');
+		$error_msg = _('Missing Required Parameters')._(': ');
 		foreach ($missing_params as $missing_param) {
-			$error .= $missing_param.' ';
+			$error_msg .= $missing_param.' ';
 		}
 	} else {
-		$error = sprintf(_('Missing Required Parameters'));
+		$error_msg = sprintf(_('Missing Required Parameters'));
 	}
 	$redirect_url = util_find_relative_referer($url);
 	if (!empty($redirect_url)) {
-		if (strpos($redirect_url,'?')) {
-			session_redirect($redirect_url.'&error_msg='.urlencode($error));
-		}
-		session_redirect($redirect_url.'?error_msg='.urlencode($error));
+		session_redirect($redirect_url);
 	} else {
 		exit_error($error, $toptab);
 	}
diff --git a/src/common/tracker/actions/admin-updates.php b/src/common/tracker/actions/admin-updates.php
index ccc82a5..0e9c1da 100644
--- a/src/common/tracker/actions/admin-updates.php
+++ b/src/common/tracker/actions/admin-updates.php
@@ -87,7 +87,7 @@ if (getStringFromRequest('add_extrafield')) {
 			$ath->setBrowseList(join(',', $arr));
 			$feedback .= _('Custom Field Deleted');
 			$deleteextrafield=false;
-			session_redirect('/tracker/admin/?group_id='.$group_id.'&atid='.$atid.'&add_extrafield=1&feedback='.urlencode($feedback));
+			session_redirect('/tracker/admin/?group_id='.$group_id.'&atid='.$atid.'&add_extrafield=1');
 		}
 	}
 
@@ -550,7 +550,7 @@ if (getStringFromRequest('add_extrafield')) {
 	$atw = new ArtifactWorkflow($ath, $field_id);
 
 	if (!isset($wk[100])) {
-		$error_msg .= _('Error: Initial values not saved, no initial state given.').'<br />';
+		$error_msg .= _('Error')._(': ')._('Initial values not saved, no initial state given.').'<br />';
 	} else {
 		// Save values for the submit form (from=100).
 		$atw->saveNextNodes('100', array_keys($wk[100]));
diff --git a/src/common/tracker/actions/query.php b/src/common/tracker/actions/query.php
index 333e01f..823f7c3 100644
--- a/src/common/tracker/actions/query.php
+++ b/src/common/tracker/actions/query.php
@@ -74,7 +74,7 @@ if (getStringFromRequest('submit')) {
 		}
 		$aq->makeDefault();
 		$query_id=$aq->getID();
-		session_redirect('/tracker/?atid='.$atid.'&group_id='.$group_id.'&func=browse&feedback='.urlencode($feedback));
+		session_redirect('/tracker/?atid='.$atid.'&group_id='.$group_id.'&func=browse');
 	//
 /*
 	// Make the displayed query the default
@@ -123,7 +123,7 @@ if (getStringFromRequest('submit')) {
 		}
 		$aq->makeDefault();
 		$query_id=$aq->getID();
-		session_redirect('/tracker/?atid='.$atid.'&group_id='.$group_id.'&func=browse&feedback='.urlencode($feedback));
+		session_redirect('/tracker/?atid='.$atid.'&group_id='.$group_id.'&func=browse');
 	//
 	//	Just load the query
 	//
@@ -146,14 +146,11 @@ if (getStringFromRequest('submit')) {
 		}
 		if (!$aq->delete()) {
 			$error_msg .= $aq->getErrorMessage();
-            $ret_msg = '&error_msg='.urlencode($error_msg);
 		} else {
 			$feedback .= _('Query Deleted');;
-            $ret_msg = '&feedback='.urlencode($feedback);
 		}
 		$query_id=0;
-		session_redirect('/tracker/?atid='.$atid.'&group_id='.$group_id.'&func=browse'.$ret_msg);
-		exit;
+		session_redirect('/tracker/?atid='.$atid.'&group_id='.$group_id.'&func=browse');
 	} else {
 		exit_error(_('Missing Build Query Action'),'tracker');
 	}
diff --git a/src/plugins/blocks/www/index.php b/src/plugins/blocks/www/index.php
index 8d57c85..2a318d4 100644
--- a/src/plugins/blocks/www/index.php
+++ b/src/plugins/blocks/www/index.php
@@ -1,6 +1,6 @@
 <?php
 
-/*
+/**
  * Copyright (C) 2006 Alain Peyrat, Alcatel-Lucent
  * Copyright (C) 2010 Alain Peyrat <aljeux at free.fr>
  * Copyright (C) 2012-2014 Alain Peyrat - Alcatel-Lucent
@@ -266,8 +266,8 @@ if ($type == 'admin') {
 					array($activate[$b], $id, $b));
 		}
 	}
-	$msg = _('Block Saved');
-	session_redirect('/plugins/blocks/index.php?id='.$id.'&type=admin&pluginname=blocks&feedback='.urlencode($msg));
+	$feedback = _('Block Saved');
+	session_redirect('/plugins/blocks/index.php?id='.$id.'&type=admin&pluginname=blocks');
 } elseif ($type == 'configure') {
 	blocks_Project_Header(array('title'=>'Edit Block','pagename'=>"$pluginname",'sectionvals'=>array(group_getname($id))));
 	// DO THE STUFF FOR THE PROJECT ADMINISTRATION PART HERE
@@ -353,8 +353,8 @@ if ($type == 'admin') {
 				WHERE group_id=$2 AND name=$3',
 				array($body, $id, htmlspecialchars($name)));
 	}
-	$msg = $name .' : '. _('Block configuration saved');
-	session_redirect('/plugins/blocks/index.php?id='.$id.'&type=admin&pluginname=blocks&feedback='.urlencode($msg));
+	$feedback = $name .' : '. _('Block configuration saved');
+	session_redirect('/plugins/blocks/index.php?id='.$id.'&type=admin&pluginname=blocks');
 }
 
 site_project_footer();
diff --git a/src/plugins/extsubproj/actions/addExtSubProj.php b/src/plugins/extsubproj/actions/addExtSubProj.php
index 4d590a6..60e2b93 100644
--- a/src/plugins/extsubproj/actions/addExtSubProj.php
+++ b/src/plugins/extsubproj/actions/addExtSubProj.php
@@ -29,8 +29,8 @@ $confArr['newsubprojecturl'] = getStringFromRequest('newsubprojecturl');
 
 if (!$pluginExtSubProj->addExtSubProj($group_id, $confArr['newsubprojecturl'])) {
 	$error_msg = _('Failed to add subproject.');
-	session_redirect('/plugins/'.$pluginExtSubProj->name.'/?type=admin&group_id='.$group_id.'&pluginname='.$pluginExtSubProj->name.'&error_msg='.urlencode($error_msg));
+	session_redirect('/plugins/'.$pluginExtSubProj->name.'/?type=admin&group_id='.$group_id.'&pluginname='.$pluginExtSubProj->name);
 }
 
 $feedback = _('Subproject successfully added.');
-session_redirect('/plugins/'.$pluginExtSubProj->name.'/?type=admin&group_id='.$group_id.'&pluginname='.$pluginExtSubProj->name.'&feedback='.urlencode($feedback));
+session_redirect('/plugins/'.$pluginExtSubProj->name.'/?type=admin&group_id='.$group_id.'&pluginname='.$pluginExtSubProj->name);
diff --git a/src/plugins/extsubproj/actions/delExtSubProj.php b/src/plugins/extsubproj/actions/delExtSubProj.php
index 1027c8a..df74272 100644
--- a/src/plugins/extsubproj/actions/delExtSubProj.php
+++ b/src/plugins/extsubproj/actions/delExtSubProj.php
@@ -28,8 +28,8 @@ $url = urldecode(getStringFromRequest('url'));
 
 if (!$pluginExtSubProj->delExtSubProj($group_id, $url)) {
 	$error_msg = _('Failed to delete subproject.');
-	session_redirect('/plugins/'.$pluginExtSubProj->name.'/?type=admin&group_id='.$group_id.'&pluginname='.$pluginExtSubProj->name.'&error_msg='.urlencode($error_msg));
+	session_redirect('/plugins/'.$pluginExtSubProj->name.'/?type=admin&group_id='.$group_id.'&pluginname='.$pluginExtSubProj->name);
 }
 
 $feedback = _('Subproject successfully deleted.');
-session_redirect('/plugins/'.$pluginExtSubProj->name.'/?type=admin&group_id='.$group_id.'&pluginname='.$pluginExtSubProj->name.'&feedback='.urlencode($feedback));
+session_redirect('/plugins/'.$pluginExtSubProj->name.'/?type=admin&group_id='.$group_id.'&pluginname='.$pluginExtSubProj->name);
diff --git a/src/plugins/globaldashboard/www/admin/add_account.php b/src/plugins/globaldashboard/www/admin/add_account.php
index 681ea8d..e1048de 100644
--- a/src/plugins/globaldashboard/www/admin/add_account.php
+++ b/src/plugins/globaldashboard/www/admin/add_account.php
@@ -1,24 +1,25 @@
 <?php
 /**
-* Copyright 2011, Sabri LABBENE - Institut Télécom
-*
-*
-* This file is part of FusionForge.
-*
-* FusionForge is free software; you can redistribute it and/or modify
-* it under the terms of the GNU General Public License as published by
-* the Free Software Foundation; either version 2 of the License, or
-* (at your option) any later version.
-*
-* FusionForge is distributed in the hope that it will be useful,
-* but WITHOUT ANY WARRANTY; without even the implied warranty of
-* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-* GNU General Public License for more details.
-*
-* You should have received a copy of the GNU General Public License along
-* with this program; if not, write to the Free Software Foundation, Inc.,
-* 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
-*/
+ * Copyright 2011, Sabri LABBENE - Institut Télécom
+ * Copyright 2014, Franck Villaume - TrivialDev
+ * http://fusionforge.org
+ *
+ * This file is part of FusionForge.
+ *
+ * FusionForge is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * FusionForge is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License along
+ * with this program; if not, write to the Free Software Foundation, Inc.,
+ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+ */
 
 require_once '../../../env.inc.php';
 require_once $gfwww.'include/pre.php';
@@ -38,7 +39,8 @@ $artifacts_discovery = getIntFromPost('artifacts_discovery_method');
 
 $user = session_get_user();
 if($user->getID() != $user_id) {
-	session_redirect( '/plugins/globaldashboard/admin/manage_accounts.php?type=user&id='.$user_id.'&pluginname=globaldashboard&error_msg='. urlencode(_('You can add remote accounts ONLY for yourself !!!')));
+	$error_msg = _('You can add remote accounts ONLY for yourself !!!')
+	session_redirect( '/plugins/globaldashboard/admin/manage_accounts.php?type=user&id='.$user_id.'&pluginname=globaldashboard');
 }
 
 $t_account_table = "plugin_globaldashboard_user_forge_account";
@@ -67,10 +69,13 @@ if($result) {
 				."VALUES ( $1, $2, $3)";
 	$disc_result = db_query_params($t_disc_query, array(db_insertid($result, $t_account_table, 'account_id'), $projects_discovery, $artifacts_discovery));
 	if($disc_result) {
-		session_redirect( '/plugins/globaldashboard/admin/manage_accounts.php?type=user&id='.$user_id.'&pluginname=globaldashboard&feedback='. urlencode(_('Remote Account successfully created')));
+		$feedback = _('Remote Account successfully created')
+		session_redirect('/plugins/globaldashboard/admin/manage_accounts.php?type=user&id='.$user_id.'&pluginname=globaldashboard');
 	} else {
-		session_redirect( '/plugins/globaldashboard/admin/manage_accounts.php?type=user&id='.$user_id.'&pluginname=globaldashboard&error_msg='. urlencode(printf('Remote account created but unable to create remote associated discovery parameters: '.db_error())));
+		$error_msg = _('Remote account created but unable to create remote associated discovery parameters')._(': ').db_error();
+		session_redirect('/plugins/globaldashboard/admin/manage_accounts.php?type=user&id='.$user_id.'&pluginname=globaldashboard');
 	}
 } else {
-	session_redirect( '/plugins/globaldashboard/admin/manage_accounts.php?type=user&id='.$user_id.'&pluginname=globaldashboard&error_msg='. urlencode(printf('Unable to create remote account: '.db_error())));
+	$error_msg = _('Unable to create remote account')._(': ').db_error();
+	session_redirect('/plugins/globaldashboard/admin/manage_accounts.php?type=user&id='.$user_id.'&pluginname=globaldashboard');
 }
diff --git a/src/plugins/globaldashboard/www/admin/delete_account.php b/src/plugins/globaldashboard/www/admin/delete_account.php
index a28dc5a..28d79f3 100644
--- a/src/plugins/globaldashboard/www/admin/delete_account.php
+++ b/src/plugins/globaldashboard/www/admin/delete_account.php
@@ -1,24 +1,26 @@
 <?php
 /**
-* Copyright 2011, Sabri LABBENE - Institut Télécom
-*
-*
-* This file is part of FusionForge.
-*
-* FusionForge is free software; you can redistribute it and/or modify
-* it under the terms of the GNU General Public License as published by
-* the Free Software Foundation; either version 2 of the License, or
-* (at your option) any later version.
-*
-* FusionForge is distributed in the hope that it will be useful,
-* but WITHOUT ANY WARRANTY; without even the implied warranty of
-* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-* GNU General Public License for more details.
-*
-* You should have received a copy of the GNU General Public License along
-* with this program; if not, write to the Free Software Foundation, Inc.,
-* 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
-*/
+ * Copyright 2011, Sabri LABBENE - Institut Télécom
+ * Copyright 2014, Franck Villaume - TrivialDev
+ * http://fusionforge.org
+ *
+ *
+ * This file is part of FusionForge.
+ *
+ * FusionForge is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * FusionForge is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License along
+ * with this program; if not, write to the Free Software Foundation, Inc.,
+ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+ */
 
 require_once '../../../env.inc.php';
 require_once $gfwww.'include/pre.php';
@@ -28,7 +30,8 @@ $user_id = getStringFromRequest('user_id');
 
 $user = session_get_user();
 if($user->getID() != $user_id) {
-	session_redirect( '/plugins/globaldashboard/admin/manage_accounts.php?type=user&id='.$user_id.'&pluginname=globaldashboard&error_msg='. urlencode(_('You can remove only YOUR remote accounts !!!')));
+	$error_msg  = _('You can remove only YOUR remote accounts !!!');
+	session_redirect('/plugins/globaldashboard/admin/manage_accounts.php?type=user&id='.$user_id.'&pluginname=globaldashboard');
 }
 
 $t_account_table = "plugin_globaldashboard_user_forge_account";
@@ -36,7 +39,9 @@ $t_query = "DELETE FROM $t_account_table WHERE account_id=$1 AND user_id=$2";
 
 $result = db_query_params($t_query, array($account_id, $user_id));
 if ($result) {
-	session_redirect( '/plugins/globaldashboard/admin/manage_accounts.php?type=user&id='.$user_id.'&pluginname=globaldashboard&feedback='. urlencode(_('Remote Account successfully deleted')));
+	$feedback = _('Remote Account successfully deleted');
+	session_redirect('/plugins/globaldashboard/admin/manage_accounts.php?type=user&id='.$user_id.'&pluginname=globaldashboard');
 } else {
-	session_redirect( '/plugins/globaldashboard/admin/manage_accounts.php?type=user&id='.$user_id.'&pluginname=globaldashboard&error_msg='. urlencode(_('Unable to delete remote account: ').db_error()));
+	$error_msg = _('Unable to delete remote account')._(': ').db_error();
+	session_redirect('/plugins/globaldashboard/admin/manage_accounts.php?type=user&id='.$user_id.'&pluginname=globaldashboard');
 }
diff --git a/src/plugins/globaldashboard/www/admin/edit_account.php b/src/plugins/globaldashboard/www/admin/edit_account.php
index ccb4c6d..6037a13 100644
--- a/src/plugins/globaldashboard/www/admin/edit_account.php
+++ b/src/plugins/globaldashboard/www/admin/edit_account.php
@@ -1,24 +1,25 @@
 <?php
 /**
-* Copyright 2011, Sabri LABBENE - Institut Télécom
-*
-*
-* This file is part of FusionForge.
-*
-* FusionForge is free software; you can redistribute it and/or modify
-* it under the terms of the GNU General Public License as published by
-* the Free Software Foundation; either version 2 of the License, or
-* (at your option) any later version.
-*
-* FusionForge is distributed in the hope that it will be useful,
-* but WITHOUT ANY WARRANTY; without even the implied warranty of
-* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-* GNU General Public License for more details.
-*
-* You should have received a copy of the GNU General Public License along
-* with this program; if not, write to the Free Software Foundation, Inc.,
-* 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
-*/
+ * Copyright 2011, Sabri LABBENE - Institut Télécom
+ * Copyright 2014, Franck Villaume - TrivialDev
+ * http://fusionforge.org
+ *
+ * This file is part of FusionForge.
+ *
+ * FusionForge is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * FusionForge is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License along
+ * with this program; if not, write to the Free Software Foundation, Inc.,
+ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+ */
 
 require_once '../../../env.inc.php';
 require_once $gfwww.'include/pre.php';
@@ -39,7 +40,8 @@ $artifacts_discovery = getIntFromPost('artifacts_discovery_method');
 
 $user = session_get_user();
 if($user->getID() != $user_id) {
-	session_redirect( '/plugins/globaldashboard/admin/manage_accounts.php?type=user&id='.$user_id.'&pluginname=globaldashboard&error_msg='. urlencode(_('You can edit only YOUR remote accounts !!!')));
+	$error_msg = _('You can edit only YOUR remote accounts !!!');
+	session_redirect( '/plugins/globaldashboard/admin/manage_accounts.php?type=user&id='.$user_id.'&pluginname=globaldashboard');
 }
 
 $t_account_table = "plugin_globaldashboard_user_forge_account";
@@ -70,10 +72,13 @@ if ($result) {
 					" WHERE account_id=$3";
 	$disc_result = db_query_params($t_disc_query, array($projects_discovery, $artifacts_discovery, $account_id));
 	if ($disc_result){
-		session_redirect( '/plugins/globaldashboard/admin/manage_accounts.php?type=user&id='.$user_id.'&pluginname=globaldashboard&feedback='. _('Remote Account successfully updated'));
+		$feedback = _('Remote Account successfully updated');
+		session_redirect('/plugins/globaldashboard/admin/manage_accounts.php?type=user&id='.$user_id.'&pluginname=globaldashboard');
 	} else {
-		session_redirect( '/plugins/globaldashboard/admin/manage_accounts.php?type=user&id='.$user_id.'&pluginname=globaldashboard&feedback='.printf('Unable to update remote account: '.db_error()));
+		$error_msg = _('Unable to update remote account')._(': ').db_error();
+		session_redirect('/plugins/globaldashboard/admin/manage_accounts.php?type=user&id='.$user_id.'&pluginname=globaldashboard');
 	}
 } else {
-	session_redirect( '/plugins/globaldashboard/admin/manage_accounts.php?type=user&id='.$user_id.'&pluginname=globaldashboard&feedback='.printf('Unable to update remote account: '.db_error()));
+	$error_msg = _('Unable to update remote account')._(': ').db_error();
+	session_redirect('/plugins/globaldashboard/admin/manage_accounts.php?type=user&id='.$user_id.'&pluginname=globaldashboard');
 }
diff --git a/src/plugins/headermenu/action/addLink.php b/src/plugins/headermenu/action/addLink.php
index 061a7af..af235c8 100644
--- a/src/plugins/headermenu/action/addLink.php
+++ b/src/plugins/headermenu/action/addLink.php
@@ -46,17 +46,17 @@ if (!empty($name) && !empty($linkmenu)) {
 				if (util_check_url($link)) {
 					if ($headermenu->addLink($link, $name, $description, $linkmenu)) {
 						$feedback = _('Task succeeded.');
-						session_redirect($redirect_url.'&feedback='.urlencode($feedback));
+						session_redirect($redirect_url);
 					}
 					$error_msg = _('Task failed');
-					session_redirect($redirect_url.'&error_msg='.urlencode($error_msg));
+					session_redirect($redirect_url);
 				} else {
 					$error_msg = _('Provided Link is not a valid URL.');
-					session_redirect($redirect_url.'&error_msg='.urlencode($error_msg));
+					session_redirect($redirect_url));
 				}
 			}
 			$warning_msg = _('Missing Link URL.');
-			session_redirect($redirect_url.'&warning_msg='.urlencode($warning_msg));
+			session_redirect($redirect_url);
 			break;
 		}
 		case 'outermenu':
@@ -69,27 +69,27 @@ if (!empty($name) && !empty($linkmenu)) {
 					}
 					if ($headermenu->addLink($link, $name, $description, $linkmenu, $linktype, $group_id)) {
 						$feedback = _('Task succeeded.');
-						session_redirect($redirect_url.'&feedback='.urlencode($feedback));
+						session_redirect($redirect_url);
 					}
 					$error_msg = _('Task failed');
-					session_redirect($redirect_url.'&error_msg='.urlencode($error_msg));
+					session_redirect($redirect_url);
 				} else {
 					$error_msg = _('Provided Link is not a valid URL.');
-					session_redirect($redirect_url.'&error_msg='.urlencode($error_msg));
+					session_redirect($redirect_url);
 				}
 			}
 			if (!empty($htmlcode)) {
 				if ($headermenu->addLink('', $name, $description, $linkmenu, 'htmlcode', $group_id, $htmlcode)) {
 					$feedback = _('Task succeeded.');
-					session_redirect($redirect_url.'&feedback='.urlencode($feedback));
+					session_redirect($redirect_url);
 				}
 				$error_msg = _('Task failed');
-				session_redirect($redirect_url.'&error_msg='.urlencode($error_msg));
+				session_redirect($redirect_url);
 			}
 			$warning_msg = _('Missing Link URL or HTML Page.');
-			session_redirect($redirect_url.'&warning_msg='.urlencode($warning_msg));
+			session_redirect($redirect_url));
 		}
 	}
 }
 $warning_msg = _('No link to create or name missing.');
-session_redirect($redirect_url.'&warning_msg='.urlencode($warning_msg));
+session_redirect($redirect_url);
diff --git a/src/plugins/headermenu/action/deleteLink.php b/src/plugins/headermenu/action/deleteLink.php
index 3356fec..8962f8b 100644
--- a/src/plugins/headermenu/action/deleteLink.php
+++ b/src/plugins/headermenu/action/deleteLink.php
@@ -34,10 +34,10 @@ if (isset($group_id) && $group_id) {
 if (!empty($idLink)) {
 	if ($headermenu->deleteLink($idLink)) {
 		$feedback = _('Link deleted');
-		session_redirect($redirect_url.'&feedback='.urlencode($feedback));
+		session_redirect($redirect_url);
 	}
 	$error_msg = _('Task failed');
-	session_redirect($redirect_url.'&error_msg='.urlencode($error_msg));
+	session_redirect($redirect_url);
 }
 $warning_msg = _('Missing Link to be deleted.');
-session_redirect($redirect_url.'&warning_msg='.urlencode($warning_msg));
+session_redirect($redirect_url);
diff --git a/src/plugins/headermenu/action/updateLinkStatus.php b/src/plugins/headermenu/action/updateLinkStatus.php
index b91c682..488e4d6 100644
--- a/src/plugins/headermenu/action/updateLinkStatus.php
+++ b/src/plugins/headermenu/action/updateLinkStatus.php
@@ -35,10 +35,10 @@ if (isset($group_id) && $group_id) {
 if (!empty($idLink)) {
 	if ($headermenu->updateLinkStatus($idLink, $linkStatus)) {
 		$feedback = _('Link Status updated');
-		session_redirect($redirect_url.'&feedback='.urlencode($feedback));
+		session_redirect($redirect_url);
 	}
 	$error_msg = _('Task failed');
-	session_redirect($redirect_url.'&error_msg='.urlencode($error_msg));
+	session_redirect($redirect_url);
 }
 $warning_msg = _('Missing Link or status to be updated.');
-session_redirect($redirect_url.'&warning_msg='.urlencode($warning_msg));
+session_redirect($redirect_url);
diff --git a/src/plugins/headermenu/action/updateLinkValue.php b/src/plugins/headermenu/action/updateLinkValue.php
index f216236..f9dc149 100644
--- a/src/plugins/headermenu/action/updateLinkValue.php
+++ b/src/plugins/headermenu/action/updateLinkValue.php
@@ -45,17 +45,17 @@ if (!empty($idLink) && !empty($name)) {
 				if (util_check_url($link)) {
 					if ($headermenu->updateLink($idLink, $link, $name, $description, $linkmenu)) {
 						$feedback = _('Task succeeded.');
-						session_redirect($redirect_url.'&feedback='.urlencode($feedback));
+						session_redirect($redirect_url);
 					}
 					$error_msg = _('Task failed');
-					session_redirect($redirect_url.'&error_msg='.urlencode($error_msg));
+					session_redirect($redirect_url);
 				} else {
 					$error_msg = _('Provided Link is not a valid URL.');
-					session_redirect($redirect_url.'&error_msg='.urlencode($error_msg));
+					session_redirect($redirect_url);
 				}
 			}
 			$warning_msg = _('Missing Link URL.');
-			session_redirect($redirect_url.'&warning_msg='.urlencode($warning_msg));
+			session_redirect($redirect_url);
 			break;
 		}
 		case 'outermenu':
@@ -64,27 +64,27 @@ if (!empty($idLink) && !empty($name)) {
 				if (util_check_url($link)) {
 					if ($headermenu->updateLink($idLink, $link, $name, $description, $linkmenu, $typemenu)) {
 						$feedback = _('Task succeeded.');
-						session_redirect($redirect_url.'&feedback='.urlencode($feedback));
+						session_redirect($redirect_url);
 					}
 					$error_msg = _('Task failed');
-					session_redirect($redirect_url.'&error_msg='.urlencode($error_msg));
+					session_redirect($redirect_url);
 				} else {
 					$error_msg = _('Provided Link is not a valid URL.');
-					session_redirect($redirect_url.'&error_msg='.urlencode($error_msg));
+					session_redirect($redirect_url);
 				}
 			}
 			if (!empty($htmlcode) && $typemenu == 'htmlcode') {
 				if ($headermenu->updateLink($idLink, '', $name, $description, $linkmenu, 'htmlcode', $htmlcode)) {
 					$feedback = _('Task succeeded.');
-					session_redirect($redirect_url.'&feedback='.urlencode($feedback));
+					session_redirect($redirect_url);
 				}
 				$error_msg = _('Task failed');
-				session_redirect($redirect_url.'&error_msg='.urlencode($error_msg));
+				session_redirect($redirect_url);
 			}
 			$warning_msg = _('Missing Link URL or HTML Page.');
-			session_redirect($redirect_url.'&warning_msg='.urlencode($warning_msg));
+			session_redirect($redirect_url);
 		}
 	}
 }
 $warning_msg = _('No link to update or name missing.');
-session_redirect($redirect_url.'&warning_msg='.urlencode($warning_msg));
+session_redirect($redirect_url);
diff --git a/src/plugins/headermenu/view/admin/updateLinkValue.php b/src/plugins/headermenu/view/admin/updateLinkValue.php
index 39d2bdd..06270ee 100644
--- a/src/plugins/headermenu/view/admin/updateLinkValue.php
+++ b/src/plugins/headermenu/view/admin/updateLinkValue.php
@@ -109,6 +109,6 @@ if (is_array($linkValues)) {
 	echo $HTML->closeForm();
 	echo html_e('span', array(), sprintf(_('Fields marked with %s are mandatory.'), utils_requiredField()), false);
 } else {
-	$error_msg = _('Cannot retrieve value for this link:').' '.$linkId;
-	session_redirect($redirect_url.'&error_msg='.urlencode($error_msg));
+	$error_msg = _('Cannot retrieve value for this link')._(': ').$linkId;
+	session_redirect($redirect_url);
 }
diff --git a/src/plugins/mailman/www/admin/deletelist.php b/src/plugins/mailman/www/admin/deletelist.php
index e99b9fc..6610612 100644
--- a/src/plugins/mailman/www/admin/deletelist.php
+++ b/src/plugins/mailman/www/admin/deletelist.php
@@ -19,7 +19,7 @@ require_once '../mailman_utils.php';
 $request =& HTTPRequest::instance();
 $group_id = $request->get('group_id');
 
-$feedback = '';
+global $feedback;
 
 if (!$group_id) {
 	exit_no_group();
@@ -43,7 +43,8 @@ if ($request->exist('submit')) {
 	if (!$ml->deleteList($sure,$really_sure)) {
 		exit_error('Error',$ml->getErrorMessage());
 	} else {
-		htmlRedirect('/plugins/mailman/index.php?group_id='.$group_id.'&feedback=DELETED');
+		$feedback = _('Deleted')
+		session_redirect('/plugins/mailman/index.php?group_id='.$group_id);
 	}
 }
 
diff --git a/src/plugins/mantisbt/action/addAttachment.php b/src/plugins/mantisbt/action/addAttachment.php
index 5bf4304..97b04ac 100644
--- a/src/plugins/mantisbt/action/addAttachment.php
+++ b/src/plugins/mantisbt/action/addAttachment.php
@@ -36,6 +36,6 @@ try {
 	$feedback = _('Task succeeded.');
 } catch (SoapFault $soapFault) {
 	$error_msg = _('Task failed')._(': ').$soapFault->faultstring;
-	session_redirect('plugins/mantisbt/?type=group&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&idBug='.$idBug.'&view=viewIssue&error_msg='.urlencode($error_msg));
+	session_redirect('plugins/mantisbt/?type=group&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&idBug='.$idBug.'&view=viewIssue');
 }
-session_redirect('plugins/mantisbt/?type=group&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&idBug='.$idBug.'&view=viewIssue&feedback='.urlencode($feedback));
+session_redirect('plugins/mantisbt/?type=group&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&idBug='.$idBug.'&view=viewIssue');
diff --git a/src/plugins/mantisbt/action/addCategory.php b/src/plugins/mantisbt/action/addCategory.php
index 11538a1..1600115 100644
--- a/src/plugins/mantisbt/action/addCategory.php
+++ b/src/plugins/mantisbt/action/addCategory.php
@@ -37,10 +37,10 @@ if (!empty($nameCategory)) {
 		$clientSOAP->__soapCall('mc_project_add_category', array("username" => $username, "password" => $password, "p_project_id" => $mantisbtConf['id_mantisbt'], "p_category_name" => $nameCategory));
 	} catch (SoapFault $soapFault) {
 		$error_msg = _('Task failed')._(': ').$soapFault->faultstring;
-		session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&error_msg='.urlencode($error_msg));
+		session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name);
 	}
 	$feedback = _('Category added successfully');
-	session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&feedback='.urlencode($feedback));
+	session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name);
 }
 $warning_msg = _('Missing category name');
-session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&warning_msg='.urlencode($warning_msg));
+session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name);
diff --git a/src/plugins/mantisbt/action/addIssue.php b/src/plugins/mantisbt/action/addIssue.php
index 5864dc2..a1174af 100644
--- a/src/plugins/mantisbt/action/addIssue.php
+++ b/src/plugins/mantisbt/action/addIssue.php
@@ -3,7 +3,7 @@
  * MantisBT Plugin
  *
  * Copyright 2010-2011, Franck Villaume - Capgemini
- * Copyright 2012, Franck Villaume - TrivialDev
+ * Copyright 2012,2014 Franck Villaume - TrivialDev
  * http://fusionforge.org
  *
  * This file is part of FusionForge. FusionForge is free software;
@@ -45,7 +45,7 @@ try {
 	$listStatus= $clientSOAP->__soapCall('mc_enum_status', array("username" => $username, "password" => $password));
 } catch (SoapFault $soapFault) {
 	$error_msg = _('Task failed')._(': ').$soapFault->faultstring;
-	session_redirect('plugins/mantisbt/?type='.$type.'&group_id='.$group_id.'&pluginname=mantisbt&view=viewIssues&error_msg='.urlencode($feedback));
+	session_redirect('plugins/mantisbt/?type='.$type.'&group_id='.$group_id.'&pluginname=mantisbt&view=viewIssues');
 }
 foreach($listSeverities as $key => $severity){
 	if (getStringFromRequest('severite') == $severity->name){
@@ -123,9 +123,9 @@ $defect['version'] = getStringFromRequest('version');
 
 try {
 	$newIdBug = $clientSOAP->__soapCall('mc_issue_add', array("username" => $username, "password" => $password, "issue" => $defect));
-	$feedback = _('Ticket '.$newIdBug.' created successfully.');
-	session_redirect('plugins/mantisbt/?type='.$type.'&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&idBug='.$newIdBug.'&view=viewIssue&feedback='.urlencode($feedback));
+	$feedback = sprintf(_('Ticket %s created successfully.'), $newIdBug);
+	session_redirect('plugins/mantisbt/?type='.$type.'&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&idBug='.$newIdBug.'&view=viewIssue');
 } catch (SoapFault $soapFault) {
 	$error_msg = _('Task failed')._(': ').$soapFault->faultstring;
-	session_redirect('plugins/mantisbt/?type='.$type.'&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&error_msg='.urlencode($error_msg));
+	session_redirect('plugins/mantisbt/?type='.$type.'&group_id='.$group_id.'&pluginname='.$mantisbt->name);
 }
diff --git a/src/plugins/mantisbt/action/addNote.php b/src/plugins/mantisbt/action/addNote.php
index 17f58f2..64dc389 100644
--- a/src/plugins/mantisbt/action/addNote.php
+++ b/src/plugins/mantisbt/action/addNote.php
@@ -57,6 +57,6 @@ try {
 	$feedback = _('Note added successfully.');
 } catch (SoapFault $soapFault) {
 	$error_msg = _('Task failed')._(': ').$soapFault->faultstring;
-	session_redirect('plugins/mantisbt/?type=group&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&idBug='.$idBug.'&view=viewIssue&error_msg='.urlencode($error_msg));
+	session_redirect('plugins/mantisbt/?type=group&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&idBug='.$idBug.'&view=viewIssue');
 }
-session_redirect('plugins/mantisbt/?type=group&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&idBug='.$idBug.'&view=viewIssue&feedback='.urlencode($feedback));
+session_redirect('plugins/mantisbt/?type=group&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&idBug='.$idBug.'&view=viewIssue');
diff --git a/src/plugins/mantisbt/action/addVersion.php b/src/plugins/mantisbt/action/addVersion.php
index 855ac16..990f210 100644
--- a/src/plugins/mantisbt/action/addVersion.php
+++ b/src/plugins/mantisbt/action/addVersion.php
@@ -57,18 +57,18 @@ if (!empty($version)) {
 // 						$versionStruct['project_id'] = $child;
 // 						$clientSOAP->__soapCall('mc_project_version_add', array("username" => $username, "password" => $password, "version" => $versionStruct));
 // 					} catch (SoapFault $soapFault) {
-// 						$msg = _('Task failed')._(': ').$versionStruct['name'].' '.$soapFault->faultstring;
-// 						session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&error_msg='.urlencode($msg));
+// 						$error_msg = _('Task failed')._(': ').$versionStruct['name'].' '.$soapFault->faultstring;
+// 						session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name);
 // 					}
 // 				}
 // 			}
 // 		}
 	} catch (SoapFault $soapFault) {
-		$msg = _('Task failed')._(': ').$versionStruct['name'].' '.$soapFault->faultstring;
-		session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&error_msg='.urlencode($msg));
+		$error_msg = _('Task failed')._(': ').$versionStruct['name'].' '.$soapFault->faultstring;
+		session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name);
 	}
 	$feedback = _('Task succeeded.');
-	session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&feedback='.urlencode($feedback));
+	session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name);
 }
 $warning_msg = _('Missing version.');
-session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&warning_msg='.urlencode($warning_msg));
+session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name);
diff --git a/src/plugins/mantisbt/action/deleteAttachment.php b/src/plugins/mantisbt/action/deleteAttachment.php
index fc2e6f7..0a2044d 100644
--- a/src/plugins/mantisbt/action/deleteAttachment.php
+++ b/src/plugins/mantisbt/action/deleteAttachment.php
@@ -36,9 +36,9 @@ if ($idAttachment) {
 		$feedback = _('Attachment deleted successfully.');
 	} catch (SoapFault $soapFault) {
 		$error_msg = _('Task failed')._(': ').$soapFault->faultstring;
-		session_redirect('plugins/mantisbt/?type=group&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&idBug='.$idBug.'&view=viewIssue&error_msg='.urlencode($error_msg));
+		session_redirect('plugins/mantisbt/?type=group&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&idBug='.$idBug.'&view=viewIssue');
 	}
-	session_redirect('plugins/mantisbt/?type=group&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&idBug='.$idBug.'&view=viewIssue&feedback='.urlencode($feedback));
+	session_redirect('plugins/mantisbt/?type=group&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&idBug='.$idBug.'&view=viewIssue');
 }
 $warning_msg = _('Missing Attachment ID to delete.');
-session_redirect('plugins/mantisbt/?type=group&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&idBug='.$idBug.'&view=viewIssue&warning_msg='.urlencode($warning_msg));
+session_redirect('plugins/mantisbt/?type=group&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&idBug='.$idBug.'&view=viewIssue');
diff --git a/src/plugins/mantisbt/action/deleteCategory.php b/src/plugins/mantisbt/action/deleteCategory.php
index 10fb4d7..e59250d 100644
--- a/src/plugins/mantisbt/action/deleteCategory.php
+++ b/src/plugins/mantisbt/action/deleteCategory.php
@@ -3,7 +3,7 @@
  * MantisBT plugin
  *
  * Copyright 2010-2011, Franck Villaume - Capgemini
- * Copyright 2012, Franck Villaume - TrivialDev
+ * Copyright 2012,2014 Franck Villaume - TrivialDev
  * http://fusionforge.org
  *
  * This file is part of FusionForge. FusionForge is free software;
@@ -30,16 +30,16 @@ global $password;
 $deleteCategory = getStringFromRequest('deleteCategory');
 
 if ($deleteCategory) {
-    try {
-	    $clientSOAP = new SoapClient($mantisbtConf['url']."/api/soap/mantisconnect.php?wsdl", array('trace'=>true, 'exceptions'=>true));
-	    $clientSOAP->__soapCall('mc_project_delete_category', array("username" => $username, "password" => $password, "p_project_id" => $mantisbtConf['id_mantisbt'], "p_category_name" => $deleteCategory));
-    } catch (SoapFault $soapFault) {
-        $msg = _('Task failed')._(': ').$soapFault->faultstring;
-        session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&error_msg='.urlencode($msg));
-    }
-    $feedback = sprintf(_('Category %s deleted successfully.'), $deleteCategory);
-    session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&feedback='.urlencode($feedback));
+	try {
+		$clientSOAP = new SoapClient($mantisbtConf['url']."/api/soap/mantisconnect.php?wsdl", array('trace'=>true, 'exceptions'=>true));
+		$clientSOAP->__soapCall('mc_project_delete_category', array("username" => $username, "password" => $password, "p_project_id" => $mantisbtConf['id_mantisbt'], "p_category_name" => $deleteCategory));
+	} catch (SoapFault $soapFault) {
+		$error_msg = _('Task failed')._(': ').$soapFault->faultstring;
+		session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name);
+	}
+	$feedback = sprintf(_('Category %s deleted successfully.'), $deleteCategory);
+	session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name);
 } else {
-    $warning = _('Missing parameters to delete category.');
-    session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&warning_msg='.urlencode($warning));
+	$warning_msg = _('Missing parameters to delete category.');
+	session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name);
 }
diff --git a/src/plugins/mantisbt/action/deleteNote.php b/src/plugins/mantisbt/action/deleteNote.php
index b35958d..bcdf303 100644
--- a/src/plugins/mantisbt/action/deleteNote.php
+++ b/src/plugins/mantisbt/action/deleteNote.php
@@ -33,8 +33,8 @@ try {
 
 	$clientSOAP->__soapCall('mc_issue_note_delete', array("username" => $username, "password" => $password, "issue_note_id" => $idNote));
 } catch (SoapFault $soapFault) {
-	$feedback = _('Task failed')._(': ').$soapFault->faultstring;
-	session_redirect('plugins/mantisbt/?type=group&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&idBug='.$idBug.'&view=viewIssue&error_msg='.urlencode($feedback));
+	$error_msg = _('Task failed')._(': ').$soapFault->faultstring;
+	session_redirect('plugins/mantisbt/?type=group&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&idBug='.$idBug.'&view=viewIssue');
 }
 $feedback = _('Note deleted successfully');
-session_redirect('plugins/mantisbt/?type=group&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&idBug='.$idBug.'&view=viewIssue&feedback='.urlencode($feedback));
+session_redirect('plugins/mantisbt/?type=group&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&idBug='.$idBug.'&view=viewIssue');
diff --git a/src/plugins/mantisbt/action/deleteVersion.php b/src/plugins/mantisbt/action/deleteVersion.php
index e164cc2..eacffc8 100644
--- a/src/plugins/mantisbt/action/deleteVersion.php
+++ b/src/plugins/mantisbt/action/deleteVersion.php
@@ -3,7 +3,7 @@
  * MantisBT plugin
  *
  * Copyright 2010-2011, Franck Villaume - Capgemini
- * Copyright 2012, Franck Villaume - TrivialDev
+ * Copyright 2012,2014, Franck Villaume - TrivialDev
  * http://fusionforge.org
  *
  * This file is part of FusionForge. FusionForge is free software;
@@ -36,14 +36,15 @@ if (!empty($deleteVersion)) {
 	try {
 		if(!isset($clientSOAP))
 			$clientSOAP = new SoapClient($mantisbtConf['url']."/api/soap/mantisconnect.php?wsdl", array('trace'=>true, 'exceptions'=>true));
-	    $clientSOAP->__soapCall('mc_project_version_delete', array("username" => $username, "password" => $password, "version_id" => $deleteVersion));
+
+		$clientSOAP->__soapCall('mc_project_version_delete', array("username" => $username, "password" => $password, "version_id" => $deleteVersion));
 	} catch (SoapFault $soapFault) {
-		$msg = _('Task failed')._(': ').$soapFault->faultstring;
-		session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&error_msg='.urlencode($msg));
+		$error_msg = _('Task failed')._(': ').$soapFault->faultstring;
+		session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name);
 	}
 	$feedback = _('Version deleted successfully.');
-	session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&feedback='.urlencode($feedback));
+	session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name);
 } else {
-	$warning = _('Missing parameters to delete version.');
-	session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&warning_msg='.urlencode($warning));
+	$warning_msg = _('Missing parameters to delete version.');
+	session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name);
 }
diff --git a/src/plugins/mantisbt/action/init.php b/src/plugins/mantisbt/action/init.php
index 8ee595f..fea11bd 100644
--- a/src/plugins/mantisbt/action/init.php
+++ b/src/plugins/mantisbt/action/init.php
@@ -3,7 +3,7 @@
  * MantisBT plugin
  *
  * Copyright 2011, Franck Villaume - Capgemini
- * Copyright 2012, Franck Villaume - TrivialDev
+ * Copyright 2012,2014, Franck Villaume - TrivialDev
  * http://fusionforge.org
  *
  * This file is part of FusionForge. FusionForge is free software;
@@ -35,8 +35,10 @@ $confArr['soap_password'] = getStringFromRequest('soap_password');
 $confArr['mantisbtcreate'] = getIntFromRequest('mantisbtcreate');
 $confArr['mantisbtname'] = getStringFromRequest('mantisbtname');
 
-if (!$mantisbt->initialize($group_id, $confArr))
-	session_redirect('/plugins/'.$mantisbt->name.'/?type=group&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&error_msg='.urlencode($group->getErrorMessage()));
+if (!$mantisbt->initialize($group_id, $confArr)) {
+	$error_msg = $group->getErrorMessage();
+	session_redirect('/plugins/'.$mantisbt->name.'/?type=group&group_id='.$group_id.'&pluginname='.$mantisbt->name);
+}
 
 $feedback = _('MantisBT plugin successfully initialized.');
-session_redirect('/plugins/'.$mantisbt->name.'/?type=group&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&feedback='.urlencode($feedback));
+session_redirect('/plugins/'.$mantisbt->name.'/?type=group&group_id='.$group_id.'&pluginname='.$mantisbt->name);
diff --git a/src/plugins/mantisbt/action/inituser.php b/src/plugins/mantisbt/action/inituser.php
index b886e1b..a69e80b 100644
--- a/src/plugins/mantisbt/action/inituser.php
+++ b/src/plugins/mantisbt/action/inituser.php
@@ -32,8 +32,8 @@ $confArr['mantisbt_useglobal'] = getIntFromRequest('mantisbtuseglobal');
 
 if (!$mantisbt->initializeUser($confArr)) {
 	$error_msg = _('Failed to initialize user.').' '.$user->getErrorMessage();
-	session_redirect('/plugins/mantisbt/?type=user&pluginname='.$mantisbt->name.'&error_msg='.urlencode($error_msg));
+	session_redirect('/plugins/mantisbt/?type=user&pluginname='.$mantisbt->name);
 }
 
 $feedback = _('MantisBT User successfully initialized.');
-session_redirect('/plugins/mantisbt/?type=user&pluginname='.$mantisbt->name.'&feedback='.urlencode($feedback));
+session_redirect('/plugins/mantisbt/?type=user&pluginname='.$mantisbt->name);
diff --git a/src/plugins/mantisbt/action/renameCategory.php b/src/plugins/mantisbt/action/renameCategory.php
index 74fa7ed..63b74b5 100644
--- a/src/plugins/mantisbt/action/renameCategory.php
+++ b/src/plugins/mantisbt/action/renameCategory.php
@@ -35,17 +35,17 @@ $renameCategory = getStringFromRequest('renameCategory');
 if ($newCategoryName && $renameCategory) {
 	if ( $newCategoryName === $renameCategory ) {
 		$warning_msg = _('No action, same category name.');
-		session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&warning_msg='.urlencode($warning_msg));
+		session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name);
 	}
 	try {
 		$clientSOAP = new SoapClient($mantisbtConf['url']."/api/soap/mantisconnect.php?wsdl", array('trace'=>true, 'exceptions'=>true));
 		$clientSOAP->__soapCall('mc_project_rename_category_by_name', array("username" => $username, "password" => $password, "p_project_id" => $mantisbtConf['id_mantisbt'], "p_category_name" => $renameCategory, "p_category_name_new" => $newCategoryName, "p_assigned_to" => ''));
 	} catch (SoapFault $soapFault) {
-		$msg = _('Task failed')._(': ').$soapFault->faultstring;
-		session_redirect('plugins/mantisbt/?type=admin&id='.$id.'&pluginname='.$mantisbt->name.'&error_msg='.urlencode($msg));
+		$error_msg = _('Task failed')._(': ').$soapFault->faultstring;
+		session_redirect('plugins/mantisbt/?type=admin&id='.$id.'&pluginname='.$mantisbt->name);
 	}
 	$feedback = _('Category renamed successfully.');
-	session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&feedback='.urlencode($feedback));
+	session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name);
 }
 $warning_msg = _('Missing category name');
-session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&warning_msg='.urlencode($warning_msg));
+session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name);
diff --git a/src/plugins/mantisbt/action/updateConf.php b/src/plugins/mantisbt/action/updateConf.php
index 567753f..2d2fa5e 100644
--- a/src/plugins/mantisbt/action/updateConf.php
+++ b/src/plugins/mantisbt/action/updateConf.php
@@ -3,7 +3,7 @@
  * MantisBT plugin
  *
  * Copyright 2011, Franck Villaume - Capgemini
- * Copyright 2011-2012, Franck Villaume - TrivialDev
+ * Copyright 2011-2012,2014 Franck Villaume - TrivialDev
  * http://fusionforge.org
  *
  * This file is part of FusionForge. FusionForge is free software;
@@ -33,8 +33,10 @@ $confArr['soap_user'] = getStringFromRequest('soap_user');
 $confArr['soap_password'] = getStringFromRequest('soap_password');
 $confArr['global_conf'] = getIntFromRequest('global_conf');
 
-if (!$mantisbt->updateConf($group_id, $confArr))
-	session_redirect('/plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&error_msg='.urlencode($group->getErrorMessage()));
+if (!$mantisbt->updateConf($group_id, $confArr)) {
+	$error_msg = $group->getErrorMessage();
+	session_redirect('/plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name);
+}
 
 $feedback = _('MantisBT configuration successfully updated.');
-session_redirect('/plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&feedback='.urlencode($feedback));
+session_redirect('/plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name);
diff --git a/src/plugins/mantisbt/action/updateGlobalConf.php b/src/plugins/mantisbt/action/updateGlobalConf.php
index f7f101b..a80c6be 100644
--- a/src/plugins/mantisbt/action/updateGlobalConf.php
+++ b/src/plugins/mantisbt/action/updateGlobalConf.php
@@ -31,8 +31,8 @@ $confArr['soap_password'] = getStringFromRequest('soap_password');
 
 if (!$mantisbt->updateGlobalConf($confArr)) {
 	$error_msg = _('Failed to update global configuration.');
-	session_redirect('/plugins/mantisbt/?type=globaladmin&pluginname='.$mantisbt->name.'&error_msg='.urlencode($error_msg));
+	session_redirect('/plugins/mantisbt/?type=globaladmin&pluginname='.$mantisbt->name);
 }
 
 $feedback = _('MantisBT global configuration successfully updated.');
-session_redirect('/plugins/mantisbt/?type=globaladmin&pluginname='.$mantisbt->name.'&feedback='.urlencode($feedback));
+session_redirect('/plugins/mantisbt/?type=globaladmin&pluginname='.$mantisbt->name);
diff --git a/src/plugins/mantisbt/action/updateIssue.php b/src/plugins/mantisbt/action/updateIssue.php
index 73d5dfd..a7b1266 100644
--- a/src/plugins/mantisbt/action/updateIssue.php
+++ b/src/plugins/mantisbt/action/updateIssue.php
@@ -43,7 +43,7 @@ switch ($type) {
 	}
 	default: {
 		$error_msg = _('No type found.');
-		session_redirect('plugins/mantisbt/&error_msg='.urlencode($error_msg));
+		session_redirect('plugins/mantisbt/');
 	}
 }
 
@@ -177,8 +177,8 @@ try {
 	$clientSOAP->__soapCall('mc_issue_update', array("username" => $username, "password" => $password, "issue_id" => $idBug, "issue" => $defect));
 } catch (SoapFault $soapFault) {
 	$error_msg = _('Task failed')._(': ').$soapFault->faultstring;
-	session_redirect($redirect_url.'&pluginname='.$mantisbt->name.'&idBug='.$idBug.'&view=viewIssue&error_msg='.urlencode($feedback));
+	session_redirect($redirect_url.'&pluginname='.$mantisbt->name.'&idBug='.$idBug.'&view=viewIssue');
 }
 
 $feedback = _('Task succeeded.');
-session_redirect($redirect_url.'&pluginname='.$mantisbt->name.'&idBug='.$idBug.'&view=viewIssue&feedback='.urlencode($feedback));
+session_redirect($redirect_url.'&pluginname='.$mantisbt->name.'&idBug='.$idBug.'&view=viewIssue');
diff --git a/src/plugins/mantisbt/action/updateNote.php b/src/plugins/mantisbt/action/updateNote.php
index a118af5..921711e 100644
--- a/src/plugins/mantisbt/action/updateNote.php
+++ b/src/plugins/mantisbt/action/updateNote.php
@@ -35,7 +35,7 @@ try {
 	$listViewStates = $clientSOAP->__soapCall('mc_enum_view_states', array("username" => $username, "password" => $password));
 } catch (SoapFault $soapFault) {
 	$error_msg = _('Task failed')._(': ').$soapFault->faultstring;
-	session_redirect('plugins/mantisbt/?type=group&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&idBug='.$idBug.'&view=viewIssue&error_msg='.urlencode($error_msg));
+	session_redirect('plugins/mantisbt/?type=group&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&idBug='.$idBug.'&view=viewIssue');
 }
 
 foreach($defect->notes as $key => $note){
@@ -59,8 +59,8 @@ try {
 	$clientSOAP->__soapCall('mc_issue_note_update', array("username" => $username, "password" => $password, "note" => $noteEdit));
 } catch (SoapFault $soapFault) {
 	$error_msg = _('Task failed')._(': ').$soapFault->faultstring;
-	session_redirect('plugins/mantisbt/?type=group&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&idBug='.$idBug.'&view=viewIssue&error_msg='.urlencode($error_msg));
+	session_redirect('plugins/mantisbt/?type=group&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&idBug='.$idBug.'&view=viewIssue');
 }
 
 $feedback = _('Task succeeded.');
-session_redirect('plugins/mantisbt/?type=group&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&idBug='.$idBug.'&view=viewIssue&feedback='.urlencode($feedback));
+session_redirect('plugins/mantisbt/?type=group&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&idBug='.$idBug.'&view=viewIssue');
diff --git a/src/plugins/mantisbt/action/updateVersion.php b/src/plugins/mantisbt/action/updateVersion.php
index 5edeced..bf3dc88 100644
--- a/src/plugins/mantisbt/action/updateVersion.php
+++ b/src/plugins/mantisbt/action/updateVersion.php
@@ -57,7 +57,7 @@ try {
 //     }
 } catch (SoapFault $soapFault) {
 	$error_msg = _('Task failed')._(': ').$version_data['name'].' '.$soapFault->faultstring;
-	session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&error_msg='.urlencode($error_msg));
+	session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name);
 }
 $feedback = _('Task succeeded.');
-session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&feedback='.urlencode($feedback));
+session_redirect('plugins/mantisbt/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name);
diff --git a/src/plugins/mantisbt/action/updateuserConf.php b/src/plugins/mantisbt/action/updateuserConf.php
index 2f1bfb5..ca90f09 100644
--- a/src/plugins/mantisbt/action/updateuserConf.php
+++ b/src/plugins/mantisbt/action/updateuserConf.php
@@ -3,6 +3,7 @@
  * MantisBT plugin
  *
  * Copyright 2011, Franck Villaume - Capgemini
+ * Copyright 2014, Franck Villaume - TrivialDev
  * http://fusionforge.org
  *
  * This file is part of FusionForge. FusionForge is free software;
@@ -28,8 +29,10 @@ $confArr = array();
 $confArr['mantisbt_user'] = getStringFromRequest('mantisbt_user');
 $confArr['mantisbt_password'] = getStringFromRequest('mantisbt_password');
 
-if (!$mantisbt->updateUserConf($confArr))
-	session_redirect('/plugins/mantisbt/?type=user&pluginname='.$mantisbt->name.'&error_msg='.urlencode($user->getErrorMessage()));
+if (!$mantisbt->updateUserConf($confArr)) {
+	$error_msg = $user->getErrorMessage();
+	session_redirect('/plugins/mantisbt/?type=user&pluginname='.$mantisbt->name);
+}
 
 $feedback = _('MantisBT User configuration successfully updated.');
-session_redirect('/plugins/mantisbt/?type=user&pluginname='.$mantisbt->name.'&view=adminuser&feedback='.urlencode($feedback));
+session_redirect('/plugins/mantisbt/?type=user&pluginname='.$mantisbt->name.'&view=adminuser');
diff --git a/src/plugins/mantisbt/www/getAttachment.php b/src/plugins/mantisbt/www/getAttachment.php
index 151f786..55368ce 100644
--- a/src/plugins/mantisbt/www/getAttachment.php
+++ b/src/plugins/mantisbt/www/getAttachment.php
@@ -74,7 +74,7 @@ if ($mantisbtUserConf) {
 // no user init ? we shoud force this user to init his account
 if (!isset($username) || !isset($password)) {
 	$warning_msg = _('Your mantisbt user is not initialized.');
-	session_redirect('/plugins/'.$mantisbt->name.'/?type=user&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&view=inituser&warning_msg='.urlencode($warning_msg));
+	session_redirect('/plugins/'.$mantisbt->name.'/?type=user&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&view=inituser');
 }
 
 if ($idAttachment) {
@@ -82,7 +82,8 @@ if ($idAttachment) {
 		$clientSOAP = new SoapClient($mantisbtConf['url']."/api/soap/mantisconnect.php?wsdl", array('trace'=>true, 'exceptions'=>true));
 		$content = $clientSOAP->__soapCall('mc_issue_attachment_get', array("username" => $username, "password" => $password, "issue_attachment_id" => $idAttachment));
 	} catch (SoapFault $soapFault) {
-		session_redirect('plugins/mantisbt/?type=group&group_id='.$group_id.'&pluginname=mantisbt&error_msg='.urlencode($soapFault->faultstring));
+		$error_msg = $soapFault->faultstring
+		session_redirect('plugins/mantisbt/?type=group&group_id='.$group_id.'&pluginname=mantisbt');
 	}
 
 	header( 'Content-Disposition: filename="'.urlencode($arr[6]).'"' );
diff --git a/src/plugins/mantisbt/www/index.php b/src/plugins/mantisbt/www/index.php
index 301cc4d..90acfa6 100644
--- a/src/plugins/mantisbt/www/index.php
+++ b/src/plugins/mantisbt/www/index.php
@@ -72,10 +72,7 @@ switch ($type) {
 		$view = getStringFromRequest('view');
 		if ($mantisbtConf['id_mantisbt'] === 0) {
 			$warning_msg = _('The mantisbt plugin for this project is not initialized.');
-			$redirect_url = '/plugins/'.$mantisbt->name.'/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&view=init&warning_msg='.urlencode($warning_msg);
-			if ($error_msg) {
-				$redirect_url .= '&error_msg='.urlencode($error_msg);
-			}
+			$redirect_url = '/plugins/'.$mantisbt->name.'/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&view=init';
 			session_redirect($redirect_url);
 		}
 
@@ -89,7 +86,7 @@ switch ($type) {
 					$password = $mantisbtUserConf['password'];
 				} else {
 					$warning_msg = _('Your mantisbt user is not initialized.');
-					session_redirect('/plugins/'.$mantisbt->name.'/?type=user&pluginname='.$mantisbt->name.'&view=inituser&warning_msg='.urlencode($warning_msg));
+					session_redirect('/plugins/'.$mantisbt->name.'/?type=user&pluginname='.$mantisbt->name.'&view=inituser');
 				}
 				$action = getStringFromRequest('action');
 			}
@@ -168,10 +165,7 @@ switch ($type) {
 				$password = $mantisbtConf['password'];
 			}  else {
 				$warning_msg = _('Your mantisbt user is not initialized.');
-				$redirect_url = '/plugins/'.$mantisbt->name.'/?type=user&pluginname='.$mantisbt->name.'&view=inituser&warning_msg='.urlencode($warning_msg);
-				if ($error_msg) {
-					$redirect_url .= '&error_msg='.urlencode($error_msg);
-				}
+				$redirect_url = '/plugins/'.$mantisbt->name.'/?type=user&pluginname='.$mantisbt->name.'&view=inituser';
 				session_redirect($redirect_url);
 			}
 		}
@@ -238,10 +232,7 @@ switch ($type) {
 		if ($view != 'init' && $action != 'init') {
 			if ($mantisbtConf['id_mantisbt'] === 0) {
 				$warning_msg = _('The mantisbt plugin for this project is not initialized.');
-				$redirect_url = '/plugins/'.$mantisbt->name.'/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&view=init&warning_msg='.urlencode($warning_msg);
-				if ($error_msg) {
-					$redirect_url .= '&error_msg='.urlencode($error_msg);
-				}
+				$redirect_url = '/plugins/'.$mantisbt->name.'/?type=admin&group_id='.$group_id.'&pluginname='.$mantisbt->name.'&view=init';
 				session_redirect($redirect_url);
 			}
 
@@ -256,7 +247,7 @@ switch ($type) {
 			// no user init ? we shoud force this user to init his account
 			if (!isset($username) || !isset($password)) {
 				$warning_msg = _('Your mantisbt user is not initialized.');
-				session_redirect('/plugins/'.$mantisbt->name.'/?type=user&pluginname='.$mantisbt->name.'&view=inituser&warning_msg='.urlencode($warning_msg));
+				session_redirect('/plugins/'.$mantisbt->name.'/?type=user&pluginname='.$mantisbt->name.'&view=inituser');
 			}
 		}
 
diff --git a/src/plugins/projects-hierarchy/actions/updateGlobalConf.php b/src/plugins/projects-hierarchy/actions/updateGlobalConf.php
index f191020..29c2444 100644
--- a/src/plugins/projects-hierarchy/actions/updateGlobalConf.php
+++ b/src/plugins/projects-hierarchy/actions/updateGlobalConf.php
@@ -31,8 +31,8 @@ $confArr['delegate'] = 0;
 
 if (!$projectsHierarchy->updateGlobalConf($confArr)) {
 	$error_msg = _('Failed to update global configuration.');
-	session_redirect('/plugins/'.$projectsHierarchy->name.'/?type=globaladmin&error_msg='.urlencode($error_msg));
+	session_redirect('/plugins/'.$projectsHierarchy->name.'/?type=globaladmin');
 }
 
 $feedback = _('Projects Hierarchy global configuration successfully updated.');
-session_redirect('/plugins/'.$projectsHierarchy->name.'/?type=globaladmin&feedback='.urlencode($feedback));
+session_redirect('/plugins/'.$projectsHierarchy->name.'/?type=globaladmin');
diff --git a/src/plugins/projects-hierarchy/actions/updateProjectConf.php b/src/plugins/projects-hierarchy/actions/updateProjectConf.php
index 123aa4e..5ae5a36 100644
--- a/src/plugins/projects-hierarchy/actions/updateProjectConf.php
+++ b/src/plugins/projects-hierarchy/actions/updateProjectConf.php
@@ -32,8 +32,8 @@ $confArr['globalconf'] = getIntFromRequest('globalconf');
 
 if (!$projectsHierarchy->updateConf($group_id, $confArr)) {
 	$error_msg = _('Failed to update configuration.');
-	session_redirect('/plugins/'.$projectsHierarchy->name.'/?type=admin&group_id='.$group_id.'&pluginname='.$projectsHierarchy->name.'&error_msg='.urlencode($error_msg));
+	session_redirect('/plugins/'.$projectsHierarchy->name.'/?type=admin&group_id='.$group_id.'&pluginname='.$projectsHierarchy->name);
 }
 
 $feedback = _('Projects Hierarchy configuration successfully updated.');
-session_redirect('/plugins/'.$projectsHierarchy->name.'/?type=admin&group_id='.$group_id.'&pluginname='.$projectsHierarchy->name.'&feedback='.urlencode($feedback));
+session_redirect('/plugins/'.$projectsHierarchy->name.'/?type=admin&group_id='.$group_id.'&pluginname='.$projectsHierarchy->name);
diff --git a/src/plugins/projects-hierarchy/common/projects-hierarchyPlugin.class.php b/src/plugins/projects-hierarchy/common/projects-hierarchyPlugin.class.php
index a0382c1..c415902 100644
--- a/src/plugins/projects-hierarchy/common/projects-hierarchyPlugin.class.php
+++ b/src/plugins/projects-hierarchy/common/projects-hierarchyPlugin.class.php
@@ -330,20 +330,21 @@ class projects_hierarchyPlugin extends Plugin {
 	 */
 	function redirect($http_referer, $type, $message) {
 		switch ($type) {
-			case 'warning_msg':
-			case 'error_msg':
+			case 'warning_msg': {
+				$warning_msg = $message;
+				break;
+			}
 			case 'feedback': {
+				$feedback = $message;
 				break;
 			}
+			case 'error_msg':
 			default: {
-				$type = 'error_msg';
+				$error_msg = $message;
 			}
 		}
 		$url = util_find_relative_referer($http_referer);
-		if (strpos($url,'?')) {
-			session_redirect($url.'&'.$type.'='.urlencode($message));
-		}
-		session_redirect($url.'?'.$type.'='.urlencode($message));
+		session_redirect($url);
 	}
 
 	/**
diff --git a/src/plugins/webanalytics/action/addLink.php b/src/plugins/webanalytics/action/addLink.php
index f14b5cf..48a5f16 100644
--- a/src/plugins/webanalytics/action/addLink.php
+++ b/src/plugins/webanalytics/action/addLink.php
@@ -30,10 +30,10 @@ $name = getStringFromRequest('name');
 if (!empty($link) && !empty($name)) {
 	if ($webanalytics->addLink($link, $name)) {
 		$feedback = _('Task succeeded.');
-		session_redirect('plugins/'.$webanalytics->name.'/?type=globaladmin&feedback='.urlencode($feedback));
+		session_redirect('plugins/'.$webanalytics->name.'/?type=globaladmin');
 	}
 	$error_msg = _('Task failed');
-	session_redirect('plugins/'.$webanalytics->name.'/?type=globaladmin&error_msg='.urlencode($error_msg));
+	session_redirect('plugins/'.$webanalytics->name.'/?type=globaladmin');
 }
 $warning_msg = _('Missing Link URL or name.');
-session_redirect('plugins/'.$webanalytics->name.'/?type=globaladmin&warning_msg='.urlencode($warning_msg));
+session_redirect('plugins/'.$webanalytics->name.'/?type=globaladmin');
diff --git a/src/plugins/webanalytics/action/deleteLink.php b/src/plugins/webanalytics/action/deleteLink.php
index fbbf2e4..9af8a15 100644
--- a/src/plugins/webanalytics/action/deleteLink.php
+++ b/src/plugins/webanalytics/action/deleteLink.php
@@ -29,10 +29,10 @@ $idLink = getIntFromRequest('linkid');
 if (!empty($idLink)) {
 	if ($webanalytics->deleteLink($idLink)) {
 		$feedback = _('Link deleted');
-		session_redirect('plugins/'.$webanalytics->name.'/?type=globaladmin&feedback='.urlencode($feedback));
+		session_redirect('plugins/'.$webanalytics->name.'/?type=globaladmin');
 	}
 	$error_msg = _('Task failed');
-	session_redirect('plugins/'.$webanalytics->name.'/?type=globaladmin&error_msg='.urlencode($error_msg));
+	session_redirect('plugins/'.$webanalytics->name.'/?type=globaladmin');
 }
 $warning_msg = _('Missing Link to be deleted.');
-session_redirect('plugins/'.$webanalytics->name.'/?type=globaladmin&warning_msg='.urlencode($warning_msg));
+session_redirect('plugins/'.$webanalytics->name.'/?type=globaladmin');
diff --git a/src/plugins/webanalytics/action/updateLinkStatus.php b/src/plugins/webanalytics/action/updateLinkStatus.php
index 9875922..160b769 100644
--- a/src/plugins/webanalytics/action/updateLinkStatus.php
+++ b/src/plugins/webanalytics/action/updateLinkStatus.php
@@ -30,10 +30,10 @@ $linkStatus = getIntFromRequest('linkstatus');
 if (!empty($idLink)) {
 	if ($webanalytics->updateLinkStatus($idLink, $linkStatus)) {
 		$feedback = _('Link Status updated');
-		session_redirect('plugins/'.$webanalytics->name.'/?type=globaladmin&feedback='.urlencode($feedback));
+		session_redirect('plugins/'.$webanalytics->name.'/?type=globaladmin');
 	}
 	$error_msg = _('Task failed');
-	session_redirect('plugins/'.$webanalytics->name.'/?type=globaladmin&error_msg='.urlencode($error_msg));
+	session_redirect('plugins/'.$webanalytics->name.'/?type=globaladmin');
 }
 $warning_msg = _('Missing Link or status to be updated.');
-session_redirect('plugins/'.$webanalytics->name.'/?type=globaladmin&warning_msg='.urlencode($warning_msg));
+session_redirect('plugins/'.$webanalytics->name.'/?type=globaladmin');
diff --git a/src/plugins/webanalytics/action/updateLinkValue.php b/src/plugins/webanalytics/action/updateLinkValue.php
index d83bef5..1afc9a2 100644
--- a/src/plugins/webanalytics/action/updateLinkValue.php
+++ b/src/plugins/webanalytics/action/updateLinkValue.php
@@ -31,8 +31,8 @@ $name = getStringFromRequest('name');
 if (!empty($idLink) && !empty($link)) {
 	if ($webanalytics->updateLink($idLink, $link, $name)) {
 		$feedback = _('Link updated');
-		session_redirect('plugins/'.$webanalytics->name.'/?type=globaladmin&feedback='.urlencode($feedback));
+		session_redirect('plugins/'.$webanalytics->name.'/?type=globaladmin');
 	}
 	$error_msg = _('Task failed');
-	session_redirect('plugins/'.$webanalytics->name.'/?type=globaladmin&error_msg='.urlencode($error_msg));
+	session_redirect('plugins/'.$webanalytics->name.'/?type=globaladmin');
 }
diff --git a/src/plugins/webanalytics/view/admin/updateLinkValue.php b/src/plugins/webanalytics/view/admin/updateLinkValue.php
index 1df29c5..85615c5 100644
--- a/src/plugins/webanalytics/view/admin/updateLinkValue.php
+++ b/src/plugins/webanalytics/view/admin/updateLinkValue.php
@@ -51,5 +51,5 @@ if (is_array($linkValues)) {
 	echo $HTML->addRequiredFieldsInfoBox();
 } else {
 	$error_msg = _('Cannot retrieve value for this link:').' '.$linkId;
-	session_redirect('plugins/'.$webanalytics->name.'/?type=globaladmin&error_msg='.urlencode($error_msg));
+	session_redirect('plugins/'.$webanalytics->name.'/?type=globaladmin');
 }
diff --git a/src/www/admin/globalroledelete.php b/src/www/admin/globalroledelete.php
index 5f29dbf..a714236 100644
--- a/src/www/admin/globalroledelete.php
+++ b/src/www/admin/globalroledelete.php
@@ -3,6 +3,8 @@
  * Global Role Delete Page
  *
  * Copyright 2011, Roland Mas
+ * Copyright 2014, Franck Villaume - TrivialDev
+ * http://fusionforge.org
  *
  * This file is part of FusionForge. FusionForge is free software;
  * you can redistribute it and/or modify it under the terms of the
@@ -23,12 +25,14 @@
 require_once '../env.inc.php';
 require_once $gfcommon.'include/pre.php';
 
+global $error_msg;
+
 $role_id = getIntFromRequest('role_id');
 
 session_require_global_perm ('forge_admin') ;
 
 if (!$role_id) {
-	session_redirect('/admin');
+	session_redirect('/admin/');
 }
 
 $role = RBACEngine::getInstance()->getRoleById($role_id);
@@ -44,17 +48,16 @@ if ($role->getHomeProject() != NULL) {
 }
 
 if (getStringFromRequest('submit')) {
-	$error_msg = '';
 	if (getIntFromRequest('sure')) {
 		if (!$role->delete()) {
 			$error_msg = _('Error')._(': ').$role->getErrorMessage();
 		} else {
 			$feedback = _('Successfully Deleted Role');
-			session_redirect('/admin/index.php?feedback='.urlencode($feedback));
+			session_redirect('/admin/');
 		}
 	} else {
-		$error_msg = _('Error: Please confirm the deletion of the role.');
+		$error_msg = _('Error')._(': ')._('Please confirm the deletion of the role.');
 	}
 
-	session_redirect('/admin/globalroleedit.php?role_id='.$role_id.'&error_msg='.urlencode($error_msg));
+	session_redirect('/admin/globalroleedit.php?role_id='.$role_id);
 }
diff --git a/src/www/docman/view.php b/src/www/docman/view.php
index 6131789..564ff4f 100644
--- a/src/www/docman/view.php
+++ b/src/www/docman/view.php
@@ -138,16 +138,16 @@ if (is_numeric($docid)) {
 			if(!readfile_chunked($file)) {
 				@unlink($file);
 				$error_msg = _('Unable to download backup file');
-				session_redirect('/docman/?group_id='.$group_id.'&view=admin&error_msg='.urlencode($error_msg));
+				session_redirect('/docman/?group_id='.$group_id.'&view=admin');
 			}
 			@unlink($file);
 		} else {
 			$warning_msg = _('No documents to backup.');
-			session_redirect('/docman/?group_id='.$group_id.'&view=admin&warning_msg='.urlencode($warning_msg));
+			session_redirect('/docman/?group_id='.$group_id.'&view=admin');
 		}
 	} else {
 		$warning_msg = _('ZIP extension is missing: no backup function');
-		session_redirect('/docman/?group_id='.$group_id.'&view=admin&warning_msg='.urlencode($warning_msg));
+		session_redirect('/docman/?group_id='.$group_id.'&view=admin');
 	}
 } elseif ($docid === 'webdav') {
 	if (forge_get_config('use_webdav') && $g->useWebDav()) {
@@ -165,7 +165,7 @@ if (is_numeric($docid)) {
 		$server->ServeRequest();
 	} else {
 		$warning_msg = _('No Webdav interface enabled.');
-		session_redirect('/docman/?group_id='.$group_id.'&warning_msg='.urlencode($warning_msg));
+		session_redirect('/docman/?group_id='.$group_id);
 	}
 } elseif ($docid === 'zip') {
 	session_require_perm('docman', $group_id, 'read');
@@ -227,12 +227,12 @@ if (is_numeric($docid)) {
 				if(!readfile_chunked($file)) {
 					unlink($file);
 					$error_msg = _('Unable to download ZIP archive');
-					session_redirect('/docman/?group_id='.$group_id.'&view=admin&error_msg='.urlencode($error_msg));
+					session_redirect('/docman/?group_id='.$group_id.'&view=admin');
 				}
 				unlink($file);
 			} else {
 				$warning_msg = _('This documents folder is empty.');
-				session_redirect('/docman/?group_id='.$group_id.'&view=listfile&dirid='.$dirid.'&warning_msg='.urlencode($warning_msg));
+				session_redirect('/docman/?group_id='.$group_id.'&view=listfile&dirid='.$dirid);
 			}
 		} elseif ( $arr[5] === 'selected' ) {
 			$dirid = $arr[6];
@@ -267,7 +267,7 @@ if (is_numeric($docid)) {
 					$zip->close();
 					unlink($file);
 					$warning_msg = _('No action to perform');
-					session_redirect('/docman/?group_id='.$group_id.'&view=listfile&dirid='.$dirid.'&warning_msg='.urlencode($warning_msg));
+					session_redirect('/docman/?group_id='.$group_id.'&view=listfile&dirid='.$dirid);
 				}
 			}
 			if ( !$zip->close()) {
@@ -283,7 +283,7 @@ if (is_numeric($docid)) {
 			if(!readfile_chunked($file)) {
 				unlink($file);
 				$error_msg = _('Unable to download ZIP archive');
-				session_redirect('/docman/?group_id='.$group_id.'&view=admin&error_msg='.urlencode($error_msg));
+				session_redirect('/docman/?group_id='.$group_id.'&view=admin');
 			}
 			unlink($file);
 		} else {
diff --git a/src/www/forum/admin/index.php b/src/www/forum/admin/index.php
index 9ffb643..5382db4 100644
--- a/src/www/forum/admin/index.php
+++ b/src/www/forum/admin/index.php
@@ -270,7 +270,8 @@ if (getStringFromRequest('add_forum')) {
 		if ($fm->updatemsg($forum_id,$posted_by,$subject,$body,$post_date,$is_followup_to,$thread_id,$has_followups,$most_recent_date)) {
 			$feedback .= _('Message Edited Successfully');
 		} else {
-			session_redirect('/forum/admin/index.php?editmsg='.$msg_id.'&group_id='.$group_id.'&thread_id='.$thread_id.'&forum_id='.$forum_id.'&error_msg='.urlencode($fm->getErrorMessage()));
+			$error_msg = $fm->getErrorMessage();
+			session_redirect('/forum/admin/index.php?editmsg='.$msg_id.'&group_id='.$group_id.'&thread_id='.$thread_id.'&forum_id='.$forum_id);
 		}
 		forum_header(array('title'=>_('Edit a Message')));
 		echo '<p>'.util_make_link('/forum/forum.php?forum_id=' . $forum_id, _("Return to the forum")) ;
diff --git a/src/www/forum/forum.php b/src/www/forum/forum.php
index 1a25eb7..c69ff92 100644
--- a/src/www/forum/forum.php
+++ b/src/www/forum/forum.php
@@ -108,7 +108,8 @@ if ($forum_id) {
 
 		if (!$fm->create($subject, $body, $thread_id, $is_followup_to,$has_attach) || $fm->isError()) {
 			form_release_key(getStringFromRequest("form_key"));
-			session_redirect('/forum/new.php?forum_id='.$forum_id.'&group_id='.$group_id.'&error_msg='.urlencode($fm->getErrorMessage()));
+			$error_msg = $fm->getErrorMessage();
+			session_redirect('/forum/new.php?forum_id='.$forum_id.'&group_id='.$group_id);
 		} else {
 			if ($fm->isPending() ) {
 				$feedback = _('Message Queued for moderation -> Please wait until the admin approves/rejects it');
diff --git a/src/www/forum/monitor.php b/src/www/forum/monitor.php
index 3de48fc..52bb843 100644
--- a/src/www/forum/monitor.php
+++ b/src/www/forum/monitor.php
@@ -69,13 +69,15 @@ if ($forum_id && $group_id) {
 		if (!$f->stopMonitor()) {
 			exit_error($f->getErrorMessage(),'forums');
 		} else {
-			session_redirect('/forum/forum.php?forum_id='.$forum_id.'&group_id='.$group_id.'&feedback='.urlencode(_('Forum Monitoring Deactivated')));
+			$feedback = _('Forum Monitoring Deactivated');
+			session_redirect('/forum/forum.php?forum_id='.$forum_id.'&group_id='.$group_id);
 		}
 	} elseif(getIntFromRequest('start')) {
 		if (!$f->setMonitor()) {
 			exit_error($f->getErrorMessage(),'forums');
 		} else {
-			session_redirect('/forum/forum.php?forum_id='.$forum_id.'&group_id='.$group_id.'&feedback='.urlencode(_('Forum Monitoring Started')));
+			$feedback = _('Forum Monitoring Started');
+			session_redirect('/forum/forum.php?forum_id='.$forum_id.'&group_id='.$group_id);
 		}
 	}
 } else {
diff --git a/src/www/forum/save.php b/src/www/forum/save.php
index b2c0ed7..dd4d596 100644
--- a/src/www/forum/save.php
+++ b/src/www/forum/save.php
@@ -52,7 +52,8 @@ if ($forum_id && $group_id) {
 	if (!$f->savePlace()) {
 		exit_error($f->getErrorMessage(),'forums');
 	} else {
-		session_redirect('/forum/forum.php?forum_id='.$forum_id.'&group_id='.$group_id.'&feedback='.urlencode(_('Forum Position Saved. New messages will be highlighted when you return')));
+		$feedback = _('Forum Position Saved. New messages will be highlighted when you return');
+		session_redirect('/forum/forum.php?forum_id='.$forum_id.'&group_id='.$group_id);
 	}
 } else {
 	exit_missing_param('',array(_('Forum ID'),_('Project ID')),'forums');
diff --git a/src/www/mail/admin/deletelist.php b/src/www/mail/admin/deletelist.php
index b5aa5dc..aa9bddf 100644
--- a/src/www/mail/admin/deletelist.php
+++ b/src/www/mail/admin/deletelist.php
@@ -56,7 +56,7 @@ if (getStringFromPost('submit')) {
 		exit_error($ml->getErrorMessage(),'home');
 	} else {
 		$feedback= _('Mailing List Successfully deleted');
-		session_redirect('/mail/admin/?group_id='.$group_id.'&feedback='.urlencode($feedback));
+		session_redirect('/mail/admin/?group_id='.$group_id);
 	}
 }
 
diff --git a/src/www/project/admin/group_trove.php b/src/www/project/admin/group_trove.php
index 169a3c9..1f503eb 100644
--- a/src/www/project/admin/group_trove.php
+++ b/src/www/project/admin/group_trove.php
@@ -40,7 +40,7 @@ if (getStringFromRequest('submit') && getStringFromRequest('root1')) {
 	 if (!form_key_is_valid(getStringFromRequest('form_key'))) {
 		exit_form_double_submit('summary');
 	 }
-	group_add_history ('Changed Trove', '', $group_id);
+	group_add_history(_('Changed Trove'), '', $group_id);
 
 	// there is at least a $root1[xxx]
 	$allroots = array();
@@ -68,7 +68,7 @@ if (getStringFromRequest('submit') && getStringFromRequest('root1')) {
 		}
 	}
 	$feedback = _('Trove Update Success');
-	session_redirect('/project/admin/?group_id='.$group_id.'&feedback='.urlencode($feedback));
+	session_redirect('/project/admin/?group_id='.$group_id);
 }
 
 html_use_tooltips();
diff --git a/src/www/project/admin/massfinish.php b/src/www/project/admin/massfinish.php
index a3da2e3..4cd83a6 100644
--- a/src/www/project/admin/massfinish.php
+++ b/src/www/project/admin/massfinish.php
@@ -46,8 +46,8 @@ if (getStringFromRequest('finished')) {
 	for ($i=0; $i<count($keys); $i++) {
 		$group->addUser($keys[$i],$addrole[$keys[$i]]);
 	}
-    $feedback = _('Successful');
-	session_redirect('/project/admin/index.php?group_id='.$group_id.'&feedback='.urlencode($feedback));
+	$feedback = _('Successful');
+	session_redirect('/project/admin/index.php?group_id='.$group_id);
 }
 
 if (!$accumulated_ids) {
@@ -68,7 +68,7 @@ echo '
 <form action="'.getStringFromServer('PHP_SELF').'?group_id='.$group_id.'" method="post">';
 
 if (!$res || db_numrows($res) < 1) {
-	echo "No Matching Users Found";
+	echo _('No Matching Users Found');
 } else {
 
 	$titles[]=_('Real Name');
diff --git a/src/www/project/admin/roledelete.php b/src/www/project/admin/roledelete.php
index 0df015a..4f10e8d 100644
--- a/src/www/project/admin/roledelete.php
+++ b/src/www/project/admin/roledelete.php
@@ -65,13 +65,12 @@ if (getStringFromRequest('submit')) {
 		} else {
 			$feedback = _('Successfully Deleted Role');
 			$group->addHistory(_('Deleted Role'), $role_name);
-			session_redirect('/project/admin/users.php?group_id='.$group_id.'&feedback='.urlencode($feedback));
+			session_redirect('/project/admin/users.php?group_id='.$group_id);
 		}
 	} else {
-		$error_msg = _('Error: Please check “I am Sure” to confirm or return to previous page to cancel.');
+		$error_msg = _('Error')._(': ')._('Please check “I am Sure” to confirm or return to previous page to cancel.');
 	}
-
-	session_redirect('/project/admin/users.php?group_id='.$group_id.'&error_msg='.urlencode($error_msg));
+	session_redirect('/project/admin/users.php?group_id='.$group_id);
 }
 
 $title = sprintf(_('Permanently Delete Role %s'), $role->getName());
diff --git a/src/www/survey/rating_resp.php b/src/www/survey/rating_resp.php
index dfe9b2b..e712bf8 100644
--- a/src/www/survey/rating_resp.php
+++ b/src/www/survey/rating_resp.php
@@ -53,12 +53,12 @@ if (!session_loggedin()) {
 						 $response,
 						 time()));
 		if (!$result) {
-			$error_msg .= _('Insert Error').db_error();
-			session_redirect('/&error_msg='.urlencode($error_msg));
+			$error_msg .= _('Insert Error')._(': ').db_error();
+			session_redirect('/');
 		} else {
 			$feedback .= _('Vote registered');
 			$warning_msg .= _('If you vote again, your old vote will be erased.');
-			session_redirect('/&feedback='.urlencode($feedback).'&warning_msg='.urlencode($warning_msg));
+			session_redirect('/');
 		}
 	} else {
 		exit_missing_param('',array(_('Vote ID'),_('Response'),_('Flag')),'survey');

-----------------------------------------------------------------------

Summary of changes:
 src/common/account/actions/addsshkey.php           |   14 +++---
 src/common/account/actions/deletesshkey.php        |    5 +-
 src/common/docman/actions/monitorfile.php          |   21 ++++----
 src/common/docman/views/addfile.php                |   10 ++--
 src/common/docman/views/additem.php                |    4 +-
 src/common/docman/views/addsubdocgroup.php         |    4 +-
 src/common/docman/views/admin.php                  |    4 +-
 src/common/docman/views/editdocgroup.php           |   10 ++--
 src/common/docman/views/editfile.php               |    4 +-
 src/common/docman/views/help.php                   |    4 +-
 src/common/docman/views/listfile.php               |   13 ++---
 src/common/docman/views/listtrashfile.php          |   13 ++---
 src/common/docman/views/menu.php                   |    4 +-
 src/common/docman/views/movefile.php               |    4 +-
 src/common/docman/views/pendingfiles.php           |    4 +-
 src/common/docman/views/reporting.php              |    4 +-
 src/common/docman/views/search.php                 |    4 +-
 src/common/docman/views/tree.php                   |    4 +-
 src/common/docman/views/views.php                  |    4 +-
 src/common/include/account.php                     |    4 +-
 src/common/include/exit.php                        |   16 +++---
 src/common/tracker/actions/admin-updates.php       |    4 +-
 src/common/tracker/actions/query.php               |    9 ++--
 src/plugins/blocks/www/index.php                   |   10 ++--
 src/plugins/extsubproj/actions/addExtSubProj.php   |    4 +-
 src/plugins/extsubproj/actions/delExtSubProj.php   |    4 +-
 .../globaldashboard/www/admin/add_account.php      |   51 +++++++++++---------
 .../globaldashboard/www/admin/delete_account.php   |   49 ++++++++++---------
 .../globaldashboard/www/admin/edit_account.php     |   51 +++++++++++---------
 src/plugins/headermenu/action/addLink.php          |   22 ++++-----
 src/plugins/headermenu/action/deleteLink.php       |    6 +--
 src/plugins/headermenu/action/updateLinkStatus.php |    6 +--
 src/plugins/headermenu/action/updateLinkValue.php  |   22 ++++-----
 .../headermenu/view/admin/updateLinkValue.php      |    4 +-
 src/plugins/mailman/www/admin/deletelist.php       |    5 +-
 src/plugins/mantisbt/action/addAttachment.php      |    4 +-
 src/plugins/mantisbt/action/addCategory.php        |    6 +--
 src/plugins/mantisbt/action/addIssue.php           |   10 ++--
 src/plugins/mantisbt/action/addNote.php            |    4 +-
 src/plugins/mantisbt/action/addVersion.php         |   12 ++---
 src/plugins/mantisbt/action/deleteAttachment.php   |    6 +--
 src/plugins/mantisbt/action/deleteCategory.php     |   24 ++++-----
 src/plugins/mantisbt/action/deleteNote.php         |    6 +--
 src/plugins/mantisbt/action/deleteVersion.php      |   15 +++---
 src/plugins/mantisbt/action/init.php               |   10 ++--
 src/plugins/mantisbt/action/inituser.php           |    4 +-
 src/plugins/mantisbt/action/renameCategory.php     |   10 ++--
 src/plugins/mantisbt/action/updateConf.php         |   10 ++--
 src/plugins/mantisbt/action/updateGlobalConf.php   |    4 +-
 src/plugins/mantisbt/action/updateIssue.php        |    6 +--
 src/plugins/mantisbt/action/updateNote.php         |    6 +--
 src/plugins/mantisbt/action/updateVersion.php      |    4 +-
 src/plugins/mantisbt/action/updateuserConf.php     |    9 ++--
 src/plugins/mantisbt/www/getAttachment.php         |    5 +-
 src/plugins/mantisbt/www/index.php                 |   19 ++------
 .../actions/updateGlobalConf.php                   |    4 +-
 .../actions/updateProjectConf.php                  |    4 +-
 .../common/projects-hierarchyPlugin.class.php      |   15 +++---
 src/plugins/webanalytics/action/addLink.php        |    6 +--
 src/plugins/webanalytics/action/deleteLink.php     |    6 +--
 .../webanalytics/action/updateLinkStatus.php       |    6 +--
 .../webanalytics/action/updateLinkValue.php        |    4 +-
 .../webanalytics/view/admin/updateLinkValue.php    |    2 +-
 src/www/admin/globalroledelete.php                 |   13 +++--
 src/www/docman/view.php                            |   16 +++---
 src/www/forum/admin/index.php                      |    3 +-
 src/www/forum/forum.php                            |    3 +-
 src/www/forum/monitor.php                          |    6 ++-
 src/www/forum/save.php                             |    3 +-
 src/www/mail/admin/deletelist.php                  |    2 +-
 src/www/project/admin/group_trove.php              |    4 +-
 src/www/project/admin/massfinish.php               |    6 +--
 src/www/project/admin/roledelete.php               |    7 ++-
 src/www/survey/rating_resp.php                     |    6 +--
 74 files changed, 358 insertions(+), 333 deletions(-)


hooks/post-receive
-- 
FusionForge



More information about the Fusionforge-commits mailing list