ConnorWiseman/jcink-custom-structure

View on GitHub

Showing 26 of 41 total issues

Avoid deeply nested control flow statements.
Open

                switch (titleContents) {
                    case 'Announcements':
                        rowClass = ' announcement-topic';
                        topicsContent += '<div class="topic-title-row">' + this.config.announcementsDefault + '</div>';
                        break;
Severity: Major
Found in src/cs.js - About 45 mins to fix

    Avoid deeply nested control flow statements.
    Open

                        if (postLinks[l].href.indexOf('act=Post&CODE=08') !== -1) {
    
                            // ... attach an event listener.
                            postLinks[l].addEventListener('click', function(event) {
                                event.preventDefault();
    Severity: Major
    Found in src/cs.js - About 45 mins to fix

      Avoid deeply nested control flow statements.
      Open

                          if (child.nodeType === 1 && child.tagName === 'TD') {
                              if (child.parentNode.parentNode.parentNode === table) {
                                  cells.push(child);
                              }
                          }
      Severity: Major
      Found in src/cs.js - About 45 mins to fix

        Avoid deeply nested control flow statements.
        Open

                            if (!topicTitle.getAttribute('title')) {
                                topicTitle = cells[2].getElementsByTagName('a')[1];
                            }
        Severity: Major
        Found in src/cs.js - About 45 mins to fix

          Avoid deeply nested control flow statements.
          Open

                              if (topicSpans[0].textContent.indexOf('(Pages ') !== -1) {
                                  this.setValue('pagination', topicSpans[0].innerHTML);
                                  this.setValue('topicDescription', topicSpans[1].textContent);
                              } else {
                                  this.setValue('pagination', this.config.paginationDefault);
          Severity: Major
          Found in src/cs.js - About 45 mins to fix

            Function createEditForm has 6 arguments (exceeds 4 allowed). Consider refactoring.
            Open

            $cs.module.Posts.prototype.createEditForm = function(forumId, topicId, postId, pageId, response, contentContainer) {
            Severity: Minor
            Found in src/cs.js - About 45 mins to fix
              Severity
              Category
              Status
              Source
              Language