Katello/katello

Function drawImage has 77 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  contextPrototype.drawImage = function(image, var_args) {
    var dx, dy, dw, dh, sx, sy, sw, sh;

    // to find the original width we overide the width and height
    var oldRuntimeWidth = image.runtimeStyle.width;

File cvModuleStreamFilter.test.js has 295 lines of code (exceeds 250 allowed). Consider refactoring.

Open

import React from 'react';
import { renderWithRedux, patientlyWaitFor, fireEvent } from 'react-testing-lib-wrapper';
import { Route } from 'react-router-dom';

import ContentViewFilterDetails from '../ContentViewFilterDetails';

Function BulkErrataReviewFooter has 77 lines of code (exceeds 25 allowed). Consider refactoring.

Open

export const BulkErrataReviewFooter = () => {
  const {
    finishButtonText,
    finishButtonLoading,
    setFinishButtonLoading,

File ContentViewDetailSelectors.js has 294 lines of code (exceeds 250 allowed). Consider refactoring.

Open

import { STATUS } from 'foremanReact/constants';
import {
  selectAPIError,
  selectAPIResponse,
  selectAPIStatus,

Function ContentService has 76 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    function ContentService($injector, Nutupane, $state, translate) {
        var contentTypes;

        function currentState() {
            return $state.current.name.split('.').pop();

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

Open

  render() {
    const { subscriptionDetails } = this.props;


    const resource = {

File ContentViewDetailsCard.js has 292 lines of code (exceeds 250 allowed). Consider refactoring.

Open

import React, { useState } from 'react';
import {
  Card,
  CardHeader,
  CardTitle,

File CVModuleStreamFilterContent.js has 292 lines of code (exceeds 250 allowed). Consider refactoring.

Open

import React, { useState, useEffect, useCallback } from 'react';
import useDeepCompareEffect from 'use-deep-compare-effect';
import PropTypes from 'prop-types';
import { shallowEqual, useSelector, useDispatch } from 'react-redux';
import { omit } from 'lodash';

Function menu has 75 lines of code (exceeds 25 allowed). Consider refactoring.

Open

KT.menu = (function(){
  return {
    menuSetup: function(){
      //set some useful vars
      var topLevel = $('#appnav li.top_level');

Function AcsUrlPaths has 75 lines of code (exceeds 25 allowed). Consider refactoring.

Open

const AcsUrlPaths = () => {
  const {
    acsType, url, setUrl, subpaths, setSubpaths,
  } = useContext(ACSCreateContext);

Class Ping has 26 methods (exceeds 20 allowed). Consider refactoring.

Open

  class Ping
    OK_RETURN_CODE = 'ok'.freeze
    FAIL_RETURN_CODE = 'FAIL'.freeze
    PACKAGES = %w(katello candlepin pulp foreman hammer dynflow).freeze

Class RepositoryTypeManager has 26 methods (exceeds 20 allowed). Consider refactoring.

Open

  class RepositoryTypeManager
    PULP3_FEATURE = "Pulpcore".freeze

    @defined_repository_types = {}
    @enabled_repository_types = {}

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

Open

          {Object.keys(subscriptionAttributes).map(key => (
            <tr key={key}>
              <td><b>{__(subscriptionAttributes[key])}</b></td>
              <td>{subscriptionDetailValue(subscriptionDetails, key)}</td>
            </tr>

• Read up
  Read up

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

Open

  await patientlyWaitFor(() => {
    expect(queryByText(results[20].name)).toBeInTheDocument();
    expect(queryByText(results[39].name)).toBeInTheDocument();
    expect(queryByText(results[41].name)).not.toBeInTheDocument();
  });

• Read up
  Read up

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

Open

          {Object.keys(subscriptionPurposeAttributes).map(key => (
            <tr key={key}>
              <td><b>{__(subscriptionPurposeAttributes[key])}</b></td>
              <td>{subscriptionDetailValue(subscriptionDetails, key)}</td>
            </tr>

• Read up
  Read up

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

Open

export const moduleStreamAction = ({ hostname, action, moduleSpec }) => post({
  type: API_OPERATIONS.POST,
  key: REX_JOB_INVOCATIONS_KEY,
  url: foremanApi.getApiUrl('/job_invocations'),
  params: katelloModuleStreamActionsParams({ hostname, action, moduleSpec }),

• Read up
  Read up

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

Open

                {products.map(product =>
                  (
                    <ListItem key={product?.id} aria-label="product_value">
                      <a href={urlBuilder(`products/${product?.id}`, '')}><b>{product?.name}</b></a>
                    </ListItem>

• Read up
  Read up

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

Open

export const removePackages = ({ hostname, search, descriptionFormat }) => post({
  type: API_OPERATIONS.POST,
  key: REX_JOB_INVOCATIONS_KEY,
  url: foremanApi.getApiUrl('/job_invocations'),
  params: katelloPackagesRemoveParams({ hostname, search, descriptionFormat }),

• Read up
  Read up

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

Open

              {smartProxies?.length > 0 && smartProxies.map(sp =>
                (
                  <ListItem key={sp?.id} aria-label="smartproxy_value">
                    <a href={urlBuilder(`smart_proxies/${sp?.id}`, '')}><b>{sp?.name}</b></a>
                  </ListItem>

• Read up
  Read up

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

Open

  await patientlyWaitFor(() => {
    expect(queryByText(results[0].name)).toBeInTheDocument();
    expect(queryByText(results[19].name)).toBeInTheDocument();
    expect(queryByText(results[21].name)).not.toBeInTheDocument();
  });

• Read up
  Read up