cleanup: trim compact resource comments
This commit is contained in:
@@ -505,3 +505,56 @@
|
|||||||
- посмотреть рабочее дерево после этого мини-этапа
|
- посмотреть рабочее дерево после этого мини-этапа
|
||||||
- при необходимости взять еще 1-2 похожих компактных resource-файла
|
- при необходимости взять еще 1-2 похожих компактных resource-файла
|
||||||
- затем собрать следующий локальный cleanup-коммит без смешения с рискованными файлами
|
- затем собрать следующий локальный cleanup-коммит без смешения с рискованными файлами
|
||||||
|
|
||||||
|
## Обновление: зафиксирован локальный коммит по компактным resource-файлам
|
||||||
|
|
||||||
|
Дата фиксации: 2026-04-29.
|
||||||
|
|
||||||
|
### Коммит
|
||||||
|
|
||||||
|
Создан локальный коммит:
|
||||||
|
- `81b8fee`
|
||||||
|
- сообщение: `cleanup: trim small resource debug comments`
|
||||||
|
|
||||||
|
В него вошли:
|
||||||
|
- `analysis/cleanup-worklog-2026-04-29.md`
|
||||||
|
- `v1/resources/resource_realm_type_ls.cfc`
|
||||||
|
- `v1/resources/bookmark.cfc`
|
||||||
|
|
||||||
|
### Следующий мини-этап
|
||||||
|
|
||||||
|
Для следующего такого же безопасного прохода выбраны:
|
||||||
|
- `v1/resources/param_value_list.cfc`
|
||||||
|
- `v1/resources/resource_realm_ls.cfc`
|
||||||
|
|
||||||
|
План по этапу:
|
||||||
|
- убрать закомментированные debug-строки и мертвые фрагменты
|
||||||
|
- убрать комментарии-рассуждения рядом с helper-инициализацией
|
||||||
|
- не менять запросы, условия доступа и runtime-поведение
|
||||||
|
|
||||||
|
### Результат этапа
|
||||||
|
|
||||||
|
Выполнен еще один точечный cleanup-проход по двум ресурсам:
|
||||||
|
- `v1/resources/param_value_list.cfc`
|
||||||
|
- `v1/resources/resource_realm_ls.cfc`
|
||||||
|
|
||||||
|
Что убрано:
|
||||||
|
- комментарии-рассуждения рядом с созданием helper-компонента
|
||||||
|
- закомментированные debug-возвраты и старые временные строки
|
||||||
|
- неиспользуемые закомментированные фрагменты, не влияющие на runtime
|
||||||
|
|
||||||
|
Что не менялось:
|
||||||
|
- текст запросов
|
||||||
|
- проверка входных параметров по существующей логике
|
||||||
|
- условия доступа и структура ответа
|
||||||
|
|
||||||
|
### Проверка
|
||||||
|
|
||||||
|
Измененные файлы проверены на ошибки.
|
||||||
|
Результат:
|
||||||
|
- синтаксических ошибок в `v1/resources/param_value_list.cfc` не обнаружено
|
||||||
|
- синтаксических ошибок в `v1/resources/resource_realm_ls.cfc` не обнаружено
|
||||||
|
|
||||||
|
### Вывод
|
||||||
|
|
||||||
|
Паттерн повторяется: в небольших list-resource можно безопасно удалять слой исторических и отладочных комментариев отдельными локальными коммитами без вмешательства в бизнес-логику.
|
||||||
|
|||||||
@@ -2,7 +2,7 @@
|
|||||||
|
|
||||||
<cfsilent>
|
<cfsilent>
|
||||||
<cfimport prefix="m" taglib="../lib"/>
|
<cfimport prefix="m" taglib="../lib"/>
|
||||||
<cfset this.helper=CreateObject("component","lib.rest_api_helper")/><!---*** странно, почему мы его видим?---><!---вынести в апп?--->
|
<cfset this.helper=CreateObject("component","lib.rest_api_helper")/>
|
||||||
</cfsilent>
|
</cfsilent>
|
||||||
|
|
||||||
|
|
||||||
@@ -22,7 +22,4 @@
|
|||||||
<cfreturn noData().withStatus(204, "OK") />
|
<cfreturn noData().withStatus(204, "OK") />
|
||||||
</cffunction>
|
</cffunction>
|
||||||
|
|
||||||
<!---
|
|
||||||
/instanceOperationCfsParams/{instanceOperationCfsParamUid}" --->
|
|
||||||
|
|
||||||
</cfcomponent>
|
</cfcomponent>
|
||||||
|
|||||||
@@ -2,7 +2,7 @@
|
|||||||
|
|
||||||
<cfsilent>
|
<cfsilent>
|
||||||
<cfimport prefix="m" taglib="../lib"/>
|
<cfimport prefix="m" taglib="../lib"/>
|
||||||
<cfset this.helper=CreateObject("component","lib.rest_api_helper")/><!---*** странно, почему мы его видим?---><!---вынести в апп?--->
|
<cfset this.helper=CreateObject("component","lib.rest_api_helper")/>
|
||||||
</cfsilent>
|
</cfsilent>
|
||||||
|
|
||||||
<!---спецификация полей, пригодных для фильтрации (?и сортировки)--->
|
<!---спецификация полей, пригодных для фильтрации (?и сортировки)--->
|
||||||
@@ -24,16 +24,15 @@
|
|||||||
<cfset local={}/>
|
<cfset local={}/>
|
||||||
|
|
||||||
|
|
||||||
<!---parse and validate request parameters--->
|
<!---разбор и проверка параметров запроса--->
|
||||||
<cftry>
|
<cftry>
|
||||||
<cfset this.helper.validateField(arguments, "pageSize", "integer")/>
|
<cfset this.helper.validateField(arguments, "pageSize", "integer")/>
|
||||||
<cfset this.helper.validateField(arguments, "page", "integer")/>
|
<cfset this.helper.validateField(arguments, "page", "integer")/>
|
||||||
|
|
||||||
<!---мы мирно игнорируем поля, отсутствующие в спецификации, что позволяет не делать исключения для orderBy и т.п.--->
|
<!---мы мирно игнорируем поля, отсутствующие в спецификации, что позволяет не делать исключения для orderBy и т.п.--->
|
||||||
<cfset var filter=this.helper.parseFilterParams(this.fieldsSpec)/>
|
<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)/>
|
<cfreturn representationOf(this.helper.formatBadRequestError(cfcatch)).withStatus(400)/>
|
||||||
</cfcatch>
|
</cfcatch>
|
||||||
<cfcatch type="any">
|
<cfcatch type="any">
|
||||||
@@ -46,9 +45,6 @@
|
|||||||
<cfset var maxrows=arguments.pageSize*arguments.page/>
|
<cfset var maxrows=arguments.pageSize*arguments.page/>
|
||||||
<cfset var startrow=arguments.pageSize*(arguments.page-1)+1/>
|
<cfset var startrow=arguments.pageSize*(arguments.page-1)+1/>
|
||||||
|
|
||||||
<!--- <cfset request.locateIamService()/> --->
|
|
||||||
|
|
||||||
|
|
||||||
<cfquery name="local.qRead" result="local.result">
|
<cfquery name="local.qRead" result="local.result">
|
||||||
select
|
select
|
||||||
<m:field_set titleMapOut="local.titleMap" lengthOut="local.fieldCount">
|
<m:field_set titleMapOut="local.titleMap" lengthOut="local.fieldCount">
|
||||||
@@ -87,8 +83,6 @@
|
|||||||
<cfreturn representationOf(cfcatch)/>
|
<cfreturn representationOf(cfcatch)/>
|
||||||
</cfcatch>
|
</cfcatch>
|
||||||
</cftry>
|
</cftry>
|
||||||
|
|
||||||
<!--- <cfreturn representationOf("Проверка связи 7")/> --->
|
|
||||||
|
|
||||||
<cfset "out.queryDurationMs"=getTickCount() - request.startTickCount/>
|
<cfset "out.queryDurationMs"=getTickCount() - request.startTickCount/>
|
||||||
<cfset "out.total"=#local.qTotal.cnt#/>
|
<cfset "out.total"=#local.qTotal.cnt#/>
|
||||||
|
|||||||
Reference in New Issue
Block a user