From 0bb4cd20cd1b1de4c2b73f95119d116b9427807a Mon Sep 17 00:00:00 2001
From: =?utf8?q?Sven=20Sch=C3=B6ling?= <s.schoeling@linet-services.de>
Date: Wed, 18 Jan 2012 10:44:45 +0100
Subject: [PATCH] html strict: table layout.
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

2 einfache regeln:
1. table > tr > td in dieser Reihenfolge muss auch korrekt wieder zu gemacht werden.
2. tr darf als Kindelemente nur und ausschließlich td und th haben. Keine hiddens, kein anderer Schnickschnack.
---
 templates/webpages/is/_payments.html   | 18 +++++++++++++----
 templates/webpages/is/form_footer.html | 14 ++++++++-----
 templates/webpages/is/form_header.html | 28 ++++++++++++++++----------
 templates/webpages/webdav/_list.html   |  3 +++
 4 files changed, 43 insertions(+), 20 deletions(-)

diff --git a/templates/webpages/is/_payments.html b/templates/webpages/is/_payments.html
index 4a64d0db4..771cede74 100644
--- a/templates/webpages/is/_payments.html
+++ b/templates/webpages/is/_payments.html
@@ -33,9 +33,12 @@
   [% SET paid          = 'paid_'          _ i %]
   [% SET AR_paid       = 'AR_paid_'       _ i %]
   [% SET selectAR_paid_ref = 'selectAR_paid_' _ i %]
-
-     <input type=hidden name="acc_trans_id_[% i %]" value=[% $acc_trans_id %]>
-     <input type=hidden name="gldate_[% i %]" value=[% $gldate %]>
+     <tr style='display:none'>
+       <td>
+         <input type=hidden name="acc_trans_id_[% i %]" value="[% $acc_trans_id %]">
+         <input type=hidden name="gldate_[% i %]" value="[% $gldate %]">
+       </td>
+     </tr>
 
      <tr>
      <td align="center">
@@ -99,11 +102,15 @@
 
     </tr>
   [% IF $changeable %]
+    <tr style='display:none'>
+     <td>
     <script type='text/javascript'>
      Calendar.setup({ inputField : "datepaid_[% i %]", ifFormat :"[% myconfig_jsc_dateformat %]", align : "TR", button : "trigger_datepaid_[% i %]" });
      $('input[name="paid_[% i %]"]').blur(function(){ check_right_number_format(this) });
      $('#datepaid_[% i %]').blur(function(){ check_right_date_format(this) });
     </script>
+     </td>
+    </tr>
   [% END %]
 
 [% END # foreach %]
@@ -120,12 +127,15 @@
       <td align="center">[%- 'Missing amount' | $T8 %]</td>
       <td align="center">[% LxERP.format_amount(paid_missing, 2) | html %]</td>
     </tr>
-
+    <tr style='display:none'>
+     <td>
      <input type="hidden" name="paidaccounts" value="[% paidaccounts %]">
      <input type="hidden" name="selectAR_paid" value="[% selectAR_paid %]">
      <input type="hidden" name="oldinvtotal" value="[% oldinvtotal %]">
 
      <input type="hidden" name="oldtotalpaid" value="[% totalpaid %]">
+    </td>
+   </tr>
     </table>
 
     </td>
diff --git a/templates/webpages/is/form_footer.html b/templates/webpages/is/form_footer.html
index d1e5e82cc..3a86b2ab8 100644
--- a/templates/webpages/is/form_footer.html
+++ b/templates/webpages/is/form_footer.html
@@ -46,14 +46,18 @@
        <table>
         <tr>
          <th  align=left>[% 'Ertrag' | $T8 %]</th>
-         <td>[% LxERP.format_amount(marge_total, 2, 0) %]</td>
+         <td>
+          [% LxERP.format_amount(marge_total, 2, 0) %]
+          <input type=hidden name="marge_total" value="[% marge_total %]">
+         </td>
         </tr>
         <tr>
          <th  align=left>[% 'Ertrag prozentual' | $T8 %]</th>
-         <td>[% LxERP.format_amount(marge_percent, 2, 0) %] %</td>
+         <td>
+          [% LxERP.format_amount(marge_percent, 2, 0) %] %
+          <input type=hidden name="marge_percent" value="[% marge_percent %]">
+         </td>
         </tr>
-        <input type=hidden name="marge_total" value="[% marge_total %]">
-        <input type=hidden name="marge_percent" value="[% marge_percent %]">
        </table>
       </td>
       <td align="right">
@@ -163,7 +167,7 @@
   [% END %]
 
 <input type="hidden" name="rowcount" value="[% rowcount %]">
-<input type="hidden" name="callback" value="[% callback %]">
+<input type="hidden" name="callback" value="[% callback | html %]">
 <input type="hidden" name="draft_id" value="[% draft_id %]">
 <input type="hidden" name="draft_description" value="[% draft_description %]">
 <input type="hidden" name="customer_discount" value="[% customer_discount %]">
diff --git a/templates/webpages/is/form_header.html b/templates/webpages/is/form_header.html
index 78d5ef27b..c322a777e 100644
--- a/templates/webpages/is/form_header.html
+++ b/templates/webpages/is/form_header.html
@@ -45,11 +45,11 @@
                  allow_textbox = 1,
                  onChange      = "document.getElementById('update_button').click();" -%]
             <input type="button" value="[% 'Details (one letter abbreviation)' | $T8 %]" onclick="show_vc_details('[% HTML.escape(vc) %]')">
-          </td>
           <input type="hidden" name="customer_klass" value="[% HTML.escape(customer_klass) %]">
           <input type="hidden" name="customer_id" value="[% HTML.escape(customer_id) %]">
           <input type="hidden" name="oldcustomer" value="[% HTML.escape(oldcustomer) %]">
           <input type="hidden" name="selectcustomer" value="[% HTML.escape(selectcustomer) %]">
+          </td>
         </tr>
 [%- IF ALL_CONTACTS.size %]
         <tr>
@@ -90,8 +90,10 @@
 [%- END %]
         <tr>
           <th align="right" nowrap>[% 'Record in' | $T8 %]</th>
-          <td colspan="3"><select name="AR" style="width:250px;">[% selectAR %]</select></td>
-          <input type="hidden" name="selectAR" value="[% selectAR %]">
+          <td colspan="3">
+            <select name="AR" style="width:250px;">[% selectAR %]</select>
+            <input type="hidden" name="selectAR" value="[% selectAR | html %]">
+          </td>
         </tr>
         <tr>
           <th align="right">[% 'Steuersatz' | $T8 %]</th>
@@ -100,6 +102,7 @@
   [%- IF id %]
           <input type='hidden' name='taxzone_id' value='[% taxzone_id %]'>
   [%- END %]
+          </td>
         </tr>
 [%- IF all_departments %]
         <tr>
@@ -115,8 +118,12 @@
           <td>[% currencies %]</td>
         </tr>
 [%- END %]
-        <input type="hidden" name="fxgain_accno" value="[% fxgain_accno %]">
-        <input type="hidden" name="fxloss_accno" value="[% fxloss_accno %]">
+        <tr style='display:none'>
+          <td>
+            <input type="hidden" name="fxgain_accno" value="[% fxgain_accno %]">
+            <input type="hidden" name="fxloss_accno" value="[% fxloss_accno %]">
+          </td>
+        </tr>
 [%- IF show_exchangerate %]
         <tr>
           <th align="right">[% 'Exchangerate' | $T8 %]</th>
@@ -225,13 +232,13 @@
       </table>
     </td>
   </tr>
-</table>
-    </td>
-  </tr>
-  <tr>
+  <tr style='display:none'>
     <td>
+      <input type="hidden" name="webdav" value="[% webdav %]">
     </td>
   </tr>
+</table>
+
   <script type="text/javascript" src="js/show_form_details.js"></script>
   <script type="text/javascript" src="js/show_vc_details.js"></script>
   <script type="text/javascript" src="js/show_history.js"></script>
@@ -265,5 +272,4 @@
      }
    //-->
   </script>
-
-  <input type="hidden" name="webdav" value="[% webdav %]">
+<table width="100%">
diff --git a/templates/webpages/webdav/_list.html b/templates/webpages/webdav/_list.html
index 3432fc614..5e86becb9 100644
--- a/templates/webpages/webdav/_list.html
+++ b/templates/webpages/webdav/_list.html
@@ -6,6 +6,8 @@
   <tr>
    <th class="listtop" align="left">Dokumente im Webdav-Repository</th>
   </tr>
+  <tr>
+   <td>
    <table width="100%">
     <tr>
      <td align="left" width="30%"><b>Dateiname</b></td>
@@ -18,5 +20,6 @@
     </tr>
  [%- END %]
    </table>
+   </td>
   </tr>
 [% END %]
-- 
2.20.1