loomio/loomio

File thread_loader.js has 445 lines of code (exceeds 250 allowed). Consider refactoring.

Open

import Records from '@/shared/services/records';
import { some, last, cloneDeep, max, uniq, compact, orderBy } from 'lodash-es';
import Vue from 'vue';
import RangeSet         from '@/shared/services/range_set';
import EventBus         from '@/shared/services/event_bus';

ThreadLoader has 34 functions (exceeds 20 allowed). Consider refactoring.

Open

export default class ThreadLoader {
  constructor(discussion) {
    this.discussion = discussion;
    this.reset();
  }

Function updateCollection has 44 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  updateCollection() {
    this.records = [];
    this.rules.forEach(rule => {
      let chain = Records.events.collection.chain();
      chain.find(rule.local.find);

Function addLoadPositionKeyRule has 36 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  addLoadPositionKeyRule(positionKey) {
    this.loading = positionKey;
    this.addRule({
      name: "positionKey from url",
      local: {

Function addLoadUnreadRule has 28 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  addLoadUnreadRule() {
    if (this.discussion.updatedAt > this.discussion.lastReadAt) {
      this.addRule({
        name: "context updated",
        local: {

Function addMetaData has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  addMetaData(collection) {
    const positions = collection.map(e => e.event.position);
    const ranges = RangeSet.arrayToRanges(positions);
    const parentExists = collection[0] && collection[0].event && collection[0].event.parent();
    const lastPosition = (parentExists && (collection[0].event.parent().childCount)) || 0;

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

Open

      local: {
        find: {
          discussionId: this.discussion.id,
          positionKey: {$jlt: positionKey}
        },

• Read up
  Read up

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

Open

      local: {
        find: {
          discussionId: this.discussion.id,
          positionKey: {$jgte: positionKey}
        },

• Read up
  Read up

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

Open

      local: {
        find: {
          discussionId: this.discussion.id,
          positionKey: {
            $jgt: event.positionKey

• Read up
  Read up

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

Open

      local: {
        find: {
          discussionId: this.discussion.id,
          positionKey: {
            $jlt: event.positionKey

• Read up
  Read up

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

Open

      this.addRule({
        name: "context updated",
        local: {
          find: {
            id: this.discussion.createdEvent().id

• Read up
  Read up

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

Open

      remote: {
        discussion_id: this.discussion.id,
        parent_id: event.id,
        order_by: 'position_key',
        order_desc: 1,

• Read up
  Read up

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

Open

    return this.addRule({
      name: 'context',
      local: {
        find: {
          id: this.discussion.createdEvent().id

• Read up
  Read up

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

Open

      remote: {
        discussion_id: this.discussion.id,
        position_key_lt: event.positionKey,
        order_by: 'position_key',
        order_desc: 1,

• Read up
  Read up