Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BUG: Unable to import CCDA XML files in Docker 7.0.3 #7411

Closed
see-k-er opened this issue Apr 27, 2024 · 7 comments · Fixed by #7416
Closed

BUG: Unable to import CCDA XML files in Docker 7.0.3 #7411

see-k-er opened this issue Apr 27, 2024 · 7 comments · Fixed by #7416

Comments

@see-k-er
Copy link

Describe the bug

Import of CCDA XML file to OpenEMR is unsuccessful in Docker 7.0.3. The following error is seen in the error.log file.

PHP Warning: Undefined array key 1 in /var/www/localhost/htdocs/openemr/controllers/C_Document.class.php on line 806, referer: http://localhost:8009/interface/modules/zend_modules/public/carecoordination/upload?id=1

To Reproduce

image

Steps to reproduce the behavior:

  1. Go to Modules, then CareCordination
  2. Go to Import and then CCDA or QRDA
  3. Click on the import button and choose a CCDA file to import
  4. Page reloads with no new import

Expected behavior

In Docker 7.0.2, the import of CCDA file is successful and it's entry can be seen in the Import CCDA page. Similar behaviour is expected in Docker 7.0.3.

Client configuration

  • Browser: Chrome
  • OpenEMR version: 7.0.3
  • Operating system: Ubuntu 22.04
@see-k-er see-k-er changed the title Unable to import CCDA XML files in Docker 7.0.3 BUG: Unable to import CCDA XML files in Docker 7.0.3 Apr 30, 2024
@stephenwaite
Copy link
Sponsor Member

hi @see-k-er , do you have a sample ccda without PHI that you could share for testing purposes please?

@see-k-er
Copy link
Author

see-k-er commented May 1, 2024

The attached file shared contains synthea generated CCDA XML files (in zip format as GitHub didn't let me attach XML files).
ccda_xml_synthea.zip

@stephenwaite
Copy link
Sponsor Member

stephenwaite commented May 1, 2024

hi @sjpadgett , what happens when there's only 1 component count in the zip?

https://github.com/openemr/openemr/blob/master/interface/modules/zend_modules/module/Carecoordination/src/Carecoordination/Controller/CarecoordinationController.php#L1018

it appears that the import won't proceed if documents aren't included...

@sjpadgett
Copy link
Sponsor Member

The document referred is the ccda xml to import. I'll take a quick look at sample xml.

@sjpadgett
Copy link
Sponsor Member

We don't like the zip for some reason but imports fine using unzipped

image

@sjpadgett
Copy link
Sponsor Member

Looks like zip of ccdas need to be in a directory. I think I did this because all the zips I've used have directory even ONC.
However the reason I put path component count was to look for directories so did I forget or was this refactored at some point!
I'll see what I can do to fix but tell user if he wants to continue testing give a directory of ccdas.

@stephenwaite
Copy link
Sponsor Member

hi @sjpadgett, here's some minor stuff I fixed while debugging, #7416

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 a pull request may close this issue.

3 participants