From 26225b8bdfc9b0a75393feeb7ee2936cc8d57259 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?J=C3=A9rome=20Perrin?= <jerome@nexedi.com>
Date: Sun, 21 Apr 2024 22:08:59 +0900
Subject: [PATCH] hal_json_style: pylint py3

---
 .../erp5_hal_json_style/Base_callDialogMethod.py             | 2 +-
 .../erp5_hal_json_style/ERP5Document_getHateoas.py           | 5 +++--
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/bt5/erp5_hal_json_style/SkinTemplateItem/portal_skins/erp5_hal_json_style/Base_callDialogMethod.py b/bt5/erp5_hal_json_style/SkinTemplateItem/portal_skins/erp5_hal_json_style/Base_callDialogMethod.py
index 47436f30f9c..aba5653c4f0 100644
--- a/bt5/erp5_hal_json_style/SkinTemplateItem/portal_skins/erp5_hal_json_style/Base_callDialogMethod.py
+++ b/bt5/erp5_hal_json_style/SkinTemplateItem/portal_skins/erp5_hal_json_style/Base_callDialogMethod.py
@@ -216,7 +216,7 @@ if dialog_method != update_method and kw.get('deferred_style', 0):
 # At this point the 'dialog_method' should point to a form (if we are in report)
 # if we are not in Deferred mode - then it points to `Base_activateSimpleView`
 
-if True:
+if True:  # pylint:disable=using-constant-test
   if dialog_method != update_method:
     # When we are not executing the update action, we have to change the skin
     # manually,
diff --git a/bt5/erp5_hal_json_style/SkinTemplateItem/portal_skins/erp5_hal_json_style/ERP5Document_getHateoas.py b/bt5/erp5_hal_json_style/SkinTemplateItem/portal_skins/erp5_hal_json_style/ERP5Document_getHateoas.py
index 64297d0adbf..0814fd7f118 100644
--- a/bt5/erp5_hal_json_style/SkinTemplateItem/portal_skins/erp5_hal_json_style/ERP5Document_getHateoas.py
+++ b/bt5/erp5_hal_json_style/SkinTemplateItem/portal_skins/erp5_hal_json_style/ERP5Document_getHateoas.py
@@ -782,8 +782,9 @@ def renderField(traversed_document, field, form, value=MARKER, meta_type=None,
     if list_method is not None:
       selectKwargsForCallable(list_method, list_method_query_dict, REQUEST)
 
-    if (True):  # editable_column_list (we need that template fields resolution
-                # (issued by existence of `form_relative_url`) always kicks in
+    if True:  # pylint:disable=using-constant-test
+      # editable_column_list (we need that template fields resolution
+      # (issued by existence of `form_relative_url`) always kicks in
       extra_param_dict = {
         # in case of a dialog the form_id points to previous form, otherwise current form
         "form_id": REQUEST.get('form_id', form.id)
-- 
2.30.9