Added capability to filter apps with system installed apps or user installed apps #539
Open
ruwin
wants to merge 3 commits from ruwin/device-mgt-core:classify-apps
into master
Loading…
Reference in new issue
There is no content yet.
Delete Branch 'ruwin/device-mgt-core:classify-apps'
Deleting a branch is permanent. It CANNOT be undone. Continue?
Purpose
Description
This pull request addresses an improvement to filter apps based on user installed apps and system installed apps.
byte[] appProperties = rs.getBytes("APP_PROPERTIES");
bais = new ByteArrayInputStream(appProperties);
ois = new ObjectInputStream(bais);
It is better to use try with resources when using ByteArrayInputStream and ObjectInputStream.
} catch (ClassNotFoundException e) {
throw new DeviceManagementDAOException("Class not found error occurred fetch at app properties", e);
} catch (SQLException e) {
throw new DeviceManagementDAOException("SQL error occurred fetch at application", e);
Log the exceptions before throwing
stmt.setInt(3, tenantId);
stmt.setInt(4, limit);
stmt.setInt(5, offset);
int paramIndex = 4;
Isn't it more elegant to use this paramIndex starting from the 1st index instead of from here.
Reviewers