Closed Tigrov closed 11 months ago
Q | A |
---|---|
Is bugfix? | ❌ |
New feature? | ❌ |
Breaks BC? | ❌ |
Fixed issues | - |
Introduction of a Changelog Entry
A new record has been added to the changelog documentation, detailing the modification of a critical programming function, specifically AbstractSchema::getDataType()
(linked to improvement request #786).
Optimization of the getDataType()
Function
The getDataType()
function within the AbstractSchema.php
file has been restructured to utilize a match
expression which is a simplified and efficient way to handle multiple conditions. This replaces the prior method of using an array mapping, which results in cleaner and easy-to-maintain code.
Removal of Redundant Case for PHP_TYPE_STRING
A specific case for handling PHP_TYPE_STRING
within the getDataType()
function has been removed. This is due to the function's redesign, which now effectively manages such scenarios using a default
case. This declutters the code and eliminates redundancy.
All modified and coverable lines are covered by tests :white_check_mark:
Comparison is base (
f655cb7
) 98.88% compared to head (dbb35ec
) 99.64%.:exclamation: Current head dbb35ec differs from pull request most recent head 5854fb7. Consider uploading reports for the commit 5854fb7 to get more accurate results
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.