X-Git-Url: https://vcs.fsf.org/?a=blobdiff_plain;f=src%2Faddressbook.php;h=2df29cdba9e572719b4cb41ca60edbe8ac30a5b1;hb=6316707656c195f1d33e54a065a0b4aff3f4636e;hp=ee38cf7f8b87d57077c67ce8fea3b8acf661b251;hpb=daba719eea8875a13aaedbba933f122046096df9;p=squirrelmail.git diff --git a/src/addressbook.php b/src/addressbook.php index ee38cf7f..2df29cdb 100644 --- a/src/addressbook.php +++ b/src/addressbook.php @@ -3,7 +3,7 @@ /** * addressbook.php * - * Copyright (c) 1999-2001 The Squirrelmail Development Team + * Copyright (c) 1999-2002 The SquirrelMail Project Team * Licensed under the GNU GPL. For full terms see the file COPYING. * * Manage personal address book. @@ -11,38 +11,11 @@ * $Id$ */ -/*****************************************************************/ -/*** THIS FILE NEEDS TO HAVE ITS FORMATTING FIXED!!! ***/ -/*** PLEASE DO SO AND REMOVE THIS COMMENT SECTION. ***/ -/*** + Base level indent should begin at left margin, as ***/ -/*** the require_once below looks. ***/ -/*** + All identation should consist of four space blocks ***/ -/*** + Tab characters are evil. ***/ -/*** + all comments should use "slash-star ... star-slash" ***/ -/*** style -- no pound characters, no slash-slash style ***/ -/*** + FLOW CONTROL STATEMENTS (if, while, etc) SHOULD ***/ -/*** ALWAYS USE { AND } CHARACTERS!!! ***/ -/*** + Please use ' instead of ", when possible. Note " ***/ -/*** should always be used in _( ) function calls. ***/ -/*** Thank you for your help making the SM code more readable. ***/ -/*****************************************************************/ - require_once('../src/validate.php'); require_once('../functions/array.php'); require_once('../functions/display_messages.php'); require_once('../functions/addressbook.php'); - -/* Sort array by the key "name" */ -function alistcmp($a,$b) { - if($a['backend'] > $b['backend']) { - return 1; - } else { - if($a['backend'] < $b['backend']) { - return -1; - } - } - return (strtolower($a['name']) > strtolower($b['name'])) ? 1 : -1; -} +require_once('../functions/strings.php'); /* Make an input field */ function adressbook_inp_field($label, $field, $name, $size, $values, $add) { @@ -77,8 +50,8 @@ function address_form($name, $submittext, $values = array()) { } -// Open addressbook, with error messages on but without LDAP (the -// second "true"). Don't need LDAP here anyway +/* Open addressbook, with error messages on but without LDAP (the * + * second "true"). Don't need LDAP here anyway */ $abook = addressbook_init(true, true); if($abook->localbackend == 0) { plain_error_message( @@ -97,19 +70,19 @@ $showaddrlist = true; $defselected = array(); -// Handle user's actions +/* Handle user's actions */ if($REQUEST_METHOD == 'POST') { - // *********************************************** - // Add new address - // *********************************************** - if(!empty($addaddr['nickname'])) { + /************************************************** + * Add new address * + **************************************************/ + if (!empty($addaddr['nickname'])) { $r = $abook->add($addaddr, $abook->localbackend); - // Handle error messages - if(!$r) { - // Remove backend name from error string + /* Handle error messages */ + if (!$r) { + /* Remove backend name from error string */ $errstr = $abook->error; $errstr = ereg_replace('^\[.*\] *', '', $errstr); @@ -120,28 +93,28 @@ if($REQUEST_METHOD == 'POST') { } else { - // *********************************************** - // Delete address(es) - // *********************************************** - if((!empty($deladdr)) && sizeof($sel) > 0) { + /************************************************ + * Delete address(es) * + ************************************************/ + if ((!empty($deladdr)) && sizeof($sel) > 0) { $orig_sel = $sel; sort($sel); - // The selected addresses are identidied by "backend:nickname". - // Sort the list and process one backend at the time + /* The selected addresses are identidied by "backend:nickname". * + * Sort the list and process one backend at the time */ $prevback = -1; $subsel = array(); $delfailed = false; - for($i = 0 ; (($i < sizeof($sel)) && !$delfailed) ; $i++) { + for ($i = 0 ; (($i < sizeof($sel)) && !$delfailed) ; $i++) { list($sbackend, $snick) = explode(':', $sel[$i]); - // When we get to a new backend, process addresses in - // previous one. - if($prevback != $sbackend && $prevback != -1) { + /* When we get to a new backend, process addresses in * + * previous one. */ + if ($prevback != $sbackend && $prevback != -1) { $r = $abook->remove($subsel, $prevback); - if(!$r) { + if (!$r) { $formerror = $abook->error; $i = sizeof($sel); $delfailed = true; @@ -150,32 +123,32 @@ if($REQUEST_METHOD == 'POST') { $subsel = array(); } - // Queue for processing + /* Queue for processing */ array_push($subsel, $snick); $prevback = $sbackend; } - if(!$delfailed) { + if (!$delfailed) { $r = $abook->remove($subsel, $prevback); - if(!$r) { // Handle errors + if (!$r) { /* Handle errors */ $formerror = $abook->error; $delfailed = true; } } - if($delfailed) { + if ($delfailed) { $showaddrlist = true; $defselected = $orig_sel; } } else { - // *********************************************** - // Update/modify address - // *********************************************** - if(!empty($editaddr)) { + /*********************************************** + * Update/modify address * + ***********************************************/ + if (!empty($editaddr)) { - // Stage one: Copy data into form + /* Stage one: Copy data into form */ if (isset($sel) && sizeof($sel) > 0) { if(sizeof($sel) > 1) { $formerror = _("You can only edit one address at the time"); @@ -186,58 +159,58 @@ if($REQUEST_METHOD == 'POST') { list($ebackend, $enick) = explode(':', $sel[0]); $olddata = $abook->lookup($enick, $ebackend); - // Display the "new address" form - print "
\n"; - print "\n"; - print "\n"; - print "
\n"; - print _("Update address"); - print "\n
\n"; + /* Display the "new address" form */ + echo '' . + "\n" . + '' . "\n" . + '\n
' . "\n" . '' . + _("Update address") . + "\n
\n"; address_form("editaddr", _("Update address"), $olddata); - printf("\n", - htmlspecialchars($olddata["nickname"])); - printf("\n", - htmlspecialchars($olddata["backend"])); - print "\n"; - print '
'; + echo '\n" . + '\n" . + '' . "\n" . + ''; } } else { - // Stage two: Write new data - if($doedit = 1) { + /* Stage two: Write new data */ + if ($doedit = 1) { $newdata = $editaddr; $r = $abook->modify($oldnick, $newdata, $backend); - // Handle error messages - if(!$r) { - // Display error - print "\n"; - print "\n"; - print "
\n
"; - print ""._("ERROR").": ". - $abook->error.""; - print "\n
\n"; - - // Display the "new address" form again - printf("
\n", $PHP_SELF); - print "\n"; - print "\n"; - print "
\n"; - print _("Update address"); - print "\n
\n"; + /* Handle error messages */ + if (!$r) { + /* Display error */ + echo '' . + "\n" . '\n
' . "\n" . + '
' . _("ERROR") . ": " . $abook->error . + '' . "\n
\n"; + + /* Display the "new address" form again */ + echo '' . "\n" . + '' . + "\n" . '\n
' . "\n" . '' . + _("Update address") . + "\n
\n"; address_form("editaddr", _("Update address"), $newdata); - printf("\n", - htmlspecialchars($oldnick)); - printf("\n", - htmlspecialchars($backend)); - print "\n"; - print '
'; - + echo '\n" . + '\n" . + '' . + "\n" . ''; $abortform = true; } } else { - // Should not get here... + /* Should not get here... */ plain_error_message(_("Unknown error"), $color); $abortform = true; } @@ -248,30 +221,29 @@ if($REQUEST_METHOD == 'POST') { // Some times we end output before forms are printed if($abortform) { - print "\n"; + echo "\n"; exit(); } } -// =================================================================== -// The following is only executed on a GET request, or on a POST when -// a user is added, or when "delete" or "modify" was successful. -// =================================================================== +/* =================================================================== * + * The following is only executed on a GET request, or on a POST when * + * a user is added, or when "delete" or "modify" was successful. * + * =================================================================== */ -// Display error messages -if(!empty($formerror)) { - print "\n"; - print "\n"; - print "
\n
"; - print ""._("ERROR").": $formerror"; - print "\n
\n"; +/* Display error messages */ +if (!empty($formerror)) { + echo '' . "\n" . + '\n
' . "\n" . '
' . + '' . _("ERROR") . ': ' . $formerror . + '' . "\n
\n"; } -// Display the address management part -if($showaddrlist) { - // Get and sort address list +/* Display the address management part */ +if ($showaddrlist) { + /* Get and sort address list */ $alist = $abook->list_addr(); if(!is_array($alist)) { plain_error_message($abook->error, $color); @@ -282,96 +254,97 @@ if($showaddrlist) { $prevbackend = -1; $headerprinted = false; - echo "

" . + echo '

' . _("Add address") . "

\n"; - // List addresses - printf("
\n", $PHP_SELF); - while(list($undef,$row) = each($alist)) { - - // New table header for each backend - if($prevbackend != $row["backend"]) { - if($prevbackend >= 0) { - print "\n"; - printf("\n", - _("Edit selected")); - printf("\n", - _("Delete selected")); - echo "\n"; - print ''; - print " 
\n"; + /* List addresses */ + if (count($alist) > 0) { + echo '' . "\n"; + while(list($undef,$row) = each($alist)) { + + /* New table header for each backend */ + if($prevbackend != $row['backend']) { + if($prevbackend < 0) { + echo '' . "\n" . + '\n" . + '\n\n" . + '' . + ' 
' . "\n"; + } + + echo '' . "\n" . + '\n
' . "\n" . + '' . $row['source'] . + "\n
\n" . + '' . + '\n"; + + $line = 0; + $headerprinted = true; + } /* End of header */ + + $prevbackend = $row['backend']; + + /* Check if this user is selected */ + if(in_array($row['backend'] . ':' . $row['nickname'], $defselected)) { + $selected = 'CHECKED'; + } else { + $selected = ''; } - - print "
 ' . + _("Nickname") . '' . _("Name") . + '' . _("E-mail") . + '' . _("Info") . "
\n"; - print "\n"; - print "
\n"; - print $row["source"]; - print "\n
\n"; - - print ''; - printf('\n", $color[9], "1%", - "1%", _("Nickname"), - "1%", _("Name"), - "1%", _("E-mail"), - "%", _("Info")); - $line = 0; - $headerprinted = true; - } // End of header - - $prevbackend = $row['backend']; - - // Check if this user is selected - if(in_array($row['backend'].':'.$row['nickname'], $defselected)) { - $selected = 'CHECKED'; - } else { - $selected = ''; + + /* Print one row */ + echo '' . + '' . + '', + ''."\n", + '' . + "\n"; + $line++; } - - // Print one row - printf("", - (($line % 2) ? " bgcolor=\"$color[0]\"" : "")); - print '', - $selected, $row["backend"], $row["nickname"]); - printf(''. - '', - "1%", $row["nickname"], - "1%", $row["name"]); - printf(''."\n", - "1%", rawurlencode($row["email"]), $row["email"]); - printf('', - "%", $row["label"]); - print "\n"; - $line++; - } - - // End of list. Close table. - if($headerprinted) { - print "
 '. - '%s%s'. - '%s%s'. - "
' . + ' ' . $row['nickname'] . + '  ' . $row['name'] . + '  ' . + '' . $row['email'] . '  ' . $row['label'] . ' 
'; - printf(' %s  %s  %s  %s 
\n"; - printf("\n", - _("Edit selected")); - printf("\n", - _("Delete selected")); - print "
"; + + /* End of list. Close table. */ + if ($headerprinted) { + echo '' . "\n" . + '\n" . + '\n" . ''; + } + echo ''; } -} // end of addresslist +} /* end of addresslist */ -// Display the "new address" form -echo "\n" . - "
\n". - "\n". - "\n". +/* Display the "new address" form */ +echo '' . "\n" . + '' . "\n" . + '
\n"; -printf(_("Add to %s"), $abook->localbackendname); -echo "\n
' . "\n" . + '\n" . "
' . "\n" . '', + sprintf(_("Add to %s"), $abook->localbackendname) . + "\n
\n"; address_form('addaddr', _("Add address"), $defdata); echo '
'; -// Add hook for anything that wants on the bottom +/* Add hook for anything that wants on the bottom */ do_hook('addressbook_bottom'); ?>