Skip to main content
summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorddunne2011-02-22 14:41:03 -0500
committerRyan D. Brooks2011-02-22 14:41:03 -0500
commitafde5c22f1c89192f5a25e22ab35276d0de52d25 (patch)
treec1c8840a0d208c96ca0bfb1e6128fb8704f15681 /plugins/org.eclipse.osee.coverage
parent6087a905390eca5608bdc04940ab8da989be70ac (diff)
downloadorg.eclipse.osee-afde5c22f1c89192f5a25e22ab35276d0de52d25.tar.gz
org.eclipse.osee-afde5c22f1c89192f5a25e22ab35276d0de52d25.tar.xz
org.eclipse.osee-afde5c22f1c89192f5a25e22ab35276d0de52d25.zip
bug[ats_0ZGPF]: Modification of coverage assignee or notes to nothing does not save
Diffstat (limited to 'plugins/org.eclipse.osee.coverage')
-rw-r--r--plugins/org.eclipse.osee.coverage/src/org/eclipse/osee/coverage/store/OseeCoverageUnitStore.java19
1 files changed, 16 insertions, 3 deletions
diff --git a/plugins/org.eclipse.osee.coverage/src/org/eclipse/osee/coverage/store/OseeCoverageUnitStore.java b/plugins/org.eclipse.osee.coverage/src/org/eclipse/osee/coverage/store/OseeCoverageUnitStore.java
index 10295e3c27..1c63c95907 100644
--- a/plugins/org.eclipse.osee.coverage/src/org/eclipse/osee/coverage/store/OseeCoverageUnitStore.java
+++ b/plugins/org.eclipse.osee.coverage/src/org/eclipse/osee/coverage/store/OseeCoverageUnitStore.java
@@ -100,8 +100,8 @@ public class OseeCoverageUnitStore extends OseeCoverageStore {
coverageUnit.setNotes(artifact.getSoleAttributeValueAsString(CoverageAttributeTypes.Notes, ""));
coverageUnit.setFolder(artifact.isOfType(CoverageArtifactTypes.CoverageFolder));
coverageUnit.setAssignees(artifact.getSoleAttributeValueAsString(CoverageAttributeTypes.Assignees, ""));
- coverageUnit.setWorkProductTaskGuid(artifact.getSoleAttributeValueAsString(CoverageAttributeTypes.WorkProductTaskGuid,
- ""));
+ coverageUnit.setWorkProductTaskGuid(artifact.getSoleAttributeValueAsString(
+ CoverageAttributeTypes.WorkProductTaskGuid, ""));
coverageUnit.setNamespace(artifact.getSoleAttributeValueAsString(CoverageAttributeTypes.Namespace, ""));
coverageUnit.setOrderNumber(artifact.getSoleAttributeValueAsString(CoverageAttributeTypes.Order, ""));
coverageUnit.setLocation(artifact.getSoleAttributeValueAsString(CoverageAttributeTypes.Location, ""));
@@ -173,12 +173,19 @@ public class OseeCoverageUnitStore extends OseeCoverageStore {
}
if (Strings.isValid(coverageUnit.getNotes())) {
artifact.setSoleAttributeFromString(CoverageAttributeTypes.Notes, coverageUnit.getNotes());
+ } else {
+ artifact.deleteAttributes(CoverageAttributeTypes.Notes);
}
if (Strings.isValid(coverageUnit.getNamespace())) {
artifact.setSoleAttributeFromString(CoverageAttributeTypes.Namespace, coverageUnit.getNamespace());
+ } else {
+ artifact.deleteAttributes(CoverageAttributeTypes.Namespace);
}
if (Strings.isValid(coverageUnit.getWorkProductTaskGuid())) {
- artifact.setSoleAttributeFromString(CoverageAttributeTypes.WorkProductTaskGuid, coverageUnit.getWorkProductTaskGuid());
+ artifact.setSoleAttributeFromString(CoverageAttributeTypes.WorkProductTaskGuid,
+ coverageUnit.getWorkProductTaskGuid());
+ } else {
+ artifact.deleteAttributes(CoverageAttributeTypes.WorkProductTaskGuid);
}
if (coverageUnit.getFileContentsProvider() != null && coverageUnit.getFileContentsProvider() != OseeCoverageUnitFileContentsProvider.getInstance(branch)) {
String fileContents = coverageUnit.getFileContents();
@@ -189,12 +196,18 @@ public class OseeCoverageUnitStore extends OseeCoverageStore {
}
if (Strings.isValid(coverageUnit.getOrderNumber())) {
artifact.setSoleAttributeFromString(CoverageAttributeTypes.Order, coverageUnit.getOrderNumber());
+ } else {
+ artifact.deleteAttributes(CoverageAttributeTypes.Order);
}
if (Strings.isValid(coverageUnit.getAssignees())) {
artifact.setSoleAttributeFromString(CoverageAttributeTypes.Assignees, coverageUnit.getAssignees());
+ } else {
+ artifact.deleteAttributes(CoverageAttributeTypes.Assignees);
}
if (Strings.isValid(coverageUnit.getLocation())) {
artifact.setSoleAttributeFromString(CoverageAttributeTypes.Location, coverageUnit.getLocation());
+ } else {
+ artifact.deleteAttributes(CoverageAttributeTypes.Location);
}
if (coverageUnit.getParent() != null) {
Artifact parentArt = ArtifactQuery.getArtifactFromId(coverageUnit.getParent().getGuid(), branch);

Back to the top