Skip to content

Commit 6e25077

Browse files
Removed extra method and added necessary comments
1 parent 4415697 commit 6e25077

2 files changed

Lines changed: 4 additions & 7 deletions

File tree

checkmarx-ast-eclipse-plugin/src/com/checkmarx/eclipse/views/CheckmarxView.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1330,7 +1330,7 @@ public void selectionChanged(SelectionChangedEvent event) {
13301330
String currentState = selectedItem.getState();
13311331
selectedState = selectedItem.getResult().getState();
13321332

1333-
// Fetch dynamic states from DataProvider
1333+
// [AST-92100] Fetch dynamic states from DataProvider
13341334
List<String> state = DataProvider.getInstance().getStatesForEngine(selectedItem.getType());
13351335

13361336
triageStateComboViewer.setContentProvider(ArrayContentProvider.getInstance());

checkmarx-ast-eclipse-plugin/src/com/checkmarx/eclipse/views/DataProvider.java

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -202,7 +202,7 @@ public List<DisplayModel> getResultsForScanId(String scanId) {
202202
Results scanResults = null;
203203

204204
setCurrentScanId(scanId);
205-
205+
// [AST-92100] Fetch all platform states before fetching results
206206
try {
207207
platformStates = getAllStatesFromPlatform();
208208
} catch (Exception e) {
@@ -246,11 +246,8 @@ public Scan getScanInformation(String scanId) throws Exception {
246246

247247
return scan;
248248
}
249-
250-
public List<String> getAvailableStates() {
251-
return platformStates != null ? platformStates : Collections.emptyList();
252-
}
253249

250+
// [AST-92100] Get all states for a specific engine type
254251
public List<String> getStatesForEngine(String engineType) {
255252
if ("SAST".equalsIgnoreCase(engineType)) {
256253
return platformStates != null ? platformStates : Collections.emptyList();
@@ -793,7 +790,7 @@ public boolean isScanAllowed() throws CxException, IOException, InterruptedExcep
793790
}
794791

795792
/**
796-
* Fetch ALL platform states (predefined + custom) irrespective of
793+
* [AST-92100] Fetch ALL platform states (predefined + custom) irrespective of
797794
* vulnerabilities.
798795
*/
799796
private List<String> getAllStatesFromPlatform() throws Exception {

0 commit comments

Comments
 (0)