From d440aa097f47460e6022a88edf8f889db2ad0ad4 Mon Sep 17 00:00:00 2001
From: Thomas Heck <theck@linet-services.de>
Date: Thu, 3 Jan 2013 14:45:01 +0100
Subject: [PATCH] =?utf8?q?HTML=20aus=20der=20Funktion=20bin/ap.pl:form=5Ff?=
 =?utf8?q?ooter=20in=20ein=20Template=20=C3=BCbertragen.?=
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

---
 bin/mozilla/ar.pl                      | 88 ++++----------------------
 templates/webpages/ar/form_footer.html | 52 +++++++++++++++
 2 files changed, 66 insertions(+), 74 deletions(-)
 create mode 100644 templates/webpages/ar/form_footer.html

diff --git a/bin/mozilla/ar.pl b/bin/mozilla/ar.pl
index 6466aa24e..f8b4f0ba7 100644
--- a/bin/mozilla/ar.pl
+++ b/bin/mozilla/ar.pl
@@ -456,89 +456,29 @@ sub form_footer {
   my $locale   = $main::locale;
   my $cgi      = $::request->{cgi};
 
-  my ($transdate, $closedto);
-
-  my $follow_ups_block;
-  if ($form->{id}) {
+  if ( $form->{id} ) {
     my $follow_ups = FU->follow_ups('trans_id' => $form->{id});
-
-    if (@{ $follow_ups} ) {
-      my $num_due       = sum map { $_->{due} * 1 } @{ $follow_ups };
-      $follow_ups_block = qq|<p>| . $locale->text("There are #1 unfinished follow-ups of which #2 are due.", scalar @{ $follow_ups }, $num_due) . qq|</p>|;
+    if ( @{ $follow_ups} ) {
+      $form->{follow_up_length} = scalar(@{$follow_ups});
+      $form->{follow_up_due_length} = sum(map({ $_->{due} * 1 } @{ $follow_ups }));
     }
   }
 
-  print qq|
-
-$follow_ups_block
-
-<input name=gldate type=hidden value="| . Q($form->{gldate}) . qq|">
-
-<input name=callback type=hidden value="$form->{callback}">
-|
-. $cgi->hidden('-name' => 'draft_id', '-default' => [$form->{draft_id}])
-. $cgi->hidden('-name' => 'draft_description', '-default' => [$form->{draft_description}])
-. qq|
-
-<br>
-|;
-
-  if (!$form->{id} && $form->{draft_id}) {
-    print(NTI($cgi->checkbox('-name' => 'remove_draft', '-id' => 'remove_draft',
-                             '-value' => 1, '-checked' => $form->{remove_draft},
-                             '-label' => '')) .
-          qq|&nbsp;<label for="remove_draft">| .
-          $locale->text("Remove draft when posting") .
-          qq|</label><br>|);
-  }
-
-  $transdate = $form->datetonum($form->{transdate}, \%myconfig);
-  $closedto  = $form->datetonum($form->{closedto},  \%myconfig);
+  my $transdate = $form->datetonum($form->{transdate}, \%myconfig);
+  my $closedto  = $form->datetonum($form->{closedto},  \%myconfig);
 
-  print qq|<input class="submit" type="submit" name="action" id="update_button" value="| . $locale->text('Update') . qq|">\n|;
+  $form->{is_closed} = $transdate <= $closedto;
 
   # ToDO: - insert a global check for stornos, so that a storno is only possible a limited time after saving it
-  print qq| <input class=submit type=submit name=action value="| . $locale->text('Storno') . qq|"> |
-    if ($form->{id} && !IS->has_storno(\%myconfig, $form, 'ar') && !IS->is_storno(\%myconfig, $form, 'ar') && (($form->{totalpaid} == 0) || ($form->{totalpaid} eq "")));
-
-  if ($form->{id}) {
-    if ($form->{radier}) {
-      print qq|
-        <input class=submit type=submit name=action value="| . $locale->text('Post') .            qq|">
-        <input class=submit type=submit name=action value="| . $locale->text('Delete') .          qq|"> |;
-    }
-    if ($transdate > $closedto) {
-      print qq|
-        <input class=submit type=submit name=action value="| . $locale->text('Use As New') . qq|"> |;
-    }
-    print qq|
-        <input class=submit type=submit name=action value="| . $locale->text('Post Payment') .    qq|">
-        <input type="button" class="submit" onclick="follow_up_window()" value="|
-      . $locale->text('Follow-Up')
-      . qq|"> |;
+  $form->{show_storno_button} =
+    $form->{id} &&
+    !IS->has_storno(\%myconfig, $form, 'ar') &&
+    !IS->is_storno(\%myconfig, $form, 'ar') &&
+    ($form->{totalpaid} == 0 || $form->{totalpaid} eq "");
 
-  } else {
-    if ($transdate > $closedto) {
-      print qq| <input class=submit type=submit name=action value="| . $locale->text('Post') .     qq|"> | .
-        NTI($cgi->submit('-name' => 'action', '-value' => $locale->text('Save draft'), '-class' => 'submit'));
-    }
-  }
+  $form->{show_mark_as_paid_button} = $form->{id} && $::instance_conf->get_ar_show_mark_as_paid();
 
-  # button for saving history
-  if($form->{id} ne "") {
-    print qq| <input type=button class=submit onclick=set_history_window($form->{id}); name=history id=history value=| . $locale->text('history') . qq|> |;
-  }
-  # /button for saving history
-  # mark_as_paid button
-  if(($form->{id} ne "") && $::instance_conf->get_ar_show_mark_as_paid) {
-    print qq|<input type="submit" class="submit" name="action" value="|
-          . $locale->text('mark as paid') . qq|">|;
-  }
-  # /mark_as_paid button
-
-  print "
-</form>
-";
+  print $::form->parse_html_template('ar/form_footer');
 
   $main::lxdebug->leave_sub();
 }
diff --git a/templates/webpages/ar/form_footer.html b/templates/webpages/ar/form_footer.html
new file mode 100644
index 000000000..6fa8b4441
--- /dev/null
+++ b/templates/webpages/ar/form_footer.html
@@ -0,0 +1,52 @@
+[% USE LxERP %]
+[% USE T8 %]
+
+  [% IF ( follow_up_length && follow_up_due_length ) %]
+    [% LxERP.t8('There are #1 unfinished follow-ups of which #2 are due.', follow_up_length , follow_up_due_length) %]
+  [% END %]
+
+  <input type="hidden" name="gldate" value="[% gldate | html %]">
+  <input type="hidden" name="callback" value="[% callback | html %]">
+  <input type="hidden" name="draft_id" value="[% draft_id | html %]">
+  <input type="hidden" name="draft_description" value="[% draft_description | html %]">
+
+  <br>
+
+  [% IF ( !id && draft_id ) %]
+    <input type="checkbox" name="remove_draft" id="remove_draft" value="1" [% IF ( remove_draft ) %]checked[% END %]>
+    <label for="remove_draft">[% 'Remove draft when posting' | $T8 %]</label>
+  [% END %]
+
+  <br>
+
+  <input class="submit" type="submit" name="action" id="update_button" value="[% 'Update' | $T8 %]">
+
+  [% IF ( show_storno_button ) %]
+    <input class="submit" type="submit" name="action" value="[% 'Storno' | $T8 %]">
+  [% END %]
+
+  [% IF ( id ) %]
+    [% IF ( radier ) %]
+      <input class="submit" type="submit" name="action" value="[% 'Post' | $T8 %]">
+      <input class="submit" type="submit" name="action" value="[% 'Delete' | $T8 %]">
+    [% END %]
+
+    [% IF ( !is_closed ) %]
+      <input class="submit" type="submit" name="action" value="[% 'Use As New' | $T8 %]">
+    [% END %]
+
+    <input class="submit" type="submit" name="action" value="[% 'Post Payment' | $T8 %]">
+    <input type="button" class="submit" onclick="follow_up_window()" value="[% 'Follow-Up' | $T8 %]">
+    <input type="button" class="submit" onclick="set_history_window([% id %]);" name="history" id="history" value="[% 'history' | $T8 %]">
+  [% ELSE %]
+    [% IF ( !is_closed ) %]
+      <input class="submit" type="submit" name="action" value="[% 'Post' | $T8 %]">
+      <input class="submit" type="submit" name="action" value="[% 'Save draft' | $T8 %]">
+    [% END %]
+  [% END %]
+
+  [% IF ( show_mark_as_paid_button ) %]
+    <input type="submit" class="submit" name="action" value="[% 'mark as paid' | $T8 %]">
+  [% END %]
+
+</form>
-- 
2.20.1