Author: bleny Date: 2013-03-27 15:02:44 +0100 (Wed, 27 Mar 2013) New Revision: 50 Url: http://forge.codelutin.com/projects/franciaflex-magalie/repository/revisions... Log: remove deprectad method Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/StorageMovementOrder.java trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ArticleStorageService.java trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/ArticleStorageServiceTest.java Modified: trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/StorageMovementOrder.java =================================================================== --- trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/StorageMovementOrder.java 2013-03-27 14:01:45 UTC (rev 49) +++ trunk/magalie-persistence/src/main/java/com/franciaflex/magalie/persistence/entity/StorageMovementOrder.java 2013-03-27 14:02:44 UTC (rev 50) @@ -66,13 +66,6 @@ this.storageMovements = storageMovements; } - /** - * @deprecated use {@link #getStorageMovements()} - */ - public List<StorageMovement> getSites() { - return storageMovements; - } - protected Set<Article> getArticles() { Set<Article> articles = Sets.newHashSet( @@ -104,8 +97,7 @@ storageMovements.add(storageMovement); } - // TODO brendan 26/03/13 rename to getSites - public Set<Site> getSites2() { + public Set<Site> getSites() { Set<Site> sites = Sets.newHashSet( Iterables.transform( Modified: trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ArticleStorageService.java =================================================================== --- trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ArticleStorageService.java 2013-03-27 14:01:45 UTC (rev 49) +++ trunk/magalie-services/src/main/java/com/franciaflex/magalie/services/service/ArticleStorageService.java 2013-03-27 14:02:44 UTC (rev 50) @@ -141,7 +141,7 @@ Article article = storageMovementOrder.getArticle(); - Set<Site> sitesInOrder = storageMovementOrder.getSites2(); + Set<Site> sitesInOrder = storageMovementOrder.getSites(); List<StoredArticle> storedArticles = getStoredArticles(article); Modified: trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/ArticleStorageServiceTest.java =================================================================== --- trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/ArticleStorageServiceTest.java 2013-03-27 14:01:45 UTC (rev 49) +++ trunk/magalie-services/src/test/java/com/franciaflex/magalie/services/service/ArticleStorageServiceTest.java 2013-03-27 14:02:44 UTC (rev 50) @@ -99,7 +99,7 @@ Assert.fail(); } - Assert.assertEquals(1, storageMovementOrder.getSites().size()); + Assert.assertEquals(1, storageMovementOrder.getStorageMovements().size()); } @Test @@ -126,10 +126,10 @@ Assert.fail(); } - Assert.assertEquals(2, storageMovementOrder.getSites().size()); - StorageMovement site1 = Iterables.get(storageMovementOrder.getSites(), 0, null); + Assert.assertEquals(2, storageMovementOrder.getStorageMovements().size()); + StorageMovement site1 = Iterables.get(storageMovementOrder.getStorageMovements(), 0, null); Assert.assertEquals(-20., site1.getQuantity(), DELTA); - StorageMovement site2 = Iterables.get(storageMovementOrder.getSites(), 1, null); + StorageMovement site2 = Iterables.get(storageMovementOrder.getStorageMovements(), 1, null); Assert.assertEquals(-30., site2.getQuantity(), DELTA); } @@ -148,11 +148,11 @@ if (log.isDebugEnabled()) { log.debug("storageMovementOrder = " + storageMovementOrder); } - Assert.assertEquals(2, storageMovementOrder.getSites().size()); - StorageMovement storageMovement1 = Iterables.get(storageMovementOrder.getSites(), 0, null); + Assert.assertEquals(2, storageMovementOrder.getStorageMovements().size()); + StorageMovement storageMovement1 = Iterables.get(storageMovementOrder.getStorageMovements(), 0, null); Assert.assertEquals(-20., storageMovement1.getQuantity(), DELTA); Assert.assertEquals("B2", storageMovement1.getSite().getLocationCode()); - StorageMovement storageMovement2 = Iterables.get(storageMovementOrder.getSites(), 1, null); + StorageMovement storageMovement2 = Iterables.get(storageMovementOrder.getStorageMovements(), 1, null); Assert.assertEquals(-10., storageMovement2.getQuantity(), DELTA); Assert.assertEquals("A1", storageMovement2.getSite().getLocationCode()); } catch (RequiredDriverLicenceException e) { @@ -172,8 +172,8 @@ if (log.isDebugEnabled()) { log.debug("storageMovementOrder = " + storageMovementOrder); } - Assert.assertEquals(1, storageMovementOrder.getSites().size()); - StorageMovement storageMovement = Iterables.get(storageMovementOrder.getSites(), 0, null); + Assert.assertEquals(1, storageMovementOrder.getStorageMovements().size()); + StorageMovement storageMovement = Iterables.get(storageMovementOrder.getStorageMovements(), 0, null); Assert.assertEquals(-30., storageMovement.getQuantity(), DELTA); Assert.assertEquals("A1", storageMovement.getSite().getLocationCode()); } catch (RequiredDriverLicenceException e) { @@ -207,8 +207,8 @@ if (log.isDebugEnabled()) { log.debug("storageMovementOrder = " + storageMovementOrder); } - Assert.assertEquals(1, storageMovementOrder.getSites().size()); - StorageMovement storageMovement = Iterables.get(storageMovementOrder.getSites(), 0, null); + Assert.assertEquals(1, storageMovementOrder.getStorageMovements().size()); + StorageMovement storageMovement = Iterables.get(storageMovementOrder.getStorageMovements(), 0, null); Assert.assertNotEquals( "site2 must not be returned because it was reported in error", site2.getId(),