From a83177a34fc0cd0004d49212621a1a7b73f6504a Mon Sep 17 00:00:00 2001
From: Tim DiLauro <tdilauro@users.noreply.github.com>
Date: Thu, 26 Sep 2024 20:20:30 -0400
Subject: [PATCH] Restore total patron count. (PP-1744) (#137)

---
 src/components/LibraryStats.tsx                |  1 +
 src/components/SingleStatListItem.tsx          |  1 -
 src/components/StatsPatronGroup.tsx            | 10 +++++++++-
 src/components/__tests__/LibraryStats-test.tsx |  7 ++++---
 4 files changed, 14 insertions(+), 5 deletions(-)

diff --git a/src/components/LibraryStats.tsx b/src/components/LibraryStats.tsx
index 5cdea404c..62e091bb7 100644
--- a/src/components/LibraryStats.tsx
+++ b/src/components/LibraryStats.tsx
@@ -65,6 +65,7 @@ const LibraryStats = ({ stats, library }: LibraryStatsProps) => {
       <ul className={`stats ${statsLayoutClass}`}>
         <li className="stat-group stat-patrons-group">
           <StatsPatronGroup
+            total={patrons.total}
             withActiveLoan={patrons.withActiveLoan}
             withActiveLoanOrHold={patrons.withActiveLoanOrHold}
             heading="Current Circulation Activity"
diff --git a/src/components/SingleStatListItem.tsx b/src/components/SingleStatListItem.tsx
index 85352da38..c7f1f2b7b 100644
--- a/src/components/SingleStatListItem.tsx
+++ b/src/components/SingleStatListItem.tsx
@@ -1,5 +1,4 @@
 import * as React from "react";
-import * as numeral from "numeral";
 import { formatNumber, roundedNumber } from "../utils/sharedFunctions";
 
 export interface SingleStatListItemProps {
diff --git a/src/components/StatsPatronGroup.tsx b/src/components/StatsPatronGroup.tsx
index 319865b3b..3aa61e60a 100644
--- a/src/components/StatsPatronGroup.tsx
+++ b/src/components/StatsPatronGroup.tsx
@@ -24,7 +24,15 @@ const StatsPatronGroup = ({
           <SingleStatListItem
             label="Total Patrons"
             value={total}
-            tooltip="Total number of patrons in the Palace System."
+            tooltip={`
+              Total number of patrons in the Palace System.
+              Please note:
+              this number could be artificially inflated if you have an
+              Aspen integration. We are working to address the issue and
+              will update you when it’s resolved.
+              `
+              .replace(/(?:\s|\r\n|\r|\n)+/g, " ")
+              .trim()}
           />
         )}
         <SingleStatListItem
diff --git a/src/components/__tests__/LibraryStats-test.tsx b/src/components/__tests__/LibraryStats-test.tsx
index 5f58e2fcf..b2051a322 100644
--- a/src/components/__tests__/LibraryStats-test.tsx
+++ b/src/components/__tests__/LibraryStats-test.tsx
@@ -106,13 +106,14 @@ describe("LibraryStats", () => {
       /* Patrons */
       expect(groups.at(0).text()).to.contain("Current Circulation Activity");
       statItems = groups.at(0).find("SingleStatListItem");
-      expect(statItems.length).to.equal(2);
-      expectStats(statItems.at(0).props(), "Patrons With Active Loans", 21);
+      expectStats(statItems.at(0).props(), "Total Patrons", 132);
+      expectStats(statItems.at(1).props(), "Patrons With Active Loans", 21);
       expectStats(
-        statItems.at(1).props(),
+        statItems.at(2).props(),
         "Patrons With Active Loans or Holds",
         23
       );
+      expect(groups.at(0).text()).to.contain("132Total Patrons");
       expect(groups.at(0).text()).to.contain("21Patrons With Active Loans");
       expect(groups.at(0).text()).to.contain(
         "23Patrons With Active Loans or Holds"