thepracticaldev/dev.to

View on GitHub

Showing 308 of 308 total issues

File chat.jsx has 990 lines of code (exceeds 250 allowed). Consider refactoring.
Open

import { h, Component } from 'preact';
import PropTypes from 'prop-types';
import ConfigImage from '../../assets/images/three-dots.svg';
import {
  conductModeration,
Severity: Major
Found in app/javascript/chat/chat.jsx - About 2 days to fix

    Class User has 83 methods (exceeds 20 allowed). Consider refactoring.
    Open

    class User < ApplicationRecord
      include CloudinaryHelper
    
      attr_accessor(
        :scholar_email, :new_note, :note_for_current_role, :user_status, :pro, :merge_user_id,
    Severity: Major
    Found in app/models/user.rb - About 1 day to fix

      Function initializePodcastPlayback has 265 lines of code (exceeds 25 allowed). Consider refactoring.
      Open

      function initializePodcastPlayback() {
        function getById(name) {
          return document.getElementById(name);
        }
      
      
      Severity: Major
      Found in app/assets/javascripts/initializers/initializePodcastPlayback.js - About 1 day to fix

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

          render() {
            const { user } = this.props;
            const channelId = this.props.activeChannelId;
            const channel = this.props.activeChannel || {};
            const socialIcons = [];
        Severity: Major
        Found in app/javascript/chat/userDetails.jsx - About 1 day to fix

          File user.rb has 531 lines of code (exceeds 250 allowed). Consider refactoring.
          Open

          class User < ApplicationRecord
            include CloudinaryHelper
          
            attr_accessor(
              :scholar_email, :new_note, :note_for_current_role, :user_status, :pro, :merge_user_id,
          Severity: Major
          Found in app/models/user.rb - About 1 day to fix

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

              render() {
                const {
                  listings,
                  query,
                  tags,
            Severity: Major
            Found in app/javascript/listings/listings.jsx - About 1 day to fix

              File article.rb has 529 lines of code (exceeds 250 allowed). Consider refactoring.
              Open

              class Article < ApplicationRecord
                include CloudinaryHelper
                include ActionView::Helpers
                include AlgoliaSearch
                include Storext.model
              Severity: Major
              Found in app/models/article.rb - About 1 day to fix

                Class Article has 60 methods (exceeds 20 allowed). Consider refactoring.
                Open

                class Article < ApplicationRecord
                  include CloudinaryHelper
                  include ActionView::Helpers
                  include AlgoliaSearch
                  include Storext.model
                Severity: Major
                Found in app/models/article.rb - About 1 day to fix

                  Chat has 54 functions (exceeds 20 allowed). Consider refactoring.
                  Open

                  export default class Chat extends Component {
                    static propTypes = {
                      pusherKey: PropTypes.number.isRequired,
                      chatChannels: PropTypes.string.isRequired,
                      chatOptions: PropTypes.string.isRequired,
                  Severity: Major
                  Found in app/javascript/chat/chat.jsx - About 7 hrs to fix

                    File listings.jsx has 486 lines of code (exceeds 250 allowed). Consider refactoring.
                    Open

                    import { h, Component } from 'preact';
                    import { SingleListing } from './singleListing';
                    
                    export class Listings extends Component {
                      state = {
                    Severity: Minor
                    Found in app/javascript/listings/listings.jsx - About 7 hrs to fix

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

                      const orgOptions = (organizations, organizationId) => {
                        const orgs = organizations.map(organization => {
                          if (organizationId === organization.id) {
                            return (
                              <option value={organization.id} selected>
                      Severity: Major
                      Found in app/javascript/article-form/elements/orgSettings.jsx and 1 other location - About 7 hrs to fix
                      app/javascript/listings/elements/orgSettings.jsx on lines 5..26

                      Duplicated Code

                      Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                      Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                      When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                      Tuning

                      This issue has a mass of 187.

                      We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                      The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                      If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                      See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                      Refactorings

                      Further Reading

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

                      const orgOptions = (organizations, organizationId) => {
                        const orgs = organizations.map(organization => {
                          if (organizationId === organization.id) {
                            return (
                              <option value={organization.id} selected>
                      Severity: Major
                      Found in app/javascript/listings/elements/orgSettings.jsx and 1 other location - About 7 hrs to fix
                      app/javascript/article-form/elements/orgSettings.jsx on lines 4..25

                      Duplicated Code

                      Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                      Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                      When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                      Tuning

                      This issue has a mass of 187.

                      We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                      The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                      If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                      See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                      Refactorings

                      Further Reading

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

                        render() {
                          // cover image url should asking for url OR providing option to upload an image
                          const {
                            title,
                            tagList,
                      Severity: Major
                      Found in app/javascript/article-form/articleForm.jsx - About 7 hrs to fix

                        File articleForm.jsx has 448 lines of code (exceeds 250 allowed). Consider refactoring.
                        Open

                        import 'preact/devtools';
                        import { h, Component } from 'preact';
                        import linkState from 'linkstate';
                        import postscribe from 'postscribe';
                        import ImageUploadIcon from 'images/image-upload.svg';
                        Severity: Minor
                        Found in app/javascript/article-form/articleForm.jsx - About 6 hrs to fix

                          File Onboarding.jsx has 436 lines of code (exceeds 250 allowed). Consider refactoring.
                          Open

                          import { h, Component } from 'preact';
                          import OnboardingWelcome from './components/OnboardingWelcome';
                          import OnboardingFollowTags from './components/OnboardingFollowTags';
                          import OnboardingFollowUsers from './components/OnboardingFollowUsers';
                          import OnboardingWelcomeThread from './components/OnboardingWelcomeThread';
                          Severity: Minor
                          Found in app/javascript/src/Onboarding.jsx - About 6 hrs to fix

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

                                    <button
                                      onClick={() => {
                                        const modal = document.getElementById('userdetails__blockmsg');
                                        const otherModal = document.getElementById(
                                          'userdetails__reportabuse',
                            Severity: Major
                            Found in app/javascript/chat/userDetails.jsx and 1 other location - About 6 hrs to fix
                            app/javascript/chat/userDetails.jsx on lines 118..135

                            Duplicated Code

                            Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                            Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                            When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                            Tuning

                            This issue has a mass of 162.

                            We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                            The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                            If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                            See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                            Refactorings

                            Further Reading

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

                                      <button
                                        onClick={() => {
                                          const modal = document.getElementById('userdetails__reportabuse');
                                          const otherModal = document.getElementById(
                                            'userdetails__blockmsg',
                            Severity: Major
                            Found in app/javascript/chat/userDetails.jsx and 1 other location - About 6 hrs to fix
                            app/javascript/chat/userDetails.jsx on lines 69..86

                            Duplicated Code

                            Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                            Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                            When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                            Tuning

                            This issue has a mass of 162.

                            We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                            The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                            If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                            See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                            Refactorings

                            Further Reading

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

                              render() {
                                const { previousLocation, prev, next, variant } = this.props;
                            
                                const previousLocationListElement = () => {
                                  if (variant === '6' || variant === '8') {
                            Severity: Major
                            Found in app/javascript/onboarding/components/ClosingSlide.jsx - About 6 hrs to fix

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

                                render() {
                                  const {
                                    listings,
                                    orgListings,
                                    userCredits,
                              Severity: Major
                              Found in app/javascript/listings/listingDashboard.jsx - About 5 hrs to fix

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

                                            <a
                                              tabIndex="0"
                                              className="no"
                                              onClick={() => {
                                                document.getElementById(
                                Severity: Major
                                Found in app/javascript/chat/userDetails.jsx and 1 other location - About 5 hrs to fix
                                app/javascript/chat/userDetails.jsx on lines 221..239

                                Duplicated Code

                                Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                                Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                                When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                                Tuning

                                This issue has a mass of 148.

                                We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                                The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                                If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                                See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                                Refactorings

                                Further Reading

                                Severity
                                Category
                                Status
                                Source
                                Language