nponiros/sync_server

View on GitHub
lib/sync/get_server_changes.js

Summary

Maintainability
A
1 hr
Test Coverage

Function getServerChanges has 29 lines of code (exceeds 25 allowed). Consider refactoring.
Open

function getServerChanges(db, syncedRevision, clientIdentity, partialsThreshold, currentDBRevision) {
  // Get all changes after syncedRevision that was not performed by the client we're talking to.
  return db.getChangesData(syncedRevision, clientIdentity)
      // Compact changes so that multiple changes on same object are merged into a single change.
      .then((changes) => reduceChanges(changes))
Severity: Minor
Found in lib/sync/get_server_changes.js - About 1 hr to fix

    Function getServerChanges has 5 arguments (exceeds 4 allowed). Consider refactoring.
    Open

    function getServerChanges(db, syncedRevision, clientIdentity, partialsThreshold, currentDBRevision) {
    Severity: Minor
    Found in lib/sync/get_server_changes.js - About 35 mins to fix

      There are no issues that match your filters.

      Category
      Status