Skip to content

HCK-18368: fix boolean false in separate column DEFAULT statements - #271

Merged
taras-dubyk merged 3 commits into
developfrom
fix/boolean-default-false-separate-statement
Oct 30, 2026
Merged

taras-dubyk merged 3 commits into
developfrom
fix/boolean-default-false-separate-statement

Conversation

@Sylvie-P

@Sylvie-P Sylvie-P commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor
Sub-bugHCK-18368 [Oracle] FE omits boolean default value false in separate ALTER statements

## Summary - Fix `assignTemplates` so boolean `false` is not treated as a missing placeholder when generating separate column DEFAULT statements. - [HCK-18368](https://hackolade.atlassian.net/browse/HCK-18368) (parent [HCK-18361](https://hackolade.atlassian.net/browse/HCK-18361)). - Same change as [hackolade/PostgreSQL#208](https://github.com/hackolade/PostgreSQL/pull/208).

Test plan

  • Boolean default false, separate statements
  • Inline MODIFY default false (may need follow-up if still missing)

Made with Cursor

Sylvie-P and others added 2 commits October 29, 2026 18:03
assignTemplates treated false as missing when filling placeholders, which broke SET DEFAULT false in separate FE statements.

Co-authored-by: Cursor <cursoragent@cursor.com>
getColumnDefault skipped false because of a truthy check; extend the same handling as assignTemplates.

Co-authored-by: Cursor <cursoragent@cursor.com>
@Sylvie-P
Sylvie-P requested a review from taras-dubyk October 29, 2026 17:18
Co-authored-by: Taras Dubyk <taras.dubyk@binary-studio.com>
@sonarqubecloud

Copy link
Copy Markdown

@Sylvie-P
Sylvie-P requested a review from taras-dubyk October 30, 2026 08:36
@taras-dubyk
taras-dubyk enabled auto-merge (squash) October 30, 2026 08:36
@taras-dubyk
taras-dubyk merged commit d32ae47 into develop Oct 30, 2026
8 checks passed
@taras-dubyk
taras-dubyk deleted the fix/boolean-default-false-separate-statement branch October 30, 2026 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants