Your Newsletter Module

stokerpoons · 07-11-2004, 10:09 PM

Quote:

Originally Posted by Mighty_Y

find in modules/Newsletter/index.php

Code:

    $query = "SELECT email, uid FROM ${prefix}_users WHERE uid = '$cookie[0]'";

and replace with

Code:

    $query = "SELECT user_email, user_id FROM $prefix_users WHERE user_id = '$cookie[0]'";

try that and report back

Hi Mighty Y! I have tried this but still get the error with line 48

error

Quote:

Warning: Supplied argument is not a valid MySQL result resource in /home/stokerpoons/public_html/modules/Newsletter/index.php on line 48

line 48

Quote:

list($usermail) = mysql_fetch_row($result);

Thanks anyway!

[/quote]

Mighty_Y · 07-12-2004, 11:02 AM

replace

Code:

global $prefix, $user, $modul_name;

with

Code:

global $prefix, $user, $modul_name, $db;

and replace

Code:

    $query = "SELECT email, uid FROM ${prefix}_users WHERE uid = '$cookie[0]'";
    $result = mysql_query($query);
    list($usermail) = mysql_fetch_row($result);

with

Code:

    $query = "SELECT user_email, user_id FROM ".$prefix."_users WHERE uid = '$cookie[0]'";
    $result = $db->sql_query($query);
    list($usermail) = $db->sql_fetchrow($result);

Normally it should work now, or atleast another error should pop in

stokerpoons · 07-12-2004, 11:23 AM

I dont get any errors anymore but there are still no emails added to the list even though it confirms that the email has been added :?
It even sends the email saying that the user has been added but no joy!

Mighty_Y · 07-12-2004, 11:38 AM

I'm gonna modify this thing

A little bit patience please

stokerpoons · 07-12-2004, 11:49 AM

No worries Mighty_Y!

Your a Star

Mighty_Y · 07-12-2004, 12:18 PM

here is the fully working (atleast on my system

) version of the newsletter module:
http://users.skynet.be/fa091229/newsletter1.70.zip

It won't stay on my server forever, so you are warned

(it stays there, till I forget what it is or why it is there, and then I delete it

)

stokerpoons · 07-12-2004, 12:20 PM

Cheers Mighty_Y. I will give it a shot later on tonight!

mikem · 07-12-2004, 01:35 PM

Sorry to chime in again.
While you wait, you can "try" this one.
http://www.fauxpas.org/mystuff/Newsletter6.5.zip

mikem

Mighty_Y · 07-12-2004, 01:42 PM

Mikem, I compared yours with my modified version and it seems I went a little bit further in adjusting (I use $db almost everywhere) so i have put my index.php in your package (with the block and more structurised):
http://users.skynet.be/fa091229/Newsletter.zip

mikem · 07-12-2004, 01:49 PM

Quote:

Originally Posted by Mighty_Y

Mikem, I compared yours with my modified version and it seems I went a little bit further in adjusting (I use $db almost everywhere) so i have put my index.php in your package (with the block and more structurised):
http://users.skynet.be/fa091229/Newsletter.zip

roger..yes, I had not made the new db changes for the new sql layering..blah blah.... :roll: ..thanks Mighty..hopefully that will work for him :wink: