fixed sonarlint issue

Author: a-brandt

src/main/java/com/arangodb/entity/BaseDocument.java

@@ -61,44 +61,6 @@ public BaseDocument(String documentKey) {
 		this.documentKey = documentKey;
 	}
 
-	// /**
-	// * @param keyValues a set of key/value pairs containing the attributes for
-	// the document.
-	// * The length has to be even and each even entry has to be of type String.
-	// * If not an empty document will be created
-	// */
-	// public BaseDocument(Object ...keyValues) {
-	// this(null, keyValues);
-	// }
-	//
-	// /**
-	// * create a BaseDocument with a given key and attributes defined in
-	// keyValues
-	// *
-	// * @param documentKey the unique key of the document
-	// * @param keyValues a set of key/value pairs containing the attributes for
-	// the document.
-	// * The length has to be even and each even entry has to be of type String.
-	// * If not an empty document will be created
-	// */
-	// public BaseDocument(String documentKey, Object ...keyValues) {
-	// this.init();
-	// if (documentKey != null) {
-	// this.documentKey = documentKey;
-	// }
-	// if (checkKeyValues(keyValues)) {
-	// for (int i = 0; i < keyValues.length; i = i+2) {
-	// if (keyValues[i] == REV) {
-	// this.documentRevision = (Long) keyValues[i+1];
-	// } else if (keyValues[i] == KEY && documentKey == null) {
-	// this.documentKey = (String) keyValues[i+1];
-	// } else {
-	// this.addAttribute((String) keyValues[i], keyValues[i + 1]);
-	// }
-	// }
-	// }
-	// }
-
 	/**
 	 * create an BaseDocument with given attributes
 	 *
@@ -221,23 +183,4 @@ public String toString() {
 				+ ", documentKey=" + documentKey + ", properties=" + properties + "]";
 	}
 
-	// /**
-	// * check the list if it is suitable
-	// *
-	// * @param keyValues
-	// * @return true, if the list has an even number and is an alternating
-	// sequence of instances of String and Object.
-	// */
-	// private boolean checkKeyValues(Object... keyValues) {
-	// if (keyValues.length %2 != 0) {
-	// return false;
-	// }
-	// for (int i = 0; i < keyValues.length; i = i+2) {
-	// if (! (keyValues[i] instanceof String)) {
-	// return false;
-	// }
-	// }
-	// return true;
-	// }
-
 }

src/main/java/com/arangodb/entity/BaseMapEntity.java

@@ -30,54 +30,66 @@
  */
 public abstract class BaseMapEntity<K, V> extends BaseEntity implements Map<K, V> {
 
-  private final TreeMap<K, V> innerMap = new TreeMap<K, V>();
+	private final TreeMap<K, V> innerMap = new TreeMap<K, V>();
+
+	@Override
+	public int size() {
+		return innerMap.size();
+	}
+
+	@Override
+	public boolean isEmpty() {
+		return innerMap.isEmpty();
+	}
+
+	@Override
+	public boolean containsKey(Object key) {
+		return innerMap.containsKey(key);
+	}
+
+	@Override
+	public boolean containsValue(Object value) {
+		return innerMap.containsValue(value);
+	}
+
+	@Override
+	public V get(Object key) {
+		return innerMap.get(key);
+	}
+
+	@Override
+	public V put(K key, V value) {
+		return innerMap.put(key, value);
+	}
+
+	@Override
+	public V remove(Object key) {
+		return innerMap.remove(key);
+	}
+
+	@Override
+	public void putAll(Map<? extends K, ? extends V> t) {
+		innerMap.putAll(t);
+	}
+
+	@Override
+	public void clear() {
+		innerMap.clear();
+	}
+
+	@Override
+	public Set<K> keySet() {
+		return innerMap.keySet();
+	}
+
+	@Override
+	public Collection<V> values() {
+		return innerMap.values();
+	}
+
+	@Override
+	public Set<java.util.Map.Entry<K, V>> entrySet() {
+		return innerMap.entrySet();
+	}
 
-  public int size() {
-    return innerMap.size();
-  }
-
-  public boolean isEmpty() {
-    return innerMap.isEmpty();
-  }
-
-  public boolean containsKey(Object key) {
-    return innerMap.containsKey(key);
-  }
-
-  public boolean containsValue(Object value) {
-    return innerMap.containsValue(value);
-  }
-
-  public V get(Object key) {
-    return innerMap.get(key);
-  }
-
-  public V put(K key, V value) {
-    return innerMap.put(key, value);
-  }
-
-  public V remove(Object key) {
-    return innerMap.remove(key);
-  }
-
-  public void putAll(Map<? extends K, ? extends V> t) {
-    innerMap.putAll(t);
-  }
-
-  public void clear() {
-    innerMap.clear();
-  }
-
-  public Set<K> keySet() {
-    return innerMap.keySet();
-  }
-
-  public Collection<V> values() {
-    return innerMap.values();
-  }
-
-  public Set<java.util.Map.Entry<K, V>> entrySet() {
-    return innerMap.entrySet();
-  }
-  
 }

src/main/java/com/arangodb/entity/DocumentEntity.java

@@ -45,12 +45,15 @@ public class DocumentEntity<T> extends BaseEntity implements DocumentHolder {
 	T entity;
 
 	public DocumentEntity() {
+		// do nothing here
 	}
 
+	@Override
 	public long getDocumentRevision() {
 		return documentRevision;
 	}
 
+	@Override
 	public String getDocumentHandle() {
 		return documentHandle;
 	}
@@ -59,10 +62,12 @@ public T getEntity() {
 		return entity;
 	}
 
+	@Override
 	public void setDocumentRevision(long documentRevision) {
 		this.documentRevision = documentRevision;
 	}
 
+	@Override
 	public void setDocumentHandle(String documentHandle) {
 		this.documentHandle = documentHandle;
 	}
@@ -71,10 +76,12 @@ public void setEntity(T entity) {
 		this.entity = entity;
 	}
 
+	@Override
 	public String getDocumentKey() {
 		return documentKey;
 	}
 
+	@Override
 	public void setDocumentKey(String documentKey) {
 		this.documentKey = documentKey;
 	}

src/main/java/com/arangodb/entity/DocumentsEntity.java

@@ -34,6 +34,7 @@ public class DocumentsEntity extends BaseEntity implements Iterable<String> {
 	 */
 	List<String> documents;
 
+	@Override
 	public Iterator<String> iterator() {
 		return CollectionUtils.safetyIterator(documents);
 	}

src/main/java/com/arangodb/entity/StreamEntity.java

@@ -25,25 +25,25 @@
  */
 public class StreamEntity extends BaseEntity {
 
-  public StreamEntity() {
-
-  }
-
-  public StreamEntity(InputStream stream) {
-    this.stream = stream;
-  }
-
-  /**
-   * Input stream.
-   */
-  InputStream stream;
-
-  public InputStream getStream() {
-    return stream;
-  }
-
-  public void setStream(InputStream stream) {
-    this.stream = stream;
-  }
+	/**
+	 * Input stream.
+	 */
+	InputStream stream;
+
+	public StreamEntity() {
+		// do nothing here
+	}
+
+	public StreamEntity(InputStream stream) {
+		this.stream = stream;
+	}
+
+	public InputStream getStream() {
+		return stream;
+	}
+
+	public void setStream(InputStream stream) {
+		this.stream = stream;
+	}
 
 }