Fix getNameUnsanitizedHtml name override - Githubissues

alma / alma-monthlypayments-magento2

Magento 2 Payment Gateway integration for Alma

Other

6 stars 9 forks source link

Fix getNameUnsanitizedHtml name override #193

Closed Francois-Gomis closed 2 months ago

Francois-Gomis commented 2 months ago

Reason for change

Code changes

Check if cart is defined and items is an iterable.

How to test

Check order summary with an insurance prodcut - Impossible to reproduce bug As a reviewer, you are encouraged to test the PR locally.

Checklist for authors and reviewers

[X] The title of the PR uses business wording, not technical jargon, for the changelog readers to understand it

Non applicable

[ ] The PR implements the changes asked in the referenced task / issue
[ ] The automated tests are compliant with the testing strategy
[ ] The tests are relevant, and cover the corner/error cases, not only the happy path
[ ] You understand the impact of this PR on existing code/features
[ ] The changes include adequate logging and Datadog traces
[ ] Documentation is updated (API, developer documentation, ADR, Notion...)

github-actions[bot] commented 2 months ago

⏳E2E tests are currently running. ➡️ You can follow their progression here.

sonarcloud[bot] commented 2 months ago

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

github-actions[bot] commented 2 months ago

❌ E2E tests have failed. ➡️ You can find the results here.