From fee6365f74c0b4dfb3fd6d3702ffd3dc3427bed1 Mon Sep 17 00:00:00 2001 From: tjc <tjc@ee4ac58c-ac51-4696-9907-e4b3aa274f04> Date: Thu, 18 Sep 2008 08:17:51 +0000 Subject: [PATCH] test for null value when retieving featurepubs git-svn-id: svn+ssh://svn.internal.sanger.ac.uk/repos/svn/pathsoft/artemis/trunk@8858 ee4ac58c-ac51-4696-9907-e4b3aa274f04 --- .../components/genebuilder/GeneUtils.java | 26 ++++++++++++------- 1 file changed, 16 insertions(+), 10 deletions(-) diff --git a/uk/ac/sanger/artemis/components/genebuilder/GeneUtils.java b/uk/ac/sanger/artemis/components/genebuilder/GeneUtils.java index e7fc60390..b5a79a2c6 100644 --- a/uk/ac/sanger/artemis/components/genebuilder/GeneUtils.java +++ b/uk/ac/sanger/artemis/components/genebuilder/GeneUtils.java @@ -185,17 +185,23 @@ public class GeneUtils // feature pubs - literature final Collection featurePubs = feature.getChadoLazyFeature().getFeaturePubs(); - final Iterator it4 = featurePubs.iterator(); - while(it4.hasNext()) + + if(featurePubs != null) { - FeaturePub featurePub = (FeaturePub)it4.next(); - - Qualifier qualifier = feature.getQualifiers().getQualifierByName("literature"); - if(qualifier == null) - qualifier = new Qualifier("literature", featurePub.getPub().getUniqueName()); - else - qualifier.addValue(featurePub.getPub().getUniqueName()); - feature.getQualifiers().setQualifier(qualifier); + final Iterator it4 = featurePubs.iterator(); + while(it4.hasNext()) + { + FeaturePub featurePub = (FeaturePub) it4.next(); + + Qualifier qualifier = feature.getQualifiers().getQualifierByName( + "literature"); + if(qualifier == null) + qualifier = new Qualifier("literature", featurePub.getPub() + .getUniqueName()); + else + qualifier.addValue(featurePub.getPub().getUniqueName()); + feature.getQualifiers().setQualifier(qualifier); + } } feature.setLazyLoaded(true); -- GitLab