From 1c4be522a9d0840e5df1d5b6ba0596e577a6fabc Mon Sep 17 00:00:00 2001
From: Yash S <119652212+yashmsonkusare@users.noreply.github.com>
Date: Tue, 12 Mar 2024 11:41:21 +0530
Subject: [PATCH] MOSIP-25073:Added the code fix for bugs in sonar analysis of
 admin-services

Signed-off-by: Yash S <119652212+yashmsonkusare@users.noreply.github.com>
---
 .../packetstatusupdater/util/RestClient.java     |  3 ++-
 .../service/impl/ApplicantDetailServiceImpl.java | 16 ++++++++++------
 2 files changed, 12 insertions(+), 7 deletions(-)

diff --git a/admin/admin-service/src/main/java/io/mosip/admin/packetstatusupdater/util/RestClient.java b/admin/admin-service/src/main/java/io/mosip/admin/packetstatusupdater/util/RestClient.java
index 23187f4162e..000ee0258fc 100644
--- a/admin/admin-service/src/main/java/io/mosip/admin/packetstatusupdater/util/RestClient.java
+++ b/admin/admin-service/src/main/java/io/mosip/admin/packetstatusupdater/util/RestClient.java
@@ -204,7 +204,8 @@ public <T> T getApi(String url,
 		try {
 			ResponseEntity responseEntity= (ResponseEntity) restTemplate
 					.exchange(url, HttpMethod.GET, setRequestHeader(null, null), responseType);
-			if(url.contains("datashare") && responseEntity.getHeaders().getContentType().equals(MediaType.APPLICATION_JSON)){
+			MediaType contentType = responseEntity.getHeaders().getContentType();
+			if(url.contains("datashare") && contentType == null || contentType.equals(MediaType.APPLICATION_JSON)){
 				throw new MasterDataServiceException(ApplicantDetailErrorCode.DATA_SHARE_EXPIRED_EXCEPTION.getErrorCode(),
 						ApplicantDetailErrorCode.DATA_SHARE_EXPIRED_EXCEPTION.getErrorMessage());
 			}
diff --git a/admin/admin-service/src/main/java/io/mosip/admin/service/impl/ApplicantDetailServiceImpl.java b/admin/admin-service/src/main/java/io/mosip/admin/service/impl/ApplicantDetailServiceImpl.java
index b37790e84ed..df9d14c5146 100644
--- a/admin/admin-service/src/main/java/io/mosip/admin/service/impl/ApplicantDetailServiceImpl.java
+++ b/admin/admin-service/src/main/java/io/mosip/admin/service/impl/ApplicantDetailServiceImpl.java
@@ -110,12 +110,16 @@ public ApplicantDetailsDto getApplicantDetails(String rid) throws Exception {
             JSONObject mapperIdentity=utility.getJSONObject(idenitityJsonObject,IDENTITY);
             List<String> mapperJsonKeys = new ArrayList<>(mapperIdentity.keySet());
             for(String valueObj: applicantDetails){
-                if(valueObj!=null && !valueObj.equalsIgnoreCase(ApplicantPhoto)){
-                    LinkedHashMap<String, String> jsonObject = utility.getJSONValue(mapperIdentity, valueObj);
-                    String value = jsonObject.get(VALUE);
-                    applicantDataMap.put(value,identityObj.get(value).toString());
-                } else if (valueObj.equalsIgnoreCase(ApplicantPhoto)) {
-                    getImageData(documents,applicantDataMap);
+                if(valueObj!=null) {
+                    if (!valueObj.equalsIgnoreCase(ApplicantPhoto)) {
+                        LinkedHashMap<String, String> jsonObject = utility.getJSONValue(mapperIdentity, valueObj);
+                        String value = jsonObject.get(VALUE);
+                        applicantDataMap.put(value, identityObj.get(value).toString());
+                    } else if (valueObj.equalsIgnoreCase(ApplicantPhoto)) {
+                        getImageData(documents, applicantDataMap);
+                    }
+                } else {
+                    throw new NullPointerException("valueObj is null");
                 }
             }
             saveApplicantLoginDetails();