Уведомления администратора на мультивитринном сайте

rus · 03.Сентябрь.2021 12:43:42

Где возникла проблема: 4.13.2

Суть проблемы: На мультивитринном сайте в Уведомлениях администратора невозможно для разных витрин указать разные Email администраторов.

Как воспроизвести проблему: Администрирование - Уведомления - Уведомления администратора

Мне кажется это баг, т.к. вся суть функционала для владельцев мультивитринных лицензий в таком случае теряется.
Данный вопрос периодически поднимается в различных темах на форуме - Уведомления в мультивитринном сайте

lexizli · 17.Ноябрь.2021 08:46:20

@ikoshkin @ichaykin @pavel.sh и прочие дамы и господа от разработчиков, почему на этот явный косяк нет два с половиной месяца никакой реакции? А ведь это явный и очень грубый косяк.

Зачем менеджеру одной витрины получать на почту уведомления о заказах всех остальных витрин?

Когда этот косяк будет исправлен?

lexizli · 17.Ноябрь.2021 09:45:26

И еще дополню по этому багу:

в результате имеем нарушение федерального закона о персональных данных №152-ФЗ. Персональные данные одной витрины (фамилия, имя, отчество, телефон, адрес) могут попасть к администраторам других витрин, хотя посетитель не давал согласия на такое.

Ответственность за такое нарушение немалая.

Есть и уголовная, до двух лет лишения свободы.

lexizli · 19.Ноябрь.2021 06:13:56

Реакцию хотелось бы видеть

AndreyJ · 02.Апрель.2023 11:47:50

Апну тепу.
@ikoshkin

Asya · 07.Апрель.2023 13:14:09

Здравствуйте, @rus @lexizli @AndreyJ

Спасибо за ваши сообщения.

Я передала информацию разработчикам для рассмотрения.

daniil.b · 06.Июнь.2023 06:30:03

Здравствуйте, @rus!

Исправили проблему. Теперь уведомления о заказах, обратных звонках и отзывах отправляются только администратору, который привязан к текущей витрине. Фикс войдет в следующий релиз.
Для исправления проблемы собственными силами можно использовать патч:

gist.github.com

https://gist.github.com/danielborzov/b11560cfd9c872ad727fc53cb979f48d

app-Tygh-Notifications-Transports-Mail-MailMessageSchema.php.diff

diff --git a/app/Tygh/Notifications/Transports/Mail/MailMessageSchema.php b/app/Tygh/Notifications/Transports/Mail/MailMessageSchema.php
index e6c73803ce..614592dced 100644
--- a/app/Tygh/Notifications/Transports/Mail/MailMessageSchema.php
+++ b/app/Tygh/Notifications/Transports/Mail/MailMessageSchema.php
@@ -77,6 +77,11 @@ class MailMessageSchema extends BaseMessageSchema
      */
     public $storefront_id;
 
+    /**
+     * @var int

This file has been truncated. show original

app-Tygh-Notifications-Transports-Mail-ReceiverFinders-UserIdFinder.php.diff

diff --git a/app/Tygh/Notifications/Transports/Mail/ReceiverFinders/UserIdFinder.php b/app/Tygh/Notifications/Transports/Mail/ReceiverFinders/UserIdFinder.php
index fc670b9d9a..7d1b19442f 100644
--- a/app/Tygh/Notifications/Transports/Mail/ReceiverFinders/UserIdFinder.php
+++ b/app/Tygh/Notifications/Transports/Mail/ReceiverFinders/UserIdFinder.php
@@ -44,6 +44,10 @@ class UserIdFinder implements ReceiverFinderInterface
             $conditions['users.company_id'] = [0, $message_schema->to_company_id];
         }
 
+        if (fn_allowed_for('MULTIVENDOR') && !empty($message_schema->to_storefront_id)) {
+            $conditions['users.storefront_id'] = [0, $message_schema->to_storefront_id];

This file has been truncated. show original

app-addons-call_requests-func.php.diff

diff --git a/app/addons/call_requests/func.php b/app/addons/call_requests/func.php
index 355cb5d82d..a068daf62b 100644
--- a/app/addons/call_requests/func.php
+++ b/app/addons/call_requests/func.php
@@ -322,6 +322,11 @@ function fn_do_call_request($params, $product_data, &$cart, &$auth)
 
     $params['company_id'] = $company_id;
 
+    if (fn_allowed_for('MULTIVENDOR') && empty($params['storefront_id'])) {
+        /** @var \Tygh\Storefront\Storefront $storefront */

This file has been truncated. show original

There are more than three files. show original

О том, как применить патч, можно прочитать в нашей документации: Как применить diff-файл — Документация docs.cs-cart.ru 4.16.x

lexizli · 29.Август.2023 12:06:18

Пришлось снова вернуться к этой проблеме. Изменения внес (ядро пока не обновляли). Но, простите, совершенно не понимаю, что означает «администратор, который привязан к текущей витрине»?

Раньше почту, на которую должны приходить уведомления, можно было указать в настройках компании. Это оно?

Или это администраторы, которые тут:

Или нужно добавить почты в настройки уведомлений:

A2N · 30.Август.2023 09:48:24

Да, которые тут.
Я тоже мучился с этим вопросом. В итоге создал на каждую витрину по администратору с “главной почтой витрины” и вот этих псевдоадминов и прикрепил к уведомлениям для администраторов в качестве получателей. Работает.