Scanned Notes form modified.

Brought to you by: bradymiller, cmswest, robertdown, sjpadgett, sunsetsystems

#98 Scanned Notes form modified.

Status: open

Owner: nobody

Labels: None

Priority: 5

Updated: 2011-03-29

Created: 2011-03-25

Creator: ZH Healthcare

Private: No

The form 'Scanned Notes' is modified to upload files.
The errors in the form are removed.
Any type of file can be uploaded,earlier only it was allowing images.

The files uploaded gets saved in the path \sites\default\documents\pid\encounter\ The present document management system is used to store the details of the uploaded file.Table 'documents' stores the details.
A category 'Scanned Encounter Notes' is added.
The table 'form_scanned_notes' stores the note and the document id.A field is added to store the id.

Selecting the encounter it displays the file along with the form listing.
Can change the file in the edit screen.

The Past Encounter List(Clinical View) shows the files against the form 'Scanned Notes' in the corresponding encounters.
This is the same with the Report link from the demo screen.Report can the generated by ticking the check box.

More over the documents uploaded at the patient level through the Documents link from demo are not affected.
They come in the Clinical View and the Report link as usuall.

Discussion

ZH Healthcare - 2011-03-25

Scanned Notes form patch

0001-The-form-Scanned-Notes.patch

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

ZH Healthcare - 2011-03-26

Issue-of-image-not-getting-displayed

0002-Issue-of-image-not-getting-displayed.patch

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

ZH Healthcare - 2011-03-29

The discussions are going on and finally concluded at the url
https://sourceforge.net/projects/openemr/forums/forum/202505/topic/4399421

bradymiller's comments in it regarding the image not getting displayed is also addressed in the second commit.

Updated in git to the branch
(doc_db_to_file_correct) accessible at
https://github.com/zhhealthcare/openemr/tree/doc_db_to_file_correct

Please review the code so that necessary action can be taken.

Thanks
Paul
www.zhservices.com

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

ZH Healthcare - 2011-03-29

assigned_to: nobody --> bradymiller
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Brady Miller - 2011-03-29

assigned_to: bradymiller --> nobody
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Brady Miller - 2011-03-29

hi,
Placed review on github.
-brady

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

ZH Healthcare - 2011-04-01

Scanned_notes complete new one.

0001-scanned_notes.patch

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

ZH Healthcare - 2011-04-01

The document screen is working for normal case with subdirectories.
For those porting from other system, with subdirectories having different path in url, is dealt with.
As said by bradymiller, code in the function retrieve_action() is utilized.
For the above, 2 issues were there in the documents screen.
The file was not getting displayed and also it was not able to rename.
Both the issues are fixed.

If file exist in the same path timestamp is added to the second one.

Updated in git to the branch
(scanned_notes) accessible at
https://github.com/zhhealthcare/openemr/tree/scanned_notes
Patch file named '0001-scanned_notes.patch' is uploaded here.

Please review the code so that necessary action can be taken.

Thanks
Paul
www.zhservices.com

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Brady Miller - 2011-04-27

Placed a github review on 4/6/11 (this is just a reminder to myself)

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Arnab Naha - 2012-03-11

Has this been pushed to official codebase??

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Brady Miller - 2012-03-13

No,
There has been no movement on this since the last review.
-brady

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Arnab Naha - 2012-06-17

Zh Healthcare...
Please throw some light on this topic...

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Log in to post a comment.