Add braces to clarify the logic structure, which was not as apparently intended.
This commit is contained in:
parent
f0c7ab7b95
commit
16d14cb86c
1 changed files with 12 additions and 7 deletions
|
@ -163,13 +163,18 @@ public class ProhibitedFromUpdateBelowRoleLevelFilter extends VitroFiltersImpl {
|
||||||
|
|
||||||
try {
|
try {
|
||||||
ObjectProperty prop = stmt.getProperty();
|
ObjectProperty prop = stmt.getProperty();
|
||||||
if( prop == null )
|
if (prop == null) {
|
||||||
prop = wdf.getObjectPropertyDao().getObjectPropertyByURI( stmt.getPropertyURI() );
|
prop = wdf.getObjectPropertyDao().getObjectPropertyByURI(stmt.getPropertyURI());
|
||||||
if( prop == null )
|
}
|
||||||
if( ! canViewOddItems() ){ return false; }
|
if (prop == null) {
|
||||||
else
|
if (!canViewOddItems()) {
|
||||||
if( sameLevelOrHigher( prop.getProhibitedFromUpdateBelowRoleLevel()) == false)
|
return false;
|
||||||
return false;
|
}
|
||||||
|
} else {
|
||||||
|
if (sameLevelOrHigher(prop.getProhibitedFromUpdateBelowRoleLevel()) == false) {
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
Individual subject =
|
Individual subject =
|
||||||
(stmt.getSubject() != null ? stmt.getSubject() : wdf.getIndividualDao().getIndividualByURI( stmt.getSubjectURI()));
|
(stmt.getSubject() != null ? stmt.getSubject() : wdf.getIndividualDao().getIndividualByURI( stmt.getSubjectURI()));
|
||||||
|
|
Loading…
Add table
Reference in a new issue