first working implementation

Author: xb058t

web/server/vue-cli/src/views/Reports.vue

@@ -2,6 +2,7 @@
   <splitpanes class="default-theme">
     <pane size="20" :style="{ 'min-width': '300px' }">
       <report-filter
+        ref="reportFilter"
         v-fill-height
         :namespace="namespace"
         :report-count="totalItems"
@@ -236,8 +237,9 @@
             <v-treeview
               :items="formattedDirectoriesForTreeViewRunResults"
               activatable
-              item-key="name"
+              item-key="fullPath"
               open-on-click
+              @update:active="onTreeFileClick"
             >
               <template #prepend="{ item, open }">
                 <v-icon v-if="item.children.length > 0">
@@ -260,8 +262,9 @@
             <v-treeview
               :items="formattedDirectoriesForTreeViewFileCounts"
               activatable
-              item-key="name"
+              item-key="fullPath"
               open-on-click
+              @update:active="onTreeFileClick"
             >
               <template #prepend="{ item, open }">
                 <v-icon v-if="item.children.length > 0">
@@ -286,10 +289,13 @@
 <script>
 import { Pane, Splitpanes } from "splitpanes";
 
-import { mapGetters } from "vuex";
+import { mapGetters, mapMutations } from "vuex";
 
 import { ccService, handleThriftError } from "@cc-api";
-import { Checker, Order, SortMode, SortType } from "@cc/report-server-types";
+import {
+  Checker, Order, SortMode, SortType
+} from "@cc/report-server-types";
+import { SET_REPORT_FILTER } from "@/store/mutations.type";
 
 import { FillHeight } from "@/directives";
 import { BugPathLengthColorMixin, DetectionStatusMixin } from "@/mixins";
@@ -329,11 +335,12 @@ const TreeItem = {
   `
 };
 
-// codechecker analyze enable all + store it + show in tree view
-// measure the time between current impelementation and the one with getCheckerCounts
-// implementation A and B for thesis documentation
-// getFileCounts(report server.thrift) 
-// 
+// codechecker analyze enable all + store it + show in tree view DONE
+// measure the time between current impelementation and the one with getCheckerCounts DONE
+// implementation A and B for thesis documentation DONE
+// getFileCounts(report server.thrift)  DONE
+
+// getCheckerCounts(report server.thrift) - if performance with getFileCounts is not good, we can try to implement this and use it instead, but it is needed to add new endpoint to get counts for all checkers and not only total count, so it is going to be filename, checker name -> count DONE
 // Introduce new hash in table, in some cases when different checkers report to the same line they may report the same error, so it is needed to group them up
 // In cases the reports are different we should do mapping table to a bug type and add that to hash again
 // Filename, line, bug type -> new hash
@@ -563,13 +570,18 @@ export default {
       this.allReportsRunResults.forEach(report => {
         const pathParts = report.checkedFile.split("/").slice(0, -1);
         let currentLevel = items;
+        let currentPath = "";
         pathParts.forEach(part => {
           if (part === "") return;
 
-          let existingPart = currentLevel.find(item => item.name === part);
+          currentPath += "/" + part;
+          let existingPart = currentLevel.find(
+            item => item.name === part
+          );
           if (!existingPart) {
             existingPart = {
               name: part,
+              fullPath: currentPath,
               children: [],
               findings: 0
             };
@@ -579,14 +591,19 @@ export default {
         });
 
         // append filename as a child of the last directory
-        const fileName = report.checkedFile.split("/").slice(-1)[0];
+        const fileName = report.checkedFile
+          .split("/").slice(-1)[0];
         if (fileName) {
-          let existingFile = currentLevel.find(item => item.name === fileName);
+          const filePath = currentPath + "/" + fileName;
+          const existingFile = currentLevel.find(
+            item => item.name === fileName
+          );
           if (existingFile) {
             existingFile.findings += 1;
           } else {
             currentLevel.push({
               name: fileName,
+              fullPath: filePath,
               children: [],
               findings: 1
             });
@@ -613,17 +630,24 @@ export default {
     formattedDirectoriesForTreeViewFileCounts() {
       const items = [];
       
-      Object.entries(this.allReportsFileCounts || {}).forEach(([filePath, count]) => {
+      Object.entries(
+        this.allReportsFileCounts || {}
+      ).forEach(([ filePath, count ]) => {
         if (!filePath) return;
         const pathParts = filePath.split("/").slice(0, -1);
         let currentLevel = items;
+        let currentPath = "";
         pathParts.forEach(part => {
           if (part === "") return;
 
-          let existingPart = currentLevel.find(item => item.name === part);
+          currentPath += "/" + part;
+          let existingPart = currentLevel.find(
+            item => item.name === part
+          );
           if (!existingPart) {
             existingPart = {
               name: part,
+              fullPath: currentPath,
               children: [],
               findings: 0
             };
@@ -635,12 +659,15 @@ export default {
         // append filename as a child of the last directory
         const fileName = filePath.split("/").slice(-1)[0];
         if (fileName) {
-          let existingFile = currentLevel.find(item => item.name === fileName);
+          const existingFile = currentLevel.find(
+            item => item.name === fileName
+          );
           if (existingFile) {
             existingFile.findings += count;
           } else {
             currentLevel.push({
               name: fileName,
+              fullPath: filePath,
               children: [],
               findings: count
             });
@@ -692,6 +719,32 @@ export default {
   },
 
   methods: {
+    ...mapMutations(namespace, {
+      setReportFilter: SET_REPORT_FILTER
+    }),
+
+    onTreeFileClick(activeItems) {
+      // activeItems is an array of item-key values (fullPath)
+      if (!activeItems || activeItems.length === 0) return;
+
+      const filePath = activeItems[0];
+      if (!filePath) return;
+
+      // Find the FilePathFilter instance inside ReportFilter
+      // and call its setSelectedItems to select this file.
+      const filters = this.$refs.reportFilter.$refs.filters;
+      const filePathFilter = filters.find(
+        f => f.id === "filepath"
+      );
+      if (filePathFilter) {
+        filePathFilter.setSelectedItems([
+          { id: filePath, title: filePath, count: "N/A" }
+        ]);
+      }
+
+      this.viewMode = "table";
+    },
+
     itemExpanded(expandedItem) {
       if (expandedItem.item.sameReports) return;