HCK-13030: ignore generating FK/UK if no columns (#170)

* HCK-13030: ignore generating FK/UK if no columns

* line position

* fix explicitly in PK/UK

Author: AlikRakhmonov

forward_engineering/alterScript/alterScriptHelpers/entityHelpers/primaryKeyHelper.js

@@ -259,10 +259,13 @@ const getAddCompositePkScriptDtos = collection => {
 	return newPrimaryKeys
 		.map(newPk => {
 			const ddlConfig = getCreateCompositePKDDLProviderConfig(newPk, entityName, collection);
+			if (_.isEmpty(ddlConfig.columns)) {
+				return null;
+			}
 			const statementDto = alterKeyConstraint(fullTableName, isCollectionActivated, ddlConfig);
 			return new KeyScriptModificationDto(statementDto.statement, fullTableName, false, statementDto.isActivated);
 		})
-		.filter(scriptDto => Boolean(scriptDto.script));
+		.filter(scriptDto => Boolean(scriptDto?.script));
 };
 
 /**

forward_engineering/alterScript/alterScriptHelpers/entityHelpers/uniqueKeyHelper.js

@@ -277,10 +277,13 @@ const getAddCompositeUniqueKeyScriptDtos = (collection, dbVersion) => {
 				collection,
 				dbVersion,
 			);
+			if (_.isEmpty(ddlConfig.columns)) {
+				return null;
+			}
 			const statementDto = alterKeyConstraint(fullTableName, isCollectionActivated, ddlConfig);
 			return new KeyScriptModificationDto(statementDto.statement, fullTableName, false, statementDto.isActivated);
 		})
-		.filter(scriptDto => Boolean(scriptDto.script));
+		.filter(scriptDto => Boolean(scriptDto?.script));
 };
 
 /**

forward_engineering/ddlProvider/ddlHelpers/constraintsHelper.js

@@ -150,7 +150,7 @@ const alterKeyConstraint = (tableName, isParentActivated, keyData) => {
 	const constraintStatementDto = createKeyConstraint(templates, isParentActivated)(keyData);
 	return {
 		statement: assignTemplates(templates.addPkConstraint, {
-			constraintStatement: (constraintStatementDto.statement || '').trim(),
+			constraintStatement: constraintStatementDto.statement.trim(),
 			tableName,
 		}),
 		isActivated: constraintStatementDto.isActivated,