projects / squirrelmail.git / commitdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | patch | inline | side by side (from parent 1: 62b9c98)
removed require_once() calls that are already loaded by include/validate.php
authortokul <tokul@7612ce4b-ef26-0410-bec9-ea0150e637f0>
Tue, 7 Sep 2004 13:54:11 +0000 (13:54 +0000)
committertokul <tokul@7612ce4b-ef26-0410-bec9-ea0150e637f0>
Tue, 7 Sep 2004 13:54:11 +0000 (13:54 +0000)
converted $_GET's to sqgetGlobalVar's and added validation.

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

src/printer_friendly_bottom.php patch | blob | blame | history
src/printer_friendly_main.php patch | blob | blame | history
src/printer_friendly_top.php patch | blob | blame | history

diff --git a/src/printer_friendly_bottom.php b/src/printer_friendly_bottom.php
index 7e02275432150363da1ea5fda16c24bc4a2ff43b..a4888333626b0e051545addf1eb0bd80fce71480 100644 (file)
--- a/src/printer_friendly_bottom.php
+++ b/src/printer_friendly_bottom.php
@@ -23,12 +23,7 @@ define('SM_PATH','../');
 
 /* SquirrelMail required files. */
 require_once(SM_PATH . 'include/validate.php');
-require_once(SM_PATH . 'functions/strings.php');
-require_once(SM_PATH . 'config/config.php');
-require_once(SM_PATH . 'include/load_prefs.php');
 require_once(SM_PATH . 'functions/imap.php');
-require_once(SM_PATH . 'functions/page_header.php');
-require_once(SM_PATH . 'functions/html.php');
 
 /* get some of these globals */
 sqgetGlobalVar('username', $username, SQ_SESSION);
@@ -43,7 +38,7 @@ if (! sqgetGlobalVar('passed_ent_id', $passed_ent_id, SQ_GET) ) {
 } 
 /* end globals */
 
-$pf_cleandisplay = getPref($data_dir, $username, 'pf_cleandisplay');
+$pf_cleandisplay = getPref($data_dir, $username, 'pf_cleandisplay', false);
 $imapConnection = sqimap_login($username, $key, $imapServerAddress, $imapPort, 0);
 $mbx_response = sqimap_mailbox_select($imapConnection, $mailbox);
 if (isset($messages[$mbx_response['UIDVALIDITY']][$passed_id])) {
@@ -86,7 +81,7 @@ if ($ent_ar[0] != '') {
 
  /* now, if they choose to, we clean up the display a bit... */
  
-if ( empty($pf_cleandisplay) || $pf_cleandisplay != 'no' ) {
+if ($pf_cleandisplay) {
 
     $num_leading_spaces = 9; // nine leading spaces for indentation
 
@@ -155,7 +150,16 @@ echo '<body text="#000000" bgcolor="#FFFFFF" link="#000000" vlink="#000000" alin
 
 /* --start pf-specific functions-- */
 
-/* $string = pf_clean_string($string, 9); */
+/**
+ * Function should clean layout of printed messages when user
+ * enables "Printer Friendly Clean Display" option.
+ *
+ * @param string unclean_string
+ * @param integer num_leading_spaces
+ * @return string
+ * @example $string = pf_clean_string($string, 9);
+ * @access private 
+ */
 function pf_clean_string ( $unclean_string, $num_leading_spaces ) {
     global $data_dir, $username;
     $unclean_string = str_replace('&nbsp;',' ',$unclean_string);
@@ -190,5 +194,4 @@ 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 b8493e23a6cc9ba5182da8e93e85084b13e45ceb..37320c2849c755629a1e381403fad2e0de807514 100644 (file)
--- a/src/printer_friendly_main.php
+++ b/src/printer_friendly_main.php
@@ -1,11 +1,9 @@
 <?php
-
 /**
- * printer_friendly_main.php
- *
- * Copyright (c) 1999-2004 The SquirrelMail Project Team
- * Licensed under the GNU GPL. For full terms see the file COPYING.
+ * printer_friendly frameset
  *
+ * @copyright (c) 1999-2004 The SquirrelMail Project Team
+ * @license http://opensource.org/licenses/gpl-license.php GNU Public License
  * @version $Id$
  * @package squirrelmail
  */
@@ -18,21 +16,26 @@ define('SM_PATH','../');
 
 /* SquirrelMail required files. */
 require_once(SM_PATH . 'include/validate.php');
-require_once(SM_PATH . 'functions/page_header.php');
-
-displayHtmlHeader( _("Printer Friendly"), '', FALSE );
 
 /* get those globals into gear */
-$passed_ent_id = $_GET['passed_ent_id'];
-$passed_id = (int) $_GET['passed_id'];
-$mailbox = $_GET['mailbox'];
+global $color;
+if ( ! sqgetGlobalVar('passed_ent_id',$passed_ent_id,SQ_GET))
+    $passed_ent_id = 0;
+if ( ! sqgetGlobalVar('mailbox',$mailbox,SQ_GET) ||
+     ! sqgetGlobalVar('passed_id',$passed_id,SQ_GET)) {
+    error_box(_("Invalid URL"),$color);
+} else {
+    $passed_id= (int) $passed_id;
 /* end globals */
+    displayHtmlHeader( _("Printer Friendly"), '', false, true );
+    echo '<frameset rows="60, *">' . "\n";
+    echo '<frame src="printer_friendly_top.php" name="top_frame" '
+        . 'scrolling="no" noresize="noresize" frameborder="0">' . "\n";
+    echo '<frame src="printer_friendly_bottom.php?passed_ent_id='
+        . urlencode($passed_ent_id) . '&amp;mailbox=' . urlencode($mailbox) 
+        . '&amp;passed_id=' . $passed_id
+        . '" name="bottom_frame" frameborder="0">' . "\n";
+    echo "</frameset>\n";
+}
 ?>
-<frameset rows="60, *" noresize="noresize" border="0">
-    <frame src="printer_friendly_top.php" name="top_frame" scrolling="no" />
-    <frame src="printer_friendly_bottom.php?passed_ent_id=<?php
-        echo urlencode($passed_ent_id) . '&amp;mailbox=' . urlencode($mailbox) .
-             '&amp;passed_id=' . $passed_id;
-        ?>" name="bottom_frame" />
-</frameset>
 </html>
diff --git a/src/printer_friendly_top.php b/src/printer_friendly_top.php
index c7510b7fdcba68cff970dab953c0248f7072fe6a..fe6c1dc6e107f6b8739d2221932d18dd0dc91bfd 100644 (file)
--- a/src/printer_friendly_top.php
+++ b/src/printer_friendly_top.php
@@ -1,14 +1,12 @@
 <?php
-
 /**
- * printer_friendly_top.php
- *
- * Copyright (c) 1999-2004 The SquirrelMail Project Team
- * Licensed under the GNU GPL. For full terms see the file COPYING.
+ * printer_friendly top frame
  *
  * top frame of printer_friendly_main.php
  * displays some javascript buttons for printing & closing
  *
+ * @copyright (c) 1999-2004 The SquirrelMail Project Team
+ * @license http://opensource.org/licenses/gpl-license.php GNU Public License
  * @version $Id$
  * @package squirrelmail
  */
@@ -21,11 +19,6 @@ define('SM_PATH','../');
 
 /* SquirrelMail required files. */
 require_once(SM_PATH . 'include/validate.php');
-require_once(SM_PATH . 'functions/strings.php');
-require_once(SM_PATH . 'config/config.php');
-require_once(SM_PATH . 'include/load_prefs.php');
-require_once(SM_PATH . 'functions/page_header.php');
-require_once(SM_PATH . 'functions/html.php');
 
 displayHtmlHeader( _("Printer Friendly"),
              "<script language=\"javascript\" type=\"text/javascript\">\n".
Unnamed repository; edit this file 'description' to name the repository.