Sign upLogin

microting/eform-sdk-dotnet

View on GitHub
Star

Showing 3,203 of 3,219 total issues

Method Down has 184 lines of code (exceeds 25 allowed). Consider refactoring.
Open

        protected override void Down(MigrationBuilder migrationBuilder)
        {
            migrationBuilder.DropTable(
                name: "CheckLisTranslations");
Severity: Major
Found in eFormCore/Migrations/20201223104631_AddingTranslations.cs - About 7 hrs to fix

    File AnswerValuesUTest.cs has 470 lines of code (exceeds 250 allowed). Consider refactoring.
    Open

    using System;
using System.Collections.Generic;
using System.Linq;
using System.Threading.Tasks;
using Microsoft.EntityFrameworkCore;
    Severity: Minor
    Found in eFormSDK.InSight.Tests/AnswerValuesUTest.cs - About 7 hrs to fix

      Method SQL_Unit_UnitGetAll_ReturnsAllUnits has 175 lines of code (exceeds 25 allowed). Consider refactoring.
      Open

              [Test]
        public async Task SQL_Unit_UnitGetAll_ReturnsAllUnits()
        {

            // Arrance
      Severity: Major
      Found in eFormSDK.Integration.Tests/SqlControllerTestUnit.cs - About 7 hrs to fix

        Class Communicator has 50 methods (exceeds 20 allowed). Consider refactoring.
        Open

            public class Communicator
    {
        #region var

        //SqlController sqlController;
        Severity: Minor
        Found in eFormCore/Communication/Communicator.cs - About 7 hrs to fix

          Method MoveTranslations has 173 lines of code (exceeds 25 allowed). Consider refactoring.
          Open

                  public static async Task MoveTranslations(MicrotingDbContext dbContext)
        {
            try
            {
                List<Field> fields = await dbContext.Fields.ToListAsync();
          Severity: Major
          Found in eFormCore/Infrastructure/Data/Entities/Field.cs - About 6 hrs to fix

            Method Cases_Delete_DoesSetWorkflowStateToRemoved has 171 lines of code (exceeds 25 allowed). Consider refactoring.
            Open

                    [Test]
        public async Task Cases_Delete_DoesSetWorkflowStateToRemoved()
        {
            //Arrange
            Severity: Major
            Found in eFormSDK.CheckLists.Tests/CasesUTest.cs - About 6 hrs to fix

              Class IHttp has 49 methods (exceeds 20 allowed). Consider refactoring.
              Open

                  internal interface IHttp
    {
        Task<string> Post(string xmlData, string siteId, string contentType = "application/xml");

        Task<string> Status(string elementId, string siteId);
              Severity: Minor
              Found in eFormCore/Communication/IHttp.cs - About 6 hrs to fix

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

                            modelBuilder.Entity("Microting.eForm.Infrastructure.Data.Entities.WorkerVersion", b =>
                {
                    b.Property<int>("Id")
                        .ValueGeneratedOnAdd()
                        .HasColumnType("int");
                Severity: Major
                Found in eFormCore/Migrations/20230607084834_AddingOriginalFileLocationToUploadedData.Designer.cs and 1 other location - About 6 hrs to fix
                eFormCore/Migrations/20230506062507_AddingInitialsToWorkers.Designer.cs on lines 3875..3922

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

                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

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

                            modelBuilder.Entity("Microting.eForm.Infrastructure.Data.Entities.WorkerVersion", b =>
                {
                    b.Property<int>("Id")
                        .ValueGeneratedOnAdd()
                        .HasColumnType("int");
                Severity: Major
                Found in eFormCore/Migrations/20230506062507_AddingInitialsToWorkers.Designer.cs and 1 other location - About 6 hrs to fix
                eFormCore/Migrations/20230607084834_AddingOriginalFileLocationToUploadedData.Designer.cs on lines 3881..3928

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

                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

                using Microsoft.EntityFrameworkCore.Migrations;

#nullable disable

namespace Microting.eForm.Migrations
                Severity: Major
                Found in eFormCore/Migrations/20220207094729_AddingIsLockedToSiteUnitWorkers.cs and 1 other location - About 6 hrs to fix
                eFormCore/Migrations/20211014105943_CLAttributes.cs on lines 1..80

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

                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

                using Microsoft.EntityFrameworkCore.Migrations;

namespace Microting.eForm.Migrations
{
    public partial class CLAttributes : Migration
                Severity: Major
                Found in eFormCore/Migrations/20211014105943_CLAttributes.cs and 1 other location - About 6 hrs to fix
                eFormCore/Migrations/20220207094729_AddingIsLockedToSiteUnitWorkers.cs on lines 1..82

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

                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

                File 20201223104631_AddingTranslations.cs has 445 lines of code (exceeds 250 allowed). Consider refactoring.
                Open

                using System;
using Microsoft.EntityFrameworkCore.Metadata;
using Microsoft.EntityFrameworkCore.Migrations;

namespace Microting.eForm.Migrations
                Severity: Minor
                Found in eFormCore/Migrations/20201223104631_AddingTranslations.cs - About 6 hrs to fix

                  Method SQL_Unit_UnitRead_ReadsUnit has 166 lines of code (exceeds 25 allowed). Consider refactoring.
                  Open

                          [Test]
        public async Task SQL_Unit_UnitRead_ReadsUnit()
        {

            // Arrance
                  Severity: Major
                  Found in eFormSDK.Integration.Tests/SqlControllerTestUnit.cs - About 6 hrs to fix

                    Method SQL_Template_TemplateFieldReadAll_ReturnsFieldList has 165 lines of code (exceeds 25 allowed). Consider refactoring.
                    Open

                            [Test]
        public async Task SQL_Template_TemplateFieldReadAll_ReturnsFieldList()
        {
            // Arrance
                    Severity: Major
                    Found in eFormSDK.Integration.Case.SqlControllerTests/SqlControllerTestCaseReadAllShort.cs - About 6 hrs to fix

                      Method Core_AdvancedTemplate_Advanced_TemplateFieldReadAll_returnslistofids has 165 lines of code (exceeds 25 allowed). Consider refactoring.
                      Open

                              [Test]
        public async Task Core_AdvancedTemplate_Advanced_TemplateFieldReadAll_returnslistofids()
        {
            #region Templates
                      Severity: Major
                      Found in eFormSDK.Integration.CheckLists.CoreTests/CoreTestAdvanced.cs - About 6 hrs to fix

                        Method SQL_Unit_UnitUpdate_UpdatesUnit has 165 lines of code (exceeds 25 allowed). Consider refactoring.
                        Open

                                [Test]
        public async Task SQL_Unit_UnitUpdate_UpdatesUnit()
        {

            // Arrance
                        Severity: Major
                        Found in eFormSDK.Integration.Tests/SqlControllerTestUnit.cs - About 6 hrs to fix

                          Method SQL_Unit_UnitDelete_DeletesUnit has 165 lines of code (exceeds 25 allowed). Consider refactoring.
                          Open

                                  [Test]
        public async Task SQL_Unit_UnitDelete_DeletesUnit()
        {  // Arrance
            #region Arrance
                          Severity: Major
                          Found in eFormSDK.Integration.Tests/SqlControllerTestUnit.cs - About 6 hrs to fix

                            Method SQL_answerValues_Update_DoesUpdate has 162 lines of code (exceeds 25 allowed). Consider refactoring.
                            Open

                                    [Test]
        public async Task SQL_answerValues_Update_DoesUpdate()
        {
            // Arrange
            Random rnd = new Random();
                            Severity: Major
                            Found in eFormSDK.Integration.Tests/SqlControllerTestAnswerValue.cs - About 6 hrs to fix

                              Method Core_eForm_MultiLvleFormCreateInDB_ReturnseFormId has 159 lines of code (exceeds 25 allowed). Consider refactoring.
                              Open

                                      [Test]
        public async Task Core_eForm_MultiLvleFormCreateInDB_ReturnseFormId()
        {
            // Arrange
            string xmlstring = @"
                              Severity: Major
                              Found in eFormSDK.Integration.CheckLists.CoreTests/CoreTesteFormCreateInDB.cs - About 6 hrs to fix

                                Method SQL_Worker_WorkerGetAll_ReturnsAllWorkers has 158 lines of code (exceeds 25 allowed). Consider refactoring.
                                Open

                                        [Test]
        public async Task SQL_Worker_WorkerGetAll_ReturnsAllWorkers()
        {
            // Arrance
            #region Arrance
                                Severity: Major
                                Found in eFormSDK.Integration.Tests/SqlControllerTestWorker.cs - About 6 hrs to fix
                                  Severity
                                  Category
                                  Status
                                  Source
                                  Language