Devassist | Hover solution (AST-16742^)

.gitignore

@@ -7,4 +7,4 @@
 *.jar
 !checkmarx-ast-eclipse-plugin/lib/*.jar
 !devassist-lib/lib/*.jar
-!common-lib/lib/*.jar
+!common-lib/lib/*.jar

checkmarx-ast-eclipse-plugin/src/com/checkmarx/eclipse/utils/PluginUtils.java

@@ -51,7 +51,8 @@ public static String convertStringTimeStamp(String timestamp) {
 
 			Instant instant = Instant.parse(timestamp);
 
-			DateTimeFormatter dateTimeFormatter = DateTimeFormatter.ofPattern(PARAM_TIMESTAMP_PATTERN).withZone(ZoneId.systemDefault());
+			DateTimeFormatter dateTimeFormatter = DateTimeFormatter.ofPattern(PARAM_TIMESTAMP_PATTERN)
+					.withZone(ZoneId.systemDefault());
 			parsedDate = dateTimeFormatter.format(instant);
 		} catch (Exception e) {
 			CxLogger.warning(String.format("[TIME-STAMP] Returning time stamp", e.getMessage()));
@@ -99,19 +100,21 @@ public static void setTextForComboViewer(ComboViewer comboViewer, String text) {
 	 */
 	public static void updateFiltersEnabledAndCheckedState(List<Action> filterActions) {
 		for (Action action : filterActions) {
-			// avoid to disable group by severity , group by query name and group by state actions
-			if (!action.getId().equals(ActionName.GROUP_BY_SEVERITY.name()) && !action.getId().equals(ActionName.GROUP_BY_QUERY_NAME.name()) && !action.getId().equals(ActionName.GROUP_BY_STATE_NAME.name()) ) {
+			// avoid to disable group by severity , group by query name and group by state
+			// actions
+			if (!action.getId().equals(ActionName.GROUP_BY_SEVERITY.name())
+					&& !action.getId().equals(ActionName.GROUP_BY_QUERY_NAME.name())
+					&& !action.getId().equals(ActionName.GROUP_BY_STATE_NAME.name())) {
 				action.setEnabled(DataProvider.getInstance().containsResults());
 			}
-			
-			if(!action.getId().equals(ActionName.FILTER_CHANGED.name())) {
+
+			if (!action.getId().equals(ActionName.FILTER_CHANGED.name())) {
 				action.setChecked(FilterState.isSeverityEnabled(action.getId()));
 			}
 
-
 		}
 	}
-	
+
 	/**
 	 * Create a display model to be presented in the tree
 	 * 
@@ -121,7 +124,7 @@ public static void updateFiltersEnabledAndCheckedState(List<Action> filterAction
 	public static DisplayModel message(String message) {
 		return new DisplayModel.DisplayModelBuilder(message).build();
 	}
-	
+
 	/**
 	 * Show message in the tree
 	 * 
@@ -132,17 +135,16 @@ public static void showMessage(DisplayModel rootModel, TreeViewer viewer, String
 		rootModel.children.add(PluginUtils.message(message));
 		viewer.refresh();
 	}
-
-
+
 	/**
 	 * Clear message in the tree
 	 * 
 	 */
-	public static void clearMessage(DisplayModel rootModel,TreeViewer viewer) {
+	public static void clearMessage(DisplayModel rootModel, TreeViewer viewer) {
 		rootModel.children.clear();
 		viewer.refresh();
 	}
-		
+
 	/**
 	 * Get Event Broker
 	 * 
@@ -151,7 +153,7 @@ public static void clearMessage(DisplayModel rootModel,TreeViewer viewer) {
 	public static IEventBroker getEventBroker() {
 		return (IEventBroker) PlatformUI.getWorkbench().getService(IEventBroker.class);
 	}
-	
+
 	/**
 	 * Check if checkmarx credentials are defined in the Preferences
 	 * 
@@ -160,7 +162,7 @@ public static IEventBroker getEventBroker() {
 	public static boolean areCredentialsDefined() {
 		return StringUtils.isNotBlank(Preferences.getApiKey());
 	}
-	
+
 	/**
 	 * Add Checkmarx vulnerabilities to Problems View
 	 * 
@@ -194,7 +196,7 @@ public static void addVulnerabilitiesToProblemsView(List<Result> resultsList) {
 			}
 		}
 	}
-	
+
 	/**
 	 * Get IMarker severity based on each checkmarx result severity
 	 * 
@@ -203,25 +205,25 @@ public static void addVulnerabilitiesToProblemsView(List<Result> resultsList) {
 	 */
 	private static Integer getIMarkerSeverity(String resultSeverity) {
 		Severity severity = Severity.getSeverity(resultSeverity);
-		
+
 		switch (severity) {
-		case CRITICAL:
-			return IMarker.SEVERITY_ERROR;
-		case HIGH:
-			return IMarker.SEVERITY_ERROR;
-		case MEDIUM:
-			return IMarker.SEVERITY_WARNING;
-		case LOW:
-			return IMarker.SEVERITY_INFO;
-		case INFO:
-			return IMarker.SEVERITY_INFO;
-		default:
-			break;
+			case CRITICAL:
+				return IMarker.SEVERITY_ERROR;
+			case HIGH:
+				return IMarker.SEVERITY_ERROR;
+			case MEDIUM:
+				return IMarker.SEVERITY_WARNING;
+			case LOW:
+				return IMarker.SEVERITY_INFO;
+			case INFO:
+				return IMarker.SEVERITY_INFO;
+			default:
+				break;
 		}
-		
+
 		return IMarker.SEVERITY_INFO;
 	}
-	
+
 	/**
 	 * Find files in workspace
 	 * 
@@ -231,7 +233,8 @@ private static Integer getIMarkerSeverity(String resultSeverity) {
 	public static List<IFile> findFileInWorkspace(final String fileName) {
 		final List<IFile> foundFiles = new ArrayList<IFile>();
 		try {
-			// visiting only resources proxy because we obtain the resource only when matching name, thus the workspace traversal is much faster
+			// visiting only resources proxy because we obtain the resource only when
+			// matching name, thus the workspace traversal is much faster
 			ResourcesPlugin.getWorkspace().getRoot().accept(new IResourceProxyVisitor() {
 				@Override
 				public boolean visit(IResourceProxy resourceProxy) throws CoreException {
@@ -250,23 +253,24 @@ public boolean visit(IResourceProxy resourceProxy) throws CoreException {
 		}
 		return foundFiles;
 	}
-	
+
 	/**
 	 * Clear checkmarx vulnerabilities from Problems View
 	 */
 	public static void clearVulnerabilitiesFromProblemsView() {
 		IWorkspace workspace = ResourcesPlugin.getWorkspace();
 		IResource resource = workspace.getRoot();
 		IMarker[] markers;
-		
+
 		try {
 			markers = resource.findMarkers(IMarker.MARKER, true, IResource.DEPTH_INFINITE);
-			
+
 			for (IMarker m : markers) {
-				if(m.getAttribute(IMarker.SOURCE_ID) != null && m.getAttribute(IMarker.SOURCE_ID).equals(PluginConstants.PROBLEM_SOURCE_ID)) {
+				if (m.getAttribute(IMarker.SOURCE_ID) != null
+						&& m.getAttribute(IMarker.SOURCE_ID).equals(PluginConstants.PROBLEM_SOURCE_ID)) {
 					m.delete();
 				}
-			}			
+			}
 		} catch (CoreException e) {
 			CxLogger.error(String.format(PluginConstants.ERROR_FINDING_OR_DELETING_MARKER, e.getMessage()), e);
 		}