litvinovg/vitro
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fixed check for redundant possible object properties (#358)

Browse source 
Co-authored-by: Georgy Litvinov <georgy.litvinov@tib.eu>
This commit is contained in:
Georgy Litvinov 2023-01-24 13:09:57 +01:00 • committed by GitHub
parent a7215cc932
commit 7dabc6b2ab
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 11 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
api/src/main/java/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/GroupedPropertyList.java
View file

@ -258,11 +258,9 @@ public class GroupedPropertyList extends BaseTemplateModel {
if (possibleOP == null) { if (possibleOP == null) {
continue; continue;
} }
for(ObjectProperty populatedOP : populatedOPs) { if (isInPopulatedOPs(populatedOPs, possibleOP)) {
if (redundant(populatedOP, possibleOP)) {
continue; continue;
} }
}
if (!vClassUris.contains(possibleOP.getDomainVClassURI())) { if (!vClassUris.contains(possibleOP.getDomainVClassURI())) {
continue; continue;
} }
@ -288,6 +286,15 @@ public class GroupedPropertyList extends BaseTemplateModel {
return possibleProperties; return possibleProperties;
} }
private boolean isInPopulatedOPs(List<ObjectProperty> populatedOPs, ObjectProperty possibleOP) {
for(ObjectProperty populatedOP : populatedOPs) {
if (redundant(populatedOP, possibleOP)) {
return true;
}
}
return false;
}
/** /**
* Don't know what the real problem is with VIVO-976, but somehow we have the same property * Don't know what the real problem is with VIVO-976, but somehow we have the same property
* showing up once with a blank node as a domain, and once with null or OWL:Thing as a domain. * showing up once with a blank node as a domain, and once with null or OWL:Thing as a domain.