From ee10efd6247b53bb9dd053ccb47ea4325c64cd33 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Iv=C3=A1n=20Todorovich?= Date: Sat, 15 Sep 2018 13:01:26 -0300 Subject: [PATCH] FIX hooks Because of default='draft' on the state field, state IS NULL wasn't selecting anything. Left the condition in case this is changed in the future. --- document_page_approval/hooks.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/document_page_approval/hooks.py b/document_page_approval/hooks.py index 212e6583..973032d6 100644 --- a/document_page_approval/hooks.py +++ b/document_page_approval/hooks.py @@ -1,20 +1,24 @@ # Copyright 2018 Ivan Todorovich () # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html). +import logging +_logger = logging.getLogger(__name__) def post_init_hook(cr, registry): # pragma: no cover # Set all pre-existing pages history to approved + _logger.info('Setting history to approved.') cr.execute(""" UPDATE document_page_history SET state='approved', approved_uid=create_uid, approved_date=create_date - WHERE state IS NULL + WHERE state IS NULL OR state = 'draft' """) def uninstall_hook(cr, registry): # pragma: no cover # Remove unapproved pages + _logger.info('Deleting unapproved Change Requests.') cr.execute( "DELETE FROM document_page_history " "WHERE state != 'approved'"