Fixed small typo mistakes and comments.
Fixed Import Bank statement. Not needed "i." in SET clause.
This commit is contained in:
parent
7619ded269
commit
1040f591ce
|
|
@ -28,22 +28,22 @@ import org.compiere.impexp.BankStatementLoaderInterface;
|
||||||
* bank statement loader class to use.
|
* bank statement loader class to use.
|
||||||
* It also inserts the data into the I_BankStatement table.
|
* It also inserts the data into the I_BankStatement table.
|
||||||
*
|
*
|
||||||
* The loader objects can acces their configuration (e.g. file, URL,
|
* The loader objects can access their configuration (e.g. file, URL,
|
||||||
* password etc) using the corresponding get methods provided by this class.
|
* password etc) using the corresponding get methods provided by this class.
|
||||||
* As this class is derrived from PO, "Persistent Object" it has acces to
|
* As this class is derived from PO, "Persistent Object" it has access to
|
||||||
* the loader configuration that has been entered for this loader.
|
* the loader configuration that has been entered for this loader.
|
||||||
* How these values are interpreted is the responsibility of the loader.
|
* How these values are interpreted is the responsibility of the loader.
|
||||||
* There are two file names provided, getFileName() and getLocalFileName().
|
* There are two file names provided, getFileName() and getLocalFileName().
|
||||||
* The first one is the file name as entered on the loader configuration tab,
|
* The first one is the file name as entered on the loader configuration tab,
|
||||||
* the second one is the file name parameter from the loader process.
|
* the second one is the file name parameter from the loader process.
|
||||||
* Reccomended behaviour for a file based loader would be to use the file
|
* Recommended behavior for a file based loader would be to use the file
|
||||||
* name from the process parameter if available, and resort to the file anme
|
* name from the process parameter if available, and resort to the file name
|
||||||
* from the loader configuration as a second option.
|
* from the loader configuration as a second option.
|
||||||
*
|
*
|
||||||
* For a HTTP based loader the file name from the loader configuration should
|
* For a HTTP based loader the file name from the loader configuration should
|
||||||
* be used to construct the URL to connect to (combined with the other parameters).
|
* be used to construct the URL to connect to (combined with the other parameters).
|
||||||
* In this scenario the file name from the process parameter can be used to save
|
* In this scenario the file name from the process parameter can be used to save
|
||||||
* the aquired statement data to disk.
|
* the acquired statement data to disk.
|
||||||
*
|
*
|
||||||
* author Maarten Klinker, Eldir Tomassen
|
* author Maarten Klinker, Eldir Tomassen
|
||||||
* @version $Id: MBankStatementLoader.java,v 1.3 2006/07/30 00:51:04 jjanke Exp $
|
* @version $Id: MBankStatementLoader.java,v 1.3 2006/07/30 00:51:04 jjanke Exp $
|
||||||
|
|
@ -150,7 +150,7 @@ import org.compiere.impexp.BankStatementLoaderInterface;
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Start loading Bankstatements
|
* Start loading Bankstatements
|
||||||
* @return true if loading completed succesfully
|
* @return true if loading completed successfully
|
||||||
*/
|
*/
|
||||||
public boolean loadLines()
|
public boolean loadLines()
|
||||||
{
|
{
|
||||||
|
|
@ -188,7 +188,7 @@ import org.compiere.impexp.BankStatementLoaderInterface;
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Load a bank statement into the I_BankStatement table
|
* Load a bank statement into the I_BankStatement table
|
||||||
* @return Statement line was loaded succesfully
|
* @return Statement line was loaded successfully
|
||||||
* This method is called by the BankStatementLoadere whenever a complete
|
* This method is called by the BankStatementLoadere whenever a complete
|
||||||
* statement line has been read.
|
* statement line has been read.
|
||||||
*/
|
*/
|
||||||
|
|
@ -236,7 +236,7 @@ import org.compiere.impexp.BankStatementLoaderInterface;
|
||||||
log.config( "MBankStatementLoader.importLine Reference No=" + m_loader.getReference());
|
log.config( "MBankStatementLoader.importLine Reference No=" + m_loader.getReference());
|
||||||
imp.setReferenceNo(m_loader.getReference());
|
imp.setReferenceNo(m_loader.getReference());
|
||||||
imp.setEftReference(m_loader.getReference());
|
imp.setEftReference(m_loader.getReference());
|
||||||
log.config( "MBankStatementLoader.importLine Check No=" + m_loader.getReference());
|
log.config( "MBankStatementLoader.importLine Check No=" + m_loader.getCheckNo());
|
||||||
imp.setEftCheckNo(m_loader.getCheckNo());
|
imp.setEftCheckNo(m_loader.getCheckNo());
|
||||||
log.config( "MBankStatementLoader.importLine Memo=" + m_loader.getMemo());
|
log.config( "MBankStatementLoader.importLine Memo=" + m_loader.getMemo());
|
||||||
imp.setMemo(m_loader.getMemo());
|
imp.setMemo(m_loader.getMemo());
|
||||||
|
|
@ -283,9 +283,9 @@ import org.compiere.impexp.BankStatementLoaderInterface;
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Return the most recent error description
|
* Return the most recent error description
|
||||||
* @return Error discription
|
* @return Error description
|
||||||
* This is an additional error description, it can be used to provided
|
* This is an additional error description, it can be used to provided
|
||||||
* descriptive iformation, such as a file name or SQL error, that can not
|
* descriptive information, such as a file name or SQL error, that can not
|
||||||
* be translated by the Adempiere message system.
|
* be translated by the Adempiere message system.
|
||||||
*/
|
*/
|
||||||
public String getErrorDescription()
|
public String getErrorDescription()
|
||||||
|
|
|
||||||
|
|
@ -190,7 +190,7 @@ public class ImportBankStatement extends SvrProcess
|
||||||
log.info("Set Currency=" + no);
|
log.info("Set Currency=" + no);
|
||||||
//
|
//
|
||||||
sql = new StringBuffer("UPDATE I_BankStatement i "
|
sql = new StringBuffer("UPDATE I_BankStatement i "
|
||||||
+ "SET i.C_Currency_ID=(SELECT C_Currency_ID FROM C_BankAccount WHERE C_BankAccount_ID=i.C_BankAccount_ID) "
|
+ "SET C_Currency_ID=(SELECT C_Currency_ID FROM C_BankAccount WHERE C_BankAccount_ID=i.C_BankAccount_ID) "
|
||||||
+ "WHERE i.C_Currency_ID IS NULL "
|
+ "WHERE i.C_Currency_ID IS NULL "
|
||||||
+ "AND i.ISO_Code IS NULL").append(clientCheck);
|
+ "AND i.ISO_Code IS NULL").append(clientCheck);
|
||||||
no = DB.executeUpdate(sql.toString(), get_TrxName());
|
no = DB.executeUpdate(sql.toString(), get_TrxName());
|
||||||
|
|
@ -315,7 +315,7 @@ public class ImportBankStatement extends SvrProcess
|
||||||
+ "WHERE i.I_isImported='N' "
|
+ "WHERE i.I_isImported='N' "
|
||||||
+ "AND s.C_BankStatement_ID=l.C_BankStatement_ID "
|
+ "AND s.C_BankStatement_ID=l.C_BankStatement_ID "
|
||||||
+ "AND i.EftTrxID IS NOT NULL AND "
|
+ "AND i.EftTrxID IS NOT NULL AND "
|
||||||
// Concatinate EFT Info
|
// Concatenate EFT Info
|
||||||
+ "(l.EftTrxID||l.EftAmt||l.EftStatementLineDate||l.EftValutaDate||l.EftTrxType||l.EftCurrency||l.EftReference||s.EftStatementReference "
|
+ "(l.EftTrxID||l.EftAmt||l.EftStatementLineDate||l.EftValutaDate||l.EftTrxType||l.EftCurrency||l.EftReference||s.EftStatementReference "
|
||||||
+ "||l.EftCheckNo||l.EftMemo||l.EftPayee||l.EftPayeeAccount) "
|
+ "||l.EftCheckNo||l.EftMemo||l.EftPayee||l.EftPayeeAccount) "
|
||||||
+ "= "
|
+ "= "
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue