Fixing #184

eppic-model/src/main/java/eppic/model/dto/PdbInfo.java

@@ -197,6 +197,23 @@ public List<Assembly> getAssemblies() {
 		return assemblies;
 	}
 
+	/**
+	 * Get the list of valid assemblies, i.e. those with id larger than 0.
+	 * Assemblies with id=0 represent the full unit cell, whilst assemblies with
+	 * id <0 represent topologically invalid assemblies (but annotated by PDB).
+	 * @return a list of topologically valid assemblies
+	 * @since 3.1.0
+	 */
+	public List<Assembly> getValidAssemblies() {
+		List<Assembly> validAssemblies = new ArrayList<Assembly>();
+		for (Assembly assembly : assemblies) {
+			if (assembly.getId()>0) {
+				validAssemblies.add(assembly);
+			}
+		}
+		return validAssemblies;
+	}
+
 	public void setAssemblies(List<Assembly> assemblies) {
 		this.assemblies = assemblies;
 	}

eppic-wui/src/main/java/ch/systemsx/sybit/crkwebui/client/results/gui/panels/AssemblyInfoPanel.java

@@ -38,7 +38,7 @@ public AssemblyInfoPanel(PdbInfo pdbInfo){
 		for(InterfaceCluster ic : clusters){
 			num_interfaces += ic.getInterfaces().size();
 		}
-    	assembly_info = new HTML("Number of assemblies: " + ApplicationContext.getPdbInfo().getAssemblies().size() + "<br>Number of interfaces: " + num_interfaces + "<br>Number of interface clusters: " + ApplicationContext.getPdbInfo().getInterfaceClusters().size());
+    	assembly_info = new HTML("Number of assemblies: " + ApplicationContext.getPdbInfo().getValidAssemblies().size() + "<br>Number of interfaces: " + num_interfaces + "<br>Number of interface clusters: " + ApplicationContext.getPdbInfo().getInterfaceClusters().size());
 
     	mainContainer.add(assembly_info);