From 84622027d96aab77f215eb316fc33989f0b03d66 Mon Sep 17 00:00:00 2001 From: Hernan Ponce de Leon <ponce@fortiss.org> Date: Fri, 6 Apr 2018 06:57:39 +0000 Subject: [PATCH] [RED] Code review + removed unused methods --- .../tooling/base/ui/utils/StatusUtils.java | 44 +------------------ 1 file changed, 1 insertion(+), 43 deletions(-) diff --git a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/utils/StatusUtils.java b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/utils/StatusUtils.java index d4572b85e..08f5013ee 100644 --- a/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/utils/StatusUtils.java +++ b/org.fortiss.tooling.base.ui/trunk/src/org/fortiss/tooling/base/ui/utils/StatusUtils.java @@ -15,15 +15,7 @@ +--------------------------------------------------------------------------*/ package org.fortiss.tooling.base.ui.utils; -import static org.conqat.ide.commons.ui.ui.WorkbenchUtils.getActiveWorkbenchPage; - import org.eclipse.emf.common.util.EList; -import org.eclipse.jface.action.IStatusLineManager; -import org.eclipse.ui.IActionBars; -import org.eclipse.ui.IViewSite; -import org.eclipse.ui.IWorkbenchPage; -import org.eclipse.ui.IWorkbenchPart; -import org.eclipse.ui.IWorkbenchPartSite; import org.fortiss.tooling.kernel.model.constraints.ConstraintInstance; import org.fortiss.tooling.kernel.model.constraints.FailedConstraintInstanceStatus; import org.fortiss.tooling.kernel.model.constraints.IConstraintInstanceContainer; @@ -35,41 +27,6 @@ import org.fortiss.tooling.kernel.model.constraints.IConstraintInstanceContainer */ public class StatusUtils { - /** - * Returns a Status Line object of the current window. - * - * @return the status line manager - */ - public static IStatusLineManager getStatusLine() { - IWorkbenchPage page = getActiveWorkbenchPage(); - IWorkbenchPart part = page.getActivePart(); - IWorkbenchPartSite site = part.getSite(); - IViewSite vSite = (IViewSite)site; - IActionBars actionBars = vSite.getActionBars(); - - if(actionBars == null) { - return null; - } - - IStatusLineManager statusLine = actionBars.getStatusLineManager(); - if(statusLine == null) { - return null; - } - return statusLine; - } - - /** - * Writes messages into the status line of the current window. - * - * @param message - * the message - */ - public static void setStatusMessage(String message) { - if(getStatusLine() != null) { - getStatusLine().setMessage(message); - } - } - /** * This function is to calculate the status of all of the instances of a particular constraint. * @@ -86,6 +43,7 @@ public class StatusUtils { IConstraintInstanceContainer instanceContainer, EList<String> currentActiveConstraints) { for(ConstraintInstance ci : instanceContainer.getConstraintInstances()) { + // TODO(HP) why using toString()? constraintToFind is already a string. if(constraintToFind.toString().equalsIgnoreCase(ci.getConstraintName())) { if(currentActiveConstraints.contains(constraintToFind.toString())) { if(ci.getConstraintName().equalsIgnoreCase(constraintToFind.toString())) { -- GitLab