Added SessionAnnotationInfoView

chris-adam · chris-adam · commit 8fa7b44ab412 · 2026-03-18T13:38:49.000+01:00
diff --git a/CHANGES.rst b/CHANGES.rst
@@ -43,6 +43,8 @@ Changelog
   [gbastien]
 - Use `@CachedProperty` for `FacetedSessionInfoViewlet.sessions` and `ItemSessionInfoViewlet.sessions`.
   [gbastien]
+- Added SessionAnnotationInfoView.
+  [chris-adam]
 
 1.0a2 (2026-02-06)
 ------------------
diff --git a/src/imio/esign/browser/actions.py b/src/imio/esign/browser/actions.py
@@ -1,12 +1,22 @@
 # -*- coding: utf-8 -*-
+from AccessControl import Unauthorized
+from html import escape
+from imio.esign.utils import persistent_to_native
 from imio.esign import _
 from imio.esign.adapters import ISignable
 from imio.esign.utils import add_files_to_session
 from imio.esign.utils import get_session_annotation
 from imio.esign.utils import remove_context_from_session
 from imio.esign.utils import remove_files_from_session
+from imio.helpers.content import uuidToObject
+from imio.helpers.security import check_zope_admin
 from plone import api
+from Products.CMFPlone.utils import safe_unicode
 from Products.Five import BrowserView
+from Products.Five.browser.pagetemplatefile import ViewPageTemplateFile
+
+import pprint
+import re
 
 
 class AddToSessionView(BrowserView):
@@ -126,3 +136,67 @@ def available(self):
         """Defines if the action is available or not."""
         annot = get_session_annotation()
         return self.context.UID() in annot.get("uids", {})
+
+
+class SessionAnnotationInfoView(BrowserView):
+    """Admin-only view displaying imio.esign session annotations for a specific context item."""
+
+    index = ViewPageTemplateFile("templates/session_annotation_info.pt")
+
+    def __call__(self):
+        if not check_zope_admin():
+            raise Unauthorized
+        return self.index()
+
+    def _uid_to_link(self, uid):
+        """Return an HTML link for an object UID, or the UID if not found."""
+        obj = uuidToObject(uid, unrestricted=True)
+        if obj is None:
+            return u"<span title='not found'>{}</span>".format(safe_unicode(uid))
+        url = escape(obj.absolute_url() + "/view", quote=True)
+        path = escape(u"/".join(obj.getPhysicalPath()))
+        title = escape(safe_unicode(getattr(obj, "title", "") or path))
+        return u"<a href='{}' title='{}'>{}</a>".format(url, path, title)
+
+    def _render_value(self, value, indent=u""):
+        """Render a value, replacing UIDs with links where possible."""
+        inner = indent + u"  "
+        if isinstance(value, dict):
+            if not value:
+                return u"{}"
+            lines = [u"{"]
+            for k, v in sorted(value.items()):
+                key = escape(safe_unicode(pprint.pformat(k)))
+                lines.append(u"{}{}: {},".format(inner, key, self._render_value(v, inner)))
+            lines.append(u"{}}}".format(indent))
+            return u"\n".join(lines)
+        elif isinstance(value, (list, tuple)):
+            if not value:
+                return u"[]"
+            lines = [u"["]
+            for item in value:
+                lines.append(u"{}{},".format(inner, self._render_value(item, inner)))
+            lines.append(u"{}]".format(indent))
+            return u"\n".join(lines)
+        elif isinstance(value, basestring) and re.match(r"^[0-9a-f]{32}$", value):
+            # Looks like a UUID
+            return self._uid_to_link(value)
+        else:
+            return escape(safe_unicode(pprint.pformat(value)))
+
+    @property
+    def esign_sessions(self):
+        """Return list of (session_id, session_data) for all sessions."""
+        annot = get_session_annotation()
+        c_uid = self.context.UID()
+        result = []
+        for session_id in annot['sessions']:
+            session = annot.get("sessions", {}).get(session_id)
+            # If any file in this session is in this context
+            if any(f['context_uid'] == c_uid for f in session['files']):
+                result.append((session_id, persistent_to_native(session)))
+        return sorted(result)
+
+    def esign_session_html(self, session_data):
+        """Renders esign session annot in HTML"""
+        return self._render_value(session_data)
diff --git a/src/imio/esign/browser/configure.zcml b/src/imio/esign/browser/configure.zcml
@@ -112,6 +112,13 @@
             allowed_attributes="available"
     />
 
+    <browser:page
+        for="*"
+        name="session-annotation-info"
+        class=".actions.SessionAnnotationInfoView"
+        permission="cmf.ManagePortal"
+        />
+
     <browser:page
             name="signing-users-csv"
             for="Products.CMFPlone.interfaces.IPloneSiteRoot"
diff --git a/src/imio/esign/browser/templates/session_annotation_info.pt b/src/imio/esign/browser/templates/session_annotation_info.pt
@@ -0,0 +1,16 @@
+<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en"
+    xmlns:tal="http://xml.zope.org/namespaces/tal"
+    xmlns:metal="http://xml.zope.org/namespaces/metal"
+    xmlns:i18n="http://xml.zope.org/namespaces/i18n"
+    metal:use-macro="context/main_template/macros/master"
+    i18n:domain="imio.esign">
+    <body>
+      <metal:bodytext fill-slot="content-core">
+          <tal:esign tal:repeat="session view/esign_sessions">
+            <h2 tal:content="python:'imio.esign session (id: {})'.format(session[0])"></h2>
+            <pre style="background: #f5f5f5; color: #333; padding: 1em; overflow: auto;"
+                 tal:content="structure python:view.esign_session_html(session[1])" />
+          </tal:esign>
+      </metal:bodytext>
+    </body>
+</html>
diff --git a/src/imio/esign/tests/test_actions.py b/src/imio/esign/tests/test_actions.py
@@ -1,10 +1,13 @@
 # -*- coding: utf-8 -*-
 """actions tests for this package."""
+from AccessControl import Unauthorized
 from collective.iconifiedcategory.utils import calculate_category_id
+from imio.esign.browser.actions import SessionAnnotationInfoView
 from imio.esign.testing import IMIO_ESIGN_INTEGRATION_TESTING
 from imio.esign.utils import add_files_to_session
 from imio.esign.utils import get_session_annotation
 from plone import api
+from plone.app.testing import login
 from plone.app.testing import setRoles
 from plone.app.testing import TEST_USER_ID
 from plone.namedfile.file import NamedBlobFile
@@ -17,9 +20,25 @@
 import unittest
 
 
+try:
+    from html import unescape
+except ImportError:  # Python 2
+    from HTMLParser import HTMLParser
+    unescape = HTMLParser().unescape
+
+try:
+    string_types = (basestring,)
+except NameError:
+    string_types = (str,)
+
+
 class BaseRemoveFromSession(unittest.TestCase):
     """Base class to centralize setUp."""
 
+
+class TestRemoveItemFromSessionView(unittest.TestCase):
+    """Test RemoveItemFromSessionView browser view."""
+
     layer = IMIO_ESIGN_INTEGRATION_TESTING
 
     def setUp(self):
@@ -152,3 +171,173 @@ def test_available(self):
         view()
         self.assertFalse(annot["sessions"])
         self.assertFalse(view.available())
+
+
+class TestSessionAnnotationInfoView(unittest.TestCase):
+    """Test SessionAnnotationInfoView"""
+
+    layer = IMIO_ESIGN_INTEGRATION_TESTING
+
+    def setUp(self):
+        self.portal = self.layer["portal"]
+        setRoles(self.portal, TEST_USER_ID, ["Manager"])
+        at_folder = api.content.create(
+            container=self.portal,
+            id="annexes_types",
+            title="Annexes Types",
+            type="ContentCategoryConfiguration",
+            exclude_from_nav=True,
+        )
+        category_group = api.content.create(
+            type="ContentCategoryGroup",
+            title="Annexes",
+            container=at_folder,
+            id="annexes",
+        )
+        icon_path = os.path.join(os.path.dirname(collective.iconifiedcategory.__file__), "tests", "icône1.png")
+        with open(icon_path, "rb") as fl:
+            api.content.create(
+                type="ContentCategory",
+                title="To sign",
+                container=category_group,
+                icon=NamedBlobImage(fl.read(), filename=u"icône1.png"),
+                id="to_sign",
+                predefined_title="To be signed",
+                to_sign=True,
+                show_preview=False,
+            )
+        self.folder = api.content.create(
+            container=self.portal, type="Folder", id="test_session_folder", title="Test Session Folder"
+        )
+        tests_dir = os.path.dirname(__file__)
+        self.annexes = []
+        for i in range(2):
+            with open(os.path.join(tests_dir, "annex1.pdf"), "rb") as f:
+                annex = api.content.create(
+                    container=self.folder,
+                    type="annex",
+                    id="annex{}".format(i),
+                    title=u"Annex {}".format(i),
+                    content_category=calculate_category_id(self.portal["annexes_types"]["annexes"]["to_sign"]),
+                    scan_id="012345600000{:02d}".format(i),
+                    file=NamedBlobFile(data=f.read(), filename=u"annex{}.pdf".format(i), contentType="application/pdf"),
+                )
+                self.annexes.append(annex)
+        self.signers = [
+            ("user1", "user1@sign.com", u"User 1", u"Position 1"),
+            ("user2", "user2@sign.com", u"User 2", u"Position 2"),
+        ]
+        self.view = SessionAnnotationInfoView(self.folder, self.portal.REQUEST)
+
+    def test_call(self):
+        setRoles(self.portal, TEST_USER_ID, ["Member"])
+        with self.assertRaises(Unauthorized):
+            self.view()
+        login(self.layer["app"], "admin")
+        self.assertIsInstance(self.view(), string_types)
+
+    def test_render_value(self):
+        # Dict
+        self.assertEqual(self.view._render_value({}), u"{}")
+        self.assertEqual(
+            self.view._render_value({"key": "val"}),
+            u"{\n  &#x27;key&#x27;: &#x27;val&#x27;,\n}",
+        )
+
+        # Indentation: nested value increases indent level
+        self.assertEqual(
+            self.view._render_value({"key": ["a"]}),
+            u"{\n  &#x27;key&#x27;: [\n    &#x27;a&#x27;,\n  ],\n}",
+        )
+
+        # List
+        self.assertEqual(self.view._render_value([]), u"[]")
+        self.assertEqual(
+            self.view._render_value(["a", "b"]),
+            u"[\n  &#x27;a&#x27;,\n  &#x27;b&#x27;,\n]",
+        )
+
+        # Tuple
+        self.assertEqual(self.view._render_value(()), u"[]")
+
+        # String
+        self.assertEqual(self.view._render_value(u"hello"), u"u&#x27;hello&#x27;")
+
+    def test_uid_to_link(self):
+        uid = self.folder.UID()
+        result = self.view._uid_to_link(uid)
+        self.assertEqual(
+            result,
+            u"<a href='http://nohost/plone/test_session_folder/view' title='/plone/test_session_folder'>Test Session Folder</a>",
+        )
+
+        result = self.view._uid_to_link(u"a" * 32)
+        self.assertEqual(
+            result,
+            u"<span title='not found'>aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa</span>",
+        )
+
+    def test_esign_sessions(self):
+        uids = [a.UID() for a in self.annexes]
+        add_files_to_session(self.signers, uids, title=u"[ia.parapheo] Session {sign_id}")
+        view = SessionAnnotationInfoView(self.folder, self.portal.REQUEST)
+
+        esign_sessions = view.esign_sessions
+        self.assertEqual(len(esign_sessions), 1)
+        esign_session = esign_sessions[0]
+        self.assertIsInstance(esign_session, tuple)
+        self.assertEqual(esign_session[0], 0)
+
+        self.assertEqual(
+            unescape(view.esign_session_html(esign_session[1])),
+            u"""{{
+  'acroform': True,
+  'client_id': '0123456',
+  'discriminators': [],
+  'files': [
+    {{
+      'context_uid': <a href='http://nohost/plone/test_session_folder/view' title='/plone/test_session_folder'>Test Session Folder</a>,
+      'filename': u'annex0.pdf',
+      'scan_id': '01234560000000',
+      'status': '',
+      'title': u'Annex 0',
+      'uid': <a href='http://nohost/plone/test_session_folder/annex0/view' title='/plone/test_session_folder/annex0'>Annex 0</a>,
+    }},
+    {{
+      'context_uid': <a href='http://nohost/plone/test_session_folder/view' title='/plone/test_session_folder'>Test Session Folder</a>,
+      'filename': u'annex1.pdf',
+      'scan_id': '01234560000001',
+      'status': '',
+      'title': u'Annex 1',
+      'uid': <a href='http://nohost/plone/test_session_folder/annex1/view' title='/plone/test_session_folder/annex1'>Annex 1</a>,
+    }},
+  ],
+  'last_update': {},
+  'returns': [],
+  'seal': None,
+  'sign_id': '012345600000',
+  'sign_url': None,
+  'signers': [
+    {{
+      'email': 'user1@sign.com',
+      'fullname': u'User 1',
+      'position': u'Position 1',
+      'status': '',
+      'userid': 'user1',
+    }},
+    {{
+      'email': 'user2@sign.com',
+      'fullname': u'User 2',
+      'position': u'Position 2',
+      'status': '',
+      'userid': 'user2',
+    }},
+  ],
+  'size': 13936,
+  'state': 'draft',
+  'title': u'[ia.parapheo] Session 012345600000',
+  'watchers': [],
+}}""".format(
+                repr(esign_session[1]['last_update']),
+            ),
+        )
diff --git a/src/imio/esign/utils.py b/src/imio/esign/utils.py
@@ -500,6 +500,15 @@ def get_suid_from_uuid(uid):
     return shortuid_encode_id(uid, separator="-", block_size=5)
 
 
+def persistent_to_native(value):
+    """Convert persistent object to native object recursively."""
+    if isinstance(value, (PersistentMapping, dict)):
+        return {k: persistent_to_native(v) for k, v in value.items()}
+    elif isinstance(value, (PersistentList, list, tuple)):
+        return [persistent_to_native(v) for v in value]
+    return value
+
+
 def get_state_description(state):
     """
     Get a human readable description for a given session state.
