Index: openacs-4/packages/acs-tcl/tcl/utilities-procs.tcl
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/packages/acs-tcl/tcl/utilities-procs.tcl,v
diff -u -r1.189.2.40 -r1.189.2.41
--- openacs-4/packages/acs-tcl/tcl/utilities-procs.tcl	29 Nov 2019 14:28:48 -0000	1.189.2.40
+++ openacs-4/packages/acs-tcl/tcl/utilities-procs.tcl	29 Nov 2019 15:30:57 -0000	1.189.2.41
@@ -401,12 +401,21 @@
     return $new_set_id
 }
 
-ad_proc -public merge_form_with_query {
+ad_proc -deprecated merge_form_with_query {
     {-bind {}}
     form statement_name sql_qry
 } {
     Merges a form with a query string.
 
+    DEPRECATED: this proc does not comply with OpenACS naming
+    convention. Furthermore, ns_formvalueput supports a limited number
+    of HTML variants and input tag types and is subject to various
+    other limitations. For a modern implementation addressing the
+    use-case of this proc one should probably use tools such as tDOM.
+
+    @see tDOM
+    @see https://panoptic.com/wiki/aolserver/Ns_formvalueput
+
     @param form the form to be stuffed.
     @param statement_name An identifier for the sql_qry to be executed.
     @param sql_qry The sql that must be executed.