UBERF-7419: Fix various sentry errors (#5931)

Author: haiodo

packages/core/src/operator.ts

@@ -38,6 +38,9 @@ function $push (document: Doc, keyval: Record<string, PropertyType>): void {
         arr.push(val)
       }
     } else {
+      if (doc[key] == null) {
+        doc[key] = []
+      }
       doc[key].push(val)
     }
   }
@@ -53,7 +56,7 @@ function $pull (document: Doc, keyval: Record<string, PropertyType>): void {
     if (typeof keyval[key] === 'object' && keyval[key] !== null) {
       const { $in } = keyval[key] as PullArray<PropertyType>
 
-      doc[key] = arr.filter((val) => {
+      doc[key] = (arr ?? []).filter((val) => {
         if ($in !== undefined) {
           return !$in.includes(val)
         } else {
@@ -67,7 +70,7 @@ function $pull (document: Doc, keyval: Record<string, PropertyType>): void {
         }
       })
     } else {
-      doc[key] = arr.filter((val) => val !== keyval[key])
+      doc[key] = (arr ?? []).filter((val) => val !== keyval[key])
     }
   }
 }
@@ -119,7 +122,7 @@ function $move (document: Doc, keyval: Record<string, PropertyType>): void {
     }
     const arr = doc[key] as Array<any>
     const desc = keyval[key]
-    doc[key] = arr.filter((val) => val !== desc.$value)
+    doc[key] = (arr ?? []).filter((val) => val !== desc.$value)
     doc[key].splice(desc.$position, 0, desc.$value)
   }
 }
@@ -134,7 +137,7 @@ function $pushMixin (document: Doc, options: any): void {
   const keyval = options.values
   for (const key in keyval) {
     const arr = mixin[key]
-    if (arr === undefined) {
+    if (arr == null) {
       mixin[key] = [keyval[key]]
     } else {
       arr.push(keyval[key])

packages/presentation/src/components/DocPopup.svelte

@@ -71,7 +71,10 @@
     created.length > 0 ||
     objects.map((it) => getObjectValue(groupBy, it)).filter((it, index, arr) => arr.indexOf(it) === index).length > 1
 
-  const checkSelected = (item: Doc): void => {
+  const checkSelected = (item?: Doc): void => {
+    if (item === undefined) {
+      return
+    }
     if (selectedElements.has(item._id)) {
       selectedElements.delete(item._id)
     } else {

packages/ui/src/colors.ts

@@ -304,7 +304,9 @@ export function getPlatformColors (darkTheme: boolean): readonly ColorDefinition
 }
 
 function hashCode (str: string): number {
-  return str.split('').reduce((prevHash, currVal) => ((prevHash << 5) - prevHash + currVal.charCodeAt(0)) | 0, 0)
+  return (str ?? '')
+    .split('')
+    .reduce((prevHash, currVal) => ((prevHash << 5) - prevHash + currVal.charCodeAt(0)) | 0, 0)
 }
 
 /**

packages/ui/src/components/DropdownLabelsPopup.svelte

@@ -41,6 +41,9 @@
 
   async function handleSelection (evt: Event | undefined, selection: number): Promise<void> {
     const item = objects[selection]
+    if (item == null) {
+      return
+    }
     if (multiselect && Array.isArray(selected)) {
       const index = selected.indexOf(item.id)
       if (index !== -1) {

packages/ui/src/components/Separator.svelte

@@ -208,10 +208,10 @@
 
   const checkSizes = (): void => {
     if (sState === SeparatorState.FLOAT) {
-      if (parentElement) initSize(parentElement, panel)
+      if (parentElement != null && panel != null) initSize(parentElement, panel)
     } else if (sState === SeparatorState.NORMAL) {
-      if (prevElement) initSize(prevElement, prevElSize)
-      if (nextElement) initSize(nextElement, nextElSize, true)
+      if (prevElement != null && prevElSize != null) initSize(prevElement, prevElSize)
+      if (nextElement != null && nextElSize != null) initSize(nextElement, nextElSize, true)
     }
   }
 

plugins/attachment-resources/src/components/AddAttachment.svelte

@@ -69,6 +69,7 @@
     name="file"
     id="file"
     style="display: none"
+    disabled={inputFile == null}
     on:change={fileSelected}
   />
 </div>

plugins/attachment-resources/src/components/AttachmentPopup.svelte

@@ -107,6 +107,7 @@
     name="file"
     id="file"
     style="display: none"
+    disabled={inputFile == null}
     on:change={fileSelected}
   />
   <div class="flex flex-between flex-grow header clear-mins">

plugins/attachment-resources/src/components/AttachmentRefInput.svelte

@@ -308,6 +308,7 @@
 <div class="no-print" bind:this={refContainer}>
   <input
     bind:this={inputFile}
+    disabled={inputFile == null}
     multiple
     type="file"
     name="file"

plugins/attachment-resources/src/components/AttachmentStyleBoxCollabEditor.svelte

@@ -243,6 +243,7 @@
 
 <input
   bind:this={inputFile}
+  disabled={inputFile == null}
   multiple
   type="file"
   name="file"

plugins/attachment-resources/src/components/Attachments.svelte

@@ -65,9 +65,7 @@
       loading--
     }
 
-    if (inputFile) {
-      inputFile.value = ''
-    }
+    inputFile.value = ''
 
     dispatch('attached')
   }
@@ -103,6 +101,7 @@
 
   <input
     bind:this={inputFile}
+    disabled={inputFile == null}
     multiple
     type="file"
     name="file"