Skip to content
Snippets Groups Projects
Commit 84622027 authored by Hernan Ponce de Leon's avatar Hernan Ponce de Leon
Browse files

[RED] Code review + removed unused methods

parent 68776913
No related branches found
No related tags found
No related merge requests found
...@@ -15,15 +15,7 @@ ...@@ -15,15 +15,7 @@
+--------------------------------------------------------------------------*/ +--------------------------------------------------------------------------*/
package org.fortiss.tooling.base.ui.utils; 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.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.ConstraintInstance;
import org.fortiss.tooling.kernel.model.constraints.FailedConstraintInstanceStatus; import org.fortiss.tooling.kernel.model.constraints.FailedConstraintInstanceStatus;
import org.fortiss.tooling.kernel.model.constraints.IConstraintInstanceContainer; import org.fortiss.tooling.kernel.model.constraints.IConstraintInstanceContainer;
...@@ -35,41 +27,6 @@ import org.fortiss.tooling.kernel.model.constraints.IConstraintInstanceContainer ...@@ -35,41 +27,6 @@ import org.fortiss.tooling.kernel.model.constraints.IConstraintInstanceContainer
*/ */
public class StatusUtils { 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. * This function is to calculate the status of all of the instances of a particular constraint.
* *
...@@ -86,6 +43,7 @@ public class StatusUtils { ...@@ -86,6 +43,7 @@ public class StatusUtils {
IConstraintInstanceContainer instanceContainer, EList<String> currentActiveConstraints) { IConstraintInstanceContainer instanceContainer, EList<String> currentActiveConstraints) {
for(ConstraintInstance ci : instanceContainer.getConstraintInstances()) { for(ConstraintInstance ci : instanceContainer.getConstraintInstances()) {
// TODO(HP) why using toString()? constraintToFind is already a string.
if(constraintToFind.toString().equalsIgnoreCase(ci.getConstraintName())) { if(constraintToFind.toString().equalsIgnoreCase(ci.getConstraintName())) {
if(currentActiveConstraints.contains(constraintToFind.toString())) { if(currentActiveConstraints.contains(constraintToFind.toString())) {
if(ci.getConstraintName().equalsIgnoreCase(constraintToFind.toString())) { if(ci.getConstraintName().equalsIgnoreCase(constraintToFind.toString())) {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment