microting/eform-sdk-dotnet

View on GitHub

Showing 3,203 of 3,219 total issues

Method UploadedData_Create_DoesCreate has 51 lines of code (exceeds 25 allowed). Consider refactoring.
Open

        [Test]
        public async Task UploadedData_Create_DoesCreate()
        {
            //Arrange
            Random rnd = new Random();
Severity: Major
Found in eFormSDK.CheckLists.Tests/UploadedDataUTest.cs - About 2 hrs to fix

    Method Core_Case_CaseCreate_CreatesCase has 51 lines of code (exceeds 25 allowed). Consider refactoring.
    Open

            [Test]//needs http mock done
            public async Task Core_Case_CaseCreate_CreatesCase()
            {
                // Arrange
                #region Template1
    Severity: Major
    Found in eFormSDK.Integration.Tests/CoreTestCaseCreate.cs - About 2 hrs to fix

      Method Core_Folders_CreateFolder_DoesCreateNewFolderWithTranslations has 51 lines of code (exceeds 25 allowed). Consider refactoring.
      Open

              [Test]
              public async Task Core_Folders_CreateFolder_DoesCreateNewFolderWithTranslations()
              {
                  // Arrange
      
      Severity: Major
      Found in eFormSDK.Integration.Base.CoreTests/CoreTestFolders.cs - About 2 hrs to fix

        Method EntityItems_Create_DoesCreate has 51 lines of code (exceeds 25 allowed). Consider refactoring.
        Open

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

          Method question_Create_DoesCreate_QuestionSetFalse_BackButtonEnabledImageTrue has 51 lines of code (exceeds 25 allowed). Consider refactoring.
          Open

                  [Test]
                  public async Task question_Create_DoesCreate_QuestionSetFalse_BackButtonEnabledImageTrue()
                  {
                      // Arrange
                      string name = Guid.NewGuid().ToString();
          Severity: Major
          Found in eFormSDK.Integration.Tests/SqlControllerTestQuestion.cs - About 2 hrs to fix

            Method SQL_answers_Delete_DoesDelete_UTCAdjustedTrue has 51 lines of code (exceeds 25 allowed). Consider refactoring.
            Open

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

              Method SQL_answers_Delete_DoesDelete_UTCAdjustedFalse has 51 lines of code (exceeds 25 allowed). Consider refactoring.
              Open

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

                Method CheckRead has 51 lines of code (exceeds 25 allowed). Consider refactoring.
                Open

                        public async Task<ReplyElement> CheckRead(int microtingUId, int checkUId, Language language)
                        {
                            string methodName = "SqlController.CheckRead";
                            try
                            {
                Severity: Major
                Found in eFormCore/Infrastructure/SqlController.cs - About 2 hrs to fix

                  Method ReadSorted has 51 lines of code (exceeds 25 allowed). Consider refactoring.
                  Open

                          public static async Task<Models.EntityGroup> ReadSorted(MicrotingDbContext dbContext, string entityGroupMUId,
                              string sort,
                              string nameFilter)
                          {
                              EntityGroup eG =
                  Severity: Major
                  Found in eFormCore/Infrastructure/Data/Entities/EntityGroup.cs - About 2 hrs to fix

                    Method FolderUpdate has 51 lines of code (exceeds 25 allowed). Consider refactoring.
                    Open

                            public async Task FolderUpdate(int id, List<KeyValuePair<string, string>> name,
                                List<KeyValuePair<string, string>> description, int? parentId)
                            {
                                string methodName = "Core.FolderUpdate";
                                try
                    Severity: Major
                    Found in eFormCore/Core.cs - About 2 hrs to fix

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

                              public async Task<bool> TemplateDelete(int templateId)
                              {
                                  string methodName = "Core.TemplateDelete";
                                  try
                                  {
                      Severity: Major
                      Found in eFormCore/Core.cs and 1 other location - About 2 hrs to fix
                      eFormCore/Core.cs on lines 4676..4692

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

                      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

                              public async Task<string> Advanced_WorkerNameRead(int workerId)
                              {
                                  string methodName = "Core.Advanced_WorkerNameRead";
                                  try
                                  {
                      Severity: Major
                      Found in eFormCore/Core.cs and 1 other location - About 2 hrs to fix
                      eFormCore/Core.cs on lines 1147..1163

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

                      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

                                      if (dataElement.Label.Split("|").Length > 1)
                                      {
                                          checkListTranslation = new CheckListTranslation
                                          {
                                              CheckListId = cl.Id,
                      Severity: Major
                      Found in eFormCore/Infrastructure/SqlController.cs and 1 other location - About 2 hrs to fix
                      eFormCore/Infrastructure/SqlController.cs on lines 6019..6033

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

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

                              public async Task<SiteNameDto> Advanced_SiteItemRead(int microting_uuid)
                              {
                                  string methodName = "Core.Advanced_SiteItemRead";
                                  try
                                  {
                      Severity: Major
                      Found in eFormCore/Core.cs and 2 other locations - About 2 hrs to fix
                      eFormCore/Core.cs on lines 2893..2909
                      eFormCore/Core.cs on lines 4694..4710

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

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

                              public async Task<SiteDto> SiteRead(int microtingUid)
                              {
                                  string methodName = "Core.SiteRead";
                                  try
                                  {
                      Severity: Major
                      Found in eFormCore/Core.cs and 2 other locations - About 2 hrs to fix
                      eFormCore/Core.cs on lines 4481..4497
                      eFormCore/Core.cs on lines 4694..4710

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

                      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

                                      if (dataElement.Label.Split("|").Length > 2)
                                      {
                                          checkListTranslation = new CheckListTranslation
                                          {
                                              CheckListId = cl.Id,
                      Severity: Major
                      Found in eFormCore/Infrastructure/SqlController.cs and 1 other location - About 2 hrs to fix
                      eFormCore/Infrastructure/SqlController.cs on lines 6003..6017

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

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

                              public async Task<WorkerDto> Advanced_WorkerRead(int workerId)
                              {
                                  string methodName = "Core.Advanced_WorkerRead";
                                  try
                                  {
                      Severity: Major
                      Found in eFormCore/Core.cs and 2 other locations - About 2 hrs to fix
                      eFormCore/Core.cs on lines 2893..2909
                      eFormCore/Core.cs on lines 4481..4497

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

                      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

                      Method Core_Tags_TemplateSetTags_DoesAssignTagToTemplateWithoutDuplicates has 50 lines of code (exceeds 25 allowed). Consider refactoring.
                      Open

                              [Test]
                              public async Task Core_Tags_TemplateSetTags_DoesAssignTagToTemplateWithoutDuplicates()
                              {
                                  // Arrance
                                  CheckList cl1 = new CheckList
                      Severity: Minor
                      Found in eFormSDK.Integration.Base.CoreTests/CoreTestTag.cs - About 2 hrs to fix

                        Method TemplateSetTags has 50 lines of code (exceeds 25 allowed). Consider refactoring.
                        Open

                                public async Task<bool> TemplateSetTags(int templateId, List<int> tagIds)
                                {
                                    string methodName = "SqlController.TemplateSetTags";
                                    try
                                    {
                        Severity: Minor
                        Found in eFormCore/Infrastructure/SqlController.cs - About 2 hrs to fix

                          Method Restart has 50 lines of code (exceeds 25 allowed). Consider refactoring.
                          Open

                                  public override async Task Restart(int sameExceptionCount, int sameExceptionCountMax)
                                  {
                                      string methodName = "Core.Restart";
                                      try
                                      {
                          Severity: Minor
                          Found in eFormCore/Core.cs - About 2 hrs to fix
                            Severity
                            Category
                            Status
                            Source
                            Language