VClassGroupCache updates on class name changes NIHVIVO-2934
This commit is contained in:
briancaruso
parent
7f8db83ec1
commit
0cd6c1227f
2 changed files with 127 additions and 18 deletions
|
|
@ -16,11 +16,13 @@ import javax.servlet.ServletContextListener;
|
|||
import org.apache.commons.logging.Log;
|
||||
import org.apache.commons.logging.LogFactory;
|
||||
|
||||
import com.hp.hpl.jena.ontology.Individual;
|
||||
import com.hp.hpl.jena.ontology.OntModel;
|
||||
import com.hp.hpl.jena.rdf.listeners.StatementListener;
|
||||
import com.hp.hpl.jena.rdf.model.ResourceFactory;
|
||||
import com.hp.hpl.jena.rdf.model.Statement;
|
||||
import com.hp.hpl.jena.shared.Lock;
|
||||
import com.hp.hpl.jena.vocabulary.OWL;
|
||||
import com.hp.hpl.jena.vocabulary.RDF;
|
||||
import com.hp.hpl.jena.vocabulary.RDFS;
|
||||
|
||||
|
|
@ -208,6 +210,24 @@ public class VClassGroupCache {
|
|||
return groups;
|
||||
}
|
||||
|
||||
protected static boolean isClassNameChange(Statement stmt, OntModel jenaOntModel) {
|
||||
// Check if the stmt is a rdfs:label change and that the
|
||||
// subject is an owl:Class.
|
||||
if( RDFS.label.equals( stmt.getPredicate() )) {
|
||||
jenaOntModel.enterCriticalSection(Lock.READ);
|
||||
try{
|
||||
return jenaOntModel.contains(
|
||||
ResourceFactory.createStatement(
|
||||
ResourceFactory.createResource(stmt.getSubject().getURI()),
|
||||
RDF.type,
|
||||
OWL.Class));
|
||||
}finally{
|
||||
jenaOntModel.leaveCriticalSection();
|
||||
}
|
||||
}else{
|
||||
return false;
|
||||
}
|
||||
}
|
||||
|
||||
/* ******************** RebuildGroupCacheThread **************** */
|
||||
|
||||
|
|
@ -310,7 +330,7 @@ public class VClassGroupCache {
|
|||
}
|
||||
|
||||
/* ****************** Jena Model Change Listener***************************** */
|
||||
private class VClassGroupCacheChangeListener extends StatementListener {
|
||||
protected class VClassGroupCacheChangeListener extends StatementListener {
|
||||
public void addedStatement(Statement stmt) {
|
||||
checkAndDoUpdate(stmt);
|
||||
}
|
||||
|
|
@ -332,24 +352,11 @@ public class VClassGroupCache {
|
|||
requestCacheUpdate();
|
||||
} else if(VitroVocabulary.DISPLAY_RANK.equals(stmt.getPredicate().getURI())){
|
||||
requestCacheUpdate();
|
||||
} else if( isVClassGroupNameChange(stmt) ) {
|
||||
requestCacheUpdate();
|
||||
}
|
||||
}
|
||||
|
||||
protected boolean isVClassGroupNameChange(Statement stmt) {
|
||||
// Check if the stmt is a rdfs:label change and that the
|
||||
// subject is a VClassGroup.
|
||||
if( RDFS.label.equals( stmt.getPredicate() )) {
|
||||
} else {
|
||||
OntModel jenaOntModel = ModelContext.getJenaOntModel(context);
|
||||
jenaOntModel.enterCriticalSection(Lock.READ);
|
||||
try{
|
||||
return jenaOntModel.contains(stmt.getSubject(), RDF.type, ResourceFactory.createResource(VitroVocabulary.CLASSGROUP));
|
||||
}finally{
|
||||
jenaOntModel.leaveCriticalSection();
|
||||
if( isClassNameChange(stmt, jenaOntModel) ) {
|
||||
requestCacheUpdate();
|
||||
}
|
||||
}else{
|
||||
return false;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
|
|
|||
|
|
@ -0,0 +1,102 @@
|
|||
/* $This file is distributed under the terms of the license in /doc/license.txt$ */
|
||||
|
||||
package edu.cornell.mannlib.vitro.webapp.dao.jena;
|
||||
|
||||
import java.io.StringReader;
|
||||
|
||||
import org.junit.Assert;
|
||||
import org.junit.Before;
|
||||
import org.junit.Test;
|
||||
|
||||
import com.hp.hpl.jena.ontology.OntModel;
|
||||
import com.hp.hpl.jena.rdf.model.ModelFactory;
|
||||
import com.hp.hpl.jena.rdf.model.ResourceFactory;
|
||||
import com.hp.hpl.jena.rdf.model.Statement;
|
||||
import com.hp.hpl.jena.vocabulary.RDFS;
|
||||
|
||||
public class VClassGroupCacheTest {
|
||||
|
||||
@Before
|
||||
public void setUp() throws Exception {
|
||||
}
|
||||
|
||||
@Test
|
||||
public void testIsVClassGroupNameChange() {
|
||||
|
||||
|
||||
//protected static boolean isVClassGroupNameChange(Statement stmt, OntModel jenaOntModel) {
|
||||
|
||||
String rdf =
|
||||
"core:Summer \n" +
|
||||
" a owl:Class ; \n" +
|
||||
" rdfs:label \"Spring and toast 3\"@en-US ; \n" +
|
||||
" rdfs:subClassOf owl:Thing ; \n" +
|
||||
" vitro:descriptionAnnot \n" +
|
||||
" \"cgaa\"^^xsd:string ; \n" +
|
||||
" vitro:displayLimitAnnot \n" +
|
||||
" \"-1\"^^xsd:int ; \n" +
|
||||
" vitro:displayRankAnnot \n" +
|
||||
" \"-1\"^^xsd:int ; \n" +
|
||||
" vitro:exampleAnnot \"serefs\"^^xsd:string ; \n" +
|
||||
" vitro:hiddenFromDisplayBelowRoleLevelAnnot \n" +
|
||||
" <http://vitro.mannlib.cornell.edu/ns/vitro/role#public> ; \n" +
|
||||
" vitro:inClassGroup <http://vivoweb.org/ontology#vitroClassGroupequipment> ; \n" +
|
||||
" vitro:prohibitedFromUpdateBelowRoleLevelAnnot \n" +
|
||||
" <http://vitro.mannlib.cornell.edu/ns/vitro/role#public> ; \n" +
|
||||
" vitro:shortDef \"sfsfe\"^^xsd:string ; \n" +
|
||||
" owl:equivalentClass core:Summer . ";
|
||||
|
||||
OntModel om = ModelFactory.createOntologyModel();
|
||||
om.read( new StringReader( prefixes + rdf) , null , "N3");
|
||||
|
||||
Statement stmt = ResourceFactory.createStatement(
|
||||
ResourceFactory.createResource("http://vivoweb.org/ontology/core#Summer"),
|
||||
RDFS.label,
|
||||
ResourceFactory.createPlainLiteral("some old label"));
|
||||
|
||||
boolean isNameChange = VClassGroupCache.isClassNameChange(stmt, om);
|
||||
Assert.assertTrue("Expected it to be a name change but it wasn't.", isNameChange);
|
||||
|
||||
stmt = ResourceFactory.createStatement(
|
||||
ResourceFactory.createResource("http://vivoweb.org/ontology/core#bogus"),
|
||||
ResourceFactory.createProperty("http://example.com/nonLabelProperty"),
|
||||
ResourceFactory.createPlainLiteral("some old label"));
|
||||
|
||||
boolean notNameChange = ! VClassGroupCache.isClassNameChange(stmt, om);
|
||||
Assert.assertTrue("Expected it to NOT be a name change but it was.", notNameChange);
|
||||
|
||||
stmt = ResourceFactory.createStatement(
|
||||
ResourceFactory.createResource("http://vivoweb.org/ontology/core#bogus"),
|
||||
RDFS.label,
|
||||
ResourceFactory.createPlainLiteral("some old label"));
|
||||
|
||||
notNameChange = ! VClassGroupCache.isClassNameChange(stmt, om);
|
||||
Assert.assertTrue("Expected it to NOT be a name change but it was.", notNameChange);
|
||||
}
|
||||
|
||||
static final String prefixes =
|
||||
"@prefix dc: <http://purl.org/dc/elements/1.1/> . \n" +
|
||||
"@prefix pvs: <http://vivoweb.org/ontology/provenance-support#> . \n" +
|
||||
"@prefix geo: <http://aims.fao.org/aos/geopolitical.owl#> . \n" +
|
||||
"@prefix foaf: <http://xmlns.com/foaf/0.1/> . \n" +
|
||||
"@prefix scires: <http://vivoweb.org/ontology/scientific-research#> . \n" +
|
||||
"@prefix scripps: <http://vivo.scripps.edu/> . \n" +
|
||||
"@prefix dcterms: <http://purl.org/dc/terms/> . \n" +
|
||||
"@prefix rdfs: <http://www.w3.org/2000/01/rdf-schema#> . \n" +
|
||||
"@prefix swrl: <http://www.w3.org/2003/11/swrl#> . \n" +
|
||||
"@prefix vitro: <http://vitro.mannlib.cornell.edu/ns/vitro/0.7#> . \n" +
|
||||
"@prefix event: <http://purl.org/NET/c4dm/event.owl#> . \n" +
|
||||
"@prefix bibo: <http://purl.org/ontology/bibo/> . \n" +
|
||||
"@prefix xsd: <http://www.w3.org/2001/XMLSchema#> . \n" +
|
||||
"@prefix owl: <http://www.w3.org/2002/07/owl#> . \n" +
|
||||
"@prefix swrlb: <http://www.w3.org/2003/11/swrlb#> . \n" +
|
||||
"@prefix rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#> . \n" +
|
||||
"@prefix core: <http://vivoweb.org/ontology/core#> . \n" +
|
||||
"@prefix skos: <http://www.w3.org/2004/02/skos/core#> . \n" +
|
||||
"@prefix vivo: <http://vivo.library.cornell.edu/ns/0.1#> . \n" +
|
||||
"@prefix dcelem: <http://purl.org/dc/elements/1.1/> . \n" +
|
||||
"@prefix ero: <http://purl.obolibrary.org/obo/> . \n" +
|
||||
" \n" ;
|
||||
|
||||
|
||||
}
|
||||
Loading…
Add table
Add a link
Reference in a new issue