Remove the load_fail value in table_contents

Removes the load_fail value in table_contents due to finding a better implementation. Table data existence is checked by length of "values" in table_data.
HEPData · Nov 8, 2023 · de500f0 · de500f0
1 parent b214fd9
commit de500f0
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 5 deletions.
diff --git a/hepdata/modules/records/assets/js/hepdata_tables.js b/hepdata/modules/records/assets/js/hepdata_tables.js
@@ -103,8 +103,7 @@ HEPDATA.table_renderer = {
 
         $("#hepdata_table_loader").addClass("hidden");
         $("#hepdata_table_content").removeClass("hidden");
-
-        if(table_data.load_fail) {
+        if(table_data.values.length == 0) {
           $("#hepdata_table_loader").addClass("hidden");
           $("#hepdata_filesize_fail").removeClass("hidden");
           $("#hep_table").addClass("hidden");

diff --git a/hepdata/modules/records/utils/data_processing_utils.py b/hepdata/modules/records/utils/data_processing_utils.py
@@ -216,8 +216,7 @@ def generate_table_structure(table_contents):
               "review": table_contents["review"],
               "associated_files": table_contents["associated_files"],
               "keywords": {},
-              "values": [],
-              "load_fail": table_contents["load_fail"]}
+              "values": []}
 
     record["description"] = sanitize_html(table_contents["title"])
 

diff --git a/hepdata/modules/records/views.py b/hepdata/modules/records/views.py
@@ -333,7 +333,6 @@ def get_table_details(recid, data_recid, version, load_all):
                 table_contents["dependent_variables"] = []
                 table_contents["independent_variables"] = []
 
-            table_contents["load_fail"] = load_fail
             table_contents["name"] = datasub_record.name
             table_contents["title"] = datasub_record.description
             table_contents["keywords"] = datasub_record.keywords