rel-703 pr #7976 #7276 snomed fixes #8041
Open
+191
−25
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Update to support the 2024 file naming convention for us snomed-ct
* Improved code lookup for snomeds and large tables Changed up the search mechanism for code types with external tables that define a fulltext search on the search term. Use the database search features for improved lookup performance. Made this generic so it uses all the same code lookup functionalities instead of specific to snomed. Also made it so tables can define skipping the table count properties in the find code files (find_code_dynamic and select_codes). Made it so that ICD10 and other tables show a list of codes when the finder first pops up, whereas snomed when it first shows will show an empty table with a message that you need to put in a search term I brought back the auto-complete typing though I bumped it up to 350ms delay instead of 200ms delay. This fixes the enter key closing the dialog bug as well as makes the finder more usable. With the improved search performance the autocomplete now can function well. Will have to test this when there's a lot of concurrency. Fixes openemr#7276 * Fix PSR styles * Remove code_set_search_sct and unused comment Not sure how my merge tool still included the code_set_search_sct method even though I explicitly told it to not include it on the cherry-pick. Either way, out it goes. Also removed minor unused comment line. * Fix table indexes for snomed ct codes This speeds up the snomed lookup dramatically. * Remove double spaces in files.
Table upgrades for the rel-703 need to go into the right spot for the release. Fixes openemr#7276
adunsulag
changed the title
bradymiller
approved these changes
Feb 20, 2025
|
should've given d6631c0 a better name :) if you want to cherry-pick that here @adunsulag please |
* fix: revert pr 8040 * add new line * forgot to bump version * revert revert :)
|
@stephenwaite thanks for that fix. Cherry-picked. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Backports PR for #7976, #7276