Skip to content

Commit 341ae94

Browse files
committed
fixes and formatting
1 parent 0a8235a commit 341ae94

File tree

5 files changed

+9
-2
lines changed

5 files changed

+9
-2
lines changed

core/model-api/src/main/java/org/eclipse/rdf4j/model/Literal.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,11 +56,12 @@ enum BaseDirection {
5656
this.suffix = suffix;
5757
}
5858

59+
@Override
5960
public String toString() {
6061
return suffix;
6162
}
6263

63-
public static BaseDirection fromString(String dir) {
64+
public static BaseDirection fromString(final String dir) {
6465
if (dir == null || dir.isEmpty())
6566
return NONE;
6667
if (dir.equals(LTR_SUFFIX))

core/model-api/src/main/java/org/eclipse/rdf4j/model/base/AbstractValueFactory.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -154,6 +154,7 @@ public Literal createLiteral(String label, String language, Literal.BaseDirectio
154154

155155
Objects.requireNonNull(label, "null label");
156156
Objects.requireNonNull(language, "null language");
157+
Objects.requireNonNull(baseDirection, "null baseDirection");
157158

158159
if (language.isEmpty()) {
159160
throw new IllegalArgumentException("empty language tag");

core/model/src/main/java/org/eclipse/rdf4j/model/impl/SimpleLiteral.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -177,6 +177,7 @@ protected void setLanguage(String language) {
177177
}
178178

179179
protected void setBaseDirection(BaseDirection baseDirection) {
180+
Objects.requireNonNull(baseDirection, "null baseDirection");
180181
this.baseDirection = baseDirection;
181182
if (this.baseDirection != BaseDirection.NONE) {
182183
setDatatype(CoreDatatype.RDF.DIRLANGSTRING);

core/sail/elasticsearch-store/src/main/java/org/eclipse/rdf4j/sail/elasticsearchstore/ElasticsearchDataStructure.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -244,7 +244,8 @@ private QueryBuilder getQueryBuilder(Resource subject, IRI predicate, Value obje
244244
boolQueryBuilder
245245
.must(QueryBuilders.termQuery("object_Lang", ((Literal) object).getLanguage().get()));
246246
boolQueryBuilder
247-
.must(QueryBuilders.termQuery("object_LangDir", ((Literal) object).getBaseDirection().toString()));
247+
.must(QueryBuilders.termQuery("object_LangDir",
248+
((Literal) object).getBaseDirection().toString()));
248249
}
249250
}
250251
}

core/sail/lmdb/src/main/java/org/eclipse/rdf4j/sail/lmdb/model/LmdbLiteral.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@
1111
package org.eclipse.rdf4j.sail.lmdb.model;
1212

1313
import java.io.ObjectStreamException;
14+
import java.util.Objects;
1415
import java.util.Optional;
1516

1617
import org.eclipse.rdf4j.model.IRI;
@@ -90,6 +91,8 @@ public LmdbLiteral(ValueStoreRevision revision, String label, String lang, BaseD
9091

9192
public LmdbLiteral(ValueStoreRevision revision, String label, String language, BaseDirection baseDirection,
9293
long internalID) {
94+
Objects.requireNonNull(language, "null language");
95+
Objects.requireNonNull(baseDirection, "null baseDirection");
9396
this.label = label;
9497
this.language = language;
9598
this.baseDirection = baseDirection;

0 commit comments

Comments
 (0)