Skip to content

Commit d794325

Browse files
committed
Fix the issue where the primary key column(s) were not the first columns in the table. This broke the generation of the signature of delete APIs, causing a leading comma on the first parameter specification.
1 parent 6985d23 commit d794325

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

src/model/tapi_generator.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -920,13 +920,15 @@ def _delete_api_sig(self,
920920
signature += f'{STAB}procedure {procedure_name}\n'
921921
signature += f'{STAB}(\n'
922922
table_name_lc = self.table.table_name.lower()
923+
predicate_num = 0
923924
for col_position, column_name in enumerate(self.table.columns_list, start = 1):
924925
column_name_lc = column_name.lower()
925926
# if not column_name in self.table.pk_columns_list and column_name_lc != self.table.row_vers_column_name:
926927
if column_name not in self.table.in_out_column_list and column_name not in self.table.out_column_list:
927928
continue
928-
929-
leader = f', ' if col_position > 1 else f' '
929+
predicate_num += 1
930+
leader = f', ' if predicate_num > 1 else f' '
931+
print(f'DEBUG: predicate_num = {predicate_num}')
930932
param = f'{STAB}{STAB}{leader}p_{column_name_lc.ljust(self.table.max_col_name_len + self.indent_spaces, " ")}'
931933
if column_name_lc in self.table.in_out_column_list_lc:
932934
in_out = f'{STAB}in out'

0 commit comments

Comments
 (0)