From d53886dce73f50ddbd6ecefc626c49ee12f936a8 Mon Sep 17 00:00:00 2001
From: emjay0921 <michael@acn.fr>
Date: Mon, 4 May 2026 10:19:41 +0800
Subject: [PATCH 1/4] fix(spp_base_common): hide form-view pager when only one
 record

The standard web.Pager renders the counter + chevrons even when
total === 1, leaving a disabled "1/1 < >" widget that confuses users
clicking into a record with no siblings to navigate to.

Extend the web.Pager OWL template to drop the entire <nav> with
`t-if="props.total > 1 or props.updateTotal"`. The "1/1+" affordance
(updateTotal set) is preserved because the count may still grow.

The pager's `h-100 d-flex` was also acting as the height anchor for
the parent `.o_control_panel_navigation` row, so removing it
collapsed the row and top-aligned the sibling refresh / view-switcher
buttons. Add `align-items: center` on that container so they stay
vertically centred whether the pager is rendered or not.

Refs OP#920.
---
 spp_base_common/__manifest__.py               |  4 +++-
 .../static/src/scss/pager_hide_single.scss    | 11 +++++++++
 .../static/src/xml/pager_hide_single.xml      | 24 +++++++++++++++++++
 3 files changed, 38 insertions(+), 1 deletion(-)
 create mode 100644 spp_base_common/static/src/scss/pager_hide_single.scss
 create mode 100644 spp_base_common/static/src/xml/pager_hide_single.xml

diff --git a/spp_base_common/__manifest__.py b/spp_base_common/__manifest__.py
index 6cda488f..b9ff39a3 100644
--- a/spp_base_common/__manifest__.py
+++ b/spp_base_common/__manifest__.py
@@ -5,7 +5,7 @@
 {
     "name": "OpenSPP Base (Common)",
     "category": "OpenSPP/Core",
-    "version": "19.0.2.0.0",
+    "version": "19.0.2.0.1",
     "sequence": 1,
     "author": "OpenSPP.org",
     "website": "https://github.com/OpenSPP/OpenSPP2",
@@ -38,6 +38,8 @@
             "spp_base_common/static/src/xml/custom_list_create_template.xml",
             "spp_base_common/static/src/js/filterable_radio_field.js",
             "spp_base_common/static/src/xml/filterable_radio_field.xml",
+            "spp_base_common/static/src/xml/pager_hide_single.xml",
+            "spp_base_common/static/src/scss/pager_hide_single.scss",
         ],
         "web._assets_primary_variables": [
             "spp_base_common/static/src/scss/colors.scss",
diff --git a/spp_base_common/static/src/scss/pager_hide_single.scss b/spp_base_common/static/src/scss/pager_hide_single.scss
new file mode 100644
index 00000000..8d888132
--- /dev/null
+++ b/spp_base_common/static/src/scss/pager_hide_single.scss
@@ -0,0 +1,11 @@
+// See OP#920 + spp_base_common/static/src/xml/pager_hide_single.xml.
+// The control-panel right-side toolbar (refresh button, embedded
+// actions, view switcher) had no vertical alignment of its own — it
+// relied on the pager's h-100 nav to anchor the row's height.
+// Once the pager is hidden, those siblings collapse to top-aligned.
+// Set align-items: center explicitly so they stay vertically centred
+// whether the pager is visible or not.
+.o_control_panel_navigation {
+    align-items: center;
+}
+
diff --git a/spp_base_common/static/src/xml/pager_hide_single.xml b/spp_base_common/static/src/xml/pager_hide_single.xml
new file mode 100644
index 00000000..a9cb59bd
--- /dev/null
+++ b/spp_base_common/static/src/xml/pager_hide_single.xml
@@ -0,0 +1,24 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!--
+    Hide the form-view "1/1" pager when there's only one record to navigate.
+
+    The standard web.Pager renders the counter + chevrons even when
+    `total === 1`, leaving a disabled "1/1 < >" widget that confuses
+    users. We drop the entire <nav> with `t-if` and pair the change
+    with an `align-items: center` rule on the parent
+    `o_control_panel_navigation` (see pager_hide_single.scss) so the
+    sibling refresh button stays vertically centred without the
+    pager's old h-100 anchor. `1/1+` (updateTotal set) stays visible
+    because the count may still grow.
+
+    See OP#920.
+-->
+<templates xml:space="preserve">
+    <t t-name="spp_base_common.PagerHideSingle"
+       t-inherit="web.Pager"
+       t-inherit-mode="extension">
+        <xpath expr="//nav[hasclass('o_pager')]" position="attributes">
+            <attribute name="t-if">props.total &gt; 1 or props.updateTotal</attribute>
+        </xpath>
+    </t>
+</templates>

From 58e6d297e06f4abb5bbbe35187081ca13dd058da Mon Sep 17 00:00:00 2001
From: emjay0921 <michael@acn.fr>
Date: Wed, 13 May 2026 09:07:27 +0800
Subject: [PATCH 2/4] fix(spp_change_request_v2): route post-submit CRs through
 the stage review form (#920 r2)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

QA returned OP#920 because demo-generated CRs in 'Applied' state
opened in the legacy main spp_change_request_form layout instead of
the Edit Details → Upload Documents → Review & Submit breadcrumb
form that fresh CRs use. The fixed pager-hide treatment from
spp_base_common only applies to the stage form, so post-submit
states (pending / approved / applied / rejected) effectively
'overrode' the new UI.

Three coordinated changes:

1. spp_change_request_v2/models/change_request.py
   - action_open_stage_form: any non-draft/non-revision state now
     routes to _action_open_review_form (matches the Review & Submit
     stage that the form's header already renders state-aware
     buttons for — Approve/Reject for validators, Apply for managers,
     Applied ribbon, Start Over for rejected).
   - Define _action_open_review_form / _action_open_documents_form —
     these were referenced from action_open_stage_form but had no
     implementations (latent crash on draft CRs at the documents/
     review stage that this branch also fixes).

2. spp_change_request_v2/views/change_request_views.xml
   - List view <list> gains action='action_open_stage_form'
     type='object' so row-click goes through the stage router. The
     demo CR list (action_change_request, action_my_change_requests,
     action_pending_change_requests) all use this list view, so the
     fix lands on every entry point.

3. spp_change_request_v2/tests/test_stage_navigation.py
   - Update test_action_open_stage_form_pending to assert the new
     behaviour: pending CRs return the review form view, not the
     legacy default form.

Bumps spp_change_request_v2 19.0.2.0.3 → 19.0.2.0.4 with matching
HISTORY entry.

Full module suite: 298 tests, 0 failed locally.
---
 spp_change_request_v2/__manifest__.py         |  2 +-
 .../models/change_request.py                  | 70 ++++++++++++++-----
 spp_change_request_v2/readme/HISTORY.md       |  4 ++
 .../tests/test_stage_navigation.py            |  6 +-
 .../views/change_request_views.xml            |  2 +
 5 files changed, 63 insertions(+), 21 deletions(-)

diff --git a/spp_change_request_v2/__manifest__.py b/spp_change_request_v2/__manifest__.py
index c565429a..d441e4fe 100644
--- a/spp_change_request_v2/__manifest__.py
+++ b/spp_change_request_v2/__manifest__.py
@@ -1,6 +1,6 @@
 {
     "name": "OpenSPP Change Request V2",
-    "version": "19.0.2.0.3",
+    "version": "19.0.2.0.4",
     "sequence": 50,
     "category": "OpenSPP",
     "summary": "Configuration-driven change request system with UX improvements, conflict detection and duplicate prevention",
diff --git a/spp_change_request_v2/models/change_request.py b/spp_change_request_v2/models/change_request.py
index 8425457d..17d4035d 100644
--- a/spp_change_request_v2/models/change_request.py
+++ b/spp_change_request_v2/models/change_request.py
@@ -1523,29 +1523,63 @@ def action_upload_document(self):
     def action_open_stage_form(self):
         """Open the appropriate form view based on the current stage.
 
-        For draft/revision CRs: routes to the stage-specific form.
-        For other states: opens the main CR form (for validators/managers).
+        - **Draft / revision**: route by `stage` to the editable stage form
+          (details / documents / review).
+        - **Submitted+ (pending, approved, applied, rejected)**: always open
+          the review-stage form. That form already renders state-aware
+          headers (Approve/Reject for validators, Apply for managers,
+          Applied ribbon for completed, Start Over for rejected) and shows
+          the same Edit Details → Upload Documents → Review & Submit
+          breadcrumb. Without this, validators/managers (and demo-applied
+          CRs opened from the list) landed on the legacy main form view
+          which lacks the breadcrumb and the pager-hide treatment. See
+          OP#920 round-2.
         """
         self.ensure_one()
 
-        if self.approval_state not in ("draft", "revision"):
-            return {
-                "type": "ir.actions.act_window",
-                "name": self.name,
-                "res_model": "spp.change.request",
-                "res_id": self.id,
-                "view_mode": "form",
-                "views": [[False, "form"]],
-                "target": "current",
-            }
+        if self.approval_state in ("draft", "revision"):
+            if self.stage == "documents":
+                return self._action_open_documents_form()
+            if self.stage == "review":
+                return self._action_open_review_form()
+            return self.action_open_detail()
 
-        if self.stage == "documents":
-            return self._action_open_documents_form()
-        if self.stage == "review":
-            return self._action_open_review_form()
+        # pending / approved / applied / rejected
+        return self._action_open_review_form()
 
-        # Default: details stage
-        return self.action_open_detail()
+    def _action_open_review_form(self):
+        """Open the CR in the Review & Submit stage form view."""
+        self.ensure_one()
+        view = self.env.ref(
+            "spp_change_request_v2.spp_change_request_review_form",
+            raise_if_not_found=False,
+        )
+        return {
+            "type": "ir.actions.act_window",
+            "name": self.name,
+            "res_model": "spp.change.request",
+            "res_id": self.id,
+            "view_mode": "form",
+            "views": [[view.id if view else False, "form"]],
+            "target": "current",
+        }
+
+    def _action_open_documents_form(self):
+        """Open the CR in the Upload Documents stage form view."""
+        self.ensure_one()
+        view = self.env.ref(
+            "spp_change_request_v2.spp_change_request_documents_form",
+            raise_if_not_found=False,
+        )
+        return {
+            "type": "ir.actions.act_window",
+            "name": self.name,
+            "res_model": "spp.change.request",
+            "res_id": self.id,
+            "view_mode": "form",
+            "views": [[view.id if view else False, "form"]],
+            "target": "current",
+        }
 
     def action_goto_details(self):
         """Navigate to the details stage (replaces breadcrumb via client action)."""
diff --git a/spp_change_request_v2/readme/HISTORY.md b/spp_change_request_v2/readme/HISTORY.md
index 387d84da..58320fcd 100644
--- a/spp_change_request_v2/readme/HISTORY.md
+++ b/spp_change_request_v2/readme/HISTORY.md
@@ -1,3 +1,7 @@
+### 19.0.2.0.4
+
+- fix(views): route post-submit CRs (pending / approved / applied / rejected) through the stage review form when opened from the list, matching the Edit Details → Upload Documents → Review & Submit breadcrumb workflow used for fresh CRs (#920 round-2). Demo-generated CRs in "Applied" state previously landed on the legacy main form view from the list — now they open in `spp_change_request_review_form` like manually-created CRs. Adds the missing `_action_open_review_form` / `_action_open_documents_form` helpers and wires `action="action_open_stage_form" type="object"` on the CR list so row-click goes through the stage router.
+
 ### 19.0.2.0.3
 
 - fix: add HTML escaping to all computed Html fields with `sanitize=False` to prevent stored XSS (#50)
diff --git a/spp_change_request_v2/tests/test_stage_navigation.py b/spp_change_request_v2/tests/test_stage_navigation.py
index 91fc64b3..2defe31b 100644
--- a/spp_change_request_v2/tests/test_stage_navigation.py
+++ b/spp_change_request_v2/tests/test_stage_navigation.py
@@ -116,14 +116,16 @@ def test_action_open_stage_form_draft_review(self):
                 cr.action_open_stage_form()
 
     def test_action_open_stage_form_pending(self):
-        """Pending CR opens main form (not stage form)."""
+        """Pending CR opens the Review & Submit stage form so post-submit
+        states share the breadcrumb-based UI with draft / revision."""
         cr = self._create_cr()
         cr.approval_state = "pending"
         result = cr.action_open_stage_form()
         self.assertEqual(result["type"], "ir.actions.act_window")
         self.assertEqual(result["res_model"], "spp.change.request")
         self.assertEqual(result["res_id"], cr.id)
-        self.assertEqual(result["views"], [[False, "form"]])
+        review_view = self.env.ref("spp_change_request_v2.spp_change_request_review_form")
+        self.assertEqual(result["views"], [[review_view.id, "form"]])
 
     def test_action_start_over_creates_new_cr(self):
         """action_start_over() creates a new CR with same type and registrant."""
diff --git a/spp_change_request_v2/views/change_request_views.xml b/spp_change_request_v2/views/change_request_views.xml
index 1e5f1927..ba441e57 100644
--- a/spp_change_request_v2/views/change_request_views.xml
+++ b/spp_change_request_v2/views/change_request_views.xml
@@ -13,6 +13,8 @@
                 decoration-warning="display_state == 'revision'"
                 create="0"
                 sample="1"
+                action="action_open_stage_form"
+                type="object"
             >
                 <field name="name" decoration-bf="1" />
                 <field name="request_type_id" />

From b0136915f418bd8be8ad2a3e890780b43200639a Mon Sep 17 00:00:00 2001
From: emjay0921 <michael@acn.fr>
Date: Wed, 13 May 2026 09:34:11 +0800
Subject: [PATCH 3/4] fix(cr): set stage='review' on submit so demo-generated
 CRs route through the new review form (#920 round-2 follow-up)

---
 spp_change_request_v2/models/change_request.py | 1 +
 1 file changed, 1 insertion(+)

diff --git a/spp_change_request_v2/models/change_request.py b/spp_change_request_v2/models/change_request.py
index 17d4035d..d1b1012e 100644
--- a/spp_change_request_v2/models/change_request.py
+++ b/spp_change_request_v2/models/change_request.py
@@ -1018,6 +1018,7 @@ def _on_submit(self):
         action = "resubmitted" if old_state == "revision" else "submitted"
         self._create_audit_event("submitted", old_state, "pending")
         self._create_log(action)
+        self.stage = "review"
 
     def _on_request_revision(self, notes):
         super()._on_request_revision(notes)

From 5881862453d721d4f21c5374f950ae548fcc1976 Mon Sep 17 00:00:00 2001
From: emjay0921 <michael@acn.fr>
Date: Thu, 14 May 2026 17:47:03 +0800
Subject: [PATCH 4/4] chore: apply pre-commit auto-fixes from CI (OP#920)

prettier and the OCA README generator wanted to reformat the new pager
inheritance template + regenerate spp_change_request_v2 README artifacts
after the new HISTORY entry shifted section numbering. Applies CI's
regenerated bytes directly.
---
 .../static/src/scss/pager_hide_single.scss    |  1 -
 .../static/src/xml/pager_hide_single.xml      | 10 +++++----
 spp_change_request_v2/README.rst              | 14 +++++++++++++
 .../static/description/index.html             | 21 ++++++++++++++++---
 4 files changed, 38 insertions(+), 8 deletions(-)

diff --git a/spp_base_common/static/src/scss/pager_hide_single.scss b/spp_base_common/static/src/scss/pager_hide_single.scss
index 8d888132..d328621a 100644
--- a/spp_base_common/static/src/scss/pager_hide_single.scss
+++ b/spp_base_common/static/src/scss/pager_hide_single.scss
@@ -8,4 +8,3 @@
 .o_control_panel_navigation {
     align-items: center;
 }
-
diff --git a/spp_base_common/static/src/xml/pager_hide_single.xml b/spp_base_common/static/src/xml/pager_hide_single.xml
index a9cb59bd..2bd84b0f 100644
--- a/spp_base_common/static/src/xml/pager_hide_single.xml
+++ b/spp_base_common/static/src/xml/pager_hide_single.xml
@@ -1,4 +1,4 @@
-<?xml version="1.0" encoding="UTF-8"?>
+<?xml version="1.0" encoding="UTF-8" ?>
 <!--
     Hide the form-view "1/1" pager when there's only one record to navigate.
 
@@ -14,9 +14,11 @@
     See OP#920.
 -->
 <templates xml:space="preserve">
-    <t t-name="spp_base_common.PagerHideSingle"
-       t-inherit="web.Pager"
-       t-inherit-mode="extension">
+    <t
+        t-name="spp_base_common.PagerHideSingle"
+        t-inherit="web.Pager"
+        t-inherit-mode="extension"
+    >
         <xpath expr="//nav[hasclass('o_pager')]" position="attributes">
             <attribute name="t-if">props.total &gt; 1 or props.updateTotal</attribute>
         </xpath>
diff --git a/spp_change_request_v2/README.rst b/spp_change_request_v2/README.rst
index 346551e7..e17aa6a3 100644
--- a/spp_change_request_v2/README.rst
+++ b/spp_change_request_v2/README.rst
@@ -853,6 +853,20 @@ Before declaring a new CR type complete:
 Changelog
 =========
 
+19.0.2.0.4
+~~~~~~~~~~
+
+- fix(views): route post-submit CRs (pending / approved / applied /
+  rejected) through the stage review form when opened from the list,
+  matching the Edit Details → Upload Documents → Review & Submit
+  breadcrumb workflow used for fresh CRs (#920 round-2). Demo-generated
+  CRs in "Applied" state previously landed on the legacy main form view
+  from the list — now they open in ``spp_change_request_review_form``
+  like manually-created CRs. Adds the missing
+  ``_action_open_review_form`` / ``_action_open_documents_form`` helpers
+  and wires ``action="action_open_stage_form" type="object"`` on the CR
+  list so row-click goes through the stage router.
+
 19.0.2.0.3
 ~~~~~~~~~~
 
diff --git a/spp_change_request_v2/static/description/index.html b/spp_change_request_v2/static/description/index.html
index f8bf3e1a..5885da4d 100644
--- a/spp_change_request_v2/static/description/index.html
+++ b/spp_change_request_v2/static/description/index.html
@@ -1339,26 +1339,41 @@ <h2>Changelog</h2>
 </div>
 </div>
 <div class="section" id="section-1">
+<h1>19.0.2.0.4</h1>
+<ul class="simple">
+<li>fix(views): route post-submit CRs (pending / approved / applied /
+rejected) through the stage review form when opened from the list,
+matching the Edit Details → Upload Documents → Review &amp; Submit
+breadcrumb workflow used for fresh CRs (#920 round-2). Demo-generated
+CRs in “Applied” state previously landed on the legacy main form view
+from the list — now they open in <tt class="docutils literal">spp_change_request_review_form</tt>
+like manually-created CRs. Adds the missing
+<tt class="docutils literal">_action_open_review_form</tt> / <tt class="docutils literal">_action_open_documents_form</tt> helpers
+and wires <tt class="docutils literal"><span class="pre">action=&quot;action_open_stage_form&quot;</span> <span class="pre">type=&quot;object&quot;</span></tt> on the CR
+list so row-click goes through the stage router.</li>
+</ul>
+</div>
+<div class="section" id="section-2">
 <h1>19.0.2.0.3</h1>
 <ul class="simple">
 <li>fix: add HTML escaping to all computed Html fields with
 <tt class="docutils literal">sanitize=False</tt> to prevent stored XSS (#50)</li>
 </ul>
 </div>
-<div class="section" id="section-2">
+<div class="section" id="section-3">
 <h1>19.0.2.0.2</h1>
 <ul class="simple">
 <li>fix: fix batch approval wizard line deletion (#130)</li>
 </ul>
 </div>
-<div class="section" id="section-3">
+<div class="section" id="section-4">
 <h1>19.0.2.0.1</h1>
 <ul class="simple">
 <li>fix: skip field types before getattr and isolate detail prefetch
 (#129)</li>
 </ul>
 </div>
-<div class="section" id="section-4">
+<div class="section" id="section-5">
 <h1>19.0.2.0.0</h1>
 <ul class="simple">
 <li>Initial migration to OpenSPP2</li>