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

Add subject to reindex in AdditionalURIsForObjectProperties (#225)

Browse source 
Co-authored-by: Georgy Litvinov <git@litvinovg.pro>
This commit is contained in:
Ben 2021-04-09 08:39:15 -06:00 • committed by GitHub
parent ebca021bc4
commit b283a8342a
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 10 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
api/src/main/java/edu/cornell/mannlib/vitro/webapp/searchindex/indexing/AdditionalURIsForObjectProperties.java
View file

@ -59,16 +59,16 @@ public class AdditionalURIsForObjectProperties implements IndexingUriFinder, Con
public void endIndexing() { /* nothing to do */ } public void endIndexing() { /* nothing to do */ }
protected List<String> doObjectPropertyStmt(Statement stmt) { protected List<String> doObjectPropertyStmt(Statement stmt) {
// Only need to consider the object since the subject
// will already be updated in search index as part of
// SearchReindexingListener.
// Also, context nodes are not handled here. They are // Also, context nodes are not handled here. They are
// taken care of in AdditionalURIsForContextNodex. // taken care of in AdditionalURIsForContextNodex.
if( stmt.getObject().isURIResource() ) List<String> uris = new ArrayList<String>();
return Collections.singletonList( stmt.getObject().as(Resource.class).getURI() ); if( stmt.getObject().isURIResource() ) {
else uris.add(stmt.getObject().as(Resource.class).getURI() );
return Collections.emptyList(); }
if( stmt.getSubject().isURIResource() ) {
uris.add(stmt.getSubject().as(Resource.class).getURI() );
}
return uris;
} }
protected List<String> doDataPropertyStmt(Statement stmt) { protected List<String> doDataPropertyStmt(Statement stmt) {

3
api/src/test/java/edu/cornell/mannlib/vitro/webapp/searchindex/indexing/AdditionalURIsForObjectPropertiesTest.java
View file

@ -90,13 +90,14 @@ public class AdditionalURIsForObjectPropertiesTest {
Assert.assertTrue("uris was empty", uris.size() > 0 ); Assert.assertTrue("uris was empty", uris.size() > 0 );
Assert.assertTrue("uris didn't not contain test:cheese", uris.contains(testNS+"cheese")); Assert.assertTrue("uris didn't not contain test:cheese", uris.contains(testNS+"cheese"));
Assert.assertTrue("uris didn't not contain test:bob", uris.contains(testNS+"bob"));
Assert.assertTrue("uris contained test:Person", !uris.contains(testNS+"Person")); Assert.assertTrue("uris contained test:Person", !uris.contains(testNS+"Person"));
Assert.assertTrue("uris contained owl:Thing", !uris.contains( OWL.Thing.getURI() )); Assert.assertTrue("uris contained owl:Thing", !uris.contains( OWL.Thing.getURI() ));
Assert.assertTrue("uris contained test:onions", !uris.contains(testNS+"onions")); Assert.assertTrue("uris contained test:onions", !uris.contains(testNS+"onions"));
Assert.assertTrue("uris contained test:icecream", !uris.contains(testNS+"icecream")); Assert.assertTrue("uris contained test:icecream", !uris.contains(testNS+"icecream"));
Assert.assertEquals(1, uris.size()); Assert.assertEquals(2, uris.size());
} }
@Test @Test