Skip to content

Conversation

@gennaris
Copy link

Questions Answers
Description? Misused $this->merchant_oos_emails variable was preventing the correct recipient(s) from receiving the product out-of-stock notification email. The email was sent to the shop admin instead of the address(es) configured in the module.
Type? bug fix
BC breaks? no
Deprecations? no
Fixed ticket? n/a
How to test? Configure one or more email addresses in the Email alerts module (section Out of stock > Send to). Trigger an out-of-stock event (e.g. by reducing product quantity below the threshold). Before applying this patch, notice that the email is sent to the shop admin instead of the configured address(es). After applying this patch, the notification should be correctly sent to the configured recipient(s).

@ps-jarvis
Copy link

Hello @gennaris!

This is your first pull request on ps_emailalerts repository of the PrestaShop project.

Thank you, and welcome to this Open Source community!

@github-project-automation github-project-automation bot moved this to Ready for review in PR Dashboard Oct 12, 2025
@gennaris
Copy link
Author

I just realized that this PR, while technically valid, has no actual effect in PS 1.7 > since the advanced stock management feature was removed, and that part of the code is never reached.
The environment where I initially found the bug uses a fork of the module that was modified to make this hook work in PS 1.7.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Ready for review

Development

Successfully merging this pull request may close these issues.

2 participants