Newsletter-Only Subscriptions for v1.3x

chac416:

geeee, i reinstalled the subscribtion, and now i forgot how i fixed the sidebox width issue... which file was that...

nevermind, took the patch file from ur site....

now here's another problem after reinstalling the subscription and installing the subscription manager..

thats what i get when i click on Subscribtion manager in admin...

1054 Unknown column 'confirmed' in 'field list'
in:
[select subscriber_id, customers_id, email_address, email_format, confirmed, subscribed_date from zen_subscribers order by email_address limit 0, 20]
If you were entering information, press the BACK button in your browser and re-check the information you had entered to be sure you left no blank fields.

and this when i first opened the admin page

1054 Unknown column 'confirmed' in 'where clause'
in:
[select count(*) as count from zen_subscribers where confirmed= '1' or customers_id >= '1']
If you were entering information, press the BACK button in your browser and re-check the information you had entered to be sure you left no blank fields.

okay, i found our why, fields where missing from the database...

can someone look into his database and copy/paste for me the attributes of the following fields (confirmed, subscribed_date) thank u so much in advance :yes:

NG,
i have decided to use phplist for sending out my newsletter.
there's one thing i still want to do. i want my subscribers to unsubscribe using ur module not phplist.
i.e. link them back to the unsubscribe page on zencart... can you provide me with the link of the unsubscribe page... thanks lot,

You mean the 'unsubscribe from newsletter' link or the one that goes on the bottom of newsletters?

The former is an option in the cart config 'show newsletter unsubscribe link' (don't have an admin handy here to look up the exact page for you).

The latter, should be the same link, with the email changed to the subscriber's email (or none and they enter their email). It would look something like:

http://yoursite.com/yourstore/index.php?main_page=unsubscribe

-Ng_

I installed the newsletter module and all seems good, except...I am testing the confirm subscription request...in the subscription management section it either says C or N, but never P (pending) when I have not yet responded to the confirm email (during testing).

Did I miss installing something? I think I have the newest version.

No you didn't miss anything. Someone else caught it a few posts back. It's fixed in the release coming tonight (in a couple hours...an evil wonderful friend surprised me with the new book from a fave author and I kinda had to read a couple chapters...or six so got delayed [but I told myself no more til this is done, so it will be quick.] :).

-Ng_

Okay folks, it's up. Apologies for the delay.

I've submitted it to the downloads section, but if there's a delay, you can grab it from my site: http://www.notgoddess.com/scripts/newsletter-subscribe

As ever I must stress - make backups before you install! :)

If you already have v2.0.2 installed, you should see an 'update' button instead of an install button.

This contrib uses the 'create table if not exists' syntax on the subscriber's table, and the 'replace' syntax for the various defines. If you have mysql 4.1+ you shouldn't have a problem. If you do, you'll have to peruse the file and update manually.

Report any issues and I'll get patches out asap.

...I did mention that you should backup first, right?

Enjoy.

-Ng_

Hi!

I have some strange problem with the unsubscribe action on the website.
When i click unsubcribe from newsletter and then the button Unsubscibe
it gives me the message that the emailadres below is unsubscibed....
I did not put in a e-mailadres that should be removed from the list....and offcourse there is no emailaddress showing.

Any idea on this one?

oeps, maybe just have to install the updated version 2.03 , am using 2.02 now.

Do upgrade if you wish, but it's there in 2.0.3 too :) (I guess I should have put out a last call :)

If you edit the template/your_template/templates/tpl_unsubscribe_default.php and change line 14 from:

<?php if (!isset($_GET['action']) || ($_GET['action'] != 'unsubscribe')) { ?>

to

<?php if (empty($unsubscribe_address) || ((!isset($_GET['action']) || ($_GET['action'] != 'unsubscribe')))) { ?>

It will always show the "We're sorry you want to unsubscribe" text for blank emails.

The unsubscribe module is set so if a person enters a nonsense email it'll still tell them it's unsubscribed so people can't use it as a way to suss out emails. That's why it always repeats the email a person enters (in case they misspelled their email).

If people wish I can change that so it gives a 'invalid email' error message instead....3 votes gets it.

I also noted the sorting works wonderfully...until you try to click on a string to edit.
Then it's resetting to the default display.

Also, I'm not sure if it's a 'first record' issue or 'customer' issue but clicking on the first address to edit gives you a blank edit box.

I'll work on both of these tonight and get out a patch.

-Ng_

Thanks a lot! I'll do the first patch as suggested and would appreciate it a lot if you continue with the very fine contribution!!!

NotGoddess:

This contrib uses the 'create table if not exists' syntax on the subscriber's table, and the 'replace' syntax for the various defines. If you have mysql 4.1+ you shouldn't have a problem. If you do, you'll have to peruse the file and update manually.
.

-Ng_

-NG - I'm running mysql 4.0.27 it seems
So when I pressed "upgrade" it gave errors. Of course, stupidly I didn't record the errors...

I was upgrading from newsletter subscribe 202

I presume I can force the errors to appear again if I reupload all the admin files?

also, I did the line change above for tpl_unsubscribe_default.php
if I leave the email box blank and press unsubscribe I get re-dired to the login page...is this what was supposed to happen? Just checking :D

thanks!

woodlandsprite:

I presume I can force the errors to appear again if I reupload all the admin files?

Well I tried that and no upgrade button appeared...I hope I'm okay :P

[quote]
also, I did the line change above for tpl_unsubscribe_default.php
if I leave the email box blank and press unsubscribe I get re-dired to the login page...is this what was supposed to happen? Just checking :D
[/quote

well, put it back to the original code, and the same thing happens. I note no difference between the original code vs

Code:

re: unsubscribe-I'll have to check into it. I may have made a change with the header as well. the redirect is supposed to happen under the 'basic' model because people have to be logged in to unsubscribe normally.

re the db errors.
I was probably the 'create table if not exists' syntax. Even tho the manual says this is supported by MySQL 3.23 onward, I've found it shakey at versions lower than 4.1. I guess I'll have to find another way to ensure the table exists (I know of a few but each has it's own drawbacks).

The main alteration of the update was to add the test groups/configuration values/email groups as outlined in the readme. If you have all those, you are okay.

If anyone is hesitant, just wait...I'll work on an alternative tonight/tomorrow and get the patch out on Friday. You can still put in the new files...just don't hit the 'update' button.

-Ng_

btw. Thank you to all who've reported issues...while I in one sense hate getting them :) I need to get any problems fixed asap.

I have found an issue - I am now receiving notices whenever a newsletter only subscrition is created. I have this set to 0 in admin-email options.

(and the email address to send notice to isnt defined - it's sending to my store owner default email)

Possibly a result of my database not updating properly?

NotGoddess:

If anyone is hesitant, just wait...I'll work on an alternative tonight/tomorrow and get the patch out on Friday. You can still put in the new files...just don't hit the 'update' button.
:cry: is it too late for me? :P I'll go look at my db and see if I can decipher what is supposed to be there...
I suppose I can attempt a restore of my DB via phpmyadmin...

btw. Thank you to all who've reported issues...while I in one sense hate getting them :) I need to get any problems fixed asap.
thank you for such a wonderful contrib. I just wish I was more savvy so I could debug too!

No, it's not too late :) I just might have a little difficulty helping until late tonight (work). if you want, for now edit modules/pages/subscribe/header_php.php

I found an error that is probably the problem:
At the very bottom you'll see these lines:

 if(defined('NEWSONLY_SUBSCRIPTION_CC_STATUS') && 
    defined('NEWSONLY_SUBSCRIPTION_CC') &&
    NEWSONLY_SUBSCRIPTION_CC_STATUS != '') {

change to

if(defined('NEWSONLY_SUBSCRIPTION_CC_STATUS') && 
   defined('NEWSONLY_SUBSCRIPTION_CC') &&
   NEWSONLY_SUBSCRIPTION_CC_STATUS == 1 &&
   NEWSONLY_SUBSCRIPTION_CC != '') {

...and I changed my mind. I think, given this very very silly error on my part along with another earlier I've proven I shouldn't be doing a release at 2am.
I'll work on the patch, but will wait until tomorrow night to release so I can do it after a good night's rest. :)

-Ng_

NotGoddess:

The main alteration of the update was to add the test groups/configuration values/email groups as outlined in the readme. If you have all those, you are okay.

NG - I checked my subscribers table in my db.
The columns I have in there are:
Subscriber ID
customers ID
email address
email format
confirmed
subscribed date

In admin->email options I do have the
-send notice of newsletter subscription only to status (which seems to not be working properly as mine is set to 0 and I am still getting notified of newsletter only subscriptions)
-send notice of newsletter only subscriptions to
-newsletter test group email

Under admin->tools->send email
even tho I have defined a newsletter test group email, there is no such item/address in the dropdown in the email tool.

NotGoddess:

...and I changed my mind. I think, given this very very silly error on my part along with another earlier I've proven I shouldn't be doing a release at 2am.
I'll work on the patch, but will wait until tomorrow night to release so I can do it after a good night's rest. :)

-Ng_

lol
Boy I know what that's like! when I first started working on my cart, I worked on it until stupid stupid hours....way beyond the point of being productive!

looking forward to the patch :) (and hopefully the news also that my db is complete ;) )

woodlandsprite:

-NG - I'm running mysql 4.0.27 it seems
So when I pressed "upgrade" it gave errors. Of course, stupidly I didn't record the errors...

NG:
I went back, reinstalled my backup version of my database.
The error wasn't with the "if not exists"

The error is as follows:

1146 Table 'dbname.query_builder' doesn't exist in:

[INSERT INTO query_builder (query_category, query_name, query_description, query_string) VALUES ('email,newsletters', 'Email Test Group - Newsletter-only subscribers', 'Returns name and email address of Newsletter-only subscribers designated in Email test group configuration.', 'SELECT s.email_address as customers_email_address FROM TABLE_SUBSCRIBERS as s LEFT JOIN TABLE_CONFIGURATION as q on LOCATE( s.email_address, q.configuration_value) >= 1 WHERE configuration_key = ''NEWSONLY_SUBSCRIPTION_TEST_GROUP'' ') ]

Hope this helps you with the patch :)