- Pollen-commits - mailman.cloud.codelutin.com

r3446 - trunk/doc
by tchemit＠users.chorem.org June 11, 2012

June 11, 2012

Author: tchemit Date: 2012-06-11 17:55:33 +0200 (Mon, 11 Jun 2012) New Revision: 3446 Url: http://chorem.org/repositories/revision/pollen/3446 Log: add tests roadmap Added: trunk/doc/tests.rst Added: trunk/doc/tests.rst =================================================================== --- trunk/doc/tests.rst (rev 0) +++ trunk/doc/tests.rst 2012-06-11 15:55:33 UTC (rev 3446) @@ -0,0 +1,119 @@ +~~~~~~~~~~ +Tests d'UI +~~~~~~~~~~ + +Abstract +======== + +- Tests Pollen by ui. +- Needs a db with + - 1 admin user *admin/admin* + - 1 none admin user *user/user* + - 1 none admin user *user2/user2* + - 1 poll of each votecounting type (Normal, pourcentage, Condorcet, Nombre) + some votes on it + - 1 poll with restricted mode + +Scenarii +======== + +Security +-------- + +- authentication avec mauvais login/password (KO) +- authentication avec bon login/password (OK) + +- access aux pages connectés en mode déconnecté (KO) +- access aux pages connectés en mode connecté (OK) + +- access aux pages admin en mode déconnecté (KO) +- access aux pages admin avec un user non admin (KO) +- access aux pages admin avec un user admin (OK) + +- access aux pages d'un sondage avec un mauvais pollId (KO) +- access aux pages d'un sondage avec un bon pollId (OK) + +- access aux pages d'un sondage en mode créateur + - en mode admin (pas de accountId dans l'url) : avec un user non admin (KO) + - en mode admin (pas de accountId dans l'url) : avec un admin (OK) + - en mode créateur (avec mauvais accountId dans l'url) (KO) + - en mode créateur (avec bon accountId dans l'url) (OK) + +- access aux résultats d'un sondage (non restreint) (OK) +- access aux résultats d'un sondage restreint en anonyme (KO) +- access aux résultats d'un sondage restreint en extérieur (mauvais accountId) (KO) +- access aux résultats d'un sondage restreint en extérieur (mauvais utilisateur connecté) (KO) +- access aux résultats d'un sondage restreint avec bon accountId (OK) +- access aux résultats d'un sondage restreint avec bon utilisateur connecté (OK) +- access aux résultats d'un sondage restreint avec accountId de créateur (OK) +- access aux résultats d'un sondage restreint avec un admin connecté (OK) + +Création d'un sondage (free + text) +----------------------------------- + +- creation avec champs obligatoires remplis (OK) +- creation avec Titre non rempli (KO) +- creation avec aucun choix proposé (KO) +- creation avec deux choix identiques (KO) +- creation avec Date de fin < Date de début (KO) +- creation avec Date de fin antérieur à maintenant (KO) (TODO) +- creation avec Ajout de choix Date de fin < Date de début (KO) +- creation avec Ajout de choix Date de fin antérieur à maintenant (KO) +- creation avec Ajout de choix cochez et remplit + decochez (les dates doivent être supprimées) (OK) (TODO) +- creation avec email invalide (KO) +- creation avec heure avant la fin de sondage <=0 (KO) (TODO) +- TODO Mettre un warning si date de fin proche... (new feature) +- creation avec nombre de choix <=0 (KO) (TODO : pffff quand on veut voter on a un message Le nombre de choix maximal est de -5....) +- TODO Par défaut mettre au moins 1 + +- TODO Quand on déchoche alors il faut desactiver les champs concernés (et pas uniquement les cacher) + +Création d'un sondage (free + date) +----------------------------------- + +- creation avec aucun choix proposé (KO) +- creation avec deux choix identiques (KO) +- creation d'un sondage de Date avec mauvais format (KO) + +Création d'un sondage (free + image) +------------------------------------ + +- creation avec aucun choix proposé (KO) +- creation d'un sondage d'Image avec mauvais mimetype (KO) (TODO) +- creation avec deux images au nom identiques (KO) (TODO) +- TODO Mettre une image avec une croix ROUGE!!! quand pas chargée + +Création d'un sondage (restreint + text) +---------------------------------------- + +- création sans participant (KO) +- création avec deux participants ayant le même nom (KO) +- création avec deux participants ayant le même email (KO) +- création avec un participants ayant juste un email et pas de nom (KO) (TODO : pollenTechincalException) +- création avec un participants ayant juste un nom et pas d'email (KO) +- TODO problème d'enregistrement des plusieurs votants, le dernier est conservé et dupliqué. +- TODO L'email envoyé n'est pas le bon (il manque le token d'account) + la sécurité ne va pas car je ne dois pas avoir accès à ce sondage + +Création d'un sondage (group + text) +---------------------------------------- + +- création sans nom de groupe (KO) +- création avec deux participants ayant le même nom (KO) +- création avec deux participants ayant le même email (KO) +- création avec un participants ayant juste un email et pas de nom (KO) (TODO : pollenTechincalException) +- création avec un participants ayant juste un nom et pas d'email (KO) +- TODO problème d'enregistrement des plusieurs votants, le dernier est conservé et dupliqué. +- TODO Revoir l'enregistrement (le groupe 2 est perdu...) + + +Edition d'un sondage +--------------------- + +- edition avec champs titre vidé (OK) +- edition avec modification date de fin antérieure à maintenant (KO) +- edition avec suppression de tous les choix (KO) + +Page de résumé d'un sondage +--------------------------- + +- TODO supprimer les // dans les urls + Property changes on: trunk/doc/tests.rst ___________________________________________________________________ Added: svn:keywords + Author Date Id Revision HeadURL Added: svn:eol-style + native

1 0

r3445 - trunk
by tchemit＠users.chorem.org June 11, 2012

June 11, 2012

Author: tchemit Date: 2012-06-11 15:32:27 +0200 (Mon, 11 Jun 2012) New Revision: 3445 Url: http://chorem.org/repositories/revision/pollen/3445 Log: updates to nuiton-utils 2.5 Modified: trunk/pom.xml Modified: trunk/pom.xml =================================================================== --- trunk/pom.xml 2012-06-11 10:27:02 UTC (rev 3444) +++ trunk/pom.xml 2012-06-11 13:32:27 UTC (rev 3445) @@ -183,7 +183,7 @@ <nuitonI18nVersion>2.4.1</nuitonI18nVersion> <nuitonWebVersion>1.11</nuitonWebVersion> - <nuitonUtilsVersion>2.5-SNAPSHOT</nuitonUtilsVersion> + <nuitonUtilsVersion>2.5</nuitonUtilsVersion> <h2Version>1.3.167</h2Version> <postgresqlVersion>9.1-901-1.jdbc4</postgresqlVersion> <struts2Version>2.3.4</struts2Version>

1 0

r3444 - in trunk: . src/site/fr/resources src/site/fr/rst src/site/resources src/site/rst
by tchemit＠users.chorem.org June 11, 2012

June 11, 2012

Author: tchemit Date: 2012-06-11 12:27:02 +0200 (Mon, 11 Jun 2012) New Revision: 3444 Url: http://chorem.org/repositories/revision/pollen/3444 Log: - use last snapshot of nuiton-utils - fix rst warning Removed: trunk/src/site/fr/resources/images/ trunk/src/site/resources/images/ Modified: trunk/pom.xml trunk/src/site/fr/rst/configure.rst trunk/src/site/fr/rst/depouillement.rst trunk/src/site/fr/rst/index.rst trunk/src/site/fr/rst/install.rst.vm trunk/src/site/rst/configure.rst trunk/src/site/rst/index.rst trunk/src/site/rst/methods.rst Modified: trunk/pom.xml =================================================================== --- trunk/pom.xml 2012-06-11 07:05:22 UTC (rev 3443) +++ trunk/pom.xml 2012-06-11 10:27:02 UTC (rev 3444) @@ -183,7 +183,7 @@ <nuitonI18nVersion>2.4.1</nuitonI18nVersion> <nuitonWebVersion>1.11</nuitonWebVersion> - <nuitonUtilsVersion>2.4.9-SNAPSHOT</nuitonUtilsVersion> + <nuitonUtilsVersion>2.5-SNAPSHOT</nuitonUtilsVersion> <h2Version>1.3.167</h2Version> <postgresqlVersion>9.1-901-1.jdbc4</postgresqlVersion> <struts2Version>2.3.4</struts2Version> Modified: trunk/src/site/fr/rst/configure.rst =================================================================== --- trunk/src/site/fr/rst/configure.rst 2012-06-11 07:05:22 UTC (rev 3443) +++ trunk/src/site/fr/rst/configure.rst 2012-06-11 10:27:02 UTC (rev 3444) @@ -20,6 +20,7 @@ .. * along with this program. If not, see <http://www.gnu.org/licenses/>. .. * #L% .. - + Configurer Pollen ~~~~~~~~~~~~~~~~~ Modified: trunk/src/site/fr/rst/depouillement.rst =================================================================== --- trunk/src/site/fr/rst/depouillement.rst 2012-06-11 07:05:22 UTC (rev 3443) +++ trunk/src/site/fr/rst/depouillement.rst 2012-06-11 10:27:02 UTC (rev 3444) @@ -20,6 +20,7 @@ .. * along with this program. If not, see <http://www.gnu.org/licenses/>. .. * #L% .. - + Les méthodes de dépouillement de Pollen ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Modified: trunk/src/site/fr/rst/index.rst =================================================================== --- trunk/src/site/fr/rst/index.rst 2012-06-11 07:05:22 UTC (rev 3443) +++ trunk/src/site/fr/rst/index.rst 2012-06-11 10:27:02 UTC (rev 3444) @@ -20,6 +20,7 @@ .. * along with this program. If not, see <http://www.gnu.org/licenses/>. .. * #L% .. - + Bienvenue sur le site de Pollen ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Modified: trunk/src/site/fr/rst/install.rst.vm =================================================================== --- trunk/src/site/fr/rst/install.rst.vm 2012-06-11 07:05:22 UTC (rev 3443) +++ trunk/src/site/fr/rst/install.rst.vm 2012-06-11 10:27:02 UTC (rev 3444) @@ -21,6 +21,7 @@ .. * along with this program. If not, see <http://www.gnu.org/licenses/>. .. * #L% .. - + Installer Pollen ~~~~~~~~~~~~~~~~ Modified: trunk/src/site/rst/configure.rst =================================================================== --- trunk/src/site/rst/configure.rst 2012-06-11 07:05:22 UTC (rev 3443) +++ trunk/src/site/rst/configure.rst 2012-06-11 10:27:02 UTC (rev 3444) @@ -20,6 +20,7 @@ .. * along with this program. If not, see <http://www.gnu.org/licenses/>. .. * #L% .. - + Configure Pollen ~~~~~~~~~~~~~~~~ Modified: trunk/src/site/rst/index.rst =================================================================== --- trunk/src/site/rst/index.rst 2012-06-11 07:05:22 UTC (rev 3443) +++ trunk/src/site/rst/index.rst 2012-06-11 10:27:02 UTC (rev 3444) @@ -20,6 +20,7 @@ .. * along with this program. If not, see <http://www.gnu.org/licenses/>. .. * #L% .. - + Welcome to Pollen website ~~~~~~~~~~~~~~~~~~~~~~~~~ Modified: trunk/src/site/rst/methods.rst =================================================================== --- trunk/src/site/rst/methods.rst 2012-06-11 07:05:22 UTC (rev 3443) +++ trunk/src/site/rst/methods.rst 2012-06-11 10:27:02 UTC (rev 3444) @@ -20,6 +20,7 @@ .. * along with this program. If not, see <http://www.gnu.org/licenses/>. .. * #L% .. - + Pollen's votecounting methods ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

1 0

r3443 - trunk
by tchemit＠users.chorem.org June 11, 2012

June 11, 2012

Author: tchemit Date: 2012-06-11 09:05:22 +0200 (Mon, 11 Jun 2012) New Revision: 3443 Url: http://chorem.org/repositories/revision/pollen/3443 Log: updates to mavenpom 3.3.3 Modified: trunk/pom.xml Modified: trunk/pom.xml =================================================================== --- trunk/pom.xml 2012-06-11 06:47:02 UTC (rev 3442) +++ trunk/pom.xml 2012-06-11 07:05:22 UTC (rev 3443) @@ -12,7 +12,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>mavenpom4redmine</artifactId> - <version>3.3.3-SNAPSHOT</version> + <version>3.3.3</version> </parent> <groupId>org.chorem</groupId>

1 0

r3442 - in trunk: pollen-persistence/src/main/java/org/chorem/pollen/business/persistence pollen-services/src/main/java/org/chorem/pollen/entities/migration pollen-services/src/main/java/org/chorem/pollen/services/impl pollen-ui-struts2/src/main/java/org/chorem/pollen/ui/actions/poll
by tchemit＠users.chorem.org June 11, 2012

June 11, 2012

Author: tchemit Date: 2012-06-11 08:47:02 +0200 (Mon, 11 Jun 2012) New Revision: 3442 Url: http://chorem.org/repositories/revision/pollen/3442 Log: fixes #173: [Vote] Montrer les votes dans l'ordre chronologique inverse Modified: trunk/pollen-persistence/src/main/java/org/chorem/pollen/business/persistence/VoteDAOImpl.java trunk/pollen-services/src/main/java/org/chorem/pollen/entities/migration/PollenMigrationCallbackV1_4.java trunk/pollen-services/src/main/java/org/chorem/pollen/services/impl/VoteService.java trunk/pollen-ui-struts2/src/main/java/org/chorem/pollen/ui/actions/poll/AbstractVoteAction.java Modified: trunk/pollen-persistence/src/main/java/org/chorem/pollen/business/persistence/VoteDAOImpl.java =================================================================== --- trunk/pollen-persistence/src/main/java/org/chorem/pollen/business/persistence/VoteDAOImpl.java 2012-06-10 22:20:02 UTC (rev 3441) +++ trunk/pollen-persistence/src/main/java/org/chorem/pollen/business/persistence/VoteDAOImpl.java 2012-06-11 06:47:02 UTC (rev 3442) @@ -37,6 +37,40 @@ /** Logger. */ private static final Log log = LogFactory.getLog(VoteDAOImpl.class); + public List<E> getAllVotes(Poll poll) throws TopiaException { + + Preconditions.checkNotNull(poll); + List<E> results; + + // No need to load votes for an anonymous poll + if (poll.isAnonymous()) { + + results = Lists.newArrayList(); + } else { + + // FIXME-FD20100309 : change model to suppress link from poll to vote + // It's not necessary to have votes when retrieving the poll, this + // method do this job only when it's needed + +// if (log.isDebugEnabled()) { +// log.debug("Load votes for poll with uid = " + poll.getPollId()); +// log.debug("LIMIT : startIndex = " + startIndex + " _ " + +// "endIndex = " + endIndex); +// } + + // FIXME : refactor this to use TopiaQuery from ToPIA 2.3 + // Order the results by creation date (the last vote done will be + // at the end of the list) + TopiaQuery query = createQuery("e"). + addFrom(Poll.class, "p"). + addWhere("p", TopiaQuery.Op.EQ, poll). + addWhere("e in elements (p." + Poll.PROPERTY_VOTE + ")"). + addOrder("e.topiaCreateDate DESC"); + results = findAllByQuery(query); + } + return results; + } + public List<E> getVotesByPoll(Poll poll, TopiaFilterPagerUtil.FilterPagerBean pager) throws TopiaException { @@ -68,7 +102,7 @@ addFrom(Poll.class, "p"). addWhere("p", TopiaQuery.Op.EQ, poll). addWhere("e in elements (p." + Poll.PROPERTY_VOTE + ")"). - addOrder("topiaCreateDate"); + addOrder("e.topiaCreateDate"); TopiaFilterPagerUtil.addPagerToQuery(query, pager); results = findAllByQuery(query); } @@ -83,7 +117,7 @@ addWhere("p", TopiaQuery.Op.EQ, poll). addWhere("e in elements(p." + Poll.PROPERTY_VOTE + ")"). addWhere("e.pollAccount.votingId", TopiaQuery.Op.EQ, votingId). - addOrder("topiaCreateDate"); + addOrder("e.topiaCreateDate"); boolean result = existByQuery(query); return result; } Modified: trunk/pollen-services/src/main/java/org/chorem/pollen/entities/migration/PollenMigrationCallbackV1_4.java =================================================================== --- trunk/pollen-services/src/main/java/org/chorem/pollen/entities/migration/PollenMigrationCallbackV1_4.java 2012-06-10 22:20:02 UTC (rev 3441) +++ trunk/pollen-services/src/main/java/org/chorem/pollen/entities/migration/PollenMigrationCallbackV1_4.java 2012-06-11 06:47:02 UTC (rev 3442) @@ -94,8 +94,16 @@ // add unique on Poll.pollId and PollAccount.accountId (http://www.chorem.org/issues/583) addUniqueOnPollIds(queries); + + // Change to type 'timestamp' the topiacreatdate to have nice order (for vote and comment) (http://chorem.org/issues/173) + addTimestampOnTopiaCreateDate(queries); } + private void addTimestampOnTopiaCreateDate(List<String> queries) { + queries.add("alter TABLE vote ALTER COLUMN topiacreatedate type timestamp;"); + queries.add("alter TABLE comment ALTER COLUMN topiacreatedate type timestamp;"); + } + private void addUniqueOnPollIds(List<String> queries) { queries.add("CREATE UNIQUE INDEX idx_Pöll_pollId ON poll(pollId);"); Modified: trunk/pollen-services/src/main/java/org/chorem/pollen/services/impl/VoteService.java =================================================================== --- trunk/pollen-services/src/main/java/org/chorem/pollen/services/impl/VoteService.java 2012-06-10 22:20:02 UTC (rev 3441) +++ trunk/pollen-services/src/main/java/org/chorem/pollen/services/impl/VoteService.java 2012-06-11 06:47:02 UTC (rev 3442) @@ -355,6 +355,17 @@ pollFeedService.onVoteDeleted(poll, entityToDelete, voteAccount, reason); } + public List<Vote> getAllVotes(Poll poll) { + try { + VoteDAO dao = getDAO(Vote.class); + List<Vote> results = dao.getAllVotes(poll); + return results; + + } catch (TopiaException e) { + throw new PollenTechnicalException("Could not obtain votes", e); + } + } + public List<Vote> getVotesByPoll(Poll poll, TopiaFilterPagerUtil.FilterPagerBean pager) { try { Modified: trunk/pollen-ui-struts2/src/main/java/org/chorem/pollen/ui/actions/poll/AbstractVoteAction.java =================================================================== --- trunk/pollen-ui-struts2/src/main/java/org/chorem/pollen/ui/actions/poll/AbstractVoteAction.java 2012-06-10 22:20:02 UTC (rev 3441) +++ trunk/pollen-ui-struts2/src/main/java/org/chorem/pollen/ui/actions/poll/AbstractVoteAction.java 2012-06-11 06:47:02 UTC (rev 3442) @@ -43,7 +43,6 @@ import org.chorem.pollen.services.exceptions.PollNotFoundException; import org.chorem.pollen.ui.actions.PageSkin; -import java.util.Collections; import java.util.Date; import java.util.List; import java.util.Map; @@ -301,11 +300,8 @@ // All votes // TODO no pagination for the moment, need to retrieve the correct page depends on current pollAccount - votes = poll.getVote(); + votes = getVoteService().getAllVotes(poll); - // votes are loaded by topiaCreateDate but we want the reverse order. - Collections.reverse(votes); - voteAllowed = getVoteService().isVoteAllowed(poll, pollAccount); // Current vote

1 0

r3441 - trunk/pollen-ui-struts2/src/main/java/org/chorem/pollen/ui/actions/poll
by tchemit＠users.chorem.org June 10, 2012

June 10, 2012

Author: tchemit Date: 2012-06-11 00:20:02 +0200 (Mon, 11 Jun 2012) New Revision: 3441 Url: http://chorem.org/repositories/revision/pollen/3441 Log: fixes #173: [Vote] Montrer les votes dans l'ordre chronologique inverse Modified: trunk/pollen-ui-struts2/src/main/java/org/chorem/pollen/ui/actions/poll/AbstractVoteAction.java Modified: trunk/pollen-ui-struts2/src/main/java/org/chorem/pollen/ui/actions/poll/AbstractVoteAction.java =================================================================== --- trunk/pollen-ui-struts2/src/main/java/org/chorem/pollen/ui/actions/poll/AbstractVoteAction.java 2012-06-10 22:19:42 UTC (rev 3440) +++ trunk/pollen-ui-struts2/src/main/java/org/chorem/pollen/ui/actions/poll/AbstractVoteAction.java 2012-06-10 22:20:02 UTC (rev 3441) @@ -43,6 +43,7 @@ import org.chorem.pollen.services.exceptions.PollNotFoundException; import org.chorem.pollen.ui.actions.PageSkin; +import java.util.Collections; import java.util.Date; import java.util.List; import java.util.Map; @@ -302,6 +303,9 @@ // TODO no pagination for the moment, need to retrieve the correct page depends on current pollAccount votes = poll.getVote(); + // votes are loaded by topiaCreateDate but we want the reverse order. + Collections.reverse(votes); + voteAllowed = getVoteService().isVoteAllowed(poll, pollAccount); // Current vote

1 0

r3440 - trunk/pollen-services/src/main/java/org/chorem/pollen/services/impl
by tchemit＠users.chorem.org June 10, 2012

June 10, 2012

Author: tchemit Date: 2012-06-11 00:19:42 +0200 (Mon, 11 Jun 2012) New Revision: 3440 Url: http://chorem.org/repositories/revision/pollen/3440 Log: refs #591: Probl?\195?\168me de vote Modified: trunk/pollen-services/src/main/java/org/chorem/pollen/services/impl/VoteService.java Modified: trunk/pollen-services/src/main/java/org/chorem/pollen/services/impl/VoteService.java =================================================================== --- trunk/pollen-services/src/main/java/org/chorem/pollen/services/impl/VoteService.java 2012-06-10 22:06:51 UTC (rev 3439) +++ trunk/pollen-services/src/main/java/org/chorem/pollen/services/impl/VoteService.java 2012-06-10 22:19:42 UTC (rev 3440) @@ -161,7 +161,6 @@ PollAccountDAO pollAccountDAO = getDAO(PollAccount.class); pollAccountLoaded = createWithProperties( pollAccountDAO, PollAccount.PROPERTY_ACCOUNT_ID, generateId()); - pollAccountLoaded.setAccountId(pollAccount.getAccountId()); } else { // Load existing account (restricted poll)

1 0

r3439 - in trunk: pollen-services/src/main/java/org/chorem/pollen/services/impl pollen-ui-struts2/src/main/java/org/chorem/pollen/ui/actions/poll
by tchemit＠users.chorem.org June 10, 2012

June 10, 2012

Author: tchemit Date: 2012-06-11 00:06:51 +0200 (Mon, 11 Jun 2012) New Revision: 3439 Url: http://chorem.org/repositories/revision/pollen/3439 Log: fixes #591: Probl?\195?\168me de vote Modified: trunk/pollen-services/src/main/java/org/chorem/pollen/services/impl/PollUrlService.java trunk/pollen-services/src/main/java/org/chorem/pollen/services/impl/VoteService.java trunk/pollen-ui-struts2/src/main/java/org/chorem/pollen/ui/actions/poll/VoteForPoll.java Modified: trunk/pollen-services/src/main/java/org/chorem/pollen/services/impl/PollUrlService.java =================================================================== --- trunk/pollen-services/src/main/java/org/chorem/pollen/services/impl/PollUrlService.java 2012-06-10 22:06:02 UTC (rev 3438) +++ trunk/pollen-services/src/main/java/org/chorem/pollen/services/impl/PollUrlService.java 2012-06-10 22:06:51 UTC (rev 3439) @@ -26,6 +26,7 @@ import org.chorem.pollen.bean.PollUri; import org.chorem.pollen.bean.PollUrl; import org.chorem.pollen.business.persistence.Poll; +import org.chorem.pollen.business.persistence.PollAccount; import org.chorem.pollen.business.persistence.PollActions; import org.chorem.pollen.services.PollenServiceSupport; @@ -62,6 +63,11 @@ return result; } + public PollUrl getPollVoteUrl(Poll poll, PollAccount pollAccount) { + PollUrl result = newPollUri(PollActions.VOTE, poll, pollAccount); + return result; + } + public PollUrl getPollModerateUrl(Poll poll) { PollUrl result = newPollUri(PollActions.MODERATE, poll, true); return result; @@ -79,7 +85,12 @@ } protected PollUrl newPollUri(PollActions action, Poll poll, boolean admin) { + Preconditions.checkNotNull(poll); + return newPollUri(action, poll, admin ? poll.getCreator() : null); + } + protected PollUrl newPollUri(PollActions action, Poll poll, PollAccount pollAccount) { + Preconditions.checkNotNull(poll); Preconditions.checkNotNull(action); @@ -87,8 +98,18 @@ URL applicationUrl = serviceContext.getApplicationURL(); String baseUrl = applicationUrl + prefix; - PollUri pollUri = PollUri.newPollUri( - admin ? poll.getAdminId() : poll.getPollId()); + PollUri pollUri; + if (pollAccount == null) { + + // no account id + pollUri = PollUri.newPollUri(poll.getPollId()); + + } else { + + // add account id + pollUri = PollUri.newPollUri(poll.getPollId(), + pollAccount.getAccountId()); + } PollUrl result = PollUrl.newPollUrl(baseUrl, pollUri); return result; } Modified: trunk/pollen-services/src/main/java/org/chorem/pollen/services/impl/VoteService.java =================================================================== --- trunk/pollen-services/src/main/java/org/chorem/pollen/services/impl/VoteService.java 2012-06-10 22:06:02 UTC (rev 3438) +++ trunk/pollen-services/src/main/java/org/chorem/pollen/services/impl/VoteService.java 2012-06-10 22:06:51 UTC (rev 3439) @@ -85,11 +85,15 @@ Preconditions.checkNotNull(accountEditable); VoteDAO dao = getDAO(Vote.class); - Vote voteLoaded; - try { - voteLoaded = dao.findByAccountId(accountEditable.getAccountId()); - } catch (TopiaException e) { - throw new PollenTechnicalException(e); + Vote voteLoaded = null; + if (accountEditable.getAccountId() != null) { + // if pollAccount has no accoutId, so not yet voted, no need to + // seek for something is db. + try { + voteLoaded = dao.findByAccountId(accountEditable.getAccountId()); + } catch (TopiaException e) { + throw new PollenTechnicalException(e); + } } Vote result; @@ -155,7 +159,8 @@ // Create new account PollAccountDAO pollAccountDAO = getDAO(PollAccount.class); - pollAccountLoaded = create(pollAccountDAO); + pollAccountLoaded = createWithProperties( + pollAccountDAO, PollAccount.PROPERTY_ACCOUNT_ID, generateId()); pollAccountLoaded.setAccountId(pollAccount.getAccountId()); } else { Modified: trunk/pollen-ui-struts2/src/main/java/org/chorem/pollen/ui/actions/poll/VoteForPoll.java =================================================================== --- trunk/pollen-ui-struts2/src/main/java/org/chorem/pollen/ui/actions/poll/VoteForPoll.java 2012-06-10 22:06:02 UTC (rev 3438) +++ trunk/pollen-ui-struts2/src/main/java/org/chorem/pollen/ui/actions/poll/VoteForPoll.java 2012-06-10 22:06:51 UTC (rev 3439) @@ -77,7 +77,7 @@ if (pollAccount.getTopiaId() == null && getVoteService().hasAlreadyVoted(poll, name)) { addFieldError("pollAccount.votingId", - _("pollen.error.user.alreadyVoted")); + _("pollen.error.user.alreadyVoted", name)); } } @@ -137,26 +137,31 @@ VoteService voteService = getVoteService(); + Vote newVote; if (StringUtils.isBlank(vote.getTopiaId())) { // create a new vote - voteService.createVote(poll, vote); + newVote = voteService.createVote(poll, vote); } else { // update existing vote - voteService.updateVote(poll, vote); + newVote = voteService.updateVote(poll, vote); } //TODO tchemit-2012-05-18 Why clean messages and just messages here ? clearFlashMessages(); - // For free Poll, display the update Url (useless if user is logged) - if (PollType.FREE == poll.getPollType() && !isUserLoggued()) { + // For free Poll, display the update Url (useless if user is logged or + // not using a modify url) + if (PollType.FREE == poll.getPollType() && + !isUserLoggued() && + StringUtils.isBlank(getAccountId())) { String pollId = poll.getPollId(); - String accountId = getPollAccount().getAccountId(); + String accountId = newVote.getPollAccount().getAccountId(); - PollUrl updateUrl = getPollUrlService().getPollModerateUrl(poll); + PollUrl updateUrl = getPollUrlService().getPollVoteUrl( + poll, newVote.getPollAccount()); if (log.isDebugEnabled()) { log.debug(String.format(

1 0

r3438 - in trunk/pollen-services/src/main/java/org/chorem/pollen/services: . impl
by tchemit＠users.chorem.org June 10, 2012

June 10, 2012

Author: tchemit Date: 2012-06-11 00:06:02 +0200 (Mon, 11 Jun 2012) New Revision: 3438 Url: http://chorem.org/repositories/revision/pollen/3438 Log: move generateId method to abstract service Modified: trunk/pollen-services/src/main/java/org/chorem/pollen/services/PollenServiceSupport.java trunk/pollen-services/src/main/java/org/chorem/pollen/services/impl/PollService.java Modified: trunk/pollen-services/src/main/java/org/chorem/pollen/services/PollenServiceSupport.java =================================================================== --- trunk/pollen-services/src/main/java/org/chorem/pollen/services/PollenServiceSupport.java 2012-06-10 21:18:17 UTC (rev 3437) +++ trunk/pollen-services/src/main/java/org/chorem/pollen/services/PollenServiceSupport.java 2012-06-10 22:06:02 UTC (rev 3438) @@ -26,6 +26,8 @@ import org.apache.commons.lang3.StringUtils; import org.chorem.pollen.PollenConfiguration; import org.chorem.pollen.PollenTechnicalException; +import org.chorem.pollen.business.persistence.Poll; +import org.chorem.pollen.business.persistence.PollAccount; import org.chorem.pollen.entities.PollenDAOHelper; import org.nuiton.topia.TopiaContext; import org.nuiton.topia.TopiaException; @@ -37,6 +39,7 @@ import java.util.Date; import java.util.List; import java.util.Locale; +import java.util.UUID; import static org.nuiton.i18n.I18n.l_; @@ -192,4 +195,15 @@ tx.getHibernate().flush(); } + /** + * Generates a unique id usable for {@link Poll#getPollId()} or + * {@link PollAccount#getAccountId()}. + * + * @return a fresh generated id + * @since 1.4 + */ + protected String generateId() { + return UUID.randomUUID().toString().replaceAll("-", ""); + } + } Modified: trunk/pollen-services/src/main/java/org/chorem/pollen/services/impl/PollService.java =================================================================== --- trunk/pollen-services/src/main/java/org/chorem/pollen/services/impl/PollService.java 2012-06-10 21:18:17 UTC (rev 3437) +++ trunk/pollen-services/src/main/java/org/chorem/pollen/services/impl/PollService.java 2012-06-10 22:06:02 UTC (rev 3438) @@ -1107,15 +1107,4 @@ + thumbCopied.length() + ")"); } } - - /** - * Generates a unique id usable for {@link Poll#getPollId()} or - * {@link PollAccount#getAccountId()}. - * - * @return a fresh generated id - * @since 1.4 - */ - public String generateId() { - return UUID.randomUUID().toString().replaceAll("-", ""); - } }

1 0

r3437 - trunk/pollen-ui-struts2/src/main/java/org/chorem/pollen/ui/actions/poll
by tchemit＠users.chorem.org June 10, 2012

June 10, 2012

Author: tchemit Date: 2012-06-10 23:18:17 +0200 (Sun, 10 Jun 2012) New Revision: 3437 Url: http://chorem.org/repositories/revision/pollen/3437 Log: refs #588: Can not vote for normal poll type Modified: trunk/pollen-ui-struts2/src/main/java/org/chorem/pollen/ui/actions/poll/VoteForPoll.java Modified: trunk/pollen-ui-struts2/src/main/java/org/chorem/pollen/ui/actions/poll/VoteForPoll.java =================================================================== --- trunk/pollen-ui-struts2/src/main/java/org/chorem/pollen/ui/actions/poll/VoteForPoll.java 2012-06-10 21:07:58 UTC (rev 3436) +++ trunk/pollen-ui-struts2/src/main/java/org/chorem/pollen/ui/actions/poll/VoteForPoll.java 2012-06-10 21:18:17 UTC (rev 3437) @@ -137,7 +137,7 @@ VoteService voteService = getVoteService(); - if (StringUtils.isNotBlank(vote.getTopiaId())) { + if (StringUtils.isBlank(vote.getTopiaId())) { // create a new vote voteService.createVote(poll, vote);

1 0