Merge pull request #38 from bimberlabinternal/24.7_fb_merge

Merge discvr-24.3 to discvr-24.7

Author: bbimber

laboratory/src/org/labkey/laboratory/ExtraDataSourcesUserSchema.java

@@ -53,8 +53,7 @@ public TableInfo createTable(String name, ContainerFilter cf)
             {
                 if (!getContainer().isWorkbook() || source.isImportIntoWorkbooks())
                 {
-                    // TODO: QueryDefinition.getTable doesn't yet have ContainerFilter option
-                    TableInfo ti = source.getTableInfo(getContainer(), getUser());
+                    TableInfo ti = createWrappedTable(name, source.getTableInfo(getContainer(), getUser()), cf);
                     new WrappingTableCustomizer().customize(ti);
 
                     return ti;
@@ -65,6 +64,14 @@ public TableInfo createTable(String name, ContainerFilter cf)
         return null;
     }
 
+    @Override
+    protected void afterConstruct(TableInfo info)
+    {
+        // No-op to avoid double-adding query metadata. Rely on the source table for this.
+        // This bug appears in the schema browser in production mode only. If there is a container with any extra data sources present, the table details
+        // page will asynchronously load an 'Error: null' message, coming from the query analyzer
+    }
+
     @Override
     public DbSchema getDbSchema()
     {