bug[ats_9RZ0S]: Update AtsWorkDefinitionCache to cache by guids
Change-Id: I259d0a42d92c88eadd2712a6a91d413b2d54807a
diff --git a/plugins/org.eclipse.osee.ats.core.client.test/src/org/eclipse/osee/ats/core/client/internal/workdef/AtsWorkDefinitionAdminImplTest.java b/plugins/org.eclipse.osee.ats.core.client.test/src/org/eclipse/osee/ats/core/client/internal/workdef/AtsWorkDefinitionAdminImplTest.java
index d0b25ca..5d4f457 100644
--- a/plugins/org.eclipse.osee.ats.core.client.test/src/org/eclipse/osee/ats/core/client/internal/workdef/AtsWorkDefinitionAdminImplTest.java
+++ b/plugins/org.eclipse.osee.ats.core.client.test/src/org/eclipse/osee/ats/core/client/internal/workdef/AtsWorkDefinitionAdminImplTest.java
@@ -88,7 +88,7 @@
AtsWorkDefinitionCache cache = new AtsWorkDefinitionCache();
when(cacheProvider.get()).thenReturn(cache);
- when(peerReview.getHumanReadableId()).thenReturn(MyPeerToPeerWorkDefId);
+ when(peerReview.getGuid()).thenReturn(MyPeerToPeerWorkDefId);
workDefAmin =
new AtsWorkDefinitionAdminImpl(cacheProvider, teamDefService, workItemService, workDefinitionService,
diff --git a/plugins/org.eclipse.osee.ats.core.client/src/org/eclipse/osee/ats/core/client/internal/workdef/AtsWorkDefinitionCache.java b/plugins/org.eclipse.osee.ats.core.client/src/org/eclipse/osee/ats/core/client/internal/workdef/AtsWorkDefinitionCache.java
index 5714914..f576fd6 100644
--- a/plugins/org.eclipse.osee.ats.core.client/src/org/eclipse/osee/ats/core/client/internal/workdef/AtsWorkDefinitionCache.java
+++ b/plugins/org.eclipse.osee.ats.core.client/src/org/eclipse/osee/ats/core/client/internal/workdef/AtsWorkDefinitionCache.java
@@ -22,7 +22,7 @@
public class AtsWorkDefinitionCache {
// Cache the WorkDefinition used for each AbstractWorkflowId so don't have to recompute each time
- private final Map<String, WorkDefinitionMatch> awaArtIdToWorkDefinition =
+ private final Map<String, WorkDefinitionMatch> awaGuidToWorkDefinition =
new ConcurrentHashMap<String, WorkDefinitionMatch>();
// Cache the WorkDefinition object for each WorkDefinition id so don't have to reload
// This grows as WorkDefinitions are requested/loaded
@@ -38,11 +38,11 @@
}
public void cache(IAtsWorkItem workItem, WorkDefinitionMatch match) {
- awaArtIdToWorkDefinition.put(workItem.getHumanReadableId(), match);
+ awaGuidToWorkDefinition.put(workItem.getGuid(), match);
}
public WorkDefinitionMatch getWorkDefinition(IAtsWorkItem workItem) {
- return awaArtIdToWorkDefinition.get(workItem.getHumanReadableId());
+ return awaGuidToWorkDefinition.get(workItem.getGuid());
}
//IAtsWorkDefinition
@@ -59,7 +59,7 @@
}
public void invalidateAll() {
- awaArtIdToWorkDefinition.clear();
+ awaGuidToWorkDefinition.clear();
workDefIdToWorkDefintion.clear();
}