diff --git a/analysis/cleanup-worklog-2026-04-29.md b/analysis/cleanup-worklog-2026-04-29.md index 63d5f7f..838d192 100644 --- a/analysis/cleanup-worklog-2026-04-29.md +++ b/analysis/cleanup-worklog-2026-04-29.md @@ -773,3 +773,31 @@ Результат: - синтаксических ошибок в `v1/resources/notification_ls.cfc` не обнаружено - синтаксических ошибок в `v1/resources/instance_operation_default.cfc` не обнаружено + +## Обновление: выбран cleanup-проход по `instance_operation_cfs_param_ls.cfc` + +Дата фиксации: 2026-04-29. + +### Цель этапа + +- снять с большого файла верхний слой явных рассуждений и закомментированного кода +- не вычищать насильно все технические пояснения за один проход +- спорные смысловые комментарии при необходимости оставлять и учитывать через внешний список кандидатов + +### Результат этапа + +В `v1/resources/instance_operation_cfs_param_ls.cfc` удалены: +- большой верхний блок авторских рассуждений про URI и контекст +- закомментированные альтернативные поля и отладочные хвосты +- часть комментариев-оценок вокруг helper-вызовов, уникальности и временных веток +- закомментированные catch/debug-блоки, не участвующие в runtime + +Что оставлено: +- технические комментарии, которые все еще объясняют ограничения валидации и semantics параметров +- спорные пояснения, которые могут быть полезны при дальнейшей разборке файла + +### Проверка + +Измененный файл проверен на ошибки. +Результат: +- синтаксических ошибок в `v1/resources/instance_operation_cfs_param_ls.cfc` не обнаружено diff --git a/analysis/comment-removal-candidates-2026-04-29.md b/analysis/comment-removal-candidates-2026-04-29.md index d03b686..326d5e4 100644 --- a/analysis/comment-removal-candidates-2026-04-29.md +++ b/analysis/comment-removal-candidates-2026-04-29.md @@ -53,12 +53,13 @@ ### 4. `v1/resources/instance_operation_cfs_param_ls.cfc` -Ожидаемо спорный файл для будущего этапа: -- содержит большой пласт рассуждений автора, предупреждений и объяснений по валидации, уникальности и доступу +После частичной зачистки в файле все еще оставлены: +- комментарии по смыслу валидации, уникальности и ограничениям параметров +- часть пояснений к SQL-условиям и поведению проверок Почему вынесено заранее: - файл крупный и неоднозначный - лучше разбирать отдельным этапом, чем смешивать с обычной зачисткой компактных ресурсов Рекомендация: -- перед правкой сначала отделить чистый commented-out код от действительно полезных пояснений к ограничениям параметров \ No newline at end of file +- если потребуется более жесткая зачистка, пройтись по оставшимся поясняющим комментариям отдельно и удалить те, которые не нужны для понимания правил валидации \ No newline at end of file diff --git a/v1/resources/instance_operation_cfs_param_ls.cfc b/v1/resources/instance_operation_cfs_param_ls.cfc index f56477a..f12738a 100644 --- a/v1/resources/instance_operation_cfs_param_ls.cfc +++ b/v1/resources/instance_operation_cfs_param_ls.cfc @@ -1,27 +1,8 @@ - - - - + @@ -33,11 +14,6 @@ arguments.requestArguments.specificationId=usrCustomerInfo.specificationId; //In sort={prefix="sop", type="integer"}, is_sensitive={prefix="sop", type="boolean"}, depends_on_cfs_params={prefix="sop", type="string"} - } /> @@ -48,9 +24,7 @@ arguments.requestArguments.specificationId=usrCustomerInfo.specificationId; //In - - @@ -59,7 +33,6 @@ arguments.requestArguments.specificationId=usrCustomerInfo.specificationId; //In - @@ -71,7 +44,6 @@ arguments.requestArguments.specificationId=usrCustomerInfo.specificationId; //In - select @@ -125,15 +97,12 @@ arguments.requestArguments.specificationId=usrCustomerInfo.specificationId; //In - - - - + }/> + @@ -166,7 +135,6 @@ arguments.requestArguments.specificationId=usrCustomerInfo.specificationId; //In - @@ -176,22 +144,11 @@ arguments.requestArguments.specificationId=usrCustomerInfo.specificationId; //In - - @@ -239,9 +196,6 @@ arguments.requestArguments.specificationId=usrCustomerInfo.specificationId; //In - insert into instance_operation_cfs_param ( instance_operation_cfs_param_uid,instance_operation_uid,svc_operation_cfs_param_id,param_value,note,dt_created,creator_id @@ -255,10 +209,6 @@ arguments.requestArguments.specificationId=usrCustomerInfo.specificationId; //In , ); - - @@ -324,10 +274,6 @@ arguments.requestArguments.specificationId=usrCustomerInfo.specificationId; //In - - - - AND p.svc_operation_cfs_param_id=/*limit to the current svc param*/ - AND p.instance_operation_cfs_param_uid <> /*exclude itself*/ AND (select cast(st.instance_data->>'isDeleted' as boolean) from instance_state st where st.instance_uid=e.instance_uid order by version desc limit 1) IS NOT TRUE /*ignore deleted instances*/ - - + @@ -384,12 +328,10 @@ arguments.requestArguments.specificationId=usrCustomerInfo.specificationId; //In AND svc_operation_cfs_param_id=/*limit to the current svc param*/ AND u.usr_id=/*limit to the contragent of current user*/ - AND p.instance_operation_cfs_param_uid <> /*exclude itself*/ AND (select cast(st.instance_data->>'isDeleted' as boolean) from instance_state st where st.instance_uid=e.instance_uid order by version desc limit 1) IS NOT TRUE /*ignore deleted instances*/ - - + @@ -405,9 +347,6 @@ arguments.requestArguments.specificationId=usrCustomerInfo.specificationId; //In - - @@ -450,10 +389,8 @@ arguments.requestArguments.specificationId=usrCustomerInfo.specificationId; //In - -