From 3715333a1e32b0c54b5bac2f878f133c76f386b5 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Jan=20B=C3=BCren?= <jan@kivitendo.de>
Date: Wed, 2 Jun 2021 09:09:19 +0200
Subject: [PATCH] =?utf8?q?DB-Buchungsvorlage:=20Auf=20Betrag=20im=20Soll?=
 =?utf8?q?=20und=20Haben=20pr=C3=BCfen?=
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

Andernfalls kann der Kontoauszugsbetrag nicht richtig gesetzt werden
---
 bin/mozilla/gl.pl | 8 +++++++-
 locale/de/all     | 1 +
 locale/en/all     | 1 +
 3 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/bin/mozilla/gl.pl b/bin/mozilla/gl.pl
index 886d756f8..dbaddc596 100644
--- a/bin/mozilla/gl.pl
+++ b/bin/mozilla/gl.pl
@@ -158,9 +158,15 @@ sub save_record_template {
   my $template = $::form->{record_template_id} ? SL::DB::RecordTemplate->new(id => $::form->{record_template_id})->load : SL::DB::RecordTemplate->new;
   my $js       = SL::ClientJS->new(controller => SL::Controller::Base->new);
   my $new_name = $template->template_name_to_use($::form->{record_template_new_template_name});
-
   $js->dialog->close('#record_template_dialog');
 
+
+  # bank transactions need amounts for assignment
+  my $can_save = 0;
+  $can_save    = 1 if ($::form->{credit_1} > 0 && $::form->{debit_2} > 0 && $::form->{credit_2} == 0 && $::form->{debit_1} == 0);
+  $can_save    = 1 if ($::form->{credit_2} > 0 && $::form->{debit_1} > 0 && $::form->{credit_1} == 0 && $::form->{debit_2} == 0);
+  return $js->flash('error', t8('Can only save template if amounts,i.e. 1 for debit and credit are set.'))->render unless $can_save;
+
   my @items = grep {
     $_->{chart_id} && (($_->{tax_id} // '') ne '')
   } map {
diff --git a/locale/de/all b/locale/de/all
index bc89120dd..fd983ea01 100755
--- a/locale/de/all
+++ b/locale/de/all
@@ -544,6 +544,7 @@ $self->{texts} = {
   'Calling #1 now'              => 'Wähle jetzt #1',
   'Can not create that quantity with current stock' => 'Diese Anzahl kann mit dem gegenwärtigen Lagerbestand nicht hergestellt werden.',
   'Can only delete the "Storno zu" part of the cancellation pair.' => 'Löschen von R(S) Rechnung nicht erlaubt. Löschen der entsprechenden "Storno zu" Gutschrift reaktiviert diese Rechnung wieder.',
+  'Can only save template if amounts,i.e. 1 for debit and credit are set.' => 'Kann die Vorlage nicht speichern. Es wird mindestens ein Betrag im Soll und im Haben benötigt (bspw. 1), damit bspw. Beträge aus Kontoauszügen korrekt gesetzt werden können.',
   'Can\'t connect to shop. #1'  => 'Kann keine Verbindung zu Shop #1 herstellen.',
   'Can\'t load item without a valid part.id' => 'Kann Artikel ohne gültige part.id nicht laden',
   'Cancel'                      => 'Abbrechen',
diff --git a/locale/en/all b/locale/en/all
index 7dbf694da..3aef25ce9 100644
--- a/locale/en/all
+++ b/locale/en/all
@@ -544,6 +544,7 @@ $self->{texts} = {
   'Calling #1 now'              => '',
   'Can not create that quantity with current stock' => '',
   'Can only delete the "Storno zu" part of the cancellation pair.' => '',
+  'Can only save template if amounts,i.e. 1 for debit and credit are set.' => '',
   'Can\'t connect to shop. #1'  => '',
   'Can\'t load item without a valid part.id' => '',
   'Cancel'                      => '',
-- 
2.20.1