Support feedback page after answering a risk

reinhardt · reinhardt · commit 23577d26329e · 2025-02-14T11:20:11.000+01:00
Ref syslabcom/scrum#2857
diff --git a/docs/changes.rst b/docs/changes.rst
@@ -38,6 +38,10 @@ Changelog
 - Updated resources from proto and applied a number of markup fixes in a review session with Daniel
   [pilz]
 
+- Support feedback page after answering a risk
+  (`#2857 <https://github.com/syslabcom/scrum/issues/2857>`_)
+  [reinhardt]
+
 
 16.2.7 (2025-01-15)
 -------------------
diff --git a/src/euphorie/client/browser/configure.zcml b/src/euphorie/client/browser/configure.zcml
@@ -683,6 +683,13 @@
       permission="euphorie.client.ViewSurvey"
       layer="euphorie.client.interfaces.IClientSkinLayer"
       />
+  <browser:page
+      name="identification_feedback"
+      for="euphorie.client.model.Risk"
+      class=".risk.IdentificationFeedbackView"
+      permission="euphorie.client.ViewSurvey"
+      layer="euphorie.client.interfaces.IClientSkinLayer"
+      />
 
   <browser:page
       name="actionplan"
diff --git a/src/euphorie/client/browser/risk.py b/src/euphorie/client/browser/risk.py
@@ -1009,6 +1009,11 @@ def proceed_to_next(self, reply):
                     session_url=self.webhelpers.traversed_session.absolute_url()
                 )
                 return self.request.response.redirect(url)
+
+        elif _next == "next" and getattr(self.risk, "feedback_text", None):
+            url = self.context.absolute_url() + "/@@identification_feedback"
+            return self.request.response.redirect(url)
+
         elif _next in ("next", "skip"):
             target = self.next_question
             if target is None:
@@ -1119,6 +1124,13 @@ def calculatePriority(self, risk, reply):
         return self.context.priority
 
 
+class IdentificationFeedbackView(IdentificationView):
+    default_template = ViewPageTemplateFile("templates/risk_identification_feedback.pt")
+    def post(self):
+        reply = self.request.form
+        return self.proceed_to_next(reply)
+
+
 class ImageUpload(BrowserView):
     def redirect(self):
         return self.request.response.redirect(
diff --git a/src/euphorie/client/browser/templates/risk_identification_feedback.pt b/src/euphorie/client/browser/templates/risk_identification_feedback.pt
@@ -0,0 +1,55 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml"
+      xmlns:i18n="http://xml.zope.org/namespaces/i18n"
+      xmlns:meta="http://xml.zope.org/namespaces/meta"
+      xmlns:metal="http://xml.zope.org/namespaces/metal"
+      xmlns:tal="http://xml.zope.org/namespaces/tal"
+      meta:interpolation="true"
+      metal:use-macro="context/@@shell/macros/shell"
+      tal:define="
+        client nocall:request/client;
+        webhelpers nocall:context/@@webhelpers;
+      "
+      i18n:domain="euphorie"
+>
+
+  <body>
+    <metal:slot fill-slot="content"
+                tal:define="
+                  risk nocall:view/risk;
+                "
+    >
+      <tal:block replace="tile:statusmessages" />
+      <form class="pat-form pat-inject panel-style"
+            accept-charset="UTF-8"
+            action="${here/absolute_url}/@@${view/__name__}"
+            enctype="multipart/form-data"
+            method="post"
+            data-pat-inject="history: record; source: #step-2-topics; target: #step-2-topics &amp;&amp; source: #main-content; target: #main-content &amp;&amp; source: #toolbar; target: #toolbar &amp;&amp; source: #status; target: #status"
+      >
+        <a class="pat-scroll pat-scroll-animated"
+           hidden
+           href="#content-pane"
+           data-pat-scroll="selector: #content-pane; trigger: auto; offset: 0"
+        >Scroll to top</a>
+
+        <div class="pat-scroll-box"
+             id="content-pane"
+        >
+          <article tal:content="structure:risk/feedback_text|nothing"></article>
+        </div>
+
+        <p class="button-bar pat-bumper"
+           id="nav-bar"
+        >
+          <button class="pat-button continue"
+                  name="next"
+                  type="submit"
+                  value="skip"
+                  i18n:translate="label_continue"
+          >Continue</button>
+        </p>
+      </form>
+    </metal:slot>
+  </body>
+</html>
diff --git a/src/euphorie/content/risk.py b/src/euphorie/content/risk.py
@@ -236,6 +236,12 @@ class IRisk(model.Schema, IRichDescription, IBasic):
         default="low",
     )
 
+    feedback_text = HtmlText(
+        title=_("label_feedback_text", default="Feedback after answer"),
+        required=False,
+    )
+    directives.widget(feedback_text="plone.app.z3cform.wysiwyg.WysiwygFieldWidget")
+
     model.fieldset(
         "main_image",
         label=_("header_main_image", default="Main image"),
