pmd icon indicating copy to clipboard operation
pmd copied to clipboard

[plsql] Fixed issue with missing optional table alias in MERGE usage

Open duursma opened this issue 1 year ago • 1 comments

Describe the PR

The newly introduced MERGE support didn't handle the case in which the table alias was missing as the next keyword was accepted as being the alias.

Related issues

Test-cases did not have an example of this variant.

  • Fixes # Added a LOOKAHEAD checking whether the found alias is not the expect keyword following the optional alias. And added test-code which would break in the original version.

Ready?

  • [X] Added unit tests for fixed bug/feature
  • [X] Passing all unit tests
  • [X] Complete build ./mvnw clean verify passes (checked automatically by github actions)
  • [X] Added (in-code) documentation (if needed)

duursma avatar Jun 27 '24 11:06 duursma

1 Message
:book: Compared to master:
This changeset changes 0 violations,
introduces 10 new violations, 0 new errors and 0 new configuration errors,
removes 20 violations, 0 errors and 0 configuration errors.
Download full report as build artifact

Generated by :no_entry_sign: Danger

ghost avatar Jun 28 '24 15:06 ghost