chore: remove redundant code

Signed-off-by: Francisco Buceta <frbuceta@gmail.com>

Author: frbuceta

lib/discovery.js

@@ -9,8 +9,6 @@ const g = require('strong-globalize')();
 module.exports = mixinDiscovery;
 
 function mixinDiscovery(PostgreSQL) {
-  const async = require('async');
-
   PostgreSQL.prototype.paginateSQL = function(sql, orderBy, options) {
     options = options || {};
     let limit = '';

lib/migration.js

@@ -14,9 +14,7 @@ module.exports = mixinMigration;
 
 function mixinMigration(PostgreSQL) {
   PostgreSQL.prototype.checkFieldAndIndex = function(fields, indexes) {
-    if (fields && indexes)
-      return true;
-    return false;
+    return !!(fields && indexes);
   };
 
   PostgreSQL.prototype.showFields = function(model, cb) {
@@ -223,11 +221,10 @@ function mixinMigration(PostgreSQL) {
 
   PostgreSQL.prototype.modifyDatatypeInActual = function(model, propName) {
     const self = this;
-    const sqlCommand = self.columnEscaped(model, propName) + ' TYPE ' +
+    return self.columnEscaped(model, propName) + ' TYPE ' +
       self.columnDataType(model, propName) + ' USING ' +
       self.columnEscaped(model, propName) + '::' +
       self.columnDataType(model, propName);
-    return sqlCommand;
   };
 
   PostgreSQL.prototype.modifyNullabilityInActual = function(model, propName) {
@@ -519,8 +516,7 @@ function mixinMigration(PostgreSQL) {
   };
 
   PostgreSQL.prototype.buildColumnType = function buildColumnType(propertyDefinition) {
-    const p = propertyDefinition;
-    switch (p.type.name) {
+    switch (propertyDefinition.type.name) {
       default:
       case 'String':
       case 'JSON':