0021820: Wrong calculation of a journal entry under some circumstancies

diff --git a/src/org/openbravo/erpCommon/ad_forms/Fact.java b/src/org/openbravo/erpCommon/ad_forms/Fact.java
--- a/src/org/openbravo/erpCommon/ad_forms/Fact.java
+++ b/src/org/openbravo/erpCommon/ad_forms/Fact.java
@@ -260,6 +260,9 @@
     log4jFact.debug("createLine - " + m_doc.DocumentNo);
     log4jFact.debug("********************* Fact - createLine - DocumentNo - " + m_doc.DocumentNo
         + " -  m_lines.size() - " + m_lines.size());
+
+    line.roundToCurrencyPrecision();
+
     m_lines.add(line);
     return line;
   } // createLine
diff --git a/src/org/openbravo/erpCommon/ad_forms/FactLine.java b/src/org/openbravo/erpCommon/ad_forms/FactLine.java
--- a/src/org/openbravo/erpCommon/ad_forms/FactLine.java
+++ b/src/org/openbravo/erpCommon/ad_forms/FactLine.java
@@ -799,22 +799,6 @@
         log4jFactLine.debug("FactLine - m_C_WithHolding_ID "
             + ((m_docLine != null) ? m_docLine.m_C_WithHolding_ID : ""));
 
-        // Applies currency precision
-        Currency currency = OBDal.getInstance().get(Currency.class, m_C_Currency_ID);
-        org.openbravo.model.financialmgmt.accounting.coa.AcctSchema schema = OBDal.getInstance()
-            .get(org.openbravo.model.financialmgmt.accounting.coa.AcctSchema.class,
-                m_C_AcctSchema_ID);
-        m_AmtSourceCr = new BigDecimal(m_AmtSourceCr).setScale(
-            currency.getStandardPrecision().intValue(), BigDecimal.ROUND_HALF_EVEN).toString();
-        m_AmtSourceDr = new BigDecimal(m_AmtSourceDr).setScale(
-            currency.getStandardPrecision().intValue(), BigDecimal.ROUND_HALF_EVEN).toString();
-        m_AmtAcctCr = new BigDecimal(m_AmtAcctCr).setScale(
-            schema.getCurrency().getStandardPrecision().intValue(), BigDecimal.ROUND_HALF_EVEN)
-            .toString();
-        m_AmtAcctDr = new BigDecimal(m_AmtAcctDr).setScale(
-            schema.getCurrency().getStandardPrecision().intValue(), BigDecimal.ROUND_HALF_EVEN)
-            .toString();
-
         no = FactLineData.insertFactAct(con, conn, m_Fact_Acct_ID, AD_Client_ID, AD_Org_ID, vars
             .getUser(), m_C_AcctSchema_ID, Account_ID, cuenta[0].value, cuenta[0].description,
             DateDoc, DateAcct, C_Period_ID, m_AD_Table_ID, m_Record_ID, m_Line_ID,
@@ -835,6 +819,23 @@
     return no == 1;
   } // save
 
+  void roundToCurrencyPrecision() {
+    // Applies currency precision
+    Currency currency = OBDal.getInstance().get(Currency.class, m_C_Currency_ID);
+    org.openbravo.model.financialmgmt.accounting.coa.AcctSchema schema = OBDal.getInstance().get(
+        org.openbravo.model.financialmgmt.accounting.coa.AcctSchema.class, m_C_AcctSchema_ID);
+    m_AmtSourceCr = new BigDecimal(m_AmtSourceCr).setScale(
+        currency.getStandardPrecision().intValue(), BigDecimal.ROUND_HALF_EVEN).toString();
+    m_AmtSourceDr = new BigDecimal(m_AmtSourceDr).setScale(
+        currency.getStandardPrecision().intValue(), BigDecimal.ROUND_HALF_EVEN).toString();
+    m_AmtAcctCr = new BigDecimal(m_AmtAcctCr).setScale(
+        schema.getCurrency().getStandardPrecision().intValue(), BigDecimal.ROUND_HALF_EVEN)
+        .toString();
+    m_AmtAcctDr = new BigDecimal(m_AmtAcctDr).setScale(
+        schema.getCurrency().getStandardPrecision().intValue(), BigDecimal.ROUND_HALF_EVEN)
+        .toString();
+  }
+
   /**
    * Get AD_Client
    *