Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Wrong mimetype returned by getMimeType from Magento library #716 #717

Closed
wants to merge 4 commits into from

Conversation

Dorn-
Copy link

@Dorn- Dorn- commented Oct 28, 2014

Fix the issue which is present in getMimeType function.
In some cases it would return an integer (constant) instead of a string, for example 2 instead of "image/jpeg"

flavien.chantelot added 4 commits January 8, 2014 12:19
Just load the needed weee attributs and avoid a sort.
The difference is really big when you have a lot of attributs
Conflicts:
	app/code/Magento/Weee/Model/Tax.php
@verklov
Copy link
Contributor

verklov commented Dec 9, 2014

Hello @Dorn- , as I see, the team resolved issue #716 that you reported. Please expect the fix to be deployed to the repository shortly.

@verklov verklov closed this Dec 9, 2014
okorshenko pushed a commit that referenced this pull request Feb 7, 2017
[TSG] Backporting for 2.1 (prs2)
VitaliyBoyko pushed a commit to VitaliyBoyko/magento2 that referenced this pull request Jun 22, 2018
VitaliyBoyko pushed a commit to VitaliyBoyko/magento2 that referenced this pull request Jun 22, 2018
magento-engcom-team added a commit that referenced this pull request Jun 19, 2019
…ng cart #717

 - Merge Pull Request magento/graphql-ce#717 from magento/graphql-ce:683-add-discount-information-cart
 - Merged commits:
   1. 31b34ad
   2. 8515566
   3. fe708a2
   4. 382d41f
   5. a1e126c
   6. 2d8bc6d
magento-engcom-team pushed a commit that referenced this pull request Dec 5, 2019
- Reverted BIC actiongroup change
- Changed actionGroup to extend existing
- Moved test materials from incorrect modules back into correct module
- added cleanup steps
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants