Sign upLogin

Reconmap/web-client

View on GitHub
Star

Showing 102 of 450 total issues

Consider simplifying this complex logical expression.
Open

            if ((key === 'LC') || (key === 'LI') || (key === 'LAV') || (key === 'LAC')
                || (key === 'FD') || (key === 'RD') || (key === 'NC') || (key === 'PV')) {
                sum += parseInt(value);
            }
Severity: Critical
Found in src/components/vulnerabilities/OwaspRR.jsx - About 1 hr to fix

Consider simplifying this complex logical expression.
Open

            if ((key === 'SL') || (key === 'M') || (key === 'O') || (key === 'S')
                || (key === 'ED') || (key === 'EE') || (key === 'A') || (key === 'ID')) {
                sum += parseInt(value);
            }
Severity: Critical
Found in src/components/vulnerabilities/OwaspRR.jsx - About 1 hr to fix

Function ProjectEdit has 33 lines of code (exceeds 25 allowed). Consider refactoring.
Open

const ProjectEdit = () => {
    const navigate = useNavigate();
    const { projectId } = useParams();
    const [serverProject] = useFetch(`/projects/${projectId}`);
    const [clientProject, setClientProject] = useState(null);
Severity: Minor
Found in src/components/projects/Edit.jsx - About 1 hr to fix

Function SearchBox has 33 lines of code (exceeds 25 allowed). Consider refactoring.
Open

const SearchBox = () => {
    const navigate = useNavigate();
    const inputRef = createRef();

    const onKeyDownListener = useCallback(
Severity: Minor
Found in src/components/search/Box.jsx - About 1 hr to fix

Function HeaderUserMenu has 32 lines of code (exceeds 25 allowed). Consider refactoring.
Open

const HeaderUserMenu = () => {
    const { user, logout } = useContext(AuthContext);

    return <Menu closeOnBlur closeOnSelect>
        <MenuButton>
Severity: Minor
Found in src/components/ui/HeaderUserMenu.jsx - About 1 hr to fix

Function reloadVulnerabilities has 32 lines of code (exceeds 25 allowed). Consider refactoring.
Open

    const reloadVulnerabilities = useCallback(() => {
        setTableModel((tableModel) => ({
            ...tableModel,
            vulnerabilities: null,
        }));
Severity: Minor
Found in src/components/vulnerabilities/List.jsx - About 1 hr to fix

Function VulnerabilitiesTable has a Cognitive Complexity of 11 (exceeds 5 allowed). Consider refactoring.
Open

const VulnerabilitiesTable = ({
    tableModel,
    tableModelSetter: setTableModel,
    reloadCallback,
    showSelection = true,
Severity: Minor
Found in src/components/vulnerabilities/VulnerabilitiesTable.jsx - About 1 hr to fix

Cognitive Complexity

Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

A method's cognitive complexity is based on a few simple rules:

  • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
  • Code is considered more complex for each "break in the linear flow of the code"
  • Code is considered more complex when "flow breaking structures are nested"

Further reading

Function ProjectForm has a Cognitive Complexity of 11 (exceeds 5 allowed). Consider refactoring.
Open

const ProjectForm = ({
    isEdit = false,
    project,
    projectSetter: setProject,
    onFormSubmit,
Severity: Minor
Found in src/components/projects/Form.jsx - About 1 hr to fix

Cognitive Complexity

Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

A method's cognitive complexity is based on a few simple rules:

  • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
  • Code is considered more complex for each "break in the linear flow of the code"
  • Code is considered more complex when "flow breaking structures are nested"

Further reading

Function EditClientPage has 31 lines of code (exceeds 25 allowed). Consider refactoring.
Open

const EditClientPage = () => {
    const navigate = useNavigate();
    const { clientId } = useParams();

    const [serverClient] = useFetch(`/clients/${clientId}`);
Severity: Minor
Found in src/components/clients/Edit.jsx - About 1 hr to fix

Function EditCommandPage has 31 lines of code (exceeds 25 allowed). Consider refactoring.
Open

const EditCommandPage = () => {
    const navigate = useNavigate();
    const { commandId } = useParams();

    const [serverCommand] = useFetch(`/commands/${commandId}`);
Severity: Minor
Found in src/components/commands/Edit.jsx - About 1 hr to fix

Function registerValidSW has 31 lines of code (exceeds 25 allowed). Consider refactoring.
Open

function registerValidSW(swUrl, config) {
    navigator.serviceWorker
        .register(swUrl)
        .then(registration => {
            registration.onupdatefound = () => {
Severity: Minor
Found in src/serviceWorker.js - About 1 hr to fix

Function EditDocumentPage has 31 lines of code (exceeds 25 allowed). Consider refactoring.
Open

const EditDocumentPage = () => {
    const navigate = useNavigate();
    const { documentId } = useParams();

    const [serverDocument] = useFetch(`/documents/${documentId}`);
Severity: Minor
Found in src/components/documents/Edit.jsx - About 1 hr to fix

Function ProjectCreate has 29 lines of code (exceeds 25 allowed). Consider refactoring.
Open

const ProjectCreate = () => {
    const navigate = useNavigate();
    const query = useQuery();
    const isTemplate = 'true' === query.get('isTemplate');
    const [newProject, setNewProject] = useState({ ...Project, is_template: isTemplate });
Severity: Minor
Found in src/components/projects/Create.jsx - About 1 hr to fix

Function EditUserPage has 29 lines of code (exceeds 25 allowed). Consider refactoring.
Open

const EditUserPage = () => {
    const navigate = useNavigate();
    const { userId } = useParams();
    const [serverUser] = useFetch(`/users/${userId}`);
    const [clientUser, setClientUser] = useState(null);
Severity: Minor
Found in src/components/users/Edit.jsx - About 1 hr to fix

Function onUploadButtonClick has 28 lines of code (exceeds 25 allowed). Consider refactoring.
Open

    const onUploadButtonClick = ev => {
        const formData = new FormData();
        formData.append('parentType', parentType);
        formData.append('parentId', parentId);
        acceptedFiles.forEach(file => {
Severity: Minor
Found in src/components/attachments/ImageDropzone.jsx - About 1 hr to fix

Function onFormChange has 28 lines of code (exceeds 25 allowed). Consider refactoring.
Open

    const onFormChange = (ev) => {
        const target = ev.target;
        const name = target.name;
        let value = target.type === "checkbox" ? target.checked : target.value;
Severity: Minor
Found in src/components/vulnerabilities/Form.jsx - About 1 hr to fix

Function onFormSubmit has 28 lines of code (exceeds 25 allowed). Consider refactoring.
Open

    const onFormSubmit = (ev) => {
        ev.preventDefault();

        const params = {
            projectId: projectId,
Severity: Minor
Found in src/components/reports/ModalDialog.jsx - About 1 hr to fix

Function AttachmentsDropzone has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring.
Open

const AttachmentsDropzone = ({ parentType, parentId, onUploadFinished = null, attachmentId = null }) => {
    const onFileDrop = (newFiles) => {
        setAcceptedFiles(newFiles);
    };
Severity: Minor
Found in src/components/attachments/Dropzone.jsx - About 1 hr to fix

Cognitive Complexity

Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

A method's cognitive complexity is based on a few simple rules:

  • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
  • Code is considered more complex for each "break in the linear flow of the code"
  • Code is considered more complex when "flow breaking structures are nested"

Further reading

Function AttachmentsImageDropzone has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring.
Open

const AttachmentsImageDropzone = ({ parentType, parentId, onUploadFinished = null, uploadFinishedParameter = null, attachmentId = null, maxFileCount = Infinity }) => {
    const onFileDrop = (newFiles) => {
        setAcceptedFiles(newFiles);
    };
Severity: Minor
Found in src/components/attachments/ImageDropzone.jsx - About 1 hr to fix

Cognitive Complexity

Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

A method's cognitive complexity is based on a few simple rules:

  • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
  • Code is considered more complex for each "break in the linear flow of the code"
  • Code is considered more complex when "flow breaking structures are nested"

Further reading

Function CreateUserPage has 27 lines of code (exceeds 25 allowed). Consider refactoring.
Open

const CreateUserPage = () => {
    const navigate = useNavigate();
    const [userData, setUserData] = useState(User);

    const handleCreate = async (ev) => {
Severity: Minor
Found in src/components/users/Create.jsx - About 1 hr to fix
Severity
Category
Status
Source
Language