This commit is contained in:
@@ -85,9 +85,9 @@ public class TariffService {
|
|||||||
public TariffDTO updateTariff(Long id, TariffDTO tariffDTO, TariffRequest tariffRequest) {
|
public TariffDTO updateTariff(Long id, TariffDTO tariffDTO, TariffRequest tariffRequest) {
|
||||||
Tariff tariff = tariffRepository.findById(id)
|
Tariff tariff = tariffRepository.findById(id)
|
||||||
.orElseThrow(() -> new TariffNotFoundException("Tariff not found"));
|
.orElseThrow(() -> new TariffNotFoundException("Tariff not found"));
|
||||||
updateTariffFromDTO(tariff, tariffDTO, tariffRequest);
|
Tariff updateTariffFromDTO = updateTariffFromDTO(tariff, tariffDTO, tariffRequest);
|
||||||
|
|
||||||
return convertToDTO(tariffRepository.save(tariff));
|
return convertToDTO(tariffRepository.save(updateTariffFromDTO));
|
||||||
}
|
}
|
||||||
|
|
||||||
@Transactional
|
@Transactional
|
||||||
@@ -112,7 +112,7 @@ public class TariffService {
|
|||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
private void updateTariffFromDTO(Tariff tariff, TariffDTO dto, TariffRequest tariffRequest) {
|
private Tariff updateTariffFromDTO(Tariff tariff, TariffDTO dto, TariffRequest tariffRequest) {
|
||||||
tariff.setType(tariffRequest.getType() == null ? dto.getType(): tariffRequest.getType());
|
tariff.setType(tariffRequest.getType() == null ? dto.getType(): tariffRequest.getType());
|
||||||
tariff.setName(tariffRequest.getName() == null ? dto.getName(): tariffRequest.getName());
|
tariff.setName(tariffRequest.getName() == null ? dto.getName(): tariffRequest.getName());
|
||||||
tariff.setPrice(tariffRequest.getPrice() == 0 ? dto.getPrice(): tariffRequest.getPrice());
|
tariff.setPrice(tariffRequest.getPrice() == 0 ? dto.getPrice(): tariffRequest.getPrice());
|
||||||
@@ -124,5 +124,7 @@ public class TariffService {
|
|||||||
tariff.setDescription(tariffRequest.getDescription() == null ? dto.getDescription(): tariffRequest.getDescription());
|
tariff.setDescription(tariffRequest.getDescription() == null ? dto.getDescription(): tariffRequest.getDescription());
|
||||||
tariff.setTariffTerm(tariffRequest.getTariffTerm() == null ? TariffTimeTerm.valueOf(dto.getTariffTerm().toUpperCase()):
|
tariff.setTariffTerm(tariffRequest.getTariffTerm() == null ? TariffTimeTerm.valueOf(dto.getTariffTerm().toUpperCase()):
|
||||||
TariffTimeTerm.valueOf(tariffRequest.getTariffTerm()));
|
TariffTimeTerm.valueOf(tariffRequest.getTariffTerm()));
|
||||||
|
|
||||||
|
return tariff;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user