cleanup: trim small resource debug comments
This commit is contained in:
“Naeel”
@@ -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-коммит без смешения с рискованными файлами
|
||||
|
||||
@@ -19,8 +19,6 @@
|
||||
|
||||
<cfset var out={}/>
|
||||
|
||||
<!--- <cfset request.locateIamService()/> --->
|
||||
|
||||
<cfquery name="local.qRead" result="local.result">
|
||||
select
|
||||
<m:field_set titleMapOut="local.titleMap" lengthOut="local.fieldCount">
|
||||
@@ -39,8 +37,6 @@
|
||||
where b.bookmark_uid=<cfqueryparam cfsqltype="cf_sql_other" value="#arguments.bookmarkUid#" null=#!isValid('guid',arguments.bookmarkUid)#/>
|
||||
AND (b.contragent_id = <cfqueryparam cfsqltype="cf_sql_integer" value=#arguments.contragentId# null=#!isNumeric(arguments.contragentId)#/> OR b.contragent_id=-1)
|
||||
</cfquery>
|
||||
<!--- <cfdump var=#local.qRead#/><cfabort/> --->
|
||||
|
||||
|
||||
<cfset "out.queryDurationMs"=getTickCount() - request.startTickCount/>
|
||||
<cfset "out.bookmark" = this.helper.appendRecord(structNew("linked"), "", local.titleMap, local.qRead, this.helper.snake2camel)/>
|
||||
@@ -63,10 +59,7 @@
|
||||
<cfargument name="iconUrl" type="string" required=false hint="type:string description: url иконки"/><!--- чистить против XSS --->
|
||||
<cfargument name="sort" type="string" required=false hint="type:integer description: целое число для сортировки"/>
|
||||
<cfargument name="isEnabled" type="string" required=false hint="type:boolean description: вкл"/>
|
||||
<!--- arguments.contragentId injected implicitly --->
|
||||
<!--- arguments.usrId injected implicitly --->
|
||||
|
||||
<!--- ****** проверка уникальности --->
|
||||
<cftry>
|
||||
|
||||
<cfset this.helper.keyExistsAndValid(arguments, "bookmarkUid", "guid")/>
|
||||
@@ -93,14 +86,9 @@
|
||||
,is_enabled=<cfqueryparam cfsqltype="cf_sql_bit" value="#arguments.isEnabled#"/>
|
||||
</cfif>
|
||||
where
|
||||
contragent_id=<cfqueryparam cfsqltype="cf_sql_integer" value="#arguments.contragentId#"/><!--- isolation --->
|
||||
contragent_id=<cfqueryparam cfsqltype="cf_sql_integer" value="#arguments.contragentId#"/>
|
||||
AND bookmark_uid=<cfqueryparam cfsqltype="cf_sql_other" value="#arguments.bookmarkUid#"/>;
|
||||
--select @@rowcount as cnt;
|
||||
</cfquery>
|
||||
|
||||
<!--- <cfif local.qSave.cnt EQ 0>
|
||||
<cfreturn noData().withStatus(404, "Record Not Found") />
|
||||
</cfif> --->
|
||||
|
||||
<cfcatch type="invalidParamValue">
|
||||
<cfreturn representationOf(this.helper.formatBadRequestError(cfcatch)).withStatus(400)/>
|
||||
|
||||
@@ -2,7 +2,7 @@
|
||||
|
||||
<cfsilent>
|
||||
<cfimport prefix="m" taglib="../lib"/>
|
||||
<cfset this.helper=CreateObject("component","lib.rest_api_helper")/><!---*** странно, почему мы его видим?---><!---вынести в апп?--->
|
||||
<cfset this.helper=CreateObject("component","lib.rest_api_helper")/>
|
||||
</cfsilent>
|
||||
|
||||
<!---спецификация полей, пригодных для фильтрации (?и сортировки)--->
|
||||
@@ -20,16 +20,15 @@
|
||||
<cfset local={}/>
|
||||
|
||||
|
||||
<!---parse and validate request parameters--->
|
||||
<!---разбор и проверка параметров запроса--->
|
||||
<cftry>
|
||||
<cfset this.helper.validateField(arguments, "pageSize", "integer")/>
|
||||
<cfset this.helper.validateField(arguments, "page", "integer")/>
|
||||
|
||||
<!---мы мирно игнорируем поля, отсутствующие в спецификации, что позволяет не делать исключения для orderBy и т.п.--->
|
||||
<cfset var filter=this.helper.parseFilterParams(this.fieldsSpec)/>
|
||||
<!--- <cfset var order=this.helper.parseOrderBy(this.fieldsSpec, arguments.orderBy)/> --->
|
||||
|
||||
<cfcatch type="invalidParamValue"><!--- <cfreturn representationOf("Проверка связи 3")> --->
|
||||
<cfcatch type="invalidParamValue">
|
||||
<cfreturn representationOf(this.helper.formatBadRequestError(cfcatch)).withStatus(400)/>
|
||||
</cfcatch>
|
||||
<cfcatch type="any">
|
||||
@@ -42,15 +41,11 @@
|
||||
<cfset var maxrows=arguments.pageSize*arguments.page/>
|
||||
<cfset var startrow=arguments.pageSize*(arguments.page-1)+1/>
|
||||
|
||||
<!--- <cfset request.locateIamService()/> --->
|
||||
|
||||
|
||||
<cfquery name="local.qRead" result="local.result">
|
||||
select
|
||||
<m:field_set titleMapOut="local.titleMap" lengthOut="local.fieldCount">
|
||||
<m:field title="ID" cfSqlType="CF_SQL_INTEGER">t.resource_realm_type_id</m:field>
|
||||
<m:field>t.resource_realm_type</m:field>
|
||||
<!--- <m:field>t.descr</m:field> --->
|
||||
<m:field>t.man</m:field>
|
||||
</m:field_set>
|
||||
from resource_realm_type t
|
||||
@@ -69,8 +64,6 @@
|
||||
<cfreturn representationOf(cfcatch)/>
|
||||
</cfcatch>
|
||||
</cftry>
|
||||
|
||||
<!--- <cfreturn representationOf("Проверка связи 7")/> --->
|
||||
|
||||
<cfset "out.queryDurationMs"=getTickCount() - request.startTickCount/>
|
||||
<cfset "out.total"=#local.qTotal.cnt#/>
|
||||
|
||||
Reference in New Issue
Block a user