[hibernate-commits] [hibernate/hibernate-validator] 3ffafb: HV-1164 Only cast dynamicPayload if it is != null ...

[GitHub]

  Branch: refs/heads/5.3
  Home:   https://github.com/hibernate/hibernate-validator
  Commit: 3ffafbc81672991e79f03ac3c8110e000907ad4c
      https://github.com/hibernate/hibernate-validator/commit/3ffafbc81672991e79f03ac3c8110e000907ad4c
  Author: bayerls <sebastian.bayerl at googlemail.com>
  Date:   2016-12-07 (Wed, 07 Dec 2016)

  Changed paths:
    M engine/src/main/java/org/hibernate/validator/internal/engine/ConstraintViolationImpl.java
    M engine/src/test/java/org/hibernate/validator/test/internal/engine/constraintvalidation/HibernateConstraintValidatorContextTest.java

  Log Message:
  -----------
  HV-1164 Only cast dynamicPayload if it is != null and add a test case


  Commit: fe240319f72ac1889db580e8b79e9f523373191c
      https://github.com/hibernate/hibernate-validator/commit/fe240319f72ac1889db580e8b79e9f523373191c
  Author: bayerls <sebastian.bayerl at googlemail.com>
  Date:   2016-12-07 (Wed, 07 Dec 2016)

  Changed paths:
    M engine/src/main/java/org/hibernate/validator/internal/engine/ConstraintViolationImpl.java

  Log Message:
  -----------
  HV-1164 remove occurences of this. for consistency


  Commit: 743c5dd803139472ddce88104ade59a1de08fc3c
      https://github.com/hibernate/hibernate-validator/commit/743c5dd803139472ddce88104ade59a1de08fc3c
  Author: Gunnar Morling <gunnar.morling at googlemail.com>
  Date:   2016-12-07 (Wed, 07 Dec 2016)

  Changed paths:
    M engine/src/main/java/org/hibernate/validator/engine/HibernateConstraintViolation.java
    M engine/src/test/java/org/hibernate/validator/test/internal/engine/constraintvalidation/HibernateConstraintValidatorContextTest.java

  Log Message:
  -----------
  HV-1164 Improving JavaDocs, removing superfluous empty lines


  Commit: 822614cba55c4a1c6cc7db22f8841c5a156644b1
      https://github.com/hibernate/hibernate-validator/commit/822614cba55c4a1c6cc7db22f8841c5a156644b1
  Author: Gunnar Morling <gunnar.morling at googlemail.com>
  Date:   2016-12-07 (Wed, 07 Dec 2016)

  Changed paths:
    M copyright.txt

  Log Message:
  -----------
  HV-1164 Adding Sebastian to copyright.txt


  Commit: 509a4c2ec043fcd4b5446b96eab7fb8d5dbffa0e
      https://github.com/hibernate/hibernate-validator/commit/509a4c2ec043fcd4b5446b96eab7fb8d5dbffa0e
  Author: Gunnar Morling <gunnar.morling at googlemail.com>
  Date:   2016-12-07 (Wed, 07 Dec 2016)

  Changed paths:
    M engine/src/test/java/org/hibernate/validator/test/internal/engine/constraintvalidation/HibernateConstraintValidatorContextTest.java

  Log Message:
  -----------
  HV-1164 Fixing formatting glitch


  Commit: 02b2ba28c571b780ed6f8f99a4d097c1ef0c4639
      https://github.com/hibernate/hibernate-validator/commit/02b2ba28c571b780ed6f8f99a4d097c1ef0c4639
  Author: Guillaume Smet <guillaume.smet at gmail.com>
  Date:   2016-12-07 (Wed, 07 Dec 2016)

  Changed paths:
    M engine-jdk8-tests/src/test/java/org/hibernate/validator/test/internal/engine/typeannotationconstraint/TypeAnnotationConstraintTest.java
    M engine/src/main/java/org/hibernate/validator/internal/engine/ValidatorImpl.java
    M engine/src/main/java/org/hibernate/validator/internal/metadata/provider/AnnotationMetaDataProvider.java
    M engine/src/main/java/org/hibernate/validator/internal/util/ReflectionHelper.java
    M engine/src/test/java/org/hibernate/validator/test/internal/util/ReflectionHelperTest.java

  Log Message:
  -----------
  HV-1165 Support type argument constraints for Set

This is a follow-up of HV-1062: it fixed the Map case but introduced a
regression for the Set case.

Also unify the tests so that they cover correctly all cases.

Also fix a misleading comment which led to this mistake.

Note that a few tests for Optional method validation got removed, they
will be reintroduced in a follow-up commit.


  Commit: bcfd0359998d5c35bb7e24aea03764e54e754b61
      https://github.com/hibernate/hibernate-validator/commit/bcfd0359998d5c35bb7e24aea03764e54e754b61
  Author: Guillaume Smet <guillaume.smet at gmail.com>
  Date:   2016-12-07 (Wed, 07 Dec 2016)

  Changed paths:
    M engine-jdk8-tests/src/test/java/org/hibernate/validator/test/internal/engine/typeannotationconstraint/TypeAnnotationConstraintTest.java
    M engine/src/main/java/org/hibernate/validator/internal/metadata/raw/ConstrainedExecutable.java
    M engine/src/main/java/org/hibernate/validator/internal/metadata/raw/ConstrainedField.java
    M engine/src/main/java/org/hibernate/validator/internal/metadata/raw/ConstrainedParameter.java

  Log Message:
  -----------
  HV-1176 Fix type argument constraints detection.

They could be ignored if they were the only constraints of a method's
parameters or return value.

This issue was detected by separating the tests of List and Optional:
the presence of an @Valid annotation for the List made the test work.


Compare: https://github.com/hibernate/hibernate-validator/compare/6c504858636a...bcfd0359998d