cleanup: trim default resource comments
This commit is contained in:
@@ -682,3 +682,43 @@
|
|||||||
Результат:
|
Результат:
|
||||||
- синтаксических ошибок в `v1/resources/svc_operation_cfs_subparam_compute.cfc` не обнаружено
|
- синтаксических ошибок в `v1/resources/svc_operation_cfs_subparam_compute.cfc` не обнаружено
|
||||||
- синтаксических ошибок в `v1/resources/instance_operation_ls.cfc` не обнаружено
|
- синтаксических ошибок в `v1/resources/instance_operation_ls.cfc` не обнаружено
|
||||||
|
|
||||||
|
## Обновление: выбран cleanup-проход по default-ресурсам
|
||||||
|
|
||||||
|
Дата фиксации: 2026-04-29.
|
||||||
|
|
||||||
|
### Выбранные файлы
|
||||||
|
|
||||||
|
- `v1/resources/instance_default.cfc`
|
||||||
|
- `v1/resources/svc_default.cfc`
|
||||||
|
- `v1/resources/instance_operation_default.cfc`
|
||||||
|
|
||||||
|
### Ограничения этапа
|
||||||
|
|
||||||
|
- убирать только закомментированные аргументы, TODO-заметки, явные debug-хвосты и комментарии-рассуждения
|
||||||
|
- не править живые участки, где cleanup уже превращается в bugfix
|
||||||
|
- не менять формирование `value_list`, `default_value` и остальную логику шаблонов
|
||||||
|
|
||||||
|
### Результат этапа
|
||||||
|
|
||||||
|
Выполнен cleanup-проход по default-ресурсам:
|
||||||
|
- `v1/resources/instance_default.cfc`
|
||||||
|
- `v1/resources/svc_default.cfc`
|
||||||
|
- `v1/resources/instance_operation_default.cfc`
|
||||||
|
|
||||||
|
Что убрано:
|
||||||
|
- старые TODO/рассуждения в заголовках методов
|
||||||
|
- закомментированные неиспользуемые аргументы и debug-хвосты
|
||||||
|
- часть локальных закомментированных альтернатив, не влияющих на текущий runtime
|
||||||
|
|
||||||
|
Что сознательно не трогалось:
|
||||||
|
- живой `cfdump/cfabort` в обработке ошибки `instance_operation_default.cfc`
|
||||||
|
- логика генерации `value_list`, `nested_ref_data` и `default_value`
|
||||||
|
|
||||||
|
### Проверка
|
||||||
|
|
||||||
|
Измененные файлы проверены на ошибки.
|
||||||
|
Результат:
|
||||||
|
- синтаксических ошибок в `v1/resources/instance_default.cfc` не обнаружено
|
||||||
|
- синтаксических ошибок в `v1/resources/svc_default.cfc` не обнаружено
|
||||||
|
- синтаксических ошибок в `v1/resources/instance_operation_default.cfc` не обнаружено
|
||||||
|
|||||||
@@ -5,9 +5,8 @@
|
|||||||
<cfset this.helper=CreateObject("component","lib.rest_api_helper")/>
|
<cfset this.helper=CreateObject("component","lib.rest_api_helper")/>
|
||||||
</cfsilent>
|
</cfsilent>
|
||||||
|
|
||||||
<cffunction name="get" hint="Шаблон экземпляра"><!--- *** TODO проверка принадлежности тенанту --->
|
<cffunction name="get" hint="Шаблон экземпляра">
|
||||||
<cfargument name="svcId" type="string" required=true hint="type:integer"/>
|
<cfargument name="svcId" type="string" required=true hint="type:integer"/>
|
||||||
<!--- <cfargument name="usrId" type="string" hint="type:integer; no need to provide this argument in request, it is injected by IAM"/> ---><!--- arguments.usrId неявно вбрасывается из Application.cfm --->
|
|
||||||
|
|
||||||
<cftry>
|
<cftry>
|
||||||
<cfset this.helper.validateField(arguments, "svcId", "integer")/>
|
<cfset this.helper.validateField(arguments, "svcId", "integer")/>
|
||||||
@@ -36,15 +35,12 @@
|
|||||||
<cfset "out.instance.displayName"="#generateDefaultName(arguments.svcId,arguments.usrId)#"/>
|
<cfset "out.instance.displayName"="#generateDefaultName(arguments.svcId,arguments.usrId)#"/>
|
||||||
<cfset "out.instance.descr"="#local.qSvc.descr#"/>
|
<cfset "out.instance.descr"="#local.qSvc.descr#"/>
|
||||||
<cfset "out.instance.man"="#local.qSvc.man#"/>
|
<cfset "out.instance.man"="#local.qSvc.man#"/>
|
||||||
|
|
||||||
<!--- <cfset var out = structCopy(svc)/> ---><!--- *** иначе, видимо, циклическая ссылка, если мы еще скопируем это внутрь структуры ---><!--- <cfset "out.instance" = instance/> --->
|
|
||||||
|
|
||||||
<cfset "out.queryDurationMs"=getTickCount() - request.startTickCount/>
|
<cfset "out.queryDurationMs"=getTickCount() - request.startTickCount/>
|
||||||
<cfset "out.runDurationMs"=getTickCount() - request.startTickCount/>
|
<cfset "out.runDurationMs"=getTickCount() - request.startTickCount/>
|
||||||
<cfreturn representationOf(out) />
|
<cfreturn representationOf(out) />
|
||||||
</cffunction>
|
</cffunction>
|
||||||
|
|
||||||
<!--- хотя такая зависимость очень не нравится своим направлением, как бы задом наперед--->
|
|
||||||
<cffunction name="generateDefaultName">
|
<cffunction name="generateDefaultName">
|
||||||
<cfargument name="service_id" type="numeric"/>
|
<cfargument name="service_id" type="numeric"/>
|
||||||
<cfargument name="usr_id" type="numeric"/>
|
<cfargument name="usr_id" type="numeric"/>
|
||||||
|
|||||||
@@ -1,6 +1,4 @@
|
|||||||
<cfcomponent extends="taffy.core.resource" taffy:uri="/instanceOperations/{instanceUid}/{svcOperationId}" hint="template for default instance operation. Legacy URL: /instanceOperations/default/{svcOperationId}">
|
<cfcomponent extends="taffy.core.resource" taffy:uri="/instanceOperations/{instanceUid}/{svcOperationId}" hint="template for default instance operation. Legacy URL: /instanceOperations/default/{svcOperationId}">
|
||||||
<!--- *** вот похоже, что для deafult неправильно считается valueList (конкретно resourceRealm) --->
|
|
||||||
<!--- ну и как мне передать второй параметр для GET? --->
|
|
||||||
|
|
||||||
<cfsilent>
|
<cfsilent>
|
||||||
<cfimport prefix="m" taglib="../lib"/>
|
<cfimport prefix="m" taglib="../lib"/>
|
||||||
@@ -8,10 +6,9 @@
|
|||||||
</cfsilent>
|
</cfsilent>
|
||||||
|
|
||||||
|
|
||||||
<cffunction name="get" hint="Шаблон операции экземпляра"><!--- *** TODO проверка принадлежности тенанту --->
|
<cffunction name="get" hint="Шаблон операции экземпляра">
|
||||||
<cfargument name="instanceUid" type="string" required=true hint="type:guid|'default'"/>
|
<cfargument name="instanceUid" type="string" required=true hint="type:guid|'default'"/>
|
||||||
<cfargument name="svcOperationId" type="string" required=true hint="type:integer"/>
|
<cfargument name="svcOperationId" type="string" required=true hint="type:integer"/>
|
||||||
<!--- <cfargument name="usrId" type="string" hint="type:integer; no need to provide this argument in request, it is injected by IAM"/> ---><!--- arguments.usrId неявно инжектируется из Application.cfm, а с ним contractId, contragentId ... --->
|
|
||||||
|
|
||||||
<cftry>
|
<cftry>
|
||||||
<cfif arguments.instanceUid NEQ 'default'>
|
<cfif arguments.instanceUid NEQ 'default'>
|
||||||
@@ -118,7 +115,6 @@
|
|||||||
</cfquery>
|
</cfquery>
|
||||||
|
|
||||||
|
|
||||||
<!--- тут разорвана обработка, ниже есть повторный проход по параметрам --->
|
|
||||||
<cfloop query="local.qCfsParam">
|
<cfloop query="local.qCfsParam">
|
||||||
<cfif len(local.qCfsParam.func)><!--- приоритет отдается функции перед списком значений--->
|
<cfif len(local.qCfsParam.func)><!--- приоритет отдается функции перед списком значений--->
|
||||||
<cfset var args = {
|
<cfset var args = {
|
||||||
@@ -126,15 +122,12 @@
|
|||||||
"svcId":#local.qSvcOperation.svc_id#,
|
"svcId":#local.qSvcOperation.svc_id#,
|
||||||
"contractId":#arguments.contractId#
|
"contractId":#arguments.contractId#
|
||||||
}/> <!--- если появятся новые аргументы у новых функций, будем добавлять их сюда --->
|
}/> <!--- если появятся новые аргументы у новых функций, будем добавлять их сюда --->
|
||||||
<!--- <cfset local.qCfsParam.value_list=generateValueList(argumentCollection=#args#)/> --->
|
|
||||||
<cfinvoke component="instance_operation_cfs_param" method="generateValueList" argumentCollection=#args# returnVariable="local.qCfsParam.value_list"/>
|
<cfinvoke component="instance_operation_cfs_param" method="generateValueList" argumentCollection=#args# returnVariable="local.qCfsParam.value_list"/>
|
||||||
|
|
||||||
<cfif len(trim(local.qCfsParam.value_list)) AND listLen(local.qCfsParam.value_list) EQ 1>
|
<cfif len(trim(local.qCfsParam.value_list)) AND listLen(local.qCfsParam.value_list) EQ 1>
|
||||||
<cfset local.qCfsParam.default_value=local.qCfsParam.value_list/><!--- *** не очень красиво перетирать query, но делаем это довольно часто --->
|
<cfset local.qCfsParam.default_value=local.qCfsParam.value_list/><!--- *** не очень красиво перетирать query, но делаем это довольно часто --->
|
||||||
</cfif>
|
</cfif>
|
||||||
<!--- <cfdump var=#local.qCfsParam.value_list# abort=true/> --->
|
|
||||||
<cfelseif len(local.qCfsParam.nested_ref)><!--- *** спрашивается, зачем нам изгаляться и возвращать замыкание, когда нам нужен json --->
|
<cfelseif len(local.qCfsParam.nested_ref)><!--- *** спрашивается, зачем нам изгаляться и возвращать замыкание, когда нам нужен json --->
|
||||||
<!--- <cfset var fNestedRef = generateClosure(local.qCfsParam.nested_ref, local.qCfsParam.config)/> --->
|
|
||||||
<cfset var args = {
|
<cfset var args = {
|
||||||
"functionName":"#local.qCfsParam.nested_ref#",
|
"functionName":"#local.qCfsParam.nested_ref#",
|
||||||
"config":"#local.qCfsParam.config#"
|
"config":"#local.qCfsParam.config#"
|
||||||
|
|||||||
@@ -5,7 +5,7 @@
|
|||||||
<cfset this.helper=CreateObject("component","lib.rest_api_helper")/>
|
<cfset this.helper=CreateObject("component","lib.rest_api_helper")/>
|
||||||
</cfsilent>
|
</cfsilent>
|
||||||
|
|
||||||
<cffunction name="get" hint="DEPRECATED. Шаблон экземпляра"><!--- *** TODO проверка принадлежности тенанту --->
|
<cffunction name="get" hint="DEPRECATED. Шаблон экземпляра">
|
||||||
<cfargument name="svcId" type="string" required=true hint="type:integer"/>
|
<cfargument name="svcId" type="string" required=true hint="type:integer"/>
|
||||||
|
|
||||||
<cftry>
|
<cftry>
|
||||||
|
|||||||
Reference in New Issue
Block a user