Merge pull request #793 from biocryst/smallfixes

josemduarte · web-flow · commit c7e12e1963ae · 2018-08-24T11:15:09.000-07:00
fix for cloning with null seqres groups
diff --git a/biojava-structure/src/main/java/org/biojava/nbio/structure/ChainImpl.java b/biojava-structure/src/main/java/org/biojava/nbio/structure/ChainImpl.java
@@ -637,7 +637,9 @@ public List<Group> getSeqResGroups() {
 	@Override
 	public void setSeqResGroups(List<Group> groups){
 		for (Group g: groups){
-			g.setChain(this);
+			if (g != null) {
+				g.setChain(this);
+			}
 		}
 		this.seqResGroups = groups;
 	}

Original file line number	Diff line number	Diff line change
`@@ -637,7 +637,9 @@ public List<Group> getSeqResGroups() {`
`637`	`637`	`@Override`
`638`	`638`	`public void setSeqResGroups(List<Group> groups){`
`639`	`639`	`for (Group g: groups){`
`640`		`- g.setChain(this);`
	`640`	`+ if (g != null) {`
	`641`	`+ g.setChain(this);`
	`642`	`+ }`
`641`	`643`	`}`
`642`	`644`	`this.seqResGroups = groups;`
`643`	`645`	`}`