smart-on-fhir / cumulus-library

https://docs.smarthealthit.org/cumulus/library/
Apache License 2.0
2 stars 0 forks source link

Removed extra indexing on schema removal #119

Closed dogversioning closed 1 year ago

dogversioning commented 1 year ago

This PR removes a second instance of array indexing in the base_table_builder, which was causing us to search only the first character of a potential string for a name like "schema"."table"

This is currently only an issue when we are creating a completely empty table.

Checklist

dogversioning commented 1 year ago

Since this is affecting sites right now, I'm electing to not write a unit test this second, but am tracking the need for catching this case at https://github.com/smart-on-fhir/cumulus-library/issues/120