From b899bb854b1ba789969d001ea005af044ec5fbcf Mon Sep 17 00:00:00 2001
From: kink <kink@7612ce4b-ef26-0410-bec9-ea0150e637f0>
Date: Sun, 18 May 2003 11:25:20 +0000
Subject: [PATCH] Fix comp_in_new.

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@4876 7612ce4b-ef26-0410-bec9-ea0150e637f0
---
 ChangeLog           |  2 ++
 doc/compose.txt     | 12 ++++--------
 src/addressbook.php |  2 +-
 src/search.php      |  2 +-
 4 files changed, 8 insertions(+), 10 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 504c5f3d..15061e31 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -18,6 +18,8 @@ Version 1.5.0 -- CVS
   - Fix for Folders being listed in create/remove/rename operations 
     (#725443, #722823, #729225)
   - Fix for bad attachment view link (#697381, #729295)
+  - Fix comp_in_new in search and addressbook not having right parameters (#731768).
+
 
 **************************************
 *** SquirrelMail Stable Series 1.4 ***
diff --git a/doc/compose.txt b/doc/compose.txt
index 3d507353..9cf0904e 100644
--- a/doc/compose.txt
+++ b/doc/compose.txt
@@ -3,16 +3,12 @@ Using the comp_in_new function
 
 comp_in_new is the javascript function to popup a new compose window.
 
-the function needs two arguments:
-1) compose new message (true or false)
-2) url with arguments to the compose window.
-
-if argument 1 is true then argument 2 should be ""
-
+the function needs one argument:
+1) url with arguments to the compose window.
 
 example 1:
-<a href="javascript:void(0)" onclick="comp_in_new(true,"")> 
+<a href="javascript:void(0)" onclick="comp_in_new("")> 
 
 example 2:
-<a href="javascript:void(0)" onclick="comp_in_new(false,"../../src/compose.php?argument1=true&amp;argument2=test")> 
+<a href="javascript:void(0)" onclick="comp_in_new("../../src/compose.php?argument1=true&amp;argument2=test")> 
 
diff --git a/src/addressbook.php b/src/addressbook.php
index b37701f1..4ba7e969 100644
--- a/src/addressbook.php
+++ b/src/addressbook.php
@@ -387,7 +387,7 @@ if ($showaddrlist) {
                 }
             $email = $abook->full_address($row);
             if ($compose_new_win == '1') {
-                echo '<a href="javascript:void(0)" onclick=comp_in_new(false,"compose.php?send_to='.rawurlencode($email).'")>';
+                echo '<a href="javascript:void(0)" onclick=comp_in_new("compose.php?send_to='.rawurlencode($email).'")>';
             }
             else {
                 echo '<A HREF="compose.php?send_to=' . rawurlencode($email).'">';
diff --git a/src/search.php b/src/search.php
index 89823839..4fca3ea9 100644
--- a/src/search.php
+++ b/src/search.php
@@ -254,7 +254,7 @@ if ($mailbox == 'All Folders') {
 if (isset($composenew) && $composenew) {
     $comp_uri = "../src/compose.php?mailbox=". urlencode($mailbox).
 		"&amp;session=$composesession&amp;attachedmessages=true&amp";
-    displayPageHeader($color, $mailbox, "comp_in_new(false,'$comp_uri');", false);
+    displayPageHeader($color, $mailbox, "comp_in_new('$comp_uri');", false);
 } else {
     displayPageHeader($color, $mailbox);
 }
-- 
2.25.1