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

Fixed Mage_Uploader_Helper_File::getDataMaxSize() when checking against different size units #4039

Merged
merged 5 commits into from
Jun 26, 2024
Merged

Conversation

F1Red5
Copy link
Contributor

@F1Red5 F1Red5 commented Jun 10, 2024

Description (*)

Fixes Mage_Uploader_Helper_File::getDataMaxSize() when different units are used.

Manual testing scenarios (*)

  1. min('1M', '1G')returns 1G

@github-actions github-actions bot added Component: Catalog Relates to Mage_Catalog Component: Uploader Relates to Mage_Uploader composer Relates to composer.json labels Jun 10, 2024
@F1Red5 F1Red5 changed the title Fixes getDataMaxSize() Fixes Mage_Uploader_Helper_File::getDataMaxSize() Jun 10, 2024
@fballiano fballiano changed the title Fixes Mage_Uploader_Helper_File::getDataMaxSize() Fixed Mage_Uploader_Helper_File::getDataMaxSize() when checking against different size units Jun 26, 2024
@fballiano
Copy link
Contributor

when porting to next I'll not include the polyfills since they're not needed (next is running on 8.2 minimum)

@fballiano fballiano merged commit 0243985 into OpenMage:main Jun 26, 2024
19 checks passed
@sreichel sreichel deleted the fix-getDataMaxSize branch June 27, 2024 00:06
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Component: Catalog Relates to Mage_Catalog Component: Uploader Relates to Mage_Uploader composer Relates to composer.json
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants