From: Moritz Bunkus <m.bunkus@linet-services.de>
Date: Thu, 11 Jul 2013 07:58:58 +0000 (+0200)
Subject: Löschen von Angeboten/Aufträgen mit JavaScript-Sicherheitsabfrage
X-Git-Tag: release-3.1.0beta1~195
X-Git-Url: http://wagnertech.de/git?a=commitdiff_plain;h=3deafde7289c8c01b046de42574dac0cd8fa9b79;p=kivitendo-erp.git

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

diff --git a/bin/mozilla/oe.pl b/bin/mozilla/oe.pl
index 5db8f8e4d..8f42a067b 100644
--- a/bin/mozilla/oe.pl
+++ b/bin/mozilla/oe.pl
@@ -1239,31 +1239,6 @@ sub save {
 }
 
 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;
diff --git a/js/locale/de.js b/js/locale/de.js
index ee83fd692..0d45d3094 100644
--- 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 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 2fb88ce56..a376ed561 100755
--- 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&ouml;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?',
diff --git a/templates/webpages/oe/delete.html b/templates/webpages/oe/delete.html
deleted file mode 100644
index 6ef6c0e56..000000000
--- 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 8e4ad3c1d..c9281ac66 100644
--- 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_delete" value="[% 'Delete' | $T8 %]">
+    [% L.submit_tag('action_delete', LxERP.t8('Delete'), confirm=LxERP.t8('Are you sure?')) %]
   [%- END %]
 
   [%- IF is_sales_quo %]