fixed/modified setApprovalFlag
parent
4ecaea20f8
commit
64b0916970
|
|
@ -135,21 +135,21 @@ public class Change {
|
|||
change.setChangeImplementerPersonId(getValueStringByID(entry, "ChangeImplementerPersonId"));
|
||||
change.setPackageType(getValueStringByID(entry, "PackageType"));
|
||||
change.setContract(getValueStringByID(entry, "Contract"));
|
||||
try {
|
||||
change.setApprovalFlag(inApprovalList(api.getUser(), getValueStringByID(entry, "ChangerNr")));
|
||||
} catch (ARException e) {
|
||||
e.printStackTrace();
|
||||
}
|
||||
change.setImplementerEdit(
|
||||
Optional.ofNullable(entry.get(queryChange.getFieldId("ChangeCoordinator")))
|
||||
.map(Object::toString)
|
||||
.filter(peopleFullName::equals)
|
||||
.isPresent());
|
||||
try {
|
||||
var flag = inApprovalList(api.getUser(), getValueStringByID(entry, "ChangeNr"));
|
||||
change.setApprovalFlag(flag);
|
||||
} catch (ARException e) {
|
||||
e.printStackTrace();
|
||||
}
|
||||
changes.add(change);
|
||||
});
|
||||
|
||||
api.freeImpersonatedUser();
|
||||
|
||||
return new ChangeResponse(entriesSize, changes);
|
||||
}
|
||||
|
||||
|
|
@ -160,8 +160,8 @@ public class Change {
|
|||
var approversOI = api.queryFieldsById("\'Ticketnumber\' = \"" + changeNr + "\"",
|
||||
queryApprovalList.getFieldIds(), queryApprovalList.getFormName(), null, 0, 0);
|
||||
|
||||
String approvers = approversOI.get(0).get(queryApprovalList.getFieldId("Approvers")).toString();
|
||||
return approvers.contains(user);
|
||||
var approvers = !approversOI.isEmpty() ? approversOI.get(0).get(queryApprovalList.getFieldId("Approvers")).toString() : null;
|
||||
return approvers != null ? approvers.contains(user) : false;
|
||||
}
|
||||
|
||||
/**
|
||||
|
|
|
|||
Loading…
Reference in New Issue