Skip to content

Commit 07a1c10

Browse files
author
Francesco Faraone
committed
LITE-20982 return latest db value in previous data instead of the value before the latest save
1 parent b907097 commit 07a1c10

File tree

3 files changed

+8
-7
lines changed

3 files changed

+8
-7
lines changed

dj_cqrs/mixins.py

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -118,11 +118,12 @@ def save(self, *args, **kwargs):
118118
def save_tracked_fields(self):
119119
if hasattr(self, FIELDS_TRACKER_FIELD_NAME):
120120
tracker = getattr(self, FIELDS_TRACKER_FIELD_NAME)
121-
if self._state.adding:
122-
data = tracker.changed_initial()
123-
else:
124-
data = tracker.changed()
125-
setattr(self, TRACKED_FIELDS_ATTR_NAME, data)
121+
if self.is_initial_cqrs_save:
122+
if self._state.adding:
123+
data = tracker.changed_initial()
124+
else:
125+
data = tracker.changed()
126+
setattr(self, TRACKED_FIELDS_ATTR_NAME, data)
126127

127128
@property
128129
def _update_cqrs_fields_default(self):

requirements/dev.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
Django >= 1.11.20
1+
Django>= 1.11.20,<4
22
pika>=1.0.0
33
kombu==4.6.*
44
ujson==3.0.0

tests/test_master/test_mixin.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -734,7 +734,7 @@ def test_transaction_instance_saved_once_simple_case(mocker):
734734

735735
mapper = (
736736
(i0.pk, 0, 'old', None),
737-
(i1.pk, 0, '2', '1'),
737+
(i1.pk, 0, '2', None),
738738
(i2.pk, 0, 'a', None),
739739
(i3.pk, 0, '.', None),
740740
(i0.pk, 1, 'new', 'old'),

0 commit comments

Comments
 (0)