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

Почта #264

Open
dima-bzz opened this issue Oct 3, 2014 · 10 comments
Open

Почта #264

dima-bzz opened this issue Oct 3, 2014 · 10 comments

Comments

@dima-bzz
Copy link
Contributor

dima-bzz commented Oct 3, 2014

После обновления не отправляются письма. Если убрать строчку и вернуть старую строчку то работает.
$stmt = $dbConnection->prepare('SELECT email,login from users where id=:client_id and status=:n and priv=:n1');
$stmt->execute(array(':n'=>'1',':client_id'=>$client_id,':n1'=>'2'));

@yaroslavsnisarua
Copy link
Collaborator

это вы вернули старую строку?

@dima-bzz
Copy link
Contributor Author

dima-bzz commented Oct 3, 2014

Нет это новая строка

@yaroslavsnisarua
Copy link
Collaborator

можете кинуть ссылку на старую или тут написать как было и что стало?

@dima-bzz
Copy link
Contributor Author

dima-bzz commented Oct 3, 2014

Было так
$stmt = $dbConnection->prepare('SELECT email from users where id=:client_id and status=:n');
$stmt->execute(array(':n'=>'1',':client_id'=>$client_id));

Стало
$stmt = $dbConnection->prepare('SELECT email,login from users where id=:client_id and status=:n and priv=:n1');
$stmt->execute(array(':n'=>'1',':client_id'=>$client_id,':n1'=>'2'));

@yaroslavsnisarua
Copy link
Collaborator

Эта строка отвечает за отправку писем обычному пользователю.
Попробуйте вернуть как на мастер-ветке, и создайте заявку на обычного пользователя (должны прийти письма ему и нач его отдела) а так же желательно смотреть логи сервера.
PS. у меня на продакшн - работает отлично.
А вобще то мы будем полностью перерабатывать систему уведомлений. Так что это пока временное решение.

@dima-bzz
Copy link
Contributor Author

dima-bzz commented Oct 3, 2014

Так priv = 2 Это главный начальник!

@yaroslavsnisarua
Copy link
Collaborator

Да всё верно, так я хочу понять, при каких именно условиях не отправляет заявку главному начальнику?

@dima-bzz
Copy link
Contributor Author

dima-bzz commented Oct 3, 2014

Попробую разобраться в понедельник в чем у меня проблема. Но отдно я знаю что как я поменял код на исправленный вами у меня перестала отправляться почта. Буду разбираться)))

@dima-bzz
Copy link
Contributor Author

dima-bzz commented Oct 6, 2014

Посмотрел код. Получается что когда создаешь заявку то почта отправляется пользователь по type=new user. А там получается что в новом коде идет выборка по пользователям с правами главного начальника. У меня получается в этом загвоздка. Поправьте если я неправ. Может просто запутался(((((

@yaroslavsnisarua
Copy link
Collaborator

$type == "new_all" - отправляется всем пользователям одного отдела

$type == "new_coord" - отправляется координатору (-ам) отдела

$type == "new_user" - отправляется пользователю

Можете пока не заморачиваться, всё-равно буду перерабатывать систему уведомлений!

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