- Wikitty-commits - mailman.cloud.codelutin.com

r1506 - in trunk: . wikitty-api wikitty-dto wikitty-generators wikitty-hessian-client wikitty-hessian-server wikitty-jdbc wikitty-perf-test wikitty-solr wikitty-struts
by maven-release＠users.nuiton.org 24 Aug '12

24 Aug '12

Author: maven-release Date: 2012-08-24 15:13:30 +0200 (Fri, 24 Aug 2012) New Revision: 1506 Url: http://nuiton.org/repositories/revision/wikitty/1506 Log: [maven-release-plugin] prepare release wikitty-3.8 Modified: trunk/pom.xml trunk/wikitty-api/pom.xml trunk/wikitty-dto/pom.xml trunk/wikitty-generators/pom.xml trunk/wikitty-hessian-client/pom.xml trunk/wikitty-hessian-server/pom.xml trunk/wikitty-jdbc/pom.xml trunk/wikitty-perf-test/pom.xml trunk/wikitty-solr/pom.xml trunk/wikitty-struts/pom.xml Modified: trunk/pom.xml =================================================================== --- trunk/pom.xml 2012-08-24 13:11:02 UTC (rev 1505) +++ trunk/pom.xml 2012-08-24 13:13:30 UTC (rev 1506) @@ -15,7 +15,7 @@ <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.8-SNAPSHOT</version> + <version>3.8</version> <modules> @@ -802,10 +802,10 @@  <scm> - <connection>scm:svn:http://svn.nuiton.org/svn/wikitty/trunk</connection> - <developerConnection>scm:svn:http://svn.nuiton.org/svn/wikitty/trunk + <connection>scm:svn:http://svn.nuiton.org/svn/wikitty/tags/wikitty-3.8</connection> + <developerConnection>scm:svn:http://svn.nuiton.org/svn/wikitty/tags/wikitty-3.8 </developerConnection> - <url>http://www.nuiton.org/repositories/browse/wikitty/trunk</url> + <url>http://www.nuiton.org/repositories/browse/wikitty/tags/wikitty-3.8</url> </scm> <distributionManagement> Modified: trunk/wikitty-api/pom.xml =================================================================== --- trunk/wikitty-api/pom.xml 2012-08-24 13:11:02 UTC (rev 1505) +++ trunk/wikitty-api/pom.xml 2012-08-24 13:13:30 UTC (rev 1506) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.8-SNAPSHOT</version> + <version>3.8</version> </parent>  Modified: trunk/wikitty-dto/pom.xml =================================================================== --- trunk/wikitty-dto/pom.xml 2012-08-24 13:11:02 UTC (rev 1505) +++ trunk/wikitty-dto/pom.xml 2012-08-24 13:13:30 UTC (rev 1506) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.8-SNAPSHOT</version> + <version>3.8</version> </parent>  Modified: trunk/wikitty-generators/pom.xml =================================================================== --- trunk/wikitty-generators/pom.xml 2012-08-24 13:11:02 UTC (rev 1505) +++ trunk/wikitty-generators/pom.xml 2012-08-24 13:13:30 UTC (rev 1506) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.8-SNAPSHOT</version> + <version>3.8</version> </parent>  Modified: trunk/wikitty-hessian-client/pom.xml =================================================================== --- trunk/wikitty-hessian-client/pom.xml 2012-08-24 13:11:02 UTC (rev 1505) +++ trunk/wikitty-hessian-client/pom.xml 2012-08-24 13:13:30 UTC (rev 1506) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.8-SNAPSHOT</version> + <version>3.8</version> </parent>  Modified: trunk/wikitty-hessian-server/pom.xml =================================================================== --- trunk/wikitty-hessian-server/pom.xml 2012-08-24 13:11:02 UTC (rev 1505) +++ trunk/wikitty-hessian-server/pom.xml 2012-08-24 13:13:30 UTC (rev 1506) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.8-SNAPSHOT</version> + <version>3.8</version> </parent>  Modified: trunk/wikitty-jdbc/pom.xml =================================================================== --- trunk/wikitty-jdbc/pom.xml 2012-08-24 13:11:02 UTC (rev 1505) +++ trunk/wikitty-jdbc/pom.xml 2012-08-24 13:13:30 UTC (rev 1506) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.8-SNAPSHOT</version> + <version>3.8</version> </parent> <groupId>org.nuiton.wikitty</groupId> Modified: trunk/wikitty-perf-test/pom.xml =================================================================== --- trunk/wikitty-perf-test/pom.xml 2012-08-24 13:11:02 UTC (rev 1505) +++ trunk/wikitty-perf-test/pom.xml 2012-08-24 13:13:30 UTC (rev 1506) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.8-SNAPSHOT</version> + <version>3.8</version> </parent> <groupId>org.nuiton.wikitty</groupId> Modified: trunk/wikitty-solr/pom.xml =================================================================== --- trunk/wikitty-solr/pom.xml 2012-08-24 13:11:02 UTC (rev 1505) +++ trunk/wikitty-solr/pom.xml 2012-08-24 13:13:30 UTC (rev 1506) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.8-SNAPSHOT</version> + <version>3.8</version> </parent>  Modified: trunk/wikitty-struts/pom.xml =================================================================== --- trunk/wikitty-struts/pom.xml 2012-08-24 13:11:02 UTC (rev 1505) +++ trunk/wikitty-struts/pom.xml 2012-08-24 13:13:30 UTC (rev 1506) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.8-SNAPSHOT</version> + <version>3.8</version> </parent>

1 0

r1505 - in trunk: . wikitty-api/src/main/java/org/nuiton/wikitty/entities
by maven-release＠users.nuiton.org 24 Aug '12

24 Aug '12

Author: maven-release Date: 2012-08-24 15:11:02 +0200 (Fri, 24 Aug 2012) New Revision: 1505 Url: http://nuiton.org/repositories/revision/wikitty/1505 Log: updates to mavenpom 3.3.6 + fix license header Modified: trunk/pom.xml trunk/wikitty-api/src/main/java/org/nuiton/wikitty/entities/WikittyImpl.java Modified: trunk/pom.xml =================================================================== --- trunk/pom.xml 2012-08-23 23:33:41 UTC (rev 1504) +++ trunk/pom.xml 2012-08-24 13:11:02 UTC (rev 1505) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>mavenpom4redmine</artifactId> - <version>3.3.4</version> + <version>3.3.6</version> </parent>  Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/entities/WikittyImpl.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/entities/WikittyImpl.java 2012-08-23 23:33:41 UTC (rev 1504) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/entities/WikittyImpl.java 2012-08-24 13:11:02 UTC (rev 1505) @@ -6,7 +6,7 @@ * $HeadURL$ * %% * Copyright (C) 2009 - 2010 CodeLutin, Benjamin Poussin - * %%gete + * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU Lesser General Public License as * published by the Free Software Foundation, either version 3 of the

1 0

r1504 - trunk/wikitty-solr/src/main/java/org/nuiton/wikitty/storage/solr
by bpoussin＠users.nuiton.org 23 Aug '12

23 Aug '12

Author: bpoussin Date: 2012-08-24 01:33:41 +0200 (Fri, 24 Aug 2012) New Revision: 1504 Url: http://nuiton.org/repositories/revision/wikitty/1504 Log: amelioration du convertisseur, si une sous requete ne retourne pas de resultat, ca ne plante plus Modified: trunk/wikitty-solr/src/main/java/org/nuiton/wikitty/storage/solr/WikittyQueryVisitorToSolr.java Modified: trunk/wikitty-solr/src/main/java/org/nuiton/wikitty/storage/solr/WikittyQueryVisitorToSolr.java =================================================================== --- trunk/wikitty-solr/src/main/java/org/nuiton/wikitty/storage/solr/WikittyQueryVisitorToSolr.java 2012-08-23 11:06:44 UTC (rev 1503) +++ trunk/wikitty-solr/src/main/java/org/nuiton/wikitty/storage/solr/WikittyQueryVisitorToSolr.java 2012-08-23 23:33:41 UTC (rev 1504) @@ -156,17 +156,18 @@ private String escape2solr(String value) { String result; - if (value != null) { + if (StringUtils.isNotEmpty(value)) { final String LUCENE_REPLACE_PATTERN = "\\+" + "|-" + "|&&" + "|\\|" + "|!" + "|\$|\$" + "|\\[|\\]" + "|\\{|\\}" + "|\"" + "|:"; result = value.replaceAll(LUCENE_REPLACE_PATTERN, "\\\\$0"); + if (result.contains(" ")) { + result = "\"" + result + "\""; + } } else { - throw new WikittyException("Parse error, value must be not empty"); +// throw new WikittyException("Parse error, value must be not empty"); + result = "\"\""; } - if (result.contains(" ")) { - result = "\"" + result + "\""; - } return result; }

1 0

r1503 - trunk/wikitty-api/src/main/java/org/nuiton/wikitty/entities
by bpoussin＠users.nuiton.org 23 Aug '12

23 Aug '12

Author: bpoussin Date: 2012-08-23 13:06:44 +0200 (Thu, 23 Aug 2012) New Revision: 1503 Url: http://nuiton.org/repositories/revision/wikitty/1503 Log: ajout de setVersion pour permettre la modif d'une extension programmatiquement Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/entities/WikittyExtension.java Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/entities/WikittyExtension.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/entities/WikittyExtension.java 2012-08-23 09:04:02 UTC (rev 1502) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/entities/WikittyExtension.java 2012-08-23 11:06:44 UTC (rev 1503) @@ -315,6 +315,10 @@ return version; } + public void setVersion(String version) { + this.version = version; + } + public List<String> getRequires() { return requires; }

1 0

r1502 - in trunk/wikitty-api/src/main/java/org/nuiton/wikitty: query services
by bpoussin＠users.nuiton.org 23 Aug '12

23 Aug '12

Author: bpoussin Date: 2012-08-23 11:04:02 +0200 (Thu, 23 Aug 2012) New Revision: 1502 Url: http://nuiton.org/repositories/revision/wikitty/1502 Log: Evolution #1871: Change query in Security service to add condition on user logged for right management (debut d'implantation) Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/query/WikittyQuery.java trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyServiceAuthorisation.java Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/query/WikittyQuery.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/query/WikittyQuery.java 2012-08-23 08:23:10 UTC (rev 1501) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/query/WikittyQuery.java 2012-08-23 09:04:02 UTC (rev 1502) @@ -34,6 +34,7 @@ import org.nuiton.wikitty.query.conditions.Condition; import org.nuiton.wikitty.entities.Element; import org.nuiton.wikitty.entities.ElementField; +import org.nuiton.wikitty.query.conditions.Select; /** * Classe permettant de faire des recherches dans les données. @@ -107,6 +108,15 @@ dangereuse pour les performances. */ protected int wikittyFieldSearchDepth = 0; + /** + * indique si dans les resultats on doit ajouter les id des objets que + * l'utilisateur ne peut pas lire. + * par defaut on ne remonte pas les objets que l'utilisateur ne peut pas lire. + * Si vrai alors on ne remonte pas les objets, si faux et que la requete + * n'est pas un select on remonte les objets non lisible + */ + protected boolean checkAuthorisation = true; + /** create anonymous query */ public WikittyQuery() { } @@ -115,6 +125,27 @@ setCondition(condition); } + /** + * indique si dans les resultats on doit ajouter les id des objets que + * l'utilisateur ne peut pas lire. + * par defaut on ne remonte pas les objets que l'utilisateur ne peut pas lire. + * Si vrai alors on ne remonte pas les objets, si faux et que la requete + * n'est pas un select on remonte les objets non lisible + * + * si checkAuthorisation est: + * <li>true alors filtre actif + * <li>false et requete sans select alors filtre inactif + * <li>false et requete select alors filtre <strong>actif</strong> + */ + public boolean isCheckAuthorisation() { + boolean result = checkAuthorisation || condition instanceof Select; + return result; + } + + public void setCheckAuthorisation(boolean checkAuthorisation) { + this.checkAuthorisation = checkAuthorisation; + } + public WikittyQuery copy() { WikittyQueryVisitorCopy v = new WikittyQueryVisitorCopy(); accept(v); Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyServiceAuthorisation.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyServiceAuthorisation.java 2012-08-23 08:23:10 UTC (rev 1501) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyServiceAuthorisation.java 2012-08-23 09:04:02 UTC (rev 1502) @@ -24,6 +24,11 @@ */ package org.nuiton.wikitty.services; +import java.util.ArrayList; +import java.util.Collection; +import java.util.Collections; +import java.util.List; +import java.util.Set; import org.apache.commons.lang3.StringUtils; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; @@ -43,15 +48,9 @@ import org.nuiton.wikitty.entities.WikittyTreeNode; import org.nuiton.wikitty.query.WikittyQuery; import org.nuiton.wikitty.query.WikittyQueryMaker; +import org.nuiton.wikitty.query.WikittyQueryResult; +import org.nuiton.wikitty.query.WikittyQueryResultTreeNode; -import java.util.ArrayList; -import java.util.Collection; -import java.util.Collections; -import java.util.List; -import java.util.Set; - -import static org.nuiton.i18n.I18n._; - /** * Cette classe gere les autorisations d'acces en utilisant les * WikittyAuthorization, il faut donc que la personne soit loggue et donc @@ -97,16 +96,6 @@ } @Override - public void addWikittyServiceListener(WikittyListener listener, ServiceListenerType type) { - getDelegate().addWikittyServiceListener(listener, type); - } - - @Override - public void removeWikittyServiceListener(WikittyListener listener, ServiceListenerType type) { - getDelegate().addWikittyServiceListener(listener, type); - } - - @Override public WikittyEvent clear(String securityToken) { String userId = getUserId(securityToken); if (isAppAdmin(securityToken, userId)) { @@ -114,7 +103,9 @@ WikittyEvent result = getDelegate().clear(securityToken); return result; } else { - throw new SecurityException(_("user %s can't clear data", userId)); + throw new SecurityException(String.format( + "user %s can't clear data", + userId)); } } @@ -135,7 +126,9 @@ // peut sortir de la boucle break; } else { - throw new SecurityException(_("user %s can't clear data", userId)); + throw new SecurityException(String.format( + "user %s can't replay data", + userId)); } } if (e.getType().contains(WikittyEvent.WikittyEventType.PUT_WIKITTY)) { @@ -190,7 +183,96 @@ return result; } + @Override + public List<Wikitty> restore(String securityToken, List<String> ids) { + String userId = getUserId(securityToken); + List<Wikitty> wikitties = getDelegate().restore(securityToken, ids); + + long start = TimeLog.getTime(); + for (Wikitty wikitty : wikitties) { + if (wikitty != null) { + refuseUnauthorizedRead(securityToken, userId, wikitty); + } + } + timeLog.log(start, "restore"); + return wikitties; + } + /** + * Ajoute dans les queries si demande (WikittyQuery#checkAuthorisation) ou si + * necessaire (WikittyQuery commence par un Select) une condition supplementaire + * qui filtre les objets remontes en fonction de ce qu'a le droit l'utilisateur + * courant. + * @param securityToken + * @param queries + * @return + */ + @Override + public List<WikittyQueryResult<String>> findAllByQuery( + String securityToken, List<WikittyQuery> queries) { + List<WikittyQuery> qs = new ArrayList<WikittyQuery>(queries.size()); + for (WikittyQuery q : queries) { + if (q != null && q.isCheckAuthorisation()) { + q = addAuthorisationCondition(q); + } + qs.add(q); + } + return super.findAllByQuery(securityToken, qs); + } + + /** + * Ajoute dans les queries si demande (WikittyQuery#checkAuthorisation) ou si + * necessaire (WikittyQuery commence par un Select) une condition supplementaire + * qui filtre les objets remontes en fonction de ce qu'a le droit l'utilisateur + * courant. + * @param securityToken + * @param queries + * @return + */ + @Override + public List<String> findByQuery(String securityToken, List<WikittyQuery> queries) { + List<WikittyQuery> qs = new ArrayList<WikittyQuery>(queries.size()); + for (WikittyQuery q : queries) { + if (q != null && q.isCheckAuthorisation()) { + q = addAuthorisationCondition(q); + } + qs.add(q); + } + return super.findByQuery(securityToken, qs); + } + + /** + * Ajoute dans filter (si non null) et si demande (WikittyQuery#checkAuthorisation) + * une condition supplementaire qui filtre les objets remontes en fonction + * de ce qu'a le droit l'utilisateur courant. + * + * @param securityToken + * @param queries + * @return + */ + @Override + public WikittyQueryResultTreeNode<String> findTreeNode( + String securityToken, String wikittyId, int depth, boolean count, WikittyQuery filter) { + if (filter != null && filter.isCheckAuthorisation()) { + filter = addAuthorisationCondition(filter); + } + return super.findTreeNode(securityToken, wikittyId, depth, count, filter); + } + + /** + * Ajoute une condition pour filtre en plus les objets seulement lisible + * par l'utilisateur loggue. + * @param q La requete a modifier + * @return une nouvelle requete avec la clause en plus + */ + protected WikittyQuery addAuthorisationCondition(WikittyQuery q) { + // FIXME poussin 20120823 a implanter, il faut trouver la bonne + // requete en fonction du stockage des droits dans solr. + // il faut donc faire un point sur le stockage des droits + return q; + } + + /** * Indique si on a bien le droit d'enregistrer tout les wikitties de la * collection. Des que pour un wikitty on a pas les droits, une exception * est levee. @@ -225,7 +307,7 @@ boolean canCreate = extensionRights == null || canRead(securityToken, userId, null, extensionRights); if ( ! canCreate ) { - throw new SecurityException(_( + throw new SecurityException(String.format( "user %s can't create instance of extension %s", userId, extensionRights)); } @@ -271,11 +353,12 @@ // Object newValue = wikitty.getFqField(fqFieldDirtyName); // oldVersion.setFqField(fqFieldDirtyName, newValue); // } else { -// throw new SecurityException(_("user %s can't write field %s on wikitty %s", +// throw new SecurityException(String.format("user %s can't write field %s on wikitty %s", // userId, fqFieldDirtyName, wikitty)); // } if (!canChange) { - throw new SecurityException(_("user %s can't write field %s on wikitty %s", + throw new SecurityException(String.format( + "user %s can't write field %s on wikitty %s", userId, fqFieldDirtyName, wikitty)); } } @@ -283,21 +366,6 @@ } } - @Override - public List<Wikitty> restore(String securityToken, List<String> ids) { - String userId = getUserId(securityToken); - List<Wikitty> wikitties = getDelegate().restore(securityToken, ids); - - long start = TimeLog.getTime(); - for (Wikitty wikitty : wikitties) { - if (wikitty != null) { - refuseUnauthorizedRead(securityToken, userId, wikitty); - } - } - timeLog.log(start, "restore"); - return wikitties; - } - /** throw an exception if read is not allowed */ protected void refuseUnauthorizedRead( String securityToken, String userId, @@ -305,7 +373,7 @@ if (wikitty != null) { for (String extensionName : wikitty.getExtensionNames()) { if ( ! canRead(securityToken, userId, extensionName, wikitty)) { - throw new SecurityException(_( + throw new SecurityException(String.format( "user %s can't read extension %s on wikitty %s," + " it may be due to a global policy on the wikitty", userId, extensionName, wikitty)); @@ -416,7 +484,7 @@ if (wikitty != null) { for (String extensionName : wikitty.getExtensionNames()) { if ( ! canWrite(securityToken, userId, extensionName, wikitty)) { - throw new SecurityException(_( + throw new SecurityException(String.format( "user %s doesn't have rights on extension %s on wikitty %s", userId, extensionName, wikitty)); } @@ -485,7 +553,9 @@ // canWrite is true if this user can modify the field for this extension boolean canWrite = canWrite(securityToken, userId, null, extensionAuthorisation); if ( ! canWrite) { - throw new SecurityException(_("user %s don't have write right for extension %s", userId, extension)); + throw new SecurityException(String.format( + "user %s don't have write right for extension %s", + userId, extension)); } } } @@ -552,7 +622,8 @@ // seul les AppAdmin on le droit a cette method getDelegate().syncSearchEngine(securityToken); } else { - throw new SecurityException(_("user %s can't sync search engine", + throw new SecurityException(String.format( + "user %s can't sync search engine", getUserId(securityToken))); } }

1 0

r1501 - trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services
by bpoussin＠users.nuiton.org 23 Aug '12

23 Aug '12

Author: bpoussin Date: 2012-08-23 10:23:10 +0200 (Thu, 23 Aug 2012) New Revision: 1501 Url: http://nuiton.org/repositories/revision/wikitty/1501 Log: ajout d'un FIXME pour la securite Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyEvent.java Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyEvent.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyEvent.java 2012-08-23 08:22:26 UTC (rev 1500) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyEvent.java 2012-08-23 08:23:10 UTC (rev 1501) @@ -56,6 +56,10 @@ * Last update : $Date$ * By : $Author$ */ +// FIXME poussin 20120823 probleme de securte certain event remonte des wikitties +// que les utilisateurs n'auraient potentiellement pas le droit de lire. Il ne faudrait +// jamais remonter des objets mais seulement des ids. Voir l'impact sur le fonctionne +// global avant de faire quoi que se soit public class WikittyEvent extends EventObject { /** serialVersionUID. */

1 0

r1500 - trunk/src/site/rst/user
by bpoussin＠users.nuiton.org 23 Aug '12

23 Aug '12

Author: bpoussin Date: 2012-08-23 10:22:26 +0200 (Thu, 23 Aug 2012) New Revision: 1500 Url: http://nuiton.org/repositories/revision/wikitty/1500 Log: renomage du fichier Added: trunk/src/site/rst/user/generation.rst Removed: trunk/src/site/rst/user/generation.rst.vm Added: trunk/src/site/rst/user/generation.rst =================================================================== --- trunk/src/site/rst/user/generation.rst (rev 0) +++ trunk/src/site/rst/user/generation.rst 2012-08-23 08:22:26 UTC (rev 1500) @@ -0,0 +1,191 @@ +.. - +.. * #%L +.. * Wikitty +.. * +.. * $Id: generation.rst.vm 1442 2012-03-05 19:01:08Z bpoussin $ +.. * $HeadURL: http://svn.nuiton.org/svn/wikitty/trunk/src/site/rst/user/generation.rst.vm $ +.. * %% +.. * Copyright (C) 2009 - 2010 CodeLutin +.. * %% +.. * This program is free software: you can redistribute it and/or modify +.. * it under the terms of the GNU Lesser General Public License as +.. * published by the Free Software Foundation, either version 3 of the +.. * License, or (at your option) any later version. +.. * +.. * This program is distributed in the hope that it will be useful, +.. * but WITHOUT ANY WARRANTY; without even the implied warranty of +.. * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +.. * GNU General Lesser Public License for more details. +.. * +.. * You should have received a copy of the GNU General Lesser Public +.. * License along with this program. If not, see +.. * <http://www.gnu.org/licenses/lgpl-3.0.html>. +.. * #L% +.. - + +====================================== +Générer son modèle métier vers Wikitty +====================================== + +Modélisation +============ + +Vous pouvez modéliser votre modèle métier en utilisant n'importe quel modéliseur +UML pouvant enregistrer le modèle au format xmi (ArgoUML, Topcased,...). Les +entités que vous souhaitez générer doivent avoir le stéréotype *entity*. + +Configuration Maven +=================== + +Pour générer le code, nous utilisons EUGene_ et son plugin Maven. + +Dans le pom de votre projet, rajoutez la définition du plugin:: + + <plugin> + <groupId>org.nuiton.eugene</groupId> + <artifactId>maven-eugene-plugin</artifactId> + <version>${eugeneVersion}/version> + <executions> + <execution> + <id>api generator</id> + <phase>process-sources</phase> + <configuration> + <inputs> + <input>zargo:src/main/xmi/:*.zargo</input> + <input>classpath:model:/:wikitty.objectmodel + </input> + </inputs> + <fullPackagePath>your.app.package</fullPackagePath> + <defaultPackage>your.app.package</defaultPackage> + <extractedPackages> + your.app.package,org.nuiton.wikitty + </extractedPackages> + <generatedPackages>your.app.package</generatedPackages> + <templates> + org.nuiton.wikitty.generator.WikittyMetaGenerator, + org.nuiton.wikitty.generator.WikittyDTOGenerator + </templates> + </configuration> + <goals> + <goal>smart-generate</goal> + </goals> + </execution> + </executions> + </plugin> + +Dans la section *inputs*, on spécifie le(s) modèle(s) métier:: + + zargo:src/main/xmi/:*.zargo + +ainsi que le modèle Wikitty (en cas de dépendance de votre modèle métier vers le +modèle Wikitty). + +Dans la section *templates*, vous pouvez spécifier quoi générer : +*org.nuiton.wikitty.generator.WikittyMetaGenerator* pour la génération Wikitty +standard et *org.nuiton.wikitty.generator.WikittyDTOGenerator* pour la +génération des DTOs si besoin (utilisation d'une UI GWT par exemple). + +Génération +========== + +La génération est effectuée ensuite à chaque fois que vous lancez un build. + +Vous pourriez être étonné que le lien entre vos entités soit représenté par une +chaîne de caractère. En fait, dans vos entité Wikitty, un lien vers un autre +Wikitty est représenté par son identifiant qui est une chaîne de caractères. +Ainsi, un getXXXXX vous retournera l'identifiant d'un Wikitty qu'il faudra +récupérer en passant par le WikittyClient. Vous pouvez alors utiliser le cache +pour limiter les appels côté serveur. + +Subtilités de modélisation +========================== + +Dépendances vers l'API de Wikitty +--------------------------------- + +Il est possible de rajouter une dépendance vers un objet de l'API Wikitty (par +example WikittyUser) en rajoutant simplement la classe dans votre modèle dans le +package *org.nuiton.wikitty.entities*. Elle ne sera pas générée mais interviendra +comme une autre classe de votre modèle (héritage,...). + +.. image:: wikittyDependency.png + +Arbres +------ + +Si vous souhaitez qu'une entité crée un arbre, vous avez juste à faire hériter +cette entité de WikittyTreeNode (pas besoin de référence sur elle-même). De +cette manière, la gestion de l'arbre est assurée par Wikitty, de même que +l'indexation et la recherche. + +.. image:: wikittyTree.png + +.. _EUGene:http://maven-site.nuiton.org/eugene/ + +Tag Value disponible +==================== + +L'ensemble des tags disponible se trouve dans la classe WikittyTagValue. + +:version: s'applique au extension, indique la version de l'extension. Il est + obligatoire de modifier la valeur de la version lorsque vous modifiez la + définition de votre extension. + +:alternativeName: s'applique au champs, ce tag peut etre mis pour qu'a la + generation un des accesseur avec un autre nom soit genere. + +:documentation: s'applicque au extension et au champs. Il indique la + documentation a mettre lors de la generation peut aussi servir de + documentation utilisateur a l'execution. + +:toString: s'applique au extension et indique la facon de representer une + extension textuellement. Par exemple:: + + %Person.lastName$s %Person.firstName$s: %Person.birthday$tm %Person.birthday$te,%Person.birthday$tY + Hello %Person.firstName|unknow$s + +In last example, if firstName field doesn't exist, unknow is used + +:sortOrder: s'applique au extension et indique le tri par defaut pour cette + extension. C'est une liste des champs dans l'importance de l'ordre de trie. + Il est possible d'ajouter asc ou desc apres le champs pour indique qu'il faut + trier en ordre croissant ou decroissant. Par exemple:: + + "Person.lastName asc, Person.firstName, Person.birthday desc" + +:Unique: s'applique au champs de type collection et indique s'il vaut + true que la collection ne peut pas contenir de doublon (Set) si unique + est false alors la collection peut contenir des doublons (List). + +:notNull: s'applique au champs et indique que le champs ne peut pas etre null, + il doit forcement avoir une valeur lors de la sauvegarde. + +:pattern: s'applique au champs et indique pour une String indique que le champs + doit respecter un certain pattern, sinon la sauvegarde echoue. Pour un numeric + ou une date pattern peut-etre utilise comme masque de saisie. Exemple:: + + on"A.*" le champs doit commencer par 'A' + "[0-9]+\.[0-9][0-9]" le nombre doit avoir deux chiffres apres la virgule + +:indexed: s'applique au champs et indique si un champs doit etre indexe ou non. + par defaut si ce tag value n'existe pas il vaut true. Il faut donc le + positionner explicitement a false si on ne veut pas indexer un champs + +:crypt: s'applique au champs et indique que le champs doit etre crypte avant + d'etre sauve et decrypter lors de la restauration, de plus le champs ne + sera pas indexe. Si le cryptage n'est pas possible, la sauvegarde ne se + fera pas (Exeption). Par exemple:: + + crypt=Blowfish:password + +:extensionAllowed: s'applique au champs de type Wikitty et indique que le champs + ne peut prendre comme valeur que des wikitties ayant les extensions specifiees. + Par exemple:: + + Person,Employee;Company + Ici il faut que l'objet ait les extensions Person et Employee ou l'extension Company + +:preload: s'applique au extension et indique les champs de type Wikitty qui + doivent etre preloade lors du chargement de cette extension. Par exemple:: + + preload="Company.employee,Employee.person;Company.address" Deleted: trunk/src/site/rst/user/generation.rst.vm =================================================================== --- trunk/src/site/rst/user/generation.rst.vm 2012-08-23 08:10:35 UTC (rev 1499) +++ trunk/src/site/rst/user/generation.rst.vm 2012-08-23 08:22:26 UTC (rev 1500) @@ -1,188 +0,0 @@ -.. - -.. * #%L -.. * Wikitty -.. * -.. * $Id$ -.. * $HeadURL$ -.. * %% -.. * Copyright (C) 2009 - 2010 CodeLutin -.. * %% -.. * This program is free software: you can redistribute it and/or modify -.. * it under the terms of the GNU Lesser General Public License as -.. * published by the Free Software Foundation, either version 3 of the -.. * License, or (at your option) any later version. -.. * -.. * This program is distributed in the hope that it will be useful, -.. * but WITHOUT ANY WARRANTY; without even the implied warranty of -.. * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the -.. * GNU General Lesser Public License for more details. -.. * -.. * You should have received a copy of the GNU General Lesser Public -.. * License along with this program. If not, see -.. * <http://www.gnu.org/licenses/lgpl-3.0.html>. -.. * #L% -.. - -====================================== -Générer son modèle métier vers Wikitty -====================================== - -Modélisation -============ - -Vous pouvez modéliser votre modèle métier en utilisant n'importe quel modéliseur -UML pouvant enregistrer le modèle au format xmi (ArgoUML, Topcased,...). Les -entités que vous souhaitez générer doivent avoir le stéréotype *entity*. - -Configuration Maven -=================== - -Pour générer le code, nous utilisons EUGene_ et son plugin Maven. - -Dans le pom de votre projet, rajoutez la définition du plugin:: - - <plugin> - <groupId>org.nuiton.eugene</groupId> - <artifactId>maven-eugene-plugin</artifactId> - <version>${eugeneVersion}/version> - <executions> - <execution> - <id>api generator</id> - <phase>process-sources</phase> - <configuration> - <inputs> - <input>zargo:src/main/xmi/:*.zargo</input> - <input>classpath:model:/:wikitty.objectmodel - </input> - </inputs> - <fullPackagePath>your.app.package</fullPackagePath> - <defaultPackage>your.app.package</defaultPackage> - <extractedPackages> - your.app.package,org.nuiton.wikitty - </extractedPackages> - <generatedPackages>your.app.package</generatedPackages> - <templates> - org.nuiton.wikitty.generator.WikittyMetaGenerator, - org.nuiton.wikitty.generator.WikittyDTOGenerator - </templates> - </configuration> - <goals> - <goal>smart-generate</goal> - </goals> - </execution> - </executions> - </plugin> - -Dans la section *inputs*, on spécifie le(s) modèle(s) métier:: - - zargo:src/main/xmi/:*.zargo - -ainsi que le modèle Wikitty (en cas de dépendance de votre modèle métier vers le -modèle Wikitty). - -Dans la section *templates*, vous pouvez spécifier quoi générer : -*org.nuiton.wikitty.generator.WikittyMetaGenerator* pour la génération Wikitty -standard et *org.nuiton.wikitty.generator.WikittyDTOGenerator* pour la -génération des DTOs si besoin (utilisation d'une UI GWT par exemple). - -Génération -========== - -La génération est effectuée ensuite à chaque fois que vous lancez un build. - -Vous pourriez être étonné que le lien entre vos entités soit représenté par une -chaîne de caractère. En fait, dans vos entité Wikitty, un lien vers un autre -Wikitty est représenté par son identifiant qui est une chaîne de caractères. -Ainsi, un getXXXXX vous retournera l'identifiant d'un Wikitty qu'il faudra -récupérer en passant par le WikittyClient. Vous pouvez alors utiliser le cache -pour limiter les appels côté serveur. - -Subtilités de modélisation -========================== - -Dépendances vers l'API de Wikitty ---------------------------------- - -Il est possible de rajouter une dépendance vers un objet de l'API Wikitty (par -example WikittyUser) en rajoutant simplement la classe dans votre modèle dans le -package *org.nuiton.wikitty.entities*. Elle ne sera pas générée mais interviendra -comme une autre classe de votre modèle (héritage,...). - -.. image:: wikittyDependency.png - -Arbres ------- - -Si vous souhaitez qu'une entité crée un arbre, vous avez juste à faire hériter -cette entité de WikittyTreeNode (pas besoin de référence sur elle-même). De -cette manière, la gestion de l'arbre est assurée par Wikitty, de même que -l'indexation et la recherche. - -.. image:: wikittyTree.png - -.. _EUGene::http://maven-site.nuiton.org/eugene/ - -Tag Value disponible -==================== - -L'ensemble des tags disponible se trouve dans la classe WikittyTagValue. - -:version: s'applique au extension, indique la version de l'extension. Il est - obligatoire de modifier la valeur de la version lorsque vous modifiez la - définition de votre extension. - -:alternativeName: s'applique au champs, ce tag peut etre mis pour qu'a la - generation un des accesseur avec un autre nom soit genere. - -:documentation: s'applicque au extension et au champs. Il indique la - documentation a mettre lors de la generation peut aussi servir de - documentation utilisateur a l'execution. - -:toString: s'applique au extension et indique la facon de representer une - extension textuellement. Par exemple:: - - %Person.lastName$s %Person.firstName$s: %Person.birthday$tm %Person.birthday$te,%Person.birthday$tY" - "Hello %Person.firstName|unknow$s" if firstName field doesn't exist, unknow is used - -:sortOrder: s'applique au extension et indique le tri par defaut pour cette - extension. C'est une liste des champs dans l'importance de l'ordre de trie. - Il est possible d'ajouter asc ou desc apres le champs pour indique qu'il faut - trier en ordre croissant ou decroissant. Par exemple:: - - "Person.lastName asc, Person.firstName, Person.birthday desc" - -:Unique: s'applique au champs de type collection et indique s'il vaut - true que la collection ne peut pas contenir de doublon (Set) si unique - est false alors la collection peut contenir des doublons (List). - -:notNull: s'applique au champs et indique que le champs ne peut pas etre null, - il doit forcement avoir une valeur lors de la sauvegarde. - -:pattern: s'applique au champs et indique pour une String indique que le champs - doit respecter un certain pattern, sinon la sauvegarde echoue. Pour un numeric - ou une date pattern peut-etre utilise comme masque de saisie. Exemple:: - - "A.*" le champs doit commencer par 'A' - "[0-9]+\.[0-9][0-9]" le nombre doit avoir deux chiffres apres la virgule - -:indexed: s'applique au champs et indique si un champs doit etre indexe ou non. - par defaut si ce tag value n'existe pas il vaut true. Il faut donc le - positionner explicitement a false si on ne veut pas indexer un champs - -:crypt: s'applique au champs et indique que le champs doit etre crypte avant - d'etre sauve et decrypter lors de la restauration, de plus le champs ne - sera pas indexe. Si le cryptage n'est pas possible, la sauvegarde ne se - fera pas (Exeption). Par exemple:: - - crypt=Blowfish:password - -:extensionAllowed: s'applique au champs de type Wikitty et indique que le champs - ne peut prendre comme valeur que des wikitties ayant les extensions specifiees. - Par exemple:: - - Person,Employee;Company - Ici il faut que l'objet ait les extensions Person et Employee ou l'extension Company - -:preload: s'applique au extension et indique les champs de type Wikitty qui - doivent etre preloade lors du chargement de cette extension. Par exemple:: - - preload="Company.employee,Employee.person;Company.address"

1 0

r1499 - trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services
by bpoussin＠users.nuiton.org 23 Aug '12

23 Aug '12

Author: bpoussin Date: 2012-08-23 10:10:35 +0200 (Thu, 23 Aug 2012) New Revision: 1499 Url: http://nuiton.org/repositories/revision/wikitty/1499 Log: refactoring de la classe Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyServiceAuthentication.java Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyServiceAuthentication.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyServiceAuthentication.java 2012-08-22 06:09:40 UTC (rev 1498) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyServiceAuthentication.java 2012-08-23 08:10:35 UTC (rev 1499) @@ -24,19 +24,15 @@ */ package org.nuiton.wikitty.services; -import java.util.Arrays; import java.util.Collections; -import java.util.Date; +import org.apache.commons.lang3.StringUtils; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.nuiton.util.ApplicationConfig; import org.nuiton.util.TimeLog; import org.nuiton.wikitty.WikittyConfigOption; import org.nuiton.wikitty.WikittyService; -import org.nuiton.wikitty.WikittyUtil; import org.nuiton.wikitty.entities.Wikitty; -import org.nuiton.wikitty.entities.WikittyImpl; -import org.nuiton.wikitty.entities.WikittyTokenHelper; import org.nuiton.wikitty.entities.WikittyUser; import org.nuiton.wikitty.entities.WikittyUserHelper; import org.nuiton.wikitty.query.WikittyQuery; @@ -68,38 +64,64 @@ super(config, ws); if (config != null) { long timeToLogInfo = config.getOptionAsInt(WikittyConfigOption. - WIKITTY_SECURITY_TIME_TO_LOG_INFO.getKey()); + WIKITTY_SERVICE_TIME_TO_LOG_INFO.getKey()); long timeToLogWarn = config.getOptionAsInt(WikittyConfigOption. - WIKITTY_SECURITY_TIME_TO_LOG_WARN.getKey()); + WIKITTY_SERVICE_TIME_TO_LOG_WARN.getKey()); timeLog.setTimeToLogInfo(timeToLogInfo); timeLog.setTimeToLogWarn(timeToLogWarn); } } + /** + * L'exception lever en cas de mauvais login ou mot de passe contient toujours + * le même message pour ne pas aider les attaquants a trouver des comptes + * existant. + * + * @param login le login (ne doit pas etre vide) + * @param password le mot de passe de l'utilisateur + * @return le token de securite a utiliser pour les autres appels + * @throws SecurityException si l'authentification echoue + */ @Override public String login(String login, String password) { long start = TimeLog.getTime(); - String tokenId; + if (StringUtils.isBlank(login)) { + if (log.isDebugEnabled()) { + log.debug(String.format("User try to authenticate with bad blank login: '%s'", login)); + } + throw new SecurityException("bad login or password"); + } + // recherche de l'utilisateur WikittyQuery criteria = new WikittyQueryMaker() .eq(WikittyUser.FQ_FIELD_WIKITTYUSER_LOGIN, login).end(); String userId = getDelegate().findByQuery(null, Collections.singletonList(criteria)).get(0); if (userId == null) { - throw new SecurityException(String.format("no such account '%s'", login)); - } else { - Wikitty user = WikittyServiceEnhanced.restore( - getDelegate(), null, userId); - // check password is valid - if (WikittyUserHelper.getPassword(user).equals(password)) { - tokenId = getToken(user); - } else { - throw new SecurityException("bad login or password"); + if (log.isDebugEnabled()) { + log.debug(String.format("User try to authenticate with bad login: '%s'", login)); } + throw new SecurityException("bad login or password"); } + // on a trouver l'utilisateur on le restore pour verifier le mot de passe + Wikitty user = WikittyServiceEnhanced.restore( + getDelegate(), null, userId); + // check password is valid + if (!StringUtils.equals(WikittyUserHelper.getPassword(user), password)) { + if (log.isDebugEnabled()) { + log.debug(String.format("User '%s' try to authenticate with bad password", login)); + } + throw new SecurityException("bad login or password"); + } + + String tokenId = getToken(user); + if (log.isDebugEnabled()) { + log.debug(String.format("User logged: '%s'", login)); + } + timeLog.log(start, "login"); return tokenId; }

1 0

r1498 - trunk/wikitty-api/src/main/java/org/nuiton/wikitty
by bpoussin＠users.nuiton.org 22 Aug '12

22 Aug '12

Author: bpoussin Date: 2012-08-22 08:09:40 +0200 (Wed, 22 Aug 2012) New Revision: 1498 Url: http://nuiton.org/repositories/revision/wikitty/1498 Log: meilleur parsing des NUMBER (BigDecimal) Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyUtil.java Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyUtil.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyUtil.java 2012-08-15 17:48:49 UTC (rev 1497) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyUtil.java 2012-08-22 06:09:40 UTC (rev 1498) @@ -48,6 +48,8 @@ import java.net.InetAddress; import java.net.NetworkInterface; import java.security.Key; +import java.text.DecimalFormat; +import java.text.NumberFormat; import java.text.ParseException; import java.text.SimpleDateFormat; import java.util.ArrayList; @@ -141,6 +143,9 @@ "yyyy-MM-dd", "yyyy-MM-dd hh:mm", }; + + /** no break space, used to parse BigDecimal */ + static final public String nbsp = new String(new int[]{160}, 0, 1); // /** All date format parser used to convert string to date */ // static final protected DateFormat[] parserDateFormats = new DateFormat[] { @@ -545,12 +550,39 @@ } else if (value instanceof BigDecimal) { result = (BigDecimal) value; } else { + String s = value.toString(); + // suppression de tous les blancs pour un parsing plus sur + s = StringUtils.deleteWhitespace(s); + s = StringUtils.remove(s, nbsp); try { - result = new BigDecimal(value.toString()); + // on commence par essayer par la methode la plus simple + result = new BigDecimal(s); } catch (NumberFormatException eee) { - throw new WikittyException( - String.format("Can't convert value '%s' to numeric", - getClass(value)), eee); + // on essaie avec un parse, en dernier ressort + // si jamais le nombre etait localise (ex: 99,987.76) + try { + DecimalFormat parser = new DecimalFormat(); + // on veut des BigDecimal en resultat et pas un Double + parser.setParseBigDecimal(true); + + Number n = parser.parse(s); + // meme si on demande au parser un BigDecimal, il retourne + // certaine fois un Double (NaN, Infinit, ...). + // mais dans ce cas la, ca revient a une erreur de parsing + // qui explosera avec le cast + result = (BigDecimal)n; + } catch (Exception zzz) { + // on fait suivre eee et non pas zzz car l'erreur initial + // est bien eee, zzz est juste le fallback, et certain fois + // on aurait un ClassCastException qui n'a rien a voir, + // mais on log tout de meme zzz + log.debug(String.format( + "Can't convert value '%s(%s)' to numeric, this exception is not throw but just logged", + value, getClass(value)), zzz); + throw new WikittyException( + String.format("Can't convert value '%s(%s)' to numeric", + value, getClass(value)), eee); + } } } return result.stripTrailingZeros();

1 0

r1497 - in trunk/wikitty-api/src/main/java/org/nuiton/wikitty: . storage
by bpoussin＠users.nuiton.org 15 Aug '12

15 Aug '12

Author: bpoussin Date: 2012-08-15 19:48:49 +0200 (Wed, 15 Aug 2012) New Revision: 1497 Url: http://nuiton.org/repositories/revision/wikitty/1497 Log: Anomalie #2258: When we have query with aggregate and use findByQuery the result is false Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyClient.java trunk/wikitty-api/src/main/java/org/nuiton/wikitty/storage/WikittySearchEngineHelper.java Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyClient.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyClient.java 2012-08-07 17:53:58 UTC (rev 1496) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyClient.java 2012-08-15 17:48:49 UTC (rev 1497) @@ -64,6 +64,7 @@ import org.nuiton.wikitty.query.WikittyQueryMaker; import org.nuiton.wikitty.query.WikittyQueryResult; import org.nuiton.wikitty.query.WikittyQueryResultTreeNode; +import org.nuiton.wikitty.query.conditions.Aggregate; import org.nuiton.wikitty.query.conditions.Select; import org.nuiton.wikitty.services.WikittyEvent; import org.nuiton.wikitty.services.WikittyExtensionMigrationRegistry; @@ -793,8 +794,13 @@ if (limitToFirst) { for (WikittyQuery query : serviceQueries) { - query.setOffset(0); - query.setLimit(1); + // si on a une fonction d'aggreagation (SUM, MIN, ...) il ne + // faut pas modifier l'offset ni le limit + if (!(query.getCondition() instanceof Select) || + ((Select)query.getCondition()).getAggregate() == null ) { + query.setOffset(0); + query.setLimit(1); + } // lorsqu'on limit au premier, c'est qu'on utilise la methode // pour retourne seulement un objet et pas un WikittyQueryResult // on n'a donc pas a calculer les facets Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/storage/WikittySearchEngineHelper.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/storage/WikittySearchEngineHelper.java 2012-08-07 17:53:58 UTC (rev 1496) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/storage/WikittySearchEngineHelper.java 2012-08-15 17:48:49 UTC (rev 1497) @@ -139,7 +139,15 @@ // on ne garde que ce qui est demande if (query.getOffset() < topics.size()) { int offset = query.getOffset(); - int last = Math.min(topics.size(), query.getOffset() + query.getLimit()); + int limit = query.getLimit(); + + if (limit == Integer.MAX_VALUE) { + // WARNING It is necessary to substract 'offset' otherwise, + // there is a capacity overlow + limit = Integer.MAX_VALUE - offset; + } + + int last = Math.min(topics.size(), offset + limit); selectList = selectList.subList(offset, last); }

1 0