Issue 9811
- Title
- on_change_with_product_uom_category fails on line without product
- Priority
- bug
- Status
- resolved
- Nosy list
- ced, reviewbot, roundup-bot, xavier
- Assigned to
- ced
- Keywords
- easy, review
Created on 2020-11-09.20:16:29 by ced, last changed 3 months ago by roundup-bot.
Messages
New changeset 1cf7bffbc569 by Cédric Krier in branch 'default': Support amend line without product https://hg.tryton.org/tryton-env/rev/1cf7bffbc569
New changeset 0e81e4cf1d32 by Cédric Krier in branch 'default': Support amend line without product https://hg.tryton.org/modules/sale_amendment/rev/0e81e4cf1d32
New changeset ab1d1564f619 by Cédric Krier in branch 'default': Support amend line without product https://hg.tryton.org/modules/purchase_amendment/rev/ab1d1564f619
Here is review331611002 which extend the test scenario.
Could you make a unique review when the fixes are common?
Found on https://discuss.tryton.org/t/woocommerce-connector-for-tryton/2795/31
The line may not have a product defined so it should return None but also the domain on product should allow then to add any product.
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2021-01-18 23:27:05 | roundup-bot | set | messages: + msg63966 |
| 2021-01-18 23:27:01 | roundup-bot | set | messages: + msg63965 |
| 2021-01-18 23:26:54 | roundup-bot | set | messages:
+ msg63964 nosy: + roundup-bot status: testing -> resolved |
| 2021-01-03 20:22:39 | reviewbot | set | messages: + msg63748 |
| 2021-01-03 19:56:51 | ced | set | assignedto: ced messages: + msg63747 reviews: 328881002, 300941002 -> 328881002, 300941002, 331611002 status: chatting -> testing |
| 2020-11-24 13:12:44 | xavier | set | assignedto: xavier -> (no value) |
| 2020-11-16 11:03:38 | ced | set | messages: + msg61959 |
| 2020-11-16 10:57:06 | reviewbot | set | messages: + msg61958 |
| 2020-11-16 10:57:05 | reviewbot | set | reviews: 328881002 -> 328881002, 300941002 |
| 2020-11-16 10:57:02 | reviewbot | set | messages: + msg61957 |
Showing 10 items. Show all history (warning: this could be VERY long)