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
Fix dom observer not calling all listeners (regression) #38623
base: 2.4-develop
Are you sure you want to change the base?
Conversation
Hi @thlassche. Thank you for your contribution! Add the comment under your pull request to deploy test or vanilla Magento instance:
❗ Automated tests can be triggered manually with an appropriate comment:
Allowed build names are:
You can find more information about the builds here For more details, review the Code Contributions documentation. |
@thlassche wasn't it fixed in #38276 ? |
@magento run all tests |
@magento run all tests |
@magento give me test instance |
Hi @engcom-Bravo. Thank you for your request. I'm working on Magento instance for you. |
Hi @engcom-Bravo, here is your Magento Instance: https://16dd4518b17f0dfee16359646226dbfd.instances-prod.magento-community.engineering |
Hi @thlassche, Thanks for the collaboration & contribution! ✔️ QA Passed Preconditions:
Steps to reproduce
Before: ✖️ After: ✔️ Builds are failed. Hence, moving this PR to Extended Testing. Thanks. |
Description (*)
A change in
dom-observer.js
in 5b931fe changed the logic, making it only call the first listener instead of all listeners. This leads to unexpected behaviourThe
return
here now exits the loop early, which wasn't the case before, since it was inside a_
function before: 5b931fe#diff-844cfb7b272693bacaa64be3a0fd96a543cc145b478330777c39682cc15b5961R152Fixed Issues (if relevant)
Contribution checklist (*)