(#4815) Fix PDQ phpstan issues#4923
Conversation
f868416 to
fa1dfed
Compare
|
@blairlearn what is the status of this? |
|
From Teams conversation
|
|
To test, we'll need to get this onto an ODE and then run the new loader to confirm that things work as expected and all summaries are loaded with no crazy errors/fires/natural disasters. (Orphaned cleanup part of the loading process runs without issue.) |
fa1dfed to
ec192c8
Compare
ODE DeploymentCode has been deployed to ODE 1091. |
|
@jfrank-nih to check with @blairlearn in terms of any testing steps to review on the ODE where it is currently deployed. |
In pdq_cancer_information_summary/src/OrphanCleanup.php: - remove line of code inadvertently copied from another method - reassure PHP (and phpstan) that we have the the right interface for invoking deleteRevision() Closes #4815
ec192c8 to
07d54f5
Compare
|
Tested on ODE1091 with CIS/DIS loader workflows. Cleanup still working as expected. To test:
Drug Info Summaries:
|
welshja
left a comment
There was a problem hiding this comment.
Looking good to me after testing
In pdq_cancer_information_summary/src/OrphanCleanup.php:
Closes #4815