diff options
author | ddunne | 2011-07-29 20:32:32 +0000 |
---|---|---|
committer | Ryan D. Brooks | 2011-07-29 20:32:32 +0000 |
commit | b3c690d4a9c7cba9df4c618179f635976a30402e (patch) | |
tree | e3d08bc77ded83dbc6b165190177d278549d859d /plugins/org.eclipse.osee.ats.core | |
parent | d4a926cd290a200a9c046bfc72382aef859f03b9 (diff) | |
download | org.eclipse.osee-b3c690d4a9c7cba9df4c618179f635976a30402e.tar.gz org.eclipse.osee-b3c690d4a9c7cba9df4c618179f635976a30402e.tar.xz org.eclipse.osee-b3c690d4a9c7cba9df4c618179f635976a30402e.zip |
bug: Fix AtsXCommitManagerValidator to check if commit is necessary
Diffstat (limited to 'plugins/org.eclipse.osee.ats.core')
-rw-r--r-- | plugins/org.eclipse.osee.ats.core/src/org/eclipse/osee/ats/core/validator/AtsXCommitManagerValidator.java | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/plugins/org.eclipse.osee.ats.core/src/org/eclipse/osee/ats/core/validator/AtsXCommitManagerValidator.java b/plugins/org.eclipse.osee.ats.core/src/org/eclipse/osee/ats/core/validator/AtsXCommitManagerValidator.java index e907cc1f0d1..987fcb82324 100644 --- a/plugins/org.eclipse.osee.ats.core/src/org/eclipse/osee/ats/core/validator/AtsXCommitManagerValidator.java +++ b/plugins/org.eclipse.osee.ats.core/src/org/eclipse/osee/ats/core/validator/AtsXCommitManagerValidator.java @@ -31,12 +31,14 @@ public class AtsXCommitManagerValidator extends AtsXWidgetValidator { try { if (provider instanceof ArtifactValueProvider && ((ArtifactValueProvider) provider).getArtifact() instanceof TeamWorkFlowArtifact) { TeamWorkFlowArtifact teamArt = (TeamWorkFlowArtifact) ((ArtifactValueProvider) provider).getArtifact(); - if (!AtsBranchManagerCore.isAllObjectsToCommitToConfigured(teamArt)) { - return new WidgetResult(WidgetStatus.Invalid_Incompleted, widgetDef, - "All branches must be configured and committed."); - } else if (!AtsBranchManagerCore.isBranchesAllCommitted(teamArt)) { - return new WidgetResult(WidgetStatus.Invalid_Incompleted, widgetDef, - "All branches must be committed."); + if (AtsBranchManagerCore.isWorkingBranchInWork(teamArt) || AtsBranchManagerCore.isCommittedBranchExists(teamArt)) { + if (!AtsBranchManagerCore.isAllObjectsToCommitToConfigured(teamArt)) { + return new WidgetResult(WidgetStatus.Invalid_Incompleted, widgetDef, + "All branches must be configured and committed."); + } else if (!AtsBranchManagerCore.isBranchesAllCommitted(teamArt)) { + return new WidgetResult(WidgetStatus.Invalid_Incompleted, widgetDef, + "All branches must be committed."); + } } } } catch (OseeCoreException ex) { |