From 937a3abca513fefc1596d90ececcf5b63446dc4d Mon Sep 17 00:00:00 2001 From: Tim Worrall Date: Tue, 25 Aug 2015 10:56:15 -0400 Subject: [PATCH] No Jira issue. When a person has no webpages, the group_concat in the query resulted in a non-empty result set: one that had no values but was nonetheless not empty. Added a check to see if the node was null. --- .../vitro/webapp/dao/jena/ObjectPropertyStatementDaoJena.java | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/webapp/src/edu/cornell/mannlib/vitro/webapp/dao/jena/ObjectPropertyStatementDaoJena.java b/webapp/src/edu/cornell/mannlib/vitro/webapp/dao/jena/ObjectPropertyStatementDaoJena.java index 7ecb92dc0..895c0a439 100644 --- a/webapp/src/edu/cornell/mannlib/vitro/webapp/dao/jena/ObjectPropertyStatementDaoJena.java +++ b/webapp/src/edu/cornell/mannlib/vitro/webapp/dao/jena/ObjectPropertyStatementDaoJena.java @@ -299,7 +299,6 @@ public class ObjectPropertyStatementDaoJena extends JenaBaseDao implements Objec if("desc".equalsIgnoreCase( sortDirection ) ){ queryString = queryString.replaceAll(" ASC\\(", " DESC("); } - ResultSet results = (constructedModel == null) ? selectFromRDFService( queryString, subjectUri, propertyUri, domainUri, rangeUri) : selectFromConstructedModel( queryString, subjectUri, propertyUri, domainUri, rangeUri, constructedModel); @@ -307,7 +306,7 @@ public class ObjectPropertyStatementDaoJena extends JenaBaseDao implements Objec while (results.hasNext()) { QuerySolution soln = results.nextSolution(); RDFNode node = soln.get(objectKey); - if (node.isURIResource()) { + if (node != null && node.isURIResource()) { list.add(QueryUtils.querySolutionToStringValueMap(soln)); } }