(superset) branch master updated: fix(drill-to-detail): drill to detail by correctly filtering by metric (#39766)

[mi...@apache.org]

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

michael-s-molina 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 df396aa6e9d fix(drill-to-detail): drill to detail by correctly filtering by metric (#39766)
df396aa6e9d is described below

commit df396aa6e9d6614c33d432c072ae1920475d33bc
Author: Luiz Otavio <45...@users.noreply.github.com>
AuthorDate: Thu Apr 30 08:40:16 2026 -0300

    fix(drill-to-detail): drill to detail by correctly filtering by metric (#39766)
    
    Co-authored-by: Michael S. Molina <mi...@gmail.com>
---
 superset/models/helpers.py              |  8 ++++++
 tests/unit_tests/models/helpers_test.py | 46 +++++++++++++++++++++++++++++++++
 2 files changed, 54 insertions(+)

diff --git a/superset/models/helpers.py b/superset/models/helpers.py
index c48cba355aa..89d3fb8b219 100644
--- a/superset/models/helpers.py
+++ b/superset/models/helpers.py
@@ -3027,6 +3027,14 @@ class ExploreMixin:  # pylint: disable=too-many-public-methods
                 col_obj = columns_by_name.get(cast(str, flt_col))
                 # If not found in columns, check if it's a metric
                 # This supports filtering on metric columns for any chart type
+                if col_obj is None:
+                    # Fall back to verbose_name so filters produced by
+                    # right-click "Drill to detail by" (which can pass a
+                    # column's verbose label) still resolve to a real column.
+                    col_obj = next(
+                        (c for c in self.columns if c.verbose_name == flt_col),
+                        None,
+                    )
                 if (
                     col_obj is None
                     and isinstance(flt_col, str)
diff --git a/tests/unit_tests/models/helpers_test.py b/tests/unit_tests/models/helpers_test.py
index 62505a8d759..8973423f968 100644
--- a/tests/unit_tests/models/helpers_test.py
+++ b/tests/unit_tests/models/helpers_test.py
@@ -2256,3 +2256,49 @@ def test_numeric_adhoc_filter_value_is_unquoted_in_where_clause(
     # The numeric value should be emitted without quotes.
     assert "IN (3)" in sql, f"Expected numeric IN-list, got SQL: {sql}"
     assert "IN ('3')" not in sql, f"Value should not be quoted, got SQL: {sql}"
+
+
+def test_filter_by_verbose_name_resolves_to_column(
+    database: Database,
+) -> None:
+    """
+    A filter whose "col" value matches a column's verbose_name
+    (e.g. the label emitted by "Drill to detail by") must resolve to that
+    column and produce a WHERE clause on the underlying column_name.
+    """
+    from superset.connectors.sqla.models import SqlaTable, TableColumn
+
+    table = SqlaTable(
+        database=database,
+        schema=None,
+        table_name="t",
+        columns=[
+            TableColumn(
+                column_name="country_code",
+                verbose_name="Country",
+                type="TEXT",
+            ),
+            TableColumn(column_name="b", type="TEXT"),
+        ],
+    )
+
+    sqla_query = table.get_sqla_query(
+        columns=["b"],
+        # Filter uses the verbose label, as "Drill to detail by" does.
+        filter=[{"col": "Country", "op": "==", "val": "US"}],
+        is_timeseries=False,
+        row_limit=10,
+    )
+
+    with database.get_sqla_engine() as engine:
+        sql = str(
+            sqla_query.sqla_query.compile(
+                dialect=engine.dialect,
+                compile_kwargs={"literal_binds": True},
+            )
+        )
+
+    # The filter should be translated to a WHERE clause on the real column.
+    assert "WHERE" in sql, f"Expected WHERE clause, got SQL: {sql}"
+    assert "country_code" in sql, f"Expected filter on 'country_code', got SQL: {sql}"
+    assert "'US'" in sql, f"Expected filter value 'US', got SQL: {sql}"