balderdashy/waterline

Function destroy has 233 lines of code (exceeds 25 allowed). Consider refactoring.

Open

module.exports = function destroy(/* criteria, explicitCbMaybe, metaContainer */) {

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

Function _afterRunningBeforeLC has 164 lines of code (exceeds 25 allowed). Consider refactoring.

Open

      })(function _afterRunningBeforeLC(err, query) {
        if (err) {
          return done(err);
        }

Function _afterPotentiallyLookingUpRecordsToCascade has 112 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        })(function _afterPotentiallyLookingUpRecordsToCascade(err, idsOfRecordsBeingDestroyedMaybe) {
          if (err) { return done(err); }


          // Now we'll actually perform the `destroy`.

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

Open

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

Function _afterPotentiallyWipingCollections has 65 lines of code (exceeds 25 allowed). Consider refactoring.

Open

            })(function _afterPotentiallyWipingCollections(err) {// ~∞%°
              if (err) {
                return done(err);
              }

Function _maybeWipeAssociatedCollections has 37 lines of code (exceeds 25 allowed). Consider refactoring.

Open

            (function _maybeWipeAssociatedCollections(proceed) {

              // If `cascade` meta key is NOT enabled, then just proceed.
              if (!query.meta || !query.meta.cascade) {
                return proceed();

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

Open

              (function _maybeTransformRecords(proceed){

                // If `fetch` was not enabled, return.
                if (!_.has(query.meta, 'fetch') || query.meta.fetch === false) {

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

Open

        (function _maybeFindIdsToDestroy(proceed) {

          // If `cascade` meta key is NOT enabled, then just proceed.
          if (!query.meta || !query.meta.cascade) {
            return proceed();

Avoid too many return statements within this function.

Open

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

Avoid too many return statements within this function.

Open

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

Avoid too many return statements within this function.

Open

                return proceed(undefined, transformedRecords);

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

Open

                async.each(transformedRecordsMaybe, 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 2 locations. Consider refactoring.

Open

  if (!_.isFunction(arguments[0])) {
    query.criteria = arguments[0];
    explicitCbMaybe = arguments[1];
    query.meta = arguments[2];
  }

• Read up
  Read up

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

Open

                if (!_.isArray(rawAdapterResult)) {
                  return proceed(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 `destroy` method.  But instead, got: '+util.inspect(rawAdapterResult, {depth:5})+''

• 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 3 locations. Consider refactoring.

Open

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

• Read up
  Read up