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

r240 - in trunk: wikitty-api/src/main/java/org/nuiton/wikitty wikitty-solr-impl/src/test/java/org/nuiton/wikitty/solr/test
by bleny＠users.nuiton.org Aug. 6, 2010

Aug. 6, 2010

Author: bleny Date: 2010-08-06 16:41:51 +0200 (Fri, 06 Aug 2010) New Revision: 240 Url: http://nuiton.org/repositories/revision/wikitty/240 Log: fixing WikittyUtil date pattern using a pattern compatible with solr + test Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyUtil.java trunk/wikitty-solr-impl/src/test/java/org/nuiton/wikitty/solr/test/SolrSearchTest.java Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyUtil.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyUtil.java 2010-08-06 13:23:30 UTC (rev 239) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyUtil.java 2010-08-06 14:41:51 UTC (rev 240) @@ -52,7 +52,7 @@ public static final String DEFAULT_VERSION = "0.0"; - public static final String DATE_FORMAT = "yyyy-MM-dd'T'HH:mm:ss.SSS'Z'Z"; + public static final String DATE_FORMAT = "yyyy-MM-dd'T'HH:mm:ss.SSSZ'Z'"; public static final String[] DATE_FORMAT_ALLOWED = { DATE_FORMAT, new SimpleDateFormat().toPattern(), Modified: trunk/wikitty-solr-impl/src/test/java/org/nuiton/wikitty/solr/test/SolrSearchTest.java =================================================================== --- trunk/wikitty-solr-impl/src/test/java/org/nuiton/wikitty/solr/test/SolrSearchTest.java 2010-08-06 13:23:30 UTC (rev 239) +++ trunk/wikitty-solr-impl/src/test/java/org/nuiton/wikitty/solr/test/SolrSearchTest.java 2010-08-06 14:41:51 UTC (rev 240) @@ -4,6 +4,8 @@ import static junit.framework.Assert.assertTrue; import java.util.ArrayList; +import java.util.Calendar; +import java.util.Date; import java.util.List; import org.apache.commons.logging.Log; @@ -410,4 +412,31 @@ list = result.getAll(); assertEquals(0, list.size()); } + + /** test that doing a search with a date criteria is possible */ + @Test + public void testSearchByDate() throws Exception { + Calendar cal = Calendar.getInstance(); + cal.set(Calendar.DAY_OF_MONTH, 20); + cal.set(Calendar.MONTH, 9); + cal.set(Calendar.YEAR, 2009); + + // this must return a date in a format understandable for solr + // pattern in WikittyUtil has to be compatible + String dateString = WikittyUtil.formatDate(cal.getTime()); + + Criteria criteria = Search.query() + .gt("Test.buildDate", dateString) + .criteria() + .setFirstIndex(0).setEndIndex(Criteria.ALL_ELEMENTS); + + // If an exception is thrown, check that the pattern in WikittyUtil + // is compatible with solr, in particular that the trailing Z + // is present and respect http://wiki.apache.org/solr/IndexingDates + PagedResult<String> result = ws.findAllByCriteria(null, criteria); + + + List<String> list = result.getAll(); + assertEquals(1, list.size()); + } }

1 0

r239 - trunk/wikitty-api/src/main/java/org/nuiton/wikitty
by sletellier＠users.nuiton.org Aug. 6, 2010

Aug. 6, 2010

Author: sletellier Date: 2010-08-06 15:23:30 +0200 (Fri, 06 Aug 2010) New Revision: 239 Url: http://nuiton.org/repositories/revision/wikitty/239 Log: Fix npe : restored wikitty can be null Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyServiceCached.java Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyServiceCached.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyServiceCached.java 2010-08-06 10:55:52 UTC (rev 238) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyServiceCached.java 2010-08-06 13:23:30 UTC (rev 239) @@ -101,6 +101,12 @@ /** wrap the wikitty or copy it according to allwaysRestoreCopies value */ protected Wikitty wrapWikitty(Wikitty wikitty) { + + // Restored wikitty can be null + if (wikitty == null) { + return null; + } + Wikitty result = null; if (allwaysRestoreCopies) { try {

1 0

r238 - in branches: . wikitty-eugene-migration
by bleny＠users.nuiton.org Aug. 6, 2010

Aug. 6, 2010

Author: bleny Date: 2010-08-06 12:55:52 +0200 (Fri, 06 Aug 2010) New Revision: 238 Url: http://nuiton.org/repositories/revision/wikitty/238 Log: creating a branch to migrate generation templates Added: branches/wikitty-eugene-migration/ Property changes on: branches/wikitty-eugene-migration ___________________________________________________________________ Added: svn:ignore + .settings .project target *.iml *.ipr *.iws Added: svn:mergeinfo + /branches/2.0-eugene2:164-179

1 0

r237 - trunk/wikitty-api/src/main/java/org/nuiton/wikitty
by echatellier＠users.nuiton.org Aug. 6, 2010

Aug. 6, 2010

Author: echatellier Date: 2010-08-06 12:40:40 +0200 (Fri, 06 Aug 2010) New Revision: 237 Url: http://nuiton.org/repositories/revision/wikitty/237 Log: Add log error message Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyServiceImpl.java Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyServiceImpl.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyServiceImpl.java 2010-08-06 09:30:04 UTC (rev 236) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyServiceImpl.java 2010-08-06 10:40:40 UTC (rev 237) @@ -430,6 +430,9 @@ transaction.commit(); return result; } catch (Exception eee) { + if (log.isErrorEnabled()) { + log.error("Can't restore wikitty collection", eee); + } transaction.rollback(); throw new WikittyException(eee); }

1 0

r236 - in trunk: wikitty-api/src/main/java/org/nuiton/wikitty wikitty-solr-impl/src/test/java/org/nuiton/wikitty/solr/test
by bleny＠users.nuiton.org Aug. 6, 2010

Aug. 6, 2010

Author: bleny Date: 2010-08-06 11:30:04 +0200 (Fri, 06 Aug 2010) New Revision: 236 Url: http://nuiton.org/repositories/revision/wikitty/236 Log: bugfixes cache ; ignoring a test that fail randomly Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/Wikitty.java trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyCopyOnWrite.java trunk/wikitty-solr-impl/src/test/java/org/nuiton/wikitty/solr/test/SolrServerTest.java Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/Wikitty.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/Wikitty.java 2010-08-06 09:09:56 UTC (rev 235) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/Wikitty.java 2010-08-06 09:30:04 UTC (rev 236) @@ -1,13 +1,14 @@ package org.nuiton.wikitty; import java.beans.PropertyChangeListener; +import java.io.Serializable; import java.math.BigDecimal; import java.util.Collection; import java.util.Date; import java.util.List; import java.util.Set; -public interface Wikitty extends Cloneable { +public interface Wikitty extends Cloneable, Serializable { void addPropertyChangeListener(PropertyChangeListener listener); Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyCopyOnWrite.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyCopyOnWrite.java 2010-08-06 09:09:56 UTC (rev 235) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyCopyOnWrite.java 2010-08-06 09:30:04 UTC (rev 236) @@ -25,10 +25,14 @@ */ public class WikittyCopyOnWrite implements Wikitty { + private static final long serialVersionUID = 1L; + private static final Log log = LogFactory.getLog(WikittyCopyOnWrite.class); protected Wikitty target; + protected boolean targetIsACopy = false; + /** only WikittyService with cache should create instances */ protected WikittyCopyOnWrite(Wikitty target) { this.target = target; @@ -39,13 +43,17 @@ * this method must be called to prevent any modification on target */ protected void substituteTargetWithCopy() { - try { - target = target.clone(); - if (log.isTraceEnabled()) { - log.trace(this + " now has for target " + target); + // test make a the copy once + if (! targetIsACopy) { + try { + target = target.clone(); + if (log.isTraceEnabled()) { + log.trace(this + " now has for target " + target); + } + } catch (CloneNotSupportedException e) { + log.error("unable to clone wikitty " + target, e); } - } catch (CloneNotSupportedException e) { - log.error("unable to clone wikitty " + target, e); + targetIsACopy = true; } } @@ -68,9 +76,7 @@ // return a clone of the target return target.clone(); } - - - + /* ** below are only delegation code with copy-on-write */ public void addPropertyChangeListener(PropertyChangeListener listener) { Modified: trunk/wikitty-solr-impl/src/test/java/org/nuiton/wikitty/solr/test/SolrServerTest.java =================================================================== --- trunk/wikitty-solr-impl/src/test/java/org/nuiton/wikitty/solr/test/SolrServerTest.java 2010-08-06 09:09:56 UTC (rev 235) +++ trunk/wikitty-solr-impl/src/test/java/org/nuiton/wikitty/solr/test/SolrServerTest.java 2010-08-06 09:30:04 UTC (rev 236) @@ -212,6 +212,7 @@ assertNotNull(result); } + @Ignore // FIXME 20100806 bleny randomly fail @Test public void testReader() throws Exception { SolrServer solrServer1 = getSolrServer();

1 0

r235 - trunk/wikitty-api/src/main/java/org/nuiton/wikitty
by bleny＠users.nuiton.org Aug. 6, 2010

Aug. 6, 2010

Author: bleny Date: 2010-08-06 11:09:56 +0200 (Fri, 06 Aug 2010) New Revision: 235 Url: http://nuiton.org/repositories/revision/wikitty/235 Log: rename cache policy option with explicit name Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyServiceCached.java Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyServiceCached.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyServiceCached.java 2010-08-06 09:09:15 UTC (rev 234) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyServiceCached.java 2010-08-06 09:09:56 UTC (rev 235) @@ -52,7 +52,7 @@ static public final String WIKITTY_CACHE_LISTENEVENTS_OPTION = "wikitty.service.cache.listenevents"; /** used as property name in props given in the constructor */ - public static final String WIKITTY_CACHE_POLICY_OPTION = "wikitty.service.cache.allwaysRestoreCopies"; + public static final String WIKITTY_CACHE_ALLWAYS_RESTORE_COPIES_POLICY_OPTION = "wikitty.service.cache.allwaysRestoreCopies"; /** Cache. */ protected WikittyCache cache = null; @@ -93,9 +93,9 @@ registerWikittyServiceListener(props); // reading configuration and set allwaysRestoreCopies accordingly - if (props != null && props.containsKey(WIKITTY_CACHE_POLICY_OPTION)) { + if (props != null && props.containsKey(WIKITTY_CACHE_ALLWAYS_RESTORE_COPIES_POLICY_OPTION)) { allwaysRestoreCopies = Boolean.parseBoolean( - props.getProperty(WIKITTY_CACHE_POLICY_OPTION)); + props.getProperty(WIKITTY_CACHE_ALLWAYS_RESTORE_COPIES_POLICY_OPTION)); } }

1 0

r234 - trunk/wikitty-api/src/test/java/org/nuiton/wikitty/cache
by bleny＠users.nuiton.org Aug. 6, 2010

Aug. 6, 2010

Author: bleny Date: 2010-08-06 11:09:15 +0200 (Fri, 06 Aug 2010) New Revision: 234 Url: http://nuiton.org/repositories/revision/wikitty/234 Log: rename cache policy option with explicit name Modified: trunk/wikitty-api/src/test/java/org/nuiton/wikitty/cache/WikittyServiceCachedTest.java Modified: trunk/wikitty-api/src/test/java/org/nuiton/wikitty/cache/WikittyServiceCachedTest.java =================================================================== --- trunk/wikitty-api/src/test/java/org/nuiton/wikitty/cache/WikittyServiceCachedTest.java 2010-08-05 16:10:03 UTC (rev 233) +++ trunk/wikitty-api/src/test/java/org/nuiton/wikitty/cache/WikittyServiceCachedTest.java 2010-08-06 09:09:15 UTC (rev 234) @@ -96,7 +96,7 @@ public void testRestoreAllwaysCopyPolicy() throws Exception { Properties props = new Properties(); - props.setProperty(WikittyServiceCached.WIKITTY_CACHE_POLICY_OPTION, "true"); + props.setProperty(WikittyServiceCached.WIKITTY_CACHE_ALLWAYS_RESTORE_COPIES_POLICY_OPTION, "true"); service = new WikittyServiceCached(new WikittyServiceInMemory(), props); token = service.login(null, null);

1 0

r233 - in trunk: wikitty-jdbc-impl/src/test/resources wikitty-solr-impl/src/test/resources
by echatellier＠users.nuiton.org Aug. 5, 2010

Aug. 5, 2010

Author: echatellier Date: 2010-08-05 18:10:03 +0200 (Thu, 05 Aug 2010) New Revision: 233 Url: http://nuiton.org/repositories/revision/wikitty/233 Log: Add jbossts file in each module that use jta Added: trunk/wikitty-jdbc-impl/src/test/resources/jbossts-properties.xml trunk/wikitty-solr-impl/src/test/resources/jbossts-properties.xml Added: trunk/wikitty-jdbc-impl/src/test/resources/jbossts-properties.xml =================================================================== --- trunk/wikitty-jdbc-impl/src/test/resources/jbossts-properties.xml (rev 0) +++ trunk/wikitty-jdbc-impl/src/test/resources/jbossts-properties.xml 2010-08-05 16:10:03 UTC (rev 233) @@ -0,0 +1,135 @@ +<?xml version="1.0" encoding="UTF-8"?> +<!DOCTYPE properties SYSTEM "http://java.sun.com/dtd/properties.dtd"> + +<properties> +  + +  + <entry key="LoggingEnvironmentBean.debugLevel">0x00000000</entry> + +  + <entry key="CoordinatorEnvironmentBean.commitOnePhase">YES</entry> + +  + <entry key="ObjectStoreEnvironmentBean.objectStoreDir">target${/}jbosstm</entry> + +  + <entry key="ObjectStoreEnvironmentBean.transactionSync">ON</entry> + +  + <entry key="CoreEnvironmentBean.nodeIdentifier">1</entry> + +  + <entry key="JTAEnvironmentBean.xaRecoveryNodes">1</entry> + +  + <entry key="CoreEnvironmentBean.socketProcessIdPort">0</entry> + + + +  + <entry key="RecoveryEnvironmentBean.recoveryExtensions"> + com.arjuna.ats.internal.arjuna.recovery.AtomicActionRecoveryModule + com.arjuna.ats.internal.txoj.recovery.TORecoveryModule + com.arjuna.ats.internal.jta.recovery.arjunacore.XARecoveryModule + </entry> + +  + <entry key="RecoveryEnvironmentBean.expiryScanners"> + com.arjuna.ats.internal.arjuna.recovery.ExpiredTransactionStatusManagerScanner + </entry> + +  + +  + <entry key="RecoveryEnvironmentBean.recoveryPort">4712</entry> + + <entry key="RecoveryEnvironmentBean.recoveryAddress"></entry> + +  + <entry key="RecoveryEnvironmentBean.transactionStatusManagerPort">0</entry> + +  + <entry key="RecoveryEnvironmentBean.transactionStatusManagerAddress"></entry> + +  + <entry key="RecoveryEnvironmentBean.recoveryListener">YES</entry> + +</properties> Added: trunk/wikitty-solr-impl/src/test/resources/jbossts-properties.xml =================================================================== --- trunk/wikitty-solr-impl/src/test/resources/jbossts-properties.xml (rev 0) +++ trunk/wikitty-solr-impl/src/test/resources/jbossts-properties.xml 2010-08-05 16:10:03 UTC (rev 233) @@ -0,0 +1,135 @@ +<?xml version="1.0" encoding="UTF-8"?> +<!DOCTYPE properties SYSTEM "http://java.sun.com/dtd/properties.dtd"> + +<properties> +  + +  + <entry key="LoggingEnvironmentBean.debugLevel">0x00000000</entry> + +  + <entry key="CoordinatorEnvironmentBean.commitOnePhase">YES</entry> + +  + <entry key="ObjectStoreEnvironmentBean.objectStoreDir">target${/}jbosstm</entry> + +  + <entry key="ObjectStoreEnvironmentBean.transactionSync">ON</entry> + +  + <entry key="CoreEnvironmentBean.nodeIdentifier">1</entry> + +  + <entry key="JTAEnvironmentBean.xaRecoveryNodes">1</entry> + +  + <entry key="CoreEnvironmentBean.socketProcessIdPort">0</entry> + + + +  + <entry key="RecoveryEnvironmentBean.recoveryExtensions"> + com.arjuna.ats.internal.arjuna.recovery.AtomicActionRecoveryModule + com.arjuna.ats.internal.txoj.recovery.TORecoveryModule + com.arjuna.ats.internal.jta.recovery.arjunacore.XARecoveryModule + </entry> + +  + <entry key="RecoveryEnvironmentBean.expiryScanners"> + com.arjuna.ats.internal.arjuna.recovery.ExpiredTransactionStatusManagerScanner + </entry> + +  + +  + <entry key="RecoveryEnvironmentBean.recoveryPort">4712</entry> + + <entry key="RecoveryEnvironmentBean.recoveryAddress"></entry> + +  + <entry key="RecoveryEnvironmentBean.transactionStatusManagerPort">0</entry> + +  + <entry key="RecoveryEnvironmentBean.transactionStatusManagerAddress"></entry> + +  + <entry key="RecoveryEnvironmentBean.recoveryListener">YES</entry> + +</properties>

1 0

r232 - trunk/wikitty-api/src/test/java/org/nuiton/wikitty/conform
by echatellier＠users.nuiton.org Aug. 5, 2010

Aug. 5, 2010

Author: echatellier Date: 2010-08-05 17:30:20 +0200 (Thu, 05 Aug 2010) New Revision: 232 Url: http://nuiton.org/repositories/revision/wikitty/232 Log: Move import to junit 4 Modified: trunk/wikitty-api/src/test/java/org/nuiton/wikitty/conform/StorageTest.java Modified: trunk/wikitty-api/src/test/java/org/nuiton/wikitty/conform/StorageTest.java =================================================================== --- trunk/wikitty-api/src/test/java/org/nuiton/wikitty/conform/StorageTest.java 2010-08-05 15:29:39 UTC (rev 231) +++ trunk/wikitty-api/src/test/java/org/nuiton/wikitty/conform/StorageTest.java 2010-08-05 15:30:20 UTC (rev 232) @@ -1,11 +1,11 @@ package org.nuiton.wikitty.conform; -import static junit.framework.Assert.assertEquals; -import static junit.framework.Assert.assertFalse; -import static junit.framework.Assert.assertNotNull; -import static junit.framework.Assert.assertNull; -import static junit.framework.Assert.assertTrue; -import static junit.framework.Assert.fail; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertFalse; +import static org.junit.Assert.assertNotNull; +import static org.junit.Assert.assertNull; +import static org.junit.Assert.assertTrue; +import static org.junit.Assert.fail; import java.util.ArrayList; import java.util.Arrays;

1 0

r231 - in trunk/wikitty-api/src: main/java/org/nuiton/wikitty test/java/org/nuiton/wikitty/conform
by echatellier＠users.nuiton.org Aug. 5, 2010

Aug. 5, 2010

Author: echatellier Date: 2010-08-05 17:29:39 +0200 (Thu, 05 Aug 2010) New Revision: 231 Url: http://nuiton.org/repositories/revision/wikitty/231 Log: Fix business entity created by introspection with wikitty contructor. Add test on that regression. Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyUtil.java trunk/wikitty-api/src/test/java/org/nuiton/wikitty/conform/StorageTest.java Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyUtil.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyUtil.java 2010-08-05 15:14:20 UTC (rev 230) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyUtil.java 2010-08-05 15:29:39 UTC (rev 231) @@ -672,7 +672,7 @@ try { // try to find constructor with wikitty argument Constructor<?> cons = clazzInstanciable - .getConstructor(WikittyImpl.class); + .getConstructor(Wikitty.class); Object[] parms = { null }; result = (E) cons.newInstance(parms); Modified: trunk/wikitty-api/src/test/java/org/nuiton/wikitty/conform/StorageTest.java =================================================================== --- trunk/wikitty-api/src/test/java/org/nuiton/wikitty/conform/StorageTest.java 2010-08-05 15:14:20 UTC (rev 230) +++ trunk/wikitty-api/src/test/java/org/nuiton/wikitty/conform/StorageTest.java 2010-08-05 15:29:39 UTC (rev 231) @@ -468,4 +468,17 @@ currentNode.addChildren( wikitty.getId() ); ws.store(null, currentNode.getWikitty()); } + + /** + * Test la creation d'une entité business par introspection avec un wikitty. + */ + @Test + public void testWikittyUtilNewInstanceWithWikitty() { + + Wikitty w = new WikittyImpl(); + w.addExtension(TreeNodeImpl.extensions); + + TreeNode treeNode = WikittyUtil.newInstance(null, ws, TreeNode.class, w); + Assert.assertNotNull(treeNode); + } }

1 0