Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Blocked users get the newsletter (admin/mailing) #656

Closed
q2apro opened this issue Aug 15, 2018 · 1 comment
Closed

Blocked users get the newsletter (admin/mailing) #656

q2apro opened this issue Aug 15, 2018 · 1 comment

Comments

@q2apro
Copy link

q2apro commented Aug 15, 2018

Blocked users are not excluded from the mailing. But they should.

I guess the mailing function is in \db\users.php

function qa_db_users_get_mailing_next($lastuserid, $count)
{
	return qa_db_read_all_assoc(qa_db_query_sub(
		'SELECT userid, email, handle, emailcode, flags FROM ^users WHERE userid># ORDER BY userid LIMIT #',
		$lastuserid, $count
	));
}

The query needs an addition to exclude the users with flag QA_USER_FLAGS_USER_BLOCKED.

@svivian
Copy link
Collaborator

svivian commented Dec 1, 2018

Fixed by #657

@svivian svivian closed this as completed Dec 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants