@konfuzio/document-validation-ui 0.2.8-dev.0 → 0.2.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@konfuzio/document-validation-ui",
-  "version": "0.2.8-dev.0",
+  "version": "0.2.8",
   "repository": "https://github.com/konfuzio-ai/document-validation-ui",
   "main": "dist/app.js",
   "scripts": {
@@ -44,6 +44,7 @@
   "devDependencies": {
     "@4tw/cypress-drag-drop": "^2.3.0",
     "@babel/preset-env": "^7.26.9",
+    "@vue/cli-plugin-eslint": "^5.0.9",
     "@vue/cli-service": "^5.0.9",
     "@vue/compiler-sfc": "^3.1.0",
     "@vue/test-utils": "^2.4.6",
@@ -60,6 +61,7 @@
   },
   "overrides": {
     "postcss": "8.4.31",
-    "webpack-dev-server": "5.2.1"
+    "webpack-dev-server": "5.2.1",
+    "node-forge": "1.3.2"
   }
 }

package/src/api.js

@@ -119,7 +119,6 @@ const makeGetPaginatedRequest = (request, hasParams = false) => {
       } catch (error) {
         reject(error);
         toFinishLoop = true;
-        console.log(error);
       }
     } while (!toFinishLoop);
     resolve(returnResponse);

package/src/components/DocumentAnnotations/AnnotationContent.vue

@@ -127,8 +127,8 @@ export default {
               this.$refs.contentEditable.focus();
             }
           })
-          .catch((error) => {
-            console.log(error);
+          .catch(() => {
+            // Error is silently ignored
           });
         // check if this is part of a group of spans to show the whole bounding box as a placeholder
         if (this.annotation.selection_bbox && this.annotation.span.length > 1) {

package/src/components/DocumentAnnotations/AnnotationRow.vue

@@ -113,7 +113,7 @@
               v-for="(span, index) in spanForEditing
                 ? spanSelection
                 : annotation.span"
-              :key="index"
+              :key="`span-${span.x0}-${span.y0}-${span.page_index}-${index}`"
               @mouseenter="onAnnotationHoverEnter(span)"
               @mouseleave="onAnnotationHoverLeave"
             >
@@ -139,7 +139,7 @@
           >
             <EmptyAnnotation
               v-for="(span, index) in spanSelection"
-              :key="index"
+              :key="`empty-span-${span.x0}-${span.y0}-${span.page_index}-${index}`"
               :span="span"
               :span-index="index"
               :label="label"

package/src/components/DocumentAnnotations/DocumentLabel.vue

@@ -2,8 +2,8 @@
   <div class="label">
     <div v-if="annotationSet && annotationsToShow().length > 0">
       <AnnotationRow
-        v-for="(annotation, index) in annotationsToShow()"
-        :key="index"
+        v-for="annotation in annotationsToShow()"
+        :key="`annotation-${annotation.id}`"
         :annotation="annotation"
         :label="label"
         :annotation-set="annotationSet"

package/src/components/DocumentAnnotations/MultiAnnotationTableOverlay.vue

@@ -21,8 +21,8 @@
       @columndragleave="columndragleave"
     >
       <b-table-column
-        v-for="(item, index) in isLoading ? [columns[0]] : columns"
-        :key="index"
+        v-for="item in isLoading ? [columns[0]] : columns"
+        :key="`column-${item.field}`"
         :field="item.field"
         :label="item.label.name"
       >

package/src/components/DocumentEdit/RenameAndCategorize.vue

@@ -18,7 +18,7 @@
     <div class="new-documents-container">
       <div
         v-for="(page, index) in updatedDocument"
-        :key="index"
+        :key="`page-${page.pages?.[0]?.id || index}`"
         class="document-details"
       >
         <div class="rename-and-categorize-thumbnails">

package/src/components/DocumentPage/DocumentPage.vue

@@ -50,62 +50,66 @@
           }"
         />
         <template v-if="pageInVisibleRange && !editMode">
-          <template v-if="searchResults.length > 0">
-            <v-rect
-              v-for="(bbox, index) in searchResults"
-              :key="'sr' + index"
-              :config="{
-                ...selectionTextRect(bbox, bbox === currentSearchResultForPage),
-              }"
-            ></v-rect>
-          </template>
-          <v-group
-            v-if="
-              !categorizeModalIsActive || !publicView || !isDocumentReviewed
-            "
-            ref="entities"
-          >
-            <v-rect
-              v-for="(entity, index) in scaledEntities(page.entities, page)"
-              :key="index"
-              :config="entityRect(entity)"
-              @click="handleClickedEntity(entity)"
-              @mouseenter="onElementEnter"
-              @mouseleave="onElementLeave"
-            />
-          </v-group>
-          <template v-for="(annotationSet, index) in pageAnnotationSets">
-            <v-group>
+          <!-- Main Group -->
+          <v-group>
+            <!-- Search Results -->
+            <template v-if="searchResults.length > 0">
               <v-rect
-                :config="groupAnnotationRect(annotationSet, index)"
-                @click="handleClickedAnnotationSet(annotationSet)"
-                @mouseenter="onAnnotationSetEnter(annotationSet)"
-                @mouseleave="onAnnotationSetLeave"
+                v-for="(bbox, index) in searchResults"
+                :key="'sr' + index"
+                :config="{
+                  ...selectionTextRect(
+                    bbox,
+                    bbox === currentSearchResultForPage
+                  ),
+                }"
               />
-            </v-group>
-          </template>
-          <template v-for="annotation in pageAnnotations">
+            </template>
+
+            <!-- Grouped Ann Sets Box -->
+            <v-rect
+              v-for="(annotationSet, index) in pageAnnotationSets"
+              :key="`annotation-set-${annotationSet.id || index}`"
+              :config="groupAnnotationRect(annotationSet, index)"
+              @click="handleClickedAnnotationSet(annotationSet)"
+              @mouseenter="onAnnotationSetEnter(annotationSet)"
+              @mouseleave="onAnnotationSetLeave"
+            />
+
+            <!-- Entities -->
             <template
-              v-for="(bbox, index) in annotation.span.filter(
-                (bbox) => bbox.page_index + 1 == page.number
-              )"
+              v-if="
+                !categorizeModalIsActive || !publicView || !isDocumentReviewed
+              "
             >
-              <v-group>
+              <v-rect
+                v-for="(entity, index) in scaledEntities(page.entities, page)"
+                :key="`entity-${entity.id || index}`"
+                :config="entityRect(entity)"
+                @click="handleClickedEntity(entity)"
+                @mouseenter="onElementEnter"
+                @mouseleave="onElementLeave"
+              />
+            </template>
+
+            <!-- Annotations -->
+            <template v-for="annotation in pageAnnotations">
+              <template v-if="!isAnnotationInEditMode(annotation.id)">
                 <v-rect
-                  v-if="!isAnnotationInEditMode(annotation.id)"
+                  v-for="(bbox, index) in annotation.span.filter(
+                    (bbox) => bbox.page_index + 1 == page.number
+                  )"
+                  :key="`annotation-${annotation.id}-bbox-${bbox.x0}-${bbox.y0}-${index}`"
                   :config="annotationRect(bbox, annotation.id)"
                   @click="handleFocusedAnnotation(annotation)"
                   @mouseenter="onElementEnter(annotation, bbox)"
                   @mouseleave="onElementLeave"
                 />
-              </v-group>
-            </template>
-            <template
-              v-if="annotation.metadata && annotation.metadata.checkbox"
-            >
-              <v-group>
+
+                <!-- Annotations Checkboxes-->
                 <v-rect
-                  v-if="!isAnnotationInEditMode(annotation.id)"
+                  v-if="annotation.metadata && annotation.metadata.checkbox"
+                  :key="`annotation-${annotation.id}-checkbox`"
                   :config="
                     annotationRect(
                       annotation.metadata.checkbox.bbox,
@@ -121,24 +125,25 @@
                   "
                   @mouseleave="onElementLeave"
                 />
-              </v-group>
+              </template>
             </template>
-          </template>
+          </v-group>
         </template>
       </v-layer>
       <v-layer>
-        <template
+        <span-selection
           v-for="(span, index) in spanSelectionsForPage(page)"
-          :key="index"
-        >
-          <span-selection :id="index" :span="span" :page="page" />
-        </template>
-        <template
+          :id="index"
+          :key="`span-selection-${span.x0}-${span.y0}-${index}`"
+          :span="span"
+          :page="page"
+        />
+        <placeholder-selection
           v-for="(span, index) in placeholderSelectionForPage(page)"
           :key="`${index}_placeholder`"
-        >
-          <placeholder-selection :span="span" :page="page" />
-        </template>
+          :span="span"
+          :page="page"
+        />
         <template v-if="page.number === selectionPage">
           <box-selection :page="page" />
         </template>
@@ -551,7 +556,7 @@ export default {
           .then((myBlob) => {
             convertBlob(myBlob);
           })
-          .catch((error) => {});
+          .catch(() => {});
       } else {
         convertBlob(this.imageBlob);
       }

package/src/components/DocumentPage/DocumentToolbar.vue

@@ -260,7 +260,6 @@ export default {
             serverErrorMessage: this.$t("server_error"),
             defaultErrorMessage: this.$t("error_downloading_file"),
           });
-          console.log(error);
         });
     },
     cancelAnnotationEditMode() {

package/src/store/category.js

@@ -158,8 +158,8 @@ const actions = {
           commit("SET_CATEGORIES", results);
         }
       })
-      .catch((error) => {
-        console.log(error, "Could not fetch categories from the backend");
+      .catch(() => {
+        // Error is silently ignored
       });
   },
 };

package/src/store/display.js

@@ -287,7 +287,7 @@ const actions = {
         break;
       }
       default:
-        console.log("No fit defined");
+        // No fit defined
         break;
     }
   },

package/src/store/document.js

@@ -1263,7 +1263,6 @@ const actions = {
         }
       })
       .catch((error) => {
-        console.log(error, "Could not fetch document details from the backend");
         dispatch("display/setPageError", error.response.data.detail, {
           root: true,
         });
@@ -1293,8 +1292,8 @@ const actions = {
       .then((response) => {
         commit("ADD_PAGE", response.data);
       })
-      .catch((error) => {
-        console.log(error);
+      .catch(() => {
+        // Error is silently ignored
       });
   },
 
@@ -1311,8 +1310,8 @@ const actions = {
       .then((response) => {
         commit("SET_DOC_SET", response.data);
       })
-      .catch((error) => {
-        console.log(error);
+      .catch(() => {
+        // Error is silently ignored
       });
   },
 
@@ -1366,7 +1365,6 @@ const actions = {
         })
         .catch((error) => {
           reject(error.response);
-          console.log(error);
         });
     });
   },
@@ -1392,7 +1390,6 @@ const actions = {
         })
         .catch((error) => {
           reject(error.response);
-          console.log(error);
         });
     });
   },
@@ -1432,7 +1429,6 @@ const actions = {
         })
         .catch((error) => {
           reject(error.response);
-          console.log(error);
         });
     });
   },
@@ -1490,7 +1486,6 @@ const actions = {
         })
         .catch((error) => {
           reject(error.response);
-          console.log(error);
         });
     });
   },
@@ -1508,7 +1503,6 @@ const actions = {
         })
         .catch((error) => {
           reject(error.response);
-          console.log(error);
         });
     });
   },
@@ -1524,7 +1518,6 @@ const actions = {
         })
         .catch((error) => {
           reject(error.response);
-          console.log(error);
         });
     });
   },
@@ -1575,7 +1568,6 @@ const actions = {
         })
         .catch((error) => {
           reject(error.response);
-          console.log(error);
         });
     });
   },

package/src/store/edit.js

@@ -260,7 +260,6 @@ const actions = {
         })
         .catch((error) => {
           reject(error.response);
-          console.log(error);
         });
     });
   },

package/src/store/project.js

@@ -22,7 +22,6 @@ const actions = {
         })
         .catch((error) => {
           reject(error);
-          console.log(error);
         });
     });
   },
@@ -34,8 +33,8 @@ const actions = {
       .then((results) => {
         commit("SET_LABEL_SETS", results);
       })
-      .catch((error) => {
-        console.log(error);
+      .catch(() => {
+        // Error is silently ignored
       });
   },
 
@@ -44,8 +43,8 @@ const actions = {
       .then((response) => {
         commit("SET_CURRENT_USER", response.data);
       })
-      .catch((error) => {
-        console.log(error);
+      .catch(() => {
+        // Error is silently ignored
       });
   },
 
@@ -71,9 +70,8 @@ const actions = {
         .then((results) => {
           resolve(results);
         })
-        .catch((error) => {
+        .catch(() => {
           reject();
-          console.log(error, "Could not fetch document list from the backend");
         });
     });
   },
@@ -87,9 +85,8 @@ const actions = {
         .then((results) => {
           resolve(results);
         })
-        .catch((error) => {
+        .catch(() => {
           reject();
-          console.log(error, "Could not fetch document list from the backend");
         });
     });
   },