diff options
| author | ddunne | 2010-06-08 16:03:48 +0000 |
|---|---|---|
| committer | ddunne | 2010-06-08 16:03:48 +0000 |
| commit | 2188835e0d4758068a28266ec663688956073ace (patch) | |
| tree | eabc0da1fddb48fa58dd14b89f021b04a4f63281 | |
| parent | 12dde10bbf67388d7cfc6f155b4d78bc6a480f2d (diff) | |
| download | org.eclipse.osee-2188835e0d4758068a28266ec663688956073ace.tar.gz org.eclipse.osee-2188835e0d4758068a28266ec663688956073ace.tar.xz org.eclipse.osee-2188835e0d4758068a28266ec663688956073ace.zip | |
added error notification for exception in attribute check
| -rw-r--r-- | plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/health/ValidateAtsDatabase.java | 88 |
1 files changed, 47 insertions, 41 deletions
diff --git a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/health/ValidateAtsDatabase.java b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/health/ValidateAtsDatabase.java index 2f9685ec561..70c9be0f83d 100644 --- a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/health/ValidateAtsDatabase.java +++ b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/health/ValidateAtsDatabase.java @@ -359,65 +359,71 @@ public class ValidateAtsDatabase extends WorldXNavigateItemAction { SkynetTransaction transaction = new SkynetTransaction(AtsUtil.getAtsBranch(), "Validate ATS Database");
for (Artifact artifact : artifacts) {
- // Test for null attribute values
- for (Attribute<?> attr : artifact.getAttributes()) {
- if (attr.getValue() == null) {
- testNameToResultsMap.put(
- "testAtsAttributeValues",
- "Error: Artifact: " + XResultData.getHyperlink(artifact) + " Types: " + artifact.getArtifactTypeName() + " - Null Attribute");
- if (fixAttributeValues) {
- attr.delete();
+ try {
+ // Test for null attribute values
+ for (Attribute<?> attr : artifact.getAttributes()) {
+ if (attr.getValue() == null) {
+ testNameToResultsMap.put(
+ "testAtsAttributeValues",
+ "Error: Artifact: " + XResultData.getHyperlink(artifact) + " Types: " + artifact.getArtifactTypeName() + " - Null Attribute");
+ if (fixAttributeValues) {
+ attr.delete();
+ }
}
}
- }
- if (artifact instanceof StateMachineArtifact) {
- checkAndResolveDuplicateAttributesForAttributeNameContains("ats", artifact, fixAttributeValues,
- testNameToResultsMap, transaction);
- }
+ if (artifact instanceof StateMachineArtifact) {
+ checkAndResolveDuplicateAttributesForAttributeNameContains("ats", artifact, fixAttributeValues,
+ testNameToResultsMap, transaction);
+ }
+
+ // Test for ats.State Completed;;;<num> or Cancelled;;;<num> and cleanup
+ if (artifact instanceof StateMachineArtifact) {
+ XStateDam stateDam = new XStateDam((StateMachineArtifact) artifact);
+ for (SMAState state : stateDam.getStates()) {
+ if (state.getName().equals(DefaultTeamState.Completed.name()) || state.getName().equals(
+ DefaultTeamState.Cancelled.name())) {
+ if (state.getHoursSpent() != 0.0 || state.getPercentComplete() != 0) {
+ testNameToResultsMap.put(
+ "testAtsAttributeValues",
+ "Error: ats.State error for SMA: " + XResultData.getHyperlink(artifact) + " State: " + state.getName() + " Hours Spent: " + state.getHoursSpentStr() + " Percent: " + state.getPercentComplete());
+ if (fixAttributeValues) {
+ state.setHoursSpent(0);
+ state.setPercentComplete(0);
+ stateDam.setState(state);
+ testNameToResultsMap.put("testAtsAttributeValues", "Fixed");
+ }
+ }
+ }
+ }
+ }
- // Test for ats.State Completed;;;<num> or Cancelled;;;<num> and cleanup
- if (artifact instanceof StateMachineArtifact) {
- XStateDam stateDam = new XStateDam((StateMachineArtifact) artifact);
- for (SMAState state : stateDam.getStates()) {
+ // Test for ats.CurrentState Completed;;;<num> or Cancelled;;;<num> and cleanup
+ if (artifact instanceof StateMachineArtifact) {
+ XCurrentStateDam currentStateDam = new XCurrentStateDam((StateMachineArtifact) artifact);
+ SMAState state = currentStateDam.getState();
if (state.getName().equals(DefaultTeamState.Completed.name()) || state.getName().equals(
DefaultTeamState.Cancelled.name())) {
if (state.getHoursSpent() != 0.0 || state.getPercentComplete() != 0) {
testNameToResultsMap.put(
"testAtsAttributeValues",
- "Error: ats.State error for SMA: " + XResultData.getHyperlink(artifact) + " State: " + state.getName() + " Hours Spent: " + state.getHoursSpentStr() + " Percent: " + state.getPercentComplete());
+ "Error: ats.CurrentState error for SMA: " + XResultData.getHyperlink(artifact) + " State: " + state.getName() + " Hours Spent: " + state.getHoursSpentStr() + " Percent: " + state.getPercentComplete());
if (fixAttributeValues) {
state.setHoursSpent(0);
state.setPercentComplete(0);
- stateDam.setState(state);
+ currentStateDam.setState(state);
testNameToResultsMap.put("testAtsAttributeValues", "Fixed");
}
}
}
}
- }
-
- // Test for ats.CurrentState Completed;;;<num> or Cancelled;;;<num> and cleanup
- if (artifact instanceof StateMachineArtifact) {
- XCurrentStateDam currentStateDam = new XCurrentStateDam((StateMachineArtifact) artifact);
- SMAState state = currentStateDam.getState();
- if (state.getName().equals(DefaultTeamState.Completed.name()) || state.getName().equals(
- DefaultTeamState.Cancelled.name())) {
- if (state.getHoursSpent() != 0.0 || state.getPercentComplete() != 0) {
- testNameToResultsMap.put(
- "testAtsAttributeValues",
- "Error: ats.CurrentState error for SMA: " + XResultData.getHyperlink(artifact) + " State: " + state.getName() + " Hours Spent: " + state.getHoursSpentStr() + " Percent: " + state.getPercentComplete());
- if (fixAttributeValues) {
- state.setHoursSpent(0);
- state.setPercentComplete(0);
- currentStateDam.setState(state);
- testNameToResultsMap.put("testAtsAttributeValues", "Fixed");
- }
- }
+ if (artifact.hasDirtyAttributes()) {
+ artifact.persist(transaction);
}
- }
- if (artifact.hasDirtyAttributes()) {
- artifact.persist(transaction);
+ } catch (OseeCoreException ex) {
+ OseeLog.log(AtsPlugin.class, Level.SEVERE, ex);
+ testNameToResultsMap.put("testAtsAttributeValues",
+ "Error: Artifact: " + XResultData.getHyperlink(artifact) + " Exception: " + ex.getLocalizedMessage());
}
}
transaction.execute();
|