balderdashy/waterline

Function update has 174 lines of code (exceeds 25 allowed). Consider refactoring.

Open

module.exports = function update(criteria, valuesToSet, explicitCbMaybe, metaContainer) {

  // Verify `this` refers to an actual Sails/Waterline model.
  verifyModelMethodContext(this);

Function _afterTalkingToAdapter has 69 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        adapter.update(WLModel.datastore, query, function _afterTalkingToAdapter(err, rawAdapterResult) {
          if (err) {
            err = forgeAdapterError(err, omen, 'update', modelIdentity, orm);
            return done(err);
          }//-•

Avoid too many return statements within this function.

Open

          } catch (e) { return done(e); }

Avoid too many return statements within this function.

Open

          } catch (e) { return done(e); }

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

Open

            async.each(transformedRecords, function _eachRecord(record, next) {

              // If the `skipAllLifecycleCallbacks` meta flag was set, don't run any of
              // the methods.
              if (_.has(query.meta, 'skipAllLifecycleCallbacks') && query.meta.skipAllLifecycleCallbacks) {

• Read up
  Read up

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

Open

          if (!fetch) {

            // > Note: This `if` statement is a convenience, for cases where the result from
            // > the adapter may have been coerced from `undefined` to `null` automatically.
            // > (we want it to be `undefined` still, for consistency)

• Read up
  Read up

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

Open

          if (!_.isArray(rawAdapterResult)) {
            return done(new Error(
              'Unexpected behavior in database adapter: Since `fetch: true` was enabled, this adapter '+
              '(for datastore `'+WLModel.datastore+'`) should have sent back an array of records as the '+
              '2nd argument when triggering the callback from its `update` method.  But instead, got: '+

• Read up
  Read up

Identical blocks of code found in 3 locations. Consider refactoring.

Open

        try {
          query = forgeStageThreeQuery({
            stageTwoQuery: query,
            identity: modelIdentity,
            transformer: WLModel._transformer,

• Read up
  Read up

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

Open

  var query = {
    method: 'update',
    using: modelIdentity,
    criteria: criteria,
    valuesToSet: valuesToSet,

• Read up
  Read up

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

Open

        if (_.keys(collectionResets).length > 0) {
          // Build a modified shallow clone of the originally-provided `meta`
          // that also has `fetch: true`.
          modifiedMetaForCollectionResets = _.extend({}, query.meta || {}, { fetch: true });
        }//>-

• Read up
  Read up

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

Open

          case 'E_NOOP':
            // Determine the appropriate no-op result.
            // If `fetch` meta key is set, use `[]`-- otherwise use `undefined`.
            //
            // > Note that future versions might simulate output from the raw driver.

• Read up
  Read up

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

Open

          async.each(_.keys(collectionResets), function _eachReplaceCollectionOp(collectionAttrName, next) {

            WLModel.replaceCollection(targetIds, collectionAttrName, collectionResets[collectionAttrName], function(err){
              if (err) { return next(err); }
              return next();

• Read up
  Read up

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

Open

        WLModel._callbacks.beforeUpdate(query.valuesToSet, function(err){
          if (err) { return proceed(err); }
          return proceed(undefined, query);
        });

• Read up
  Read up

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

Open

        if (!adapter.update) {
          return done(new Error('The adapter used by this model (`' + modelIdentity + '`) doesn\'t support the `'+query.method+'` method.'));
        }

• Read up
  Read up