Skip to content

[#34] VistA Functions: handle SQL NULLs (missed in previous commit)

Sam Habiel requested to merge shabiel/YDBOctoVistA:ddl34 into master

A couple of things missed in the previous commit:

  • Octo sends EMPTY strings from Octo if specified as literals as SQL NULLs. That means that REPLACE('AB','B','') did not work because the third parameter was not an empty string as expected by the M function.
  • To be safe, FMGET (which is always supposed to be used with literals for the first and second arguments) will also protect itself if the first two arguments are passed as empty strings.
  • Ditto for second argument of DATETIME.
  • The rest of the functions were reviewed and no changes were thought to be currently needed.
  • In addition, DATETIME function is supposed to accept one or two arguements. This was missed in the last commit, and a one argument form has been created; and tests now added to TVD0011 and TVD0012 tests in the Octo repo.
Edited by Sam Habiel

Merge request reports