diff options
Diffstat (limited to 'plugins/org.eclipse.osee.ats/src/org/eclipse/osee')
25 files changed, 49 insertions, 47 deletions
diff --git a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/actions/ExportChangeReportsAction.java b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/actions/ExportChangeReportsAction.java index c35ae4eb0fe..62522a67197 100644 --- a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/actions/ExportChangeReportsAction.java +++ b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/actions/ExportChangeReportsAction.java @@ -131,7 +131,7 @@ public class ExportChangeReportsAction extends Action { } } if (minTransactionId == -1) { - throw new OseeStateException("no transaction records found for " + workflow); + throw new OseeStateException("no transaction records found for [%s]", workflow); } return TransactionManager.getTransactionId(minTransactionId); } diff --git a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/artifact/ATSLog.java b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/artifact/ATSLog.java index 22b18c07305..f7c62d2f6a7 100644 --- a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/artifact/ATSLog.java +++ b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/artifact/ATSLog.java @@ -66,7 +66,7 @@ public class ATSLog { return (LogType) e; } } - throw new OseeArgumentException("Unhandled LogType: \"" + type + "\""); + throw new OseeArgumentException("Unhandled LogType: [%s]", type); } }; diff --git a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/artifact/ActionArtifact.java b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/artifact/ActionArtifact.java index 0495262f328..91bf4460df0 100644 --- a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/artifact/ActionArtifact.java +++ b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/artifact/ActionArtifact.java @@ -753,8 +753,8 @@ public class ActionArtifact extends ATSArtifact implements IWorldViewArtifact { for (TeamWorkFlowArtifact teamArt : getTeamWorkFlowArtifacts()) { if (teamArt.getTeamDefinition().equals(teamDef)) { AWorkbench.popup("ERROR", "Team already exist"); - throw new OseeArgumentException( - "Team \"" + teamDef + "\" already exists for Action " + getHumanReadableId()); + throw new OseeArgumentException("Team [%s] already exists for Action [%s]", teamDef, + getHumanReadableId()); } } } @@ -803,6 +803,8 @@ public class ActionArtifact extends ATSArtifact implements IWorldViewArtifact { PriorityType priType = null; if (Strings.isValid(priorityStr)) { priType = PriorityType.getPriority(priorityStr); + } else { + throw new OseeArgumentException("Invalid priority [%s]", priorityStr); } setArtifactIdentifyData(toTeam, fromAction.getName(), fromAction.getSoleAttributeValue(AtsAttributeTypes.Description, ""), diff --git a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/artifact/AtsArtifactFactory.java b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/artifact/AtsArtifactFactory.java index c895df9098d..ed97d556aac 100644 --- a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/artifact/AtsArtifactFactory.java +++ b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/artifact/AtsArtifactFactory.java @@ -70,7 +70,7 @@ public class AtsArtifactFactory extends ArtifactFactory { if (artifactType.equals(AtsArtifactTypes.Goal)) { return new GoalArtifact(this, guid, humandReadableId, branch, artifactType); } - throw new OseeArgumentException("did not recognize the artifact type: " + artifactType.getName()); + throw new OseeArgumentException("did not recognize the artifact type [%s]", artifactType); } @Override diff --git a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/artifact/TaskArtifact.java b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/artifact/TaskArtifact.java index 9e7ba3fa5df..f64f37ce5b8 100644 --- a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/artifact/TaskArtifact.java +++ b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/artifact/TaskArtifact.java @@ -311,7 +311,7 @@ public class TaskArtifact extends StateMachineArtifact implements IATSStateMachi Collection<StateMachineArtifact> smas = getRelatedArtifacts(AtsRelationTypes.SmaToTask_Sma, StateMachineArtifact.class); if (smas.isEmpty()) { - throw new OseeStateException("Task has no parent - " + getHumanReadableId()); + throw new OseeStateException("Task has no parent [%s]", getHumanReadableId()); } parentSma = smas.iterator().next(); return parentSma; diff --git a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/artifact/TeamDefinitionArtifact.java b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/artifact/TeamDefinitionArtifact.java index 90ba9724bf4..db09a20bd0f 100644 --- a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/artifact/TeamDefinitionArtifact.java +++ b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/artifact/TeamDefinitionArtifact.java @@ -462,7 +462,7 @@ public class TeamDefinitionArtifact extends Artifact implements ICommitConfigArt if (!hasWorkRule(ruleId)) { Artifact artifact = WorkItemDefinitionFactory.getWorkItemDefinitionArtifact(ruleId); if (artifact == null) { - throw new OseeArgumentException("Rule \"" + ruleId + "\" does not exist."); + throw new OseeArgumentException("Rule [%s] does not exist.", ruleId); } else { addRelation(CoreRelationTypes.WorkItem__Child, artifact); } diff --git a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/artifact/TeamWorkFlowArtifact.java b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/artifact/TeamWorkFlowArtifact.java index bd72fe6a9f9..ae11da2ee72 100644 --- a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/artifact/TeamWorkFlowArtifact.java +++ b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/artifact/TeamWorkFlowArtifact.java @@ -214,8 +214,7 @@ public class TeamWorkFlowArtifact extends TaskableStateMachineArtifact implement public TeamDefinitionArtifact getTeamDefinition() throws OseeCoreException, OseeCoreException { String guid = this.getSoleAttributeValue(AtsAttributeTypes.TeamDefinition, ""); if (!Strings.isValid(guid)) { - throw new OseeArgumentException( - "TeamWorkflow [" + getHumanReadableId() + "] has no TeamDefinition associated."); + throw new OseeArgumentException("TeamWorkflow [%s] has no TeamDefinition associated.", getHumanReadableId()); } return AtsCacheManager.getTeamDefinitionArtifact(guid); } @@ -286,9 +285,9 @@ public class TeamWorkFlowArtifact extends TaskableStateMachineArtifact implement Collection<ActionArtifact> arts = getRelatedArtifacts(AtsRelationTypes.ActionToWorkflow_Action, ActionArtifact.class); if (arts.isEmpty()) { - throw new OseeStateException("Team " + getHumanReadableId() + " has no parent Action"); + throw new OseeStateException("Team [%s] has no parent Action", getGuid()); } else if (arts.size() > 1) { - throw new OseeStateException("Team " + getHumanReadableId() + " has multiple parent Actions"); + throw new OseeStateException("Team [%s] has multiple parent Actions", getGuid()); } parentAction = arts.iterator().next(); return parentAction; diff --git a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/editor/SMAWorkFlowSection.java b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/editor/SMAWorkFlowSection.java index 6e1bd709536..604510a35e8 100644 --- a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/editor/SMAWorkFlowSection.java +++ b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/editor/SMAWorkFlowSection.java @@ -366,7 +366,7 @@ public class SMAWorkFlowSection extends SectionPart { if (sma.isCancelled()) { LogItem item = sma.getLog().getStateEvent(LogType.StateCancelled); if (item == null) { - throw new OseeStateException("ats.Log: Cancelled state has no logItem for " + sma.getHumanReadableId()); + throw new OseeStateException("ats.Log: Cancelled state has no logItem for [%s]", sma.getGuid()); } if (item.getState().equals(atsWorkPage.getName())) { sb.append(" - Cancelled"); diff --git a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/health/ValidateChangeReports.java b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/health/ValidateChangeReports.java index aceabaa6d84..a637ffc2f04 100644 --- a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/health/ValidateChangeReports.java +++ b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/health/ValidateChangeReports.java @@ -221,7 +221,7 @@ public class ValidateChangeReports extends XNavigateItemAction { String storedChangeReport = null; Artifact artifactForStore = null; if (arts.size() > 1) { - throw new OseeStateException("Multiple artifacts found of name \"" + name + "\""); + throw new OseeStateException("Multiple artifacts found named [%s]", name); } else if (arts.size() == 1) { artifactForStore = arts.iterator().next(); storedChangeReport = artifactForStore.getSoleAttributeValue(CoreAttributeTypes.GeneralStringData, null); diff --git a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/navigate/UserSearchWorkflowSearchItem.java b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/navigate/UserSearchWorkflowSearchItem.java index a352c27cc3d..0faf4d32090 100644 --- a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/navigate/UserSearchWorkflowSearchItem.java +++ b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/navigate/UserSearchWorkflowSearchItem.java @@ -24,6 +24,7 @@ import org.eclipse.osee.ats.world.WorldEditorParameterSearchItem; import org.eclipse.osee.ats.world.search.UserWorldSearchItem; import org.eclipse.osee.ats.world.search.UserWorldSearchItem.UserSearchOption; import org.eclipse.osee.framework.core.exception.OseeCoreException; +import org.eclipse.osee.framework.core.exception.OseeStateException; import org.eclipse.osee.framework.jdk.core.util.Strings; import org.eclipse.osee.framework.logging.OseeLevel; import org.eclipse.osee.framework.logging.OseeLog; @@ -257,7 +258,7 @@ public class UserSearchWorkflowSearchItem extends WorldEditorParameterSearchItem } } - public void setSelected(UserSearchOption userSearchOption, boolean set) { + public void setSelected(UserSearchOption userSearchOption, boolean set) throws OseeStateException { if (userSearchOption == UserSearchOption.Assignee) { assigneeCheckbox.set(set); } else if (userSearchOption == UserSearchOption.Favorites) { @@ -277,7 +278,7 @@ public class UserSearchWorkflowSearchItem extends WorldEditorParameterSearchItem } else if (userSearchOption == UserSearchOption.IncludeTasks) { tasksCheckbox.set(set); } else { - throw new IllegalStateException(String.format("Unhandled checkbox [%s]", userSearchOption)); + throw new OseeStateException("Unhandled checkbox [%s]", userSearchOption); } } diff --git a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/AtsBranchManager.java b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/AtsBranchManager.java index 2a12cc0b01e..1103092f022 100644 --- a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/AtsBranchManager.java +++ b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/AtsBranchManager.java @@ -499,7 +499,8 @@ public class AtsBranchManager { return null; } else if (branches.size() > 1) { throw new MultipleBranchesExist( - "Unexpected multiple associated un-deleted working branches found for workflow " + teamArt.getHumanReadableId()); + "Unexpected multiple associated un-deleted working branches found for workflow [%s]", + teamArt.getHumanReadableId()); } else { return branches.get(0); } @@ -661,7 +662,7 @@ public class AtsBranchManager { public static void createNecessaryBranchEventReviews(StateEventType stateEventType, TeamWorkFlowArtifact teamArt, SkynetTransaction transaction) throws OseeCoreException { if (stateEventType != StateEventType.CommitBranch && stateEventType != StateEventType.CreateBranch) { - throw new OseeStateException("Invalid stateEventType = " + stateEventType); + throw new OseeStateException("Invalid stateEventType [%s]", stateEventType); } // Create any decision and peerToPeer reviews for createBranch and commitBranch for (String ruleId : Arrays.asList(AtsAddDecisionReviewRule.ID, AtsAddPeerToPeerReviewRule.ID)) { @@ -766,7 +767,7 @@ public class AtsBranchManager { */ public ChangeData getChangeData(ICommitConfigArtifact commitConfigArt) throws OseeCoreException { if (commitConfigArt != null && commitConfigArt.getParentBranch() == null) { - throw new OseeArgumentException("Parent Branch not configured for " + commitConfigArt); + throw new OseeArgumentException("Parent Branch not configured for [%s]", commitConfigArt); } Collection<Change> changes = new ArrayList<Change>(); diff --git a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/AtsFolderUtil.java b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/AtsFolderUtil.java index 0bdfd741def..31d4dd329a9 100644 --- a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/AtsFolderUtil.java +++ b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/AtsFolderUtil.java @@ -68,7 +68,7 @@ public class AtsFolderUtil { StaticIdManager.getSingletonArtifact(atsFolder.artifactType, atsFolder.staticId, AtsUtil.getAtsBranch(), true); if (artifact == null) { - throw new OseeStateException(String.format("Can't retrieve Ats folder [%s]", atsFolder.displayName)); + throw new OseeStateException("Can't retrieve Ats folder [%s]", atsFolder.displayName); } folderMap.put(atsFolder, artifact); } diff --git a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/AtsPriority.java b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/AtsPriority.java index d172dcf9854..8f503b5d93b 100644 --- a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/AtsPriority.java +++ b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/AtsPriority.java @@ -74,7 +74,7 @@ public class AtsPriority { return pri; } } - throw new OseeStateException("Invalid priority => " + type); + throw new OseeStateException("Invalid priority [%s]", type); } }; diff --git a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/Import/ExcelAtsTaskArtifactExtractor.java b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/Import/ExcelAtsTaskArtifactExtractor.java index 26357f43bba..1a9c91149b5 100644 --- a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/Import/ExcelAtsTaskArtifactExtractor.java +++ b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/Import/ExcelAtsTaskArtifactExtractor.java @@ -234,8 +234,8 @@ public class ExcelAtsTaskArtifactExtractor { percent = percent * 100; } } catch (Exception ex) { - throw new OseeArgumentException(String.format("Invalid Percent Complete \"%s\" for row %d", - str, rowNum)); + throw new OseeArgumentException("Invalid Percent Complete \"%s\" for row %d", + str, rowNum); } int percentInt = percent.intValue(); sma.getStateMgr().updateMetrics(0, percentInt, true); @@ -247,8 +247,8 @@ public class ExcelAtsTaskArtifactExtractor { try { hours = new Double(str); } catch (Exception ex) { - throw new OseeArgumentException(String.format("Invalid Hours Spent \"%s\" for row %d", str, - rowNum)); + throw new OseeArgumentException("Invalid Hours Spent \"%s\" for row %d", str, + rowNum); } sma.getStateMgr().updateMetrics(hours, sma.getStateMgr().getPercentComplete(), true); } @@ -259,8 +259,8 @@ public class ExcelAtsTaskArtifactExtractor { try { hours = new Double(str); } catch (Exception ex) { - throw new OseeArgumentException(String.format("Invalid Estimated Hours \"%s\" for row %d", str, - rowNum)); + throw new OseeArgumentException("Invalid Estimated Hours \"%s\" for row %d", str, + rowNum); } taskArt.setSoleAttributeValue(AtsAttributeTypes.EstimatedHours, hours); } diff --git a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/StateManager.java b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/StateManager.java index 8887323e668..dae9ec2f783 100644 --- a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/StateManager.java +++ b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/StateManager.java @@ -183,7 +183,7 @@ public class StateManager { */ public void setAssignee(String stateName, User assignee) throws OseeCoreException { if (!isStateVisited(stateName)) { - throw new OseeArgumentException("State " + stateName + " does not exist."); + throw new OseeArgumentException("State [%s] does not exist.", stateName); } SMAState state = getSMAState(stateName, false); state.setAssignee(assignee); diff --git a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/widgets/SMAState.java b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/widgets/SMAState.java index 8dd8875ac3e..48db5005ac9 100644 --- a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/widgets/SMAState.java +++ b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/widgets/SMAState.java @@ -189,7 +189,7 @@ public class SMAState { } assignees = UsersByIds.getUsers(m.group(2)); } else { - throw new OseeArgumentException("Can't unpack state data => " + xml); + throw new OseeArgumentException("Can't unpack state data [%s]", xml); } } diff --git a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/widgets/XCurrentStateDam.java b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/widgets/XCurrentStateDam.java index 358d020a024..7c8cf4329d9 100644 --- a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/widgets/XCurrentStateDam.java +++ b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/widgets/XCurrentStateDam.java @@ -37,8 +37,8 @@ public class XCurrentStateDam extends XStateAssigneesDam { public SMAState getState() throws OseeCoreException { Set<SMAState> states = getStates(); if (states.size() != 1) { - throw new OseeArgumentException( - "Must be one current state. Found " + states.size() + " for " + getArtifact().getGuid()); + throw new OseeArgumentException("Must be one current state. Found %d for %s", states.size(), + getArtifact().getGuid()); } return states.iterator().next(); } diff --git a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/widgets/commit/XCommitManager.java b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/widgets/commit/XCommitManager.java index 7d2baf41616..e5768c78a1b 100644 --- a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/widgets/commit/XCommitManager.java +++ b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/widgets/commit/XCommitManager.java @@ -355,7 +355,7 @@ public class XCommitManager extends XWidget implements IArtifactWidget, IBranchE @Override public void setArtifact(Artifact artifact) throws OseeCoreException { if (!(artifact instanceof TeamWorkFlowArtifact)) { - throw new OseeStateException("Must be TeamWorkflowArtifact, set was a " + artifact.getArtifactTypeName()); + throw new OseeStateException("Must be TeamWorkflowArtifact, set was a [%s]", artifact.getArtifactTypeName()); } this.teamArt = (TeamWorkFlowArtifact) artifact; loadTable(); diff --git a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/widgets/dialog/TaskResOptionDefinition.java b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/widgets/dialog/TaskResOptionDefinition.java index 4447ed2a309..cd821e3d6c3 100644 --- a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/widgets/dialog/TaskResOptionDefinition.java +++ b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/util/widgets/dialog/TaskResOptionDefinition.java @@ -63,7 +63,7 @@ public class TaskResOptionDefinition { } else if (nodeName.equals(Field.percent.name())) { percent = node.getNodeValue(); } else { - throw new OseeArgumentException("Unknow Task Resolution Option Attribute \"" + nodeName + "\""); + throw new OseeArgumentException("Unknow Task Resolution Option Attribute [%s]", nodeName); } } } diff --git a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/workflow/editor/AtsWorkflowConfigEditor.java b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/workflow/editor/AtsWorkflowConfigEditor.java index 5da7626ac1a..0afef5de624 100644 --- a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/workflow/editor/AtsWorkflowConfigEditor.java +++ b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/workflow/editor/AtsWorkflowConfigEditor.java @@ -249,7 +249,7 @@ public class AtsWorkflowConfigEditor extends GraphicalEditorWithFlyoutPalette im int yLoc = 0; WorkPageDefinition startPage = workflowDef.getStartPage(); if (startPage == null || startPage.getName().equals("")) { - throw new OseeArgumentException("StartPage null for workflow " + workflowDef); + throw new OseeArgumentException("StartPage null for workflow [%s]", workflowDef); } // Create states List<WorkPageDefinition> pages = workflowDef.getPagesOrdered(); diff --git a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/workflow/vue/Diagram.java b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/workflow/vue/Diagram.java index e097a9c3e5a..c7dafc0db63 100644 --- a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/workflow/vue/Diagram.java +++ b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/workflow/vue/Diagram.java @@ -49,8 +49,7 @@ public class Diagram { public DiagramNode getPage(String pageName) throws OseeCoreException { List<DiagramNode> pages = getPages(pageName); if (pages.size() > 1) { - throw new OseeArgumentException( - "Multiple node of same name found in workflow => " + getId() + " Use getPages."); + throw new OseeArgumentException("Multiple node of same name found in workflow [%s]. Use getPages.", getId()); } return pages.iterator().next(); } @@ -63,7 +62,7 @@ public class Diagram { } } if (foundPages.isEmpty()) { - throw new OseeArgumentException("Invalid node Name \"" + pageName + "\" from workflow => " + getId()); + throw new OseeArgumentException("Invalid node Name [%s] from workflow [%s]", pageName, getId()); } return foundPages; } diff --git a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/workflow/vue/LoadAIsAndTeamsAction.java b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/workflow/vue/LoadAIsAndTeamsAction.java index 99c2c215fb9..a120b63f441 100644 --- a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/workflow/vue/LoadAIsAndTeamsAction.java +++ b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/workflow/vue/LoadAIsAndTeamsAction.java @@ -208,8 +208,8 @@ public class LoadAIsAndTeamsAction { User u = getUserByName(name, allowUserCreation, transaction); members.add(u); } else { - throw new OseeArgumentException( - "Unhandled AtsConfig Line\"" + line + "\" in diagram page \"" + page.getName() + "\""); + throw new OseeArgumentException("Unhandled AtsConfig Line [%s] in diagram page [%s]", line, + page.getName()); } } } @@ -222,8 +222,8 @@ public class LoadAIsAndTeamsAction { if (actItem != null) { actionableItems.add(actItem); } else { - throw new OseeArgumentException( - "Can't retrieve Actionable Item \"" + childPage.getName() + "\" with id " + childPage.getId()); + throw new OseeArgumentException("Can't retrieve Actionable Item [%s] with id [%s]", + childPage.getName(), childPage.getId()); } } } diff --git a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/workflow/vue/VueDiagram.java b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/workflow/vue/VueDiagram.java index 44f3d8801ed..a3ed6ee5c16 100644 --- a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/workflow/vue/VueDiagram.java +++ b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/workflow/vue/VueDiagram.java @@ -54,12 +54,12 @@ public class VueDiagram { for (VueLink link : links) { VueNode fromVuePage = getPageFromVueId(link.getFromVueId()); if (fromVuePage == null) { - throw new OseeArgumentException("Can't retrieve fromVuePage with id " + link.getFromVueId()); + throw new OseeArgumentException("Can't retrieve fromVuePage with id [%s]", link.getFromVueId()); } VueNode toVuePage = getPageFromVueId(link.getToVueId()); if (toVuePage == null) { - throw new OseeArgumentException( - "Can't retrieve toVuePage " + link.getToVueId() + " fromVuePage " + link.getFromVueId() + " named \"" + fromVuePage.getWorkPage().getName() + "\""); + throw new OseeArgumentException("Can't retrieve toVuePage [%s] fromVuePage [%s] named [%s]", + link.getToVueId(), link.getFromVueId(), fromVuePage.getWorkPage().getName()); } fromVuePage.getWorkPage().addToPage(toVuePage.getWorkPage(), link.getName().equals("return")); toVuePage.getWorkPage().addFromPage(fromVuePage.getWorkPage()); @@ -71,8 +71,8 @@ public class VueDiagram { if (fromVuePage.getWorkPage().getDefaultToPage() == null) { fromVuePage.getWorkPage().setDefaultToPage(toVuePage.getWorkPage()); } else { - throw new OseeArgumentException( - "Can't have 2 default transitions. Page " + fromVuePage.getWorkPage().getName()); + throw new OseeArgumentException("Can't have 2 default transitions. Page [%s]", + fromVuePage.getWorkPage().getName()); } } } diff --git a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/workflow/vue/VueLink.java b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/workflow/vue/VueLink.java index 76cf34dbf70..438cfd2db02 100644 --- a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/workflow/vue/VueLink.java +++ b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/workflow/vue/VueLink.java @@ -49,7 +49,7 @@ public class VueLink { toVueId = m.group(2); multiDirectional = true; } else if (xml.contains("arrowState=\"0\"")) { - throw new OseeArgumentException("Non-directional links not supported. id = " + getVueId()); + throw new OseeArgumentException("Non-directional links not supported. id [%s]", getVueId()); } } } diff --git a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/world/search/GroupWorldSearchItem.java b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/world/search/GroupWorldSearchItem.java index 24c0234df85..40745af1200 100644 --- a/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/world/search/GroupWorldSearchItem.java +++ b/plugins/org.eclipse.osee.ats/src/org/eclipse/osee/ats/world/search/GroupWorldSearchItem.java @@ -79,7 +79,7 @@ public class GroupWorldSearchItem extends WorldUISearchItem { group = UniversalGroup.getGroups(groupName, branch).iterator().next(); } if (group == null) { - throw new OseeArgumentException("Can't Find Universal Group for " + getName()); + throw new OseeArgumentException("Can't Find Universal Group for [%s]", getName()); } } |