Author: mallon Date: 2012-06-21 16:38:23 +0200 (Thu, 21 Jun 2012) New Revision: 3469 Url: http://chorem.org/repositories/revision/lima/3469 Log: La suppression d'une transaction ne g?\195?\169n?\195?\168re plus d'erreurs, et les entr?\195?\169es correspondantes sont bien supprim?\195?\169es. Modified: trunk/lima-business/src/main/java/org/chorem/lima/business/ejb/FinancialTransactionServiceImpl.java trunk/lima-callao/src/main/java/org/chorem/lima/entity/EntryDAOImpl.java trunk/lima-callao/src/main/java/org/chorem/lima/entity/FinancialTransactionDAOImpl.java trunk/lima-swing/src/main/java/org/chorem/lima/ui/financialtransaction/FinancialTransactionViewHandler.java Modified: trunk/lima-business/src/main/java/org/chorem/lima/business/ejb/FinancialTransactionServiceImpl.java =================================================================== --- trunk/lima-business/src/main/java/org/chorem/lima/business/ejb/FinancialTransactionServiceImpl.java 2012-06-21 12:49:13 UTC (rev 3468) +++ trunk/lima-business/src/main/java/org/chorem/lima/business/ejb/FinancialTransactionServiceImpl.java 2012-06-21 14:38:23 UTC (rev 3469) @@ -255,15 +255,18 @@ * call accounting rules */ @Override - public void removeFinancialTransaction(FinancialTransaction financialtransaction) throws LimaException { + public void removeFinancialTransaction(FinancialTransaction financialTransaction) throws LimaException { // check if the financial period is blocked AccountingRules accountingRules = LimaConfig.getInstance().getAccountingRules(); - accountingRules.checkFinancialPeriodBlockedWithFinancialTransaction(financialtransaction); + accountingRules.checkFinancialPeriodBlockedWithFinancialTransaction(financialTransaction); try { + FinancialTransactionDAO transactionDAO = getDaoHelper().getFinancialTransactionDAO(); - transactionDAO.delete(financialtransaction); + FinancialTransaction financialTransactionOld = transactionDAO.findByTopiaId(financialTransaction.getTopiaId()); + transactionDAO.delete(financialTransactionOld); + } catch (Exception ex) { throw new LimaException("Can't remove financial transaction", ex); } Modified: trunk/lima-callao/src/main/java/org/chorem/lima/entity/EntryDAOImpl.java =================================================================== --- trunk/lima-callao/src/main/java/org/chorem/lima/entity/EntryDAOImpl.java 2012-06-21 12:49:13 UTC (rev 3468) +++ trunk/lima-callao/src/main/java/org/chorem/lima/entity/EntryDAOImpl.java 2012-06-21 14:38:23 UTC (rev 3469) @@ -163,7 +163,7 @@ "endDate", endDate, "entryBook", entryBook); return entries; } - + /** * Retourne la somme des entrées des transaction entre * deux dates pour un journal donné. Modified: trunk/lima-callao/src/main/java/org/chorem/lima/entity/FinancialTransactionDAOImpl.java =================================================================== --- trunk/lima-callao/src/main/java/org/chorem/lima/entity/FinancialTransactionDAOImpl.java 2012-06-21 12:49:13 UTC (rev 3468) +++ trunk/lima-callao/src/main/java/org/chorem/lima/entity/FinancialTransactionDAOImpl.java 2012-06-21 14:38:23 UTC (rev 3469) @@ -246,7 +246,7 @@ return result; } - + /** * Search financial transaction. * Modified: trunk/lima-swing/src/main/java/org/chorem/lima/ui/financialtransaction/FinancialTransactionViewHandler.java =================================================================== --- trunk/lima-swing/src/main/java/org/chorem/lima/ui/financialtransaction/FinancialTransactionViewHandler.java 2012-06-21 12:49:13 UTC (rev 3468) +++ trunk/lima-swing/src/main/java/org/chorem/lima/ui/financialtransaction/FinancialTransactionViewHandler.java 2012-06-21 14:38:23 UTC (rev 3469) @@ -30,6 +30,7 @@ import java.awt.event.ItemEvent; import java.util.ArrayList; import java.util.Calendar; +import java.util.Collection; import java.util.Collections; import java.util.Date; import java.util.List; @@ -426,8 +427,14 @@ if (response == JOptionPane.YES_OPTION) { if (selectedValue instanceof FinancialTransaction) { - FinancialTransaction currentTransaction = - (FinancialTransaction) selectedValue; + FinancialTransaction currentTransaction = (FinancialTransaction) selectedValue; + + //must delete the entries of the deleted transaction + Collection<Entry> entries = currentTransaction.getEntry(); + for (Entry entry : entries){ + tableModel.deleteRow(tableModel.indexOf(entry)); + } + financialTransactionService.removeFinancialTransaction(currentTransaction); } else if (selectedValue instanceof Entry) {