Patient Documents for portal and staff rework

That’s it.
Please open a different forum issue or a git issue for this as we’re coopting this thread. I’ll pay attention to this as I see occasionally on my dev system.
When open new issue please see if user can give a description of what else is going on such as several tabs open for different patients or any info relevant. This is very helpful for me to troubleshoot.

Anyway thanks David.

Ok, will do once I get a description from the Doctor.

Also, Pre-Patch calendar files are owned by root with root group and 0644 attributes. Is that Ok?
Reported Properties are;
File: /var/www/html/openemr/interface/main/calendar/add_edit_event.php
Type: PHP script, ASCII text, with very long lines, with CRLF line terminators
Size: 88.97 KiB Blocks: 184 IO Block: 4096 regular file
Device: fd01h/64769d Inode: 35262615 Links: 1
Attrs: --------------e—
Access: (0644/-rw-r–r--) Uid: (0/root) Gid: (0/root)
Access: 2021-08-09 17:10:16.098620227 -0400
Modify: 2021-07-26 18:51:39.000000000 -0400
Change: 2021-08-09 17:05:51.140055002 -0400

No, group should be whatever group your instance is using like apache or such. 0644 is correct.
You shouldn’t unzip as a root user but whatever user account is elevated from root.

1 Like

Asked because other files in this directory have www-data / www-data as owner and group and I’m guessing the patch files should be the same.
I use Webmin to extract patches as it produces same file properties as doing “sudo unzip -o” from command line as in this example compared with my previous post above.

File: /var/www/html/openemr/interface/main/calendar/add_edit_event.php
Type: PHP script, ASCII text, with very long lines, with CRLF line terminators
Size: 88.97 KiB Blocks: 184 IO Block: 4096 regular file
Device: fd01h/64769d Inode: 43254587 Links: 1
Attrs: --------------e—
Access: (0644/-rw-r–r--) Uid: (0/root) Gid: (0/root)
Access: 2021-08-10 07:30:44.428319063 -0400
Modify: 2021-07-26 18:51:39.000000000 -0400
Change: 2021-08-10 07:29:15.166479182 -0400

I always run a chown (permissions) and restorecon (SELinux) after all patches.

As this project is developed against Debian/Ubuntu you will almost always need to fix permissions if you are not on that.

I also never actually run as root, so always things with sudo.
I have an active deployment on Oracle Enterprise Linux 8. So I always do this:

[jbusch@openemr ~]$ sudo chown -R apache:apache /var/www/html
[jbusch@openemr ~]$ sudo restorecon -FR /var/www/html/openemr/

Jerry do you plan to have the document names display without the .tpl and the underscores convert to spaces at some point? Would make it more neat for patients.

Will you also be adding in file management to move files between general and repository?
And the ability to push documents from the repository to a patient? Not sure if it needs to copy the file, but maybe it could just become visible as a priority request in their forms section.

yes. currently I’m refactoring portal to get rid of sidebar(continuation of @benmarte work) and I’m doing some more work in documents for easier patient work flows plus some new feature.
This is much better as we gain lots more view port size.

2 Likes

Looks amazing, looking forward to seeing it. Will there be a prepatch nearer the time this is ready? Would love to try this out.

Depends on how different globals is but, I do plan to give out for testing soon.

@madmax Here is a prepatch for testing on v6(2). Be sure to backup portal directory otherwise the rest is new to core.

Doc title

1 Like

@madmax
Did the patch install okay? I’m trying to decide if I want to include in patch three Brady is working on getting out next week.
I’d feel better knowing if an experienced user okay’d the install. Thanks

Yes, appears to be working thanks!

Okay thanks. I’ll post the final here with the new fixes as it will be for patch 3. An hour or two…

1 Like

@madmax @gutiersa and others

Okay all here is the portal version that will be in patch v6.0.0(3).
It’s a very different look and feel with concentration on easier patient work flows.

Doing some work concerning logging in as related persons(parents, guardians, occupational health etc.) and more is coming concerning documents like downloads, uploads and document patient profiles. Will also work on secure chat by providing events that can hook to telehealth.

final_portal(v6.0.3).zip (54.3 KB)

1 Like

Found a bug, if you have valid or fake login data present (such as from a password manager) but want to register another account, if the fields are filled then the login page actually errors/refreshes.

If the fields are blank it appears to go to the registration page normally.

I tested with “123@123.com” in username and email and 123 in password and the page just refreshes but does produce an error in the browser console.

Uncaught TypeError: document.getElementById(…) is null
[portal:28:36]

This was caught on firefox, Chrome appears to freak out even more with console messages.

1 Like