Build #535

Plan branch:
OpenMRS Core Master

Build: #535 failed

Job: Integration Test was successful

Stages & jobs

  1. Build and Test

Code commits

openmrs-core

  • Daniel Kayiwa

    Daniel Kayiwa 367d95ae80288f62763fc671785bd2a969939c79

    Merge pull request #995 from chocobar/master
    TRUNK-4208 Gender and Birth date should not be required for persons linked to providers or users but to patients

  • Priya Chandran <prichand@thoughtworks.com>

    Priya Chandran <prichand@thoughtworks.com> 795ad4117f55098ba1e8d044e928a88668233fcd

    TRUNK-4208: Removed gender as a requirement during PersonValidation, added it instead to PatientValidation.
    - Moved unit tests from PersonValidatorTest to PatientValidatorTest to reflect this new behaviour.

    - Birthdate is already *not* a requirement for Persons.

    - Added test for PersonValidator to check that it's not validating gender requirement.

    TRUNK-4208: Refactored PersonValidator. Extracted birth date validation into it's own method.

    • api/src/main/java/org/openmrs/validator/PatientValidator.java (version 795ad4117f55098ba1e8d044e928a88668233fcd)
    • api/src/main/java/org/openmrs/validator/PersonValidator.java (version 795ad4117f55098ba1e8d044e928a88668233fcd)
    • api/src/test/java/org/openmrs/validator/PatientValidatorTest.java (version 795ad4117f55098ba1e8d044e928a88668233fcd)
    • api/src/test/java/org/openmrs/validator/PersonValidatorTest.java (version 795ad4117f55098ba1e8d044e928a88668233fcd)