Revert "Belege runden, und Rundungsdifferenzen auf Extrakonten buchen."
authorSven Schöling <s.schoeling@linet-services.de>
Fri, 13 May 2016 16:36:36 +0000 (18:36 +0200)
committerSven Schöling <s.schoeling@linet-services.de>
Fri, 13 May 2016 16:36:36 +0000 (18:36 +0200)
This reverts commit 422f2f17365ea7879c14b902c5db1e4cd7023e48.

15 files changed:
SL/DB/MetaSetup/Default.pm
SL/Form.pm
SL/IS.pm
SL/OE.pm
bin/mozilla/am.pl
bin/mozilla/is.pl
bin/mozilla/oe.pl
locale/de/all
locale/en/all
sql/Pg-upgrade2/defaults_add_rnd_accno_ids.sql [deleted file]
templates/webpages/am/edit_accounts.html
templates/webpages/client_config/_default_accounts.html
templates/webpages/is/form_footer.html
templates/webpages/is/form_header.html
templates/webpages/oe/form_footer.html

index 96c4b0f..d8f3506 100644 (file)
@@ -91,8 +91,6 @@ __PACKAGE__->meta->columns(
   revtrans                                  => { type => 'boolean', default => 'false' },
   rfqnumber                                 => { type => 'text' },
   rmanumber                                 => { type => 'text' },
-  rndgain_accno_id                          => { type => 'integer' },
-  rndloss_accno_id                          => { type => 'integer' },
   sales_delivery_order_show_delete          => { type => 'boolean', default => 'true' },
   sales_order_show_delete                   => { type => 'boolean', default => 'true' },
   sales_purchase_order_ship_missing_column  => { type => 'boolean', default => 'false' },
index fbcbe56..d712c94 100644 (file)
@@ -2886,9 +2886,7 @@ sub create_links {
            d.closedto, d.revtrans,
            (SELECT cu.name FROM currencies cu WHERE cu.id=d.currency_id) AS defaultcurrency,
            (SELECT c.accno FROM chart c WHERE d.fxgain_accno_id = c.id) AS fxgain_accno,
-           (SELECT c.accno FROM chart c WHERE d.fxloss_accno_id = c.id) AS fxloss_accno,
-           (SELECT c.accno FROM chart c WHERE d.rndgain_accno_id = c.id) AS rndgain_accno,
-           (SELECT c.accno FROM chart c WHERE d.rndloss_accno_id = c.id) AS rndloss_accno
+           (SELECT c.accno FROM chart c WHERE d.fxloss_accno_id = c.id) AS fxloss_accno
          FROM defaults d|;
     $ref = selectfirst_hashref_query($self, $dbh, $query);
     map { $self->{$_} = $ref->{$_} } keys %$ref;
@@ -2901,9 +2899,7 @@ sub create_links {
             current_date AS transdate, d.closedto, d.revtrans,
             (SELECT cu.name FROM currencies cu WHERE cu.id=d.currency_id) AS defaultcurrency,
             (SELECT c.accno FROM chart c WHERE d.fxgain_accno_id = c.id) AS fxgain_accno,
-            (SELECT c.accno FROM chart c WHERE d.fxloss_accno_id = c.id) AS fxloss_accno,
-            (SELECT c.accno FROM chart c WHERE d.rndgain_accno_id = c.id) AS rndgain_accno,
-            (SELECT c.accno FROM chart c WHERE d.rndloss_accno_id = c.id) AS rndloss_accno
+            (SELECT c.accno FROM chart c WHERE d.fxloss_accno_id = c.id) AS fxloss_accno
           FROM defaults d|;
     $ref = selectfirst_hashref_query($self, $dbh, $query);
     map { $self->{$_} = $ref->{$_} } keys %$ref;
index caa2c34..10892de 100644 (file)
--- a/SL/IS.pm
+++ b/SL/IS.pm
@@ -968,14 +968,7 @@ SQL
     }
   }
 
-  # Invoice Summary includes Rounding
-  my $rounding = $form->round_amount(
-    $form->round_amount($netamount + $tax, 2, 1) - $form->round_amount($netamount + $tax, 2), 2
-  );
-  my $rnd_accno = $rounding == 0 ? 0
-                : $rounding > 0  ? $form->{rndgain_accno}
-                :                  $form->{rndloss_accno};
-  $form->{amount}{ $form->{id} }{ $form->{AR} } = $form->round_amount($netamount + $tax, 2, 1);
+  $form->{amount}{ $form->{id} }{ $form->{AR} } = $netamount + $tax;
   $form->{paid} =
     $form->round_amount($form->{paid} * $form->{exchangerate} + $diff, 2);
 
@@ -1080,14 +1073,6 @@ SQL
         do_query($form, $dbh, $query, @values);
       }
     }
-    if (!$payments_only && ($rnd_accno != 0)) {
-      $query =
-        qq|INSERT INTO acc_trans (trans_id, chart_id, amount, transdate, tax_id, taxkey, project_id, chart_link)
-             VALUES (?, (SELECT id FROM chart WHERE accno = ?), ?, ?, (SELECT id FROM tax WHERE taxkey=0), 0, ?, (SELECT link FROM chart WHERE accno = ?))|;
-      @values = (conv_i($trans_id), $rnd_accno, $rounding, conv_date($form->{invdate}), conv_i($project_id), $rnd_accno);
-      do_query($form, $dbh, $query, @values);
-      $rnd_accno = 0;
-    }
   }
 
   # deduct payment differences from diff
@@ -1252,7 +1237,7 @@ SQL
     return;
   }
 
-  $amount = $form->round_amount( $netamount + $tax, 2, 1);
+  $amount = $netamount + $tax;
 
   # save AR record
   #erweiterung fuer lieferscheinnummer (donumber) 12.02.09 jb
@@ -1897,9 +1882,7 @@ sub retrieve_invoice {
          (SELECT c.accno FROM chart c WHERE d.income_accno_id = c.id)    AS income_accno,
          (SELECT c.accno FROM chart c WHERE d.expense_accno_id = c.id)   AS expense_accno,
          (SELECT c.accno FROM chart c WHERE d.fxgain_accno_id = c.id)    AS fxgain_accno,
-         (SELECT c.accno FROM chart c WHERE d.fxloss_accno_id = c.id)    AS fxloss_accno,
-         (SELECT c.accno FROM chart c WHERE d.rndgain_accno_id = c.id)   AS rndgain_accno,
-         (SELECT c.accno FROM chart c WHERE d.rndloss_accno_id = c.id)   AS rndloss_accno
+         (SELECT c.accno FROM chart c WHERE d.fxloss_accno_id = c.id)    AS fxloss_accno
          ${query_transdate}
        FROM defaults d|;
 
index 61af621..7cd5de1 100644 (file)
--- a/SL/OE.pm
+++ b/SL/OE.pm
@@ -690,7 +690,7 @@ SQL
   my $tax = 0;
   map { $tax += $form->round_amount($taxaccounts{$_}, 2) } keys %taxaccounts;
 
-  $amount = $form->round_amount($netamount + $tax, 2, 1);
+  $amount = $form->round_amount($netamount + $tax, 2);
   $netamount = $form->round_amount($netamount, 2);
 
   if ($form->{currency} eq $form->{defaultcurrency}) {
@@ -956,9 +956,7 @@ sub retrieve {
                      (SELECT c.accno FROM chart c WHERE d.income_accno_id    = c.id) AS income_accno,
                      (SELECT c.accno FROM chart c WHERE d.expense_accno_id   = c.id) AS expense_accno,
                      (SELECT c.accno FROM chart c WHERE d.fxgain_accno_id    = c.id) AS fxgain_accno,
-                     (SELECT c.accno FROM chart c WHERE d.fxloss_accno_id    = c.id) AS fxloss_accno,
-                     (SELECT c.accno FROM chart c WHERE d.rndgain_accno_id   = c.id) AS rndgain_accno,
-                     (SELECT c.accno FROM chart c WHERE d.rndloss_accno_id   = c.id) AS rndloss_accno
+                     (SELECT c.accno FROM chart c WHERE d.fxloss_accno_id    = c.id) AS fxloss_accno
               $query_add
               FROM defaults d|;
   my $ref = selectfirst_hashref_query($form, $dbh, $query);
index 62c645a..09a671e 100644 (file)
@@ -590,7 +590,7 @@ sub delete_account {
   $form->{title} = $locale->text('Delete Account');
 
   foreach my $id (
-    qw(inventory_accno_id income_accno_id expense_accno_id fxgain_accno_id fxloss_accno_id rndgain_accno_id rndloss_accno_id)
+    qw(inventory_accno_id income_accno_id expense_accno_id fxgain_accno_id fxloss_accno_id)
     ) {
     if ($form->{id} == $form->{$id}) {
       $form->error($locale->text('Cannot delete default account!'));
index 37981eb..3a3d433 100644 (file)
@@ -463,11 +463,6 @@ sub form_footer {
     }
   }
 
-  $form->{rounding} = $form->round_amount(
-    $form->round_amount($form->{invtotal}, 2, 1) - $form->round_amount($form->{invtotal}, 2), 2
-  );
-  $form->{invtotal} = $form->round_amount( $form->{invtotal}, 2, 1 );
-
   # follow ups
   if ($form->{id}) {
     $form->{follow_ups}            = FU->follow_ups('trans_id' => $form->{id}) || [];
index 11e220a..86f0298 100644 (file)
@@ -539,6 +539,9 @@ sub form_footer {
               </tr> |;
       }
     }
+
+#    $form->{invsubtotal} = $form->format_amount(\%myconfig, $form->{invsubtotal}, 2, 0); # template does this
+
   } else {
     foreach my $item (split / /, $form->{taxaccounts}) {
       if ($form->{"${item}_base"}) {
@@ -560,10 +563,6 @@ sub form_footer {
     }
   }
 
-  $form->{rounding} = $form->round_amount(
-    $form->round_amount($form->{invtotal}, 2, 1) - $form->round_amount($form->{invtotal}, 2)
-  );
-  $form->{invtotal} = $form->round_amount( $form->{invtotal}, 2, 1);
   $form->{oldinvtotal} = $form->{invtotal};
 
   $TMPL_VAR{ALL_DELIVERY_TERMS} = SL::DB::Manager::DeliveryTerm->get_all_sorted();
index 393f441..796aa9c 100755 (executable)
@@ -2320,9 +2320,6 @@ $self->{texts} = {
   'Risk'                        => 'Risiko',
   'Risk levels'                 => 'Risikograde',
   'Risks'                       => 'Risikograde',
-  'Rounding'                    => 'Rundung',
-  'Rounding Gain'               => 'Rundungserträge',
-  'Rounding Loss'               => 'Rundungsaufwendungen',
   'Row'                         => 'Zeile',
   'Row #1: amount has to be different from zero.' => 'Zeile #1: Der Wert darf nicht 0 sein.',
   'Row number'                  => 'Zeilennummer',
index e371413..bf275f6 100644 (file)
@@ -1787,9 +1787,6 @@ $self->{texts} = {
   'Revenues EU without UStId'   => '',
   'Review of Aging list'        => '',
   'Right'                       => '',
-  'Rounding'                    => '',
-  'Rounding Gain'               => '',
-  'Rounding Loss'               => '',
   'Row #1: amount has to be different from zero.' => '',
   'Row number'                  => '',
   'Row was created from current record' => '',
diff --git a/sql/Pg-upgrade2/defaults_add_rnd_accno_ids.sql b/sql/Pg-upgrade2/defaults_add_rnd_accno_ids.sql
deleted file mode 100644 (file)
index deee948..0000000
+++ /dev/null
@@ -1,6 +0,0 @@
--- @tag: defaults_add_rnd_accno_ids
--- @description: adds new columns 'rndgain_accno_id' and 'rndloss_accno_id' in table defaults, used to book roundings
--- @depends: release_3_1_0
-ALTER TABLE defaults ADD COLUMN rndgain_accno_id Integer;
-ALTER TABLE defaults ADD COLUMN rndloss_accno_id Integer;
-
index 12a284c..2bcaa0d 100644 (file)
@@ -27,8 +27,6 @@ $(function() {
 <input type="hidden" name="expense_accno_id"   value="[% HTML.escape(expense_accno_id) %]">
 <input type="hidden" name="fxgain_accno_id"    value="[% HTML.escape(fxgain_accno_id) %]">
 <input type="hidden" name="fxloss_accno_id"    value="[% HTML.escape(fxloss_accno_id) %]">
-<input type="hidden" name="rndgain_accno_id"   value="[% HTML.escape(rndgain_accno_id) %]">
-<input type="hidden" name="rndloss_accno_id"   value="[% HTML.escape(rndloss_accno_id) %]">
 
 <fieldset>
   <legend>
index bb92517..51c9d2e 100644 (file)
    <td>[% L.chart_picker('defaults.fxloss_accno_id', SELF.defaults.fxloss_accno_id, category='E,A', choose=1, style=style) %]<td>
   </tr>
 
-  <tr>
-   <td align="right">[% LxERP.t8("Rounding Gain") %]</td>
-   <td>[% L.chart_picker('defaults.rndgain_accno_id', SELF.defaults.rndgain_accno_id, category='I,A', choose=1, style=style) %]</td>
-  </tr>
-
-  <tr>
-   <td align="right">[% LxERP.t8("Rounding Loss") %]</td>
-   <td>[% L.chart_picker('defaults.rndloss_accno_id', SELF.defaults.rndloss_accno_id,  category='E,A', choose=1, style=style) %]</td>
-  </tr>
-
   <tr>
    <td align="right">[% LxERP.t8("Current assets account") %]</td>
    <td>[% L.chart_picker('defaults.ar_paid_accno_id', SELF.defaults.ar_paid_accno_id, type='AR_paid', choose=1, style=style) %]<td>
index 7980bca..d91527c 100644 (file)
        </tr>
 [%- END %]
 
-[%- IF rounding %]
-        <tr>
-          <th align='right'>[% 'Rounding' | $T8 %]</th>
-          <td align='right'>[% LxERP.format_amount(rounding, 2) %]</td>
-        </tr>
-[%- END %]
-
 [%# tax %]
 [% FOREACH item = taxaccounts_array %]
 [% SET description_ref = item _ '_description' %]
        </tr>
   [%- END %]
 [%- END %]
+
         <tr>
          <th align="right">[% 'Total' | $T8 %]</th>
          <td align="right">[% LxERP.format_amount(invtotal, 2) %]</td>
index f9a8879..c84b85b 100644 (file)
           <td>
             <input type="hidden" name="fxgain_accno" value="[% fxgain_accno %]">
             <input type="hidden" name="fxloss_accno" value="[% fxloss_accno %]">
-            <input type="hidden" name="rndgain_accno" value="[% rndgain_accno %]">
-            <input type="hidden" name="rndloss_accno" value="[% rndloss_accno %]">
           </td>
         </tr>
 [%- IF show_exchangerate %]
index fbeb529..0956f20 100644 (file)
                 <th align='right'>[% 'Subtotal' | $T8 %]</th>
                 <td align='right'>[% LxERP.format_amount(invsubtotal, 2) %]</td>
               </tr>
-[%- END %]
-[%- IF rounding %]
-              <tr>
-                <th align='right'>[% 'Rounding' | $T8 %]</th>
-                <td align='right'>[% LxERP.format_amount(rounding, 2) %]</td>
-              </tr>
 [%- END %]
               [% tax %]
               <tr>