The if statement that uses the new ignore policy was improperly

implemented.  Updated so that any non-revision attribute will save
This commit is contained in:
Cyrus 2024-03-13 20:18:39 -04:00
parent a9e403d1a0
commit 6a13d8dbb1

View File

@ -266,13 +266,16 @@ public class CertificateAttributeScvValidator extends SupplyChainCredentialValid
componentInfos, componentResults);
int numOfAttributes = 0;
if (!remainingComponentResults.isEmpty()) {
List<ComponentAttributeResult> attributeResults = checkComponentClassMap(
componentInfos, remainingComponentResults);
boolean saveAttributeResult;
for (ComponentAttributeResult componentAttributeResult : attributeResults) {
if (componentAttributeResult.getAttribute().equalsIgnoreCase(ComponentResult.ATTRIBUTE_REVISION)
&& !ignoreRevisionAttribute) {
saveAttributeResult = true;
if (ignoreRevisionAttribute) {
saveAttributeResult = !componentAttributeResult.getAttribute()
.equalsIgnoreCase(ComponentResult.ATTRIBUTE_REVISION);
}
if (saveAttributeResult) {
componentAttributeResult.setProvisionSessionId(provisionSessionId);
componentAttributeRepository.save(componentAttributeResult);
fieldValidation &= componentAttributeResult.checkMatchedStatus();
@ -372,11 +375,14 @@ public class CertificateAttributeScvValidator extends SupplyChainCredentialValid
if (!remainingComponentResults.isEmpty()) {
List<ComponentAttributeResult> attributeResults = checkComponentClassMap(
componentInfos, remainingComponentResults);
boolean saveAttributeResult;
for (ComponentAttributeResult componentAttributeResult : attributeResults) {
if (componentAttributeResult.getAttribute()
.equalsIgnoreCase(ComponentResult.ATTRIBUTE_REVISION)
&& !ignoreRevisionAttribute) {
saveAttributeResult = true;
if (ignoreRevisionAttribute) {
saveAttributeResult = !componentAttributeResult.getAttribute()
.equalsIgnoreCase(ComponentResult.ATTRIBUTE_REVISION);
}
if (saveAttributeResult) {
componentAttributeResult.setProvisionSessionId(provisionSessionId);
componentAttributeRepository.save(componentAttributeResult);
fieldValidation &= componentAttributeResult.checkMatchedStatus();