cleanup: trim bookmark and realm list comments

analysis/cleanup-worklog-2026-04-29.md

@@ -558,3 +558,52 @@
 ### Вывод
 
 Паттерн повторяется: в небольших list-resource можно безопасно удалять слой исторических и отладочных комментариев отдельными локальными коммитами без вмешательства в бизнес-логику.
+
+## Обновление: выбран следующий mini-cleanup проход
+
+Дата фиксации: 2026-04-29.
+
+### Выбранные файлы
+
+- `v1/resources/available_resource_realms.cfc`
+- `v1/resources/bookmark_ls.cfc`
+
+### Что планируется убрать
+
+- закомментированные debug-возвраты и `cfdump`-хвосты
+- старые временные комментарии рядом с `cftry` и helper-инициализацией
+- закомментированные неиспользуемые строки и хвосты в `cfscript`
+
+### Что в этот этап не входит
+
+- изменение SQL-логики
+- пересмотр бизнес-ограничений по `contragent_id`
+- правка спорных комментариев, которые еще могут нести смысл бизнес-правил
+
+### Результат этапа
+
+Выполнен cleanup-проход по двум следующим ресурсам:
+- `v1/resources/available_resource_realms.cfc`
+- `v1/resources/bookmark_ls.cfc`
+
+Что убрано:
+- комментарии-рассуждения рядом с helper-инициализацией
+- закомментированные debug-возвраты и `cfdump`-хвосты
+- закомментированные неиспользуемые строки и временные следы в `cfscript`
+- старые закомментированные альтернативы сортировки и выборки полей
+
+Что оставлено специально:
+- рабочая SQL-логика
+- комментарий про общую букмарку, так как он still поясняет текущее бизнес-правило
+- структура ответа и набор аргументов
+
+### Проверка
+
+Измененные файлы проверены на ошибки.
+Результат:
+- синтаксических ошибок в `v1/resources/available_resource_realms.cfc` не обнаружено
+- синтаксических ошибок в `v1/resources/bookmark_ls.cfc` не обнаружено
+
+### Вывод
+
+Даже в чуть более длинных list-resource по-прежнему удается безопасно снимать debug/comment слой малыми независимыми коммитами, если не заходить в спорные бизнес-комментарии и не трогать логику запросов.

v1/resources/available_resource_realms.cfc

@@ -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>
 
 	<!---спецификация полей, пригодных для фильтрации (?и сортировки)--->
@@ -24,15 +24,14 @@
 		<cfset local={}/>
 		
 		
-		<!---parse and validate request parameters--->
+		<!---разбор и проверка параметров запроса--->
 		<cftry>
 			<cfset this.helper.validateField(arguments, "svcId", "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">
@@ -57,7 +56,7 @@
 				left outer join contract c on (s.contract_id=c.contract_id)
 				left outer join usr u on (c.contragent_id=u.contragent_id)
 				left outer join contragent z on (u.contragent_id=z.contragent_id)
-				where u.usr_id=<cfqueryparam cfsqltype="cf_sql_integer" value=#ARGUMENTS.usrId#/><!--- *** undeclared --->
+				where u.usr_id=<cfqueryparam cfsqltype="cf_sql_integer" value=#ARGUMENTS.usrId#/>
 				order by specification_id desc;				
 			</cfquery>		
 		
@@ -72,7 +71,6 @@
 		
 
 		<cfinvoke component="instance_operation_cfs_param" method="generateValueList" argumentCollection=#args# returnVariable="local.value_list"/>
-<!--- 		<cfreturn representationOf("Проверка связи 7")/> --->
 
 		
 

v1/resources/bookmark_ls.cfc

@@ -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>
 
 	<!---спецификация полей, пригодных для фильтрации (?и сортировки)--->
@@ -28,17 +28,16 @@
 		<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={}/>
-		<cftry>	<!--- отладка на время проблемы 404 --->	
+		<cftry>
 						
 		
-			<!---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(this.helper.formatBadRequestError(cfcatch)).withStatus(400)/>
@@ -52,8 +51,6 @@
 			<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">
@@ -65,26 +62,20 @@
 					<m:field>b.contragent_id</m:field>
 					<m:field>b.sort</m:field>
 					<m:field formatter=#request.castToBool#>b.is_enabled</m:field>
-					<!--- <m:field>k.external_uid</m:field> --->
 					<m:field>to_char(b.dt_created, 'YYYY-MM-DD"T"HH24:MI:SS.FF3TZHTZM') as dt_created</m:field>
 					<m:field>to_char(b.dt_updated, 'YYYY-MM-DD"T"HH24:MI:SS.FF3TZHTZM') as dt_updated</m:field>
 				</m:field_set>	
 				from bookmark b 
-				/*left outer join contragent k on (b.contragent_id=k.contragent_id)*/
 				where 1=1 <m:filter_build filter=#filter#/>
 				AND (b.contragent_id = <cfqueryparam cfsqltype="cf_sql_integer" value=#arguments.contragentId# null=#!isNumeric(arguments.contragentId)#/>
 				OR b.contragent_id=-1)/*общая букмарка*/
 				order by b.contragent_id desc, b.sort asc
-				<!--- <m:order_build sortCollection=#this.helper.parseNumericOrder(local.titleMap, arguments.orderBy)# fieldCount=0/>--->
 				limit #maxrows#
 			</cfquery>
-			<!--- <cfdump var=#local.qRead#/><cfabort/> --->			
-			
 
 			<cfquery name="local.qTotal">
 				select count(*) as cnt
 				from bookmark b 
-				/*left outer join contragent k on (b.contragent_id=k.contragent_id)*/
 				where 1=1				
 				AND (b.contragent_id = <cfqueryparam cfsqltype="cf_sql_integer" value=#arguments.contragentId# null=#!isNumeric(arguments.contragentId)#/>)
 			</cfquery> 
@@ -106,7 +97,6 @@
 			<cfset "out.size"=#arrayLen(resultCollection)#/>
 			<cfset "out.results"=#resultCollection#/>
 			<cfset "out.runDurationMs"=getTickCount()-request.startTickCount/>	
-			<!---<cfset "out.sql"=#local.result.sql#/>--->	
 			<cfreturn representationOf(out)/>	
 
 			<cfcatch type="any">
@@ -171,17 +161,14 @@
 	function plain2htm(s) {
 		return replace(replace(s, chr(13),'',"ALL"),chr(10),'<br/>', "ALL"); 
 	}
-	//request.plain2htm = plain2htm;
 
 	function htm2plain(s) {
 		return replaceNoCase(s, '<br/>', '#chr(13)##chr(10)#', "ALL"); 
 	}
-	//request.htm2plain = htm2plain;
 
 	function cleanHtm(s) {
 		return replaceList(s, '<,>,"', '&lt;,&gt;,&quot;'); 
 	}
-	//request.cleanHtm = cleanHtm;
 	</cfscript>
 	
 	<cffunction name="plain2HtmClean">