fix: ensure public is in sync with isVisible when using DAP

bsod90 · bsod90 · commit 641770274624 · 2024-11-21T16:18:32.000-08:00
diff --git a/packages/cubejs-server-core/src/core/CompilerApi.js b/packages/cubejs-server-core/src/core/CompilerApi.js
@@ -473,7 +473,8 @@ export class CompilerApi {
       }
       return (item) => ({
         ...item,
-        isVisible: item.isVisible && isMemberVisibleInContext[item.name]
+        isVisible: item.isVisible && isMemberVisibleInContext[item.name],
+        public: item.public && isMemberVisibleInContext[item.name]
       });
     };
 

Original file line number	Diff line number	Diff line change
`@@ -473,7 +473,8 @@ export class CompilerApi {`
`473`	`473`	`}`
`474`	`474`	`return (item) => ({`
`475`	`475`	`...item,`
`476`		`- isVisible: item.isVisible && isMemberVisibleInContext[item.name]`
	`476`	`+ isVisible: item.isVisible && isMemberVisibleInContext[item.name],`
	`477`	`+ public: item.public && isMemberVisibleInContext[item.name]`
`477`	`478`	`});`
`478`	`479`	`};`
`479`	`480`