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.

Sign up for GitHub

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

Jump to bottom

feat: Extract local providers list selector logic from options.inc.php into its own class #7368

Closed
adunsulag opened this issue Apr 17, 2024 · 0 comments · Fixed by #7369
Closed

feat: Extract local providers list selector logic from options.inc.php into its own class #7368

adunsulag opened this issue Apr 17, 2024 · 0 comments · Fixed by #7369
Assignees
@adunsulag
Labels
developers This issue targets an issue that is for developers/collaborators/module writers/technical users
Milestone
7.0.2.1

Comments

@adunsulag
Copy link
Sponsor Member

I need to be able to use the local providers list selector in a module without having to grab everything needed to load the options.inc.php or LBF forms. As a user I want to just include the widget that is needed for the this widget and not have to deal with everything else.
@adunsulag adunsulag self-assigned this Apr 17, 2024
adunsulag added a commit to DiscoverAndChange/openemr that referenced this issue Apr 17, 2024
@adunsulag
Fixes openemr#7368 local provider's form list type
3f56369 
Fixes openemr#7368

This is a local provider's selector widget that was used in options.inc.php. I extracted it into its own class file so I can use it in a module as well as use it in options.inc.php.

I added options to inject a static list of providers into the widget for
caching/performance reasons.  Also added option to inject the query used
to retrieve the list of providers into the widget.
@adunsulag adunsulag mentioned this issue Apr 17, 2024
Merged
adunsulag added a commit that referenced this issue Apr 18, 2024
@adunsulag
Fixes #7368 local provider's form list type (#7369)
632d2f7 
Fixes #7368

This is a local provider's selector widget that was used in options.inc.php. I extracted it into its own class file so I can use it in a module as well as use it in options.inc.php.

I added options to inject a static list of providers into the widget for
caching/performance reasons.  Also added option to inject the query used
to retrieve the list of providers into the widget.
sjpadgett pushed a commit to sjpadgett/openemr that referenced this issue Apr 22, 2024
@adunsulag @sjpadgett
Fixes openemr#7368 local provider's form list type (openemr#7369)
72a8097 
Fixes openemr#7368

This is a local provider's selector widget that was used in options.inc.php. I extracted it into its own class file so I can use it in a module as well as use it in options.inc.php.

I added options to inject a static list of providers into the widget for
caching/performance reasons.  Also added option to inject the query used
to retrieve the list of providers into the widget.

(cherry picked from commit 632d2f7)
@adunsulag adunsulag added this to the 7.0.2.1 milestone Apr 22, 2024
@adunsulag adunsulag added the developers This issue targets an issue that is for developers/collaborators/module writers/technical users label May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@adunsulag adunsulag

Labels
developers This issue targets an issue that is for developers/collaborators/module writers/technical users
Projects
None yet
Milestone
7.0.2.1
Development

Successfully merging a pull request may close this issue.

Fixes #7368 local provider's form list type DiscoverAndChange/openemr
1 participant
@adunsulag