Index: openacs-4/packages/xowiki/tcl/bootstrap-procs.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/packages/xowiki/tcl/bootstrap-procs.tcl,v diff -u -r1.11.2.1 -r1.11.2.2 --- openacs-4/packages/xowiki/tcl/bootstrap-procs.tcl 18 Apr 2019 07:56:43 -0000 1.11.2.1 +++ openacs-4/packages/xowiki/tcl/bootstrap-procs.tcl 4 Jul 2019 17:51:39 -0000 1.11.2.2 @@ -540,7 +540,7 @@ " \ -instproc render-data {line} { - set __name [:name] + set __name ${:name} if {[$line exists $__name.href] && [set href [$line set $__name.href]] ne ""} { # use the CSS class rather from the Field than not the line Index: openacs-4/packages/xowiki/tcl/chat-procs.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/packages/xowiki/tcl/chat-procs.tcl,v diff -u -r1.54.2.13 -r1.54.2.14 --- openacs-4/packages/xowiki/tcl/chat-procs.tcl 3 Jul 2019 20:33:04 -0000 1.54.2.13 +++ openacs-4/packages/xowiki/tcl/chat-procs.tcl 4 Jul 2019 17:51:39 -0000 1.54.2.14 @@ -61,7 +61,7 @@ if {![nsv_exists ${:array}-color idx]} { ::xo::clusterwide nsv_set ${:array}-color idx 0 } - if {[:user_id] != 0 || [:session_id] != 0} { + if {${:user_id} != 0 || [:session_id] != 0} { :init_user_color } :set_options Index: openacs-4/packages/xowiki/tcl/form-field-procs.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/packages/xowiki/tcl/form-field-procs.tcl,v diff -u -r1.284.2.18 -r1.284.2.19 --- openacs-4/packages/xowiki/tcl/form-field-procs.tcl 4 Jul 2019 16:20:58 -0000 1.284.2.18 +++ openacs-4/packages/xowiki/tcl/form-field-procs.tcl 4 Jul 2019 17:51:39 -0000 1.284.2.19 @@ -383,7 +383,7 @@ FormField instproc asWidgetSpec {} { set spec ${:widget_type} - if {[info exists :spell]} {append spec ",[expr {[:spell] ? {} : {no}}]spell"} + if {[info exists :spell]} {append spec ",[expr {${:spell} ? {} : {no}}]spell"} if {![:required]} {append spec ",optional"} if {[info exists :editor]} {append spec " {options {editor ${:editor}}} "} @@ -1175,7 +1175,7 @@ } hidden instproc render_item {} { # don't render the labels - if {[info exists :sign] && [:sign]} { + if {[info exists :sign] && ${:sign}} { set token_id [sec_get_random_cached_token_id] set secret [ns_config "ns/server/[ns_info server]/acs" parametersecret ""] if {[info exists :max_age]} { @@ -1195,7 +1195,7 @@ } hidden instproc check=signature {value} { set v 1 - if {[info exists :sign] && [:sign]} { + if {[info exists :sign] && ${:sign}} { set sig [::xo::cc form_parameter __${:name}:sig] set secret [ns_config "ns/server/[ns_info server]/acs" parametersecret ""] set v [ad_verify_signature -secret $secret $value $sig] Index: openacs-4/packages/xowiki/tcl/includelet-procs.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/packages/xowiki/tcl/includelet-procs.tcl,v diff -u -r1.239.2.13 -r1.239.2.14 --- openacs-4/packages/xowiki/tcl/includelet-procs.tcl 24 Jun 2019 11:41:40 -0000 1.239.2.13 +++ openacs-4/packages/xowiki/tcl/includelet-procs.tcl 4 Jul 2019 17:51:39 -0000 1.239.2.14 @@ -3789,7 +3789,7 @@ ::xo::Page requireJS "/resources/xowiki/highcharts/js/highcharts.js" ::xo::Page requireJS "/resources/xowiki/highcharts/js/themes/gray.js" set result "
\n" - set title [:title] + set title ${:title} if {![info exists :id]} {set :id [::xowiki::Includelet html_id [self]]} set id [:id] set values [list] Index: openacs-4/packages/xowiki/tcl/repeat-procs.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/packages/xowiki/tcl/repeat-procs.tcl,v diff -u -r1.14.2.5 -r1.14.2.6 --- openacs-4/packages/xowiki/tcl/repeat-procs.tcl 8 May 2019 14:33:40 -0000 1.14.2.5 +++ openacs-4/packages/xowiki/tcl/repeat-procs.tcl 4 Jul 2019 17:51:39 -0000 1.14.2.6 @@ -176,7 +176,7 @@ set values [:value] :trim_values set r [next] - #:msg name=[:name],value=[:get_compound_value] + #:msg name=${:name},value=[:get_compound_value] # # remove "unneeded" entries from instance attributes Index: openacs-4/packages/xowiki/tcl/syndicate-procs.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/packages/xowiki/tcl/syndicate-procs.tcl,v diff -u -r1.51.2.5 -r1.51.2.6 --- openacs-4/packages/xowiki/tcl/syndicate-procs.tcl 17 Jun 2019 10:20:51 -0000 1.51.2.5 +++ openacs-4/packages/xowiki/tcl/syndicate-procs.tcl 4 Jul 2019 17:51:39 -0000 1.51.2.6 @@ -272,7 +272,7 @@ set last_user "" set last_item "" set last_clock "" - if {[info exists :user_id]} { append where_clause " and o.creation_user = [:user_id] " } + if {[info exists :user_id]} { append where_clause " and o.creation_user = ${:user_id} " } if {[info exists :limit]} { set limit [:limit] } ::xo::OrderedComposite items -destroy_on_cleanup Index: openacs-4/packages/xowiki/tcl/weblog-procs.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/packages/xowiki/tcl/weblog-procs.tcl,v diff -u -r1.75.2.10 -r1.75.2.11 --- openacs-4/packages/xowiki/tcl/weblog-procs.tcl 28 Jun 2019 10:30:27 -0000 1.75.2.10 +++ openacs-4/packages/xowiki/tcl/weblog-procs.tcl 4 Jul 2019 17:51:39 -0000 1.75.2.11 @@ -252,14 +252,14 @@ set p [::xo::db::CrClass get_instance_from_db -item_id 0 -revision_id $revision_id] # in cases, the revision was created already earlier, drop the mixins if {[$p info mixin] ne ""} {$p mixin {}} - if {[info exists :entry_flag]} {$p set [:entry_flag] 1} + if {[info exists :entry_flag]} {$p set ${:entry_flag} 1} if {[:no_footer]} {$p set __no_footer 1} ad_try { $p set description [$p render -with_footer false] } on error {errorMsg} { $p set description "Render Error ($errorMsg) $revision_id $name $title" } - if {[info exists :entry_flag]} {$p unset [:entry_flag]} + if {[info exists :entry_flag]} {$p unset ${:entry_flag}} #:log "--W $p render (mixins=[$p info mixin]) => $description" } $p set pretty_date $pretty_date Index: openacs-4/packages/xowiki/tcl/xowiki-procs.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/packages/xowiki/tcl/xowiki-procs.tcl,v diff -u -r1.542.2.30 -r1.542.2.31 --- openacs-4/packages/xowiki/tcl/xowiki-procs.tcl 3 Jul 2019 19:57:46 -0000 1.542.2.30 +++ openacs-4/packages/xowiki/tcl/xowiki-procs.tcl 4 Jul 2019 17:51:39 -0000 1.542.2.31 @@ -752,19 +752,19 @@ # associative list (attribute/value pairs) for form-field attributes. # #foreach f $form_fields {lappend fns [list [$f name] [$f info class]]} - #:msg "page [:name] build_instance_attribute_map $fns" + #:msg "page ${:name} build_instance_attribute_map $fns" if {![info exists :__instance_attribute_map]} { set :__instance_attribute_map [dict create] } foreach f $form_fields { set multiple [expr {[$f exists multiple] ? [$f set multiple] : 0}] #:msg "$f [$f name] cat_tree [$f exists category_tree] is fc: [$f exists is_category_field]" if {[$f exists category_tree] && [$f exists is_category_field]} { - #:msg "page [:name] field [$f name] is a category_id from [$f category_tree]" + #:msg "page ${:name} field [$f name] is a category_id from [$f category_tree]" dict set :__instance_attribute_map [$f name] [list category [$f category_tree] $multiple] :category_export [$f category_tree] } elseif {[$f exists is_party_id]} { - #:msg "page [:name] field [$f name] is a party_id" + #:msg "page ${:name} field [$f name] is a party_id" dict set :__instance_attribute_map [$f name] [list party_id $multiple] } elseif {[$f istype "::xowiki::formfield::file"]} { dict set :__instance_attribute_map [$f name] [list file 0] @@ -877,7 +877,7 @@ # method. # if {[info exists :__instance_attribute_map]} { - # :log "+++ we have an instance_attribute_map for [:name]" + # :log "+++ we have an instance_attribute_map for ${:name}" # :log "+++ starting with instance_attributes [:instance_attributes]" array set multiple_index [list category 2 party_id 1 file 1] set ia [list] @@ -1018,22 +1018,22 @@ # default value if {![info exists :page_order]} {set :page_order ""} set is_folder_page [:is_folder_page] - #:msg "is-folder-page [:name] => $is_folder_page" + #:msg "is-folder-page ${:name} => $is_folder_page" if {$is_folder_page} { # reset names if necessary (e.g. import from old releases) set :name [:build_name] } else { # Check, if nls_language and lang are aligned. - if {[regexp {^(..):} [:name] _ lang]} { + if {[regexp {^(..):} ${:name} _ lang]} { if {[string range [:nls_language] 0 1] ne $lang} { set old_nls_language [:nls_language] :nls_language [:get_nls_language_from_lang $lang] - ns_log notice "nls_language for item [:name] set from $old_nls_language to [:nls_language]" + ns_log notice "nls_language for item ${:name} set from $old_nls_language to [:nls_language]" } } } # in the general case, no more actions required - #:msg "demarshall [:name] DONE" + #:msg "demarshall ${:name} DONE" } File instproc demarshall {args} { @@ -1090,7 +1090,7 @@ # mapped here again to internal representations :upvar $category_ids_name category_ids if {[info exists ::__xowiki_reverse_category_map($value)]} { - #:msg "map value '$value' (category tree: $use($name)) of [:name] to an ID" + #:msg "map value '$value' (category tree: $use($name)) of ${:name} to an ID" lappend category_ids $::__xowiki_reverse_category_map($value) return $::__xowiki_reverse_category_map($value) } elseif {$map_type eq "party_id"} { @@ -1102,7 +1102,7 @@ return "" } else { :msg "cannot map value '$value' (map_type $map_type)\ - of [:name] to an ID; maybe there is some\ + of ${:name} to an ID; maybe there is some\ same_named category tree with fewer entries..." :msg "reverse category map has values [lsort [array names ::__xowiki_reverse_category_map]]" return "" @@ -1117,7 +1117,7 @@ # to categorize this objects in the source instance. set category_ids [list] - #:msg "[:name] check cm=[info exists ::__xowiki_reverse_category_map] && iam=[info exists :__instance_attribute_map]" + #:msg "${:name} check cm=[info exists ::__xowiki_reverse_category_map] && iam=[info exists :__instance_attribute_map]" if {[info exists ::__xowiki_reverse_category_map] && [info exists :__instance_attribute_map] @@ -1152,7 +1152,7 @@ } } set :instance_attributes $ia - #:msg "[:name] saving instance_attributes $ia" + #:msg "${:name} saving instance_attributes $ia" } set r [next] set :__category_ids [lsort -unique $category_ids] @@ -1355,7 +1355,7 @@ return 0 } FormPage instproc is_folder_page {{-include_folder_links true}} { - set page_template_name [[:page_template] name] + set page_template_name [${:page_template} name] if {$page_template_name eq "en:folder.form"} {return 1} if {$include_folder_links && $page_template_name eq "en:link.form"} { set link_type [:get_property_from_link_page link_type] @@ -1969,7 +1969,7 @@ if {$page_name ne ""} { set page [::${:package_id} resolve_page_name_and_init_context -lang [:lang] $page_name] if {$page eq ""} { - error "Cannot find page '$page_name' to be included in page '[:name]'" + error "Cannot find page '$page_name' to be included in page '${:name}'" } } else { set page [self] @@ -2275,7 +2275,7 @@ # return the pretty_link for the current page ${:package_id} pretty_link -parent_id ${:parent_id} \ -anchor $anchor -query $query -absolute $absolute -siteurl $siteurl \ - -lang $lang -download $download -page [self] [:name] + -lang $lang -download $download -page [self] ${:name} } Page instproc detail_link {} { @@ -2391,7 +2391,7 @@ [dict get $link_info link]] } #ns_log notice "link_info $link_info" - #ns_log notice "--L link <$arg> lang [:lang] CURRENT [:name] nls_lang ${:nls_language} -> item_ref_info $item_ref_info" + #ns_log notice "--L link <$arg> lang [:lang] CURRENT ${:name} nls_lang ${:nls_language} -> item_ref_info $item_ref_info" #:log "link '[dict get $link_info link]' package_id $package_id ${:package_id} => [array get {}]" @@ -2445,7 +2445,7 @@ if {[info exists object_type]} { next } else { - set template_id [:page_template] + set template_id ${:page_template} if {![info exists parent_id]} { set parent_id [::$page_package_id folder_id] } @@ -2559,7 +2559,7 @@ # The provided content and the returned result are strings # containing HTML. # - #:msg "--adp_subst in [:name] vars=[:info vars]" + #:msg "--adp_subst in ${:name} vars=[:info vars]" foreach __v [:info vars] { if {[info exists $__v]} continue #ns_log notice "import instvar $__v into current scope" @@ -2842,7 +2842,7 @@ } # unset -nocomplain :__references # - #:log "Page [:name] render with_footer $with_footer - [ns_conn isconnected] - [catch {ns_conn content}]" + #:log "Page ${:name} render with_footer $with_footer - [ns_conn isconnected] - [catch {ns_conn content}]" # # handle footer # @@ -2926,7 +2926,7 @@ # Page instproc notification_detail_link {} { set link [:pretty_link -absolute 1] - append html "For more details, see [ns_quotehtml [:title]]
" + append html "For more details, see [ns_quotehtml ${:title}]
" append text "\nFor more details, see $link ...\n" return [list html $html text $text] } @@ -2938,9 +2938,9 @@ # Page instproc notification_subject {-instance_name {-category_label ""} -state} { if {$category_label eq ""} { - return "\[$instance_name\]: [:title] ($state)" + return "\[$instance_name\]: ${:title} ($state)" } else { - return "\[$instance_name\] $category_label: [:title] ($state)" + return "\[$instance_name\] $category_label: ${:title} ($state)" } } @@ -3210,7 +3210,7 @@ if {$source_item_id ne ""} { set source [FormPage get_instance_from_db -item_id $source_item_id] $f copy_content_vars -from_object $source - set name "[::xowiki::autoname new -parent_id $source_item_id -name [:name]]" + set name "[::xowiki::autoname new -parent_id $source_item_id -name ${:name}]" ::$package_id get_lang_and_name -name $name lang name $f set name $name #:msg nls=[$f nls_language],source-nls=[$source nls_language] @@ -3220,7 +3220,7 @@ } # Finally provide base for auto-titles - $f set __title_prefix [:title] + $f set __title_prefix ${:title} return $f } @@ -3540,7 +3540,7 @@ set name "" } - return [append name [::xowiki::utility formCSSclass [:name]]] + return [append name [::xowiki::utility formCSSclass ${:name}]] } # @@ -3585,7 +3585,7 @@ PageInstance instproc get_short_spec {name} { #set form_constraints [:get_from_template form_constraints] set form_constraints [:get_form_constraints] - #:msg "fc of [self] [:name] = $form_constraints" + #:msg "fc of [self] ${:name} = $form_constraints" if {$form_constraints ne ""} { set s [::xowiki::PageInstance get_short_spec_from_form_constraints \ -name $name -form_constraints $form_constraints] @@ -3643,7 +3643,7 @@ } PageInstance instproc get_template_object {} { - set id [:page_template] + set id ${:page_template} if {![:isobject ::$id]} { ::xo::db::CrClass get_instance_from_db -item_id $id } @@ -3689,7 +3689,7 @@ # The resulting page should be either a Form (PageTemplate) or # a FormPage (PageInstance) # - #:msg "parent of self [:name] is [$form_obj name] type [$form_obj info class]" + #:msg "parent of self ${:name} is [$form_obj name] type [$form_obj info class]" # # If it is as well a PageInstance, we find the information in the # properties of this page. Note that we cannot distinguish here between @@ -3731,7 +3731,7 @@ PageInstance instproc render_content {} { set html [:get_html_from_content [:get_from_template text]] set html [:adp_subst $html] - return "[ns_quotehtml $errorInfo]" @@ -1526,7 +1523,7 @@ if {$validation_errors == 0} { #:msg "validation ok" set ctx [::xowf::Context require [self]] - set cc [[:package_id] context] + set cc [${:package_id} context] foreach {name value} [$cc get_all_form_parameter] { if {[regexp {^__action_(.+)$} $name _ action]} { set next_state [:activate $ctx $action] @@ -1629,7 +1626,7 @@ WorkflowPage instproc save_in_hstore {} { # - if {[::xo::dc has_hstore] && [[:package_id] get_parameter use_hstore 0]} { + if {[::xo::dc has_hstore] && [${:package_id} get_parameter use_hstore 0]} { set hkey [::xowiki::hstore::dict_as_hkey [:hstore_attributes]] set revision_id ${:revision_id} xo::dc dml update_hstore "update xowiki_page_instance \ @@ -1908,7 +1905,6 @@ if {[info exists :__no_form_page_footer]} { next } else { - set package_id [:package_id] set parent_id [:parent_id] set form_item_id ${:page_template} #:msg "is wf page [:is_wf], is wf instance page [:is_wf_instance]" @@ -1925,17 +1921,17 @@ set button_objs [list] # create new workflow instance button with start form - #if {[:parent_id] != [::$package_id folder_id]} { + #if {[:parent_id] != [::${:package_id} folder_id]} { # set parent_id [:parent_id] #} - set link [::$package_id make_link -link $wf_base $wf create-new parent_id return_url] + set link [::${:package_id} make_link -link $wf_base $wf create-new parent_id return_url] lappend button_objs [::xowiki::includelet::form-menu-button-new new -volatile \ -parent_id $parent_id \ -form $wf -link $link] # list workflow instances button set obj [::xowiki::includelet::form-menu-button-wf-instances new -volatile \ - -package_id $package_id -parent_id $parent_id \ + -package_id ${:package_id} -parent_id $parent_id \ -base $wf_base -form $wf] if {[info exists return_url]} { $obj return_url $return_url @@ -1944,7 +1940,7 @@ # work flow definition button set obj [::xowiki::includelet::form-menu-button-form new -volatile \ - -package_id $package_id -parent_id $parent_id \ + -package_id ${:package_id} -parent_id $parent_id \ -base $work_flow_base -form $work_flow_form] if {[info exists return_url]} {$obj return_url $return_url} lappend button_objs $obj @@ -1973,7 +1969,7 @@ set form [::xo::db::CrClass get_instance_from_db -item_id $entry_form_item_id] set base [$form pretty_link] set obj [::xowiki::includelet::form-menu-button-form new -volatile \ - -package_id $package_id -parent_id $parent_id \ + -package_id ${:package_id} -parent_id $parent_id \ -base $base -form $form] if {[info exists return_url]} { $obj return_url $return_url @@ -1984,7 +1980,7 @@ # work flow definition button # set obj [::xowiki::includelet::form-menu-button-wf new -volatile \ - -package_id $package_id -parent_id $parent_id \ + -package_id ${:package_id} -parent_id $parent_id \ -base $work_flow_base -form $work_flow_form] if {[info exists return_url]} {$obj return_url $return_url} lappend button_objs $obj @@ -2001,7 +1997,7 @@ by the list of page names } { foreach page_name $page_names { - set page [[:package_id] get_page_from_name -parent_id [:parent_id] -name $page_name] + set page [${:package_id} get_page_from_name -parent_id [:parent_id] -name $page_name] if {$page ne ""} { $page call_action -action $action -attributes $attributes } else {