[superset] branch master updated: fix: safe removal of tab with filters still scoped to a non-existing chart (#15650)

[rusackas]

This is an automated email from the ASF dual-hosted git repository.

rusackas pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/superset.git


The following commit(s) were added to refs/heads/master by this push:
     new 52ad779  fix: safe removal of tab with filters still scoped to a 
non-existing chart (#15650)
52ad779 is described below

commit 52ad779a2705c828bab3fbf38aa8b16a21a74d00
Author: Evan Rusackas <e...@preset.io>
AuthorDate: Mon Jul 12 23:43:53 2021 -0600

    fix: safe removal of tab with filters still scoped to a non-existing chart 
(#15650)
---
 superset-frontend/src/dashboard/util/activeAllDashboardFilters.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/superset-frontend/src/dashboard/util/activeAllDashboardFilters.ts 
b/superset-frontend/src/dashboard/util/activeAllDashboardFilters.ts
index 2243c25..7b9dc10 100644
--- a/superset-frontend/src/dashboard/util/activeAllDashboardFilters.ts
+++ b/superset-frontend/src/dashboard/util/activeAllDashboardFilters.ts
@@ -104,7 +104,7 @@ export const getAllActiveFilters = ({
       };
     // Iterate over all roots to find all affected charts
     scope.rootPath.forEach((layoutItemId: string | number) => {
-      layout[layoutItemId].children.forEach((child: string) => {
+      layout[layoutItemId]?.children?.forEach((child: string) => {
         // Need exclude from affected charts, charts that located in scope 
`excluded`
         findAffectedCharts({
           child,