From 06417b1ff9e716fa447fb051244b1686b7154133 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Laur=C3=A9line=20Gu=C3=A9rin?= <zebuline@entrouvert.com>
Date: Thu, 6 Feb 2020 16:57:12 +0100
Subject: [PATCH] cells: take a snapshot when a link is added to a list of
 links cell

---
 combo/manager/views.py | 1 +
 tests/test_manager.py  | 4 ++++
 2 files changed, 5 insertions(+)

diff --git a/combo/manager/views.py b/combo/manager/views.py
index 613d401d..c8508d69 100644
--- a/combo/manager/views.py
+++ b/combo/manager/views.py
@@ -631,6 +631,7 @@ class PageListCellAddLinkView(CreateView):
             form.instance.order = max(orders)+1
         else:
             form.instance.order = 1
+        PageSnapshot.take(self.cell.page, request=self.request, comment=_('changed cell "%s"') % self.cell)
         return super(PageListCellAddLinkView, self).form_valid(form)
 
     def get_success_url(self):
diff --git a/tests/test_manager.py b/tests/test_manager.py
index 982a8293..3cc30646 100644
--- a/tests/test_manager.py
+++ b/tests/test_manager.py
@@ -1515,6 +1515,7 @@ def test_add_edit_delete_list_link_item(app, admin_user):
     cell = LinkListCell.objects.create(order=0, placeholder='content', page=page)
     app = login(app)
     resp = app.get('/manage/pages/%s/' % page.pk)
+    assert PageSnapshot.objects.count() == 0
 
     resp = resp.click(href='.*/add-link/link$')
     resp.forms[0]['title'] = 'Hello world'
@@ -1528,6 +1529,7 @@ def test_add_edit_delete_list_link_item(app, admin_user):
     assert item.url == 'http://example.com'
     assert item.page == page
     assert item.placeholder == cell.link_placeholder
+    assert PageSnapshot.objects.count() == 1
 
     resp = resp.follow()
     resp = resp.click(href='.*/link/%s/$' % item.get_reference())
@@ -1540,6 +1542,7 @@ def test_add_edit_delete_list_link_item(app, admin_user):
     item.refresh_from_db()
     assert item.title == 'Hello world 2'
     assert item.url == 'http://example2.com'
+    assert PageSnapshot.objects.count() == 2
 
     resp = resp.follow()
     resp = resp.click(href='.*/link/%s/delete' % item.get_reference())
@@ -1547,6 +1550,7 @@ def test_add_edit_delete_list_link_item(app, admin_user):
     assert resp.status_int == 302
     assert resp.location.endswith('/manage/pages/%s/#cell-%s' % (page.pk, cell.get_reference()))
     assert LinkCell.objects.count() == 0
+    assert PageSnapshot.objects.count() == 3
 
 
 def test_edit_link_list_order(app, admin_user):