Author: tchemit Date: 2012-03-07 17:57:17 +0100 (Wed, 07 Mar 2012) New Revision: 3168 Url: http://chorem.org/repositories/revision/pollen/3168 Log: prefer commit inside service than outside Modified: branches/pollen-1.2.6-struts2/pollen-services/src/main/java/org/chorem/pollen/services/impl/PollCommentService.java branches/pollen-1.2.6-struts2/pollen-services/src/main/java/org/chorem/pollen/services/impl/PollService.java Modified: branches/pollen-1.2.6-struts2/pollen-services/src/main/java/org/chorem/pollen/services/impl/PollCommentService.java =================================================================== --- branches/pollen-1.2.6-struts2/pollen-services/src/main/java/org/chorem/pollen/services/impl/PollCommentService.java 2012-03-07 16:56:43 UTC (rev 3167) +++ branches/pollen-1.2.6-struts2/pollen-services/src/main/java/org/chorem/pollen/services/impl/PollCommentService.java 2012-03-07 16:57:17 UTC (rev 3168) @@ -76,7 +76,7 @@ // add it to poll pollToUpdate.addComment(commentCreated); - commitTransaction("Could not create comment"); + commitTransaction("Could not create comment "+comment.getText()); // feed notification @@ -108,6 +108,9 @@ //FIXME Should we also delete the associated pollAccount ? dao.delete(comment); + + commitTransaction("Could not delete comment "+comment.getText()); + } catch (TopiaException e) { throw new PollenTechnicalException(e); } Modified: branches/pollen-1.2.6-struts2/pollen-services/src/main/java/org/chorem/pollen/services/impl/PollService.java =================================================================== --- branches/pollen-1.2.6-struts2/pollen-services/src/main/java/org/chorem/pollen/services/impl/PollService.java 2012-03-07 16:56:43 UTC (rev 3167) +++ branches/pollen-1.2.6-struts2/pollen-services/src/main/java/org/chorem/pollen/services/impl/PollService.java 2012-03-07 16:57:17 UTC (rev 3168) @@ -559,6 +559,8 @@ VoteDAO dao = getDAO(Vote.class); delete(dao, vote); + + commitTransaction("Could not delete vote " + vote); } public void deleteChoice(String pollId, String choiceId) @@ -577,6 +579,8 @@ ChoiceDAO dao = getDAO(Choice.class); delete(dao, choice); + + commitTransaction("Could not delete choice " + choice.getName()); } public PollAccount getRestrictedAccount(String pollId,