dce-reactkit 3.2.2-beta.23 → 3.2.2-beta.25

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dce-reactkit",
-  "version": "3.2.2-beta.23",
+  "version": "3.2.2-beta.25",
   "description": "Shared components for Harvard DCE apps",
   "main": "dist/cjs/index.js",
   "module": "dist/esm/index.js",

package/src/components/LogReviewer.tsx

@@ -524,9 +524,10 @@ const LogReviewer: React.FC<Props> = (props) => {
   // > Add "uncategorized" subcontext to each context
   Object.keys(LogMetadata.Context ?? {}).forEach((context) => {
     if (
+      // Context exists
       LogMetadata.Context
       // Context has children already
-      && typeof LogMetadata.Context[context] !== 'string'
+      && LogMetadata.Context[context] !== 'string'
     ) {
       (LogMetadata.Context as any)[context][LogBuiltInMetadata.Context.Uncategorized] = (
         LogBuiltInMetadata.Context.Uncategorized
@@ -547,6 +548,7 @@ const LogReviewer: React.FC<Props> = (props) => {
       LogMetadata.Target[target] = target;
     }
   });
+  console.log(LogMetadata);
 
   /* -------------- State ------------- */
 
@@ -578,8 +580,10 @@ const LogReviewer: React.FC<Props> = (props) => {
       // Case: subcontexts exist
       initContextFilterState[contextValue._] = {};
       Object.values((LogMetadata.Context ?? {})[context]).forEach((subcontext) => {
-        const subcontextValue = contextValue[subcontext];
-        (initContextFilterState[contextValue._] as { [k: string]: boolean })[subcontextValue] = true;
+        if (subcontext) {
+          const subcontextValue = contextValue[subcontext];
+          (initContextFilterState[contextValue._] as { [k: string]: boolean })[subcontextValue] = true;
+        }
       });
     }
   });
@@ -990,14 +994,14 @@ const LogReviewer: React.FC<Props> = (props) => {
               const children: PickableItem[] = (
                 Object.keys(value)
                   .filter((subcontext) => {
-                    return subcontext !== '_'
+                    return subcontext !== '_';
                   })
                   .map((subcontext) => {
                     return {
                       id: subcontext,
                       name: genHumanReadableName(subcontext),
                       isGroup: false,
-                      checked: !!value[subcontext],
+                      checked: (contextFilterState[context] as any)[subcontext],
                     };
                   })
               );
@@ -1017,22 +1021,17 @@ const LogReviewer: React.FC<Props> = (props) => {
             title="Context"
             items={pickableItems}
             onChanged={(updatedItems) => {
-              console.log('Items before:', pickableItems);
-              console.log('Updated Items:', updatedItems);
               // Update our state
               updatedItems.forEach((pickableItem) => {
                 if (pickableItem.isGroup) {
                   // Has subcontexts
                   pickableItem.children.forEach((subcontextItem) => {
                     if (!subcontextItem.isGroup) {
-                      console.log('Update child:', pickableItem.id, subcontextItem.id, subcontextItem.checked);
-                      console.log(contextFilterState);
                       (
                         contextFilterState[pickableItem.id] as { [k: string]: boolean }
                       )[subcontextItem.id] = (
                         subcontextItem.checked
                       );
-                      console.log(contextFilterState);
                     }
                   });
                 } else {
@@ -1042,6 +1041,7 @@ const LogReviewer: React.FC<Props> = (props) => {
                   );
                 }
               });
+              console.log('New context filter state:', contextFilterState);
               dispatch({
                 type: ActionType.UpdateContextFilterState,
                 contextFilterState,