diff options
author | Roberto E. Escobar | 2012-03-15 19:37:33 +0000 |
---|---|---|
committer | Ryan D. Brooks | 2012-03-15 19:37:33 +0000 |
commit | 648358d809b8cc8aaf920b39764c938f71fb8dd4 (patch) | |
tree | 730e5681c9c554d41be2357f8afd676c9550c42d | |
parent | 939814215ccc5edec6dcbb477b44f61c1d7d0a26 (diff) | |
download | org.eclipse.osee-648358d809b8cc8aaf920b39764c938f71fb8dd4.tar.gz org.eclipse.osee-648358d809b8cc8aaf920b39764c938f71fb8dd4.tar.xz org.eclipse.osee-648358d809b8cc8aaf920b39764c938f71fb8dd4.zip |
refactor: Move useExistingBackingData into ModificationType class
2 files changed, 18 insertions, 9 deletions
diff --git a/plugins/org.eclipse.osee.framework.core/src/org/eclipse/osee/framework/core/enums/ModificationType.java b/plugins/org.eclipse.osee.framework.core/src/org/eclipse/osee/framework/core/enums/ModificationType.java index 6ce98ca689c..b6a4a69cc4f 100644 --- a/plugins/org.eclipse.osee.framework.core/src/org/eclipse/osee/framework/core/enums/ModificationType.java +++ b/plugins/org.eclipse.osee.framework.core/src/org/eclipse/osee/framework/core/enums/ModificationType.java @@ -135,4 +135,20 @@ public enum ModificationType { } return ALL_STATES; } + + public boolean isExistingVersionUsed() { + boolean result = false; + switch (this) { + case ARTIFACT_DELETED: + case DELETED: + case INTRODUCED: + case REPLACED_WITH_VERSION: + result = true; + break; + default: + result = false; + break; + } + return result; + } }
\ No newline at end of file diff --git a/plugins/org.eclipse.osee.framework.skynet.core/src/org/eclipse/osee/framework/skynet/core/transaction/BaseTransactionData.java b/plugins/org.eclipse.osee.framework.skynet.core/src/org/eclipse/osee/framework/skynet/core/transaction/BaseTransactionData.java index 9d623533c9d..ff5176ba64b 100644 --- a/plugins/org.eclipse.osee.framework.skynet.core/src/org/eclipse/osee/framework/skynet/core/transaction/BaseTransactionData.java +++ b/plugins/org.eclipse.osee.framework.skynet.core/src/org/eclipse/osee/framework/skynet/core/transaction/BaseTransactionData.java @@ -45,15 +45,8 @@ public abstract class BaseTransactionData { this.itemId = itemId; } - protected boolean useExistingBackingData() { - switch (modificationType) { - case ARTIFACT_DELETED: - case DELETED: - case INTRODUCED: - case REPLACED_WITH_VERSION: - return true; - } - return false; + protected final boolean useExistingBackingData() { + return modificationType.isExistingVersionUsed(); } @Override |