PDA

View Full Version : Enewsletter - Bug


icarrion
12-01-2004, 09:28 AM
Hello Staff,

There is a bug in the enewsletter mod in 4.7 pro version. Basically, before it would allow the webmaster to send newsletters to their secure users but the functionality was removed which I believe is a bug (please tell me if otherwise). Right now the table "sec_users" is not being pulled to create campaigns. Below is the fix.

Please modify the file "sohoadmin/program/modules/mods_full/enewsletter/create_campaign.php" and replace line 254 with the following:

if ( eregi("UDT_", $tb_names[$x]) || $tb_names[$x] == "sec_users" ) {

Hope this helps!

icarrion
12-01-2004, 09:36 AM
Hello Staff,

There is a bug in the enewsletter mod in 4.7 pro version. Basically, before it would allow the webmaster to send newsletters to their secure users but the functionality was removed which I believe is a bug (please tell me if otherwise). Right now the table "sec_users" is not being pulled to create campaigns. Below is the fix.

Please modify the file "sohoadmin/program/modules/mods_full/enewsletter/create_campaign.php" and replace line 254 with the following:

if ( eregi("UDT_", $tb_names[$x]) || $tb_names[$x] == "sec_users" ) {

Hope this helps!

My fault, my customer notified me that there is more tables that belong like the "login" and "cart_customers" tables. So please ignore the previous code and use the following instead:

if ( eregi("UDT_", $tb_names[$x]) || $tb_names[$x] == "sec_users" || $tb_names[$x] == "cart_customers" || $tb_names[$x] == "login") {

Thanks!

icarrion
12-01-2004, 09:48 AM
My fault, my customer notified me that there is more tables that belong like the "login" and "cart_customers" tables. So please ignore the previous code and use the following instead:

if ( eregi("UDT_", $tb_names[$x]) || $tb_names[$x] == "sec_users" || $tb_names[$x] == "cart_customers" || $tb_names[$x] == "login") {

Thanks!

Oops, I hate to do this to ya. I should have stayed with the same coding convention. I guess eregi has speed benefits. So here it goes... :

if ( eregi("UDT_", $tb_names[$x]) || eregi("sec_users",$tb_names[$x]) || eregi("cart_customers",$tb_names[$x]) || eregi("login",$tb_names[$x])) {

Ruben
12-05-2004, 08:58 PM
Icarrion, can you please post this also in the bugz forum? http://bugz.soholaunch.com is you have not done this yet.

thanks,