From a6a9e82c35a2069ac9865096476c0d158cc9d0f2 Mon Sep 17 00:00:00 2001
From: Gimhan-minion <gimhan@entgra.io>
Date: Thu, 1 Jun 2023 21:57:05 +0530
Subject: [PATCH] Update debug and error logs

---
 .../mgt/impl/DeviceInformationManagerImpl.java       | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/components/device-mgt/io.entgra.device.mgt.core.device.mgt.core/src/main/java/io/entgra/device/mgt/core/device/mgt/core/device/details/mgt/impl/DeviceInformationManagerImpl.java b/components/device-mgt/io.entgra.device.mgt.core.device.mgt.core/src/main/java/io/entgra/device/mgt/core/device/mgt/core/device/details/mgt/impl/DeviceInformationManagerImpl.java
index 5c54bc8ea1..13867d1d4d 100644
--- a/components/device-mgt/io.entgra.device.mgt.core.device.mgt.core/src/main/java/io/entgra/device/mgt/core/device/mgt/core/device/details/mgt/impl/DeviceInformationManagerImpl.java
+++ b/components/device-mgt/io.entgra.device.mgt.core.device.mgt.core/src/main/java/io/entgra/device/mgt/core/device/mgt/core/device/details/mgt/impl/DeviceInformationManagerImpl.java
@@ -419,26 +419,32 @@ public class DeviceInformationManagerImpl implements DeviceInformationManager {
     @Override
     public void deleteDeviceLocation(Device device) throws DeviceDetailsMgtException {
         try {
+            if (log.isDebugEnabled()) {
+                log.debug("Deleting device location for device: " + device.getId());
+            }
             DeviceManagementDAOFactory.beginTransaction();
             DeviceLocation deviceLocation = deviceDetailsDAO.getDeviceLocation(device.getId(),
                     device.getEnrolmentInfo().getId());
             if (deviceLocation != null) {
                 deviceDetailsDAO.deleteDeviceLocation(device.getId(), device.getEnrolmentInfo().getId());
             } else {
-                throw new DeviceDetailsMgtException("Device location not found.");
+                log.error("Device location not found for device: " + device.getId());
+                throw new DeviceDetailsMgtException("Device location not found for device: " + device.getId());
             }
             DeviceManagementDAOFactory.commitTransaction();
         } catch (TransactionManagementException e) {
-            throw new DeviceDetailsMgtException("Transactional error occurred while deleting the device location " + "information.", e);
+            log.error("Transactional error occurred while deleting the device location information. Device ID: " + device.getId(), e);
+            throw new DeviceDetailsMgtException("Transactional error occurred while deleting the device location " +
+                    "information.", e);
         } catch (DeviceDetailsMgtDAOException e) {
             DeviceManagementDAOFactory.rollbackTransaction();
+            log.error("Error occurred while deleting the device location information. Device ID: " + device.getId(), e);
             throw new DeviceDetailsMgtException("Error occurred while deleting the device location information.", e);
         } finally {
             DeviceManagementDAOFactory.closeConnection();
         }
     }
 
-
     @Override
     public void addDeviceLocations(Device device, List<DeviceLocation> deviceLocations) throws DeviceDetailsMgtException {
         try {