OLMIS-1973 fixed 500 error code while converting to order
OLMIS-1973 fixed 500 error code while converting to order
OLMIS-1973 fixed 500 error code while converting to order

While converting RequisitionLineItems to OrderLineItems RequisitionLineItems marked as skipped are filtered out

OLMIS-1474 fixed debug message when supported program not found

it's because we can hardcode requested quantity for every line item but every line item has a different price per pack and size of a pack so packtoShip will also be different for every line item so...

it's because we can hardcode requested quantity for every line item but every line item has a different price per pack and size of a pack so packtoShip will also be different for every line item so I can't hardcode totalCost here

OLMIS-1474 When I enter Program UID in SupportedPrograms message error is not correct.
OLMIS-1474 When I enter Program UID in SupportedPrograms message error is not correct.
Sebastian Brudziński are issues resolved?

Sebastian Brudziński are issues resolved?

OLMIS-1474 facilityController searches program by id or code

when creating facility, controller will search program by id if code is not provided,

endpoint will return 400 if id nor code are provided,

added message to return when both id and code are null.

OLMIS-1802 removed unnecessary code, minor codestyle fixes

OLMIS-1802 fixed failing contract test

removed hardcoded totalCost values and added method to count totalCost for each line item.

looks like it is needed to obtain proper requisition to work with in later steps, without it, they throw an NPE.

looks like it is needed to obtain proper requisition to work with in later steps, without it, they throw an NPE.

because right should be named facility_approved_orderables_manage not facility_approved_product_manage like I did before Josh commented on the ticket, so I removed this file but it never left my lo...

because right should be named facility_approved_orderables_manage not facility_approved_product_manage like I did before Josh commented on the ticket, so I removed this file but it never left my local repository so I thought there is no problem since it was never on remote repo.

OLMIS-1802 added test for calculating total cost
OLMIS-1802 added test for calculating total cost
OLMIS-1802 added test for calculating total cost

OLMIS-1593 Create Right: Manage facility approved products (orderables)
OLMIS-1593 Create Right: Manage facility approved products (orderables)
OLMIS-1593 fixed failing integration tests

added equals and hashCode methods to FacilityTypeApprovedProduct,

removed unused imports,

added doc to ProgramOrderableDeserializer.

OLMIS-1593 changed right name to FACILITY_APPROVED_ORDERABLES_MANAGE

fixed some integration tests,

added deserializer for ProgramOrderable.java,

changed response code for facilityTypeApprovedProducts/{id} DELETE endpoint to 204 from 200.

    • -0
    • +12
    /src/main/resources/db/migration/20170216082252561__create_facility_approved_orderables_manage_right.sql
OLMIS-1593 Created FACILITY_APPROVED_PRODUCT_MANAGE right

added FACILITY_APPROVED_PRODUCT_MANAGE right,

added right check to facilityTypeApprovedProducts endpoints,

added integration tests,

modified raml.

    • -0
    • +16
    /src/main/resources/api-definition.yaml
    • -0
    • +12
    /src/main/resources/db/migration/20170215122344292__create_manage_facility_approved_product_right.sql
OLMIS-1913 modified number to be constatnt

Klaudia Pałkowska told me that this is the way this is calculated

Klaudia Pałkowska told me that this is the way this is calculated

OLMIS-1913 added validation for totalStockoutDays
OLMIS-1913 added validation for totalStockoutDays
OLMIS-1913 added validation for totalStockoutDays

added validation for field totalStockOutDays,

added unit test,

added proper message.

OLMIS-1859 modified 3 integration tests

simplified extracting List from endpoints responses where it was possible,

in shouldFindRequisitionWithStatuses,

in shouldReturnOnlyRequisitionsForWhichUserHasRights,

in shouldReturnEmptyListIfUserHasNoRightsToSeeFoundedRequisitions.

OLMIS-1859 modified 3 integration tests

simplified extracting List from endpoints responses where it was possible,

in shouldFindRequisitionWithStatuses,

in shouldReturnOnlyRequisitionsForWhichUserHasRights,

in shouldReturnEmptyListIfUserHasNoRightsToSeeFoundedRequisitions.

it is used in almost every test to receive a List from the response, and in 807 line test you asked me to try not use this to get List so I guess this can be done simpler way. should I change it in...

it is used in almost every test to receive a List from the response, and in 807 line test you asked me to try not use this to get List so I guess this can be done simpler way. should I change it in every other test?