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

r1473 - trunk/wikitty-api/src/main/java/org/nuiton/wikitty/entities
by bpoussin＠users.nuiton.org April 20, 2012

April 20, 2012

Author: bpoussin Date: 2012-04-20 02:58:23 +0200 (Fri, 20 Apr 2012) New Revision: 1473 Url: http://nuiton.org/repositories/revision/wikitty/1473 Log: add support to array as valide argument to set a collection attribute Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/entities/FieldType.java trunk/wikitty-api/src/main/java/org/nuiton/wikitty/entities/WikittyImpl.java Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/entities/FieldType.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/entities/FieldType.java 2012-04-06 15:17:07 UTC (rev 1472) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/entities/FieldType.java 2012-04-20 00:58:23 UTC (rev 1473) @@ -34,6 +34,8 @@ import java.util.Map; import java.util.Set; import org.apache.commons.lang3.StringUtils; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; import org.nuiton.wikitty.WikittyException; import org.nuiton.wikitty.WikittyUtil; import org.nuiton.wikitty.generator.WikittyTagValue; @@ -48,6 +50,9 @@ */ public class FieldType implements WikittyTagValue, Serializable { + /** to use log facility, just put in your code: log.info(\"...\"); */ + static private Log log = LogFactory.getLog(FieldType.class); + /** serialVersionUID. */ private static final long serialVersionUID = -4375308750387837026L; @@ -162,8 +167,8 @@ if (value == null) { result = null; } else if (isCollection()) { - if ( !(value instanceof Collection) ) { - throw new WikittyException( "A collection is expected for type " + if ( !(value instanceof Collection || value.getClass().isArray()) ) { + throw new WikittyException( "A collection or array is expected for type " + type.name() + "[" + lowerBound + " - " + upperBound + "]" ); } Collection<Object> col; @@ -174,8 +179,15 @@ } // copy all value in new collections - for ( Object o : (Collection<?>) value ) { - col.add( getContainedValidObject(o) ); + if (value instanceof Collection) { + for ( Object o : (Collection<?>) value ) { + col.add( getContainedValidObject(o) ); + } + } else { + // is Array + for ( Object o : (Object[]) value ) { + col.add( getContainedValidObject(o) ); + } } result = col; @@ -199,7 +211,23 @@ * @return true if value is valid */ public boolean isValidValue(Object value) { - return getValidValue(value) != null; + boolean result; + if (value == null && isNotNull()) { + result = false; + } else { + try { + getValidValue(value); + // no exception value is valid + result = true; + } catch (Exception eee) { + if (log.isDebugEnabled()) { + log.debug("Can't convert value, value is not valid, return false", eee); + } + // exception value is not valid + result = false; + } + } + return result; } public WikittyTypes getType() { 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-04-06 15:17:07 UTC (rev 1472) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/entities/WikittyImpl.java 2012-04-20 00:58:23 UTC (rev 1473) @@ -591,9 +591,17 @@ // we put it, in preload map if (value != null && fieldType.getType() == WikittyTypes.WIKITTY) { if (fieldType.isCollection()) { - for ( Object o : (Collection<?>) value ) { - Wikitty w = WikittyUtil.getWikitty(o); - addPreloaded(w); + if (value instanceof Collection) { + for ( Object o : (Collection<?>) value ) { + Wikitty w = WikittyUtil.getWikitty(o); + addPreloaded(w); + } + } else { + // Array + for ( Object o : (Object[]) value ) { + Wikitty w = WikittyUtil.getWikitty(o); + addPreloaded(w); + } } } else { Wikitty w = WikittyUtil.getWikitty(value);

1 0

r1472 - 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 April 6, 2012

April 6, 2012

Author: maven-release Date: 2012-04-06 17:17:07 +0200 (Fri, 06 Apr 2012) New Revision: 1472 Url: http://nuiton.org/repositories/revision/wikitty/1472 Log: [maven-release-plugin] prepare for next development iteration 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-04-06 15:17:05 UTC (rev 1471) +++ trunk/pom.xml 2012-04-06 15:17:07 UTC (rev 1472) @@ -15,7 +15,7 @@ <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.6</version> + <version>3.7-SNAPSHOT</version> <modules> @@ -802,10 +802,10 @@  <scm> - <connection>scm:svn:http://svn.nuiton.org/svn/wikitty/tags/wikitty-3.6</connection> - <developerConnection>scm:svn:http://svn.nuiton.org/svn/wikitty/tags/wikitty-3.6 + <connection>scm:svn:http://svn.nuiton.org/svn/wikitty/trunk</connection> + <developerConnection>scm:svn:http://svn.nuiton.org/svn/wikitty/trunk </developerConnection> - <url>http://www.nuiton.org/repositories/browse/wikitty/tags/wikitty-3.6</url> + <url>http://www.nuiton.org/repositories/browse/wikitty/trunk</url> </scm> <distributionManagement> Modified: trunk/wikitty-api/pom.xml =================================================================== --- trunk/wikitty-api/pom.xml 2012-04-06 15:17:05 UTC (rev 1471) +++ trunk/wikitty-api/pom.xml 2012-04-06 15:17:07 UTC (rev 1472) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.6</version> + <version>3.7-SNAPSHOT</version> </parent>  Modified: trunk/wikitty-dto/pom.xml =================================================================== --- trunk/wikitty-dto/pom.xml 2012-04-06 15:17:05 UTC (rev 1471) +++ trunk/wikitty-dto/pom.xml 2012-04-06 15:17:07 UTC (rev 1472) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.6</version> + <version>3.7-SNAPSHOT</version> </parent>  Modified: trunk/wikitty-generators/pom.xml =================================================================== --- trunk/wikitty-generators/pom.xml 2012-04-06 15:17:05 UTC (rev 1471) +++ trunk/wikitty-generators/pom.xml 2012-04-06 15:17:07 UTC (rev 1472) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.6</version> + <version>3.7-SNAPSHOT</version> </parent>  Modified: trunk/wikitty-hessian-client/pom.xml =================================================================== --- trunk/wikitty-hessian-client/pom.xml 2012-04-06 15:17:05 UTC (rev 1471) +++ trunk/wikitty-hessian-client/pom.xml 2012-04-06 15:17:07 UTC (rev 1472) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.6</version> + <version>3.7-SNAPSHOT</version> </parent>  Modified: trunk/wikitty-hessian-server/pom.xml =================================================================== --- trunk/wikitty-hessian-server/pom.xml 2012-04-06 15:17:05 UTC (rev 1471) +++ trunk/wikitty-hessian-server/pom.xml 2012-04-06 15:17:07 UTC (rev 1472) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.6</version> + <version>3.7-SNAPSHOT</version> </parent>  Modified: trunk/wikitty-jdbc/pom.xml =================================================================== --- trunk/wikitty-jdbc/pom.xml 2012-04-06 15:17:05 UTC (rev 1471) +++ trunk/wikitty-jdbc/pom.xml 2012-04-06 15:17:07 UTC (rev 1472) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.6</version> + <version>3.7-SNAPSHOT</version> </parent> <groupId>org.nuiton.wikitty</groupId> Modified: trunk/wikitty-perf-test/pom.xml =================================================================== --- trunk/wikitty-perf-test/pom.xml 2012-04-06 15:17:05 UTC (rev 1471) +++ trunk/wikitty-perf-test/pom.xml 2012-04-06 15:17:07 UTC (rev 1472) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.6</version> + <version>3.7-SNAPSHOT</version> </parent> <groupId>org.nuiton.wikitty</groupId> Modified: trunk/wikitty-solr/pom.xml =================================================================== --- trunk/wikitty-solr/pom.xml 2012-04-06 15:17:05 UTC (rev 1471) +++ trunk/wikitty-solr/pom.xml 2012-04-06 15:17:07 UTC (rev 1472) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.6</version> + <version>3.7-SNAPSHOT</version> </parent>  Modified: trunk/wikitty-struts/pom.xml =================================================================== --- trunk/wikitty-struts/pom.xml 2012-04-06 15:17:05 UTC (rev 1471) +++ trunk/wikitty-struts/pom.xml 2012-04-06 15:17:07 UTC (rev 1472) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.6</version> + <version>3.7-SNAPSHOT</version> </parent>

1 0

r1471 - in tags: . wikitty-3.6
by maven-release＠users.nuiton.org April 6, 2012

April 6, 2012

Author: maven-release Date: 2012-04-06 17:17:05 +0200 (Fri, 06 Apr 2012) New Revision: 1471 Url: http://nuiton.org/repositories/revision/wikitty/1471 Log: [maven-release-plugin] copy for tag wikitty-3.6 Added: tags/wikitty-3.6/ Property changes on: tags/wikitty-3.6 ___________________________________________________________________ Added: svn:ignore + .settings .project target *.iml *.ipr *.iws Added: svn:mergeinfo + /branches/2.0-eugene2:164-179

1 0

r1470 - 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 April 6, 2012

April 6, 2012

Author: maven-release Date: 2012-04-06 17:17:03 +0200 (Fri, 06 Apr 2012) New Revision: 1470 Url: http://nuiton.org/repositories/revision/wikitty/1470 Log: [maven-release-plugin] prepare release wikitty-3.6 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-04-06 15:00:42 UTC (rev 1469) +++ trunk/pom.xml 2012-04-06 15:17:03 UTC (rev 1470) @@ -15,7 +15,7 @@ <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.6-SNAPSHOT</version> + <version>3.6</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.6</connection> + <developerConnection>scm:svn:http://svn.nuiton.org/svn/wikitty/tags/wikitty-3.6 </developerConnection> - <url>http://www.nuiton.org/repositories/browse/wikitty/trunk</url> + <url>http://www.nuiton.org/repositories/browse/wikitty/tags/wikitty-3.6</url> </scm> <distributionManagement> Modified: trunk/wikitty-api/pom.xml =================================================================== --- trunk/wikitty-api/pom.xml 2012-04-06 15:00:42 UTC (rev 1469) +++ trunk/wikitty-api/pom.xml 2012-04-06 15:17:03 UTC (rev 1470) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.6-SNAPSHOT</version> + <version>3.6</version> </parent>  Modified: trunk/wikitty-dto/pom.xml =================================================================== --- trunk/wikitty-dto/pom.xml 2012-04-06 15:00:42 UTC (rev 1469) +++ trunk/wikitty-dto/pom.xml 2012-04-06 15:17:03 UTC (rev 1470) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.6-SNAPSHOT</version> + <version>3.6</version> </parent>  Modified: trunk/wikitty-generators/pom.xml =================================================================== --- trunk/wikitty-generators/pom.xml 2012-04-06 15:00:42 UTC (rev 1469) +++ trunk/wikitty-generators/pom.xml 2012-04-06 15:17:03 UTC (rev 1470) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.6-SNAPSHOT</version> + <version>3.6</version> </parent>  Modified: trunk/wikitty-hessian-client/pom.xml =================================================================== --- trunk/wikitty-hessian-client/pom.xml 2012-04-06 15:00:42 UTC (rev 1469) +++ trunk/wikitty-hessian-client/pom.xml 2012-04-06 15:17:03 UTC (rev 1470) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.6-SNAPSHOT</version> + <version>3.6</version> </parent>  Modified: trunk/wikitty-hessian-server/pom.xml =================================================================== --- trunk/wikitty-hessian-server/pom.xml 2012-04-06 15:00:42 UTC (rev 1469) +++ trunk/wikitty-hessian-server/pom.xml 2012-04-06 15:17:03 UTC (rev 1470) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.6-SNAPSHOT</version> + <version>3.6</version> </parent>  Modified: trunk/wikitty-jdbc/pom.xml =================================================================== --- trunk/wikitty-jdbc/pom.xml 2012-04-06 15:00:42 UTC (rev 1469) +++ trunk/wikitty-jdbc/pom.xml 2012-04-06 15:17:03 UTC (rev 1470) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.6-SNAPSHOT</version> + <version>3.6</version> </parent> <groupId>org.nuiton.wikitty</groupId> Modified: trunk/wikitty-perf-test/pom.xml =================================================================== --- trunk/wikitty-perf-test/pom.xml 2012-04-06 15:00:42 UTC (rev 1469) +++ trunk/wikitty-perf-test/pom.xml 2012-04-06 15:17:03 UTC (rev 1470) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.6-SNAPSHOT</version> + <version>3.6</version> </parent> <groupId>org.nuiton.wikitty</groupId> Modified: trunk/wikitty-solr/pom.xml =================================================================== --- trunk/wikitty-solr/pom.xml 2012-04-06 15:00:42 UTC (rev 1469) +++ trunk/wikitty-solr/pom.xml 2012-04-06 15:17:03 UTC (rev 1470) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.6-SNAPSHOT</version> + <version>3.6</version> </parent>  Modified: trunk/wikitty-struts/pom.xml =================================================================== --- trunk/wikitty-struts/pom.xml 2012-04-06 15:00:42 UTC (rev 1469) +++ trunk/wikitty-struts/pom.xml 2012-04-06 15:17:03 UTC (rev 1470) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.6-SNAPSHOT</version> + <version>3.6</version> </parent>

1 0

r1469 - trunk/wikitty-api/src/main/java/org/nuiton/wikitty/entities
by bpoussin＠users.nuiton.org April 6, 2012

April 6, 2012

Author: bpoussin Date: 2012-04-06 17:00:42 +0200 (Fri, 06 Apr 2012) New Revision: 1469 Url: http://nuiton.org/repositories/revision/wikitty/1469 Log: au lieu de retourner faux, retourne vrai pour l'instant si on a une contrainte allowed non verifier, et un message de warn est affiche a la place Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/entities/FieldTypeConstaintChecker.java Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/entities/FieldTypeConstaintChecker.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/entities/FieldTypeConstaintChecker.java 2012-04-06 10:00:12 UTC (rev 1468) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/entities/FieldTypeConstaintChecker.java 2012-04-06 15:00:42 UTC (rev 1469) @@ -35,6 +35,7 @@ import java.util.Set; import java.util.regex.Pattern; import org.apache.commons.collections.CollectionUtils; +import org.apache.commons.collections.ListUtils; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.nuiton.wikitty.WikittyClient; @@ -202,11 +203,10 @@ */ protected boolean isAllowedWikitty(String fqfield, FieldType field, Collection<String> ids, Collection<String> errors) { - boolean result = false; - if (CollectionUtils.isEmpty(ids)) { - // si la liste est vide, alors on sait que la valeur est autorisee - result = true; - } else { + boolean result = true; + // par defaut s'il n'y a rien a verifier le resultat est vrai + if (!CollectionUtils.isEmpty(ids)) { + // creation de la requete qui ramene tous les ids d'objets autorisé WikittyQuery q = null; if (field.hasAllowedQuery()) { // construire les requetes @@ -220,43 +220,89 @@ q = maker.end(); } - List<String> allowed = null; + List<String> unallowed = null; + // si il y a une requete qui contraint ... if (q != null) { - q.setOffset(0); - q.setLimit(WikittyQuery.MAX); + // ... on lui ajoute la liste des ids qu'a le champs comme valeur + // et le resultat doit etre identique a cette liste. + // sinon cela veut dire que certain id du champs ne sont pas + // dans les valeurs acceptables + WikittyQuery checkQuery = new WikittyQueryMaker().and() + .condition(q.getCondition()) + .containsOne(Element.ID, ids).end() + .setOffset(0).setLimit(WikittyQuery.MAX); + WikittyQueryResult<String> queryResult = + ws.findAllByQuery(token, Collections.singletonList(checkQuery)).get(0); + List<String> idResult = queryResult.getAll(); - WikittyQuery wq = new WikittyQueryMaker().containsOne(Element.ID, ids).end(); - wq.setOffset(0); - wq.setLimit(WikittyQuery.MAX); - - List<WikittyQueryResult<String>> queryResult = - ws.findAllByQuery(token, Arrays.asList(q, wq)); - allowed = queryResult.get(0).getAll(); - List<String> findedIds = queryResult.get(1).getAll(); - - // on ne check que sur les ids trouve dans la base et on espere - // que les autres sont bon :). En fait, lorsqu'on store plusieurs - // wikitties en meme temps donc un wikitty qui doit etre la valeur - // d'un champs d'un autre. Celui ci n'est pas retrouvable dans la - // base et donc on ne peut pas verifier facilement la contrainte. - // Pour l'instant on ne fait rien, car les solutions seraient trop - // couteuse (creeer un storage in memory y mettre les objets a - // checker, faire le check en requetant le vrai repo et le in memory - // si tout va bien faire le store reel sur le vrai repo. - result = allowed.containsAll(findedIds); - if (log.isDebugEnabled() && findedIds.size() != ids.size()) { + result = idResult.size() == ids.size(); + if (!result) { + unallowed = ListUtils.removeAll(ids, idResult); + } + if (log.isDebugEnabled() && !result) { log.debug(String.format( - "For field '%s' allowed contraint not checked for: %s", - fqfield, CollectionUtils.disjunction(findedIds, ids))); + "For field '%s' allowed contraint not checked for: %s (ids=%s, idResult=%s)(request=%s)", + fqfield, unallowed, ids, idResult, queryResult.getQueryString())); } + } + +// List<String> allowed = null; +// if (q != null) { +// q.setOffset(0); +// q.setLimit(WikittyQuery.MAX); +// +// WikittyQuery wq = new WikittyQueryMaker().containsOne(Element.ID, ids).end(); +// wq.setOffset(0); +// wq.setLimit(WikittyQuery.MAX); +// +// List<WikittyQueryResult<String>> queryResult = +// ws.findAllByQuery(token, Arrays.asList(q, wq)); +// allowed = queryResult.get(0).getAll(); +// List<String> findedIds = queryResult.get(1).getAll(); +// +// // on ne check que sur les ids trouve dans la base et on espere +// // que les autres sont bon :). En fait, lorsqu'on store plusieurs +// // wikitties en meme temps donc un wikitty qui doit etre la valeur +// // d'un champs d'un autre. Celui ci n'est pas retrouvable dans la +// // base et donc on ne peut pas verifier facilement la contrainte. +// // Pour l'instant on ne fait rien, car les solutions seraient trop +// // couteuse (creeer un storage in memory y mettre les objets a +// // checker, faire le check en requetant le vrai repo et le in memory +// // si tout va bien faire le store reel sur le vrai repo. +// result = allowed.containsAll(findedIds); +// if (log.isDebugEnabled() && findedIds.size() != ids.size()) { +// log.debug(String.format( +// "For field '%s' allowed contraint not checked for: %s", +// fqfield, CollectionUtils.disjunction(findedIds, ids))); +// } +// } + if (errors != null && !result) { errors.add(String.format( - "Field '%s' contains unallowed Wikitty values %s allowed %s", - fqfield, ids, allowed)); + "Field '%s' contains unallowed Wikitty values %s", + fqfield, unallowed)); } + + + // FIXME poussin 20120406: on ne peut pas garantir cette contrainte + // actuellement car l'objet qui est mis dan le champs a checker peut + // aussi etre un objet en cours de stockage. Donc pour l'instant on ne + // leve pas d'erreur, mais on mes un warn dans les logs. + // + // Une solution serait de creeer un storage in memory y mettre les + // objets a checker, faire le check en requetant le vrai repo et sur + // les ids en erreur refaire la meme requete sur le in memory + if (!result) { + result = true; + log.warn(String.format( + "Field '%s' contains unallowed Wikitty values %s", + fqfield, unallowed)); + } + // FIN du FIXME } + return result; }

1 0

r1468 - trunk/wikitty-api/src/main/java/org/nuiton/wikitty/entities
by bpoussin＠users.nuiton.org April 6, 2012

April 6, 2012

Author: bpoussin Date: 2012-04-06 12:00:12 +0200 (Fri, 06 Apr 2012) New Revision: 1468 Url: http://nuiton.org/repositories/revision/wikitty/1468 Log: optimisation in isAllowedWikitty, if collection is empty don't do search, juste return true Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/entities/FieldTypeConstaintChecker.java Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/entities/FieldTypeConstaintChecker.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/entities/FieldTypeConstaintChecker.java 2012-04-06 09:47:10 UTC (rev 1467) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/entities/FieldTypeConstaintChecker.java 2012-04-06 10:00:12 UTC (rev 1468) @@ -203,55 +203,59 @@ protected boolean isAllowedWikitty(String fqfield, FieldType field, Collection<String> ids, Collection<String> errors) { boolean result = false; + if (CollectionUtils.isEmpty(ids)) { + // si la liste est vide, alors on sait que la valeur est autorisee + result = true; + } else { + WikittyQuery q = null; + if (field.hasAllowedQuery()) { + // construire les requetes + q = WikittyQueryParser.parse(field.getAllowedQuery()); - WikittyQuery q = null; - if (field.hasAllowedQuery()) { - // construire les requetes - q = WikittyQueryParser.parse(field.getAllowedQuery()); - - } else if (field.hasAllowed()) { - WikittyQueryMaker maker = new WikittyQueryMaker().or(); - for (String extName : field.getAllowedAsList()) { - maker.exteq(extName); + } else if (field.hasAllowed()) { + WikittyQueryMaker maker = new WikittyQueryMaker().or(); + for (String extName : field.getAllowedAsList()) { + maker.exteq(extName); + } + q = maker.end(); } - q = maker.end(); - } - List<String> allowed = null; - if (q != null) { - q.setOffset(0); - q.setLimit(WikittyQuery.MAX); + List<String> allowed = null; + if (q != null) { + q.setOffset(0); + q.setLimit(WikittyQuery.MAX); - WikittyQuery wq = new WikittyQueryMaker().containsOne(Element.ID, ids).end(); - wq.setOffset(0); - wq.setLimit(WikittyQuery.MAX); + WikittyQuery wq = new WikittyQueryMaker().containsOne(Element.ID, ids).end(); + wq.setOffset(0); + wq.setLimit(WikittyQuery.MAX); - List<WikittyQueryResult<String>> queryResult = - ws.findAllByQuery(token, Arrays.asList(q, wq)); - allowed = queryResult.get(0).getAll(); - List<String> findedIds = queryResult.get(1).getAll(); + List<WikittyQueryResult<String>> queryResult = + ws.findAllByQuery(token, Arrays.asList(q, wq)); + allowed = queryResult.get(0).getAll(); + List<String> findedIds = queryResult.get(1).getAll(); - // on ne check que sur les ids trouve dans la base et on espere - // que les autres sont bon :). En fait, lorsqu'on store plusieurs - // wikitties en meme temps donc un wikitty qui doit etre la valeur - // d'un champs d'un autre. Celui ci n'est pas retrouvable dans la - // base et donc on ne peut pas verifier facilement la contrainte. - // Pour l'instant on ne fait rien, car les solutions seraient trop - // couteuse (creeer un storage in memory y mettre les objets a - // checker, faire le check en requetant le vrai repo et le in memory - // si tout va bien faire le store reel sur le vrai repo. - result = allowed.containsAll(findedIds); - if (log.isDebugEnabled() && findedIds.size() != ids.size()) { - log.debug(String.format( - "For field '%s' allowed contraint not checked for: %s", - fqfield, CollectionUtils.disjunction(findedIds, ids))); + // on ne check que sur les ids trouve dans la base et on espere + // que les autres sont bon :). En fait, lorsqu'on store plusieurs + // wikitties en meme temps donc un wikitty qui doit etre la valeur + // d'un champs d'un autre. Celui ci n'est pas retrouvable dans la + // base et donc on ne peut pas verifier facilement la contrainte. + // Pour l'instant on ne fait rien, car les solutions seraient trop + // couteuse (creeer un storage in memory y mettre les objets a + // checker, faire le check en requetant le vrai repo et le in memory + // si tout va bien faire le store reel sur le vrai repo. + result = allowed.containsAll(findedIds); + if (log.isDebugEnabled() && findedIds.size() != ids.size()) { + log.debug(String.format( + "For field '%s' allowed contraint not checked for: %s", + fqfield, CollectionUtils.disjunction(findedIds, ids))); + } } - } - if (errors != null && !result) { - errors.add(String.format( - "Field '%s' contains unallowed Wikitty values %s allowed %s", - fqfield, ids, allowed)); + if (errors != null && !result) { + errors.add(String.format( + "Field '%s' contains unallowed Wikitty values %s allowed %s", + fqfield, ids, allowed)); + } } return result; }

1 0

r1467 - trunk/wikitty-solr/src/main/java/org/nuiton/wikitty/storage/solr
by bpoussin＠users.nuiton.org April 6, 2012

April 6, 2012

Author: bpoussin Date: 2012-04-06 11:47:10 +0200 (Fri, 06 Apr 2012) New Revision: 1467 Url: http://nuiton.org/repositories/revision/wikitty/1467 Log: Anomalie fix #2053 Error when we try to search containsAll or containsOne with empty collection 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-04-05 16:11:56 UTC (rev 1466) +++ trunk/wikitty-solr/src/main/java/org/nuiton/wikitty/storage/solr/WikittyQueryVisitorToSolr.java 2012-04-06 09:47:10 UTC (rev 1467) @@ -318,17 +318,21 @@ @Override public boolean visitEnter(ContainsAll o) { - String element2solr = element2solr(o.getElement()); + List<String> values = evalConditionValueAsList(o.getValues()); + if (values.size() == 0) { + solrQuery += "( *:* - *:* )"; // false / rien + } else { + String element2solr = element2solr(o.getElement()); - solrQuery += element2solr + ":("; + solrQuery += element2solr + ":("; - String sep = ""; - for(String e : evalConditionValueAsList(o.getValues())) { - solrQuery += sep + fixSolrBug(element2solr, e); - sep = " AND "; + String sep = ""; + for(String e : values) { + solrQuery += sep + fixSolrBug(element2solr, e); + sep = " AND "; + } + solrQuery += ")"; } - solrQuery += ")"; - return false; } @@ -345,15 +349,20 @@ @Override public boolean visitEnter(ContainsOne o) { - String element2solr = element2solr(o.getElement()); - - solrQuery += element2solr + ":("; - String sep = ""; - for(String e : evalConditionValueAsList(o.getValues())) { - solrQuery += sep + fixSolrBug(element2solr, e); - sep = " OR "; + List<String> values = evalConditionValueAsList(o.getValues()); + if (values.size() == 0) { + solrQuery += "( *:* - *:* )"; // false / rien + } else { + String element2solr = element2solr(o.getElement()); + + solrQuery += element2solr + ":("; + String sep = ""; + for(String e : values) { + solrQuery += sep + fixSolrBug(element2solr, e); + sep = " OR "; + } + solrQuery += ")"; } - solrQuery += ")"; return false; }

1 0

r1466 - 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 April 5, 2012

April 5, 2012

Author: maven-release Date: 2012-04-05 18:11:56 +0200 (Thu, 05 Apr 2012) New Revision: 1466 Url: http://nuiton.org/repositories/revision/wikitty/1466 Log: [maven-release-plugin] prepare for next development iteration 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-04-05 16:11:54 UTC (rev 1465) +++ trunk/pom.xml 2012-04-05 16:11:56 UTC (rev 1466) @@ -15,7 +15,7 @@ <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.5</version> + <version>3.6-SNAPSHOT</version> <modules> @@ -802,10 +802,10 @@  <scm> - <connection>scm:svn:http://svn.nuiton.org/svn/wikitty/tags/wikitty-3.5</connection> - <developerConnection>scm:svn:http://svn.nuiton.org/svn/wikitty/tags/wikitty-3.5 + <connection>scm:svn:http://svn.nuiton.org/svn/wikitty/trunk</connection> + <developerConnection>scm:svn:http://svn.nuiton.org/svn/wikitty/trunk </developerConnection> - <url>http://www.nuiton.org/repositories/browse/wikitty/tags/wikitty-3.5</url> + <url>http://www.nuiton.org/repositories/browse/wikitty/trunk</url> </scm> <distributionManagement> Modified: trunk/wikitty-api/pom.xml =================================================================== --- trunk/wikitty-api/pom.xml 2012-04-05 16:11:54 UTC (rev 1465) +++ trunk/wikitty-api/pom.xml 2012-04-05 16:11:56 UTC (rev 1466) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.5</version> + <version>3.6-SNAPSHOT</version> </parent>  Modified: trunk/wikitty-dto/pom.xml =================================================================== --- trunk/wikitty-dto/pom.xml 2012-04-05 16:11:54 UTC (rev 1465) +++ trunk/wikitty-dto/pom.xml 2012-04-05 16:11:56 UTC (rev 1466) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.5</version> + <version>3.6-SNAPSHOT</version> </parent>  Modified: trunk/wikitty-generators/pom.xml =================================================================== --- trunk/wikitty-generators/pom.xml 2012-04-05 16:11:54 UTC (rev 1465) +++ trunk/wikitty-generators/pom.xml 2012-04-05 16:11:56 UTC (rev 1466) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.5</version> + <version>3.6-SNAPSHOT</version> </parent>  Modified: trunk/wikitty-hessian-client/pom.xml =================================================================== --- trunk/wikitty-hessian-client/pom.xml 2012-04-05 16:11:54 UTC (rev 1465) +++ trunk/wikitty-hessian-client/pom.xml 2012-04-05 16:11:56 UTC (rev 1466) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.5</version> + <version>3.6-SNAPSHOT</version> </parent>  Modified: trunk/wikitty-hessian-server/pom.xml =================================================================== --- trunk/wikitty-hessian-server/pom.xml 2012-04-05 16:11:54 UTC (rev 1465) +++ trunk/wikitty-hessian-server/pom.xml 2012-04-05 16:11:56 UTC (rev 1466) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.5</version> + <version>3.6-SNAPSHOT</version> </parent>  Modified: trunk/wikitty-jdbc/pom.xml =================================================================== --- trunk/wikitty-jdbc/pom.xml 2012-04-05 16:11:54 UTC (rev 1465) +++ trunk/wikitty-jdbc/pom.xml 2012-04-05 16:11:56 UTC (rev 1466) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.5</version> + <version>3.6-SNAPSHOT</version> </parent> <groupId>org.nuiton.wikitty</groupId> Modified: trunk/wikitty-perf-test/pom.xml =================================================================== --- trunk/wikitty-perf-test/pom.xml 2012-04-05 16:11:54 UTC (rev 1465) +++ trunk/wikitty-perf-test/pom.xml 2012-04-05 16:11:56 UTC (rev 1466) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.5</version> + <version>3.6-SNAPSHOT</version> </parent> <groupId>org.nuiton.wikitty</groupId> Modified: trunk/wikitty-solr/pom.xml =================================================================== --- trunk/wikitty-solr/pom.xml 2012-04-05 16:11:54 UTC (rev 1465) +++ trunk/wikitty-solr/pom.xml 2012-04-05 16:11:56 UTC (rev 1466) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.5</version> + <version>3.6-SNAPSHOT</version> </parent>  Modified: trunk/wikitty-struts/pom.xml =================================================================== --- trunk/wikitty-struts/pom.xml 2012-04-05 16:11:54 UTC (rev 1465) +++ trunk/wikitty-struts/pom.xml 2012-04-05 16:11:56 UTC (rev 1466) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.5</version> + <version>3.6-SNAPSHOT</version> </parent>

1 0

r1465 - in tags: . wikitty-3.5
by maven-release＠users.nuiton.org April 5, 2012

April 5, 2012

Author: maven-release Date: 2012-04-05 18:11:54 +0200 (Thu, 05 Apr 2012) New Revision: 1465 Url: http://nuiton.org/repositories/revision/wikitty/1465 Log: [maven-release-plugin] copy for tag wikitty-3.5 Added: tags/wikitty-3.5/ Property changes on: tags/wikitty-3.5 ___________________________________________________________________ Added: svn:ignore + .settings .project target *.iml *.ipr *.iws Added: svn:mergeinfo + /branches/2.0-eugene2:164-179

1 0

r1464 - 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 April 5, 2012

April 5, 2012

Author: maven-release Date: 2012-04-05 18:11:53 +0200 (Thu, 05 Apr 2012) New Revision: 1464 Url: http://nuiton.org/repositories/revision/wikitty/1464 Log: [maven-release-plugin] prepare release wikitty-3.5 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-04-05 16:08:38 UTC (rev 1463) +++ trunk/pom.xml 2012-04-05 16:11:53 UTC (rev 1464) @@ -15,7 +15,7 @@ <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.5-SNAPSHOT</version> + <version>3.5</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.5</connection> + <developerConnection>scm:svn:http://svn.nuiton.org/svn/wikitty/tags/wikitty-3.5 </developerConnection> - <url>http://www.nuiton.org/repositories/browse/wikitty/trunk</url> + <url>http://www.nuiton.org/repositories/browse/wikitty/tags/wikitty-3.5</url> </scm> <distributionManagement> Modified: trunk/wikitty-api/pom.xml =================================================================== --- trunk/wikitty-api/pom.xml 2012-04-05 16:08:38 UTC (rev 1463) +++ trunk/wikitty-api/pom.xml 2012-04-05 16:11:53 UTC (rev 1464) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.5-SNAPSHOT</version> + <version>3.5</version> </parent>  Modified: trunk/wikitty-dto/pom.xml =================================================================== --- trunk/wikitty-dto/pom.xml 2012-04-05 16:08:38 UTC (rev 1463) +++ trunk/wikitty-dto/pom.xml 2012-04-05 16:11:53 UTC (rev 1464) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.5-SNAPSHOT</version> + <version>3.5</version> </parent>  Modified: trunk/wikitty-generators/pom.xml =================================================================== --- trunk/wikitty-generators/pom.xml 2012-04-05 16:08:38 UTC (rev 1463) +++ trunk/wikitty-generators/pom.xml 2012-04-05 16:11:53 UTC (rev 1464) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.5-SNAPSHOT</version> + <version>3.5</version> </parent>  Modified: trunk/wikitty-hessian-client/pom.xml =================================================================== --- trunk/wikitty-hessian-client/pom.xml 2012-04-05 16:08:38 UTC (rev 1463) +++ trunk/wikitty-hessian-client/pom.xml 2012-04-05 16:11:53 UTC (rev 1464) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.5-SNAPSHOT</version> + <version>3.5</version> </parent>  Modified: trunk/wikitty-hessian-server/pom.xml =================================================================== --- trunk/wikitty-hessian-server/pom.xml 2012-04-05 16:08:38 UTC (rev 1463) +++ trunk/wikitty-hessian-server/pom.xml 2012-04-05 16:11:53 UTC (rev 1464) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.5-SNAPSHOT</version> + <version>3.5</version> </parent>  Modified: trunk/wikitty-jdbc/pom.xml =================================================================== --- trunk/wikitty-jdbc/pom.xml 2012-04-05 16:08:38 UTC (rev 1463) +++ trunk/wikitty-jdbc/pom.xml 2012-04-05 16:11:53 UTC (rev 1464) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.5-SNAPSHOT</version> + <version>3.5</version> </parent> <groupId>org.nuiton.wikitty</groupId> Modified: trunk/wikitty-perf-test/pom.xml =================================================================== --- trunk/wikitty-perf-test/pom.xml 2012-04-05 16:08:38 UTC (rev 1463) +++ trunk/wikitty-perf-test/pom.xml 2012-04-05 16:11:53 UTC (rev 1464) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.5-SNAPSHOT</version> + <version>3.5</version> </parent> <groupId>org.nuiton.wikitty</groupId> Modified: trunk/wikitty-solr/pom.xml =================================================================== --- trunk/wikitty-solr/pom.xml 2012-04-05 16:08:38 UTC (rev 1463) +++ trunk/wikitty-solr/pom.xml 2012-04-05 16:11:53 UTC (rev 1464) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.5-SNAPSHOT</version> + <version>3.5</version> </parent>  Modified: trunk/wikitty-struts/pom.xml =================================================================== --- trunk/wikitty-struts/pom.xml 2012-04-05 16:08:38 UTC (rev 1463) +++ trunk/wikitty-struts/pom.xml 2012-04-05 16:11:53 UTC (rev 1464) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>wikitty</artifactId> - <version>3.5-SNAPSHOT</version> + <version>3.5</version> </parent>

1 0