WMDE
/
LDAPAccountManager
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

less PHP notices

This commit is contained in:
Roland Gruber 2006-01-25 18:31:19 +00:00
parent 6a8082abda
commit c7917dd760
3 changed files with 10 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lam/lib/baseModule.inc
View File

@ -235,7 +235,7 @@ class baseModule {
// empty input
if (($options[$identifiers[$i]][0] == '') || !isset($options[$identifiers[$i]][0])) {
// check if option is required
if ($this->meta['profile_checks'][$identifiers[$i]]['required']) {
if (isset($this->meta['profile_checks'][$identifiers[$i]]['required']) && $this->meta['profile_checks'][$identifiers[$i]]['required']) {
$messages[] = $this->meta['profile_checks'][$identifiers[$i]]['required_message'];
}
continue;

2
lam/lib/modules.inc
View File

@ -580,7 +580,7 @@ function parseHtml($module, $input, $values, $restricted, &$tabindex, &$tabindex
if (isset($values[$input[$i][$j]['name']])) {
$output .= ' value="' . $values[$input[$i][$j]['name']][0] . '"';
}
elseif ($input[$i][$j]['value']!='') $output .= ' value="' . $input[$i][$j]['value'] . '"';
elseif (isset($input[$i][$j]['value']) && $input[$i][$j]['value']!='') $output .= ' value="' . $input[$i][$j]['value'] . '"';
}
if (isset($input[$i][$j]['disabled']) && ($input[$i][$j]['disabled'] == true)) $output .= ' disabled';
// Show taborder

15
lam/templates/profedit/profilepage.php
View File

@ -57,7 +57,7 @@ if (isset($_POST['accounttype'])) $_GET['type'] = $_POST['accounttype'];
// abort button was pressed
// back to profile editor
if ($_POST['abort']) {
if (isset($_POST['abort'])) {
metaRefresh("profilemain.php");
exit;
}
@ -69,7 +69,7 @@ echo "<title></title>\n<link rel=\"stylesheet\" type=\"text/css\" href=\"../../s
echo "</head><body><br>\n";
// save button was presed
if ($_POST['save']) {
if (isset($_POST['save'])) {
// create option array to check and save
$options = array();
$opt_keys = array_keys($_SESSION['profile_types']);
@ -80,7 +80,7 @@ if ($_POST['save']) {
}
// checkboxes
elseif ($_SESSION['profile_types'][$element] == "checkbox") {
if ($_POST[$element] == "on") $options[$element] = array('true');
if (isset($_POST[$element]) && ($_POST[$element] == "on")) $options[$element] = array('true');
else $options[$element] = array('false');
}
// dropdownbox
@ -89,14 +89,15 @@ if ($_POST['save']) {
}
// multiselect
elseif ($_SESSION['profile_types'][$element] == "multiselect") {
$options[$element] = $_POST[$element]; // value is already an array
if (isset($_POST[$element])) $options[$element] = $_POST[$element]; // value is already an array
else $options[$element] = array();
}
}
// remove double slashes if magic quotes are on
if (get_magic_quotes_gpc() == 1) {
foreach ($opt_keys as $element) {
if (is_string($options[$element][0])) $options[$element][0] = stripslashes($options[$element][0]);
if (isset($options[$element][0]) && is_string($options[$element][0])) $options[$element][0] = stripslashes($options[$element][0]);
}
}
@ -179,7 +180,7 @@ echo "<fieldset class=\"" . $type . "edit\">\n<legend><b>" . _("LDAP") . "</b></
echo "</td><td>";
echo "<select name=\"ldap_suffix\" tabindex=\"1\">";
for ($i = 0; $i < sizeof($suffixes); $i++) {
if ($old_options['ldap_suffix'][0] == $suffixes[$i]) {
if (isset($old_options['ldap_suffix']) && ($old_options['ldap_suffix'][0] == $suffixes[$i])) {
echo "<option selected>" . $suffixes[$i] . "</option>\n";
}
else {
@ -199,7 +200,7 @@ echo "<fieldset class=\"" . $type . "edit\">\n<legend><b>" . _("LDAP") . "</b></
echo "</td><td>";
echo "<select name=\"ldap_rdn\" tabindex=\"1\">";
for ($i = 0; $i < sizeof($rdns); $i++) {
if ($old_options['ldap_rdn'][0] == $rdns[$i]) {
if (isset($old_options['ldap_rdn']) && ($old_options['ldap_rdn'][0] == $rdns[$i])) {
echo "<option selected>" . $rdns[$i] . "</option>\n";
}
else {