Correcting reporting

aalhossary · aalhossary · commit e178cd140fa1 · 2021-11-28T04:33:19.000+08:00
Isn't it existing atom which will be ignored if future lookups, or I am
missing some point here?
diff --git a/biojava-structure/src/main/java/org/biojava/nbio/structure/HetatomImpl.java b/biojava-structure/src/main/java/org/biojava/nbio/structure/HetatomImpl.java
@@ -168,7 +168,7 @@ public void addAtom(Atom atom){
 				String altLocStr = "";
 				char altLoc = atom.getAltLoc();
 				if (altLoc != ' ') altLocStr = "(alt loc '" + altLoc + "')";
-				logger.warn("An atom with name " + atom.getName() + " " + altLocStr + " is already present in group: " + this.toString() + ". The atom with serial " + atom.getPDBserial() + " will be ignored in look-ups.");
+				logger.warn("An atom with name " + atom.getName() + " " + altLocStr + " is already present in group: " + this.toString() + ". The atom with serial " + existingAtom.getPDBserial() + " will be ignored in look-ups.");
 			}
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -168,7 +168,7 @@ public void addAtom(Atom atom){`
`168`	`168`	`String altLocStr = "";`
`169`	`169`	`char altLoc = atom.getAltLoc();`
`170`	`170`	`if (altLoc != ' ') altLocStr = "(alt loc '" + altLoc + "')";`
`171`		`- logger.warn("An atom with name " + atom.getName() + " " + altLocStr + " is already present in group: " + this.toString() + ". The atom with serial " + atom.getPDBserial() + " will be ignored in look-ups.");`
	`171`	`+ logger.warn("An atom with name " + atom.getName() + " " + altLocStr + " is already present in group: " + this.toString() + ". The atom with serial " + existingAtom.getPDBserial() + " will be ignored in look-ups.");`
`172`	`172`	`}`
`173`	`173`	`}`
`174`	`174`	`}`