E-Sign API

kchapple wrote on Thursday, November 28, 2013:

Greetings OpenEMR community!

I have published a branch to my github.com account for your review. The ken_esign branch contains the Mi2 E-Sign API and implementations for signing/locking forms and encounters.

The commit includes:

  • The jquery.esign.js jQuery plugin which attaches E-Sign behavior to DOM objects like links or buttons.
  • PHP interfaces that work with jQuery plugin to handle back-end processing.
  • Implementations of interfaces for forms and encounters. Future possibilities include documents, patients, etc…
  • Manditory change to CSS layout in forms.php to make room for the E-Sign encounter logging.
  • Database changes to add esign_signatures table
  • Updates to fancybox and jQuery in the encounter/forms.php.

Please let me know if you have any questions! Happy holidays from Medical Information Integration!

Ken Chapple
ken@mi-squared.com

openemrdev wrote on Friday, November 29, 2013:

Ken

I fetch your code your git branch (esign)

After download that code,The left nav frame gets blank

Please advise if i am doing anything wrong

My git commands are

git remote add ken GitHub - kchapple/openemr: Mirror of official OpenEMR Sourceforge repository
git checkout -b esign
git pull ken ken_esign

tmccormi wrote on Friday, November 29, 2013:

Blank frames are usually permissions issues, is there an error in your
apache2 log or php log files?

Tony McCormick, CTO
Medical Information Integration, LLC

Direct: 713-574-6709, cell 503-330-2239
Office: 866-735-0897
@MI2_OpenEMR

On Thu, Nov 28, 2013 at 11:58 PM, openemrdev openemrdev@users.sf.netwrote:

Ken

I fetch your code your git branch (esign)

After download that code,The left nav frame gets blank

Please advise if i am doing anything wrong

My git commands are

git remote add ken GitHub - kchapple/openemr: Mirror of official OpenEMR repository
git checkout -b esign
git pull ken ken_esign

E-Sign APIhttps://sourceforge.net/p/openemr/discussion/202506/thread/9687253b/?limit=50#2d02

Sent from sourceforge.net because you indicated interest in
OpenEMR / Discussion / Developers

To unsubscribe from further messages, please visit
SourceForge.net: Log In to SourceForge.net

kchapple wrote on Friday, November 29, 2013:

Greetings,

A white screen indicates a crash somewhere. You can check the apache2 error_log for the exact error. It would be helpful if you could post the error here. Did you run the setup wizard? There is a new database table that is required for E-Sign, and will either require a new installation, or manually add the required esign_signatures table.

To install, the easiest thing to do is create a new empty directory. Change to that directory and type the following commands:

git clone git@github.com:kchapple/openemr.git
git fetch origin ken_esign
git checkout ken_esign

Then you can point your Apache webserver’s document root to the new directory you created and run the setup wizard.

Thanks,
Ken

bradymiller wrote on Monday, December 02, 2013:

Hi Ken,

I placed a review on github (I also got the white screen of death in left menu and placed the php fatal error details on the github review):

Look forward to the next revision.

-brady
OpenEMR

kchapple wrote on Monday, December 02, 2013:

Hi Brady,

Thanks for your detailed review. Great work!

I’ve posted a revision that addresses the comments in the review. I couldn’t easily test for the crash on my environment, but it should be fixed as well.

Ken

kchapple wrote on Thursday, December 05, 2013:

Hey all, let us know if this is OK to merge.

Thanks!

bradymiller wrote on Thursday, December 05, 2013:

Hi Ken,
Still need to test and do a another look through the code. Can you rebase your two commits into one on a new branch in github (maybe ken_esign_2 or something like that)?
thanks,
-brady
OpenEMR

kchapple wrote on Thursday, December 05, 2013:

I’ve pushed a branch that contains a rebase of those two commits.

Ken

bradymiller wrote on Friday, December 06, 2013:

Hi Ken,

I placed a review on github:

thanks,
-brady
OpenEMR

kchapple wrote on Tuesday, December 10, 2013:

Hey all,

I’ve placed another revision of Esign on my github account page. I added esign log to report, changed type to is_lock (Brady’s database optimization suggestion,) added implementation for LBF forms and hopefully caught all the escaping and translation issues.

Thanks,
Ken

bradymiller wrote on Wednesday, December 11, 2013:

Hi Ken,

I placed a review on github:

-brady
OpenEMR

kchapple wrote on Wednesday, December 11, 2013:

Fixed the crash caused by the reference to Linkify, and added global lock toggle option.

Ken

bradymiller wrote on Friday, December 13, 2013:

Hi Ken,

I placed a review on github:

-brady
OpenEMR

sunsetsystems wrote on Friday, December 13, 2013:

I’ll try to take a look at this also over the weekend.

Rod
http://www.sunsetsystems.com/

kchapple wrote on Saturday, December 14, 2013:

I have pushed a 5th revision, which includes hashing and verification of the signatures themselves, and reversing the order of the log. This is all I had time to do today before I went on vacation. The stuff that is left is mostly style and doesn’t require database changes. As far as basic functionality, I should be done, and would appreciate some more testing before any final revisions. I likely won’t be able to get to any further revisions until 12/23.

Reverse the ordering of signatures (ie. most recent should be at bottom). This is important because then the signature stamp is signing off on whatever is above the signature stamp, which is what physicians are accustomed to when signing orders and notes. For example, when I sign a note, I would be signing off on all the signatures and addendums prior to my signature.

This was done.

Place addendum above the applicable signature. This is also important, because I am signing off on it so it should go above the signature stamp.

This was done.

Do a sha1 hash of the addendum itself in a new sql column (ie. hash_addendum). This is part of the note so needs to be hashed like the main note to prove no changes were made to addendum post-hoc.

This was done.

In the report, do not show anything if there are no signatures. No reason to show this in report and will make physicians feel uncomfortable sending reports with empty signature messages in it.

Not done yet.

In the report, do not show the “eSign Log” label above the signature stamps for forms (just show the addendums and signature stamps). I’d also argue to remove this label in the encounter screen also for forms since it is not just a log (ie. the addendums will end up containing clinical information).

Not done yet.

In the report, make the pdf output option signature stamps readable.

I didn’t have an issue on FF on Mac, will have to test with other browsers.

If esign is turned off, still need to show existing esignature/addendums if they exist, especially for the forms (need to realize that the addendum/signatures may contain part of note so it still needs to be shown).

This was done.

Change “eSign” button label to just “Sign”

Not done.

If time would be nice to add a Sign button to one of the forms (for example, the SOAP note place a Sign Note button to right of Save Note button), Even better to get a “Sign” button to right of the “Save” button in the LBF forms, since LBF forms are heavily used and you get them all with just one change.

Not done.

Once above is done, this thing is super ready for commit to codebase and I’d argue that community will likely want the both globals for Allowing E-Signing Individual Forms and Locking an e-signed form individually as turned on by default. The structure you have with above items incorporated are extremely useful because a huge range of things can be done, for example simply signing/freezing notes and then adding signed addendums (can range from clinical information to why the note was deleted for example, wrong patient) to things like a academic structure where an intern signs, resident signs with addendum and then attending signs with addendum. Very cool and that’s just talking about forms, considering your framework can also be extended on any category of items in the openemr database. Again, very cool.

Thanks for your input, and looking forward to contributing this code!

Ken

sunsetsystems wrote on Monday, December 16, 2013:

Looking good! I did a quick test and made a couple of comments.

Rod
http://www.sunsetsystems.com/

tmccormi wrote on Friday, January 03, 2014:

This branch contains a minor bug fix to the LBF part of the signing/locking process.

With this we consider the project ready to commit. There are many fine suggestions for enhancements and some things where opinions will differ.

I plan on committing this to the dev-tip this weekend and we hope to produce a patch for release 4.1.2 shortly thereafter.

I am certain many cool updates will be added over time by the community as well.

Thanks to all those that contributed to the Crowd Funding activity.

–Tony

bradymiller wrote on Friday, January 03, 2014:

Hi Tony,

Regarding above review, can we get clarity on what was done and not done in this revision (so do not have to re-review anything on our end).

Also wait on the commit to dev-tip; note the code is still up for another review/test (I have been holding off an any further review/testing until this next revision).

-brady
OpenEMR

tmccormi wrote on Saturday, January 04, 2014:

This revision just fixed a minor LBF bug. No other changes where made (or
will be) since the last -5 updates.

Tony McCormick, CTO
Medical Information Integration, LLC

Direct: 713-574-6709, cell 503-330-2239
Office: 866-735-0897
@MI2_OpenEMR

On Fri, Jan 3, 2014 at 12:35 AM, Brady Miller bradymiller@users.sf.netwrote:

Hi Tony,

Regarding above review, can we get clarity on what was done and not done
in this revision (so do not have to re-review anything on our end).

Also wait on the commit to dev-tip; note the code is still up for another
review/test (I have been holding off an any further review/testing until
this next revision).

-brady
OpenEMR http://www.open-emr.org/

E-Sign APIhttps://sourceforge.net/p/openemr/discussion/202506/thread/9687253b/?limit=25#939f

Sent from sourceforge.net because you indicated interest in
OpenEMR / Discussion / Developers

To unsubscribe from further messages, please visit
SourceForge.net: Log In to SourceForge.net