department-of-veterans-affairs/vets-website

Function VitalDetails has 319 lines of code (exceeds 25 allowed). Consider refactoring.

Open

const VitalDetails = props => {
  const { runningUnitTest } = props;
  const records = useSelector(state => state.mr.vitals.vitalDetails);
  const vitalsList = useSelector(state => state.mr.vitals.vitalsList);
  const user = useSelector(state => state.user.profile);

File VitalDetails.jsx has 376 lines of code (exceeds 250 allowed). Consider refactoring.

Open

import React, { useEffect, useState, useRef, useMemo } from 'react';
import PropTypes from 'prop-types';
import { useDispatch, useSelector } from 'react-redux';
import { useParams } from 'react-router-dom';
import { chunk } from 'lodash';

Consider simplifying this complex logical expression.

Open

  if (records?.length) {
    const vitalDisplayName = vitalTypeDisplayNames[records[0].type];
    return (
      <>
        <PrintHeader />

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  const generateVitalsPdf = async () => {
    setDownloadStarted(true);

    const { title, subject, preface } = generateVitalsIntro(
      records,

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

        <div className="vads-u-margin-bottom--2 no-print">
          <VaPagination
            onPageSelect={e => onPageChange(e.detail.page)}
            page={currentPage}
            pages={paginatedVitals.current.length}

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

  useEffect(
    () => {
      if (currentPage > 1 && records?.length) {
        focusElement(document.querySelector('#showingRecords'));
        window.scrollTo({

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

                <div className="vads-u-margin-bottom--0p5 vads-u-margin-left--1p5">
                  <h4 className="vads-u-display--inline vads-u-font-size--base vads-u-font-family--sans">
                    Location:{' '}
                  </h4>
                  <p className="vads-u-display--inline" data-dd-privacy="mask">

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

                <div className="vads-u-margin-bottom--0p5 vads-u-margin-left--1p5">
                  <h4 className="vads-u-display--inline vads-u-font-size--base vads-u-font-family--sans">
                    Measurement:{' '}
                  </h4>
                  <p className="vads-u-display--inline" data-dd-privacy="mask">

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

                <div className="vads-u-margin-left--1p5">
                  <h4 className="vads-u-display--inline vads-u-font-size--base vads-u-font-family--sans">
                    Provider notes:{' '}
                  </h4>
                  <p className="vads-u-display--inline" data-dd-privacy="mask">

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

  const fromToNums = (page, total) => {
    const from = (page - 1) * perPage + 1;
    const to = Math.min(page * perPage, total);
    return [from, to];
  };

• Read up
  Read up

Similar blocks of code found in 7 locations. Consider refactoring.

Open

  useListRefresh({
    listState,
    listCurrentAsOf: vitalsCurrentAsOf,
    refreshStatus: refresh.status,
    extractType: refreshExtractTypes.VPR,

• Read up
  Read up