Author: bpoussin Date: 2013-02-23 15:20:02 +0100 (Sat, 23 Feb 2013) New Revision: 1541 Url: http://nuiton.org/projects/wikitty/repository/revisions/1541 Log: simplification de la methode de migration (la complexite n'avait que peu d'interet (voir ne marchait pas dans certain cas :()) Modified: trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyServiceStorage.java 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 2013-02-23 14:19:09 UTC (rev 1540) +++ trunk/wikitty-api/src/main/java/org/nuiton/wikitty/services/WikittyServiceStorage.java 2013-02-23 14:20:02 UTC (rev 1541) @@ -648,6 +648,9 @@ /** * Upgrade wikitty but not saved it. + * On passe directement de la version du wikitty a la derniere version connue + * car une installation pourrait faire un upgrade de plusieurs version + * et donc certaine version de l'extension pourrait ne pas exister en local * * @param securityToken security token previously returned by login. If * securityToken is not valid, this method do nothing @@ -673,7 +676,8 @@ String lastExtensionVersion = lastExtension.getVersion(); if (log.isDebugEnabled()) { - log.debug("lastExtensionVersion=" + lastExtensionVersion); + log.debug(String.format("Migrate extension '%s' from '%s' to '%s'", + extensionName, currentExtensionVersion, lastExtensionVersion)); } @@ -691,31 +695,9 @@ migration = defaultExtensionMigration; } - // Loop on between extension in wikitty and last version - while(WikittyUtil.versionGreaterThan(lastExtensionVersion, currentExtensionVersion)) { - - // Get extension after the current version - String nextExtensionVersion = - WikittyUtil.incrementMajorRevision(currentExtensionVersion); - String nextExtensionId = - WikittyExtension.computeId(extensionName, nextExtensionVersion); - WikittyExtension nextExtension = restoreExtension( - securityToken, nextExtensionId); - - if (log.isDebugEnabled()) { - log.debug("currentExtensionVersion=" + currentExtensionVersion); - log.debug("nextExtensionVersion=" + nextExtensionVersion); - } - - // Test if extension is never use in this version - if(nextExtension != null) { - result = migration.migrate(this, result, - currentExtension, nextExtension); - currentExtension = nextExtension; - } - - // Follow - currentExtensionVersion = nextExtensionVersion; + if(WikittyUtil.versionGreaterThan(lastExtensionVersion, currentExtensionVersion)) { + result = migration.migrate(this, result, + currentExtension, lastExtension); } }