Merge branch 'master' into 'master'

Fix ent.app install issue when installing via roles

See merge request entgra/carbon-device-mgt-plugins!71
revert-dabc3590
Milan Perera 5 years ago
commit 2c043363db

@ -109,11 +109,16 @@ public class ApplicationOperationsImpl implements ApplicationOperations {
.getDeviceManagementService(applicationOperationAction.getTenantId()). .getDeviceManagementService(applicationOperationAction.getTenantId()).
getAllDevicesOfRole(userRole); getAllDevicesOfRole(userRole);
for (org.wso2.carbon.device.mgt.common.Device device : deviceList) { for (org.wso2.carbon.device.mgt.common.Device device : deviceList) {
if (MDMAppConstants.ACTIVE.equalsIgnoreCase(device.getEnrolmentInfo().getStatus().toString())) { if (MDMAppConstants.WEBAPP.equals(applicationOperationAction.getApp().getPlatform())
|| applicationOperationAction.getApp().getPlatform()
.equalsIgnoreCase(device.getType())) {
if (MDMAppConstants.ACTIVE
.equalsIgnoreCase(device.getEnrolmentInfo().getStatus().toString())) {
deviceIdentifiers.add(getDeviceIdentifierByDevice(device)); deviceIdentifiers.add(getDeviceIdentifierByDevice(device));
} }
} }
} }
}
} catch (DeviceManagementException devMgtEx) { } catch (DeviceManagementException devMgtEx) {
String errorMsg = "Error occurred fetch device for user role " + userRole + " at app installation"; String errorMsg = "Error occurred fetch device for user role " + userRole + " at app installation";
logError(errorMsg, devMgtEx); logError(errorMsg, devMgtEx);

Loading…
Cancel
Save