From d0c012e31e8bc6dbfd8d71fa21b9ffd3f166e88a Mon Sep 17 00:00:00 2001
From: Alan Garcia <a.garcia@nnx.com>
Date: Thu, 23 Aug 2012 14:10:38 +0000
Subject: [PATCH] Tcho notices

---
 bureau/admin/adm_mxaccount.php | 3 +--
 bureau/admin/dom_subdel.php    | 2 +-
 bureau/admin/dom_subdodel.php  | 2 +-
 bureau/admin/dom_subdoedit.php | 4 ++--
 bureau/admin/hta_adduser.php   | 5 +++--
 bureau/admin/hta_edituser.php  | 6 +++---
 bureau/admin/mem_admin.php     | 2 +-
 bureau/admin/mem_chgmail.php   | 4 ++--
 bureau/admin/sql_admin.php     | 2 +-
 9 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/bureau/admin/adm_mxaccount.php b/bureau/admin/adm_mxaccount.php
index be5e98ca..d010e27a 100644
--- a/bureau/admin/adm_mxaccount.php
+++ b/bureau/admin/adm_mxaccount.php
@@ -36,7 +36,6 @@ if (!$admin->enabled) {
 
 $fields = array (
 	"delaccount"   => array ("request", "string", ""),
-
 	"newlogin"   => array ("request", "string", ""),
 	"newpass"    => array ("request", "string", ""),
 );
@@ -64,7 +63,7 @@ include_once("head.php");
 <hr id="topbar"/>
 <br />
 <?php
-	if ($error) {
+	if (isset($error) && $error) {
 	  echo "<p class=\"error\">$error</p>";
 	}
 
diff --git a/bureau/admin/dom_subdel.php b/bureau/admin/dom_subdel.php
index a5db0966..74290bc0 100644
--- a/bureau/admin/dom_subdel.php
+++ b/bureau/admin/dom_subdel.php
@@ -56,7 +56,7 @@ $dom->unlock();
 ?>
 <h3><?php printf(_("Deleting subdomain %s"),"http://".ife($sub,$sub.".").$domain); ?> : </h3>
 <?php
-	if ($error) {
+	if (isset($error) && $error) {
 		echo "<p class=\"error\">$error</p>";
 		include_once("foot.php");
 		exit();
diff --git a/bureau/admin/dom_subdodel.php b/bureau/admin/dom_subdodel.php
index 242d77b2..ce0f301f 100644
--- a/bureau/admin/dom_subdodel.php
+++ b/bureau/admin/dom_subdodel.php
@@ -58,7 +58,7 @@ $dom->unlock();
 <hr id="topbar"/>
 <br />
 <?php
-	if ($error) {
+	if (isset($error) && $error) {
 		echo "<p class=\"error\">$error</p>";
 		include_once("foot.php");
 		exit();
diff --git a/bureau/admin/dom_subdoedit.php b/bureau/admin/dom_subdoedit.php
index f156671d..e0df0e82 100644
--- a/bureau/admin/dom_subdoedit.php
+++ b/bureau/admin/dom_subdoedit.php
@@ -35,7 +35,7 @@ $fields = array (
 	"sub_old"   => array ("request", "string", ""),
 	"type"      => array ("request", "string", $dom->type_local),
 	"type_old"  => array ("request", "string", ""),
-  "value_old" => array ("request", "string", ""),
+  	"value_old" => array ("request", "string", ""),
 );
 getFields($fields);
 
@@ -51,7 +51,7 @@ $value=$$dynamicvar;
 $dom->lock();
 
 $dt=$dom->domains_type_lst();
-if (!$isinvited && $dt[strtolower($type)]["enable"] != "ALL" ) {
+if ( (!isset($isinvited) || !$isinvited) && $dt[strtolower($type)]["enable"] != "ALL" ) {
   __("This page is restricted to authorized staff");
   exit();
 }
diff --git a/bureau/admin/hta_adduser.php b/bureau/admin/hta_adduser.php
index 3ed1bfdd..f64e5e0a 100644
--- a/bureau/admin/hta_adduser.php
+++ b/bureau/admin/hta_adduser.php
@@ -32,13 +32,14 @@ include_once("head.php");
 
 $fields = array (
 	"dir"      => array ("request", "string", ""),
+	"user"     => array ("request", "string", ""),
 );
 getFields($fields);
 
 ?>
 <h3><?php printf(_("Adding a username in %s"),$dir); ?></h3>
 <?php
-	if ($error) {
+	if (isset($error) && $error) {
 		echo "<p class=\"error\">$error</p>";
 	}
 ?>
@@ -69,4 +70,4 @@ getFields($fields);
 <script type="text/javascript">
 document.forms['main'].user.focus();
 </script>
-<?php include_once("foot.php"); ?>
\ No newline at end of file
+<?php include_once("foot.php"); ?>
diff --git a/bureau/admin/hta_edituser.php b/bureau/admin/hta_edituser.php
index 6bde93ed..cf9dcd08 100644
--- a/bureau/admin/hta_edituser.php
+++ b/bureau/admin/hta_edituser.php
@@ -41,9 +41,9 @@ getFields($fields);
 <hr id="topbar"/>
 <br />
 <?php
-	if ($error) {
-		echo "<p class=\"error\">$error</p>";
-	}
+if (isset($error) && $error) {
+	echo "<p class=\"error\">$error</p>";
+}
 ?>
 <form method="post" action="hta_doedituser.php" name="main" id="main">
 <table border="1" cellspacing="0" cellpadding="4">
diff --git a/bureau/admin/mem_admin.php b/bureau/admin/mem_admin.php
index 8926be72..7cb36f7b 100644
--- a/bureau/admin/mem_admin.php
+++ b/bureau/admin/mem_admin.php
@@ -45,7 +45,7 @@ include_once("head.php");
 ?>
 <h3><?php __("Admin preferences"); ?></h3>
 <?php
-	if ($error) {
+	if (isset($error) && $error) {
 		echo "<p class=\"error\">$error</p>";
 	}
 ?>
diff --git a/bureau/admin/mem_chgmail.php b/bureau/admin/mem_chgmail.php
index 25237fdb..e8c53a74 100644
--- a/bureau/admin/mem_chgmail.php
+++ b/bureau/admin/mem_chgmail.php
@@ -43,7 +43,7 @@ include_once("head.php");
 ?>
 <h3><?php __("Change the email of the account"); ?></h3>
 <?php
-	if ($error) {
+	if (isset($error) && $error) {
 		echo "<p class=\"error\">$error</p>";
 		include_once("foot.php");
 		exit();
@@ -51,4 +51,4 @@ include_once("head.php");
 printf(_("help_mem_chgmail %s"),$newmail);
 ?>
 <p class="code"><?php echo $cle; ?></p>
-<?php include_once("foot.php"); ?>
\ No newline at end of file
+<?php include_once("foot.php"); ?>
diff --git a/bureau/admin/sql_admin.php b/bureau/admin/sql_admin.php
index a2c542e4..4e4f7609 100644
--- a/bureau/admin/sql_admin.php
+++ b/bureau/admin/sql_admin.php
@@ -46,7 +46,7 @@ include_once("head.php");
 ?>
 <h3><?php __("SQL Admin"); ?></h3>
 <?php
-	if ($error) {
+	if (isset($error) && $error) {
 		echo "<p class=\"error\">$error</p>";
 	}
 ?>