Skip to content

Password forgotten notification - #112

Merged
gburton merged 5 commits into
CE-PhoenixCart:1.1.0.8from
blaine-cwe:password-forgotten-notification
Aug 20, 2026
Merged

Password forgotten notification#112
gburton merged 5 commits into
CE-PhoenixCart:1.1.0.8from
blaine-cwe:password-forgotten-notification

Conversation

@blaine-cwe

Copy link
Copy Markdown

Explain the details for making this change. What existing problem does it solve?

#111

Currently, the password_forgotten.php script uses a hardcoded email generation routine directly in the root file. This makes it impossible for shop owners to customize, heavily style, or convert the password reset email to HTML without modifying core files.

This proposal updates password_forgotten.php to route the reset email through the existing Notifications::notify() system. This brings the password reset architecture completely in line with the modern "Create Account" and "Order Update" modular notification systems.

Test plan (required)

Baseline Test: On a stock 1.1.0.7 install, trigger a password reset. Observe the standard plain-text email arrives via the hardcoded root file routine.

Apply Changes:

Replace the $db->query email block in password_forgotten.php with Notifications::notify('password_forgotten', [...]);

Upload the proposed stock n_password_forgotten controller, template, and language file.

Core Verification: Trigger another password reset. Observe that the exact same plain-text email arrives, proving the module perfectly replicates core functionality.

Override Verification: Install a custom/3rd-party HTML notification module mapped to the password_forgotten trigger. Run the reset form again and observe that the custom HTML email is delivered without any further core modifications.

Converting password forgotten to a notification module.
Converting password forgotten to notification module.
Language file for new password forgotten notification module.
Controller file for new Password Forgotten notification module.
Template file for new Password Forgotten notification module.
@gburton
gburton merged commit 70e60c7 into CE-PhoenixCart:1.1.0.8 Aug 20, 2026
4 checks passed
@gburton

gburton commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Thank You @blaine-cwe

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants