From 4969e04fadd6edd9a590dd23e0143b511cb13bc5 Mon Sep 17 00:00:00 2001
From: mark <mark>
Date: Wed, 3 Nov 2010 01:04:46 +0000
Subject: [PATCH] safety improvement to RT#9947

---
 httemplate/edit/bulk-cust_pkg.html | 14 ++++++--------
 1 file changed, 6 insertions(+), 8 deletions(-)

diff --git a/httemplate/edit/bulk-cust_pkg.html b/httemplate/edit/bulk-cust_pkg.html
index ab419215e..2ff38ca53 100644
--- a/httemplate/edit/bulk-cust_pkg.html
+++ b/httemplate/edit/bulk-cust_pkg.html
@@ -4,7 +4,7 @@
 
 <% include('/elements/progress-init.html',
               'OneTrueForm',
-              [qw( old_pkgpart new_pkgpart keep_dates)],
+              [qw( old_pkgpart new_pkgpart )],
               'process/bulk-cust_pkg.cgi',
               $p.'browse/part_pkg.cgi',
            )
@@ -12,7 +12,8 @@
 
 <SCRIPT TYPE="text/javascript">
 function areyousure() {
-  if(confirm('Change these packages?')) {
+  var warning = 'Change these packages?';
+  if(confirm(warning)) {
     process();
   }
 }
@@ -34,9 +35,6 @@ Change <B><% $src_part_pkg->pkg_comment %></B><BR>
 to new package definition
 <SELECT NAME="new_pkgpart">
 % foreach my $dest_part_pkg ( qsearch('part_pkg', { 'disabled' => '' } ) ) {
-% # XXX probably no way to prevent packages from violating agent restrictions
-% # maybe something like what bulk-cust_svc does with changing services 
-% # under specific pkgparts?
 
   <OPTION VALUE="<% $dest_part_pkg->pkgpart %>"><% $dest_part_pkg->pkgpart %>: <% $dest_part_pkg->pkg %>
 % } 
@@ -44,9 +42,9 @@ to new package definition
 </SELECT>
 <BR>
 <BR>
-<INPUT TYPE="checkbox" NAME="keep_dates" CHECKED> Preserve all billing dates
-<BR>
-<BR>
+%#<INPUT TYPE="checkbox" NAME="keep_dates" CHECKED> Preserve all billing dates <I>(strongly recommended)</I>
+%#<BR>
+%#<BR>
 
 <INPUT TYPE="button" VALUE="Bulk change packages" onclick="areyousure()">
 
-- 
2.11.0