Wrong username in Subscripton email

zews · August 13th, 2007, 06:21 AM

In subscription emails for a category, the username in the email is not the username of the user whom uploaded the ad.

$Globals['pp_lang']['subnotify'] = "{$tusername} has uploaded one or more ads to a category you subscribe to.

In my case the $tusername returns the username of another user who subscribes to the same category.

Chuck S · August 13th, 2007, 09:23 AM

Already a reported bug and fixed in the current downloaded code in the file image-inc.php should read like this

Code:

Content visible to verified customers only.

zews · August 13th, 2007, 10:10 AM

I have the 8/2/07 build, and that is the code above.

So I have the problem with this code.

the emails.php code is

Quote:

$Globals['pp_lang']['subnotify'] = "{$tusername} has uploaded one or more ads to a category you subscribe to.

Does $tusername refer to the poster of the ad?

Chuck S · August 13th, 2007, 10:17 AM

duh use $User['userid'] like above

zews · August 13th, 2007, 10:20 AM

Its changed back to how it was?

Chuck S · August 13th, 2007, 03:13 PM

wait my code before was right $nuserid is right

zews · August 14th, 2007, 09:37 AM

After I changed to $User['userid'] I didn't get any subscription emails, when I changed back to $nuserid I get emails, but the username in the email is wrong.

In the email the name is the name of another subscriber, not the name of the one who has posted the new ad.

Chuck S · August 14th, 2007, 10:26 AM

Code:

Content visible to verified customers only.

The code in image-inc.php as I stated above should be this. Look at the email the user's username uploading the ad is indeed $User['userid'] to grab his username.

Code:

Content visible to verified customers only.

zews · August 14th, 2007, 11:48 AM

This is not working,

what happens now is that if I upload an ad, I get 3 emails about it.

If another user that is not subscribing to the category uploads an ad, I am not getting any email about it, but that user is getting 3 notify emails.

Something must be wrong somewhere. Do you have a site running this software with the newest build where I can try this out and se that it works like its supposed to?

zews · August 14th, 2007, 11:54 AM

I also get reports about people getting another email about winning auctions every day, and after upgrading all the old auctions where reactivated.

Maybe there is something wrong in by database or something? I have read something about a timestamp-error - could it be something like that causing my problems? What can I do to investigate that?

zews · August 14th, 2007, 02:34 PM

I did a new download and a new install as a test. The email looked like this when an other user posted to a category i subsribed to:

Quote:

has uploaded one or more ads to a category you subscribe to.

Image name:
Title: Weeee
Description: ffghfgf

The username was missing.

When I uploaded to the category myself, the email was correct, containing my username.

Chuck S · August 14th, 2007, 04:41 PM

HMM mine is working I tested it and posted here let me test again

Chuck S · August 14th, 2007, 04:51 PM

HMM mine is working I tested it and posted here let me test again

This cheat line should work also

$tusername = $User['username'];

zews · August 15th, 2007, 09:36 AM

Where did you test it? Can I test also?

Where should I place that cheat line?

Chuck S · August 15th, 2007, 10:15 AM

technically you could do a hard coded test by getting rid of that list line and placing the line above thee

zews · August 15th, 2007, 10:32 AM

Like this?

Quote:

if ( $nuserid > 0 )
{
$tusername = $User['username'];
// list( $tuserid, $tusername, $tmail ) = get_userinfo( "", $nuserid );

zews · August 15th, 2007, 10:33 AM

Where can I test this script on another server than mine? Do you have a public installation of the latest build where I can test?

Chuck S · August 15th, 2007, 10:41 AM

no not public I have a test server I use for doing trouble shooting try that on yours. I mean its pretty simple the name on that email from be the person who is uploading so that variable should work. I do not understand why yours is not pulling that name with the list statement

zews · August 15th, 2007, 11:20 AM

Yes, I have installed another copy to test.

I will try to understand the logic now.

Where can i find the get_userinfo() function in the code? Which file?

zews · August 15th, 2007, 01:44 PM

To send email to all the users that subscribe, we need the $tmail for each $nuserid. So this must be right in image-inc.php:

list( $tuserid, $tusername, $tmail ) = get_userinfo( "", $nuserid );

The problem must be in emails.php where

$Globals['pp_lang']['subnotify'] = "{$tusername} has uploaded one or more ads to a category you subscribe to.

need to be changed to:

$Globals['pp_lang']['subnotify'] = "{$User['username']} has uploaded one or more ads to a category you subscribe to.

The same place this line also need to be removed:

Image name: {$realname}

Can you please confirm that this is correct?