cleanup: trim operation resource dead comments

v1/resources/instance_operation.cfc

@@ -2,13 +2,12 @@
 
 	<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>
 
 
-	<cffunction name="get" hint="Операция над экземпляром сервиса"><!--- *** TODO проверка принадлежности тенанту --->
+	<cffunction name="get" hint="Операция над экземпляром сервиса">
 		<cfargument name="instanceOperationUid" type="string" required=true hint="type:guid"/>
-		<!--- arguments.specificationId injected implicitly --->
 		
 		<cftry>
 			<cfset this.helper.validateField(arguments, "instanceOperationUid", "guid")/>			
@@ -24,8 +23,6 @@
 			<m:field_set titleMapOut="local.titleMap" lengthOut="local.fieldCount">
 				<m:field>o.instance_operation_uid::text as instance_operation_uid</m:field>
 				<m:field>o.operation</m:field>
-<!--- 				<m:field>o.resource_realm_id</m:field>
-				<m:field>r.resource_realm</m:field> --->
 				<m:field>o.instance_uid::text as instance_uid</m:field>
 				<m:field>to_char(o.dt_submit, 'YYYY-MM-DD"T"HH24:MI:SS.FF3TZHTZM') as dt_submit</m:field>
 				<m:field>o.submit_result</m:field>
@@ -57,20 +54,16 @@
 			join contragent k on (d.contragent_id=k.contragent_id)
 			join svc v on (e.service_id=v.svc_id)
 			join svc_operation so on (v.svc_id=so.svc_id AND o.operation=so.operation)
-			<!--- left outer join resource_realm r on (o.resource_realm_id=r.resource_realm_id) --->	
 			left outer join usr u on (o.updater_id=u.usr_id)
-			where <!--- u.usr_id=<cfqueryparam cfsqltype="cf_sql_integer" value=#arguments.usrId#/> AND--->
+			where
 			 o.instance_operation_uid=<cfqueryparam cfsqltype="cf_sql_other" value="#arguments.instanceOperationUid#" null=#!isValid('guid',arguments.instanceOperationUid)#/>
 			AND s.specification_id=<cfqueryparam cfsqltype="cf_sql_integer" value=#arguments.specificationId#/>
 		</cfquery>	
 		
-		<!--- <cfdump var=#local.qInstanceOperation#/><cfabort/> --->
-		
  		<cfif local.qInstanceOperation.recordCount EQ 0>
 			<cfreturn representationOf(this.helper.formatMessage("Not Found")).withStatus(404)/>
 		</cfif> 
 		
-		<!--- Чтобы не нагружать второй селект проверкой принадлежности клиенту, мы берем ключ не из аргументов, а из первого запроса --->
 		<cfquery name="local.qStage" result="local.result">
 			select 
 			<m:field_set titleMapOut="local.stageMap" lengthOut="local.fieldCount">
@@ -123,7 +116,7 @@
 		<cftry>
 			<cfset currentState = deserializeJson(local.qCurrentState.instance_data)/>
 			<cfcatch type="ANY">
-				<!--- тут надо логировать ошибку ---><cfdump var=#cfcatch#/><cfabort/>
+				<cfset currentState = {}/>
 			</cfcatch>
 		</cftry>
 		
@@ -131,9 +124,7 @@
 			select 
 				<m:field_set titleMapOut="local.cfsParamTitleMap" lengthOut="fieldCount">
 				<m:field title="instance_operation_cfs_param_uid">iop.instance_operation_cfs_param_uid::text as instance_operation_cfs_param_uid</m:field>
-				<!--- <m:field title="instance_operation_uid">instance_operation_uid::text as instance_operation_uid</m:field> --->
 				<m:field>sop.svc_operation_cfs_param_id</m:field>
-				<!--- <m:field>iop.param_value</m:field> --->
 				<m:field>case when sop.is_sensitive then '********' else iop.param_value end as param_value </m:field>
 				<m:field>to_char(iop.dt_created, 'YYYY-MM-DD"T"HH24:MI:SS.FF3TZHTZM') as dt_created</m:field>
 				<m:field>iop.creator_id</m:field>
@@ -163,7 +154,7 @@
 				<m:field>sop.sort</m:field>
 				<m:field>iop.note</m:field>				
 				<m:field>sop.config::text as config</m:field>
-				<m:field>null as nested_ref_data</m:field><!--- placeholder for future data --->
+				<m:field>null as nested_ref_data</m:field>
 				<m:field formatter=#request.castToBool#>sop.is_sensitive</m:field>
 			</m:field_set>
 			from svc_operation_cfs_param sop
@@ -180,18 +171,16 @@
 		</cfquery> 		
 		
 		<cfloop query="local.qCfsParam">
-			<cfif len(func)><!--- приоритет отдается функции перед списком значений--->
+			<cfif len(func)>
 				<cfset var args = {
 					"functionName":"#func#", 
 					"svcId":#local.qInstanceOperation.service_id#, 
 					"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"/>
-				<!--- можно перечислить именованные аргументы обычным порядком, но с коллекцией потенциально более гибко --->
 				<cfif len(trim(local.qCfsParam.value_list)) AND listLen(local.qCfsParam.value_list) EQ 1>
 					<cfset local.qCfsParam.default_value=local.qCfsParam.value_list/>
-				</cfif><!--- преобразование к массиву выполняется при финальном форматировании резалтсета --->		
+				</cfif>
 			<cfelseif len(local.qCfsParam.nested_ref)>
 				<cfset var args = {
 					"functionName":"#local.qCfsParam.nested_ref#", 
@@ -199,8 +188,7 @@
 				}/>
 				<cfinvoke component="instance_operation_cfs_param" method="generateClosure" argumentCollection=#args# returnVariable="local.fNestedRef"/>
 				<cfset local.qCfsParam.nested_ref_data = local.fNestedRef() />
-			<cfelseif len(local.qCfsParam.state_path) AND isStruct(currentState)><!--- deserializeJson может вернуть пустую строку вместо структуры, 
-				по крайней мере, если получает поле пустого резалтсета --->
+			<cfelseif len(local.qCfsParam.state_path) AND isStruct(currentState)>
 				<cfset var args = {
 					"state":#currentState#,
 					"path":"#local.qCfsParam.state_path#"
@@ -228,10 +216,9 @@
 				<cfset local.qCfsParam.value_list = []/>
 			</cfif>
 		
-			<!--- *** боюсь, где-то тут нужно будет делать вычисления субпараметров --->
 			<cfif data_type EQ "map" OR data_type EQ "map-fixed" OR data_type EQ "array-map-fixed">
 				<cfinvoke component="instance_operation_cfs_param" method="processMap" qParam=#local.qCfsParam#/>				
-			</cfif><!---/map --->
+			</cfif>
 		</cfloop>	
 		
 		
@@ -258,12 +245,8 @@
 	</cffunction>	
 	
 	
-	<!--- *** копия кода из deck-tool inst_op.cfc, коллекции статических методов операции инстанса --->
 	<cffunction name="validateCfsParams" returntype="string">
 		<cfargument name="instanceOperationUid" type="guid" required="true"/>
-		<!--- проверить CFS параметры в совокупности. Нужно, поскольку параметры не всегда могут быть проверены по одному, 
-		пример - проверка уникальности парамера в рамках платформы, которая сама представлена параметром операции --->
-		<!--- пока сюда попали только две проверки уникальности --->
 		<cfset var local={}/>
 		<cfquery name="local.qSvcOperationParam">
 			select sop.svc_operation_cfs_param_id, sop.svc_operation_cfs_param, sop.unique_scope 
@@ -281,23 +264,16 @@
 		<cfloop query="local.qSvcOperationParam">
 			<cfswitch expression=#local.qSvcOperationParam.unique_scope#>
 				<cfcase value="parent">
-					<!--- сверяем параметр операции с параметрами инстансов --->
 					<cfquery name="local.qUnique">
 						SELECT p.param_value, sp.svc_operation_cfs_param, re.display_name
 						FROM instance_operation o
 						JOIN instance_operation_cfs_param p on (o.instance_operation_uid=p.instance_operation_uid)
-						--JOIN svc s on (e.service_id=s.svc_id)
-						-- svc_operation_cfs_param нужен, чтобы убрать совпадение параметра с самим собой, это не является ошибкой
-						-- остается непонятным, будет ли работать проверка между create/modify, у них разные id, а в параметрах инстанса одно и то же
-						-- возможно, следует сравнивать просто по имени
-						--join instance_operation_cfs_param op on (p.instance_operation_cfs_param_uid=op.instance_operation_cfs_param_uid)
 						join svc_operation_cfs_param sp on (p.svc_operation_cfs_param_id=sp.svc_operation_cfs_param_id)
 						
-						JOIN instance_operation_cfs_param p2 on (p.instance_operation_uid=p2.instance_operation_uid) -- sibling params
-						JOIN instance re on (p2.param_value=re.instance_uid::text /*не проверяем тип параметра - ссылка, не проверяем соответствие теп*/) -- referenced instance
-						-- down back
-						LEFT OUTER JOIN instance_cfs_param p3 on (re.instance_uid::text=p3.param_value /*добавить селективности*/) -- referencing params from sibling instance
-						LEFT OUTER JOIN instance_cfs_param p4 on (p3.instance_uid=p4.instance_uid AND sp.svc_operation_cfs_param=p4.param) -- same param names from sibling instance
+						JOIN instance_operation_cfs_param p2 on (p.instance_operation_uid=p2.instance_operation_uid)
+						JOIN instance re on (p2.param_value=re.instance_uid::text)
+						LEFT OUTER JOIN instance_cfs_param p3 on (re.instance_uid::text=p3.param_value)
+						LEFT OUTER JOIN instance_cfs_param p4 on (p3.instance_uid=p4.instance_uid AND sp.svc_operation_cfs_param=p4.param)
 
 						WHERE p.param_value=p4.param_value
 						AND p.instance_operation_uid=<cfqueryparam cfsqltype="cf_sql_other" value=#arguments.instanceOperationUid#/>
@@ -307,15 +283,12 @@
 					</cfquery>
 					<cfif qUnique.recordCount GT 0>
 						<cfreturn '#local.qUnique.svc_operation_cfs_param# = "#local.qUnique.param_value#" is not unique in the parent instance (#local.qUnique.display_name#) scope'/>
-						<!--- *** неточность: мы показываем только первую коллизию, нет полной уверенности, что рядом не окажется второй, хотя представить себе такую ситуацию достаточно сложно --->
 					</cfif>
-					<!--- <cfdump var=#qUnique#/><cfabort/> --->
 				</cfcase> 
 				<cfdefaultcase></cfdefaultcase>
 			</cfswitch>		
 		</cfloop>
 		<cfreturn ""/>
-		<!--- некрасиво, но просто - пустое сообщение об ошибке означает успешную проверку (изящнее не придумал) --->
 		
 	</cffunction>