Skip to content
Snippets Groups Projects
Commit 7db90031 authored by Tom Robinson's avatar Tom Robinson
Browse files

Fix: Lat/Lon columns no longer automatically detected for pin maps

parent e8f05a25
Branches
Tags
No related merge requests found
......@@ -114,6 +114,8 @@ export const isNumericBaseType = (field) => isa(field && field.base_type, TYPE.N
export const isNumber = (field) => (field.special_type == null || field.special_type === TYPE.Number);
export const isCoordinate = (field) => isa(field && field.special_type, TYPE.Coordinate);
export const isLatitude = (field) => isa(field && field.special_type, TYPE.Latitude);
export const isLongitude = (field) => isa(field && field.special_type, TYPE.Longitude);
// operator argument constructors:
......
......@@ -9,7 +9,7 @@ import {
DIMENSION_METRIC_METRIC
} from "metabase/visualizations/lib/utils";
import { isNumeric, isDate, isMetric, isDimension, hasLatitudeAndLongitudeColumns } from "metabase/lib/schema_metadata";
import { isNumeric, isDate, isMetric, isDimension, isLatitude, isLongitude, hasLatitudeAndLongitudeColumns } from "metabase/lib/schema_metadata";
import Query from "metabase/lib/query";
import { capitalize } from "metabase/lib/formatting";
......@@ -564,7 +564,7 @@ const SETTINGS = {
title: "Latitude field",
widget: ChartSettingSelect,
getDefault: ([{ card, data: { cols }}]) =>
(_.findWhere(cols, { special_type: "latitude" }) || {}).name,
(_.find(cols, isLatitude) || {}).name,
getProps: ([{ card, data: { cols }}]) => ({
options: cols.filter(isNumeric).map(getOptionFromColumn)
}),
......@@ -574,7 +574,7 @@ const SETTINGS = {
title: "Longitude field",
widget: ChartSettingSelect,
getDefault: ([{ card, data: { cols }}]) =>
(_.findWhere(cols, { special_type: "longitude" }) || {}).name,
(_.find(cols, isLongitude) || {}).name,
getProps: ([{ card, data: { cols }}]) => ({
options: cols.filter(isNumeric).map(getOptionFromColumn)
}),
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment