July 2012 - Lima-devel - mailman.cloud.codelutin.com

Re: [Lima-devel] [Lima-commits] r3540 - in trunk/lima-swing/src/main: java/org/chorem/lima/ui/celleditor java/org/chorem/lima/ui/lettering resources resources/i18n
by Eric Chatellier 20 Jul '12

20 Jul '12

> Author: mallon > Date: 2012-07-19 18:07:36 +0200 (Thu, 19 Jul 2012) > New Revision: 3540 > > Url: http://chorem.org/repositories/revision/lima/3540 Le LetteringEditModel pour être un javaBean doit être serializable et donc implementer l'interface java.io.Serializable Committé ici : http://www.chorem.org/projects/lima/repository/revisions/3542 Tu peut maintenant te passer tu code suivant: > @@ -105,21 +115,122 @@ > > @Override > public void propertyChange(PropertyChangeEvent evt) { > - editModel.resetDebitCreditBalance(); > - > view.getLettringAndUnlettringSelectionModel().clearSelection(); > + onBalanceChanged(null); > + view.getLettringSelectionModel().clearSelection(); > updateAllEntries(); > + onButtonModeChanged("all"); > } > }); > > editModel.addPropertyChangeListener(new PropertyChangeListener() > { > - > @Override > public void propertyChange(PropertyChangeEvent evt) { > - updateBalanceAndLetteringButton(); > + > view.getCreditTextField().setText(editModel.getCredit().toString()); > + > view.getDebitTexttField().setText(editModel.getDebit().toString()); > + > view.getSoldeTextField().setText(editModel.getSolde().toString()); > + view.getNoLettered().setEnabled(editModel.isUnLettred()); > + view.getLettered().setEnabled(editModel.isLettred()); > } -- Éric Chatellier

2 2

[CR LIMA] - 18/07/2012
by mallon＠codelutin.com 19 Jul '12

19 Jul '12

Bonjour. Voici le compte-rendu de la réunion dev. du 18/07/2012, concernant le développement de LIMA, et les modifications à apportées durant les deux prochaines semaines : Panneau de lettrage =================== Gestion des modèles ------------------- * Pas de délégation, mais un seul modèle; * Créer un listener écoutant ce modèle, et poussant les infos dans l'edit model; Revoir l'interface ------------------ * Utiliser des panels avec titre et bordure; * Ajout d'un bouton pour les arrondis (Permettant de lettrer des entrées à la base non équilibrée) : + Faire la différence entre le débit et le crédit des deux entrées sélectionnées; + Créer une nouvelle entrée avec cette différence (En debit ou en crédit); * Ajout d'un bouton 'Refresh', afin de mettre à jour les données du tableau, après un lettrage / delettrage; Trigramme --------- * Permettre de saisir plus de 17000 lettrages en ajoutant une lettre lorsque 'Z' est atteint; Panneau de saisie des écritures =============================== * Remettre la sélection du contenu par défaut, au niveau de toutes les cellules de saisie; * Permettre de modifier cette manière de sélectionner, via lima-config; * Remplacer le système de contrôle actuel sur les cellules : si le debit et le crédit sont tous deux à zéro, le signaler en les surlignant en rouge; Panneau de recherche des écritures ================================== corriger les bugs existants; Rapports ======== idem (Notamment celui ne permettant pas de faire de rapports si aucun exercice fermé n'existe); Panneau des transactions non-équilibrées ======================================== idem;

1 0

Revue de code entre la r3506 et la r3534
by Arnaud Thimel 18 Jul '12

18 Jul '12

J'ai pris un peu de temps pour regarder le résultat des derniers commits. Matthieu, je pense que la plupart de ces remarques sont pour toi. N'hésites pas à me demander si tu ne comprends pas certaines remarques et/ou à aller demander à un lutin. Essayes de voir avec Tony ou Éric pour configurer ton IDE afin d'éviter les imports du genre : +import javax.swing.*; +import java.awt.*; "when *touch* enter pressed" ... I'm pretty sure you ment *key*, right ? + if(selectedRow != table.getRowCount()){ + table.setRowSelectionInterval(table.getSelectedRow(), table.getSelectedRow()); + } else { + table.setRowSelectionInterval(table.getSelectedRow()-1, table.getSelectedRow()-1); + } Tu peux économiser au moins 3 table.getSelectedRow() au visuel, et 1 en runtime. Des logs de ce genre devraient plutôt : - être en "debug" ; - être en anglais. + if (log.isInfoEnabled()) { + log.info("Test sur JXDatePicker"); + } Tu as encore des soucis d'indentation. Cf FinancialTransactionTable#editCellAt(int,int,EventObject) Est-ce que la TVA ne devrait pas être dans l'AccountingRule en vigueur ? FinancialTransactionViewHandler#addEntry + //Actual (2012) tva percentage : 19,6% + BigDecimal tvaPercentAdd = new BigDecimal(0.196); Sur une méthode publique, essayes de plus respecter un format de documentation type javadoc : + //to reinitialize model attribute + public void resetAttribute(){ Sur cette même méthode (UnlettringSelectionModel#resetAttribute), je pense que tu as un gros bug, et que tu supprimes une ligne sur 2 (les lignes paires en l'occurrence) : for (int i = 0; i < selectedRows.size(); i ++){ selectedRows.remove(i); } Imagines que tu as une liste de 6 éléments : A B C D E F Tu supprimes l'élément à l'index 0 (A). Tous les éléments alors APRÈS A sont donc "déplacés" vers la gauche. B a alors l'index 0, C=1, D=2, ... B C D E F Tu incrémentes ton index : i=1. Tu supprimes l'élément à l'index 1 (C). B D E F Tu incrémentes ton index : i=2. Tu supprimes l'élément à l'index 2 (E). B D F Arrêt de la boucle car ta as attend le .size() = 2. Tu as donc supprimé tous les éléments pairs de ta liste :) Le plus simple aurait été d'utiliser une méthode pour vider la liste : selectedRows.clear(); -> Même problème dans LetteringSelectionModel#resetAttribute D'ailleurs, je vais peut-être dire une connerie, mais ces 2 classes ont l'air très similaire, est-ce qu'il n'aurait pas été possible de les mutualiser ? Bon courage, Arnaud

4 3

Re: [Lima-devel] [Lima-commits] r3526 - in trunk: . lima-swing lima-swing/src/main/java/org/chorem/lima/ui lima-swing/src/main/java/org/chorem/lima/ui/financialtransaction
by Eric Chatellier 17 Jul '12

17 Jul '12

Le 13/07/2012 18:42, mallon(a)users.chorem.org a écrit : > > + //Creation of xml lima state file > + protected File createFileLimaState(){ > + File limaStateDir = new File("././.lima/reports"); > + File limaState = new File("././.lima/reports/limaState.xml"); > > + return limaState; > + } > + C'est quoi ces chemins étranges ? Ca doit sauvegarder ces fichiers dans le $HOME de l'utilisateur, chemin qui doit être configurable comme tous les autres chemin dans la classe LimaConfig. -- Éric Chatellier <chatellier(a)codelutin.com> Tel: 02.40.50.29.28 http://www.codelutin.com

2 1

Re: [Lima-devel] [Lima-commits] r3496 - in trunk: lima-business/src/main/java/org/chorem/lima/business/ejb lima-business-api/src/main/java/org/chorem/lima/business/api lima-callao/src/main/java/org/chorem/lima/entity lima-swing/src/main/java/org/chorem/lima/ui/common lima-swing/src/main/java/org/chorem/lima/ui/lettering
by Eric Chatellier 06 Jul '12

06 Jul '12

Le 06/07/2012 12:17, mallon(a)users.chorem.org a écrit : > + public List<Entry> getAllEntrieByDatesAndAccountAndLettering(Date beginDate, Date endDate, Account account, String filtreLettre) throws LimaException { > + List<FinancialTransaction> financialTransactions = null; > + List<Entry> entries = new ArrayList<Entry>(); > + > + try{ > + FinancialTransactionDAO transactionDAO = getDaoHelper().getFinancialTransactionDAO(); > + financialTransactions = transactionDAO.findAllByDates(beginDate, endDate); > + }catch (Exception ex){ > + throw new LimaException("Can't get financial transactions", ex); > + } > + > + try { > + EntryDAO entryDAO = getDaoHelper().getEntryDAO(); > + for (FinancialTransaction financialTransaction : financialTransactions){ > + entries.addAll(entryDAO.findAllEntryByAccountAndLettering(financialTransaction, account, filtreLettre)); > + } > + } catch (Exception ex) { > + throw new LimaException("Can't get entries", ex); > + } > + > + if (log.isInfoEnabled()) { > + log.info("Entries size : " + entries.size()); > + } > + > + return entries; > + } > + > + /** C'est mieux, mais c'est toujours pas bon. Tu as plusieurs appels base, alors qu'un seul suffit, dont un dans une boucle "for". Tu devrait avoir seulement quelque chose du genre: entryDAO.findAllEntryByAccountAndLettering(beginDate, endDate, account, filtreLettre); + public List<Entry> findAllEntryByAccountAndLettering(FinancialTransaction financialTransaction, Account account, String filtreLettre) throws TopiaException { + List<Entry> entries = null; + + String query = "from " + Entry.class.getName() + " E " + + " where E.financialTransaction = :financialTransaction" + + " and E.account = :account "; + if (filtreLettre.equals("null")){ + query += " and E.lettering is null"; + }else if (filtreLettre.equals("notNull")){ + query += " and E.lettering is not null"; + } + + entries = context.findAll(query, "financialTransaction", financialTransaction, "account", account); + return entries; + } Et le "filtreLettre" en string et les equals("null") c'est toujours pas bon. -- Éric Chatellier <chatellier(a)codelutin.com> Tel: 02.40.50.29.28 http://www.codelutin.com

1 0

Patch import csv
by Cédric Houbart 05 Jul '12

05 Jul '12

Bonjour J'ai ajouté l'import d'entrées pas csv pour supporter l'import des données de banques (on reçoit notre relevé de banque et on l'injecte dans le journal de banque). J'en ai également profité pour mettre à jour un peu la javadoc des méthodes d'import et quelques nettoyages de code. Le patch est en pièce jointe. Aussi est-ce que lancer les diff par mails est la bonne méthode et est-ce que vous avez une convention de code quelque part et des pratiques de découpage des commits ? :) Notamment pour le code si les commentaires sont en Français ou Anglais et quel indentation pour les instructions sur plusieurs lignes (8 espaces d'indentation, alignement à la parenthèse…), j'ai vu plusieurs méthodes à plusieurs endroits. Cordialement -- Cédric Houbart Développeur Scil 81 bis rue Gantois +33.9.51.77.68.98 http://scil.coop

2 1

Account.thirdParty
by Cédric Houbart 03 Jul '12

03 Jul '12

Bonjour Je suis en train de regarder pour importer les données d'OpenERP dans Lima. Pour l'instant pour les comptes je vois que l'objet dispose d'un attribut thirdParty non utilisé dans les données de démo et qui n'apparaît pas dans l'interface. Ce champ est-il utilisé/plus utilisé/prévu pour un usage futur (la javadoc de AccountAbstract sur ce champ semble d'ailleurs erronée) ? Merci bien -- Cédric Houbart Développeur Scil 81 bis rue Gantois +33.9.51.77.68.98 http://scil.coop

2 1

Re: [Lima-devel] [Lima-commits] r3485 - in trunk: lima-business/src/main/java/org/chorem/lima/business/ejb lima-business-api/src/main/java/org/chorem/lima/business/api lima-callao/src/main/java/org/chorem/lima lima-callao/src/main/java/org/chorem/lima/beans lima-swing/src/main/java/org/chorem/lima/ui/common lima-swing/src/main/java/org/chorem/lima/ui/financialtransaction lima-swing/src/main/java/org/chorem/lima/ui/lettering lima-swing/src/main/resources lima-swing/src/main/resources/i18n
by Eric Chatellier 03 Jul '12

03 Jul '12

Le 03/07/2012 13:54, mallon(a)users.chorem.org a écrit : > - > + > } catch (Exception ex) { > throw new LimaException("Can't get all accounts", ex); > } > - > + > return accountsList; > } J'ai le même soucis. Dans eclipse, il est possible de configurer ca: *Preferences -> Java -> Editor -> Save Actions* and configure it to *remove trailing whitespace source : http://stackoverflow.com/a/1048858/1165234 * -- Éric Chatellier <chatellier(a)codelutin.com> Tel: 02.40.50.29.28 http://www.codelutin.com

1 0