April 2010 - Lima-devel - mailman.cloud.codelutin.com

Le projet ne builde pas
by Tony Chemit 29 Apr '10

29 Apr '10

Yo, J'arrive pas builder lima. Y'a une dépendance en snpashot sur openejb ? C'est normal ? Je pense pas. Peux-t-on me donner le dépot qui contient la snpashot que je l'ajoute au nexus. Merci -- Tony Chemit -------------------- tél: +33 (0) 2 40 50 29 28 email: chemit(a)codelutin.com http://www.codelutin.com

2 1

Remarques sur r2870
by Florian Desbois 25 Apr '10

25 Apr '10

Bonjour numéro 7, Deux, trois petites remarques sur ton dernier commit : + else { + Identity existIdentity = account.getIdentity(); + // remove account + AccountDAO accountDAO = + LimaCallaoDAOHelper.getAccountDAO(transaction); + accountDAO.delete(account); + //check if account is a subledger with identity + if (existIdentity != null) { + System.out.println(existIdentity.getAddress()); + // delete identity + IdentityDAO identityDAO = + LimaCallaoDAOHelper.getIdentityDAO(transaction); + identityDAO.delete(existIdentity); + } + // commit + transaction.commitTransaction(); Il semble y avoir un problème dans ton modèle. Identity devrait être lié via une composition a Account. Au moment de la suppression de account cela supprimera automatiquement le Identity qui lui est lié. Dans le meme ordre d'idée ce qui suit est inutile : + //Check if the account is a subledger + Identity existIdentity = account.getIdentity(); + if (existIdentity != null) { + // update identity + IdentityDAO identityDAO = + LimaCallaoDAOHelper.getIdentityDAO(transaction); + identityDAO.update(existIdentity); + } l'identity sera automatiquement mis a jour a la sauvegarde de l'Account (pour le vérifier tu pourrais créer un jeu de test avec JUnit) Deuxième chose : + List<FiscalPeriod> listFiscalPeriods = getAllFiscalPeriods(); + ListIterator<FiscalPeriod> itr = listFiscalPeriods.listIterator(); + while(itr.hasNext()){ + FiscalPeriod thisFiscalPeriod = itr.next(); + if (thisFiscalPeriod.getEndDate(). + after(lastFiscalPeriod.getEndDate())){ + lastFiscalPeriod=thisFiscalPeriod; + } + } L'iterateur est inutile ici, il est préférable d'utiliser le foreach de java bien plus simple et lisible : for (FiscalPeriod period : getAllFiscalPeriods()) { if (...) } Les itérateurs sont plutôt utilisés si tu veux faire un add ou remove sur la collection directement dans la boucle. Dernière chose : + int monthNumber = fiscalPeriod.getEndDate().getMonth() + -fiscalPeriod.getBeginDate().getMonth(); + + //check if fiscalperiod have 12 months, ask a confirmation + if(monthNumber!=12){ + //TODO Exception + throw new LimaBusinessException("The fiscal period has not 12 months"); + } Je doute sincèrement que ce code marche. De plus la méthode getMonth() est déprécié et ne devrait pas être utilisé. A moins d'être sûr d'être sur une seule et même année, la soustraction de deux mois ne garantie pas d'avoir le bon nombre réel de mois entre les deux ! (ex : 03/2010, 03/2011 -> beginMonth = 3, endMonth = 3 -> 3 - 3 = 0 au lieu de 12) Tu devrais jeter un oeil sur la librairie nuiton-utils ou autre commons-lang qui te fournissent des méthodes utilitaires sur les dates (DateUtils, PeriodDates, ...) En espérant ne pas t'avoir perdu :P Cordialement, -- Florian Desbois <fdesbois(a)codelutin.com> SSLL Code Lutin Tel: 02 40 50 29 28 http://www.codelutin.com Membre du Réseau Libre-entreprise

4 3

Re: [Lima-devel] [Lima-commits] r2872 - in trunk: lima-business/src/main/java/org/chorem/lima/business lima-business/src/main/java/org/chorem/lima/business/ejb lima-swing/src/main/java/org/chorem/lima/ui/entrybook lima-swing/src/main/java/org/chorem/lima/ui/fiscalperiod lima-swing/src/main/java/org/chorem/lima/ui/fiscalperiod/model lima-swing/src/main/resources/i18n
by Eric Chatellier 23 Apr '10

23 Apr '10

Le 23/04/2010 16:40, jpepin(a)users.chorem.org a écrit : > + //FIXME PJ 23/04/2010 code à changer pour ne pas dépendre de l'impl > + TopiaContextImpl topiaContextImpl = (TopiaContextImpl) transaction; > + topiaContextImpl.getHibernate().evict(oldestUnBlockedFiscalPeriod); > + // update account > + fiscalPeriodDAO.update(fiscalPeriod); > Ce code est de moi :). En fait, ce code essaie de mettre à jour fiscalPeriod (passé en paramètre du service), et oldestUnBlockedFiscalPeriod est le même objet (période) mais associé à la transaction en cours. Par contre, hibernate refuse la mise à jour de fiscalPeriod parce que oldestUnBlockedFiscalPeriod est dans sa session. La manœuvre a consisté a dissocier oldestUnBlockedFiscalPeriod de la session (hibernate hard code donc) et là la mise à jour de fiscalPeriod fonctionne. -- Éric <chatellier(a)codelutin.com> Tel: 02 40 50 29 28 http://www.codelutin.com

1 0

Re: [Lima-devel] [Lima-commits] r2866 - in trunk: lima-business/src/main/java/org/chorem/lima/business lima-business/src/main/java/org/chorem/lima/business/ejb lima-swing/src/main/java/org/chorem/lima/ui/account lima-swing/src/main/java/org/chorem/lima/ui/account/model lima-swing/src/main/resources/i18n
by Eric Chatellier 19 Apr '10

19 Apr '10

Le 19/04/2010 17:15, jpepin(a)users.chorem.org a écrit : > @@ -587,11 +607,12 @@ > * ATTENTION : si il existe une entrée comptable associée au numéro de > * compte, il est alors impossible de supprimer le compte. > * > + * Si un compte contient des sous comptes la fonction retourne -1 > * @param account > * @throws LimaException > */ > @Override > - public void removeAccount(Account account) throws LimaException { > + public int removeAccount(Account account) throws LimaException { > /*String result = ServiceHelper.RESPOND_ERROR; > Account deleteAccount = searchAccount(accountNumber); > // Si le compte n'existe pas > @@ -667,12 +688,12 @@ > } > } > return result;*/ > - > + int result = 0; > TopiaContext transaction = null; > try { > transaction = rootContext.beginTransaction(); > > - // Vérifie si une entrée ne possède pas ce numéro de compte. > + //Check if an account has not his number > // FIXME !IMPORTANT! check that subaccounts have no entries too > EntryDAO entryDAO = LimaCallaoDAOHelper.getEntryDAO(transaction); > Entry firstEntry = entryDAO.findByAccount(account); > @@ -681,12 +702,19 @@ > throw new LimaBusinessException("Can't delete account with entries"); > } > > + //Check if the account is not empty, return -1 > + List<Account> existingSubAccounts = getChildrenAccounts(account); > + if (CollectionUtils.isNotEmpty(existingSubAccounts)){ > + result =-1; > + } > + else { > // remove account > - // FIXME !IMPORTANT! check that subaccounts are also deleted > - AccountDAO accountDAO = LimaCallaoDAOHelper.getAccountDAO(transaction); > + AccountDAO accountDAO = > + LimaCallaoDAOHelper.getAccountDAO(transaction); > accountDAO.delete(account); > // commit > transaction.commitTransaction(); > + } > } > catch (TopiaException ex) { > doCatch(transaction, ex, log); > @@ -694,7 +722,38 @@ > finally { > doFinally(transaction, log); > } > + return result Il veut mieux retourner une exception dans ce cas plutôt que -1 -- Éric <chatellier(a)codelutin.com> Tel: 02 40 50 29 28 http://www.codelutin.com

2 1

Roadmap modification lima
by Eric Chatellier 08 Apr '10

08 Apr '10

Module ------ Proposition : - lima-persistence (persistence générées et moteur par défaut) - lima-business (dao, ejb) - lima-swing (ui swing) - lima-ofbiz (je vais pas le lié au build dans un premier temps) Mais il n'y a plus de trace de callao :( Callao = les impl dans lima-persistence + les impl dans lima-business ? Roadmap 1 --------- - amélioration du modèle (renommage de certaines entité "Log") - merge des deux projets - suppression des DTO - remplacement des codes d'échec réussite par des exceptions - conservation des UI actuelle - conservation des codes métier actuel => Estimation de temps : 6J Roadmap 2 --------- - développement métier - utilisation des DOA Topia - pre version fonctionnelle niveau métier => Estimation de temps : 5J Roadmap 3 --------- - amélioration des ui - pre version fonctionnelle globale => Estimation de temps : 15J Roadmap 4 --------- - modification de topia pour générer les interface des DAO - remplacement de l'utilisation des DOA Topia par des DOA génériques - ajout de JTA ( + EJB ?) => Estimation de temps : 8J (partie très technique) -- Éric <chatellier(a)codelutin.com> Tel: 02 40 50 29 28 http://www.codelutin.com

2 3