Author: echatellier Date: 2012-06-05 17:54:08 +0200 (Tue, 05 Jun 2012) New Revision: 3444 Url: http://chorem.org/repositories/revision/lima/3444 Log: Fix NPE Modified: trunk/lima-swing/src/main/java/org/chorem/lima/ui/financialtransaction/FinancialTransactionTable.java Modified: trunk/lima-swing/src/main/java/org/chorem/lima/ui/financialtransaction/FinancialTransactionTable.java =================================================================== --- trunk/lima-swing/src/main/java/org/chorem/lima/ui/financialtransaction/FinancialTransactionTable.java 2012-06-05 15:08:46 UTC (rev 3443) +++ trunk/lima-swing/src/main/java/org/chorem/lima/ui/financialtransaction/FinancialTransactionTable.java 2012-06-05 15:54:08 UTC (rev 3444) @@ -25,6 +25,7 @@ package org.chorem.lima.ui.financialtransaction; +import org.apache.commons.collections.CollectionUtils; import org.chorem.lima.entity.Account; import org.chorem.lima.entity.Entry; import org.chorem.lima.entity.EntryBook; @@ -53,8 +54,14 @@ /** * Table des transaction qui ajoute des comportement (keys). + * + * <ul> + * <li>Auto creation des entrees si la transaction est vide (tab) + * <li>positionnement automatique sur les cellules editables + * </ul> * * @author jpepin + * @author echatellier */ public class FinancialTransactionTable extends JXTable { @@ -189,14 +196,13 @@ } } - /** + /* * Touche tab * Incrémente le curseur de case tant que la case n'est pas editable * Ajoute une entrée comptable si tab est sur - * la dernière cellule et si la transaction est non équilibré + * la dernière cellule et si la transaction est non équilibrée * sinon rajoute une transaction */ - if (e.getKeyChar() == KeyEvent.VK_TAB) { int max_x = getColumnCount(); int max_y = getRowCount(); @@ -222,7 +228,7 @@ //if transaction, add entry if (object instanceof FinancialTransaction) { FinancialTransaction financialTransaction = (FinancialTransaction) object; - if (financialTransaction.getEntry().size() == 0) { + if (CollectionUtils.isEmpty(financialTransaction.getEntry())) { handler.addEntry(); setColumnSelectionInterval(1, 1); }