gitlabhq/gitlabhq

View on GitHub

Showing 1,254 of 1,755 total issues

Class Project has 259 methods (exceeds 20 allowed). Consider refactoring.
Open

class Project < ActiveRecord::Base
  include Gitlab::ConfigHelper
  include Gitlab::ShellAdapter
  include Gitlab::VisibilityLevel
  include AccessRequestable
Severity: Major
Found in app/models/project.rb - About 5 days to fix

    File project.rb has 1668 lines of code (exceeds 250 allowed). Consider refactoring.
    Open

    require 'carrierwave/orm/activerecord'
    
    class Project < ActiveRecord::Base
      include Gitlab::ConfigHelper
      include Gitlab::ShellAdapter
    Severity: Major
    Found in app/models/project.rb - About 4 days to fix

      File notes.js has 1345 lines of code (exceeds 250 allowed). Consider refactoring.
      Open

      /* eslint-disable no-restricted-properties, func-names, no-var, wrap-iife, camelcase,
      no-unused-expressions, max-len, one-var, one-var-declaration-per-line, default-case,
      prefer-template, consistent-return, no-alert, no-return-assign,
      no-param-reassign, prefer-arrow-callback, no-else-return, vars-on-top,
      no-unused-vars, no-shadow, no-useless-escape, class-methods-use-this */
      Severity: Major
      Found in app/assets/javascripts/notes.js - About 3 days to fix

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

        class User < ActiveRecord::Base
          extend Gitlab::ConfigHelper
        
          include Gitlab::ConfigHelper
          include Gitlab::SQL::Pattern
        Severity: Major
        Found in app/models/user.rb - About 3 days to fix

          File entities.rb has 1138 lines of code (exceeds 250 allowed). Consider refactoring.
          Open

          module API
            module Entities
              class WikiPageBasic < Grape::Entity
                expose :format
                expose :slug
          Severity: Major
          Found in lib/api/entities.rb - About 2 days to fix

            Class MergeRequest has 137 methods (exceeds 20 allowed). Consider refactoring.
            Open

            class MergeRequest < ActiveRecord::Base
              include AtomicInternalId
              include IidRoutes
              include Issuable
              include Noteable
            Severity: Major
            Found in app/models/merge_request.rb - About 2 days to fix

              Class Repository has 136 methods (exceeds 20 allowed). Consider refactoring.
              Open

                  class Repository
                    include Gitlab::Git::RepositoryMirroring
                    include Gitlab::Git::Popen
                    include Gitlab::EncodingHelper
                    include Gitlab::Utils::StrongMemoize
              Severity: Major
              Found in lib/gitlab/git/repository.rb - About 2 days to fix

                Function UsersSelect has 524 lines of code (exceeds 25 allowed). Consider refactoring.
                Open

                function UsersSelect(currentUser, els, options = {}) {
                  var $els;
                  this.users = this.users.bind(this);
                  this.user = this.user.bind(this);
                  this.usersPath = "/autocomplete/users.json";
                Severity: Major
                Found in app/assets/javascripts/users_select.js - About 2 days to fix

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

                  require 'carrierwave/orm/activerecord'
                  
                  class User < ActiveRecord::Base
                    extend Gitlab::ConfigHelper
                  
                  
                  Severity: Major
                  Found in app/models/user.rb - About 2 days to fix

                    Class Repository has 129 methods (exceeds 20 allowed). Consider refactoring.
                    Open

                    class Repository
                      REF_MERGE_REQUEST = 'merge-requests'.freeze
                      REF_KEEP_AROUND = 'keep-around'.freeze
                      REF_ENVIRONMENTS = 'environments'.freeze
                      MAX_DIVERGING_COUNT = 1000
                    Severity: Major
                    Found in app/models/repository.rb - About 2 days to fix

                      File gl_dropdown.js has 935 lines of code (exceeds 250 allowed). Consider refactoring.
                      Open

                      /* eslint-disable func-names, no-underscore-dangle, no-var, one-var, one-var-declaration-per-line, max-len, vars-on-top, wrap-iife, no-unused-vars, no-shadow, no-cond-assign, prefer-arrow-callback, no-return-assign, no-else-return, camelcase, no-lonely-if, guard-for-in, no-restricted-syntax, consistent-return, prefer-template, no-param-reassign, no-loop-func */
                      /* global fuzzaldrinPlus */
                      
                      import $ from 'jquery';
                      import _ from 'underscore';
                      Severity: Major
                      Found in app/assets/javascripts/gl_dropdown.js - About 2 days to fix

                        File merge_request.rb has 899 lines of code (exceeds 250 allowed). Consider refactoring.
                        Open

                        class MergeRequest < ActiveRecord::Base
                          include AtomicInternalId
                          include IidRoutes
                          include Issuable
                          include Noteable
                        Severity: Major
                        Found in app/models/merge_request.rb - About 2 days to fix

                          File repository.rb has 866 lines of code (exceeds 250 allowed). Consider refactoring.
                          Open

                          require 'tempfile'
                          require 'forwardable'
                          require "rubygems/package"
                          
                          module Gitlab
                          Severity: Major
                          Found in lib/gitlab/git/repository.rb - About 2 days to fix

                            File services.rb has 836 lines of code (exceeds 250 allowed). Consider refactoring.
                            Open

                            module API
                              class Services < Grape::API
                                CHAT_NOTIFICATION_SETTINGS = [
                                  {
                                    required: true,
                            Severity: Major
                            Found in lib/api/services.rb - About 2 days to fix

                              Class Build has 98 methods (exceeds 20 allowed). Consider refactoring.
                              Open

                                class Build < CommitStatus
                                  prepend ArtifactMigratable
                                  include TokenAuthenticatable
                                  include AfterCommitQueue
                                  include ObjectStorage::BackgroundMove
                              Severity: Major
                              Found in app/models/ci/build.rb - About 1 day to fix

                                Function constructor has 369 lines of code (exceeds 25 allowed). Consider refactoring.
                                Open

                                  constructor(els, options = {}) {
                                    var _this, $els;
                                    _this = this;
                                
                                    $els = $(els);
                                Severity: Major
                                Found in app/assets/javascripts/labels_select.js - About 1 day to fix

                                  File repository.rb has 758 lines of code (exceeds 250 allowed). Consider refactoring.
                                  Open

                                  require 'securerandom'
                                  
                                  class Repository
                                    REF_MERGE_REQUEST = 'merge-requests'.freeze
                                    REF_KEEP_AROUND = 'keep-around'.freeze
                                  Severity: Major
                                  Found in app/models/repository.rb - About 1 day to fix

                                    File users.rb has 651 lines of code (exceeds 250 allowed). Consider refactoring.
                                    Open

                                    module API
                                      class Users < Grape::API
                                        include PaginationParams
                                        include APIGuard
                                        include Helpers::CustomAttributes
                                    Severity: Major
                                    Found in lib/api/users.rb - About 1 day to fix

                                      Notes has 74 functions (exceeds 20 allowed). Consider refactoring.
                                      Open

                                      export default class Notes {
                                        static initialize(notes_url, note_ids, last_fetched_at, view, enableGFM) {
                                          if (!this.instance) {
                                            this.instance = new Notes(notes_url, note_ids, last_fetched_at, view, enableGFM);
                                          }
                                      Severity: Major
                                      Found in app/assets/javascripts/notes.js - About 1 day to fix

                                        File users_select.js has 594 lines of code (exceeds 250 allowed). Consider refactoring.
                                        Open

                                        /* eslint-disable func-names, one-var, no-var, prefer-rest-params, quotes, max-len, one-var-declaration-per-line, vars-on-top, prefer-arrow-callback, consistent-return, comma-dangle, object-shorthand, no-shadow, no-unused-vars, no-else-return, no-self-compare, prefer-template, no-unused-expressions, yoda, prefer-spread, no-void, camelcase, no-param-reassign */
                                        /* global Issuable */
                                        /* global emitSidebarEvent */
                                        
                                        import $ from 'jquery';
                                        Severity: Major
                                        Found in app/assets/javascripts/users_select.js - About 1 day to fix
                                          Severity
                                          Category
                                          Status
                                          Source
                                          Language