cleanup: trim compute and operation list comments
This commit is contained in:
@@ -644,3 +644,41 @@
|
|||||||
Результат:
|
Результат:
|
||||||
- синтаксических ошибок в `v1/resources/catalog_service_ls.cfc` не обнаружено
|
- синтаксических ошибок в `v1/resources/catalog_service_ls.cfc` не обнаружено
|
||||||
- синтаксических ошибок в `v1/resources/resource_realm.cfc` не обнаружено
|
- синтаксических ошибок в `v1/resources/resource_realm.cfc` не обнаружено
|
||||||
|
|
||||||
|
## Обновление: выбран cleanup-проход по compute/list слою
|
||||||
|
|
||||||
|
Дата фиксации: 2026-04-29.
|
||||||
|
|
||||||
|
### Выбранные файлы
|
||||||
|
|
||||||
|
- `v1/resources/svc_operation_cfs_subparam_compute.cfc`
|
||||||
|
- `v1/resources/instance_operation_ls.cfc`
|
||||||
|
|
||||||
|
### Ограничения этапа
|
||||||
|
|
||||||
|
- убирать только закомментированные debug-хвосты и комментарии-рассуждения
|
||||||
|
- не менять активную обработку ошибок и не исправлять живые runtime-аномалии
|
||||||
|
- не заходить глубоко в `post` и крупные блоки бизнес-логики
|
||||||
|
|
||||||
|
### Результат этапа
|
||||||
|
|
||||||
|
Выполнен cleanup-проход по compute/list слою:
|
||||||
|
- `v1/resources/svc_operation_cfs_subparam_compute.cfc`
|
||||||
|
- `v1/resources/instance_operation_ls.cfc`
|
||||||
|
|
||||||
|
Что убрано:
|
||||||
|
- комментарии-рассуждения рядом с helper-инициализацией
|
||||||
|
- закомментированные `cfdump`/`cfabort` хвосты
|
||||||
|
- старые закомментированные debug-строки валидации и SQL-вывода
|
||||||
|
|
||||||
|
Что сознательно не трогалось:
|
||||||
|
- активные `cfcatch` и текущее поведение ошибок
|
||||||
|
- крупные блоки `post`
|
||||||
|
- спорные бизнес-комментарии вне явного debug-слоя
|
||||||
|
|
||||||
|
### Проверка
|
||||||
|
|
||||||
|
Измененные файлы проверены на ошибки.
|
||||||
|
Результат:
|
||||||
|
- синтаксических ошибок в `v1/resources/svc_operation_cfs_subparam_compute.cfc` не обнаружено
|
||||||
|
- синтаксических ошибок в `v1/resources/instance_operation_ls.cfc` не обнаружено
|
||||||
|
|||||||
@@ -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>
|
||||||
|
|
||||||
<!---спецификация полей, пригодных для фильтрации (и сортировки)--->
|
<!---спецификация полей, пригодных для фильтрации (и сортировки)--->
|
||||||
@@ -41,19 +41,18 @@
|
|||||||
<cfargument name="orderBy" type="string" hint="type:string, description:comma-separated list of fields to sort by, example:fld1.ASC,fld2.DESC,fld3.ASC" default=""/>
|
<cfargument name="orderBy" type="string" hint="type:string, description:comma-separated list of fields to sort by, example:fld1.ASC,fld2.DESC,fld3.ASC" default=""/>
|
||||||
|
|
||||||
<cfset var local={}/>
|
<cfset var local={}/>
|
||||||
<cftry> <!--- отладка на время проблемы 404 --->
|
<cftry>
|
||||||
|
|
||||||
|
|
||||||
<!---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">
|
||||||
@@ -113,7 +112,6 @@
|
|||||||
offset #arguments.pageSize*(arguments.page-1)#
|
offset #arguments.pageSize*(arguments.page-1)#
|
||||||
</cfif>
|
</cfif>
|
||||||
</cfquery>
|
</cfquery>
|
||||||
<!--- <cfdump var=#local.qRead#/><cfabort/> --->
|
|
||||||
|
|
||||||
<cfquery name="local.qCnt">
|
<cfquery name="local.qCnt">
|
||||||
select count(*) as cnt
|
select count(*) as cnt
|
||||||
@@ -153,7 +151,6 @@
|
|||||||
<cfset "out.size"=#arrayLen(resultCollection)#/>
|
<cfset "out.size"=#arrayLen(resultCollection)#/>
|
||||||
<cfset "out.results"=#resultCollection#/>
|
<cfset "out.results"=#resultCollection#/>
|
||||||
<cfset "out.runDurationMs"=getTickCount()-request.startTickCount/>
|
<cfset "out.runDurationMs"=getTickCount()-request.startTickCount/>
|
||||||
<!---<cfset "out.sql"=#local.result.sql#/>--->
|
|
||||||
<cfreturn representationOf(out)/>
|
<cfreturn representationOf(out)/>
|
||||||
|
|
||||||
<cfcatch type="any">
|
<cfcatch type="any">
|
||||||
|
|||||||
@@ -21,7 +21,7 @@ arguments.requestArguments.specificationId=usrCustomerInfo.specificationId; //In
|
|||||||
|
|
||||||
<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>
|
||||||
|
|
||||||
|
|
||||||
@@ -86,9 +86,6 @@ arguments.requestArguments.specificationId=usrCustomerInfo.specificationId; //In
|
|||||||
<cfelse>
|
<cfelse>
|
||||||
<cfreturn representationOf(this.helper.formatMessage("Expression Not Found")).withStatus(404)/>
|
<cfreturn representationOf(this.helper.formatMessage("Expression Not Found")).withStatus(404)/>
|
||||||
</cfif>
|
</cfif>
|
||||||
<!--- <cfdump var=#local.keys#/>
|
|
||||||
<cfdump var=#local.params#/>
|
|
||||||
<cfdump var=#local.calculatedValueList# abort=true/> --->
|
|
||||||
<!--- Тут нам еще придется сформатировать лист в массив. Но обошлось довольно просто --->
|
<!--- Тут нам еще придется сформатировать лист в массив. Но обошлось довольно просто --->
|
||||||
<cfset var out=structNew("linked")/>
|
<cfset var out=structNew("linked")/>
|
||||||
<cfset "out.queryDurationMs"=getTickCount() - request.startTickCount/>
|
<cfset "out.queryDurationMs"=getTickCount() - request.startTickCount/>
|
||||||
|
|||||||
Reference in New Issue
Block a user