Promact/slack-erp-custom-integration-mvc

View on GitHub

Showing 196 of 517 total issues

Avoid deeply nested control flow statements.
Open

                            if (!string.IsNullOrEmpty(response))
                                // If added time entry in redmine issue
                                replyText = string.Format(_stringConstant.TimeEnrtyAddSuccessfully, issueId);
                            else
                                // If error in adding time entry in redmine issue

    Avoid deeply nested control flow statements.
    Open

                                    case SlackAction.help:
                                        replyText = _stringConstant.RedmineHelp;
                                        break;

      Consider simplifying this complex logical expression.
      Open

                      if (await IsScrumStartLeaveLinkCommandAsync(scrumBotId, message, messageArray)
                         || String.Compare(message, _stringConstant.ScrumHalt, StringComparison.OrdinalIgnoreCase) == 0
                         || String.Compare(message, _stringConstant.ScrumResume, StringComparison.OrdinalIgnoreCase) == 0
                         || (scrum != null && scrum.IsOngoing && !scrum.IsHalted))
                      {

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

                private async Task<string> LeaveAsync(int projectId, string slackUserName, string slackUserId, string applicant, string applicantId)

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

                  private void AddAnswer(int scrumId, int questionId, string userId, string message, ScrumAnswerStatus scrumAnswerStatus)

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

                    private async Task<List<TaskMailReportAc>> TaskMailDetailsAsync(string role, string loginId, DateTime selectedDate, DateTime maxDate, DateTime minDate)

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

                      private async Task<string> AddScrumAnswerAsync(string slackUserName, string message, int? projectId, string slackUserId, bool isLinkCommand)

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

                        public async Task<string> PutAsync(string baseUrl, string contentString, string contentHeader, string accessToken, string accessTokenType)
                Severity: Minor
                Found in Slack.Automation/Promact.Erp.Util/HttpClient/HttpClientService.cs - About 35 mins to fix

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

                          Task<string> PostAsync(string baseUrl, string contentString, string contentHeader, string accessToken, string accessTokenType);
                  Severity: Minor
                  Found in Slack.Automation/Promact.Erp.Util/HttpClient/IHttpClientService.cs - About 35 mins to fix

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

                            public async Task<List<TaskMailReportAc>> TaskMailDetailsReportSelectedDateAsync(string userId, string role, string userName, string createdOn, string selectedDate)
                    Severity: Minor
                    Found in Slack.Automation/Promact.Erp.Core/Controllers/TaskReportController.cs - About 35 mins to fix

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

                              public async Task<string> PostAsync(string baseUrl, string contentString, string contentHeader, string accessToken, string accessTokenType)
                      Severity: Minor
                      Found in Slack.Automation/Promact.Erp.Util/HttpClient/HttpClientService.cs - About 35 mins to fix

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

                                Task<string> PutAsync(string baseUrl, string contentString, string contentHeader, string accessToken, string accessTokenType);
                        Severity: Minor
                        Found in Slack.Automation/Promact.Erp.Util/HttpClient/IHttpClientService.cs - About 35 mins to fix

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

                                  public async Task<List<TaskMailReportAc>> TaskMailDetailsReportNextPreviousDateAsync(string userId, string role, string userName, string createdOn, string pageType)
                          Severity: Minor
                          Found in Slack.Automation/Promact.Erp.Core/Controllers/TaskReportController.cs - About 35 mins to fix

                            Avoid too many return statements within this method.
                            Open

                                            return _stringConstant.ProjectChannelNotLinked;

                              Avoid too many return statements within this method.
                              Open

                                              return _stringConstant.OnlyPrivateChannel;

                                Avoid too many return statements within this method.
                                Open

                                            return false;

                                  Avoid too many return statements within this method.
                                  Open

                                                          return _stringConstant.YouAreNotInExistInOAuthServer;

                                    Avoid too many return statements within this method.
                                    Open

                                                    return ScrumStatus.InActiveProject;

                                      Avoid too many return statements within this method.
                                      Open

                                                          return _stringConstant.YouAreNotInExistInOAuthServer;

                                        Avoid too many return statements within this method.
                                        Open

                                                        return false;
                                          Severity
                                          Category
                                          Status
                                          Source
                                          Language