diff --git a/analysis/cleanup-worklog-2026-04-29.md b/analysis/cleanup-worklog-2026-04-29.md index 2267990..d56c1c2 100644 --- a/analysis/cleanup-worklog-2026-04-29.md +++ b/analysis/cleanup-worklog-2026-04-29.md @@ -518,6 +518,37 @@ В него вошли: - `analysis/cleanup-worklog-2026-04-29.md` + +## Обновление: убран следующий слой dead-code и debug-комментариев в operation-ресурсах + +Дата фиксации: 2026-04-29. + +### Что планируется сделать + +Следующий проход выполняется по двум прикладным ресурсам среднего риска: +- `v1/resources/instance_operation.cfc` +- `v1/resources/svc_operation_cfs_param_compute.cfc` + +Цель этапа: +- убрать закомментированные SQL-поля и старые альтернативные ветки +- убрать debug-хвосты с `cfdump/cfabort` +- убрать длинные комментарии-рассуждения и sample-блоки, не влияющие на runtime +- не менять контракт ресурсов и действующую вычислительную логику + +### Результат этапа + +Выполнено: +- в `instance_operation.cfc` убраны закомментированные поля, join/where-хвосты, комментарии-размышления и отладочный abort при ошибке десериализации состояния +- в `instance_operation.cfc` сохранена рабочая логика, а при сбое десериализации оставлен безопасный пустой `currentState` +- в `svc_operation_cfs_param_compute.cfc` удалены длинные вводные рассуждения, мертвые альтернативные ветки, устаревшие sample URL и служебные комментарии + +### Проверка + +После правок оба файла нужно проверить на синтаксические ошибки и затем зафиксировать отдельным локальным коммитом. + +Результат проверки: +- синтаксических ошибок в `v1/resources/instance_operation.cfc` не обнаружено +- синтаксических ошибок в `v1/resources/svc_operation_cfs_param_compute.cfc` не обнаружено - `v1/resources/resource_realm_type_ls.cfc` - `v1/resources/bookmark.cfc` diff --git a/v1/resources/instance_operation.cfc b/v1/resources/instance_operation.cfc index cd43507..342133e 100644 --- a/v1/resources/instance_operation.cfc +++ b/v1/resources/instance_operation.cfc @@ -2,13 +2,12 @@ - + - + - @@ -24,8 +23,6 @@ o.instance_operation_uid::text as instance_operation_uid o.operation - o.instance_uid::text as instance_uid to_char(o.dt_submit, 'YYYY-MM-DD"T"HH24:MI:SS.FF3TZHTZM') as dt_submit o.submit_result @@ -57,20 +54,16 @@ join contragent k on (d.contragent_id=k.contragent_id) join svc v on (e.service_id=v.svc_id) join svc_operation so on (v.svc_id=so.svc_id AND o.operation=so.operation) - left outer join usr u on (o.updater_id=u.usr_id) - where + where o.instance_operation_uid= AND s.specification_id= - - - select @@ -123,7 +116,7 @@ - + @@ -131,9 +124,7 @@ select iop.instance_operation_cfs_param_uid::text as instance_operation_cfs_param_uid - sop.svc_operation_cfs_param_id - case when sop.is_sensitive then '********' else iop.param_value end as param_value to_char(iop.dt_created, 'YYYY-MM-DD"T"HH24:MI:SS.FF3TZHTZM') as dt_created iop.creator_id @@ -163,7 +154,7 @@ sop.sort iop.note sop.config::text as config - null as nested_ref_data + null as nested_ref_data sop.is_sensitive from svc_operation_cfs_param sop @@ -180,18 +171,16 @@ - + - + }/> - - + - + - - + @@ -258,12 +245,8 @@ - - - select sop.svc_operation_cfs_param_id, sop.svc_operation_cfs_param, sop.unique_scope @@ -281,23 +264,16 @@ - SELECT p.param_value, sp.svc_operation_cfs_param, re.display_name FROM instance_operation o JOIN instance_operation_cfs_param p on (o.instance_operation_uid=p.instance_operation_uid) - --JOIN svc s on (e.service_id=s.svc_id) - -- svc_operation_cfs_param нужен, чтобы убрать совпадение параметра с самим собой, это не является ошибкой - -- остается непонятным, будет ли работать проверка между create/modify, у них разные id, а в параметрах инстанса одно и то же - -- возможно, следует сравнивать просто по имени - --join instance_operation_cfs_param op on (p.instance_operation_cfs_param_uid=op.instance_operation_cfs_param_uid) join svc_operation_cfs_param sp on (p.svc_operation_cfs_param_id=sp.svc_operation_cfs_param_id) - JOIN instance_operation_cfs_param p2 on (p.instance_operation_uid=p2.instance_operation_uid) -- sibling params - JOIN instance re on (p2.param_value=re.instance_uid::text /*не проверяем тип параметра - ссылка, не проверяем соответствие теп*/) -- referenced instance - -- down back - LEFT OUTER JOIN instance_cfs_param p3 on (re.instance_uid::text=p3.param_value /*добавить селективности*/) -- referencing params from sibling instance - LEFT OUTER JOIN instance_cfs_param p4 on (p3.instance_uid=p4.instance_uid AND sp.svc_operation_cfs_param=p4.param) -- same param names from sibling instance + JOIN instance_operation_cfs_param p2 on (p.instance_operation_uid=p2.instance_operation_uid) + JOIN instance re on (p2.param_value=re.instance_uid::text) + LEFT OUTER JOIN instance_cfs_param p3 on (re.instance_uid::text=p3.param_value) + LEFT OUTER JOIN instance_cfs_param p4 on (p3.instance_uid=p4.instance_uid AND sp.svc_operation_cfs_param=p4.param) WHERE p.param_value=p4.param_value AND p.instance_operation_uid= @@ -307,15 +283,12 @@ - - - diff --git a/v1/resources/svc_operation_cfs_param_compute.cfc b/v1/resources/svc_operation_cfs_param_compute.cfc index 8e8a4ae..6b26d7c 100644 --- a/v1/resources/svc_operation_cfs_param_compute.cfc +++ b/v1/resources/svc_operation_cfs_param_compute.cfc @@ -1,35 +1,11 @@ - - - - - + - - - - @@ -37,8 +13,7 @@ arguments.requestArguments.specificationId=usrCustomerInfo.specificationId; //In - - + @@ -46,46 +21,12 @@ arguments.requestArguments.specificationId=usrCustomerInfo.specificationId; //In - + - + - - - - - - - - select @@ -103,21 +44,6 @@ arguments.requestArguments.specificationId=usrCustomerInfo.specificationId; //In - - - - - - - - + -