Author: bleny Date: 2013-04-18 15:17:39 +0200 (Thu, 18 Apr 2013) New Revision: 114 Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revisions... Log: allow to remove user affectation: more logs Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/RequestedArticleService.java Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/RequestedArticleService.java =================================================================== --- trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/RequestedArticleService.java 2013-04-18 13:15:08 UTC (rev 113) +++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/RequestedArticleService.java 2013-04-18 13:17:39 UTC (rev 114) @@ -382,18 +382,25 @@ DeliveredRequestedListDao dao = persistenceContext.getDeliveredRequestedListDao(); - DeliveredRequestedList affectedTo = dao.findByAffectedTo(magalieUser); + DeliveredRequestedList deliveredRequestedList = + dao.findByAffectedTo(magalieUser); - if (affectedTo != null && affectedTo.getStatus() == DeliveredRequestedListStatus.AFFECTED) { + if (deliveredRequestedList != null && deliveredRequestedList.getStatus() == DeliveredRequestedListStatus.AFFECTED) { - affectedTo.setAffectedTo(null); + if (log.isInfoEnabled()) { + log.info("user " + magalieUser.getLogin() + + " is no longer affected to list " + + deliveredRequestedList.getRequestedList().getCode()); + } - affectedTo.setStatus(DeliveredRequestedListStatus.PENDING); + deliveredRequestedList.setAffectedTo(null); + deliveredRequestedList.setStatus(DeliveredRequestedListStatus.PENDING); + + persistenceContext.commit(); + } - persistenceContext.commit(); - } }