Formatting and adding FIXME to addressbook.php

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@9504 7612ce4b-ef26-0410-bec9-ea0150e637f0

diff --git a/src/about.php b/src/about.php
index 6602325b2b590859d3b90cc025c00eebcf4372f3..b048ce02539a9e1ecbf93ca13518142567debae7 100644 (file)
--- a/src/about.php
+++ b/src/about.php
@@ -83,6 +83,7 @@ if ( count ($plugins) > 0 ) {
 } else {
     echo '<em>'._("none installed")."</em>\n\n";
 }
+
 ?>
 </small>
 <br /><br />

diff --git a/src/addrbook_search.php b/src/addrbook_search.php
index 05777b89e1f7c381350d88bcc4c723e51f398141..3fe8001c4e8792cfe3d744f27e278d661496b1b6 100644 (file)
--- a/src/addrbook_search.php
+++ b/src/addrbook_search.php
@@ -308,5 +308,6 @@ if ($show == 'form' && empty($listall)) {
     }
 
 }
+
 ?>
-</body></html>
+</body></html>
\ No newline at end of file

diff --git a/src/addressbook.php b/src/addressbook.php
index 753cd2ca054ac9655f62a37b938ec2aec498eb40..a0737ee89b57ffbaf9270e7dc020862a2794a2e9 100644 (file)
--- a/src/addressbook.php
+++ b/src/addressbook.php
@@ -61,7 +61,6 @@ if($abook->localbackend == 0) {
     exit();
 }
 
-
 $defdata   = array();
 $formerror = '';
 $abortform = false;
@@ -158,6 +157,7 @@ if(sqgetGlobalVar('REQUEST_METHOD', $req_method, SQ_SERVER) && $req_method == 'P
                         $abortform = true;
                         list($ebackend, $enick) = explode(':', $sel[0]);
                         $olddata = $abook->lookup($enick, $ebackend);
+                        // FIXME: Test if $olddata really contains anything and return an error message if it doesn't
 
                         /* Display the "new address" form */
                         abook_create_form($form_url,'editaddr',_("Update address"),_("Update address"),$olddata);
@@ -175,13 +175,13 @@ if(sqgetGlobalVar('REQUEST_METHOD', $req_method, SQ_SERVER) && $req_method == 'P
                     if (!$r) {
                         /* Display error */
                         echo html_tag( 'table',
-                                       html_tag( 'tr',
-                                                 html_tag( 'td',
-                                                           "\n". '<strong><font color="' . $color[2] .
-                                                           '">' . _("ERROR") . ': ' . $abook->error . '</font></strong>' ."\n",
-                                                           'center' )
-                                                 ),
-                                       'center', '', 'width="100%"' );
+                                html_tag( 'tr',
+                                    html_tag( 'td',
+                                        "\n". '<strong><font color="' . $color[2] .
+                                        '">' . _("ERROR") . ': ' . $abook->error . '</font></strong>' ."\n",
+                                        'center' )
+                                    ),
+                                'center', '', 'width="100%"' );
 
                         /* Display the "new address" form again */
                         abook_create_form($form_url,'editaddr',_("Update address"),_("Update address"),$newdata);
@@ -291,7 +291,7 @@ if ($showaddrlist) {
                 }
 
                 // Print backend name
-                echo  html_tag( 'tr',
+                echo html_tag( 'tr',
                         html_tag( 'td', "\n" . '<strong>' . $row['source'] . '</strong>' . "\n", 'center', $color[0] ,'colspan="5"' )
                         );
 
@@ -379,5 +379,6 @@ echo "</form>\n";
 echo "<!-- start of addressbook_bottom hook-->\n";
 do_hook('addressbook_bottom');
 echo "\n<!-- end of addressbook_bottom hook-->\n";
+
 ?>
 </body></html>
\ No newline at end of file

diff --git a/src/compose.php b/src/compose.php
index 20f107918fe64444e23885ec5eb1b6a967cbbee1..eab20f32332609b035e2502ff9230b22e1a85952 100644 (file)
--- a/src/compose.php
+++ b/src/compose.php
@@ -34,8 +34,8 @@ require_once(SM_PATH . 'functions/plugin.php');
 require_once(SM_PATH . 'functions/display_messages.php');
 require_once(SM_PATH . 'class/deliver/Deliver.class.php');
 require_once(SM_PATH . 'functions/addressbook.php');
-require_once(SM_PATH . 'functions/identity.php');
 require_once(SM_PATH . 'functions/forms.php');
+require_once(SM_PATH . 'functions/identity.php');
 
 /* --------------------- Get globals ------------------------------------- */
 /** COOKIE VARS */
@@ -1658,5 +1658,4 @@ function deliverMessage($composeMessage, $draft=false) {
     return $succes;
 }
 
-// vim: et ts=4
-?>
+?>
\ No newline at end of file

diff --git a/src/configtest.php b/src/configtest.php
index 8e727d3e6775f7c2d82f399ca83e026b620ba979..f729624cf49fa4a12cbcdf9c253fbb8297cf6a25 100644 (file)
--- a/src/configtest.php
+++ b/src/configtest.php
@@ -479,6 +479,7 @@ if( empty($ldap_server) ) {
         }
     }
 }
+
 ?>
 
 <p>Congratulations, your SquirrelMail setup looks fine to me!</p>

diff --git a/src/delete_message.php b/src/delete_message.php
index 5ec635ccc1093cc60a6f41b86c346a1f81d0bd8f..2d6420b2fa57e54c1a985acf52b41681d06acda4 100644 (file)
--- a/src/delete_message.php
+++ b/src/delete_message.php
@@ -1,4 +1,5 @@
 <?php
+
 /**
  * delete_message.php
  *
@@ -23,5 +24,6 @@ include_once(SM_PATH . 'functions/display_messages.php');
 
 global $color;
 error_box('delete_message.php script is obsolete since 1.5.1.',$color);
+
 ?>
 </body></html>
\ No newline at end of file

diff --git a/src/download.php b/src/download.php
index c5b8efb6ca3e2bb5e34de92ba02eab25385dea09..91de332ef3e52c7179709b629a98a3c441edce3a 100644 (file)
--- a/src/download.php
+++ b/src/download.php
@@ -159,4 +159,5 @@ if (isset($absolute_dl) && $absolute_dl) {
 mime_print_body_lines ($imapConnection, $passed_id, $ent_id, $encoding);
 $mailbox_cache[$aMailbox['NAME']] = $aMailbox;
 sqsession_register($mailbox_cache,'mailbox_cache');
+
 ?>
\ No newline at end of file

diff --git a/src/folders.php b/src/folders.php
index 930148af02e432c1053afed344c40629210da5a3..a040f5e7c17860ad75a2573d3185e871e78a0399 100644 (file)
--- a/src/folders.php
+++ b/src/folders.php
@@ -367,6 +367,7 @@ if ($show_only_subscribed_folders) {
 
 do_hook('folders_bottom');
 sqimap_logout($imapConnection);
+
 ?>
     </td></tr>
     </table>

diff --git a/src/help.php b/src/help.php
index 690f274de70acf89bd5ed9b7f6d69ef1917573ae..609937d0c5856cda83b528b3eefe2d1ea8b15bec 100644 (file)
--- a/src/help.php
+++ b/src/help.php
@@ -211,5 +211,6 @@ do_hook('help_bottom');
 echo html_tag( 'tr',
             html_tag( 'td', ' ', 'left', $color[0] )
         );
+
 ?>
 </table></body></html>
\ No newline at end of file

diff --git a/src/image.php b/src/image.php
index f08b8784b5230e1238d1a67f3619b8f41829a1c0..1ecf75dbf4cf7f876c770f495a48081fe4dd6295 100644 (file)
--- a/src/image.php
+++ b/src/image.php
@@ -52,6 +52,7 @@ echo '<a href="'.$msg_url.'">'. _("View message") . '</a>';
 $DownloadLink = '../src/download.php?passed_id=' . $passed_id .
                '&amp;mailbox=' . urlencode($mailbox) .
                '&amp;ent_id=' . urlencode($ent_id) . '&amp;absolute_dl=true';
+
 ?>
 </b></td></tr>
 <tr><td align="center">

diff --git a/src/login.php b/src/login.php
index 7522f69c388ec1e323b338553e829a660b177a94..203938867811f0ec5edd3d41fadae0b385a1a859 100644 (file)
--- a/src/login.php
+++ b/src/login.php
@@ -173,23 +173,24 @@ echo html_tag( 'table',
                                     addHidden('js_autodetect_results', SMPREF_JS_OFF).
                                     $rcptaddress .
                                     addHidden('just_logged_in', '1'),
-                                'left', '', 'width="*"' )
-                            ) ,
-                        'center', $color[4], 'border="0" width="100%"' ) ,
-                    'left', $color[4] )
-                ) .
-                html_tag( 'tr',
-                    html_tag( 'td',
-                        '<center>'. addSubmit(_("Login")) .'</center>',
-                    'left' )
-                ),
-            '', $color[4], 'border="0" width="350"' ) . '</center>',
-        'center' )
-    ) ,
-'', $color[4], 'border="0" cellspacing="0" cellpadding="0" width="100%"' );
+                                    'left', '', 'width="*"' )
+                                ) ,
+                            'center', $color[4], 'border="0" width="100%"' ) ,
+                        'left', $color[4] )
+                        ) .
+                        html_tag( 'tr',
+                                html_tag( 'td',
+                                    '<center>'. addSubmit(_("Login")) .'</center>',
+                                    'left' )
+                                ),
+                        '', $color[4], 'border="0" width="350"' ) . '</center>',
+                        'center' )
+                        ) ,
+                        '', $color[4], 'border="0" cellspacing="0" cellpadding="0" width="100%"' );
 do_hook('login_form');
 echo '</form>' . "\n";
 
 do_hook('login_bottom');
+
 ?>
 </body></html>
\ No newline at end of file

diff --git a/src/mailto.php b/src/mailto.php
index 75146e9af3d29efc1894b531a9111025d1fb04b7..fc1bf393178d88f15501a89574cfc339fc6495ef 100644 (file)
--- a/src/mailto.php
+++ b/src/mailto.php
@@ -80,4 +80,5 @@ if($force_login == false && sqsession_is_registered('user_is_logged_in')) {
 
 session_write_close();
 header('Location: ' . get_location() . '/' . $redirect);
+
 ?>
\ No newline at end of file

diff --git a/src/options.php b/src/options.php
index 23d4db99ef75f63fc15729f22eb5c8a51a9f6607..a3982a7d83b2aa849bb3c030b5d40d49ba8761c6 100644 (file)
--- a/src/options.php
+++ b/src/options.php
@@ -486,6 +486,7 @@ if ($optpage == SMOPT_PAGE_MAIN) {
         do_hook($bottom_hook_name);
     }
 }
+
 ?>
 </td></tr>
 </table>

diff --git a/src/options_highlight.php b/src/options_highlight.php
index d724910efff8ba128bb21e6273ef8a4a62c6708b..c4dfba067c46487919b489cb7cbe7b712c35e79a 100644 (file)
--- a/src/options_highlight.php
+++ b/src/options_highlight.php
@@ -465,5 +465,6 @@ if ($action == 'edit' || $action == 'add') {
     echo "</form>\n";
 }
 do_hook('options_highlight_bottom');
+
 ?>
 </table></body></html>
\ No newline at end of file

diff --git a/src/options_identities.php b/src/options_identities.php
index a8c0e653880fb4c9b29adac3cc7afa6a8547a359..c776c4b20a5744f3b82763809eb0fce655cacb4d 100644 (file)
--- a/src/options_identities.php
+++ b/src/options_identities.php
@@ -357,4 +357,5 @@ function ShowTableInfo($full_name, $email_address, $reply_to, $signature, $post)
 
     return ($return_val);
 }
+
 ?>
\ No newline at end of file

diff --git a/src/printer_friendly_bottom.php b/src/printer_friendly_bottom.php
index ffaccb00b9ca9669881c3e3c72bdf7e4b3d6a00f..db37b09a26836d6c093bf4457e559e7d64b516ee 100644 (file)
--- a/src/printer_friendly_bottom.php
+++ b/src/printer_friendly_bottom.php
@@ -186,4 +186,5 @@ function pf_clean_string ( $unclean_string, $num_leading_spaces ) {
 } /* end pf_clean_string() function */
 
 /* --end pf-specific functions */
+
 ?>
\ No newline at end of file

diff --git a/src/printer_friendly_main.php b/src/printer_friendly_main.php
index 4d5a703b1d1dd3fcb3825689e467bfbe7ace3ba7..639671c2d47a6e353090e3318f048a0a1554987a 100644 (file)
--- a/src/printer_friendly_main.php
+++ b/src/printer_friendly_main.php
@@ -40,5 +40,6 @@ if ( ! sqgetGlobalVar('mailbox',$mailbox,SQ_GET) ||
         . '" name="bottom_frame" frameborder="0" />' . "\n";
     echo "</frameset>\n";
 }
+
 ?>
 </html>
\ No newline at end of file

diff --git a/src/printer_friendly_top.php b/src/printer_friendly_top.php
index 41294e8220a12048b77edc49264254537d142a71..33202e8fe6c9445ef1a62238ad95ce128a505f55 100644 (file)
--- a/src/printer_friendly_top.php
+++ b/src/printer_friendly_top.php
@@ -39,5 +39,6 @@ echo '<body text="'.$color[8].'" bgcolor="'.$color[3].'" link="'.$color[7].'" vl
          '<input type="button" value="' . _("Close") . '" onclick="window.parent.close()" />'.
          '</form>',
      'right' );
+
 ?>
 </body></html>
\ No newline at end of file

diff --git a/src/read_body.php b/src/read_body.php
index 7998c13cd3d879e1d7ec7d0b3a8ed804e75b84de..f3bb0fcd8460f0c5ec939387607637bd6b22d333 100644 (file)
--- a/src/read_body.php
+++ b/src/read_body.php
@@ -1009,5 +1009,6 @@ sqimap_logout($imapConnection);
  */
 $mailbox_cache[$iAccount.'_'.$aMailbox['NAME']] = $aMailbox;
 sqsession_register($mailbox_cache,'mailbox_cache');
+
 ?>
 </body></html>
\ No newline at end of file

diff --git a/src/right_main.php b/src/right_main.php
index 058ff5b88d3d89c35a8eb42c6aafc6ca595ad92c..c8380e7d611afd87807e037a7d585dbaae95cb79 100644 (file)
--- a/src/right_main.php
+++ b/src/right_main.php
@@ -1,4 +1,5 @@
 <?php
+
 /**
  * right_main.php
  *

diff --git a/src/search.php b/src/search.php
index 4401c3eae20af07b4dac119065e30d497acfb05d..b08778f5e50e55c46e8c832de7780fa37066365c 100644 (file)
--- a/src/search.php
+++ b/src/search.php
@@ -1645,4 +1645,5 @@ do_hook('search_bottom');
 sqimap_logout($imapConnection);
 echo '</body></html>';
 sqsession_register($mailbox_cache,'mailbox_cache');
+
 ?>
\ No newline at end of file

diff --git a/src/signout.php b/src/signout.php
index 775727932bc4b045ff63f3980314b5a968ce9f95..eb4f9ab0c96dc553b012a92704590544229ea552 100644 (file)
--- a/src/signout.php
+++ b/src/signout.php
@@ -95,6 +95,7 @@ html_tag( 'table',
          html_tag( 'td', '<br />', 'center' ) ,
     '', $color[0] ) ,
 'center', $color[4], 'width="50%" cellpadding="2" cellspacing="0" border="0"' )
+
 ?>
 </body>
 </html>
\ No newline at end of file

diff --git a/src/view_html.php b/src/view_html.php
index 250bfe8771d303351f53754a28481913a99ea8ba..cce20eaf7210c28b862283d8499199e42dec0c13 100644 (file)
--- a/src/view_html.php
+++ b/src/view_html.php
@@ -1,4 +1,5 @@
 <?php
+
 /**
  * $Source$
  * Displays html message parts
@@ -97,4 +98,5 @@ $body = MagicHTML( $body, $passed_id, $message, $mailbox);
 /** TODO: charset might be part of html code. */
 header('Content-Type: text/html; charset=' . $charset);
 echo $body;
+
 ?>
\ No newline at end of file

diff --git a/src/webmail.php b/src/webmail.php
index 7417dd76e20db727a0470bb11852ce7136f59251..aaad5e5fc3ae2f9213c382b4cc90e1b8c9710270 100644 (file)
--- a/src/webmail.php
+++ b/src/webmail.php
@@ -49,7 +49,7 @@ if (sqgetGlobalVar('startMessage', $startMessage)) {
     $startMessage = (int) $startMessage;
 }
 
-if (!sqgetGlobalVar('mailbox',$mailbox)) {
+if (!sqgetGlobalVar('mailbox', $mailbox)) {
     $mailbox = 'INBOX';
 }
 
@@ -180,6 +180,7 @@ if($ret != '') {
     $output = $ret;
 }
 echo $output;
+
 ?>
 </frameset>
 </html>
\ No newline at end of file