From 12e32f8f8b23a310824cfd4d03b7f44df0d2614f Mon Sep 17 00:00:00 2001 From: Chris Arnesen Date: Sat, 13 Oct 2018 16:03:09 -0500 Subject: [PATCH] refactor: Use @types/lodash.groupBy --- src/zone.ts | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/src/zone.ts b/src/zone.ts index d79dc3a..1b38a6c 100644 --- a/src/zone.ts +++ b/src/zone.ts @@ -23,7 +23,7 @@ import {promisifyAll} from '@google-cloud/promisify'; import * as extend from 'extend'; const flatten = require('lodash.flatten'); import * as fs from 'fs'; -const groupBy = require('lodash.groupby'); +import groupBy = require('lodash.groupby'); import * as is from 'is'; import {teenyRequest} from 'teeny-request'; const zonefile = require('dns-zonefile'); @@ -427,12 +427,12 @@ class Zone extends ServiceObject { if (!config || (!config.add && !config.delete)) { throw new Error('Cannot create a change with no additions or deletions.'); } - const groupByType = (changes: RecordObject[]) => { - changes = groupBy(changes, 'type'); - const changesArray: Array<{}> = []; + const groupByType = (recordsIn: RecordObject[]) => { + const recordsByType = groupBy(recordsIn, 'type'); + const recordsOut: RecordObject[] = []; // tslint:disable-next-line:forin - for (const recordType in changes) { - const recordsByName = groupBy(changes[recordType], 'name'); + for (const recordType in recordsByType) { + const recordsByName = groupBy(recordsByType[recordType], 'name'); // tslint:disable-next-line:forin for (const recordName in recordsByName) { const records = recordsByName[recordName]; @@ -442,10 +442,10 @@ class Zone extends ServiceObject { templateRecord.rrdatas = flatten(records.map((x: RecordObject) => x.rrdatas)); } - changesArray.push(templateRecord); + recordsOut.push(templateRecord); } } - return changesArray; + return recordsOut; }; const body = extend( {