Currently, a provider needs to be referenced within a nested <code class="notranslate"

CCR import: Retrieving Providers about pophealth HOT 3 CLOSED

pophealth commented on July 3, 2024

CCR import: Retrieving Providers

from pophealth.

Comments (3)

ssayer commented on July 3, 2024

Thanks for working the XPath out for me :). This issue is actually in health data standards, but I've updated that library to look for NPI instead of "care provider."

from pophealth.

kclaes commented on July 3, 2024

Provider import works if all attributes are available.
However, the extract_data method in provider_import_utils.rb does not cope well with nil because it accesses content without checking the value of the at_xpath method.
Also, in provider_importer.rb, we have the following code:

          if actor.at_xpath('./ccr:Person/ccr:Name/ccr:CurrentName/ccr:Given')
            provider[:given_name] = actor.at_xpath('./ccr:Person/ccr:Name/ccr:CurrentName/ccr:Given').content
            provider[:family_name] = actor.at_xpath('./ccr:Person/ccr:Name/ccr:CurrentName/ccr:Family').content
          end

The nil-check is done for CurrentName/Given, but not for Family -> potential error?

If you fix the extract_data function to cope with nils, I guess it can just become

            provider[:given_name] = extract_data(actor, './ccr:Person/ccr:Name/ccr:CurrentName/ccr:Given')
            provider[:family_name] = extract_data(actor, './ccr:Person/ccr:Name/ccr:CurrentName/ccr:Family')
            provider[:specialty] = extract_data(actor, './ccr:Specialty/ccr:Text')

I changed the definition of extract_data to this

  # Returns nil if result is an empty string, block allows text munging of result if there is one
  def extract_data(subject, query)
    result = subject.at_xpath(query)
    if !result || result.content == ""
      nil
    else
      result
    end
  end