Closed Deltik closed 6 months ago
Code Climate has analyzed commit 688ebdfe and detected 0 issues on this pull request.
The test coverage on the diff in this pull request is 100.0% (80% is the threshold).
This pull request will bring the total coverage in the repository to 38.7% (-0.1% change).
View more on Code Climate.
Great! Thank you @Deltik ! 👍
Motivation and Context
Fixes: https://github.com/e107inc/e107/issues/5131
Description
Switch the logging file handle check to use
is_resource()
for correctly identifying whether the file handle is open.How Has This Been Tested?
New unit test:
e107EmailTest::testLogFileHandle()
Types of Changes
Checklist