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

r719 - in trunk: wikitty-api/src/main/java/org/nuiton/wikitty/services wikitty-api/src/main/java/org/nuiton/wikitty/storage wikitty-jdbc-impl/src/main/java/org/nuiton/wikitty/jdbc wikitty-jdbc-impl/src/main/resources
by bpoussin＠users.nuiton.org 22 Feb '11

22 Feb '11

Author: bpoussin Date: 2011-02-22 19:57:43 +0100 (Tue, 22 Feb 2011) New Revision: 719 Url: http://nuiton.org/repositories/revision/wikitty/719 Log: Evolution #1352: Add information in log during reindexation process add getStatistic method on WikittyStorage interface Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyServiceStorage.java trunk/wikitty-api/src/main/java/org/nuiton/wikitty/storage/WikittyStorage.java trunk/wikitty-api/src/main/java/org/nuiton/wikitty/storage/WikittyStorageInMemory.java trunk/wikitty-jdbc-impl/src/main/java/org/nuiton/wikitty/jdbc/WikittyJDBCUtil.java trunk/wikitty-jdbc-impl/src/main/java/org/nuiton/wikitty/jdbc/WikittyStorageJDBC.java trunk/wikitty-jdbc-impl/src/main/resources/wikitty-jdbc-query.properties Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyServiceStorage.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyServiceStorage.java 2011-02-21 17:49:33 UTC (rev 718) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyServiceStorage.java 2011-02-22 18:57:43 UTC (rev 719) @@ -928,6 +928,7 @@ boolean txBeginHere = false; try { + log.info("Reindexation started"); final int numberForCommit = 1000; // TODO poussin 20110221 configurable variable final WikittySearchEngine searchEngine = getSearchEngine(); final List<Wikitty> wikitties = new ArrayList<Wikitty>(numberForCommit); @@ -944,7 +945,14 @@ tx.begin(); txBeginHere = true; } + + WikittyStorage.DataStatistic stat = getWikittyStorage().getDataStatistic(tx); + final long max = stat.getActiveWikitties(); + + log.info("Reindexation 0%"); getWikittyStorage().scanWikitties(tx, new WikittyStorage.Scanner() { + long lastPurcent = -1; + long current = 0; int count = 0; @Override @@ -952,6 +960,7 @@ Wikitty wikitty = restore(securityToken, wikittyId); Date deleteDate = wikitty.getDeleteDate(); if(deleteDate == null) { + current++; count ++; wikitties.add(wikitty); @@ -963,6 +972,16 @@ count = 0; wikitties.clear(); tx.begin(); + // si on a les statistiques a disposition on affiche + // l'etat d'avancement dans les logs + if (max > 0) { + long purcent = current * 100 / max; + // on affiche l'etat d'avancement environ tous les 10% + if (purcent / 10 > lastPurcent) { + lastPurcent = purcent / 10; + log.info(String.format("Reindexation %s%%", purcent)); + } + } } } } @@ -973,6 +992,7 @@ if (txBeginHere) { tx.commit(); } + log.info("Reindexation finished"); } catch (WikittyException ex) { if (tx != null && tx.isStarted()) { tx.rollback(); Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/storage/WikittyStorage.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/storage/WikittyStorage.java 2011-02-21 17:49:33 UTC (rev 718) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/storage/WikittyStorage.java 2011-02-22 18:57:43 UTC (rev 719) @@ -129,4 +129,58 @@ */ public WikittyEvent clear(WikittyTransaction transaction); + /** + * Return some statistique about the data + * @param transaction + * @return + * @since 3.0.5 + */ + public DataStatistic getDataStatistic(WikittyTransaction transaction); + + /** + * @see #getDataStatistic(org.nuiton.wikitty.services.WikittyTransaction) + * @since 3.0.5 + */ + static public class DataStatistic { + protected long activeWikitties = -1; + protected long deletedWikitties = -1; + + /** + * Use this constructor when your implementation don't support statistic + */ + public DataStatistic() { + } + + public DataStatistic(long activeWikitties, long deletedWikitties) { + this.activeWikitties = activeWikitties; + this.deletedWikitties = deletedWikitties; + } + /** + * Return number of non deleted wikitties, or negative value if this + * information can't be retrieved + * @return + */ + public long getActiveWikitties() { + return activeWikitties; + } + + /** + * Return number of deleted wikitties, or negative value if this + * information can't be retrieved + * @return + */ + public long getDeletedWikitties() { + return deletedWikitties; + } + + /** + * Return number of non deleted and deleted wikitties, or negative value + * if this information can't be retrieved + * @return + */ + public long getTotalWikitties() { + long result = getActiveWikitties() + getDeletedWikitties(); + return result; + } + } } Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/storage/WikittyStorageInMemory.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/storage/WikittyStorageInMemory.java 2011-02-21 17:49:33 UTC (rev 718) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/storage/WikittyStorageInMemory.java 2011-02-22 18:57:43 UTC (rev 719) @@ -175,4 +175,21 @@ result.addType(WikittyEvent.WikittyEventType.CLEAR_WIKITTY); return result; } + + @Override + public DataStatistic getDataStatistic(WikittyTransaction transaction) { + long active = 0; + long deleted = 0; + Collection<Wikitty> all = wikitties.values(); + for (Wikitty wikitty : all) { + if (wikitty.isDeleted()) { + deleted++; + } else { + active++; + } + } + DataStatistic result = new DataStatistic(active, deleted); + return result; + } + } Modified: trunk/wikitty-jdbc-impl/src/main/java/org/nuiton/wikitty/jdbc/WikittyJDBCUtil.java =================================================================== --- trunk/wikitty-jdbc-impl/src/main/java/org/nuiton/wikitty/jdbc/WikittyJDBCUtil.java 2011-02-21 17:49:33 UTC (rev 718) +++ trunk/wikitty-jdbc-impl/src/main/java/org/nuiton/wikitty/jdbc/WikittyJDBCUtil.java 2011-02-22 18:57:43 UTC (rev 719) @@ -100,6 +100,10 @@ static final public String QUERY_SELECT = "jdbc.queries.select"; /** basic selection without where clause query property name */ static final public String QUERY_SELECT_NOTDELETED = "jdbc.queries.select.notdeleted"; + /** count number of active (not deleted wikitties) */ + static final public String QUERY_COUNT_ACTIVE_WIKITTY = "jdbc.queries.select.count.active.wikitties"; + /** count number of deleted */ + static final public String QUERY_COUNT_DELETED_WIKITTY = "jdbc.queries.select.count.deleted.wikitties"; /** basic selection with where clause query property name */ static final public String QUERY_SELECT_WHERE = "jdbc.queries.select.where"; /** basic selection with where clause query property name */ Modified: trunk/wikitty-jdbc-impl/src/main/java/org/nuiton/wikitty/jdbc/WikittyStorageJDBC.java =================================================================== --- trunk/wikitty-jdbc-impl/src/main/java/org/nuiton/wikitty/jdbc/WikittyStorageJDBC.java 2011-02-21 17:49:33 UTC (rev 718) +++ trunk/wikitty-jdbc-impl/src/main/java/org/nuiton/wikitty/jdbc/WikittyStorageJDBC.java 2011-02-22 18:57:43 UTC (rev 719) @@ -48,6 +48,8 @@ import static org.nuiton.wikitty.jdbc.WikittyJDBCUtil.QUERY_INSERT_WIKITTY_ADMIN; import static org.nuiton.wikitty.jdbc.WikittyJDBCUtil.QUERY_INSERT_WIKITTY_DATA; import static org.nuiton.wikitty.jdbc.WikittyJDBCUtil.QUERY_SELECT_NOTDELETED; +import static org.nuiton.wikitty.jdbc.WikittyJDBCUtil.QUERY_COUNT_ACTIVE_WIKITTY; +import static org.nuiton.wikitty.jdbc.WikittyJDBCUtil.QUERY_COUNT_DELETED_WIKITTY; import static org.nuiton.wikitty.jdbc.WikittyJDBCUtil.QUERY_SELECT_WHERE; import static org.nuiton.wikitty.jdbc.WikittyJDBCUtil.QUERY_SELECT_TWO_WHERE; import static org.nuiton.wikitty.jdbc.WikittyJDBCUtil.QUERY_SELECT_WHERE_NOTDELETED; @@ -621,4 +623,31 @@ WikittyJDBCUtil.closeQuietly(connection); } } + + @Override + public DataStatistic getDataStatistic(WikittyTransaction transaction) { + DataStatistic result;; + Connection connection = WikittyJDBCUtil.getConnection(config); + try { + Statement statement = connection.createStatement(); + + ResultSet resultSet = statement.executeQuery( + jdbcQuery.getProperty(QUERY_COUNT_ACTIVE_WIKITTY)); + long activeWikitties = resultSet.getLong(1); + + resultSet = statement.executeQuery( + jdbcQuery.getProperty(QUERY_COUNT_DELETED_WIKITTY)); + long deletedWikitties = resultSet.getLong(1); + + result = new DataStatistic(activeWikitties, deletedWikitties); + } catch (SQLException eee) { + result = new DataStatistic(); + log.warn("Can't retrieve statisticn data", eee); + } finally { + WikittyJDBCUtil.closeQuietly(connection); + } + return result; + } + + } Modified: trunk/wikitty-jdbc-impl/src/main/resources/wikitty-jdbc-query.properties =================================================================== --- trunk/wikitty-jdbc-impl/src/main/resources/wikitty-jdbc-query.properties 2011-02-21 17:49:33 UTC (rev 718) +++ trunk/wikitty-jdbc-impl/src/main/resources/wikitty-jdbc-query.properties 2011-02-22 18:57:43 UTC (rev 719) @@ -77,6 +77,10 @@ jdbc.queries.select=SELECT %s FROM %s; #not deleted data selection query without where clause jdbc.queries.select.notdeleted=SELECT %s FROM %s WHERE deletionDate IS NULL; +#count number of active (not deleted wikitties) +jdbc.queries.select.count.active.wikitties=SELECT count(id) FROM wikitty_admin WHERE deletiondate IS NULL; +#count number of deleted wikitties +jdbc.queries.select.count.deleted.wikitties=SELECT count(id) FROM wikitty_admin WHERE deletiondate IS NOT NULL; #basic selection query with where clause jdbc.queries.select.where=SELECT %s FROM %s WHERE %s = ?; #basic 2 selections query with where clause

1 0

r718 - trunk/wikitty-solr-impl/src/main/java/org/nuiton/wikitty/storage/solr
by bpoussin＠users.nuiton.org 21 Feb '11

21 Feb '11

Author: bpoussin Date: 2011-02-21 18:49:33 +0100 (Mon, 21 Feb 2011) New Revision: 718 Url: http://nuiton.org/repositories/revision/wikitty/718 Log: Anomalie #1351: Bug in reindexation with new SolR indexation implementation when syncSearchEngine Modified: trunk/wikitty-solr-impl/src/main/java/org/nuiton/wikitty/storage/solr/AttachmentInTree.java trunk/wikitty-solr-impl/src/main/java/org/nuiton/wikitty/storage/solr/WikittySearchEngineSolr.java Modified: trunk/wikitty-solr-impl/src/main/java/org/nuiton/wikitty/storage/solr/AttachmentInTree.java =================================================================== --- trunk/wikitty-solr-impl/src/main/java/org/nuiton/wikitty/storage/solr/AttachmentInTree.java 2011-02-21 17:45:43 UTC (rev 717) +++ trunk/wikitty-solr-impl/src/main/java/org/nuiton/wikitty/storage/solr/AttachmentInTree.java 2011-02-21 17:49:33 UTC (rev 718) @@ -145,8 +145,10 @@ } /** + * Ajout les attachment du TreeNode passe en parametre via son Id + * * @param id TreeNode id - * @param ids attachment id + * @param attId attachment id */ public void add(String id, Collection<String> attId) { if (attId != null && !attId.isEmpty()) { @@ -155,7 +157,21 @@ } } + /** + * Ajout l'attachment du TreeNode passe en parametre via son Id + * + * @param id TreeNode id + * @param attId attachment id + */ + public void add(String id, String attId) { + if (attId != null) { + attachmentAddedInTree.get(id).add(attId); + allAttachmentToIndex.add(attId); + } + } + /** + * Ajout les attachment du TreeNode passe en parametre sous forme de doc Solr * @param doc TreeNode document representation */ public void add(SolrDocument doc) { @@ -168,8 +184,31 @@ } /** + * Ajout les attachment du TreeNode passe en parametre sous forme de doc Solr + * Mais seulement si l'attachment est aussi dans la liste restriction + * * @param doc TreeNode document representation + * @param restriction la liste accepte de wikitty a ajouter */ + public void add(SolrDocument doc, Set<String> restriction) { + String id = (String)doc.getFieldValue(WikittySolrConstant.SOLR_ID); + + Collection att = doc.getFieldValues(SolrUtil.getSolrFieldName( + WikittyTreeNode.FQ_FIELD_WIKITTYTREENODE_ATTACHMENT, + TYPE.WIKITTY)); + if (att != null) { + for (Object attId : att) { + if (restriction.contains(attId)) { + add(id, (String)attId); + } + } + } + } + + /** + * Ajout les attachment du TreeNode passe en parametre sous forme de doc Solr + * @param doc TreeNode document representation + */ public void add(SolrInputDocument doc) { String id = (String)doc.getFieldValue(WikittySolrConstant.SOLR_ID); @@ -180,6 +219,7 @@ } /** + * Ajout les attachment du TreeNode passe en parametre sous forme de Wikitty * @param doc TreeNode document representation */ public void add(Wikitty w) { Modified: trunk/wikitty-solr-impl/src/main/java/org/nuiton/wikitty/storage/solr/WikittySearchEngineSolr.java =================================================================== --- trunk/wikitty-solr-impl/src/main/java/org/nuiton/wikitty/storage/solr/WikittySearchEngineSolr.java 2011-02-21 17:45:43 UTC (rev 717) +++ trunk/wikitty-solr-impl/src/main/java/org/nuiton/wikitty/storage/solr/WikittySearchEngineSolr.java 2011-02-21 17:49:33 UTC (rev 718) @@ -161,6 +161,9 @@ Map<String, Wikitty> allWikitties = new HashMap<String, Wikitty>(); // les ids des wikitties en parametre reellement modifier (a reindexer) Set<String> dirtyObject = new HashSet<String>(); + // les ids des wikitties qui n'appartiennent pas a un TreeNode passé en + // parametre, mais qui pourrait appartenir a un treenode deja indexe + Set<String> possibleAttachment = new HashSet<String>(); // les ids des TreeNodes dont le champs parent a change (est aussi // contenu dans dirtyObject Set<String> dirtyParent = new HashSet<String>(); @@ -176,8 +179,8 @@ // s'il y a au moins un champs a reindexer ou que l'objet // n'a jamais ete sauve (1 > version) dirtyObject.add(w.getId()); - if (WikittyTreeNodeHelper.hasExtension(w) && - (w.getDirty().contains(WikittyTreeNode.FQ_FIELD_WIKITTYTREENODE_PARENT) + if (WikittyTreeNodeHelper.hasExtension(w) && (force + ||w.getDirty().contains(WikittyTreeNode.FQ_FIELD_WIKITTYTREENODE_PARENT) || null == WikittyTreeNodeHelper.getParent(w))) { // si le pere a change // ou qu'il est null (creation d'un nouvel arbre) @@ -205,16 +208,21 @@ // // Phase 1: on indexe les objets passe en paremetre, on copie si - // besoin #tree.attached et #tree.* des TreeNode dont - // leur champs parent n'a pas ete modifie, et dans ce cas - // on collecte les modif d'attachments des TreeNode + // besoin #tree.attached des wikitties et #tree.* des + // TreeNode dont leur champs parent n'a pas ete modifie, et + // dans ce cas on collecte les modif d'attachments des TreeNode // for(String id : dirtyObject) { Wikitty w = allWikitties.get(id); SolrDocument oldDoc = dirtyObjectDoc.get(id); SolrInputDocument doc = createIndexDocument(w); - if (oldDoc != null) { + if (oldDoc == null) { + // Le document n'existait pas, mais peut-etre qu'il etait + // dans la liste des attachments d'un TreeNode. Il faudra + // donc rechercher si c'est le cas et l'indexer convenablement + possibleAttachment.add(id); + } else { // copy des champs #tree.attached SolrUtil.copySolrDocument(oldDoc, doc, TREENODE_ATTACHED + ".*"); if (WikittyTreeNodeHelper.hasExtension(w) @@ -315,9 +323,15 @@ // Phase 3: on reindexe les attachments qui en ont besoin // - // on passe null pour tree, car tous les noeuds doivent etre dans - // solrResource - addTreeIndexField(solrResource, null, attachmentInTree); + // on ajoute les attachment au TreeNode qui ont ete store, sans + // leurs attachments (ni en base, ni dans le meme store) + Map<String, SolrDocument> tree = + SolrUtil.findAllByAttachment(solrServer, possibleAttachment); + for (SolrDocument doc : tree.values()) { + attachmentInTree.add(doc, possibleAttachment); + } + + addTreeIndexField(solrResource, tree, attachmentInTree); } catch (Exception eee) { throw new WikittyException("Can't store wikitty", eee); } @@ -369,6 +383,10 @@ attachmentInTree.remove(doc); } // recherche tous les noeuds sur lequel est attache cet objet + + // TODO poussin 20110217 optimiser cette requete pour la faire + // hors de la boucle et pour tous les doc a supprimer en meme + // temps (donc 1 seul requete au lieu de N) Set<String> treeNodeIds = SolrUtil.getAttachedTreeNode(doc); treeNodeAttachmentRemovedId.addAll(treeNodeIds); @@ -528,12 +546,12 @@ } /** - * Update attaced extra field on all objects passed in argument allAttachmentToIndex + * Update attached extra field on all objects passed in argument + * allAttachmentToIndex * * @param solrResource must contains reindexed TreeNode, that contains attachment - * @param allAttachmentToIndex id of object to update - * @param attachmentRemovedInTree index to remove - * @param attachmentAddedInTree index to add + * @param tree solr document for some TreeNode (used when TreeNode not find in solrResource) + * @param attachmentInTree attachment added and removed from TreeNode */ protected void addTreeIndexField(SolrResource solrResource, Map<String, SolrDocument> tree, AttachmentInTree attachmentInTree) { @@ -566,18 +584,29 @@ log.error("SolR doc not found in Transaction or in tree." + "This is a bug !!!"); } + // add tree indexation on all attachments for this treeNodeId for (String attId : attachmentInTree.getAdded().get(treeNodeId)) { SolrDocument oldDoc = attachments.get(attId); SolrInputDocument doc = solrResource.getAddedDoc(attId); - if (doc == null) { - doc = new SolrInputDocument(); - SolrUtil.copySolrDocument(oldDoc, doc); - solrResource.addDoc(attId, doc); + // il faut que oldDoc ou doc soit different de null pour + // pouvoir ajouter l'indexation d'arbre. Le cas on les deux + // sont nuls arrivent lorsqu'on demande la sauvegarde d'un + // TreeNode alors qu'on a pas encore sauve les attachments + // (ex: durant un syncSearchEngine ou l'on demande une + // reindexation totale). Ceci n'est pas un probleme car + // les attachments seront convenablement indexes lorsqu'ils + // seront ajoutes + if (oldDoc != null || doc != null) { + if (doc == null) { + doc = new SolrInputDocument(); + SolrUtil.copySolrDocument(oldDoc, doc); + solrResource.addDoc(attId, doc); + } + doc.removeField(TREENODE_ATTACHED + treeNodeId); + for (Object id : treeNodeParents) { + doc.addField(TREENODE_ATTACHED + treeNodeId, id); + } } - doc.removeField(TREENODE_ATTACHED + treeNodeId); - for (Object id : treeNodeParents) { - doc.addField(TREENODE_ATTACHED + treeNodeId, id); - } } } }

1 0

r717 - trunk/wikitty-api/src/test/java/org/nuiton/wikitty/api
by bpoussin＠users.nuiton.org 21 Feb '11

21 Feb '11

Author: bpoussin Date: 2011-02-21 18:45:43 +0100 (Mon, 21 Feb 2011) New Revision: 717 Url: http://nuiton.org/repositories/revision/wikitty/717 Log: use Integer.valueOf (better than new Integer) Modified: trunk/wikitty-api/src/test/java/org/nuiton/wikitty/api/CommonTest.java Modified: trunk/wikitty-api/src/test/java/org/nuiton/wikitty/api/CommonTest.java =================================================================== --- trunk/wikitty-api/src/test/java/org/nuiton/wikitty/api/CommonTest.java 2011-02-21 17:45:16 UTC (rev 716) +++ trunk/wikitty-api/src/test/java/org/nuiton/wikitty/api/CommonTest.java 2011-02-21 17:45:43 UTC (rev 717) @@ -250,7 +250,7 @@ } }); // test java.lang.Integer => int - w.setField(StorageTest.EXTNAME, "fieldName1", new Integer(123)); + w.setField(StorageTest.EXTNAME, "fieldName1", Integer.valueOf(123)); Assert.assertEquals(123, w.getFieldAsInt(StorageTest.EXTNAME, "fieldName1") ); // test String => int w.setField(StorageTest.EXTNAME, "fieldName1", "123");

1 0

r716 - in trunk/wikitty-api/src/main/java/org/nuiton/wikitty: . search
by bpoussin＠users.nuiton.org 21 Feb '11

21 Feb '11

Author: bpoussin Date: 2011-02-21 18:45:16 +0100 (Mon, 21 Feb 2011) New Revision: 716 Url: http://nuiton.org/repositories/revision/wikitty/716 Log: fix small bug Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyUtil.java trunk/wikitty-api/src/main/java/org/nuiton/wikitty/search/PagedResult.java Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyUtil.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyUtil.java 2011-02-21 17:44:48 UTC (rev 715) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyUtil.java 2011-02-21 17:45:16 UTC (rev 716) @@ -870,7 +870,7 @@ */ static public String genSecurityTokenId() { String result = "_" + Math.abs(Math.random()); - result.replace(".", ""); // on supprime le '.' dans le nombre aleatoire + result = result.replace(".", ""); // on supprime le '.' dans le nombre aleatoire result = genUID() + result; return result; } Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/search/PagedResult.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/search/PagedResult.java 2011-02-21 17:44:48 UTC (rev 715) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/search/PagedResult.java 2011-02-21 17:45:16 UTC (rev 716) @@ -179,7 +179,7 @@ w = ((BusinessEntityImpl) t).getWikitty(); } else { throw new WikittyException(String.format( - "Illegal object result class '%s' can't convert it to wikitty", + "Illegal object result class '%s' can't convert it to class '%s'", t.getClass().getName(), clazz.getName())); }

1 0

r715 - in trunk/wikitty-api/src/main/java/org/nuiton/wikitty: . services
by bpoussin＠users.nuiton.org 21 Feb '11

21 Feb '11

Author: bpoussin Date: 2011-02-21 18:44:48 +0100 (Mon, 21 Feb 2011) New Revision: 715 Url: http://nuiton.org/repositories/revision/wikitty/715 Log: add todo and javadoc Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyProxy.java trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyServiceStorage.java Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyProxy.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyProxy.java 2011-02-21 16:42:36 UTC (rev 714) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyProxy.java 2011-02-21 17:44:48 UTC (rev 715) @@ -173,7 +173,7 @@ public <E extends BusinessEntity> E store(E e) { Wikitty w = ((BusinessEntityImpl)e).getWikitty(); - w = store(w); + store(w); return e; } @@ -756,7 +756,12 @@ } /** + * Recupere une portion d'arbre a partir de l'id passer en parametre. L'id + * doit etre celui d'un WikittyTreeNode. Ce WikittyTreeNode est alors le + * root de l'arbre retourne. + * * Return just wikitty Id in result + * * @param wikittyId * @param depth * @param count @@ -826,7 +831,7 @@ * @param wikittyId * @param filter * @return - * @deprecated since 3.1: use {@link #findTreeNode} or open new ticket with your need + * @deprecated since 3.1: use {@link #findTreeNode(java.lang.Class, java.lang.String, int, boolean, org.nuiton.wikitty.search.Criteria)} or open new ticket with your need */ @Deprecated public <E extends BusinessEntity> Map.Entry<E, Integer> restoreNode( @@ -852,7 +857,7 @@ * @param wikittyId * @param filter * @param checkExtension not used - * @deprecated since 3.1: use {@link #findTreeNode} or open new ticket with your need + * @deprecated since 3.1: use {@link #findTreeNode(java.lang.Class, java.lang.String, int, boolean, org.nuiton.wikitty.search.Criteria)} or open new ticket with your need */ @Deprecated public <E extends BusinessEntity> Map.Entry<E, Integer> restoreNode( @@ -869,7 +874,7 @@ * @param filter * @return * - * @deprecated since 3.1: use {@link #findTreeNode} or open new ticket with your need + * @deprecated since 3.1: use {@link #findTreeNode(java.lang.Class, java.lang.String, int, boolean, org.nuiton.wikitty.search.Criteria) } or open new ticket with your need */ @Deprecated public <E extends BusinessEntity> Map<E, Integer> findTreeNode( @@ -892,7 +897,7 @@ * * ATTENTION: extension is never checked. * - * @deprecated since 3.1: use {@link #findTreeNode} or open new ticket with your need + * @deprecated since 3.1: use {@link #findTreeNode(java.lang.Class, java.lang.String, int, boolean, org.nuiton.wikitty.search.Criteria) } or open new ticket with your need */ @Deprecated public <E extends BusinessEntity> Map<E, Integer> findTreeNode( Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyServiceStorage.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyServiceStorage.java 2011-02-21 16:42:36 UTC (rev 714) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyServiceStorage.java 2011-02-21 17:44:48 UTC (rev 715) @@ -928,7 +928,7 @@ boolean txBeginHere = false; try { - final int numberForCommit = 1000; + final int numberForCommit = 1000; // TODO poussin 20110221 configurable variable final WikittySearchEngine searchEngine = getSearchEngine(); final List<Wikitty> wikitties = new ArrayList<Wikitty>(numberForCommit);

1 0

r714 - in trunk: wikitty-api/src/main/java/org/nuiton/wikitty/services wikitty-jdbc-impl/src/main/java/org/nuiton/wikitty/jdbc wikitty-jdbc-impl/src/main/resources
by echatellier＠users.nuiton.org 21 Feb '11

21 Feb '11

Author: echatellier Date: 2011-02-21 17:42:36 +0100 (Mon, 21 Feb 2011) New Revision: 714 Url: http://nuiton.org/repositories/revision/wikitty/714 Log: #1348 : Deleted extension not removed from wikitty service cache Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyCacheJCS.java trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyEvent.java trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyServiceCached.java trunk/wikitty-jdbc-impl/src/main/java/org/nuiton/wikitty/jdbc/WikittyExtensionStorageJDBC.java trunk/wikitty-jdbc-impl/src/main/java/org/nuiton/wikitty/jdbc/WikittyJDBCUtil.java trunk/wikitty-jdbc-impl/src/main/resources/wikitty-jdbc-query.properties Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyCacheJCS.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyCacheJCS.java 2011-02-21 14:17:58 UTC (rev 713) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyCacheJCS.java 2011-02-21 16:42:36 UTC (rev 714) @@ -192,14 +192,14 @@ } /** - * Return wikitty object if is in the cache, null otherwize. + * Return wikitty object if is in the cache, null otherwise. * - * @param id + * @param extId extension id * @return wikitty object or null */ @Override - public WikittyExtension getExtension(String id) { - WikittyExtension result = (WikittyExtension)extensionCache.get(id); + public WikittyExtension getExtension(String extId) { + WikittyExtension result = (WikittyExtension)extensionCache.get(extId); return result; } @@ -222,14 +222,18 @@ /** * Remove wikitty from cache. * - * @param id wikitty id to remove + * @param extId wikitty id to remove */ @Override - public void removeExtension(String id) { + public void removeExtension(String extId) { try { - extensionCache.remove(id); + extensionCache.remove(extId); + + if (log.isDebugEnabled()) { + log.debug("Remove extension from JCS cache " + extId); + } } catch (CacheException eee) { - log.error(String.format("Can't remove extensions %s in cache", id), eee); + log.error(String.format("Can't remove extensions %s in cache", extId), eee); } } 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 2011-02-21 14:17:58 UTC (rev 713) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyEvent.java 2011-02-21 16:42:36 UTC (rev 714) @@ -34,6 +34,8 @@ import java.util.LinkedHashSet; import java.util.Map; import java.util.Set; + +import org.nuiton.wikitty.WikittyService; import org.nuiton.wikitty.entities.Wikitty; import org.nuiton.wikitty.entities.WikittyExtension; @@ -99,14 +101,13 @@ /** Use by PUT_EXTENSION, all extensions added */ protected Map<String, WikittyExtension> extensions; - /** Use by REMOVE_EXTENSION, all extensions name deleted */ + /** Use by REMOVE_EXTENSION, all extensions id deleted */ protected Set<String> deletedExtensions; /** * Constructor with source {@link WikittyService}. * * @param source wikitty service - * @param eventId unique event id */ public WikittyEvent(Object source) { super(source); @@ -115,7 +116,8 @@ } /** - * Return time of event creation + * Return time of event creation. + * * @return */ public long getTime() { @@ -213,12 +215,12 @@ return deletedExtensions; } - public void addDeletedExtension(String extName) { + public void addDeletedExtension(String id) { if (deletedExtensions == null) { deletedExtensions = new LinkedHashSet<String>(); addType(WikittyEventType.REMOVE_EXTENSION); } - deletedExtensions.add(extName); + deletedExtensions.add(id); } Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyServiceCached.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyServiceCached.java 2011-02-21 14:17:58 UTC (rev 713) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyServiceCached.java 2011-02-21 16:42:36 UTC (rev 714) @@ -5,7 +5,7 @@ * $Id$ * $HeadURL$ * %% - * Copyright (C) 2009 - 2010 CodeLutin, Benjamin Poussin + * Copyright (C) 2009 - 2011 CodeLutin, Benjamin Poussin, Chatellier Eric * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU Lesser General Public License as @@ -81,12 +81,13 @@ /** * Constructor with configuration. * + * @param config config * @param ws delegate service - * @param props properties (can be null) + * @param cache wikity cache */ public WikittyServiceCached(ApplicationConfig config, WikittyService ws, WikittyCache cache) { super(ws); - // this.ws = ws; + // this.ws = ws; this.cache = cache; if (cache == null) { @@ -134,7 +135,8 @@ } /** - * Retourne le nombre de solicitation du cache + * Retourne le nombre de solicitation du cache. + * * @return */ public long getAsked() { @@ -143,14 +145,17 @@ /** * Retourne le nombre de fois que l'element dans le cache demande n'y etait - * pas + * pas. + * * @return */ public long getMissed() { return missed; } - /** wrap the wikitty or copy it according to allwaysRestoreCopies value */ + /** + * Wrap the wikitty or copy it according to allwaysRestoreCopies value. + */ protected Wikitty wrapWikitty(Wikitty wikitty) { Wikitty result = null; @@ -225,19 +230,19 @@ * Only WikittyImpl can be put in real cache implementation. If argument * is WikittyCopyOnWrite, we must take internal wikitty to put in cache */ - protected void cachePutExtension(WikittyExtension w) { - if (w != null) { - if (!cache.existsExtension(w.getId())) { - cache.putExtension(w); + protected void cachePutExtension(WikittyExtension ext) { + if (ext != null) { + if (!cache.existsExtension(ext.getId())) { + cache.putExtension(ext); if (log.isTraceEnabled()) { log.trace(String.format( - "Replace cached wikitty extension '%s'", w.getId())); + "Replace cached wikitty extension '%s'", ext.getId())); } } else { if (log.isTraceEnabled()) { log.trace(String.format( - "Ignoring put wikitty extension for '%s'",w.getId())); + "Ignoring put wikitty extension for '%s'", ext.getId())); } } } @@ -259,9 +264,12 @@ } } - protected void cacheRemoveExtension(String id) { - if (id != null) { - cache.removeWikitty(id); + protected void cacheRemoveExtension(String extId) { + if (extId != null) { + if (log.isDebugEnabled()) { + log.debug("Remove extension from cache " + extId); + } + cache.removeExtension(extId); } } @@ -271,9 +279,9 @@ } } - protected void cacheRemoveExtension(Collection<String> ids) { - for (String id : ids) { - cacheRemoveWikitty(id); + protected void cacheRemoveExtension(Collection<String> extIds) { + for (String extId : extIds) { + cacheRemoveExtension(extId); } } @@ -484,6 +492,11 @@ * @param e */ protected void processEvent(WikittyEvent e) { + + if (log.isDebugEnabled()) { + log.debug("Cache receive event : " + e); + } + // check clear must be the first, if event have clear and other type // clear is all time play first if (e.getType().contains( Modified: trunk/wikitty-jdbc-impl/src/main/java/org/nuiton/wikitty/jdbc/WikittyExtensionStorageJDBC.java =================================================================== --- trunk/wikitty-jdbc-impl/src/main/java/org/nuiton/wikitty/jdbc/WikittyExtensionStorageJDBC.java 2011-02-21 14:17:58 UTC (rev 713) +++ trunk/wikitty-jdbc-impl/src/main/java/org/nuiton/wikitty/jdbc/WikittyExtensionStorageJDBC.java 2011-02-21 16:42:36 UTC (rev 714) @@ -5,7 +5,7 @@ * $Id$ * $HeadURL$ * %% - * Copyright (C) 2010 CodeLutin, Benjamin Poussin + * Copyright (C) 2010 - 2011 CodeLutin, Benjamin Poussin, Chatellier Eric * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU Lesser General Public License as @@ -40,6 +40,7 @@ import static org.nuiton.wikitty.jdbc.WikittyJDBCUtil.QUERY_INSERT_EXTENSION_ADMIN; import static org.nuiton.wikitty.jdbc.WikittyJDBCUtil.QUERY_INSERT_EXTENSION_DATA; import static org.nuiton.wikitty.jdbc.WikittyJDBCUtil.QUERY_SELECT; +import static org.nuiton.wikitty.jdbc.WikittyJDBCUtil.QUERY_SELECT_EXTENSION_ADMIN; import static org.nuiton.wikitty.jdbc.WikittyJDBCUtil.QUERY_SELECT_WHERE; import static org.nuiton.wikitty.jdbc.WikittyJDBCUtil.TABLE_EXTENSION_ADMIN; import static org.nuiton.wikitty.jdbc.WikittyJDBCUtil.TABLE_EXTENSION_DATA; @@ -186,14 +187,28 @@ // # is ESCAPE caractere (see wikitty-jdbc-query file) // we escape [ and ] because some database use it as special char name = name + "#[%#]"; + + // select all wikitty id for event to contains + // id instead of only name (usefull for cache to remove cached + // extension id) + String q = String.format(jdbcQuery.getProperty(QUERY_SELECT_EXTENSION_ADMIN), COL_ID); + PreparedStatement statement = connection.prepareStatement(q); + statement.setString(1, name); + ResultSet resultSet = statement.executeQuery(); + while (resultSet.next()) { + String id = resultSet.getString(COL_ID); + result.addDeletedExtension(id); + } + // delete extension in admin and data WikittyJDBCUtil.doQuery(connection, jdbcQuery.getProperty( QUERY_DELETE_EXTENSION_DATA), name); WikittyJDBCUtil.doQuery(connection, jdbcQuery.getProperty( QUERY_DELETE_EXTENSION_ADMIN), name); - result.addDeletedExtension(name); - } + + // FIXME echatellier 20110221 : clear cache !!! + lastVersion = null; return result; } catch (SQLException eee) { Modified: trunk/wikitty-jdbc-impl/src/main/java/org/nuiton/wikitty/jdbc/WikittyJDBCUtil.java =================================================================== --- trunk/wikitty-jdbc-impl/src/main/java/org/nuiton/wikitty/jdbc/WikittyJDBCUtil.java 2011-02-21 14:17:58 UTC (rev 713) +++ trunk/wikitty-jdbc-impl/src/main/java/org/nuiton/wikitty/jdbc/WikittyJDBCUtil.java 2011-02-21 16:42:36 UTC (rev 714) @@ -5,7 +5,7 @@ * $Id$ * $HeadURL$ * %% - * Copyright (C) 2010 CodeLutin, Benjamin Poussin + * Copyright (C) 2010 - 2011 CodeLutin, Benjamin Poussin, Chatellier Eric * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU Lesser General Public License as @@ -141,6 +141,7 @@ static final public String QUERY_CLEAR_WIKITTY = "jdbc.queries.clear.wikitty"; /** clear extension query property name */ + static final public String QUERY_SELECT_EXTENSION_ADMIN = "jdbc.queries.select.extension.admin"; static final public String QUERY_DELETE_EXTENSION_ADMIN = "jdbc.queries.delete.extension.admin"; static final public String QUERY_DELETE_EXTENSION_DATA = "jdbc.queries.delete.extension.data"; Modified: trunk/wikitty-jdbc-impl/src/main/resources/wikitty-jdbc-query.properties =================================================================== --- trunk/wikitty-jdbc-impl/src/main/resources/wikitty-jdbc-query.properties 2011-02-21 14:17:58 UTC (rev 713) +++ trunk/wikitty-jdbc-impl/src/main/resources/wikitty-jdbc-query.properties 2011-02-21 16:42:36 UTC (rev 714) @@ -5,7 +5,7 @@ # $Id$ # $HeadURL$ # %% -# Copyright (C) 2010 CodeLutin, Benjamin Poussin +# Copyright (C) 2010 - 2011 CodeLutin, Benjamin Poussin, Chatellier Eric # %% # This program is free software: you can redistribute it and/or modify # it under the terms of the GNU Lesser General Public License as @@ -96,6 +96,7 @@ #extension delete admin and data (ESCAPE is needed for [ and ] because some # database use it like sqlserver) +jdbc.queries.select.extension.admin=SELECT %s FROM extension_admin WHERE id LIKE ? ESCAPE '#'; jdbc.queries.delete.extension.data=DELETE FROM extension_data WHERE id LIKE ? ESCAPE '#'; jdbc.queries.delete.extension.admin=DELETE FROM extension_admin WHERE id LIKE ? ESCAPE '#';

1 0

r713 - in trunk/src/site: . rst rst/user
by jcouteau＠users.nuiton.org 21 Feb '11

21 Feb '11

Author: jcouteau Date: 2011-02-21 15:17:58 +0100 (Mon, 21 Feb 2011) New Revision: 713 Url: http://nuiton.org/repositories/revision/wikitty/713 Log: Improve doc Added: trunk/src/site/rst/user/ trunk/src/site/rst/user/generation.rst trunk/src/site/rst/user/hessian.rst trunk/src/site/rst/user/migration.rst trunk/src/site/rst/user/security.rst Removed: trunk/src/site/rst/generation.rst trunk/src/site/rst/hessian.rst trunk/src/site/rst/migration.rst Modified: trunk/src/site/rst/fonctionnement.rst trunk/src/site/site_fr.xml Modified: trunk/src/site/rst/fonctionnement.rst =================================================================== --- trunk/src/site/rst/fonctionnement.rst 2011-02-16 21:37:57 UTC (rev 712) +++ trunk/src/site/rst/fonctionnement.rst 2011-02-21 14:17:58 UTC (rev 713) @@ -95,7 +95,7 @@ - login/logout - check de droit (read, write, delete) -Les nouveaux services (cache, securite, notification, ...) s'intercale en +Les nouveaux services (cache, securité, notification, ...) s'intercale en couche au dessus du coeur pour founir les services. Le coeur delegue le travail à trois services qui peuvent être implanté avec Deleted: trunk/src/site/rst/generation.rst =================================================================== --- trunk/src/site/rst/generation.rst 2011-02-16 21:37:57 UTC (rev 712) +++ trunk/src/site/rst/generation.rst 2011-02-21 14:17:58 UTC (rev 713) @@ -1,111 +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 souhaiter 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>2.2/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. - -Génération -========== - -La génération est effectuée ensuite à chaque fois que vous lancez un build. - -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.api*. Elle ne sera pas générée mais interviendra -comme une autre classe de votre modèle (héritage,...). - -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 Wikity, de même que -l'indexation et la recherche. - -.. _EUGene::http://maven-site.nuiton.org/eugene/ \ No newline at end of file Deleted: trunk/src/site/rst/hessian.rst =================================================================== --- trunk/src/site/rst/hessian.rst 2011-02-16 21:37:57 UTC (rev 712) +++ trunk/src/site/rst/hessian.rst 2011-02-21 14:17:58 UTC (rev 713) @@ -1,88 +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% -.. - -Hessian -======= - -Wikitty fournit de base les modules nécéssaires à une utilisation de type -client serveur basé sur hessian. - -Il est composé : - - * d'une application war qui, en lisant une configuration, est capable de - fournir un service sur n'importe quel type de stockage wikitty - * d'une factory revoyant un proxy sur un service distant - -Serveur -------- - -Le serveur est disponible sous forme d'une application web (war) prete à -l'emploi. Vous pouvez la télécharger à l'adresse suivante : -http://www.nuiton.org/projects/list_files/wikitty - -Par défaut, cette application web se base sur le fichier de configuration -``/etc/wikitty-hessian.properties`` (sous linux). Ce fichier de configuration -contient 3 options de configuration spécifique à hessian: - - * wikitty.hessian.usecache : ajout du cache - * wikitty.hessian.usenotification : ajout de la notification - * wikitty.hessian.usesecurity : ajout de la securité - -Ensuite, le fichier de configuration contient la configuration specifique -à l'implementation du stockage à utiliser. - -Voici un exemple de configuration:: - - # security - wikitty.hessian.usesecurity=true - wikitty.service.login=xxxx - wikitty.service.password=zzzz - - # notification - wikitty.hessian.usenotification=true - wikitty.service.event.jgroupschannelname=myjgroupchannel - wikitty.service.event.propagateEvent=false - - # cache - wikitty.hessian.usecache=true - wikitty.service.cache.listenevents=true - - # solr configuration - solr.data.dir=/var/lib/myapp/solr - -L'application est ensuite accessible sur l'uri "/wikitty" suivant le contexte -de déploiement. -Par exemple : http://localhost:8080/myapp/wikitty - -Client ------- - -L'interface sur le service s'obtient en utilisant la factory disponible -dans le modules ``wikitty-hessian-client``. - -Example:: - - WikittyService service = getWikittyService("http://localhost/wikitty"); - -Le ``service`` s'utilise ensuite comme un service local. Deleted: trunk/src/site/rst/migration.rst =================================================================== --- trunk/src/site/rst/migration.rst 2011-02-16 21:37:57 UTC (rev 712) +++ trunk/src/site/rst/migration.rst 2011-02-21 14:17:58 UTC (rev 713) @@ -1,87 +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érer la migration des données dans Wikitty -------------------------------------------- - -Présentation -============ - -Dans Wikitty, la migration est effectuée à chaque fois que vous chargez un -objet. Si un objet est présent dans la base de donnée dans une ancienne version, -quand il est chargé depuis la base de donnée, il est automatiquement migré dans -la nouvelle version. De cette manière, vous pouvez éviter les migrations -coûteuses à l'initialisation. - -La seule chose à garder à l'esprit est que les données qui n'ont pas encore -été migrée sont encore indexées dans l'ancienne version et peuvent ne pas être -retournées par certaines requêtes. Si vous avez besoin que les données soient -indexées dans la nouvelle version, vous pouvez toujours migrer tous vos objets -à l'initialisation en chargeant les objets et en les enregistrant. - -Comment migrer proprement les données dans Wikitty -================================================== - -Après avoir lancé votre application et avant toutes vos requêtes, vous devez -enregistrer toutes les extensions dont votre application a besoin dans leur -dernière version:: - - store(myExtension); - -ou:: - - store(myEntityImpl.extensionMyEntity) - -Si vous avez des migrations spécifiques à effectuer, ajoutez au registre toutes -les migrations spécifiques des extensions:: - - WikittyExtensionMigration.migrationRegistry.put("myExtension", myMigrationClass); - -Vous pouvez maintenant utiliser vos données simplement, elles seront migrées -au chargement. - -Si vous voulez migrer vos objets avant de les utiliser (pour des histoires de -requêtes par exemple), pour chaque version, recherchez vos objets, restaurez -les puis enregistrez-les:: - - search(Client) - restore(Client) - store(Client) - -Attention au coût d'une indexation au vu du volume de données, il n'est pas -toujours judicieux de réindexer toutes les données. - -Migrations spécifiques -====================== - -Wikitty réalise une migration automatique en cas de champs supprimés, ajoutés -ou renommés (dans ce cas, il faut ajouter un tag renameFrom sur le champ). - -Pour toutes les autre migrations, vous devrez créer votre propre classe de -migration qui implémente WikittyExtensionMigration. Cette classe n'a besoin de -faire la migration d'une version à l'autre, par exemple de la version 1 à la -version 2. Dans le cas d'une migration de la version 1 à la version 4, Wikitty -appellera automatiquement les classes pour migrer de la version 1 à la version 2 -puis de la version 2 à la version 3 et ainsi de suite. \ No newline at end of file Copied: trunk/src/site/rst/user/generation.rst (from rev 694, trunk/src/site/rst/generation.rst) =================================================================== --- trunk/src/site/rst/user/generation.rst (rev 0) +++ trunk/src/site/rst/user/generation.rst 2011-02-21 14:17:58 UTC (rev 713) @@ -0,0 +1,111 @@ +.. - +.. * #%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 souhaiter 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>2.2/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. + +Génération +========== + +La génération est effectuée ensuite à chaque fois que vous lancez un build. + +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.api*. Elle ne sera pas générée mais interviendra +comme une autre classe de votre modèle (héritage,...). + +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 Wikity, de même que +l'indexation et la recherche. + +.. _EUGene::http://maven-site.nuiton.org/eugene/ \ No newline at end of file Property changes on: trunk/src/site/rst/user/generation.rst ___________________________________________________________________ Added: svn:keywords + Author Date Id Revision HeadURL Copied: trunk/src/site/rst/user/hessian.rst (from rev 694, trunk/src/site/rst/hessian.rst) =================================================================== --- trunk/src/site/rst/user/hessian.rst (rev 0) +++ trunk/src/site/rst/user/hessian.rst 2011-02-21 14:17:58 UTC (rev 713) @@ -0,0 +1,88 @@ +.. - +.. * #%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% +.. - +Hessian +======= + +Wikitty fournit de base les modules nécéssaires à une utilisation de type +client serveur basé sur hessian. + +Il est composé : + + * d'une application war qui, en lisant une configuration, est capable de + fournir un service sur n'importe quel type de stockage wikitty + * d'une factory revoyant un proxy sur un service distant + +Serveur +------- + +Le serveur est disponible sous forme d'une application web (war) prete à +l'emploi. Vous pouvez la télécharger à l'adresse suivante : +http://www.nuiton.org/projects/list_files/wikitty + +Par défaut, cette application web se base sur le fichier de configuration +``/etc/wikitty-hessian.properties`` (sous linux). Ce fichier de configuration +contient 3 options de configuration spécifique à hessian: + + * wikitty.hessian.usecache : ajout du cache + * wikitty.hessian.usenotification : ajout de la notification + * wikitty.hessian.usesecurity : ajout de la securité + +Ensuite, le fichier de configuration contient la configuration specifique +à l'implementation du stockage à utiliser. + +Voici un exemple de configuration:: + + # security + wikitty.hessian.usesecurity=true + wikitty.service.login=xxxx + wikitty.service.password=zzzz + + # notification + wikitty.hessian.usenotification=true + wikitty.service.event.jgroupschannelname=myjgroupchannel + wikitty.service.event.propagateEvent=false + + # cache + wikitty.hessian.usecache=true + wikitty.service.cache.listenevents=true + + # solr configuration + solr.data.dir=/var/lib/myapp/solr + +L'application est ensuite accessible sur l'uri "/wikitty" suivant le contexte +de déploiement. +Par exemple : http://localhost:8080/myapp/wikitty + +Client +------ + +L'interface sur le service s'obtient en utilisant la factory disponible +dans le modules ``wikitty-hessian-client``. + +Example:: + + WikittyService service = getWikittyService("http://localhost/wikitty"); + +Le ``service`` s'utilise ensuite comme un service local. Property changes on: trunk/src/site/rst/user/hessian.rst ___________________________________________________________________ Added: svn:keywords + Author Date Id Revision HeadURL Copied: trunk/src/site/rst/user/migration.rst (from rev 694, trunk/src/site/rst/migration.rst) =================================================================== --- trunk/src/site/rst/user/migration.rst (rev 0) +++ trunk/src/site/rst/user/migration.rst 2011-02-21 14:17:58 UTC (rev 713) @@ -0,0 +1,87 @@ +.. - +.. * #%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érer la migration des données dans Wikitty +------------------------------------------- + +Présentation +============ + +Dans Wikitty, la migration est effectuée à chaque fois que vous chargez un +objet. Si un objet est présent dans la base de donnée dans une ancienne version, +quand il est chargé depuis la base de donnée, il est automatiquement migré dans +la nouvelle version. De cette manière, vous pouvez éviter les migrations +coûteuses à l'initialisation. + +La seule chose à garder à l'esprit est que les données qui n'ont pas encore +été migrée sont encore indexées dans l'ancienne version et peuvent ne pas être +retournées par certaines requêtes. Si vous avez besoin que les données soient +indexées dans la nouvelle version, vous pouvez toujours migrer tous vos objets +à l'initialisation en chargeant les objets et en les enregistrant. + +Comment migrer proprement les données dans Wikitty +================================================== + +Après avoir lancé votre application et avant toutes vos requêtes, vous devez +enregistrer toutes les extensions dont votre application a besoin dans leur +dernière version:: + + store(myExtension); + +ou:: + + store(myEntityImpl.extensionMyEntity) + +Si vous avez des migrations spécifiques à effectuer, ajoutez au registre toutes +les migrations spécifiques des extensions:: + + WikittyExtensionMigration.migrationRegistry.put("myExtension", myMigrationClass); + +Vous pouvez maintenant utiliser vos données simplement, elles seront migrées +au chargement. + +Si vous voulez migrer vos objets avant de les utiliser (pour des histoires de +requêtes par exemple), pour chaque version, recherchez vos objets, restaurez +les puis enregistrez-les:: + + search(Client) + restore(Client) + store(Client) + +Attention au coût d'une indexation au vu du volume de données, il n'est pas +toujours judicieux de réindexer toutes les données. + +Migrations spécifiques +====================== + +Wikitty réalise une migration automatique en cas de champs supprimés, ajoutés +ou renommés (dans ce cas, il faut ajouter un tag renameFrom sur le champ). + +Pour toutes les autre migrations, vous devrez créer votre propre classe de +migration qui implémente WikittyExtensionMigration. Cette classe n'a besoin de +faire la migration d'une version à l'autre, par exemple de la version 1 à la +version 2. Dans le cas d'une migration de la version 1 à la version 4, Wikitty +appellera automatiquement les classes pour migrer de la version 1 à la version 2 +puis de la version 2 à la version 3 et ainsi de suite. \ No newline at end of file Property changes on: trunk/src/site/rst/user/migration.rst ___________________________________________________________________ Added: svn:keywords + Author Date Id Revision HeadURL Added: trunk/src/site/rst/user/security.rst =================================================================== --- trunk/src/site/rst/user/security.rst (rev 0) +++ trunk/src/site/rst/user/security.rst 2011-02-21 14:17:58 UTC (rev 713) @@ -0,0 +1,111 @@ +Utiliser la sécurité dans Wikitty +================================= + +Wikitty fournit un système de gestion des droits en lecture et en écriture pour +les entitées. Vous pouvez aller jusqu'à limiter l'accès à certains objet d'un +type à certains utilisateurs. Vous pouvez également accorder des droits +différents à certains champs des objets. + +Ajouter la couche de sécurité +----------------------------- + +Pour mettre en place la sécurité dans Wikitty, il suffit de rajouter +``org.nuiton.wikitty.services.WikittyServiceSecurity`` dans la liste des +composants à utiliser de votre configuration:: + + wikitty.WikittyService.components=org.nuiton.wikitty.services.WikittyServiceStorage,\ + org.nuiton.wikitty.services.WikittyServiceCached,\ + org.nuiton.wikitty.services.WikittyServiceSecurity + +Cela rajoute la couche de sécurité à Wikitty. Vous pouvez maintenant rajouter +des droits à certaines entitées pour en limiter l'accès. + +Création de l'administrateur de l'application +--------------------------------------------- + +Par défaut il existe un *WikittyGroup* portant le nom **WikittyAppAdmin**. +Si ce groupe n'existe pas ou qu'il est vide, cela indique que tous les +utilisateurs sont AppAdmin. Cela est nécessaire pour deux choses: + +- pour permettre d'avoir une authentification, mais que l'on ne souhaite pas de + gestion d'autorisation. + +- pour servir de bootstrap au lancement de l'application seul les AppAdmin + peuvent modifier le *WikittyGroup* portant le nom **WikittyAppAdmin**, or s'il + est vide il faut pouvoir le remplir. + +Il ne doit donc exister un seul *WikittyGroup* portant le nom +**WikittyAppAdmin** + +La première chose à faire maintenant, est donc de créer l'utilisateur +administrateur de l'application et l'ajouter au groupe **WikittyAppAdmin**:: + + WikittyUser admin = new WikittyUserImpl(); + admin.setLogin("admin"); + admin.setPassword("admin"); + admin = proxy.store(admin); + + WikittyGroup adminGroup = WikittySecurityUtil.createAppAdminGroup(admin); + +Jouer avec les droits sur une entité (sécurité de niveau 1) +----------------------------------------------------------- + +A partir de maintenant, l'utilisateur administrateur (et tous les autres +utilisateurs faisant partie du groupe des **AppAdmin**) peut ajouter/retirer +des droits à des entités. + +Par exemple, pour rajouter les droits de lecture a un utilisateur (l'utilisateur +admin est déjà identifié) sur un wikitty:: + + //On commence par charger les wikittys concernés + WikittyAuthorisation skill = proxy.restore(WikittyAuthorisation.class, id); + + //On ajoute en reader notre utilisateur + skill.addReader(user.getWikittyId()); + + //On enregistre + proxy.store(skill); + +On peut de la même manière ajouter un groupe en tant que +reader/writer/owner/admin d'une entité. + +Les droits sont les suivants : + ++----------+---------+----------+-------------+----------------------+ +| Type | Lecture | Ecriture | Suppression | Changement de droits | ++==========+=========+==========+=============+======================+ +| Reader | X | | | | ++----------+---------+----------+-------------+----------------------+ +| Writer | X | X | | | ++----------+---------+----------+-------------+----------------------+ +| Owner | X | X | X | | ++----------+---------+----------+-------------+----------------------+ +| Admin | X | X | X | X | ++----------+---------+----------+-------------+----------------------+ +| AppAdmin | X | X | X | X | ++----------+---------+----------+-------------+----------------------+ + +Si on veut que tout le monde puisse lire un wikitty, il faut laisser vide la +liste des readers (vide ou null). + +Si l'on souhaite indiquer que personne ne peut lire l'objet il faut mettre le +owner en tant que reader. Car de toute façon l'owner aura toujours le droit +en lecture. + +L'authentification +------------------ + +À chaque login, un token de sécurité est fourni à l'utilisateur. L'utilisateur +doit fournir ce token à chaque opération pour montrer qu'il a les droits. +Pour un token donné, on peut savoir à quel utilisateur il appartient. Pour le +développeur, cela n'implique pas de changement car le token de sécurité est +conservé par le **WikittyProxy**. Il faut juste faire attention à bien avoir un +WikittyProxy par utilisateur. Dans le cas d'applications web, il est conseillé +de conserver le proxy dans la session de l'utilisateur. + +Les tokens sont stockés en base et supprimés au logout de l'utilisateur. + +Les SecurityToken ne sont supprimés de la base de données qu'au moment du logout +Si les utilisateurs/applications quittent sans faire le logout le nombre de +SecurityToken ne fera qu'augmenter. Pour éviter cela, il faut prévoir un petit +job qui de temps en temps fait le ménage dans les SecurityToken trop vieux. \ No newline at end of file Modified: trunk/src/site/site_fr.xml =================================================================== --- trunk/src/site/site_fr.xml 2011-02-16 21:37:57 UTC (rev 712) +++ trunk/src/site/site_fr.xml 2011-02-21 14:17:58 UTC (rev 713) @@ -53,13 +53,14 @@ <item name="Changements" href="changes.html"/> </menu> - <menu name="User"> - <item name="Generation" href="generation.html"/> - <item name="Migration" href="migration.html"/> - <item name="Hessian" href="hessian.html"/> + <menu name="Utilisateur"> + <item name="Génération" href="user/generation.html"/> + <item name="Migration" href="user/migration.html"/> + <item name="Hessian" href="user/hessian.html"/> + <item name="Sécurité" href="user/security.html"/> </menu> - <menu name="Developer"> + <menu name="Developpeur"> <item name="Todo" href="todo.html"/> </menu>

1 0

r712 - trunk
by tchemit＠users.nuiton.org 16 Feb '11

16 Feb '11

Author: tchemit Date: 2011-02-16 22:37:57 +0100 (Wed, 16 Feb 2011) New Revision: 712 Url: http://nuiton.org/repositories/revision/wikitty/712 Log: Update mavenpom4redmine to 2.5.1. Modified: trunk/pom.xml Modified: trunk/pom.xml =================================================================== --- trunk/pom.xml 2011-02-16 16:35:48 UTC (rev 711) +++ trunk/pom.xml 2011-02-16 21:37:57 UTC (rev 712) @@ -6,7 +6,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>mavenpom4redmine</artifactId> - <version>2.5</version> + <version>2.5.1</version> </parent>

1 0

r711 - trunk/wikitty-api/src/main/java/org/nuiton/wikitty
by bpoussin＠users.nuiton.org 16 Feb '11

16 Feb '11

Author: bpoussin Date: 2011-02-16 17:35:48 +0100 (Wed, 16 Feb 2011) New Revision: 711 Url: http://nuiton.org/repositories/revision/wikitty/711 Log: Evolution #1337: On Proxy add delete(BusinessEntity) and delete(List<BusinessEntity>) Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyProxy.java Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyProxy.java =================================================================== --- trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyProxy.java 2011-02-16 16:12:36 UTC (rev 710) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/WikittyProxy.java 2011-02-16 16:35:48 UTC (rev 711) @@ -387,10 +387,19 @@ public void delete(String id) { long start = TimeTrace.getTime(); wikittyService.delete(securityToken, id); - + timeTrace.add(start, "delete"); } + public <E extends BusinessEntity> void delete(E object) { + long start = TimeTrace.getTime(); + if (object != null) { + String id = object.getWikittyId(); + wikittyService.delete(securityToken, id); + } + timeTrace.add(start, "delete(BusinessEntity)"); + } + public void delete(Collection<String> ids) { long start = TimeTrace.getTime(); wikittyService.delete(securityToken, ids); @@ -398,6 +407,24 @@ timeTrace.add(start, "delete<list>"); } + public <E extends BusinessEntity> void delete(List<E> objets) { + long start = TimeTrace.getTime(); + + // prepare data to send to service + List<String> ids = new ArrayList<String>(objets.size()); + for (E e : objets) { + if (e != null) { + String id = e.getWikittyId(); + ids.add(id); + } + } + + // call the service with Wikitty + wikittyService.delete(securityToken, ids); + + timeTrace.add(start, "delete<list<BusinessEntity>>"); + } + /** * Null field are not used in search request. *

1 0

r710 - trunk/wikitty-generators/src/main/java/org/nuiton/wikitty/generator
by bpoussin＠users.nuiton.org 16 Feb '11

16 Feb '11

Author: bpoussin Date: 2011-02-16 17:12:36 +0100 (Wed, 16 Feb 2011) New Revision: 710 Url: http://nuiton.org/repositories/revision/wikitty/710 Log: Anomalie #1331: Bug in entity generation when inherite wikittyLabel (import missing) add support for more than 1 level of inheritance Modified: trunk/wikitty-generators/src/main/java/org/nuiton/wikitty/generator/WikittyAbstractGenerator.java Modified: trunk/wikitty-generators/src/main/java/org/nuiton/wikitty/generator/WikittyAbstractGenerator.java =================================================================== --- trunk/wikitty-generators/src/main/java/org/nuiton/wikitty/generator/WikittyAbstractGenerator.java 2011-02-16 14:57:46 UTC (rev 709) +++ trunk/wikitty-generators/src/main/java/org/nuiton/wikitty/generator/WikittyAbstractGenerator.java 2011-02-16 16:12:36 UTC (rev 710) @@ -470,13 +470,16 @@ // if super class is not in the same package, import its Helper if ( ! businessEntity.getPackageName().equals(superClass.getPackageName())) { + String parentToImport = superClass.getQualifiedName(); String helperToImport = superClass.getPackageName() + "." + WikittyTransformerUtil.businessEntityToHelperName(superClass); + addImport(abstractClass, parentToImport); addImport(abstractClass, helperToImport); // add the last import to the map, so sub-classes will be // able to know what imports they need + imports.add(parentToImport); imports.add(helperToImport); }

1 0