Updates to the Reports section

app/data/organisations.js

@@ -1074,7 +1074,7 @@ module.exports = [
       },
       {
         id: "RW382",
-        name: "Ear nose and throad RMCH"
+        name: "Ear nose and throat RMCH"
       },
       {
         id: "RW391",

app/data/session-data-defaults.js

@@ -22,7 +22,7 @@ module.exports = {
   vaccinationsRecorded: vaccinationsRecorded,
 
   // These are the options for extracting CSV reports
-  patientDataOptions: ["Name", "NHS number", "Gender", "Date of birth", "Address", "Postcode", "ODS code of their GP"],
+  patientDataOptions: ["Name", "NHS number", "Gender", "Date of birth", "Address", "Postcode", "GP (ODS code only)"],
   staffDataOptions: ["Recorder", "Vaccinator"],
   locationDataOptions: ["Site name", "Site ODS code", "Location type"],
   consentAndEligibilityDataOptions: ["Consent", "Eligibility", "Estimated due date"],

app/routes/reports.js

@@ -46,9 +46,15 @@ module.exports = (router) => {
   })
 
   router.post('/reports/choose-vaccines-answer', (req, res) => {
+    const data = req.session.data
+    const currentOrganisation = res.locals.currentOrganisation
+    const sites = currentOrganisation.sites || []
 
-    if (res.locals.currentOrganisation.type === "Pharmacy HQ") {
+    if (currentOrganisation.type === "Pharmacy HQ") {
       res.redirect('/reports/choose-pharmacies')
+    } else if (sites.length === 1) {
+      data.siteIdsToReport = [sites[0].id]
+      res.redirect('/reports/choose-data')
     } else {
       res.redirect('/reports/choose-site')
     }
@@ -106,9 +112,17 @@ module.exports = (router) => {
   })
 
   router.get('/reports/choose-site', (req, res) => {
+    const data = req.session.data
     const currentOrganisation = res.locals.currentOrganisation
 
     const sites = currentOrganisation.sites || []
+
+    if (sites.length === 1) {
+      data.siteIdsToReport = [sites[0].id]
+      res.redirect('/reports/choose-data')
+      return
+    }
+
     res.render('reports/choose-site', {
       sites
     })
@@ -170,7 +184,7 @@ module.exports = (router) => {
     } else {
 
       const error = {
-        text: "Select data for report",
+        text: "Select the data you want to include",
         href: "#data-1"
       }
 
@@ -186,6 +200,7 @@ module.exports = (router) => {
     const data = req.session.data
     const currentOrganisation = res.locals.currentOrganisation
     const siteIds = data.siteIdsToReport || []
+    const selectedVaccines = data.vaccinesToReport || []
     const today = new Date()
     const days = 86400000 // number of milliseconds in a day
 
@@ -203,6 +218,22 @@ module.exports = (router) => {
     const dateOption = data.date
 
     let from, to
+    let vaccinesToReportDisplay = selectedVaccines.filter((vaccineName) => vaccineName !== 'all')
+
+    if (selectedVaccines.includes('all')) {
+      const organisationVaccines = currentOrganisation.vaccines || []
+      let enabledVaccines = organisationVaccines.filter((vaccine) => vaccine.status === "enabled")
+
+      // Temporary: show all vaccines if none have batches added
+      if (enabledVaccines.length === 0) {
+        enabledVaccines = data.vaccines
+      }
+
+      vaccinesToReportDisplay = enabledVaccines.map((vaccine) => vaccine.name)
+    }
+
+    vaccinesToReportDisplay = [...new Set(vaccinesToReportDisplay)]
+      .sort((a, b) => a.localeCompare(b))
 
     switch (dateOption) {
       case 'custom_date_range':
@@ -229,6 +260,19 @@ module.exports = (router) => {
         from = new Date(today.getTime() - (31 * days)).toISOString().substring(0,10)
         to = today.toISOString().substring(0,10)
         break
+      case 'Last90days':
+        from = new Date(today.getTime() - (90 * days)).toISOString().substring(0,10)
+        to = today.toISOString().substring(0,10)
+        break
+      case 'LastCalendarMonth': {
+        const firstDayOfThisMonth = new Date(today.getFullYear(), today.getMonth(), 1)
+        const firstDayOfLastMonth = new Date(today.getFullYear(), today.getMonth() - 1, 1)
+        const lastDayOfLastMonth = new Date(firstDayOfThisMonth.getTime() - days)
+
+        from = firstDayOfLastMonth.toISOString().substring(0,10)
+        to = lastDayOfLastMonth.toISOString().substring(0,10)
+        break
+      }
     }
 
 
@@ -237,7 +281,8 @@ module.exports = (router) => {
       sites,
       pharmacies,
       from,
-      to
+      to,
+      vaccinesToReportDisplay
     })
 
   })
@@ -252,7 +297,7 @@ module.exports = (router) => {
     data.vaccinesToReport = null
 
 
-    res.redirect('/reports/download')
+    res.redirect('/reports/creating-report')
   })
 
 }

app/views/reports/alt-index.html

@@ -0,0 +1,37 @@
+{% extends 'layout.html' %}
+
+{% set pageName = "Reports" %}
+
+{% set currentSection = "reports" %}
+
+{% block content %}
+
+  <div class="nhsuk-grid-row">
+    <div class="nhsuk-grid-column-two-thirds">
+
+      {% include "includes/notification.html" %}
+
+    </div>
+  </div>
+
+  <div class="nhsuk-grid-row">
+    <div class="nhsuk-grid-column-full">
+
+      <h1 class="nhsuk-heading-l nhsuk-u-margin-bottom-2">Reports</h1>
+      <p>Create and download reports.</p>
+
+      {% from "inset-text/macro.njk" import insetText %}
+
+{% set insetTextHtml %}
+  <p>You need to wait before you can start creating a report. This is because another report is already in progress for your organisation.</p>
+{% endset %}
+
+{{ insetText({
+  html: insetTextHtml
+}) }}
+
+
+    </div>
+  </div>
+
+{% endblock %}

app/views/reports/check.html

@@ -19,37 +19,55 @@
 
       <h1 class="nhsuk-heading-l">{{ pageName }}</h1>
 
-      {% set dataFields = [] %}
-
-      {% for field in data.patientDataToReport %}
-        {% set dataFields = (dataFields.push(field), dataFields) %}
-      {% endfor %}
-      {% for field in data.staffDataToReport %}
-        {% set dataFields = (dataFields.push(field), dataFields) %}
-      {% endfor %}
-      {% for field in data.siteDataToReport %}
-        {% set dataFields = (dataFields.push(field), dataFields) %}
-      {% endfor %}
-      {% for field in data.consentAndEligibilityDataToReport %}
-        {% set dataFields = (dataFields.push(field), dataFields) %}
-      {% endfor %}
-      {% for field in data.vaccinationDataToReport %}
-        {% set dataFields = (dataFields.push(field), dataFields) %}
-      {% endfor %}
-      {% for field in data.paymentDataToReport %}
-        {% set dataFields = (dataFields.push(field), dataFields) %}
-      {% endfor %}
+      {% set patientData = data.patientDataToReport or [] %}
+      {% set staffData = data.staffDataToReport or [] %}
+      {% set siteData = data.siteDataToReport or [] %}
+      {% set consentAndEligibilityData = data.consentAndEligibilityDataToReport or [] %}
+      {% set vaccinationData = data.vaccinationDataToReport or [] %}
+      {% set paymentData = data.paymentDataToReport or [] %}
 
       {% set dataValue %}
-        {{ dataFields | join(", ") }}
+        {% if (patientData | length) > 0 %}
+          <h4 class="nhsuk-heading-xs nhsuk-u-margin-bottom-1">Patient data</h4>
+          <p class="nhsuk-u-margin-bottom-3">{{ patientData | join(", ") }}</p>
+        {% endif %}
+
+        {% if (staffData | length) > 0 %}
+          <h4 class="nhsuk-heading-xs nhsuk-u-margin-bottom-1">Staff data</h4>
+          <p class="nhsuk-u-margin-bottom-3">{{ staffData | join(", ") }}</p>
+        {% endif %}
+
+        {% if (siteData | length) > 0 %}
+          <h4 class="nhsuk-heading-xs nhsuk-u-margin-bottom-1">Site data</h4>
+          <p class="nhsuk-u-margin-bottom-3">{{ siteData | join(", ") }}</p>
+        {% endif %}
+
+        {% if (consentAndEligibilityData | length) > 0 %}
+          <h4 class="nhsuk-heading-xs nhsuk-u-margin-bottom-1">Consent and eligibility data</h4>
+          <p class="nhsuk-u-margin-bottom-3">{{ consentAndEligibilityData | join(", ") }}</p>
+        {% endif %}
+
+        {% if (vaccinationData | length) > 0 %}
+          <h4 class="nhsuk-heading-xs nhsuk-u-margin-bottom-1">Vaccination data</h4>
+          <p class="nhsuk-u-margin-bottom-3">{{ vaccinationData | join(", ") }}</p>
+        {% endif %}
+
+        {% if (paymentData | length) > 0 %}
+          <h4 class="nhsuk-heading-xs nhsuk-u-margin-bottom-1">Payment information</h4>
+          <p class="nhsuk-u-margin-bottom-0">{{ paymentData | join(", ") }}</p>
+        {% endif %}
       {% endset %}
 
       {% set sitesHtml %}
-        <ul class="nhsuk-list">
-          {% for site in sites | sort(false, false, "name") %}
-            <li>{{ site.name }} ({{ site.id }})</li>
-          {% endfor %}
-        </ul>
+        {% if (sites | length) > 2 %}
+          {{ sites | length }} sites selected
+        {% else %}
+          <ul class="nhsuk-list">
+            {% for site in sites | sort(false, false, "name") %}
+              <li>{{ site.name }} ({{ site.id }})</li>
+            {% endfor %}
+          </ul>
+        {% endif %}
       {% endset %}
 
       {% set pharmaciesHtml %}
@@ -60,14 +78,13 @@ <h1 class="nhsuk-heading-l">{{ pageName }}</h1>
         </ul>
       {% endset %}
 
-      {% set vaccinesToReportText = "" %}
-      {% if data.vaccinesToReport %}
-        {% set vaccinesToReportDisplay = [] %}
-        {% for vaccine in (data.vaccinesToReport | sort(false, false, "name")) %}
-          {% set vaccinesToReportDisplay = (vaccinesToReportDisplay.push(vaccine ), vaccinesToReportDisplay) %}
-        {% endfor %}
-        {% set vaccinesToReportText = vaccinesToReportDisplay | join(", ") %}
-      {% endif %}
+      {% set vaccinesToReportHtml %}
+        <ul class="nhsuk-list">
+          {% for vaccine in vaccinesToReportDisplay or [] %}
+            <li>{{ vaccine | capitaliseFirstLetter }}</li>
+          {% endfor %}
+        </ul>
+      {% endset %}
 
       {{ summaryList({
         rows: [
@@ -110,10 +127,10 @@ <h1 class="nhsuk-heading-l">{{ pageName }}</h1>
           },
           {
             key: {
-              text: ("Vaccines" if (data.vaccinesToReport | length) > 1 else "Vaccine")
+              text: ("Vaccines" if (vaccinesToReportDisplay | length) > 1 else "Vaccine")
             },
             value: {
-              text: vaccinesToReportText if data.vaccines
+              html: vaccinesToReportHtml
             },
             actions: {
               items: [
@@ -181,7 +198,7 @@ <h1 class="nhsuk-heading-l">{{ pageName }}</h1>
               text: "Data"
             },
             value: {
-              text: dataValue
+              html: dataValue
             },
             actions: {
               items: [

app/views/reports/choose-data.html

@@ -1,13 +1,21 @@
 {% extends 'layout.html' %}
 
-{% set pageName = "Data details" %}
+{% set pageName = "Which details do you want to include?" %}
 
 {% set currentSection = "reports" %}
 
 
 {% block beforeContent %}
+  {% if currentOrganisation.type == "Pharmacy HQ" %}
+    {% set backLinkHref = "/reports/choose-pharmacies" %}
+  {% elif (currentOrganisation.sites | length) == 1 %}
+    {% set backLinkHref = "/reports/choose-vaccines" %}
+  {% else %}
+    {% set backLinkHref = "/reports/choose-site" %}
+  {% endif %}
+
   {{ backLink({
-    href: "/reports/choose-site",
+    href: backLinkHref,
     text: "Back"
   }) }}
 {% endblock %}
@@ -178,7 +186,7 @@
           } if error,
           fieldset: {
             legend: {
-              text: "Which details would you like to include?",
+              text: "Which details do you want to include?",
               isPageHeading: true,
               classes: "nhsuk-fieldset__legend--l"
             }
@@ -215,7 +223,7 @@
             },
             {
               value: "Vaccination",
-              text: "Vaccination",
+              text: "Vaccinations",
               conditional: {
                 html: vaccinationHtml
               }

app/views/reports/choose-dates.html

@@ -143,10 +143,6 @@
               "value": "Last7days",
               "text": "Last 7 days (includes today)"
             },
-            {
-              "value": "Last14days",
-              "text": "Last 14 days (includes today)"
-            },
             {
                 "value": "Last31days",
                 "text": "Last 31 days (includes today)"
@@ -155,6 +151,10 @@
                 "value": "Last90days",
                 "text": "Last 90 days (includes today)"
               },
+            {
+                "value": "LastCalendarMonth",
+                "text": "Last calendar month"
+            },
             {
               divider: "or"
             },

app/views/reports/choose-pharmacies.html

@@ -1,6 +1,6 @@
 {% extends 'layout.html' %}
 
-{% set pageName = "Choose pharmacies" %}
+{% set pageName = "Which pharmacies do you want to include?" %}
 
 {% set currentSection = "reports" %}
 

app/views/reports/choose-vaccines.html

@@ -1,6 +1,6 @@
 {% extends 'layout.html' %}
 
-{% set pageName = "Choose vaccines" %}
+{% set pageName = "Which vaccines you want to include?" %}
 
 {% set currentSection = "reports" %}
 
@@ -46,7 +46,7 @@
           name: "vaccinesToReport",
           fieldset: {
             legend: {
-              text: "Choose vaccines",
+              text: "Which vaccines do you want to include?",
               size: "l",
               isPageHeading: true
             }