Mail catchall domain parameter being reset by changing any general setting
Bug #1311628 reported by
Stefan Rijnhart (Opener)
This bug affects 4 people
Affects | Status | Importance | Assigned to | Milestone | ||
---|---|---|---|---|---|---|
Odoo Addons (MOVED TO GITHUB) |
Fix Released
|
Low
|
OpenERP R&D Addons Team 1 | |||
OpenERP Community Backports (Addons) | Status tracked in 7.0 | |||||
7.0 |
Fix Committed
|
Undecided
|
Stefan Rijnhart (Opener) |
Bug Description
How to reproduce:
Go back to Settings -> Configuration -> General settings. Erase Alias Domain, and Apply. Note that the website url is filled in in the catchall domain field, but for now we can mail from valid email addresses configured on the user accounts because system paramenter mail.catchall.
Now, change any other setting or just reapply the same settings.
Expected results:
Because the mail catchall setting is not changed, I expect that emails are still being sent without using the catchall aliases.
Actual results:
Reapplying the settings overwrites mail.catchall.
Related branches
lp://staging/~therp-nl/openobject-addons/7.0-lp1311628-dont_overwrite_empty_mail_catchall_domain
- Olivier Dony (Odoo): Approve
- Pedro Manuel Baeza (community): Approve (code review)
- Sylvain LE GAL (GRAP) (community): Approve (code review, no test)
-
Diff: 21 lines (+3/-3)1 file modifiedmail/res_config.py (+3/-3)
lp://staging/~stefan-opener/ocb-addons/7.0-lp1311628-dont_overwrite_empty_mail_catchall_domain
- Guewen Baconnier @ Camptocamp: Approve (code review)
- Sylvain LE GAL (GRAP) (community): Approve (code review, no test)
- Pedro Manuel Baeza: Approve (code review)
- Lara (Therp) (community): Approve (test)
-
Diff: 21 lines (+3/-3)1 file modifiedmail/res_config.py (+3/-3)
description: | updated |
summary: |
- Catchall parameter being reset by changing any general setting + Mail catchall domain parameter being reset by changing any general + setting |
To post a comment you must log in.
Thanks Stefan for the bug report and the excellent fix, as usual!
Your patch was merged in 7.0 at revision 10031 rev-id <email address hidden>.