Removing a custom column used in the grouping crashes the query builder

[ngapuileung]

Describe the bug

The query builder crashes when I remove a custom column from an existing question. That custom column is neither displayed nor used for sorting in the output/results but is used in the grouping.

To Reproduce

1. Create a new question using the Orders sample table
2. Add a custom column called "Is Promotion": case([Discount] > 0, 1, 0)
3. Group by the custom column and count the number of distinct order IDs
4. Remove the custom column
5. Query builder crashes

Expected behavior

The custom column should just be dropped and the question should not group by the removed column anymore.

Logs

Information about your Metabase installation

{
  "browser-info": {
    "language": "en-GB",
    "platform": "MacIntel",
    "userAgent": "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/114.0.0.0 Safari/537.36",
    "vendor": "Google Inc."
  },
  "system-info": {
    "file.encoding": "UTF-8",
    "java.runtime.name": "OpenJDK Runtime Environment",
    "java.runtime.version": "11.0.19+7",
    "java.vendor": "Eclipse Adoptium",
    "java.vendor.url": "https://adoptium.net/",
    "java.version": "11.0.19",
    "java.vm.name": "OpenJDK 64-Bit Server VM",
    "java.vm.version": "11.0.19+7",
    "os.name": "Linux",
    "os.version": "5.10.184-175.731.amzn2.x86_64",
    "user.language": "en",
    "user.timezone": "GMT"
  },
  "metabase-info": {
    "databases": [
      "druid",
      "redshift",
      "mysql",
      "bigquery-cloud-sdk",
      "postgres",
      "mongo",
      "h2"
    ],
    "hosting-env": "unknown",
    "application-database": "postgres",
    "application-database-details": {
      "database": {
        "name": "PostgreSQL",
        "version": "14.7"
      },
      "jdbc-driver": {
        "name": "PostgreSQL JDBC Driver",
        "version": "42.5.4"
      }
    },
    "run-mode": "prod",
    "version": {
      "date": "2023-07-25",
      "tag": "vUNKNOWN",
      "branch": "master",
      "hash": "14a24c5"
    },
    "settings": {
      "report-timezone": "US/Pacific"
    }
  }
}

Severity

Annoying but there is a workaround by removing the grouping first

Additional context

No response

[darksciencebase]

Now I'm seeing this on stats. QB did not crash after I removed the custom column, so I clicked 'Visualize' and got an error that is talking about the custom column which is not there anymore (I used a custom column =1+1 named 2.
@ngapuileung could you please also check if it works for you?