Sign upLogin

Bnei-Baruch/mdb

View on GitHub
Star
importer/kmedia/kmodels/roles.go

Summary

Maintainability
F
5 days
Test Coverage

File roles.go has 673 lines of code (exceeds 500 allowed). Consider refactoring.
Open

// Code generated by SQLBoiler (https://github.com/volatiletech/sqlboiler). DO NOT EDIT.
// This file is meant to be re-generated in place and/or deleted at any time.

package kmodels
Severity: Minor
Found in importer/kmedia/kmodels/roles.go - About 6 hrs to fix

    Method Role.Upsert has 99 lines of code (exceeds 50 allowed). Consider refactoring.
    Open

    func (o *Role) Upsert(exec boil.Executor, updateOnConflict bool, conflictColumns []string, updateColumns []string, whitelist ...string) error {
    if o == nil {
        return errors.New("kmodels: no roles provided for upsert")
    }
    currTime := time.Now().In(boil.GetLocation())
    Severity: Major
    Found in importer/kmedia/kmodels/roles.go - About 2 hrs to fix

      Method Role.Insert has 67 lines of code (exceeds 50 allowed). Consider refactoring.
      Open

      func (o *Role) Insert(exec boil.Executor, whitelist ...string) error {
    if o == nil {
        return errors.New("kmodels: no roles provided for insertion")
    }
      Severity: Minor
      Found in importer/kmedia/kmodels/roles.go - About 1 hr to fix

        Method Role.Upsert has a Cognitive Complexity of 28 (exceeds 20 allowed). Consider refactoring.
        Open

        func (o *Role) Upsert(exec boil.Executor, updateOnConflict bool, conflictColumns []string, updateColumns []string, whitelist ...string) error {
    if o == nil {
        return errors.New("kmodels: no roles provided for upsert")
    }
    currTime := time.Now().In(boil.GetLocation())
        Severity: Minor
        Found in importer/kmedia/kmodels/roles.go - 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

        Method Role.Upsert has 6 return statements (exceeds 4 allowed).
        Open

        func (o *Role) Upsert(exec boil.Executor, updateOnConflict bool, conflictColumns []string, updateColumns []string, whitelist ...string) error {
    if o == nil {
        return errors.New("kmodels: no roles provided for upsert")
    }
    currTime := time.Now().In(boil.GetLocation())
        Severity: Major
        Found in importer/kmedia/kmodels/roles.go - About 40 mins to fix

          Method Role.UpsertP has 5 arguments (exceeds 4 allowed). Consider refactoring.
          Open

          func (o *Role) UpsertP(exec boil.Executor, updateOnConflict bool, conflictColumns []string, updateColumns []string, whitelist ...string) {
          Severity: Minor
          Found in importer/kmedia/kmodels/roles.go - About 35 mins to fix

            Method Role.Upsert has 5 arguments (exceeds 4 allowed). Consider refactoring.
            Open

            func (o *Role) Upsert(exec boil.Executor, updateOnConflict bool, conflictColumns []string, updateColumns []string, whitelist ...string) error {
            Severity: Minor
            Found in importer/kmedia/kmodels/roles.go - About 35 mins to fix

              Method Role.Insert has 5 return statements (exceeds 4 allowed).
              Open

              func (o *Role) Insert(exec boil.Executor, whitelist ...string) error {
    if o == nil {
        return errors.New("kmodels: no roles provided for insertion")
    }
              Severity: Major
              Found in importer/kmedia/kmodels/roles.go - About 35 mins to fix

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

                func (o *Role) Upsert(exec boil.Executor, updateOnConflict bool, conflictColumns []string, updateColumns []string, whitelist ...string) error {
    if o == nil {
        return errors.New("kmodels: no roles provided for upsert")
    }
    currTime := time.Now().In(boil.GetLocation())
                Severity: Major
                Found in importer/kmedia/kmodels/roles.go and 10 other locations - About 1 day to fix
                importer/kmedia/kmodels/catalog_descriptions.go on lines 950..1066
                importer/kmedia/kmodels/catalogs.go on lines 2213..2329
                importer/kmedia/kmodels/container_description_patterns.go on lines 907..1023
                importer/kmedia/kmodels/container_descriptions.go on lines 953..1069
                importer/kmedia/kmodels/containers.go on lines 2493..2609
                importer/kmedia/kmodels/file_asset_descriptions.go on lines 723..839
                importer/kmedia/kmodels/file_assets.go on lines 1791..1907
                importer/kmedia/kmodels/lecturer_descriptions.go on lines 559..675
                importer/kmedia/kmodels/lecturers.go on lines 556..672
                importer/kmedia/kmodels/users.go on lines 1394..1510

                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 918.

                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 11 locations. Consider refactoring.
                Open

                func (o *Role) Insert(exec boil.Executor, whitelist ...string) error {
    if o == nil {
        return errors.New("kmodels: no roles provided for insertion")
    }
                Severity: Major
                Found in importer/kmedia/kmodels/roles.go and 10 other locations - About 1 day to fix
                importer/kmedia/kmodels/catalog_descriptions.go on lines 674..757
                importer/kmedia/kmodels/catalogs.go on lines 1937..2020
                importer/kmedia/kmodels/container_description_patterns.go on lines 631..714
                importer/kmedia/kmodels/container_descriptions.go on lines 677..760
                importer/kmedia/kmodels/containers.go on lines 2217..2300
                importer/kmedia/kmodels/file_asset_descriptions.go on lines 447..530
                importer/kmedia/kmodels/file_assets.go on lines 1515..1598
                importer/kmedia/kmodels/lecturer_descriptions.go on lines 283..366
                importer/kmedia/kmodels/lecturers.go on lines 280..363
                importer/kmedia/kmodels/users.go on lines 1118..1201

                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 727.

                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 11 locations. Consider refactoring.
                Open

                func (o *Role) Update(exec boil.Executor, whitelist ...string) error {
    currTime := time.Now().In(boil.GetLocation())

    o.UpdatedAt.Time = currTime
    o.UpdatedAt.Valid = true
                Severity: Major
                Found in importer/kmedia/kmodels/roles.go and 10 other locations - About 6 hrs to fix
                importer/kmedia/kmodels/catalog_descriptions.go on lines 790..845
                importer/kmedia/kmodels/catalogs.go on lines 2053..2108
                importer/kmedia/kmodels/container_description_patterns.go on lines 747..802
                importer/kmedia/kmodels/container_descriptions.go on lines 793..848
                importer/kmedia/kmodels/containers.go on lines 2333..2388
                importer/kmedia/kmodels/file_asset_descriptions.go on lines 563..618
                importer/kmedia/kmodels/file_assets.go on lines 1631..1686
                importer/kmedia/kmodels/lecturer_descriptions.go on lines 399..454
                importer/kmedia/kmodels/lecturers.go on lines 396..451
                importer/kmedia/kmodels/users.go on lines 1234..1289

                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 479.

                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

                There are no issues that match your filters.

                Category
                Status