Löschen von Angeboten/Aufträgen mit JavaScript-Sicherheitsabfrage

diff --git a/bin/mozilla/oe.pl b/bin/mozilla/oe.pl
index 5db8f8e4db6778e7c0c4b8c15b12b76891ece148..8f42a067b7d48b73a3420dc453bdfe86e068f5d3 100644 (file)
--- a/bin/mozilla/oe.pl
+++ b/bin/mozilla/oe.pl
@@ -1239,31 +1239,6 @@ sub save {
 }
 
 sub delete {
 }
 
 sub delete {

-  $::lxdebug->enter_sub;
-
-  check_oe_access();
-
-  $::form->header;
-
-  # delete action variable
-  delete $::form->{$_} for qw(action header);
-
-  my @hiddens;
-  for my $key (keys %$::form) {
-    next if $key eq 'login' || $key eq 'password' || '' ne ref $::form->{$key};
-    push @hiddens, { key => $key, value => $::form->{$key} };
-  }
-
-  print $::form->parse_html_template('oe/delete', {
-    hiddens => \@hiddens,
-    is_order => scalar($::form->{type} =~ /_order$/),
-  });
-
-
-  $::lxdebug->leave_sub;
-}
-
-sub delete_order_quotation {

   $main::lxdebug->enter_sub();
 
   my $form     = $main::form;
   $main::lxdebug->enter_sub();
 
   my $form     = $main::form;

diff --git a/js/locale/de.js b/js/locale/de.js
index ee83fd6928d1657ee745d1f409c828abf9e8cab7..0d45d3094b651a9974d4d94f2b3d4a997eb1e253 100644 (file)
--- a/js/locale/de.js
+++ b/js/locale/de.js
@@ -4,5 +4,5 @@ namespace("kivi").setupLocale({
 "The name is missing.":"Der Name fehlt.",
 "The name must only consist of letters, numbers and underscores and start with a letter.":"Der Name darf nur aus Buchstaben (keine Umlaute), Ziffern und Unterstrichen bestehen und muss mit einem Buchstaben beginnen.",
 "The option field is empty.":"Das Optionsfeld ist leer.",
 "The name is missing.":"Der Name fehlt.",
 "The name must only consist of letters, numbers and underscores and start with a letter.":"Der Name darf nur aus Buchstaben (keine Umlaute), Ziffern und Unterstrichen bestehen und muss mit einem Buchstaben beginnen.",
 "The option field is empty.":"Das Optionsfeld ist leer.",

-"The selected database is still configured for client \"#1\". If you delete the database that client will stop working until you re-configure it. Do you still want to delete the database?":"Die auswählte Datenbank ist noch für Mandant \"#1\" konfiguriert. Wenn Sie die Datenbank löschen, wird der Mandanten nicht mehr funktionieren, bis er anders konfiguriert wurde. Wollen Sie die Datenbank trotzdem löschen?",
+"The selected database is still configured for client \"#1\". If you delete the database that client will stop working until you re-configure it. Do you still want to delete the database?":"Die auswählte Datenbank ist noch für Mandant \"#1\" konfiguriert. Wenn Sie die Datenbank löschen, wird der Mandanten nicht mehr funktionieren, bis er anders konfiguriert wurde. Wollen Sie die Datenbank trotzdem löschen?"

 });
 });

diff --git a/locale/de/all b/locale/de/all
index 2fb88ce56cb084ded532c8b19012cb64cd35f195..a376ed56153c10a358fc6ce1491b8b700d7cda9a 100755 (executable)
--- a/locale/de/all
+++ b/locale/de/all
@@ -212,8 +212,6 @@ $self->{texts} = {
   'Ar aging on %s'              => 'Offene Forderungen zum %s',
   'Are you sure you want to delete Delivery Order Number #1?' => 'Sind Sie sicher, dass Sie Lieferschein #1 löschen wollen?',
   'Are you sure you want to delete Invoice Number' => 'Soll die Rechnung mit folgender Nummer wirklich gelöscht werden:',
   'Ar aging on %s'              => 'Offene Forderungen zum %s',
   'Are you sure you want to delete Delivery Order Number #1?' => 'Sind Sie sicher, dass Sie Lieferschein #1 löschen wollen?',
   'Are you sure you want to delete Invoice Number' => 'Soll die Rechnung mit folgender Nummer wirklich gelöscht werden:',

-  'Are you sure you want to delete Order Number' => 'Soll der Auftrag mit folgender Nummer wirklich gelöscht werden:',
-  'Are you sure you want to delete Quotation Number' => 'Sind Sie sicher, dass Angebotnummer gelöscht werden soll?',

   'Are you sure you want to delete Transaction' => 'Buchung wirklich löschen?',
   'Are you sure you want to delete this background job?' => 'Sind Sie sicher, dass Sie diesen Hintergrund-Job löschen möchten?',
   'Are you sure you want to delete this business?' => 'Sind Sie sicher, dass Sie diesen Kunden-/Lieferantentyp löschen wollen?',
   'Are you sure you want to delete Transaction' => 'Buchung wirklich löschen?',
   'Are you sure you want to delete this background job?' => 'Sind Sie sicher, dass Sie diesen Hintergrund-Job löschen möchten?',
   'Are you sure you want to delete this business?' => 'Sind Sie sicher, dass Sie diesen Kunden-/Lieferantentyp löschen wollen?',

diff --git a/templates/webpages/oe/delete.html b/templates/webpages/oe/delete.html
deleted file mode 100644 (file)
index 6ef6c0e..0000000
--- a/templates/webpages/oe/delete.html
+++ /dev/null
@@ -1,24 +0,0 @@
-[%- USE T8 %]
-[%- USE L %]
-[%- USE LxERP %]
-
-
-<form method="post" action="[% script %]">
-
-<h2 class="confirm">[% 'Confirm!' | $T8 %]</h2>
-
-[%- IF is_order %]
-<h4>[% 'Are you sure you want to delete Order Number' | $T8 %] [% ordnumber %]</h4>
-[%- ELSE %]
-<h4>[% 'Are you sure you want to delete Quotation Number' | $T8 %] [% quonumber %]</h4>
-[%- END %]
-
-<br>
-
-[% L.hidden_tag(row.key, row.value) FOREACH row = hiddens %]
-[% L.hidden_tag('yes_nextsub', 'delete_order_quotation') %]
-[% L.submit_tag('action', LxERP.t8('Yes')) %]
-
-<button class=submit type=button onclick="history.back()">[% 'No' | $T8 %]</button>
-</form>
-

diff --git a/templates/webpages/oe/form_footer.html b/templates/webpages/oe/form_footer.html
index 8e4ad3c1dedf8e3e39e9afd67b796e886139536b..c9281ac66d4f8a406f44571098565749eeeba1aa 100644 (file)
--- a/templates/webpages/oe/form_footer.html
+++ b/templates/webpages/oe/form_footer.html
@@ -137,7 +137,7 @@
   <input class="submit" type="submit" name="action_save_as_new" value="[% 'Save as new' | $T8 %]">
 
   [%- UNLESS (is_sales_ord && !INSTANCE_CONF.get_sales_order_show_delete) || (is_pur_ord && !INSTANCE_CONF.get_purchase_order_show_delete) %]
   <input class="submit" type="submit" name="action_save_as_new" value="[% 'Save as new' | $T8 %]">
 
   [%- UNLESS (is_sales_ord && !INSTANCE_CONF.get_sales_order_show_delete) || (is_pur_ord && !INSTANCE_CONF.get_purchase_order_show_delete) %]

-    <input class="submit" type="submit" name="action_delete" value="[% 'Delete' | $T8 %]">
+    [% L.submit_tag('action_delete', LxERP.t8('Delete'), confirm=LxERP.t8('Are you sure?')) %]

   [%- END %]
 
   [%- IF is_sales_quo %]
   [%- END %]
 
   [%- IF is_sales_quo %]