Dorn Changes #7221

claimrevolution · 2024-02-11T21:18:38Z

Fixes #

Short description of what this resolves:

These are the changes for DORN.

Changes proposed in this pull request:

if more than one come back on a check. Increased table column size too, truncation was happening

sjpadgett

Module listeners for claims and DORN.

sjpadgett · 2024-04-08T17:33:28Z

interface/modules/custom_modules/oe-module-claimrev-connect/ModuleManagerListener.php

+/*
+    $classLoader = new \OpenEMR\Core\ModulesClassLoader($GLOBALS['fileroot']);
+    $classLoader->registerNamespaceIfNotExists("OpenEMR\\Modules\\ClaimRevConnector\\", __DIR__ . DIRECTORY_SEPARATOR . 'src');
+*/
+
+use OpenEMR\Core\AbstractModuleActionListener;
+use OpenEMR\Modules\WenoModule\Services\ModuleService;
+
+/* Allows maintenance of background tasks depending on Module Manager action. */
+
+class ModuleManagerListener extends AbstractModuleActionListener


@claimrevolution
I added this to your claim module which I setup to handle background service tasks.
It tested well for me!
Recommend you review and test!

sjpadgett · 2024-04-08T17:39:07Z

interface/modules/custom_modules/oe-module-dorn/ModuleManagerListener.php

+/*
+    $classLoader = new \OpenEMR\Core\ModulesClassLoader($GLOBALS['fileroot']);
+    $classLoader->registerNamespaceIfNotExists("OpenEMR\\Modules\\Dorn\\", __DIR__ . DIRECTORY_SEPARATOR . 'src');
+*/
+
+use OpenEMR\Core\AbstractModuleActionListener;
+use OpenEMR\Modules\WenoModule\Services\ModuleService;
+
+/* Allows maintenance of background tasks depending on Module Manager action. */
+
+class ModuleManagerListener extends AbstractModuleActionListener


Since no background services don't really need but has some useful functions you may want to implement to help manage module Delete if you want.

sjpadgett · 2024-04-08T17:44:34Z

@claimrevolution If you want me to test then I need all the current test credentials for provider and api!
I'm getting ready to move on! This PR is close to being ready for merge...

bradymiller · 2024-04-09T08:00:15Z

interface/modules/custom_modules/oe-module-dorn/src/LabCompendiumInstall.php

+                }
+            }
+            ConnectorApi::setCompendiumLastUpdate($labGuid);
+            echo "Compendium has been updated for lab: " . $compendiumResponse->compendium->labName;


text() on the variable above

bradymiller · 2024-04-09T08:00:24Z

interface/modules/custom_modules/oe-module-dorn/src/LabCompendiumInstall.php

+            ConnectorApi::setCompendiumLastUpdate($labGuid);
+            echo "Compendium has been updated for lab: " . $compendiumResponse->compendium->labName;
+        } else {
+            echo "Error Getting Compendium! " . $compendiumResponse->responseMessage;


text() on the variable above

bradymiller · 2024-04-09T08:13:02Z

review complete :)

sjpadgett · 2024-04-09T14:11:51Z

lol I can look at a script a hundred times and still miss an escape!
I do fine if I write it. This is why I don't often review others work, I'm horrible at it!
Thanks for the review. Now testing

Fix js error for incomplete dialog title

sjpadgett · 2024-04-12T21:00:44Z

@claimrevolution After setting up route I was able to download compendium. I didn't have an account so I used the lab guid.
So on transmit order I received an error of No account, lab guid etc.
Is their an account number I can use?

I really like how you auto setup Provider, easy stuff! Still for hl7 orders you'll need a couple params in provider provided.
I'm going to try to find a couple test my clinic can use so we may be able to use it for DORN cert. I'll send you a list and we can shop a lab.
If you'll allow me I'd like to duplicate your setup in module or at least collect credentials so perhaps you can test if proper settings are in place before allow the module to start up.
I noted that routes are in a table they will still persist in compendium search when deleted in DB. Maybe you are maintaining elsewhere.
Noted too that while setting D for providers mode in globals the provider is created with P.
echoing to dialog text for status if you don't format to take into account themes user will miss message i.e dark theme
Many of these minor things I can take care of.

So I think all I need is lab account for Autolab or maybe not.

I'm getting excited to see this project work.

claimrevolution · 2024-04-12T21:03:39Z

I'm getting the same error. I reached out to dorn to see what the deal is. As far as the account number, it doesn't matter in testing. I use a random key on the keyboard like "asdf3". Anyway, I got the same error you are and I emailed them the other day and now just waiting. I checked routes and account numbers in the route api and everything looked fine. This happened before and they did something and it started working again. Brad Sharp CTO Cell: 918-943-0020 Office: 918-842-9564 Bixby, OK www.claimrev.com<http://www.claimrev.com/> Book a Demo<https://www.claimrev.com/schedule-demo/>

…

________________________________ From: Jerry Padgett ***@***.***> Sent: Friday, April 12, 2024 4:01 PM To: openemr/openemr ***@***.***> Cc: Brad Sharp ***@***.***>; Mention ***@***.***> Subject: Re: [openemr/openemr] Dorn Changes (PR #7221) @claimrevolution<https://github.com/claimrevolution> After setting up route I was able to download compendium. I didn't have an account so I used the lab guid. So on transmit order I received an error of No account, lab guid etc. Is their an account number I can use? * I really like how you auto setup Provider, easy stuff! Still for hl7 orders you'll need a couple params in provider provided. * I'm going to try to find a couple test my clinic can use so we may be able to use it for DORN cert. I'll send you a list and we can shop a lab. * If you'll allow me I'd like to duplicate your setup in module or at least collect credentials so perhaps you can test if proper settings are in place before allow the module to start up. * I noted that routes are in a table they will still persist in compendium search when deleted in DB. Maybe you are maintaining elsewhere. * Noted too that while setting D for providers mode in globals the provider is created with P. * echoing to dialog text for status if you don't format to take into account themes user will miss message i.e dark theme Many of these minor things I can take care of. So I think all I need is lab account for Autolab or maybe not. I'm getting excited to see this project work. — Reply to this email directly, view it on GitHub<#7221 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AJA7JDEXOSJKADDDN64INUTY5BDRDAVCNFSM6AAAAABDD2PE4WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJSGUZDKNJWGI>. You are receiving this because you were mentioned.Message ID: ***@***.***> ALERT: EMAIL EXTERNAL TO ORGANIZATION - VERIFY BEFORE ACTING

sjpadgett · 2024-04-17T01:02:44Z

@claimrevolution
I'm tired and won't be working tonight but I've scheduled all day tomorrow to work this. I reviewed HL7 and looks okay to me so far.
Do you have some TODO's I can help you with? Just list them here or email.
Otherwise I'll probably be doings UX and stylings. Maybe some error reporting.
Let me know!

claimrevolution · 2024-04-17T15:10:32Z

I've added better error messages, to the return of the labs. also for this test your account number needs to be: 1232e5When the route is setup the provider is updated, autolab in this case. The Sending Facility Name isn't populated and should be. It will give an error saying the MSH4 is empty. Finally I'm geting an error message now that the account number doesn't match what is in the API. I've done a check and validated the route in DORN and the account number is 1232e5 but it doesn't seem to recognize it. Emailing DORN now. Brad Sharp CTO Cell: 918-943-0020 Office: 918-842-9564 Bixby, OK www.claimrev.com<http://www.claimrev.com/> Book a Demo<https://www.claimrev.com/schedule-demo/>

…

________________________________ From: Jerry Padgett ***@***.***> Sent: Tuesday, April 16, 2024 8:03 PM To: openemr/openemr ***@***.***> Cc: Brad Sharp ***@***.***>; Mention ***@***.***> Subject: Re: [openemr/openemr] Dorn Changes (PR #7221) @claimrevolution<https://github.com/claimrevolution> I'm tired and won't be working tonight but I've scheduled all day tomorrow to work this. I reviewed HL7 and looks okay to me so far. Do you have some TODO's I can help you with? Just list them here or email. Otherwise I'll probably be doings UX and stylings. Maybe some error reporting. Let me know! — Reply to this email directly, view it on GitHub<#7221 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AJA7JDFXEGVRFIPBK2OKB2TY5XC4TAVCNFSM6AAAAABDD2PE4WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRQGE2TIOJZHA>. You are receiving this because you were mentioned.Message ID: ***@***.***> ALERT: EMAIL EXTERNAL TO ORGANIZATION - VERIFY BEFORE ACTING

- no major changes to hl7. mostly code cleanup from auto tools.

sjpadgett · 2024-04-17T23:49:20Z

@claimrevolution
I have orders being validated except AOE.
One or more validation errors occurred. Status:400 OBX: Test Code: '57021-8' is missing responses to the following required AOE question(s): 'What is your name?', 'Do you seek the Holy Grail?', 'What is your favorite color?' Generic Response: labGuid, accountNumber or lab test not found
If I remember, DORN sends the AOE in their JSON. Just needs to be parsed and saved to the procedure_answer table.

sjpadgett · 2024-04-18T01:47:00Z

Looks like you have the AOE covered in code but I'm not seeing DORN sending the questions!
Found an issue when sending an order with more than one test i.e sent all three then get No diagnosis present for the second set of two tests. Unknown why yet.

claimrevolution and others added 30 commits August 21, 2023 14:36

added ability to see multiple eligibilities if

8e21bec

if more than one come back on a check. Increased table column size too, truncation was happening

fix styling

fa3ab99

added code review suggestions

77f0444

bootstrap changes

6902cb9

Merge branch 'openemr:master' into master

21cdbbb

trying to get rid of a testing doc I have in my m

f7e11a2

Merge branch 'master' of https://github.com/claimrevolution/openemr_fork

a1ab86f

remove help file

bbcb6f8

file shoudln't be here

ccba3f2

cleanup

ae389cd

remove settings file

cc7a0e6

Merge branch 'openemr:master' into master

c61826a

Merge branch 'master' of https://github.com/claimrevolution/openemr_fork

149a683

dorn start coding

0c00240

first

e84e116

start of dorn module

1f191dc

lab route start

e3f0be3

route managment

f97cda5

updates

98f93ae

Merge branch 'openemr:master' into master

b01d807

ran phpcf fix

afdc3da

additions

03dad72

route adding

9ef74b8

Merge branch 'openemr:master' into master

97d943b

dorn lab updates

441586e

changes to hl7

fb5b958

hl7 stuff

7562f5a

Merge branch 'openemr:master' into master

e2543fe

Merge branch 'master' of https://github.com/claimrevolution/openemr_fork

7372d89

Added Dorn to the list of protocols

ea0e455

sjpadgett added 2 commits April 8, 2024 08:58

review xlate and escapes and cleanups

c312051

Add new ModuleManagerListener class to claimrev and DORN modules.

aa22459

sjpadgett reviewed Apr 8, 2024

View reviewed changes

claimrevolution added 2 commits April 8, 2024 15:45

fix transations

34272f7

Merge branch 'master' of https://github.com/claimrevolution/openemr_fork

1e2c224

bradymiller reviewed Apr 9, 2024

View reviewed changes

final review escaping

b0ea4bd

sjpadgett marked this pull request as draft April 9, 2024 16:14

Fix datepicker

7803036

Fix js error for incomplete dialog title

sjpadgett marked this pull request as ready for review April 10, 2024 16:34

remove order path check it's not needed

a3165b3

- minor style fixes to procedure orders

cb612a4

add restore session

bb82ed8

- annoying address book style fix.

ef31bd8

- no major changes to hl7. mostly code cleanup from auto tools.

adunsulag added this to the 7.0.2.2 milestone Apr 22, 2024

adunsulag added the Status: Needs Work label Apr 23, 2024

claimrevolution and others added 3 commits April 28, 2024 17:15

Merge branch 'openemr:master' into master

ed332d3

Merge branch 'openemr:master' into master

40e5f76

updated to use ajax call for eligibilitiy check

6642391

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Dorn Changes #7221

Dorn Changes #7221

claimrevolution commented Feb 11, 2024

sjpadgett left a comment

sjpadgett Apr 8, 2024

sjpadgett Apr 8, 2024

sjpadgett commented Apr 8, 2024

bradymiller Apr 9, 2024

bradymiller Apr 9, 2024

bradymiller commented Apr 9, 2024

sjpadgett commented Apr 9, 2024

sjpadgett commented Apr 12, 2024

claimrevolution commented Apr 12, 2024 via email

sjpadgett commented Apr 17, 2024

claimrevolution commented Apr 17, 2024 via email

sjpadgett commented Apr 17, 2024

sjpadgett commented Apr 18, 2024

Dorn Changes #7221

Are you sure you want to change the base?

Dorn Changes #7221

Conversation

claimrevolution commented Feb 11, 2024

Short description of what this resolves:

Changes proposed in this pull request:

sjpadgett left a comment

Choose a reason for hiding this comment

sjpadgett Apr 8, 2024

Choose a reason for hiding this comment

sjpadgett Apr 8, 2024

Choose a reason for hiding this comment

sjpadgett commented Apr 8, 2024

bradymiller Apr 9, 2024

Choose a reason for hiding this comment

bradymiller Apr 9, 2024

Choose a reason for hiding this comment

bradymiller commented Apr 9, 2024

sjpadgett commented Apr 9, 2024

sjpadgett commented Apr 12, 2024

claimrevolution commented Apr 12, 2024 via email

sjpadgett commented Apr 17, 2024

claimrevolution commented Apr 17, 2024 via email

sjpadgett commented Apr 17, 2024

sjpadgett commented Apr 18, 2024