Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: fixes to allergy card in patient summary #6863

Merged

Conversation

bradymiller
Copy link
Member

@bradymiller bradymiller commented Sep 18, 2023

fixes to allergy card in patient summary

@@ -1058,16 +1058,7 @@ function filterActiveIssues(array $i): array
// ALLERGY CARD
if ($allergy === 1) {
$allergyService = new AllergyIntoleranceService();
$_rawAllergies = filterActiveIssues($allergyService->getAll(['lists.pid' => $pid])->getData());

This comment was marked as resolved.

}
return xl_list_label($row['title']);
}
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

EtiIRpM0

@bradymiller
Copy link
Member Author

This is testing well and good to go. Will bring this in (and change the function call to original GetAll call) after another PR is in the codebase: #6864

@stephenwaite
Copy link
Member

southpark_cheering_edited

@stephenwaite
Copy link
Member

all set for you @bradymiller

EtiIRpM0

@bradymiller bradymiller force-pushed the fix-allergy-card-patient-summary branch from c8cb095 to 85cf9d3 Compare September 20, 2023 22:40
@bradymiller bradymiller merged commit 9b0504a into openemr:master Sep 21, 2023
@adunsulag adunsulag added this to the 7.0.2 milestone Nov 16, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants