department-of-veterans-affairs/vets-website

Function render has 171 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  render() {
    const { supplies, order, eligibility } = this.props;
    const currentDate = moment();
    const batterySupplies = supplies.filter(
      batterySupply => batterySupply.productGroup === BATTERY,

File Batteries.jsx has 262 lines of code (exceeds 250 allowed). Consider refactoring.

Open

import PropTypes from 'prop-types';
import React, { Component } from 'react';
import { connect } from 'react-redux';

import classnames from 'classnames';

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

Open

Batteries.propTypes = {
  eligibility: PropTypes.object,
  formData: PropTypes.object,
  order: PropTypes.arrayOf(
    PropTypes.shape({

• Read up
  Read up

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

Open

        {!haveBatteriesBeenOrderedInLastTwoYears &&
          !areBatterySuppliesEligible && (
            <va-alert status="info" visible>
              <h3 slot="headline">
                Your batteries aren’t available for online ordering

• Read up
  Read up

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

Open

  handleChecked = (checked, batterySupply) => {
    const { order, formData } = this.props;
    let updatedOrder;
    const isSupplyChecked = checked ? 'yes' : 'no';
    recordEvent({

• Read up
  Read up

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

Open

              <div className="vads-u-border-left--10px vads-u-border-color--primary-alt vads-u-margin-bottom--2">
                <div className="usa-alert-body vads-u-padding-left--1">
                  <p className="vads-u-margin--1px vads-u-margin-y--1">
                    <span className="vads-u-font-weight--bold dd-privacy-mask">
                      Battery:{' '}

• Read up
  Read up

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

Open

        {batterySupplies.length > 0 && (
          <va-additional-info trigger="What if my device isn’t listed here?">
            <p>
              Your hearing aid device may not be listed here if you haven’t
              placed an order for resupply items within the last 2 years. If you

• Read up
  Read up

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

Open

const mapStateToProps = state => ({
  supplies: state.form?.data?.supplies,
  formData: state.form?.data,
  order: state.form?.data?.order,
  eligibility: state.form?.data?.eligibility,

• Read up
  Read up

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

Open

              {currentDate.diff(batterySupply.nextAvailabilityDate, 'days') <
              0 ? (
                <div className="usa-alert usa-alert-warning vads-u-background-color--white vads-u-padding-x--2p5 vads-u-padding-y--2 vads-u-width--full">
                  <div className="usa-alert-body">
                    <h3 className="usa-alert-heading vads-u-font-family--sans">

• Read up
  Read up

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

Open

                  <input
                    id={batterySupply.productId}
                    className="vads-u-margin-left--0"
                    type="checkbox"
                    onChange={e =>

• Read up
  Read up

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

Open

            <div
              key={batterySupply.productId}
              className={classnames({
                'vads-u-background-color--gray-lightest vads-u-margin-bottom--2 vads-u-margin-top--3': true,
                'vads-u-border-color--primary vads-u-border--3px vads-u-padding--21 dd-privacy-mask': isBatterySelected(

• Read up
  Read up

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

Open

  componentDidMount() {
    const areBatterySuppliesEligible = this.props.eligibility?.batteries;
    if (!areBatterySuppliesEligible) {
      recordEvent({
        event: 'bam-error',

• Read up
  Read up

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

Open

    const haveBatteriesBeenOrderedInLastTwoYears =
      batterySupplies.length > 0 &&
      batterySupplies.every(
        battery => currentDate.diff(battery.lastOrderDate, 'years') <= 2,
      );

• Read up
  Read up

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

Open

    const isBatterySelected = batteryProductId => {
      const selectedProductIds = order.map(
        selectedProduct => selectedProduct.productId,
      );
      return selectedProductIds.includes(batteryProductId);

• Read up
  Read up