[2.2][Backport] Database Media Storage - Transactional Emails will now extract image from database in Database Media Storage mode

[gwharton]

Original PR (*)

1. [2.3] Database Media Storage - Transactional Emails will now extract image from database in Database Media Storage mode #21674

Description (*)

If the email logo does not exist in pub/media and we are in database storage mode, then magento should copy the image file from database to local pub/media prior to continuing.

Fixed Issues (if relevant)

1. Database Media Storage - Transaction emails logo not used when pub/media cleared #21671: Database Media Storage - Transaction emails logo not used when pub/media cleared

Manual testing scenarios (*)

1. Setup Magento
2. Change media storage mode to database
3. Synchronize
4. Save Settings
5. Set store General Contact email address to valid email
6. Set email logo (Content->Configuration->Edit->Transactional Emails)
7. Clear Cache
8. Verify email logo exists in database <- This will fail on 2.2-develop due to Database Media Storage - Design Config fails to save transactional email logo correctly #21672
9. Verify email logo exists in pub/media
10. On Frontend, subscribe to newsletter
11. Verify newsletter email contains logo
12. Clear pub/media, simulating a new cluster node being used
13. On Frontend, subscribe to newsletter (need to use new email address, or delete 1st address in backend first)
14. Verify newsletter email contains logo

Contribution checklist (*)

• Pull request has a meaningful description of its purpose
• All commits are accompanied by meaningful commit messages
• All new or changed code is covered with unit/integration tests (if applicable)
• All automated tests passed successfully (all builds on Travis CI are green)

[magento-engcom-team]

Hi @gwharton. Thank you for your contribution
Here is some useful tips how you can test your changes using Magento test environment.
Add the comment under your pull request to deploy test or vanilla Magento instance:

• @magento-engcom-team give me test instance - deploy test instance based on PR changes
• @magento-engcom-team give me 2.2-develop instance - deploy vanilla Magento instance

For more details, please, review the Magento Contributor Assistant documentation

[gwharton]

Hold pending resolution to #21672, otherwise difficult to reproduce in isolation and prove.

[magento-engcom-team]

Hi @sidolov, thank you for the review.
ENGCOM-5302 has been created to process this Pull Request

[engcom-Delta]

✔️ QA Passed

[m2-assistant]

Hi @gwharton, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.