diff --git a/NEWS.rst b/NEWS.rst index 84837ad4e..da708f30b 100644 --- a/NEWS.rst +++ b/NEWS.rst @@ -41,6 +41,9 @@ Changes * Update Commons Math * Update Commons Collections * Update Commons Logging +* Update Commons Gettext +* Update Сommons IO +* Update Commons Fileupload * Update CXF-transports-http * Update CXF-frontend-jaxrs @@ -56,12 +59,12 @@ Changes * Update JGraphT * Update DBUnit * Update JodaTime -* Update Gettext * Update Jasper Reports * Update EasyMock * Update JDBC driver -* Update Сommons-IO * Update AspectJ Weaver +* Update JAX-RS API +* Update BeanShell * Update LibrePlan version to 1.6.0 diff --git a/ganttzk/pom.xml b/ganttzk/pom.xml index 865e33994..d19472033 100644 --- a/ganttzk/pom.xml +++ b/ganttzk/pom.xml @@ -40,7 +40,6 @@ org.apache.maven.plugins maven-source-plugin - 2.1.2 default @@ -57,6 +56,7 @@ + @@ -72,13 +72,13 @@ org.xnap.commons maven-gettext-plugin - [1.2.0,) + [1.2.4,) dist - + diff --git a/ganttzk/src/main/java/org/zkoss/ganttz/adapters/DomainDependency.java b/ganttzk/src/main/java/org/zkoss/ganttz/adapters/DomainDependency.java index 4f147e120..ea14c47b5 100644 --- a/ganttzk/src/main/java/org/zkoss/ganttz/adapters/DomainDependency.java +++ b/ganttzk/src/main/java/org/zkoss/ganttz/adapters/DomainDependency.java @@ -40,9 +40,9 @@ public class DomainDependency implements IDependency { private static final Log LOG = LogFactory.getLog(DomainDependency.class); public static List toDependencies( - IDomainAndBeansMapper mapper, - Collection> dependencies) { - List result = new ArrayList(); + IDomainAndBeansMapper mapper, Collection> dependencies) { + + List result = new ArrayList<>(); for (DomainDependency domainDependency : dependencies) { try { result.add(domainDependency.toDependency(mapper)); @@ -50,11 +50,11 @@ public class DomainDependency implements IDependency { LOG.error("error creating dependency from domainDependency", e); } } + return result; } - public static DomainDependency createDependency(T source, - T destination, DependencyType type) { + public static DomainDependency createDependency(T source, T destination, DependencyType type) { return new DomainDependency(source, destination, type); } @@ -84,7 +84,6 @@ public class DomainDependency implements IDependency { } public Dependency toDependency(IDomainAndBeansMapper mapper) { - return new Dependency(mapper.findAssociatedBean(source), mapper - .findAssociatedBean(destination), type); + return new Dependency(mapper.findAssociatedBean(source), mapper.findAssociatedBean(destination), type); } } diff --git a/ganttzk/src/test/java/org/zkoss/ganttz/data/constraint/ConstraintTest.java b/ganttzk/src/test/java/org/zkoss/ganttz/data/constraint/ConstraintTest.java index 292b05a7e..5f08f1587 100644 --- a/ganttzk/src/test/java/org/zkoss/ganttz/data/constraint/ConstraintTest.java +++ b/ganttzk/src/test/java/org/zkoss/ganttz/data/constraint/ConstraintTest.java @@ -91,22 +91,23 @@ public class ConstraintTest { return value != null && value > 5; } }; + assertThat(biggerThanFive.applyTo(5), equalTo(6)); } @SuppressWarnings("unchecked") @Test public void applyingSeveralConstraintsCallsApplyToOfEveryone() { - List> constraints = new ArrayList>(); + List> constraints = new ArrayList<>(); final int numerOfConstraints = 5; + for (int i = 0; i < numerOfConstraints; i++) { Constraint constraint = createNiceMock(Constraint.class); - expect(constraint.applyConstraintTo(isA(Object.class))) - .andReturn(2).atLeastOnce(); - expect(constraint.isSatisfiedBy(isA(Object.class))).andReturn(true) - .atLeastOnce(); + expect(constraint.applyConstraintTo(isA(Object.class))).andReturn(2).atLeastOnce(); + expect(constraint.isSatisfiedBy(isA(Object.class))).andReturn(true).atLeastOnce(); constraints.add(constraint); } + replay(constraints.toArray()); Constraint.apply(2, constraints); verify(constraints.toArray()); @@ -115,8 +116,8 @@ public class ConstraintTest { @SuppressWarnings("unchecked") @Test public void theLastConstraintHasPriority() { - Integer result = Constraint.apply(6, biggerThanFive, - lessThanFive); + Integer result = Constraint.apply(6, biggerThanFive, lessThanFive); + assertThat(result, equalTo(4)); } @@ -124,21 +125,20 @@ public class ConstraintTest { @Test public void theViolatedConstraintsNotifiesItsListeners() { final Constraint[] constraintViolated = new Constraint[1]; - biggerThanFive - .addConstraintViolationListener(new IConstraintViolationListener() { - @Override - public void constraintViolated( - Constraint constraint, Integer value) { - constraintViolated[0] = constraint; - } + biggerThanFive.addConstraintViolationListener(new IConstraintViolationListener() { + @Override + public void constraintViolated(Constraint constraint, Integer value) { + constraintViolated[0] = constraint; + } + + @Override + public void constraintSatisfied(Constraint constraint, Integer value) { + } + }); - @Override - public void constraintSatisfied( - Constraint constraint, Integer value) { - } - }); Constraint.apply(6, biggerThanFive, lessThanFive); + assertThat(constraintViolated[0], equalTo(biggerThanFive)); } @@ -146,31 +146,29 @@ public class ConstraintTest { @SuppressWarnings("unchecked") public void theSatisfiedConstraintsNotifiesItsListeners() { final Constraint[] constraintSatisfied = new Constraint[1]; - biggerThanFive - .addConstraintViolationListener(new IConstraintViolationListener() { - @Override - public void constraintViolated( - Constraint constraint, Integer value) { - } + biggerThanFive.addConstraintViolationListener(new IConstraintViolationListener() { + @Override + public void constraintViolated( + Constraint constraint, Integer value) { + } + + @Override + public void constraintSatisfied( + Constraint constraint, Integer value) { + constraintSatisfied[0] = constraint; + } + }); - @Override - public void constraintSatisfied( - Constraint constraint, Integer value) { - constraintSatisfied[0] = constraint; - } - }); Constraint.apply(6, biggerThanFive); + assertThat(constraintSatisfied[0], equalTo(biggerThanFive)); } @SuppressWarnings("unchecked") @Test public void theApplicationCanBeDoneUsingAFluentInterface() { - assertThat(Constraint.initialValue(3) - .withConstraints(biggerThanFive) - .apply(), - equalTo(6)); + assertThat(Constraint.initialValue(3).withConstraints(biggerThanFive).apply(), equalTo(6)); } @SuppressWarnings("unchecked") diff --git a/libreplan-business/pom.xml b/libreplan-business/pom.xml index e53adf8df..ab6d6d7b0 100644 --- a/libreplan-business/pom.xml +++ b/libreplan-business/pom.xml @@ -207,6 +207,7 @@ + @@ -227,7 +228,7 @@ - + diff --git a/libreplan-webapp/pom.xml b/libreplan-webapp/pom.xml index 5a6790a73..ca5192401 100644 --- a/libreplan-webapp/pom.xml +++ b/libreplan-webapp/pom.xml @@ -464,7 +464,7 @@ javax.ws.rs - jsr311-api + javax.ws.rs-api diff --git a/pom.xml b/pom.xml index 5f5986b93..6e1926ba4 100644 --- a/pom.xml +++ b/pom.xml @@ -8,9 +8,7 @@ 1.6.0 LibrePlan - + libreplan-business @@ -18,9 +16,7 @@ libreplan-webapp - + - @@ -88,8 +87,8 @@ false validate - + false @@ -124,7 +123,7 @@ jdbc:postgresql://localhost/libreplan${libreplan.mode} ${dataSource.url}test - org.hibernate.dialect.PostgreSQLDialect + org.hibernate.dialect.PostgreSQL9Dialect public. @@ -145,15 +144,12 @@ ${dataSource.url}test org.hibernate.dialect.MySQL5InnoDBDialect - + - @@ -170,18 +166,11 @@ + gettext-commons-site http://gettext-commons.googlecode.com/svn/maven-repository - @@ -301,8 +290,7 @@ @@ -522,8 +510,8 @@ javax.ws.rs - jsr311-api - 1.0 + javax.ws.rs-api + 2.0.1 @@ -589,7 +577,7 @@ - + com.fasterxml.jackson.jaxrs jackson-jaxrs-json-provider @@ -668,19 +656,16 @@ + - - + src/main/resources true @@ -690,10 +675,7 @@ - + src/main/resources @@ -741,10 +723,7 @@ - + src/test/resources true @@ -755,10 +734,7 @@ - + src/test/resources @@ -767,7 +743,6 @@ - com.googlecode.gettext-commons @@ -778,7 +753,6 @@ - org.apache.maven.plugins @@ -792,7 +766,7 @@ - + maven-assembly-plugin @@ -845,19 +819,12 @@ true - @@ -893,7 +860,6 @@ - org.codehaus.mojo @@ -905,7 +871,6 @@ - org.codehaus.mojo @@ -921,7 +886,6 @@ - org.liquibase