Skip to content

Commit 23f6f66

Browse files
committed
competency: we need to use column_id when setting default values.
1 parent 299e8c3 commit 23f6f66

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

mapserver/competency/definition.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -269,11 +269,11 @@ def make_sql(self, request: QueryRequest) -> tuple[str, list[str]]:
269269
conditions[param_def.condition].append(where_condition)
270270
used_columns.append(column_id)
271271
# NB. req_params might/will not have entries for default params
272-
for column, param_def in self.__parameters.items():
273-
if column not in used_columns:
272+
for column_id, param_def in self.__parameters.items():
273+
if column_id not in used_columns:
274274
if not param_def.optional:
275-
raise ValueError(f'Required parameter must have a value: {column}')
276-
where_condition = f'{column} = ({param_def.default_sql})'
275+
raise ValueError(f'Required parameter must have a value: {column_id}')
276+
where_condition = f'{param_def.column} = ({param_def.default_sql})'
277277
conditions[param_def.condition].append(where_condition)
278278
sql = self.__sql_defn.sql
279279
for condition, expressions in conditions.items():

0 commit comments

Comments
 (0)