email of quotes (Page 2 of 2)

@ed10: While I was documenting the code trace back to where the variables were set, you beat me to it!

You can actually dump the specific session variable:

echo print_r($_SESSION["wa_current_user"], true);

at the location of the send() and see what is the output!

Line 95 in admin/db/users_db.inc in function get_users_by_login():

$sql = "SELECT * FROM ".TB_PREF."users WHERE user_id=".db_escape($user_id);

Hence all fields must be available and in your instance, the real_name field may not have been set, though in the email instance, the username may be okay.

Ok, got it figured out. The last test email came in just like I wanted it to (almost-the salutation is not right yet). For the user I was logged in for didn't have any entry in user table for real_name so that is why it came up blank.  In the line:
$this->user = $_SESSION["wa_current_user"]->name;
'name' is taken from the 'real_name' field in the user table. (I thought it was the user_id)

At least that is what it looks like right now.  Time to take a break and look at it all again tomorrow to see if I have it all the way I want it.

Tomorrow is work on Mercurial day also, can't seem to get it to work right for me.

The patch didn't work here.  When I did the patch and took the  '-f sales@mydomain.com' out of var $add_params = '-f sales@mydomain';  then the returnpath went to: Return-Path: <"www-data@user"@mydomain.com>
so that is not right. with the -f flag in the add_parms it works.
I don't think you can set that Return-Path in the header, has to be separate parameter sent to postfix.

Take a look at:
me at arronwoods dot com
1 year ago
I've had all sorts of problems with scripts that don't set the "-f user@example.org" parameter when using mail() if postfix is the sendmail agent. ...
in http://php.net/manual/en/function.mail.php

ed10

What happens if both the headers method and the -f method are used together for Reply-to - does the mail deliver as desired? Make sure the users table has the email of the logged in user and try $_SESSION["wa_current_user"]->email and see if it appears correctly.

The RFC2822  uses CRLF and FA's class.mail.inc uses LF only.

Article 3.6.2 in the RFC2822 states:

Maybe some non compliant versions of Postfix may need the additional parameters like -f.....

Caveat - in some mail systems:

_________________________
[size=8]Composed/Posted with WYSIWYG BBCode Editor[/size]

Please check that the logged in user has an email entry in his record in the users table and then see if the $_SESSION["wa_current_user"]->email gets populated.

Unless we try having both Reply-to in header and using the -f parameter with no error, we cannot include the code in the base repo as it has to ensure that existing installs with no postfix and just sendmail will still work.

@ed10: thanks for the info.

Since the users.email field is not being used anywhere in the code if acquired using get_user_by_login() (as is done here in line 81 of includes/current_user.inc), the following patch will ensure it's availability in any script that uses it:

--- old/includes/current_user.inc    Thu Jan 17 19:16:02 2013
+++ new/includes/current_user.inc    Mon Feb 04 05:42:49 2013
@@ -22,6 +22,7 @@
     var $loginname;
     var $username;
     var $name;
+    var $email;
     var $company; // user's company
     var $pos;
     var $access;
@@ -120,6 +121,7 @@
                 $this->username = $this->loginname;
                 $this->prefs = new user_prefs($myrow);
                 $this->user = @$myrow["id"];
+                $this->email = @$myrow["email"];
                 update_user_visitdate($this->username);
                 $this->logged = true;
                 $this->last_act = time();