Author: fdesbois Date: 2009-12-14 11:05:18 +0100 (Mon, 14 Dec 2009) New Revision: 2820 Modified: trunk/pollen-business/src/main/java/org/chorem/pollen/business/services/ServiceCommentImpl.java trunk/pollen-business/src/main/java/org/chorem/pollen/business/services/ServiceListImpl.java trunk/pollen-business/src/main/java/org/chorem/pollen/business/services/ServicePollImpl.java trunk/pollen-business/src/main/java/org/chorem/pollen/business/services/ServiceResultsImpl.java Log: Add FIXME for Refactor calling Service from an other one (transaction problem) Modified: trunk/pollen-business/src/main/java/org/chorem/pollen/business/services/ServiceCommentImpl.java =================================================================== --- trunk/pollen-business/src/main/java/org/chorem/pollen/business/services/ServiceCommentImpl.java 2009-12-11 15:26:51 UTC (rev 2819) +++ trunk/pollen-business/src/main/java/org/chorem/pollen/business/services/ServiceCommentImpl.java 2009-12-14 10:05:18 UTC (rev 2820) @@ -62,6 +62,7 @@ converter.setTransaction(transaction); converter.populateCommentEntity(comment, commentEntity); + // FIXME do not call a Service from an other one ServicePollAccount spa = new ServicePollAccountImpl(); commentEntity.setPollAccount(spa.createPollAccount(comment .getPollAccountId(), null, null)); Modified: trunk/pollen-business/src/main/java/org/chorem/pollen/business/services/ServiceListImpl.java =================================================================== --- trunk/pollen-business/src/main/java/org/chorem/pollen/business/services/ServiceListImpl.java 2009-12-11 15:26:51 UTC (rev 2819) +++ trunk/pollen-business/src/main/java/org/chorem/pollen/business/services/ServiceListImpl.java 2009-12-14 10:05:18 UTC (rev 2820) @@ -74,6 +74,7 @@ votingListEntity); // Création des pollAccount de la liste si nécessaire + // FIXME do not call a Service from an other one ServicePollAccount spa = new ServicePollAccountImpl(); //spa.createPollAccounts(votingList.getPollAccountDTOs(), transaction); for (PollAccountDTO account : votingList.getPollAccountDTOs()) { @@ -123,6 +124,7 @@ votingListEntity); // Création des pollAccount de la liste si nécessaire + // FIXME do not call a Service from an other one ServicePollAccount spa = new ServicePollAccountImpl(); //spa.createPollAccounts(votingList.getPollAccountDTOs(), trans); for (PollAccountDTO account : votingList.getPollAccountDTOs()) { @@ -314,6 +316,7 @@ personListConverter.populatePersonListEntity(personList, entity); // Création des comptes de la liste + // FIXME do not call a Service from an other one ServicePollAccount spa = new ServicePollAccountImpl(); for (PollAccountDTO account : personList.getPollAccountDTOs()) { account.setPersonListId(entity.getTopiaId()); @@ -352,6 +355,7 @@ PersonList entity = personListDAO.create(); // Création des comptes de la liste + // FIXME do not call a Service from an other one ServicePollAccount spa = new ServicePollAccountImpl(); for (PollAccountDTO account : personList.getPollAccountDTOs()) { account.setPersonListId(entity.getTopiaId()); @@ -387,6 +391,7 @@ personListDAO.update(entity); // mise à jour ou création des comptes + // FIXME do not call a Service from an other one ServicePollAccount spa = new ServicePollAccountImpl(); for (PollAccountDTO pollAccountDTO : personList .getPollAccountDTOs()) { Modified: trunk/pollen-business/src/main/java/org/chorem/pollen/business/services/ServicePollImpl.java =================================================================== --- trunk/pollen-business/src/main/java/org/chorem/pollen/business/services/ServicePollImpl.java 2009-12-11 15:26:51 UTC (rev 2819) +++ trunk/pollen-business/src/main/java/org/chorem/pollen/business/services/ServicePollImpl.java 2009-12-14 10:05:18 UTC (rev 2820) @@ -78,6 +78,7 @@ converter.populatePollEntity(pollDTO, pollEntity); // Création du pollAccount associé au sondage + // FIXME do not call a Service from an other one ServicePollAccount spa = new ServicePollAccountImpl(); PollAccount pollAccountEntity = spa.createPollAccount(pollDTO .getCreatorId(), pollDTO.getCreatorEmail(), pollDTO @@ -93,21 +94,25 @@ topiaId = pollEntity.getTopiaId(); // Création des choix du sondage + // FIXME do not call a Service from an other one ServiceChoice sChoice = new ServiceChoiceImpl(); pollEntity.setChoice(sChoice.createChoices(pollDTO.getChoiceDTOs(), topiaId, transaction)); // Création des commentaires du sondage + // FIXME do not call a Service from an other one ServiceComment sComment = new ServiceCommentImpl(); pollEntity.setComment(sComment.createComments(pollDTO .getCommentDTOs(), topiaId, transaction)); // Création des listes de votants du sondage + // FIXME do not call a Service from an other one ServiceList sList = new ServiceListImpl(); pollEntity.setVotingList(sList.createVotingLists(pollDTO .getVotingListDTOs(), topiaId, transaction)); // Création des règle de notification pour le sondage + // FIXME do not call a Service from an other one ServicePreventRule sPreventRule = new ServicePreventRuleImpl(); pollEntity.setPreventRule(sPreventRule.createPreventRules(pollDTO .getPreventRuleDTOs(), topiaId, transaction)); Modified: trunk/pollen-business/src/main/java/org/chorem/pollen/business/services/ServiceResultsImpl.java =================================================================== --- trunk/pollen-business/src/main/java/org/chorem/pollen/business/services/ServiceResultsImpl.java 2009-12-11 15:26:51 UTC (rev 2819) +++ trunk/pollen-business/src/main/java/org/chorem/pollen/business/services/ServiceResultsImpl.java 2009-12-14 10:05:18 UTC (rev 2820) @@ -264,6 +264,7 @@ PollExportDTO pollExportDTO = serviceExport.executeImport(filePath); // Transformation du sondage + // FIXME do not call a Service from an other one ServicePoll servicePoll = new ServicePollImpl(); org.chorem.pollen.business.dto.PollDTO poll = DataVoteCountingConverter .createPollDTO(pollExportDTO.getPoll());