mod: валидатор форм, парсера и моделей (09) уделение дублей в синонимах других записей при редактировании через админку

lpon_site/frontend/utils.py

@@ -332,11 +332,6 @@ def validate_entity_for_admin_form(form_instance, cleaned_data,
     """
     from django.utils.html import mark_safe
 
-    # ПЕРЕД ВАЛИДАЦИЕЙ: проверяем GET параметр ignore_validate
-    # Если пользователь нажал красную кнопку, addGetParam добавит GET параметр к URL
-    if request and request.GET.get('ignore_validate') == '1':
-        return
-
     # Получаем класс модели из метаинформации формы
     model_class = form_instance.Meta.model
 
@@ -348,6 +343,9 @@ def validate_entity_for_admin_form(form_instance, cleaned_data,
     if not main_field_value:
         return
 
+    # Нормализуем основное значение для сравнения (как в validate_for_duplicates)
+    normalized_main_value = normalize_string(main_field_value)
+
     # Вызываем основной валидатор дубликатов
     result = validate_for_duplicates(
         model_class=model_class,
@@ -384,8 +382,8 @@ def validate_entity_for_admin_form(form_instance, cleaned_data,
                 # Объединяем все найденные дубликаты в один список
                 dup_list = ", ".join(dup_links)
 
-                # Для случая IS_DUPLICATE отключена проверка force_ignore_validate, т.к. это критическая ситуация
+                # Для случая IS_DUPLICATE всегда выбрасываем ошибку, т.к. это критическая ситуация
-                # и проверяемом поле часто unique=True на уровне модели.
+                # и поле часто имеет unique=True на уровне модели.
                 raise ValidationError(
                     mark_safe(
                         f"ОШИБКА: Найдено ПОЛНОЕ совпадение! "
@@ -395,6 +393,32 @@ def validate_entity_for_admin_form(form_instance, cleaned_data,
 
             case ValidateMatchType.FIND_IN_SYNONYM:
                 # ОБРАБОТКА СОВПАДЕНИЙ В СИНОНИМАХ
+                # Проверяем: это запрос с подтверждением (ignore_validate=1) или первоначальная проверка?
+                if request and request.GET.get('ignore_validate') == '1':
+                    # РЕЖИМ: ОБХОД ВАЛИДАЦИИ (пользователь нажал красную кнопку и подтвердил "Я проверил и уверен!")
+                    # Тихо удаляем найденные совпадения из синонимов других записей
+                    for duplicate_record in duplicates_queryset:
+                        # Получаем текущие метаданные записи
+                        dup_metadata = getattr(duplicate_record, metadata_field_name) or {}
+
+                        # Если в метаданных есть синонимы, удаляем из них текущее значение
+                        if KEY_SYNONYM in dup_metadata and isinstance(dup_metadata[KEY_SYNONYM], list):
+                            # Удаляем нормализованное значение из списка синонимов
+                            dup_metadata[KEY_SYNONYM] = [
+                                syn for syn in dup_metadata[KEY_SYNONYM]
+                                if normalize_string(syn) != normalized_main_value
+                            ]
+                            # Сохраняем обновленные метаданные
+                            setattr(duplicate_record, metadata_field_name, dup_metadata)
+                            # Сохраняем запись (обновляем только поле с метаданными)
+                            duplicate_record.save(update_fields=[metadata_field_name])
+                    # Выходим без ошибки в админку, т.к. пользователь "проверил и уверен!"
+                    # Синонимы из удалены, запись сохранится нормально
+                    return
+
+                else:
+                    # РЕЖИМ: ПЕРВОНАЧАЛЬНАЯ ПРОВЕРКА
+                    # Показываем пользователю красную кнопку подтверждения с информацией о совпадениях
                     for dup in duplicates_queryset:
                         rel_url = f"../{dup.pk}/change/" if form_instance.instance.pk is None else f"../../{dup.pk}/change/"
                         dup_value = getattr(dup, main_field_name, '?')
@@ -412,7 +436,7 @@ def validate_entity_for_admin_form(form_instance, cleaned_data,
                       <button type="button" onclick="markSubmitButtonsToIgnoreValidation();">
                         <big>Я проверил и уверен!</big><br/>
                         Сохранить, несмотря на синонимы.<br/>
-                        <i>Точно совпадения в синонимах других записей будут удалены.</i>
+                        <i>Точные совпадения в синонимах других записей будут удалены.</i>
                       </button>
                       <em>Теперь нажмите стандартные кнопки сохранения снизу, чтобы сохранить.</em>
                     </div>

public/static/js/form-field-watcher.js

@@ -37,12 +37,12 @@ function markSubmitButtonsToIgnoreValidation() {
   // Находим все submit-кнопки на странице и добавляем им класс
   // form-field-watcher.js потом отследит добавление класса через MutationObserver
   // и добавит соответствующие onclick обработчики
-    document.querySelectorAll('input[type=submit]').forEach(function(btn) {
+  document.querySelectorAll('input[type=submit]').forEach(function (btn) {
     btn.classList.add('force-ignore-validation');
   });
 }
 
-document.addEventListener('DOMContentLoaded', function() {
+document.addEventListener('DOMContentLoaded', function () {
   // Находим все submit-кнопки администратора
   let submitButtons = document.querySelectorAll('input[type=submit]');
 
@@ -88,7 +88,7 @@ document.addEventListener('DOMContentLoaded', function() {
     // 1. Удаляем класс force-ignore-validation (кнопки вернут нормальный цвет)
     // 2. Удаляем onclick обработчик
     // 3. Восстанавливаем оригинальный action формы
-        submitButtons.forEach(function(btn) {
+    submitButtons.forEach(function (btn) {
       if (btn.classList.contains('force-ignore-validation')) {
         btn.classList.remove('force-ignore-validation');
         removeOnclickHandler(btn);
@@ -98,12 +98,12 @@ document.addEventListener('DOMContentLoaded', function() {
 
     // Скрываем сообщения об ошибках валидации
     let errorNotes = document.querySelectorAll('.errornote, .errorlist');
-        errorNotes.forEach(function(errorElement) {
+    errorNotes.forEach(function (errorElement) {
       errorElement.style.display = 'none';
     });
   }
 
-    formInputs.forEach(function(input) {
+  formInputs.forEach(function (input) {
     // Слушаем оба события: 'change' для обычных input/select
     // и 'input' для CodeMirror и других редакторов
     input.addEventListener('change', handleChange);
@@ -112,8 +112,8 @@ document.addEventListener('DOMContentLoaded', function() {
 
   // Мониторим изменение класса force-ignore-validation на кнопках
   // Используем MutationObserver для отслеживания добавления/удаления класса
-    const observer = new MutationObserver(function(mutations) {
+  const observer = new MutationObserver(function (mutations) {
-        mutations.forEach(function(mutation) {
+    mutations.forEach(function (mutation) {
       if (mutation.type === 'attributes' && mutation.attributeName === 'class') {
         let btn = mutation.target;
         if (btn.tagName === 'INPUT' && btn.type === 'submit') {
@@ -131,7 +131,7 @@ document.addEventListener('DOMContentLoaded', function() {
   });
 
   // Настраиваем observer для отслеживания изменения класса
-    submitButtons.forEach(function(btn) {
+  submitButtons.forEach(function (btn) {
-        observer.observe(btn, { attributes: true, attributeFilter: ['class'] });
+    observer.observe(btn, {attributes: true, attributeFilter: ['class']});
   });
 });