Fix: [AEA-6014] - show relevant details with back button (#1575)

## Summary

https://nhsd-jira.digital.nhs.uk/browse/AEA-6014
https://nhsd-jira.digital.nhs.uk/browse/AEA-6013



- Routine Change


### Details
Made it so you dont get autocomplete options on the basic details form
Made it so that when you do any of the searches, thats the only details
that are saved and populated when you press the back button

Author: jonathanwelch1-nhs

packages/cpt-ui/__tests__/EpsSearchForAPrescription.test.tsx

@@ -7,6 +7,7 @@ import SearchForAPrescription from "@/pages/SearchPrescriptionPage"
 import {HERO_TEXT} from "@/constants/ui-strings/SearchForAPrescriptionStrings"
 import {AuthContext, AuthContextType} from "@/context/AuthProvider"
 import {AccessContext} from "@/context/AccessProvider"
+import {SearchProvider} from "@/context/SearchProvider"
 
 // Default mock values for contexts
 const defaultAuthContext: AuthContextType = {
@@ -45,7 +46,9 @@ const renderWithProviders = (
     <MemoryRouter>
       <AuthContext.Provider value={authContext}>
         <AccessContext.Provider value={accessContext}>
-          {ui}
+          <SearchProvider>
+            {ui}
+          </SearchProvider>
         </AccessContext.Provider>
       </AuthContext.Provider>
     </MemoryRouter>

packages/cpt-ui/__tests__/PrescriptionListTable.test.tsx

@@ -431,10 +431,8 @@ describe("PrescriptionsListTable", () => {
       expect(prescriptionButton).toBeInTheDocument()
 
       fireEvent.click(prescriptionButton)
-      expect(mockSetAllSearchParameters).toHaveBeenCalledWith({
-        prescriptionId: "C0C757-A83008-C2D93O",
-        issueNumber: "1"
-      })
+      expect(mockSetPrescriptionId).toHaveBeenCalledWith("C0C757-A83008-C2D93O")
+      expect(mockSetIssueNumber).toHaveBeenCalledWith("1")
     })
 
   })

packages/cpt-ui/src/components/prescriptionList/PrescriptionsListTable.tsx

@@ -8,7 +8,6 @@ import {PRESCRIPTION_LIST_TABLE_TEXT} from "@/constants/ui-strings/PrescriptionL
 import {Link} from "react-router-dom"
 import {FRONTEND_PATHS} from "@/constants/environment"
 import {useSearchContext} from "@/context/SearchProvider"
-import {useNavigationContext} from "@/context/NavigationProvider"
 
 export interface PrescriptionsListTableProps {
   textContent: PrescriptionsListStrings;
@@ -26,7 +25,6 @@ const PrescriptionsListTable = ({
   const initialSortConfig: SortConfig = {key: "issueDate", direction: null}
   const currentTabId: TabId = textContent.testid as TabId
   const searchContext = useSearchContext()
-  const navigationContext = useNavigationContext()
 
   // all tabs have own key in state object so each tab can be sorted individually
   const [allSortConfigs, setAllSortConfigs] = React.useState<
@@ -144,32 +142,12 @@ const PrescriptionsListTable = ({
     prescriptionId: string,
     issueNumber: string | undefined
   ) => {
-    const originalSearchParams =
-      navigationContext.getOriginalSearchParameters()
-
-    let relevantParams = {}
-    if (originalSearchParams) {
-      if (originalSearchParams.prescriptionId) {
-        relevantParams =
-          navigationContext.getRelevantSearchParameters("prescriptionId")
-      } else if (originalSearchParams.nhsNumber) {
-        relevantParams =
-          navigationContext.getRelevantSearchParameters("nhsNumber")
-      } else if (
-        originalSearchParams.firstName ||
-        originalSearchParams.lastName
-      ) {
-        relevantParams =
-          navigationContext.getRelevantSearchParameters("basicDetails")
-      }
+    // Only set the prescription-specific parameters needed for the details page
+    // Don't modify or mix with the original search parameters to avoid cross-contamination
+    searchContext.setPrescriptionId(prescriptionId)
+    if (issueNumber) {
+      searchContext.setIssueNumber(issueNumber)
     }
-
-    // only preserve relevant search parameters and add prescription-specific ones
-    searchContext.setAllSearchParameters({
-      ...relevantParams,
-      prescriptionId,
-      issueNumber
-    })
   }
 
   const getSortedItems = () => {

packages/cpt-ui/src/components/prescriptionSearch/BasicDetailsSearch.tsx

@@ -53,14 +53,14 @@ export default function BasicDetailsSearch() {
 
   // restore original search parameters when available
   useEffect(() => {
-    const originalParams = navigationContext.getOriginalSearchParameters()
-    if (originalParams) {
-      setFirstName(originalParams.firstName || "")
-      setLastName(originalParams.lastName || "")
-      setDobDay(originalParams.dobDay || "")
-      setDobMonth(originalParams.dobMonth || "")
-      setDobYear(originalParams.dobYear || "")
-      setPostcode(originalParams.postcode || "")
+    const relevantParams = navigationContext.getRelevantSearchParameters("basicDetails")
+    if (relevantParams) {
+      setFirstName(relevantParams.firstName || "")
+      setLastName(relevantParams.lastName || "")
+      setDobDay(relevantParams.dobDay || "")
+      setDobMonth(relevantParams.dobMonth || "")
+      setDobYear(relevantParams.dobYear || "")
+      setPostcode(relevantParams.postcode || "")
     }
   }, [navigationContext])
 
@@ -180,7 +180,7 @@ export default function BasicDetailsSearch() {
     <Container className="nhsuk-width-container-fluid" data-testid="basic-details-search-form-container">
       <Row className="patient-search-form">
         <Col width="three-quarters">
-          <Form onSubmit={handleSubmit} noValidate data-testid="basic-details-form">
+          <Form onSubmit={handleSubmit} noValidate autoComplete="off" data-testid="basic-details-form">
             {inlineErrors.length > 0 && (
               <ErrorSummary ref={errorRef} data-testid="error-summary">
                 <ErrorSummary.Title>{STRINGS.errorSummaryHeading}</ErrorSummary.Title>

packages/cpt-ui/src/components/prescriptionSearch/NhsNumSearch.tsx

@@ -48,9 +48,9 @@ export default function NhsNumSearch() {
   }, [searchContext.nhsNumber, searchContext.searchType])
 
   useEffect(() => {
-    const originalParams = navigationContext.getOriginalSearchParameters()
-    if (originalParams && originalParams.nhsNumber) {
-      setNhsNumber(originalParams.nhsNumber || "")
+    const relevantParams = navigationContext.getRelevantSearchParameters("nhsNumber")
+    if (relevantParams && relevantParams.nhsNumber) {
+      setNhsNumber(relevantParams.nhsNumber || "")
     }
   }, [navigationContext])
 

packages/cpt-ui/src/components/prescriptionSearch/PrescriptionIdSearch.tsx

@@ -43,9 +43,9 @@ export default function PrescriptionIdSearch() {
   const errorMessages = PRESCRIPTION_ID_SEARCH_STRINGS.errors
 
   useEffect(() => {
-    const originalParams = navigationContext.getOriginalSearchParameters()
-    if (originalParams && originalParams.prescriptionId) {
-      setPrescriptionId(originalParams.prescriptionId || "")
+    const relevantParams = navigationContext.getRelevantSearchParameters("prescriptionId")
+    if (relevantParams && relevantParams.prescriptionId) {
+      setPrescriptionId(relevantParams.prescriptionId || "")
     }
   }, [navigationContext])
 

packages/cpt-ui/src/pages/SearchPrescriptionPage.tsx

@@ -1,4 +1,4 @@
-import React, {Fragment} from "react"
+import React, {Fragment, useEffect} from "react"
 import {
   Col,
   Container,
@@ -14,10 +14,28 @@ import BasicDetailsSearch from "@/components/prescriptionSearch/BasicDetailsSear
 
 import {HERO_TEXT} from "@/constants/ui-strings/SearchForAPrescriptionStrings"
 import {PRESCRIPTION_SEARCH_TABS} from "@/constants/ui-strings/SearchTabStrings"
+import {useSearchContext} from "@/context/SearchProvider"
 
 export default function SearchPrescriptionPage() {
   const location = useLocation()
   const pathname = location.pathname
+  const searchContext = useSearchContext()
+
+  // Clear search context contamination when navigating to search pages
+  useEffect(() => {
+    const allParams = searchContext.getAllSearchParameters()
+
+    // Check if we have contamination from multiple search types
+    const hasPrescriptionId = !!allParams.prescriptionId
+    const hasNhsNumber = !!allParams.nhsNumber
+    const hasBasicDetails = !!(allParams.firstName || allParams.lastName || allParams.dobDay)
+
+    const contaminationCount = [hasPrescriptionId, hasNhsNumber, hasBasicDetails].filter(Boolean).length
+
+    if (contaminationCount > 1) {
+      searchContext.clearSearchParameters()
+    }
+  }, [pathname, searchContext])
 
   // Map paths directly to content components
   const pathContent: Record<string, React.ReactNode> = {