Index: openacs-4/contrib/obsolete-packages/acs-workflow/www/admin/task-add-2.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/contrib/obsolete-packages/acs-workflow/www/admin/task-add-2.tcl,v diff -u -r1.2 -r1.3 --- openacs-4/contrib/obsolete-packages/acs-workflow/www/admin/task-add-2.tcl 19 Nov 2001 18:29:29 -0000 1.2 +++ openacs-4/contrib/obsolete-packages/acs-workflow/www/admin/task-add-2.tcl 18 Sep 2002 13:06:04 -0000 1.3 @@ -38,7 +38,7 @@ set return_url "define?[export_vars -url {workflow_key}]" } ad_returnredirect $return_url - return + ad_script_abort } set create_new_role_p 0 Index: openacs-4/contrib/obsolete-packages/acs-workflow/www/admin/task-edit-2.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/contrib/obsolete-packages/acs-workflow/www/admin/task-edit-2.tcl,v diff -u -r1.2 -r1.3 --- openacs-4/contrib/obsolete-packages/acs-workflow/www/admin/task-edit-2.tcl 19 Nov 2001 18:29:29 -0000 1.2 +++ openacs-4/contrib/obsolete-packages/acs-workflow/www/admin/task-edit-2.tcl 18 Sep 2002 13:06:04 -0000 1.3 @@ -17,7 +17,7 @@ if { [info exists cancel] && ![empty_string_p $cancel] } { # User hit cancel ad_returnredirect $return_url - return + ad_script_abort } db_transaction { Index: openacs-4/contrib/obsolete-packages/acs-workflow/www/admin/wizard/create.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/contrib/obsolete-packages/acs-workflow/www/admin/wizard/create.tcl,v diff -u -r1.4 -r1.5 --- openacs-4/contrib/obsolete-packages/acs-workflow/www/admin/wizard/create.tcl 6 Sep 2002 21:50:18 -0000 1.4 +++ openacs-4/contrib/obsolete-packages/acs-workflow/www/admin/wizard/create.tcl 18 Sep 2002 13:06:05 -0000 1.5 @@ -17,12 +17,12 @@ if { [empty_string_p $workflow_name] } { ad_returnredirect "" - return + ad_script_abort } if { $num_tasks == 0 } { ad_returnredirect "tasks" - return + ad_script_abort } # setup loop_attribute_name, input_place Index: openacs-4/contrib/obsolete-packages/acs-workflow/www/admin/wizard/loops.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/contrib/obsolete-packages/acs-workflow/www/admin/wizard/loops.tcl,v diff -u -r1.3 -r1.4 --- openacs-4/contrib/obsolete-packages/acs-workflow/www/admin/wizard/loops.tcl 6 Sep 2002 21:50:18 -0000 1.3 +++ openacs-4/contrib/obsolete-packages/acs-workflow/www/admin/wizard/loops.tcl 18 Sep 2002 13:06:05 -0000 1.4 @@ -18,7 +18,7 @@ if { [empty_string_p $workflow_name] } { ad_returnredirect "" - return + ad_script_abort } wf_wizard_massage_tasks [ad_get_client_property wf tasks] task_list task tasks Index: openacs-4/contrib/obsolete-packages/acs-workflow/www/admin/wizard/tasks.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/contrib/obsolete-packages/acs-workflow/www/admin/wizard/tasks.tcl,v diff -u -r1.3 -r1.4 --- openacs-4/contrib/obsolete-packages/acs-workflow/www/admin/wizard/tasks.tcl 6 Sep 2002 21:50:18 -0000 1.3 +++ openacs-4/contrib/obsolete-packages/acs-workflow/www/admin/wizard/tasks.tcl 18 Sep 2002 13:06:05 -0000 1.4 @@ -14,7 +14,7 @@ if { [empty_string_p $workflow_name] } { ad_returnredirect "" - return + ad_script_abort } wf_wizard_massage_tasks [ad_get_client_property wf tasks] "" "" tasks Index: openacs-4/contrib/obsolete-packages/photo-album-lite/www/photo-add-2.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/contrib/obsolete-packages/photo-album-lite/www/photo-add-2.tcl,v diff -u -r1.2 -r1.3 --- openacs-4/contrib/obsolete-packages/photo-album-lite/www/photo-add-2.tcl 10 Sep 2002 22:23:14 -0000 1.2 +++ openacs-4/contrib/obsolete-packages/photo-album-lite/www/photo-add-2.tcl 18 Sep 2002 13:03:18 -0000 1.3 @@ -29,6 +29,7 @@ if !$caller_id { ad_return_complaint 1 "Sorry, you don't appear to be logged in. Please <a href=\"/register/index\">log in</a> and try again." + ad_script_abort } ad_require_permission $folder_id write @@ -38,6 +39,7 @@ set guessed_file_type [string tolower [ns_guesstype [set file$num]]] if { [lsearch [list image/gif image/jpeg application/x-zip-compressed] $guessed_file_type] < 0 } { ad_return_complaint 1 "Sorry, only GIF, JPEG, and ZIP formats are allowed at this time. Please make sure your images are in the correct format and that your filenames end in <code>.gif</code> or <code>.jpg</code>." + ad_script_abort } } } @@ -50,11 +52,13 @@ if [catch {pl_bulk_insert -use_default_date $use_default_date -use_default_prefix $use_default_prefix $folder_id [set file${num}.tmpfile]} errMsg] { global errorInfo doc_return 200 text/plain "sorry, there was an error: $errMsg" + ad_script_abort } } else { if [catch {pl_photo_insert -use_default_date $use_default_date -use_default_prefix $use_default_prefix $folder_id [set file${num}.tmpfile] [set file$num]} errMsg] { global errorInfo doc_return 200 text/plain "sorry, there was an error: $errMsg" + ad_script_abort } } Index: openacs-4/contrib/obsolete-packages/ticket-tracker/www/index.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/contrib/obsolete-packages/ticket-tracker/www/index.tcl,v diff -u -r1.3 -r1.4 --- openacs-4/contrib/obsolete-packages/ticket-tracker/www/index.tcl 6 Sep 2002 06:57:03 -0000 1.3 +++ openacs-4/contrib/obsolete-packages/ticket-tracker/www/index.tcl 18 Sep 2002 13:04:24 -0000 1.4 @@ -47,7 +47,7 @@ ad_returnredirect "admin/install/" set user_task_table "" set unassigned_task_table "" - return + ad_script_abort } # table definition for user's outstanding tasks Index: openacs-4/contrib/obsolete-packages/ticket-tracker/www/ticket-edit-2.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/contrib/obsolete-packages/ticket-tracker/www/ticket-edit-2.tcl,v diff -u -r1.1 -r1.2 --- openacs-4/contrib/obsolete-packages/ticket-tracker/www/ticket-edit-2.tcl 20 Apr 2001 20:51:25 -0000 1.1 +++ openacs-4/contrib/obsolete-packages/ticket-tracker/www/ticket-edit-2.tcl 18 Sep 2002 13:04:24 -0000 1.2 @@ -46,6 +46,7 @@ ns_log Error $errmsg ad_return_complaint 0 "There was an error updating the ticket information.<p> Here is the full error message:<pre>$errmsg</pre>" + ad_script_abort } if {[exists_and_not_null task_id]} { Index: openacs-4/contrib/obsolete-packages/ticket-tracker/www/admin/category-delete-2.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/contrib/obsolete-packages/ticket-tracker/www/admin/category-delete-2.tcl,v diff -u -r1.1 -r1.2 --- openacs-4/contrib/obsolete-packages/ticket-tracker/www/admin/category-delete-2.tcl 20 Apr 2001 20:51:25 -0000 1.1 +++ openacs-4/contrib/obsolete-packages/ticket-tracker/www/admin/category-delete-2.tcl 18 Sep 2002 13:04:25 -0000 1.2 @@ -46,12 +46,14 @@ where package_id = :package_id }] } { db_abort_transaction ad_return_complaint 1 "<li>You cannot delete this category. There needs to be at least one category for each ticket tracker instance." + ad_script_abort } } on_error { ns_log Error "$errmsg" } } else { ad_return_complaint 1 "You cannot delete this category. There needs to be at least one category for each ticket tracker instance." + ad_script_abort } } Index: openacs-4/contrib/obsolete-packages/wap/www/admin/add-ua-2.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/contrib/obsolete-packages/wap/www/admin/add-ua-2.tcl,v diff -u -r1.2 -r1.3 --- openacs-4/contrib/obsolete-packages/wap/www/admin/add-ua-2.tcl 10 Sep 2002 22:23:29 -0000 1.2 +++ openacs-4/contrib/obsolete-packages/wap/www/admin/add-ua-2.tcl 18 Sep 2002 13:05:15 -0000 1.3 @@ -12,4 +12,4 @@ wap_add_user_agent $name $creation_comment $active_p -ad_returnredirect index.tcl \ No newline at end of file +ad_returnredirect {} \ No newline at end of file Index: openacs-4/contrib/obsolete-packages/wap/www/admin/modify-ua-2.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/contrib/obsolete-packages/wap/www/admin/modify-ua-2.tcl,v diff -u -r1.1 -r1.2 --- openacs-4/contrib/obsolete-packages/wap/www/admin/modify-ua-2.tcl 20 Apr 2001 20:51:23 -0000 1.1 +++ openacs-4/contrib/obsolete-packages/wap/www/admin/modify-ua-2.tcl 18 Sep 2002 13:05:15 -0000 1.2 @@ -14,4 +14,4 @@ wap_modify_user_agent $user_agent_id $name $creation_comment $active_p -ad_returnredirect index.tcl \ No newline at end of file +ad_returnredirect {} \ No newline at end of file Index: openacs-4/contrib/obsolete-packages/wap/www/admin/modify-ua.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/contrib/obsolete-packages/wap/www/admin/modify-ua.tcl,v diff -u -r1.1 -r1.2 --- openacs-4/contrib/obsolete-packages/wap/www/admin/modify-ua.tcl 20 Apr 2001 20:51:23 -0000 1.1 +++ openacs-4/contrib/obsolete-packages/wap/www/admin/modify-ua.tcl 18 Sep 2002 13:05:15 -0000 1.2 @@ -33,8 +33,8 @@ } delete { wap_delete_user_agent $user_agent_id - ad_returnredirect index.tcl - return + ad_returnredirect {} + ad_script_abort } } Index: openacs-4/contrib/obsolete-packages/wap/www/admin/toggle-wap-enabled.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/contrib/obsolete-packages/wap/www/admin/toggle-wap-enabled.tcl,v diff -u -r1.1 -r1.2 --- openacs-4/contrib/obsolete-packages/wap/www/admin/toggle-wap-enabled.tcl 20 Apr 2001 20:51:23 -0000 1.1 +++ openacs-4/contrib/obsolete-packages/wap/www/admin/toggle-wap-enabled.tcl 18 Sep 2002 13:05:15 -0000 1.2 @@ -29,6 +29,7 @@ apm_generate_package_spec $version_id } on_error { ad_return_error "Database Error" "Could not create WAPEnabledP parameter for package $package_key, version $version_id." + ad_script_abort } set current_stat 0 @@ -42,4 +43,4 @@ ad_parameter -set $new_stat -package_id $package_id WAPEnabledP -ns_returnredirect "/wap/admin?[export_url_vars supertype]" \ No newline at end of file +ad_returnredirect "/wap/admin?[export_url_vars supertype]" \ No newline at end of file Index: openacs-4/packages/attachments/www/file-add-2.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/packages/attachments/www/file-add-2.tcl,v diff -u -r1.1 -r1.2 --- openacs-4/packages/attachments/www/file-add-2.tcl 2 Jul 2002 04:18:54 -0000 1.1 +++ openacs-4/packages/attachments/www/file-add-2.tcl 18 Sep 2002 13:07:14 -0000 1.2 @@ -153,7 +153,7 @@ ad_return_complaint 1 "You probably clicked on the Add button more than once. Check if the file is properly loaded on the <a href=\"index?folder_id?$folder_id\">folder</a> you wan, or you can use the Back button to return and re-enter the version file." - return + ad_script_abort } Index: openacs-4/packages/attachments/www/admin/new-root-folder-map.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/packages/attachments/www/admin/new-root-folder-map.tcl,v diff -u -r1.3 -r1.4 --- openacs-4/packages/attachments/www/admin/new-root-folder-map.tcl 10 Sep 2002 22:22:22 -0000 1.3 +++ openacs-4/packages/attachments/www/admin/new-root-folder-map.tcl 18 Sep 2002 13:07:15 -0000 1.4 @@ -28,16 +28,14 @@ # a new one if one doesn't exist. A BUG. set root_folder_id [db_string has_fs_root_folder_p_select \ "select folder_id from fs_root_folders where package_id = :package_id" \ - -default 0 -] + -default 0 ] if {$root_folder_id == 0} { # look for a fs root folder candidate, by looking for an file-storage # sibling of our parent (uncle? or aunt? node). Should generalize sibling # stuff, search by parent etc. set parent_id [site_node::get_parent \ - -node_id [site_node::get_node_id_from_object_id -object_id $package_id] - ] + -node_id [site_node::get_node_id_from_object_id -object_id $package_id] ] # # todo @@ -49,23 +47,22 @@ # else ask to create a new root folder ad_return_template - } else { if {[attachments::root_folder_p -pacakge_id $package_id]} { # sanity check that the attachments_root_folder and fs_root_folder match set attachments_root_folder [attachments::get_root_folder \ - -package_id $package_id - ] + -package_id $package_id ] if {$attachments_root_folder != $root_folder_id} { ad_return_complaint 1 "Error: Attachment root folder and fs root folder different!" - } else { - # since this pkg already has a root folder do the mapping and return - attachments::map_root_folder \ - -package_id $package_id \ - -folder_id $root_folder_id - - ad_returnredirect $referer + } else { + # since this pkg already has a root folder do the mapping and return + attachments::map_root_folder \ + -package_id $package_id \ + -folder_id $root_folder_id + + ad_returnredirect $referer + ad_script_abort } } } Index: openacs-4/packages/chat/www/chat.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/packages/chat/www/chat.tcl,v diff -u -r1.2 -r1.3 --- openacs-4/packages/chat/www/chat.tcl 4 Sep 2002 08:12:50 -0000 1.2 +++ openacs-4/packages/chat/www/chat.tcl 18 Sep 2002 13:03:47 -0000 1.3 @@ -49,6 +49,7 @@ if { ($read_p == "0" && $write_p == "0") || ($ban_p == "1") } { #Display unauthorize privilege page. ad_returnredirect unauthorized + ad_script_abort } # Get chat screen name. Index: openacs-4/packages/chat/www/room-enter.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/packages/chat/www/Attic/room-enter.tcl,v diff -u -r1.1 -r1.2 --- openacs-4/packages/chat/www/room-enter.tcl 20 Apr 2001 20:51:08 -0000 1.1 +++ openacs-4/packages/chat/www/room-enter.tcl 18 Sep 2002 13:03:47 -0000 1.2 @@ -20,6 +20,7 @@ if { ($read_p == "0" && $write_p == "0") || ($ban_p == "1") } { #Display unauthorize privilege page. ad_returnredirect unauthorized + ad_script_abort } if {$client == "html"} { chat_message_post $room_id $user_id "has entered the room." "1" Index: openacs-4/packages/chat/www/room-exit.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/packages/chat/www/room-exit.tcl,v diff -u -r1.1 -r1.2 --- openacs-4/packages/chat/www/room-exit.tcl 20 Apr 2001 20:51:08 -0000 1.1 +++ openacs-4/packages/chat/www/room-exit.tcl 18 Sep 2002 13:03:47 -0000 1.2 @@ -19,9 +19,9 @@ if { ($read_p == "0" && $write_p == "0") || ($ban_p == "1") } { #Display unauthorize privilege page. ad_returnredirect unauthorized + ad_script_abort } -chat_message_post $room_id $user_id "has leaved the room." "1" +chat_message_post $room_id $user_id "has left the room." "1" - ad_returnredirect index Index: openacs-4/packages/download/www/admin/approve-or-reject-2.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/packages/download/www/admin/approve-or-reject-2.tcl,v diff -u -r1.3 -r1.4 --- openacs-4/packages/download/www/admin/approve-or-reject-2.tcl 10 Sep 2002 22:22:35 -0000 1.3 +++ openacs-4/packages/download/www/admin/approve-or-reject-2.tcl 18 Sep 2002 13:06:33 -0000 1.4 @@ -52,6 +52,7 @@ # Everything after here is email related, so let's send the user on their way ad_returnredirect $return_url +# do not abort/return here! if {[ad_parameter -package_id [ad_conn package_id] "approval_notification" "download" 1] == 1} { # We want to send email to use who submitted the version to let Index: openacs-4/packages/download/www/admin/repository-ae.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/packages/download/www/admin/repository-ae.tcl,v diff -u -r1.3 -r1.4 --- openacs-4/packages/download/www/admin/repository-ae.tcl 10 Sep 2002 22:22:35 -0000 1.3 +++ openacs-4/packages/download/www/admin/repository-ae.tcl 18 Sep 2002 13:06:33 -0000 1.4 @@ -67,6 +67,7 @@ } } ad_returnredirect "$return_url" + ad_script_abort } ad_return_template \ No newline at end of file Index: openacs-4/packages/download/www/admin/repository-metadata-edit.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/packages/download/www/admin/repository-metadata-edit.tcl,v diff -u -r1.3 -r1.4 --- openacs-4/packages/download/www/admin/repository-metadata-edit.tcl 10 Sep 2002 22:22:35 -0000 1.3 +++ openacs-4/packages/download/www/admin/repository-metadata-edit.tcl 18 Sep 2002 13:06:33 -0000 1.4 @@ -94,6 +94,7 @@ } } ad_returnredirect "repository-metadata" + ad_script_abort } Index: openacs-4/packages/download/www/admin/repository-reasons-edit.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/packages/download/www/admin/repository-reasons-edit.tcl,v diff -u -r1.3 -r1.4 --- openacs-4/packages/download/www/admin/repository-reasons-edit.tcl 10 Sep 2002 22:22:35 -0000 1.3 +++ openacs-4/packages/download/www/admin/repository-reasons-edit.tcl 18 Sep 2002 13:06:33 -0000 1.4 @@ -31,6 +31,7 @@ where download_reason_id = :download_reason_id } ad_returnredirect "repository-reasons" + ad_script_abort } set title "Edit Reason" Index: openacs-4/packages/download/www/admin/repository-types-edit.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/packages/download/www/admin/repository-types-edit.tcl,v diff -u -r1.3 -r1.4 --- openacs-4/packages/download/www/admin/repository-types-edit.tcl 10 Sep 2002 22:22:35 -0000 1.3 +++ openacs-4/packages/download/www/admin/repository-types-edit.tcl 18 Sep 2002 13:06:33 -0000 1.4 @@ -32,6 +32,7 @@ where archive_type_id = :archive_type_id } ad_returnredirect "repository-types" + ad_script_abort } set title "Edit $pretty_name" Index: openacs-4/packages/forums/www/moderate/message-delete.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/packages/forums/www/moderate/message-delete.tcl,v diff -u -r1.4 -r1.5 --- openacs-4/packages/forums/www/moderate/message-delete.tcl 10 Sep 2002 22:22:58 -0000 1.4 +++ openacs-4/packages/forums/www/moderate/message-delete.tcl 18 Sep 2002 13:02:00 -0000 1.5 @@ -20,14 +20,21 @@ # Select the stuff forum::message::get -message_id $message_id -array message -# Confirm? -if {!$confirm_p} { - set url_vars [export_url_vars message_id return_url] - ad_return_template -} else { +# Confirmed? +if {$confirm_p} { # Delete the message and all children forum::message::delete -message_id $message_id # Redirect to the forum ad_returnredirect "../forum-view?forum_id=$message(forum_id)" + ad_script_abort } + +set url_vars [export_url_vars message_id return_url] + +ad_return_template + + + + + Index: openacs-4/packages/spam/www/spam-edit.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/packages/spam/www/spam-edit.tcl,v diff -u -r1.3 -r1.4 --- openacs-4/packages/spam/www/spam-edit.tcl 5 Sep 2002 22:14:36 -0000 1.3 +++ openacs-4/packages/spam/www/spam-edit.tcl 18 Sep 2002 13:02:24 -0000 1.4 @@ -29,6 +29,7 @@ if {$sent_p == "t"} { ad_returnredirect "[spam_base]spam-view?spam_id=$spam_id" + ad_script_abort } ad_set_client_property spam "sql_query" $sql_query @@ -63,6 +64,7 @@ set html_text $content } else { ad_return_error "invalid content type in spam: $mime_type" + ad_script_abort } } Index: openacs-4/packages/spam/www/admin/spam-edit.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/packages/spam/www/admin/spam-edit.tcl,v diff -u -r1.4 -r1.5 --- openacs-4/packages/spam/www/admin/spam-edit.tcl 5 Sep 2002 22:14:37 -0000 1.4 +++ openacs-4/packages/spam/www/admin/spam-edit.tcl 18 Sep 2002 13:02:25 -0000 1.5 @@ -28,6 +28,7 @@ if {$sent_p == "t"} { ad_returnredirect "[spam_base]spam-view?spam_id=$spam_id" + ad_script_abort } ad_set_client_property spam "sql_query" $sql_query