fixing behavour for Table Keys report

Author: ddooley

lib/AppContext.js

@@ -74,29 +74,35 @@ export default class AppContext {
     let dh = this.getCurrentDataHarmonizer();
     let class_name = dh.template_name;
     let report_type = $('#report_select_type').val();
+    const class_dh = this.dhs['Class'];
+    const class_row = class_dh?.hot.getSelected()?.[0]?.[0] ?? -1; // -1 = undefined
 
     //console.log("REFRESHTABDISPLAY", class_name, this.schemaEditor, report_type)
 
     // Display Focus path only if there is some path > 1 element.
     $("#record-hierarchy-div").toggle(Object.keys(this.dhs).length > 1);
 
-    // Record hierarchy only shown for "Records by 1-many key"
+    // Record hierarchy cookie crumbs on Display line at top of toolbar are 
+    // only shown for "Records by 1-many key"
     $('#record-hierarchy').toggle(report_type === "");
 
+    // A Display menu item  which is only shown with Schema Editor interface
+    // if user is on Slot tab.
+    //$('#slot_report_type, #slot_report_type_option')
+    //  .toggle(this.schemaEditor !== undefined && class_name === 'Slot');
+
     // The Schema tab always displays a full list, even with "record(s) by selected key" option set in menu.
     if (this.schemaEditor !== undefined && class_name === 'Schema') {
       // Report type override
       report_type = 'all';
     }
 
-    // A Schema Editor interface item that is visible only if Slot tab is showing.
-    $('#slot_report_type, #slot_report_type option')
-      .toggle(this.schemaEditor !== undefined && class_name === 'Slot');
-
     // Save, deselect, and then set cursor, Otherwise selected cell gets
     // set to same column but first row after filter!!!
+    /*
     let cursor = dh.hot.getSelected();
     dh.hot.deselectCell(); 
+    */
 
     // Schema editor SCHEMA tab should never be filtered.
     // ONLY DEPENDENT TABS/TABLES SHOULD BE FILTERED.
@@ -108,6 +114,7 @@ export default class AppContext {
         this.slotTypeFilter(dh); // Done via hidden fields. no constraint (no keys)
         break;
 
+      // The following Display > menu is specific to the Field / Slots display
       case 'slot':
         //this.slotTypeFilter(dh, ['slot']);   
         //dh.hot.getPlugin('multiColumnSorting').sort() ; //{ column: 3, sortOrder: 'asc' }
@@ -135,10 +142,24 @@ export default class AppContext {
       // Records by selected key case
       case '': 
       default:
-        let dependent_report = this.dependent_rows.get(class_name);
-        this.tabFilter(dh, dependent_report.fkey_vals); // filter by selected keys.
+
+        let fkey_vals = this.dependent_rows.get(class_name).fkey_vals;
+        // Some reports should show all class's entries if none selected in Class (Table) tab.
+
+        if (this.schemaEditor !== undefined && dh.template_name === 'UniqueKey') {
+          // If no row is selected in the Class (Table) tab, remove the class_id
+          // filter so all unique keys for the selected schema are shown.
+          alert(class_row)
+          if (class_row < 0) {
+            delete fkey_vals['class_id'];
+            //alert("here")
+          }
+        }
+
+        this.tabFilter(dh, fkey_vals); // filter by selected keys.
     }
 
+    /*
     // DON'T RESTORE CURSOR UNTIL WE KNOW THAT IT IS POINTING TO SAME KEY AS BEFORE FILTER?
     // Refreshes dependent record list.
     if (cursor) {
@@ -150,6 +171,7 @@ export default class AppContext {
       // Note this will cause a "double event" of selecting same cell twice.
 
     }
+    */
 
     //columnsorting cannot be disabled. If it is, below code will error.
     const columnSorting = dh.hot.getPlugin('multiColumnSorting');
@@ -1235,7 +1257,7 @@ export default class AppContext {
               // And dependent slots within this class via other's foreign keys
               // Only one dependent can depend on a slot. 
               relations[class_name].dependent_slots[slot_name] = {
-                foreign_class: foreign_class, 
+                foreign_class: foreign_class,
                 foreign_slot: foreign_slot_name
               };