Commit Graph

9 Commits

Author SHA1 Message Date
Ernesto Tejeda
11ecd1dd9a [MIG] mass_mailing_custom_unsubscribe: Migration to 15.0 2022-12-19 11:38:40 +01:00
Ernesto Tejeda
9b7ef0ae19 [MIG] mass_mailing_custom_unsubscribe: Migration to 14.0 2022-12-19 11:38:40 +01:00
Carlos Roca
fc38567fb2 [MIG] mass_mailing_custom_unsubscribe: Migration to v13.0 2022-12-16 13:13:16 +01:00
Carlos Roca
844c382fe7 [IMP] mass_mailing_custom_unsubscribe: black, isort, prettier 2022-12-16 13:13:16 +01:00
ernesto
42e8dd1e74 [MIG] mass_mailing_custom_unsubscribe: Migration to 12.0 2022-12-16 13:13:16 +01:00
ernesto
10c516473f mass_mailing_custom_unsubscribe: Fix testing error
Fix assertIn error thrown in testing mode
2022-12-16 13:13:16 +01:00
David
2e416ed4d0 [MIG] mass_mailing_custom_unsubscribe: Migration to 11.0 2022-12-16 13:13:16 +01:00
Jairo Llopis
70a1c997ac [MIG][mass_mailing_custom_unsubscribe] Migrate to v10 2022-12-16 13:13:16 +01:00
Jairo Llopis
708780cab9 [9.0][MIG][mass_mailing_custom_unsubscribe] Migrate.
- Imported last updates from v8.
- Adapted to v9.
- Added a saner default to `mass_mailing.salt` configuration parameter by
  reusing `database.secret` if available, hoping that some day
  https://github.com/odoo/odoo/pull/12040 gets merged.
- Updated README.
- Increase security, drop backwards compatibility.
  Security got improved upstream, which would again break compatibility among current addon and future master upstream.
  I choose to break it now and keep it secured future-wise, so I drop the backwards compatibility features.
- Includes tour tests.
- Removes outdated tests.
- Extends the mailing list management form when unsubscriber is a contact.
- Adds a reason form even if he is not.
- Avoids all methods that were not model-agnostic.

[FIX][mass_mailing_custom_unsubscribe] Reasons noupdate

After this fix, when you update the addon, you will not lose your customized reasons.

[FIX] Compatibilize with mass_mailing_partner

Current test code was based on the assumption that the `@api.model` decorator on `create()` ensured an empty recordset when running the method, but that's not true. This was causing an incompatibility betwee these tests and the `mass_mailing_partner` addon, which works assuming 0-1 recordsets.

Now records are created from an empty recordset, and thus tests work everywhere.

Update instructions

If the user does not add the unsubscribe snippet, nothing will happen, so it's added to README to avoid confusion when testing/using the addon.

[FIX] Use the right operator to preserve recordsets order

Using `|=` sorts records at will each time (treating them as Python's `set`).
Using `+=` always appends a record to the end of the set.
Since we are using the record position in the set, this caused the test to work sometimes and fail other times. Now it works always.
2022-12-16 13:13:16 +01:00