Can't create newsletters - HELP!

[np1001]

I'm having mega problems with the Newsletter manager in OSC 2.2ms2-051113.

 

When I create a newsletter and click save I get the message "Error: Newsletter module required" which I don't understand because the newsletter files are there.

 

Am I missing something here? Any help would be very welcome.

[Guest]

I'm having mega problems with the Newsletter manager in OSC 2.2ms2-051113.

 

When I create a newsletter and click save I get the message "Error: Newsletter module required" which I don't understand because the newsletter files are there.

 

Am I missing something here? Any help would be very welcome.

 

Same problem here... Anyone?

[Guest]

Same problem here... Anyone?

 

 

Yes I have the same prolem, hunting this forum for solutions. I installed the newsindex_v07 contribution that maybe the reason.

 

Wolf

[Guest]

Yes I have the same prolem, hunting this forum for solutions. I installed the newsindex_v07 contribution that maybe the reason.

 

Wolf

 

 

Found it hopefully no other problems in admin/newsletter.php, about line 44

 

if (empty($module)) {

$messageStack->add(ERROR_NEWSLETTER_MODULE, 'error');

$newsletter_error = true;

}

 

Change to

 

if (empty($module)) {

$messageStack->add(ERROR_NEWSLETTER_MODULE, 'error');

$newsletter_error = false;

}

 

 

That did it for me

 

Wolfgang

[Evertje]

worked for me too :)

[arbiterxero]

Actually it's just using the wrong variable, a better solution is to fix it properly rather than ignore the error

 

 

change

 

$newsletter_module = tep_db_prepare_input($HTTP_POST_VARS['module']);

$title = tep_db_prepare_input($HTTP_POST_VARS['title']);

$content = tep_db_prepare_input($HTTP_POST_VARS['content']);

 

$newsletter_error = false;

if (empty($title)) {

$messageStack->add(ERROR_NEWSLETTER_TITLE, 'error');

$newsletter_error = true;

}

 

if (empty($module)) {

$messageStack->add(ERROR_NEWSLETTER_MODULE, 'error');

$newsletter_error = true;

}

 

 

since they should both be the same I assume....oops!