Skip to main content
summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authordonald.g.dunne2013-12-13 21:33:12 +0000
committerRoberto E. Escobar2014-01-08 18:31:14 +0000
commit3ea8e3d9d3a8ad0c92d17a51fe900c059f51f236 (patch)
tree11c17df52cbe61ca794c36dc366cd7c3156b775b /plugins
parent0d2bed8fa12797be66054d1b5989677460e48caf (diff)
downloadorg.eclipse.osee-3ea8e3d9d3a8ad0c92d17a51fe900c059f51f236.tar.gz
org.eclipse.osee-3ea8e3d9d3a8ad0c92d17a51fe900c059f51f236.tar.xz
org.eclipse.osee-3ea8e3d9d3a8ad0c92d17a51fe900c059f51f236.zip
bug[ats_ATS4869]: Alt-left click change of assignee doesn't save
Diffstat (limited to 'plugins')
-rw-r--r--plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/column/AssigneeColumnUI.java21
1 files changed, 12 insertions, 9 deletions
diff --git a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/column/AssigneeColumnUI.java b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/column/AssigneeColumnUI.java
index 5d8d61d023..8268df47b3 100644
--- a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/column/AssigneeColumnUI.java
+++ b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/column/AssigneeColumnUI.java
@@ -23,6 +23,7 @@ import org.eclipse.osee.ats.api.data.AtsArtifactTypes;
import org.eclipse.osee.ats.api.user.IAtsUser;
import org.eclipse.osee.ats.core.client.action.ActionManager;
import org.eclipse.osee.ats.core.client.team.TeamWorkFlowArtifact;
+import org.eclipse.osee.ats.core.client.util.AtsChangeSet;
import org.eclipse.osee.ats.core.client.workflow.AbstractWorkflowArtifact;
import org.eclipse.osee.ats.core.column.AssigneeColumn;
import org.eclipse.osee.ats.internal.Activator;
@@ -35,7 +36,6 @@ import org.eclipse.osee.framework.logging.OseeLevel;
import org.eclipse.osee.framework.logging.OseeLog;
import org.eclipse.osee.framework.skynet.core.UserManager;
import org.eclipse.osee.framework.skynet.core.artifact.Artifact;
-import org.eclipse.osee.framework.skynet.core.utility.Artifacts;
import org.eclipse.osee.framework.ui.plugin.util.AWorkbench;
import org.eclipse.osee.framework.ui.skynet.FrameworkArtifactImageProvider;
import org.eclipse.osee.framework.ui.skynet.util.LogUtil;
@@ -87,14 +87,14 @@ public class AssigneeColumnUI extends XViewerAtsColumn implements IXViewerValueC
if (!(useArt instanceof AbstractWorkflowArtifact)) {
return false;
}
- boolean modified =
- promptChangeAssignees(Arrays.asList((AbstractWorkflowArtifact) useArt), isPersistViewer());
+ AbstractWorkflowArtifact awa = (AbstractWorkflowArtifact) useArt;
+ boolean modified = promptChangeAssignees(Arrays.asList(awa), isPersistViewer());
XViewer xViewer = ((XViewerColumn) treeColumn.getData()).getTreeViewer();
if (modified && isPersistViewer(xViewer)) {
- useArt.persist("persist assignees via alt-left-click");
+ AtsChangeSet.execute("persist assignees via alt-left-click", awa);
}
if (modified) {
- xViewer.update(useArt, null);
+ xViewer.update(awa, null);
return true;
}
}
@@ -124,11 +124,13 @@ public class AssigneeColumnUI extends XViewerAtsColumn implements IXViewerValueC
UserCheckTreeDialog uld = new UserCheckTreeDialog();
uld.setMessage("Select to assign.\nDeSelect to un-assign.");
if (awas.iterator().next().getParentTeamWorkflow() != null) {
- uld.setTeamMembers(AtsClientService.get().getUserAdmin().getOseeUsers(awas.iterator().next().getParentTeamWorkflow().getTeamDefinition().getMembersAndLeads()));
+ uld.setTeamMembers(AtsClientService.get().getUserAdmin().getOseeUsers(
+ awas.iterator().next().getParentTeamWorkflow().getTeamDefinition().getMembersAndLeads()));
}
if (awas.size() == 1) {
- uld.setInitialSelections(AtsClientService.get().getUserAdmin().getOseeUsers(awas.iterator().next().getStateMgr().getAssignees()));
+ uld.setInitialSelections(AtsClientService.get().getUserAdmin().getOseeUsers(
+ awas.iterator().next().getStateMgr().getAssignees()));
}
if (uld.open() != 0) {
return false;
@@ -146,7 +148,7 @@ public class AssigneeColumnUI extends XViewerAtsColumn implements IXViewerValueC
awa.getStateMgr().setAssignees(users);
}
if (persist) {
- Artifacts.persistInTransaction("Assignee - Prompt Change", awas);
+ AtsChangeSet.execute("Assignee - Prompt Change", awas);
}
return true;
}
@@ -203,7 +205,8 @@ public class AssigneeColumnUI extends XViewerAtsColumn implements IXViewerValueC
return null;
}
if (artifact instanceof AbstractWorkflowArtifact) {
- return FrameworkArtifactImageProvider.getUserImage(AtsClientService.get().getUserAdmin().getOseeUsers(((AbstractWorkflowArtifact) artifact).getStateMgr().getAssignees()));
+ return FrameworkArtifactImageProvider.getUserImage(AtsClientService.get().getUserAdmin().getOseeUsers(
+ ((AbstractWorkflowArtifact) artifact).getStateMgr().getAssignees()));
}
if (artifact.isOfType(AtsArtifactTypes.Action)) {
for (TeamWorkFlowArtifact team : ActionManager.getTeams(artifact)) {

Back to the top