fixes FPBASE-5EX (#395)

Author: tlambert03

backend/proteins/views/microscope.py

@@ -44,7 +44,10 @@ def update_scope_report(request):
     if request.POST.get("action") == "update":
         outdated = request.POST.get("outdated")
         if outdated:
-            outdated = json.loads(outdated)
+            try:
+                outdated = json.loads(outdated)
+            except json.JSONDecodeError:
+                outdated = None
         if scope_id:
             try:
                 # this is throwing connection resets

frontend/src/js/ajax-sentry.js

@@ -20,6 +20,23 @@
 
 import * as Sentry from "@sentry/browser"
 
+/**
+ * Create URLSearchParams from an object, filtering out undefined/null values.
+ * This mimics jQuery's $.param() behavior which skips undefined keys.
+ *
+ * @param {object} obj - Object to convert to URLSearchParams
+ * @returns {URLSearchParams} - URLSearchParams with undefined/null values filtered out
+ */
+export function createFormData(obj) {
+  const filtered = {}
+  for (const [key, value] of Object.entries(obj)) {
+    if (value !== undefined && value !== null) {
+      filtered[key] = value
+    }
+  }
+  return new URLSearchParams(filtered)
+}
+
 /**
  * Determine if an error should be reported to Sentry
  * Override this function to customize filtering logic

frontend/src/js/scope_report.js

@@ -1,5 +1,5 @@
 import Highcharts from "highcharts"
-import { fetchWithSentry } from "./ajax-sentry"
+import { createFormData, fetchWithSentry } from "./ajax-sentry"
 
 const $ = window.jQuery // jQuery loaded from CDN
 
@@ -36,10 +36,7 @@ $.fn.extend({
     var SCOPE_ID = config.scopeID
     var SCOPE_URL = config.scopeURL
     var JOB_ID = null
-    var OUTDATED
-    if (config.outdated.length) {
-      OUTDATED = config.outdated
-    }
+    var OUTDATED = config.outdated || []
     var interval = config.interval || 3500
     var FLUORS
     var REPORT
@@ -367,7 +364,7 @@ $.fn.extend({
     function check_status(next) {
       next = next || interval
       if (JOB_ID !== null) {
-        const formData = new URLSearchParams({
+        const formData = createFormData({
           action: "check",
           job_id: JOB_ID,
           csrfmiddlewaretoken: CSRF_TOKEN,
@@ -421,12 +418,12 @@ $.fn.extend({
         action = "cancel"
       }
 
-      const formData = new URLSearchParams({
+      const formData = createFormData({
         action: action,
         scope_id: SCOPE_ID,
         csrfmiddlewaretoken: CSRF_TOKEN,
         job_id: JOB_ID,
-        outdated: JSON.stringify(OUTDATED),
+        outdated: OUTDATED.length ? JSON.stringify(OUTDATED) : null,
       })
 
       fetchWithSentry("", {