diff --git a/analysis/cleanup-worklog-2026-04-29.md b/analysis/cleanup-worklog-2026-04-29.md index c90a3df..3032a97 100644 --- a/analysis/cleanup-worklog-2026-04-29.md +++ b/analysis/cleanup-worklog-2026-04-29.md @@ -442,3 +442,66 @@ - очистить `catalog_service_ls.cfc` и `catalog_service_param_ls.cfc` - затем проверить синтаксис измененных файлов - после этого собрать третий локальный commit-stage + +## Обновление: выбран следующий безопасный мини-этап + +Дата фиксации: 2026-04-29. + +### Что планируется сделать + +На следующем проходе берется очень маленькая и контролируемая группа resource-файлов: +- `v1/resources/resource_realm_type_ls.cfc` +- `v1/resources/bookmark.cfc` + +Цель прохода: +- убрать только очевидный закомментированный мертвый код и debug-хвосты +- убрать комментарии в формате внутренних рассуждений, если они не нужны текущей логике +- не менять SQL-логику, контракт ресурса и структуру аргументов + +### Почему выбраны именно они + +- оба файла компактнее крупных `instance*` и `operation*` ресурсов +- в них уже видны локальные и низкорисковые cleanup-кандидаты +- правки можно сделать без захода в архитектурно спорные области + +### Что сознательно не входит в этот этап + +- `v1/Application.cfc` +- крупные `instance*` и `svc_operation*` ресурсы +- vendor-слой `v1/taffy` +- автоматическое удаление файлов + +### Результат этапа + +Выполнен точечный cleanup в двух компактных resource-файлах: +- `v1/resources/resource_realm_type_ls.cfc` +- `v1/resources/bookmark.cfc` + +Что именно убрано: +- закомментированные debug-возвраты и `cfdump`-хвосты +- закомментированные неиспользуемые строки вроде старого `request.locateIamService()` +- комментарии в формате внутренних рассуждений рядом с helper-инициализацией +- неиспользуемые закомментированные SQL/field-фрагменты +- закомментированный legacy-блок с проверкой `qSave.cnt` + +Что сознательно оставлено: +- действующая SQL-логика ресурсов +- текущая структура аргументов и `hint` +- комментарии, которые еще можно отдельно разобрать на этапе перевода или дальнейшей локальной чистки + +### Проверка + +Измененные файлы проверены на ошибки. +Результат: +- синтаксических ошибок в `v1/resources/resource_realm_type_ls.cfc` не обнаружено +- синтаксических ошибок в `v1/resources/bookmark.cfc` не обнаружено + +### Вывод + +Подтверждается, что после helper-слоя можно безопасно продолжать малыми проходами по компактным `resources/*.cfc`, если ограничиваться только явным dead-code и debug-comment слоем. + +### Следующий шаг + +- посмотреть рабочее дерево после этого мини-этапа +- при необходимости взять еще 1-2 похожих компактных resource-файла +- затем собрать следующий локальный cleanup-коммит без смешения с рискованными файлами diff --git a/v1/resources/bookmark.cfc b/v1/resources/bookmark.cfc index bee6c0d..beb2f04 100644 --- a/v1/resources/bookmark.cfc +++ b/v1/resources/bookmark.cfc @@ -19,8 +19,6 @@ - - select @@ -39,8 +37,6 @@ where b.bookmark_uid= AND (b.contragent_id = OR b.contragent_id=-1) - - @@ -63,10 +59,7 @@ - - - @@ -93,14 +86,9 @@ ,is_enabled= where - contragent_id= + contragent_id= AND bookmark_uid=; - --select @@rowcount as cnt; - - diff --git a/v1/resources/resource_realm_type_ls.cfc b/v1/resources/resource_realm_type_ls.cfc index 0f16bab..c704bdd 100644 --- a/v1/resources/resource_realm_type_ls.cfc +++ b/v1/resources/resource_realm_type_ls.cfc @@ -2,7 +2,7 @@ - + @@ -20,16 +20,15 @@ - + - - + @@ -42,15 +41,11 @@ - - - select t.resource_realm_type_id t.resource_realm_type - t.man from resource_realm_type t @@ -69,8 +64,6 @@ - -