diff options
| author | Quentin Mourier <qmo@odoo.com> | 2020-03-26 08:00:06 +0000 |
|---|---|---|
| committer | Thibault Delavallée <tde@odoo.com> | 2020-03-26 15:57:43 +0000 |
| commit | 843caa270c7d13e6c81ef8f077e469bd9746a2cf (patch) | |
| tree | e2df5b36ae10a97155d44e004917615082ac64c9 /addons/website_slides/models | |
| parent | 37d281681c864f36c09de6f3895f9ef09a9dcdff (diff) | |
[FIX] website_slides: fix resequencing based on category
When new slides are added through frontend a resequencing of slides and
categories sequence is done by calling _resequence_slides on the channel.
However this method is not working as expected, and resequencing should
take a category in order to better know where to start its resequencing.
Otherwise it simply reorder slides in the same order as before, or in an
unpredicted or prodicted but fasly order if new slide category has just been
created.
Task ID 2090927
X-original-commit: f038946ae2e3f174bee086281a541c6bc770e354
Diffstat (limited to 'addons/website_slides/models')
| -rw-r--r-- | addons/website_slides/models/slide_channel.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/addons/website_slides/models/slide_channel.py b/addons/website_slides/models/slide_channel.py index 47a823b6702..214a2025f2c 100644 --- a/addons/website_slides/models/slide_channel.py +++ b/addons/website_slides/models/slide_channel.py @@ -621,13 +621,13 @@ class Channel(models.Model): }) return category_data - def _resequence_slides(self, slide): + def _resequence_slides(self, slide, force_category=False): ids_to_resequence = self.slide_ids.ids index_of_added_slide = ids_to_resequence.index(slide.id) - category_id = slide.category_id.id next_category_id = None if self.slide_category_ids: - index_of_category = self.slide_category_ids.ids.index(category_id) if category_id else None + force_category_id = force_category.id if force_category else slide.category_id.id + index_of_category = self.slide_category_ids.ids.index(force_category_id) if force_category_id else None if index_of_category is None: next_category_id = self.slide_category_ids.ids[0] elif index_of_category < len(self.slide_category_ids.ids) - 1: @@ -638,7 +638,7 @@ class Channel(models.Model): index_of_next_category = ids_to_resequence.index(next_category_id) ids_to_resequence.insert(index_of_next_category, added_slide_id) for i, record in enumerate(self.env['slide.slide'].browse(ids_to_resequence)): - record.write({'sequence': i}) + record.write({'sequence': i + 1}) # start at 1 to make people scream else: slide.write({ 'sequence': self.env['slide.slide'].browse(ids_to_resequence[-1]).sequence + 1 |